Repository navigation
build(itd-2609201925079472): the whole spec, step 1 of spc-2609301921521360 - #886
Merged
Merged
Conversation
A claude run whose event stream carries a rate-limit response (a rejected rate_limit_event, or an assistant message whose error is rate_limit) and that did not finish now fails as rate-limited, whether the harness exits non-zero or reports the error in its result. The dispatcher never falls back on it: every route spends the budget the run's window paces, so the failure reaches the caller as itself, with no fallback receipt. A runner may report its remaining quota, in agent runs, through the optional QuotaReporter; Config.Quota asks the one a role is routed to. Neither shipped runner reports one, so each reports none. Assisted-by: Claude:claude-opus-5-5
… rate limit The budget check (itd-2609201925079472 criterion 7): once every other pre-start check passes, each runner a role of the run is routed to is asked for its remaining quota and compared with the run's estimate on it, from the spec's size: per step, one implementer and one round of the two reviewers, and the fidelity audit once for an intent. An estimate over a quota refuses the start at the check `budget`, naming both numbers, and writes nothing; a runner that reports none, the host included, is named and the check skipped out loud in the start's checks, its text output and the run record. build, build next and drain hand the runner configuration they read to the check. The rate-limit checkpoint (criterion 8): a runner's rate-limit response ends the run's window early for every lane, writing next_eligible_at once from the run's pause. The lane it came from is checkpointed to its branch: its agent's uncommitted work and partial receipt are saved aside for review, never built on, its worktree is reset to its last commit, and its await is dropped so the first step after the pause hands the work to a fresh agent; a limited validator has only its partial return saved aside. Every other lane with work in flight is checkpointed at its branch's head and left running. The record names the response, the runner and the lane. The restart's save-aside is now one function both share. Assisted-by: Claude:claude-opus-5-5
quotaOf is reached only through Config.Quota, so it is not exported; the exported-reach audit refused it as scaffolding. Assisted-by: Claude:claude-opus-5-5
The implement loop's landing for itd-2609201925079472 (spc-2609301921521360, step 1): closed spc-2609301921521360 (itd-2609201925079472 planned -> shipped), audit rcp-c411be255805 ingested. Delivers: itd-2609201925079472 Assisted-by: Claude:claude-opus-5-5
REPPL
added a commit
that referenced
this pull request
Oct 10, 2026
The reviewers of PR #886 passed it with notes below their bar. Five are captured: three small defects in the rate-limit and budget code, the budget check that no shipped runner can reach, and the runners that cannot recognise a rate limit. The three defects are deferred in DECISIONS.md with the reason: none is reachable without a second fault, and the drain's next steps rework the same code. Refs: itd-2609201925079472 Assisted-by: Claude:claude-opus-5-5
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request lands step 1 of spc-2609301921521360 ("the whole spec") for itd-2609201925079472. The implement loop built it in run run-2610100550297632 as lane-1, and wrote this text from the run's records.
build/run-2610100550297632-lane-1, from 3f21f58 to 0851a50.Delivers: itd-2609201925079472
Assisted-by: abcd:dev
Assisted-by: Claude:claude-opus-5-5