Skip to content

fix: wait for Claude TUI startup readiness - #4276

Merged
atomantic merged 2 commits into
mainfrom
claim/issue-4180
Aug 15, 2026
Merged

fix: wait for Claude TUI startup readiness#4276
atomantic merged 2 commits into
mainfrom
claim/issue-4180

Conversation

@atomantic

Copy link
Copy Markdown
Owner

Summary

  • Use shared PTY readiness tracking for direct Claude one-shot runs.
  • Dismiss folder-trust and auto-mode prompts before pasting, with a safe timeout if no input prompt appears.
  • Cover split control sequences and preserve the non-Claude fallback behavior.

Test plan

  • cd server && NODE_ENV=test npm test -- tuiPromptRunner.test.js tuiHandshake.test.js
  • cd server && NODE_ENV=test npx vitest run services/agentTuiSpawning.test.js

Closes #4180

@atomantic

Copy link
Copy Markdown
Owner Author

Review gate blocked: claude is installed but not authenticated (Not logged in · Please run /login), so the required Claude CLI review could not run. Codex review completed cleanly. Please authenticate Claude and rerun the required review before merging.

…latches

needsAutoModeChoice sets autoModeAnswered=true in ackAutoModeChoice(),
which permanently blocks it from being re-armed in observe() — the
local flag could never diverge from the tracker's own state.
@atomantic
atomantic merged commit 96b099d into main Aug 15, 2026
7 checks passed
@atomantic
atomantic deleted the claim/issue-4180 branch August 15, 2026 14:41
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.

Give tuiPromptRunner the same startup-dialog dismissals as the CoS agent path

1 participant