Skip to content

fix: renew auto-turn window on explicit resume - #56

Merged
danyel117 merged 1 commit into
mainfrom
fix/issue-55-resume-turn-limit
Sep 22, 2026
Merged

danyel117 merged 1 commit into
mainfrom
fix/issue-55-resume-turn-limit

Conversation

@danyel117

Copy link
Copy Markdown
Contributor

Closes #55

Summary

  • reset the auto-turn counter when a user explicitly resumes a goal after reaching max_auto_turns
  • keep token and elapsed-time accounting intact, and require a single-use V1/V2 command authorization so the model-callable status tool cannot renew its own safety limit
  • harden limit prompts and document the resume behavior

Tests

  • bun run lint
  • bun run typecheck
  • bun test (295 passed)
  • bun run build
  • bun run pack:dry-run

Claude Opus audits

  1. Found one medium issue: the generic status tool could renew the auto-turn ceiling without an explicit user request. Fixed by gating the reset behind a single-use command authorization and adding V1/V2/security regressions.
  2. Final audit: no high or medium findings. Six low-severity notes; the highest-priority V2 hook coverage note was addressed before opening this PR.

@danyel117
danyel117 merged commit d79a61a into main Sep 22, 2026
5 checks passed
@danyel117
danyel117 deleted the fix/issue-55-resume-turn-limit branch September 22, 2026 06:44
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.

[Bug]: /goal resume, after hitting 25-turn limit, only resumes for one turn

1 participant