Skip to content

fix(mcp): share CLI timeout derivation - #96

Merged
yashranaway merged 1 commit into
mainfrom
t3code/mcp-timeout-parity
Aug 10, 2026
Merged

fix(mcp): share CLI timeout derivation#96
yashranaway merged 1 commit into
mainfrom
t3code/mcp-timeout-parity

Conversation

@yashranaway

Copy link
Copy Markdown
Collaborator

Closes #29.

What changed

  • Move command timeout derivation into shared HeadlessProtocol code.
  • Make both the CLI and stdio MCP adapter call that one helper.
  • Preserve the existing CLI bounds for normal commands, waits, tours, screenshots, screenshot series, and recording stop.
  • Mark backlog C1 complete.

This fixes MCP calls such as wait --timeout 90000, which previously inherited a flat 30-second socket timeout despite requesting a valid 90-second wait.

Verification

  • pnpm test:runtime: passed.
  • Swift 6.1 protocol/security suite in Docker: 31/31 passed.
  • Explicit Linux build of the headless product: passed.
  • Explicit Linux build of the headless-mcp product: passed.
  • git diff --check: passed.

@yashranaway
yashranaway merged commit 30bb758 into main Aug 10, 2026
7 checks passed
@yashranaway
yashranaway deleted the t3code/mcp-timeout-parity branch August 10, 2026 16:48
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.

C1 — Timeout parity

1 participant