Skip to content

fix(publish): extend state lease wait to eight minutes#731

Merged
brokemac79 merged 1 commit into
mainfrom
codex/csw-049-temp-publish-timeout-480
Jul 21, 2026
Merged

fix(publish): extend state lease wait to eight minutes#731
brokemac79 merged 1 commit into
mainfrom
codex/csw-049-temp-publish-timeout-480

Conversation

@brokemac79

@brokemac79 brokemac79 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Problem

PR #730 repairs the separate apply-lane starvation by setting a 900-second override on three rare apply jobs. It intentionally leaves normal review/event result publishers on the 180-second fallback. Post-#729 production evidence showed those normal publishers still dominated by 180-second StatePublishContentionError failures.

Implementation

After rebasing onto #730's merge commit, this changes only its fallback from 3 * 60_000 to 8 * 60_000. The CLAWSWEEPER_STATE_LEASE_ACQUIRE_TIMEOUT_MS override, three 900000 apply-lane settings, 30-minute cap, publication capacity, lease TTL, polling, queue state, retries, gates, and dispatch behavior remain unchanged.

Validation

No additional local test, Codex review, or ClawSweeper review was run at the maintainer's explicit request.

Risk and rollout

Normal contending publishers may remain leased for up to eight minutes before returning to the durable queue. Apply-lane publishers retain #730's 15-minute override. Queue depth, useful GitHub delivery, retry amplification, and state-contention failures remain under active monitoring.

Related: #729, #730, and CSW-049.

@brokemac79
brokemac79 marked this pull request as draft July 21, 2026 00:55
@brokemac79
brokemac79 force-pushed the codex/csw-049-temp-publish-timeout-480 branch from 06822e5 to 2799353 Compare July 21, 2026 01:06
@brokemac79
brokemac79 marked this pull request as ready for review July 21, 2026 01:07
@brokemac79
brokemac79 merged commit b06b856 into main Jul 21, 2026
11 checks passed
@brokemac79
brokemac79 deleted the codex/csw-049-temp-publish-timeout-480 branch July 21, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant