Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

CLI: improve stream recovery and add configurable consecutive mistake limit - #11775

Merged
cte merged 2 commits into
mainfrom
fix/cli-stream-mistake-limit-recovery
Feb 27, 2026
Merged

CLI: improve stream recovery and add configurable consecutive mistake limit#11775
cte merged 2 commits into
mainfrom
fix/cli-stream-mistake-limit-recovery

Conversation

@cte

@ctecte commented Feb 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • stop treating recoverable idle asks (mistake_limit_reached, api_req_failed) as terminal task completion in CLI stream mode
  • keep terminal completion events for completion_result and resume_completed_task
  • add --consecutive-mistake-limit <limit> CLI flag and thread it into extension settings
  • set CLI default consecutive mistake limit to 10 and document it
  • add/update tests for completion semantics and CLI settings propagation

Validation

  • pnpm -C apps/cli run -s check-types
  • pnpm -C apps/cli run -s lint
  • pnpm -C apps/cli vitest run src/agent/__tests__/events.test.ts src/agent/__tests__/extension-host.test.ts src/agent/__tests__/extension-client.test.ts src/lib/storage/__tests__/settings.test.ts
  • pnpm -C apps/cli vitest run src/commands/cli/__tests__/run.test.ts src/commands/cli/__tests__/parse-stdin-command.test.ts

Interactively review PR in Roo Code Cloud

@dosubotdosubotBot added size:L This PR changes 100-499 lines, ignoring generated files. Enhancement New feature or request labels Feb 27, 2026
@cte
cte merged commit 2f4ce36 into mainFeb 27, 2026
7 checks passed
@cte
cte deleted the fix/cli-stream-mistake-limit-recovery branch February 27, 2026 05:09
@github-actionsgithub-actionsBot mentioned this pull request Mar 4, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

EnhancementNew feature or requestsize:LThis PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@cte