Skip to content

Remove the fixed orchestration fix-pass limit - #506

Merged
lolipopshock merged 1 commit into
mainfrom
fix/explicit-orchestration-fix-pass-count
Aug 22, 2026
Merged

Remove the fixed orchestration fix-pass limit#506
lolipopshock merged 1 commit into
mainfrom
fix/explicit-orchestration-fix-pass-count

Conversation

@lolipopshock

@lolipopshock lolipopshock commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the hard-coded one-automatic-fix-pass rule from the orchestrator prompt
  • let the orchestrator decide each FIX_PR handoff from the current review synthesis, scope disposition, and actionable context
  • retain the existing requirement that self-approval fixes be concrete, in-scope branch work

Context

PR #498 added a one-pass limit that was neither an existing product requirement nor backed by reliable chain state. On PR #501, the planner incorrectly inferred that round 3 meant a fix pass had already happened and stopped before the first valid fix.

The review/synthesis guidance remains responsible for classifying work as FIX_IN_PR, FOLLOW_UP, or HUMAN_DECISION; the orchestrator is trusted to decide whether another bounded fix is useful.

Verification

  • npm --prefix .agent run build
  • npm --prefix .agent run check:workflow
  • npm --prefix .agent run check:docs
  • git diff --check

@lolipopshock
lolipopshock force-pushed the fix/explicit-orchestration-fix-pass-count branch from d98e59a to 2e79bf6 Compare August 22, 2026 08:02
@lolipopshock lolipopshock changed the title Track automatic fix passes explicitly Stop inferring fix passes from orchestration rounds Aug 22, 2026
@lolipopshock
lolipopshock force-pushed the fix/explicit-orchestration-fix-pass-count branch from 2e79bf6 to 88a7d9a Compare August 22, 2026 08:08
@lolipopshock lolipopshock changed the title Stop inferring fix passes from orchestration rounds Remove the fixed orchestration fix-pass limit Aug 22, 2026
@lolipopshock
lolipopshock marked this pull request as ready for review August 22, 2026 08:12
@lolipopshock
lolipopshock merged commit 7b05cd2 into main Aug 22, 2026
2 checks passed
@lolipopshock
lolipopshock deleted the fix/explicit-orchestration-fix-pass-count branch August 22, 2026 08:12
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