Skip to content

fix(repair): let apply-lane publishers out-wait state lease contention #730

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
steipete merged 2 commits into from
Jul 21, 2026
Merged

fix(repair): let apply-lane publishers out-wait state lease contention #730

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
daeb0cd
fix(repair): let apply-lane publishers out-wait state lease contention
steipete Jul 21, 2026
daa70ad
fix(workflow): dedupe apply-lane lease env blocks
steipete Jul 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/workflows/sweep.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3968,6 +3968,7 @@ jobs:
issues: read
pull-requests: read
env:
CLAWSWEEPER_STATE_LEASE_ACQUIRE_TIMEOUT_MS: "900000"
CLAWSWEEPER_UNCONFIRMED_PRODUCT_DIRECTION_CLOSE_ENABLED: ${{ vars.CLAWSWEEPER_UNCONFIRMED_PRODUCT_DIRECTION_CLOSE_ENABLED || 'false' }}
CLAWSWEEPER_UNSPONSORED_FEATURE_CLOSE_ENABLED: ${{ vars.CLAWSWEEPER_UNSPONSORED_FEATURE_CLOSE_ENABLED || 'false' }}
CLAWSWEEPER_IDEA_REVIVAL_REACTIONS: ${{ vars.CLAWSWEEPER_IDEA_REVIVAL_REACTIONS || '5' }}
Expand Down Expand Up @@ -4183,6 +4184,8 @@ jobs:

publish-apply-proof-action-ledger:
name: Publish immutable apply proof action ledger
env:
CLAWSWEEPER_STATE_LEASE_ACQUIRE_TIMEOUT_MS: "900000"
needs: apply-proof
if: ${{ always() && needs.apply-proof.result != 'skipped' }}
runs-on: ubuntu-latest
Expand Down Expand Up @@ -4263,6 +4266,7 @@ jobs:
cancel-in-progress: false
queue: max
env:
CLAWSWEEPER_STATE_LEASE_ACQUIRE_TIMEOUT_MS: "900000"
CLAWSWEEPER_UNCONFIRMED_PRODUCT_DIRECTION_CLOSE_ENABLED: ${{ vars.CLAWSWEEPER_UNCONFIRMED_PRODUCT_DIRECTION_CLOSE_ENABLED || 'false' }}
CLAWSWEEPER_UNSPONSORED_FEATURE_CLOSE_ENABLED: ${{ vars.CLAWSWEEPER_UNSPONSORED_FEATURE_CLOSE_ENABLED || 'false' }}
CLAWSWEEPER_IDEA_REVIVAL_REACTIONS: ${{ vars.CLAWSWEEPER_IDEA_REVIVAL_REACTIONS || '5' }}
Expand Down
12 changes: 11 additions & 1 deletion src/repair/git-publish.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,17 @@ const RECOVERY_FETCH_TIMEOUT_MS = 300_000;
const STATE_PUBLISH_LEASE_REF_ROOT = "refs/heads/clawsweeper-publish-lease";
const STATE_PUBLISH_LEASE_TTL_MS = 2 * 60_000;
const STATE_PUBLISH_LEASE_RENEW_THRESHOLD_MS = PUBLISH_FETCH_TIMEOUT_MS;
const STATE_PUBLISH_LEASE_ACQUIRE_TIMEOUT_MS = 3 * 60_000;
const STATE_PUBLISH_LEASE_ACQUIRE_TIMEOUT_MS = (() => {
const fallback = 3 * 60_000;
// Rare, high-value publishers (the apply lane) starve behind the review
// herd inside the default window; their workflows may raise the budget so
// one apply publish out-waits contention instead of failing the run.
const configured = Number(process.env.CLAWSWEEPER_STATE_LEASE_ACQUIRE_TIMEOUT_MS);
if (Number.isInteger(configured) && configured > fallback) {
return Math.min(configured, 30 * 60_000);
}
return fallback;
})();
const STATE_PUBLISH_LEASE_WAIT_MS = 1_000;
const STATE_PUBLISH_LEASE_MAX_WAIT_MS = 5_000;
const SKIP_CI_DIRECTIVE_PATTERN =
Expand Down
Loading