Repository navigation
refactor(server): name Codex's direct-only T3 MCP server t3-code-direct - #29068
Conversation
The split server from #29026 was named `t3code`, which the model sees as `mcp__t3code` next to `mcp__t3_code`. Rename it to `t3-code-direct` so its purpose is clear. Only the Codex adapter config changes; the shared tool resolver also recognizes the new name. Co-Authored-By: Claude Opus 5.5 <[email protected]>
github-actions
Bot
added
vouch:trusted
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — Renames Codex’s direct-only MCP server and updates tool-name recognition while preserving legacy aliases. Tool availability, authorization, and execution restrictions remain unchanged, so the change carries limited risk. You can add or adjust custom eligibility rules. Learn more. |
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: Scenario and decoded snapshot size10 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.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Rename server and recognize namespace apps/server/src/orchestration-v2/Adapters/CodexAdapterV2.ts, apps/server/src/orchestration-v2/Adapters/CodexAdapterV2.test.ts, docs/orchestration-v2/orchestrator-mcp-server.md, packages/shared/src/t3McpToolPresentation.ts, packages/shared/src/t3McpToolPresentation.test.ts |
The Codex adapter, test, and documentation use t3-code-direct for the direct-wait server. Shared tool presentation recognizes t3-code-direct and t3_code_direct aliases and prefixes. Existing tool settings remain unchanged. |
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Refactor
Suggested reviewers: juliusmarminge
Merge Risk: 
6f94a
-
The Codex direct MCP server rename is consistent across its registration and tool-resolution paths, so no merge-blocking risk remains.
Pre-merge checks: 
-
Check name Status Explanation Title check The title names the main change: the Codex direct-only T3 MCP server is renamed to t3-code-direct. The title follows the conventional commit format and matches the raw summary, the PR objectives, and … Description check The description follows the template's main sections in substance. It gives the problem (the direct-only server was named t3code, which Codex shows as mcp__t3code next to mcp__t3_code) and the change … 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: Title check
-
Explanation
The title names the main change: the Codex direct-only T3 MCP server is renamed to t3-code-direct. The title follows the conventional commit format and matches the raw summary, the PR objectives, and the PR description. The title is specific and concise.
Full details: Description check
-
Explanation
The description follows the template's main sections in substance. It gives the problem (the direct-only server was named t3code, which Codex shows as mcp__t3code next to mcp__t3_code) and the change (rename in the Codex adapter config, resolver recognizes the new name, t3code kept for old waits). The Verification section gives specific checks: a test row that failed before the resolver change and passed after it, and a run with Codex 0.166.0 against a mock model and mock MCP server. The Scope and approval section is missing. The description does not link a triaged issue or maintainer approval, and it does not explain why the change qualifies as focused configuration of an existing capability. The description also omits the template's note about the agent harness, although it includes a 'Made with' attribution line that names a model and harness. The description is mostly complete, so the check passes, but the author should add the scope and approval information.
-
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
**v0.1.7-nightly.20261102.4066** · 16 changes from 2 contributors - **Model and effort history.** See where you changed models or effort levels as you scroll through a thread. - **Antigravity subagent work.** See subagent work in Antigravity threads. - **Model picker in short windows.** Reach model options, effort levels, and controls more easily in short windows. ### Plus 10 more bug fixes and performance improvements. <details> <summary>Full changelog</summary> ## What's Changed * perf(server): Codex calls t3_thread_wait directly instead of from Code Mode by @t3dotgg in pingdotgg/t3code#29026 * refactor(server): name Codex's direct-only T3 MCP server t3-code-direct by @t3dotgg in pingdotgg/t3code#29068 * fix(mobile): files with brackets in their names attach on Android by @juliusmarminge in pingdotgg/t3code#28983 * fix(server): Stop ends a run whose provider turn already finished by @juliusmarminge in pingdotgg/t3code#28520 * fix(server): a linked retry closes the waiting link card it names by @juliusmarminge in pingdotgg/t3code#28387 * fix(server): a stored boolean settledOverride no longer stops startup by @juliusmarminge in pingdotgg/t3code#28314 * fix(server): remote updates no longer fail when the staged preflight is slow on a loaded host by @juliusmarminge in pingdotgg/t3code#28201 * fix(server): idle compaction survives a late update for an earlier run by @juliusmarminge in pingdotgg/t3code#28102 * fix(server): usage prices bare model names at the maker's rate when a reseller marks them up by @juliusmarminge in pingdotgg/t3code#28004 * fix(web): model picker uses the full window height in short windows by @t3dotgg in pingdotgg/t3code#29017 * fix(server): open clients no longer stay on "Updating" after a provider update by @juliusmarminge in pingdotgg/t3code#27992 * ci(release): preview releases run from any branch by @juliusmarminge in pingdotgg/t3code#27828 * fix(desktop): save and copy any image the app can show by @t3dotgg in pingdotgg/t3code#29079 * feat(chat): threads show when you switch models or effort by @t3dotgg in pingdotgg/t3code#28995 * feat(antigravity): runtime 1.4.5 with real turn failures and subagent work by @t3dotgg in pingdotgg/t3code#28629 * feat: tag threads and hide tagged threads while recording by @t3dotgg in pingdotgg/t3code#29053 **Full Changelog**: pingdotgg/t3code@v0.1.7-nightly.20261102.4061...v0.1.7-nightly.20261102.4066 </details> Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.1.7-nightly.20261102.4066
The direct-only T3 server added in #29026 was named
t3code. Codex shows it to the model asmcp__t3code, right next tomcp__t3_code, so nothing explains why the two exist.This renames it to
t3-code-direct. The model now seesmcp__t3_code_direct→t3_thread_wait. Only the Codex adapter config changes. Claude and ACP providers keept3-code. The shared tool resolver also recognizes the new name, so tool display and "waiting" detection keep working.t3codestays recognized for waits recorded since #29026.Proof: a test row for
t3-code-direct.failed before the resolver change and passes after it. Real Codex 0.166.0 against a mock model and a mock MCP server shows a directmcp__t3_code_directnamespace witht3_thread_wait.Made with Claude Opus 5.5 in Claude Code, running in T3 Code.
🤖 Generated with Claude Code