fix(terminal): identify T3 Code to integrated shells #22827 · pingdotgg/t3code

Scope and approval

Fixes #8450

Triaged bug via the automated issue flow.

Manager.test.ts: 42 → 43 tests, 43 passing after implementation. The new open/restart regression fails on the tests-only commit because the parent terminal identity remains.

Scoped lint passes (one existing warning); server typecheck passes with existing suggestions.

No regex, parser, dependency, schema, client, or render changes.

Verification

References #8450.

Validation

Evidence

Linux browser integrated terminal	Before	After
Real app, bash, node-pty; command `env	grep -E 'TERM	COLORTERM'`

Before output: COLORTERM=truecolor, TERM=xterm-256color; TERM_PROGRAM and TERM_PROGRAM_VERSION absent. After adds TERM_PROGRAM=t3code, TERM_PROGRAM_VERSION=0.1.1, matching the server package version. The app ran with T3CODE_HOME unset and explicit isolated home /home/ubuntu/wt/issue-8450/.t3, web port 7015, server port 15055. The transcript includes the checked-out after SHA. A new shell was opened after the dev server reloaded the implementation.

What else changed: no render paths changed. Runtime output adds the two terminal identity variables; TERM and COLORTERM remain as before. No screenshot/pixel diff is applicable to this server environment change.

Unverified surfaces: native Windows ConPTY and Electron desktop/macOS, including packaged desktop version. Base inherited identity, open/restart and unrelated passthrough/filtering are covered by the service test; session runtime identity override precedence was inspected in code, not independently exercised. The fixed T3 identity also replaces an explicitly supplied TERM_PROGRAM/TERM_PROGRAM_VERSION. Mixed-case Windows inherited identity keys were not exercised.

Model: GPT-6.1-Sol. Harness: Codex through T3 Code. Independent reviewers: Codex GPT-6.1-Sol and Claude Opus 5.5, both VERDICT: MERGE with no confirmed findings. Proof audit pass 1 accepted the evidence plan; pass 2 returned SUFFICIENT. New and restarted shells receive the identity; already-running shells retain their old environment.

Ggithub-actionsBot3d ago

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	4.9 KiB	5.0 KiB	+23 B (+0.5%)	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	+23 B (+1.9%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.8 KiB	20.9 KiB	+41 B (+0.2%)	29.3 KiB	✅
Codex	Live turn messages	1	2	+1 (+100.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.2 KiB	21.2 KiB	0 B (0.0%)	29.3 KiB	✅
Claude	Live turn messages	2	2	0 (0.0%)	8	✅

Baseline: 8d3b061 · PR result: acaee88 · 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.

fix(terminal): identify T3 Code to integrated shells#22827

Problem

Change