[henrychu04]

Follow-ups from the Fable review of the e923627 sync (#259, #260). The review used three read-only area reviewers: server and bridge, client-runtime, and web plus gateway. None of them found a merge regression. This PR fixes the small drift they did report.

Changes

• Command palette. isAddProjectEnvironmentView no longer checks the "environments" group. After refactor(web): command palette state lives in a store, git and branch pickers drop timer and sync effects pingdotgg/t3code#25733, every Add project view opens on ADD_PROJECT_ENTRY_GROUP, so that arm was dead.
• CI-only cuts recorded. Upstream's agent context budget test (test(server): CI fails when agent context grows past main pingdotgg/t3code#25785) and built web boot check (ci(web): releases fail when the built web app cannot boot pingdotgg/t3code#25929) are now noted in the seams doc, sync.md, and sync-resolve-deleted.mjs, so the next sync removes them automatically. The budget test measures MCP tool context, which the file bridge replaces.
• Seams doc fixes:
  • It now says upstream's connection/supervisor.test.ts is not carried.
  • It notes that the canvas tests carry no preview cases.
  • It names CommandPalette.browser.test.tsx's useCoder mock.
  • It removes a stale sentence about a focus test upstream deleted in test: delete 11k low-signal tests so CI runs faster and flakes less pingdotgg/t3code#22557.
  • The GitLab CLI fixes are now marked as checked against e923627. Upstream did not touch source-control-gitlab in this range.

Not changed, by choice

• Upstream-verbatim helpers used only by their tests: resolveAddProjectEntry and the ACP wizard helpers. Their files stay byte-identical to upstream, which keeps syncs small.
• The inert silent-socket loop that comes with the gateway route's checkHost seam. Cutting it would make the supervisor delta much larger.
• Pre-existing "T3 Code" strings in About, provider status, and keybinding text.

Verification

• pnpm typecheck:web passes.
• The CommandPalette browser and logic tests pass (35 tests).
• pnpm test:web:coder: 3944 of 3945 tests pass. The one failure, localApi.test.ts, is the known load flake (Tests that flake under full-suite load #236) and passes when run alone.
• pnpm knip:check, vp fmt --check, and node scripts/sync-resolve-deleted.mjs all pass.

🤖 Generated with Claude Code