Skip to content

Fix auto-stalled steering and long Goal plans - #187

Merged
ByBrawe merged 14 commits into
mainfrom
fix/paused-steering-long-goals-1.3.28
Aug 24, 2026
Merged

Fix auto-stalled steering and long Goal plans#187
ByBrawe merged 14 commits into
mainfrom
fix/paused-steering-long-goals-1.3.28

Conversation

@ByBrawe

@ByBrawe ByBrawe commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes the long-running Goal UX failure where a Goal auto-paused after the no-progress guard and then ignored a concrete foreground work instruction unless the user first sent /goal resume or a narrow continuation phrase. It also removes the implicit 30-turn ceiling for newly created Goals so genuinely long plans are not cut off solely because they need more assistant turns.

Changes

  • Treat only the host-generated no-progress pause as recoverable from a concrete foreground work instruction.
  • Resume through the guarded Goal command chain, consume internal command ownership, and preserve the original human instruction as normal Goal steering.
  • Keep explicit /goal pause authoritative; unrelated work instructions do not silently resume it.
  • Scale the no-progress window from 4 to 12 turns when a current native Todo plan still has open work, instead of hard-pausing every long plan after 3 read/recon/verification-heavy turns.
  • Keep prior Todo telemetry visibly stale across /goal edit and prevent an unchanged native Todo digest from being silently rebound to the new Goal revision; a genuinely changed/rebuilt plan can bind current.
  • Make cumulative turn caps opt-in for newly created Goals (maxTurns: 0 by default). Explicit --max-turns / /goal budget --max-turns limits and persisted legacy limits remain hard guards.
  • Add regressions for auto-paused steering, explicit pause safety, 100-item Todo plans, Todo revision rebinding, unlimited default turn budget, and explicit turn caps.

Why

Observed session snapshots showed Goals with multiple pending Todos (and even effectively finished work) landing in paused after three turns without a new unique host mutation fingerprint. Current lifecycle UX then treated arbitrary foreground work instructions as normal chat while paused. Separately, a 100-step plan could still hit the default 30-turn cumulative budget even after the stall behavior was fixed. This patch makes system auto-pause recoverable and new long-running Goals turn-unbounded by default without turning every user message into a Goal edit or weakening explicit user pause/budget semantics.

Copilot AI lite review requested due to automatic review settings August 23, 2026 20:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ByBrawe
ByBrawe merged commit 09b25a3 into main Aug 24, 2026
23 checks passed
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.

2 participants