[t3dotgg]

Problem

On Claude threads, ChatGPT Computer Use asks "Allow Computer Use to use X?" on every action, and the card only offered a one-time Approve. A short Computer Use task meant approving dozens of times, often several times for one tool call.

The Computer Use server never remembers an answer. The client must. Codex threads already do this: the card offers "Always allow this session" and Codex answers later prompts itself. The Claude adapter only allowed one-time accepts (#21903).

Fix

The Claude elicitation card now offers "Always allow this session". Picking it stores a grant for that MCP server and exact prompt in the adapter session. Later identical prompts get the same acceptance with no card. A different prompt (for example another app) or another server still asks. Grants end with the provider session; "Always allow" across sessions is still not offered.

The Claude CLI strips _meta from elicitation replies, so sending _meta.persist to the server would do nothing. The grant lives in T3.

Web and mobile need no change; they render the options generically and already show this one on Codex cards.

More menu on the card	After one click, three calls go through

Proof: a new adapter test that fails on main, vp run check:changed green, and an end-to-end run in a dev server (Claude Haiku thread, stdio MCP server that elicits on every call like Computer Use): one click on "Always allow this session", then all 3 tool calls accepted with a single card.

Reviewed with sol-loop: 1 pass with GPT-6.1 Sol. Sol verdict: approved

Created with Claude Opus 5.5 in Claude Code, running in T3 Code.

🤖 Generated with Claude Code

[macroscopeapp]

Approvability

Verdict: Approved at 4907789

Macroscope's review found this PR approvable — Claude MCP prompts gain an explicit “Always allow this session” choice. Grants remain in memory within the opened session, while ordinary one-time approval behavior stays unchanged.

^{You can add or adjust custom eligibility rules. Learn more.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	5.0 KiB	5.0 KiB	0 B (0.0%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.2 KiB	0 B (0.0%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	21.5 KiB	21.5 KiB	0 B (0.0%)	24.4 KiB	✅
Codex	Live turn messages	1	1	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	0 B (0.0%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	0 B (0.0%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.9 KiB	21.9 KiB	0 B (0.0%)	24.4 KiB	✅
Claude	Live turn messages	1	1	0 (0.0%)	8	✅

Baseline: 6c9adfb · PR result: 4907789 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 108.5 KiB
• Claude decoded thread snapshot: 108.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

Claude MCP form elicitation approvals now include an “Always allow this session” choice. The adapter reuses accepted responses for later requests with the same server name and message. Tests cover matching requests, different messages or servers, and the existing acceptAlways decline behavior.

Changes: Claude MCP session grants

Layer / File(s)	Summary
Grant handling and coverage apps/server/src/orchestration-v2/Adapters/ClaudeAdapterV2.ts, apps/server/src/orchestration-v2/Adapters/ClaudeAdapterV2.test.ts	The adapter records session grants by server name and request message, and returns an accepted response for a matching request without opening another approval card. Tests cover grant reuse, scope, and acceptAlways declining.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Possibly related PRs

• pingdotgg/t3code#21903: Adds the Claude MCP elicitation approval flow extended by this change.
• pingdotgg/t3code#33211: Defines elicitation choices for URL mode, including declining acceptForSession.
• pingdotgg/t3code#31315: Changes accepted responses for Claude form elicitation fields used by this grant flow.

Suggested labels: size:S

Suggested reviewers: chrischriscris, macroscopeapp

Merge Risk: Minimal · up to 49077

The session approval behavior is ready to merge after normal checks; no demonstrated consent or reuse defect remains.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly summarizes the main change: Claude threads can allow an MCP app for the session.
Description check		The description explains the problem and fix, limits the grant to the session, and reports focused tests and an end-to-end check. It does not include the required Scope and approval section or explain…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description explains the problem and fix, limits the grant to the session, and reports focused tests and an end-to-end check. It does not include the required Scope and approval section or explain why the change qualifies for an approval exemption.

Finishing Touches

Generate docstrings

• Commit to this branch
• Create a new PR

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: t3/fix-repeated-computer-use-approvals
• File: .coderabbit.config.ts
• Commit: 4907789
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Team
• Run ID: 100ccf95-525e-4ea4-88e3-36535527327a

Commits

Reviewing files that changed from the base of the PR and between 6c9adfb and 4907789.

Files selected for processing: 2

• apps/server/src/orchestration-v2/Adapters/ClaudeAdapterV2.test.ts
• apps/server/src/orchestration-v2/Adapters/ClaudeAdapterV2.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.