Repository navigation
the-supervisor-changes-agents - #821
Merged
Merged
Conversation
…rvisor-changes-agents) Under supervisor.mode "act", set in a change's own harness.json with autonomyLevel "autonomous" and more than one attempt per stage, a stage whose failure is likely to pass is attempted again on the same agent, and one that repeating cannot fix is moved to the first fallback agent the change's policy allows (allowProviderChange, allowCostIncrease). Every repeat and move is said, recorded as a supervisor message entry, and costs an attempt; a chain says failed only for a stage it does not try again. Agents name their provider; under advise the last-run suggestion names the fallback act would use; the change's Harness Settings offers Act under Autonomous with the fallback per stage and the allowances; the CLI prints why a stage is attempted again. An ACP agent's own message now reaches the failure diagnosis (claude-cli-acp's "Not logged in" was diagnosed unknown). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…sts; checkboxes keep their size The fallback per stage was a free-text field with its own suggestion list, unlike every other agent choice in Harness Settings; it is now one agent list per fallback plus one to add the next, the editor's chat excepted. The band's rule that stretches its fields stretched the two allowances' checkboxes across the row and pushed their labels out of sight. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…y the owner) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
the-supervisor-changes-agents
Change 2 of ADR 0039: the supervisor may repeat a failed stage or move it to a fallback agent, where a change's own configuration allows it, and always says so.
act(supervisor.mode), only in a change's ownharness.jsontogether withautonomyLevel: "autonomous", and only with more than one attempt per stage.fallback.<stage>the policy allows and the stage has not run on; unknown → nothing.allowProviderChangefor another provider,allowCostIncreasefor any agent but the local model. Every registered agent now names its provider. A fallback passed over is named with its rule.failedis held until decided; a stage tried again yields nofailed. Each repeat/move is a progress line, an auditmessageentry fromsupervisorwithsupervisorDecision, and the next attempt'sstageStartedwith the new agent and reason. A moved stage runs its new agent with that agent's defaults.actwould use.openspec-ui-cli runprints why a stage is attempted again.claude-cli-acpreports "Not logged in" as an ACP message, so the diagnosis missed it; the runner now diagnoses from what an ACP agent says, too.Checks
openspec validate --strict; merge gate with the absolute worktree path: only 4.4 open (Human-only, for the owner).gemini-clinot on the PATH → moved tolocal-llm-acp(passing overcopilot-cli-acpfor cost), which ran and completed apply; record intasks.md.🤖 Generated with Claude Code