chore: delete dead code, unused deps, and duplicate helpers

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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

[t3dotgg]

An audit of the repo turned up code that survived refactors after its last importer was removed, exports nothing calls, dependencies nothing imports, and helpers copied byte for byte between files. This PR deletes the parts that need no product decision.

What changed:

• 24 files deleted that nothing imports. The biggest are the old web orchestration recovery and history bootstrap modules, four unused Base UI wrappers, the mobile thread terminal panel and diff parser, and a 442 line Cursor probe script.
• About 60 exports removed that had zero callers, including six __resetForTests hooks no test used, the providerRuntime compatibility alias block, and five desktop IPC schemas that were never decoded.
• Dependencies removed: msw and its service worker, punycode, yaml, @effect/platform-node in tailscale, @t3tools/contracts in scripts. Three stale root scripts removed, including build:contracts which filtered a package with no build.
• withInstanceIdentity was copied into all five provider drivers. Now one module. Three identical expandHomePath copies fold into pathExpansion.ts. Same for splitNullSeparatedPaths, canonicalItemTypeFromAcpToolKind, and isProcessAlive.
• Four tests that could not fail now assert something. Three tests that only checked a stub returned what the stub returned are gone, as is a button test that asserted Tailwind class strings.

Left for a separate decision: LegacySidebar, the planModeEnabled flag, the undocumented T3CODE_STRICT_PROVIDER_LIFECYCLE_GUARD env switch, the deprecated layerTest statics with 149 call sites, and wall-clock sleeps in about 16 server tests.

Verified with per-package typecheck for web, desktop, server, mobile, contracts, shared, client-runtime, and tailscale, lint on every touched file, and the tests for every touched or deduped module.

Changes made by Claude Fable 5.1 in Claude Code.

---

Note

Low Risk
Mostly removals and internal deduplication; risk is limited to undiscovered out-of-tree imports of deleted exports (e.g. CLI flags, entity hooks, contract schemas).

Overview
Large repo hygiene pass that removes code and dependencies with no remaining importers, and folds repeated helpers into shared modules.

Deletions span desktop (PickLabelPosition), mobile (glass header, thread terminal panel, unified diff parser, GlassSafeAreaView, agent-awareness bulk unregister, managed-relay environment status hook), web (orchestration recovery/bootstrap, terminal UI cleanup, unused UI primitives, MSW worker, many __resetForTests hooks), and server (Cursor ACP probe script, migration layer export, provider registry merge helpers, CORS header bundle). Dependencies dropped include msw, mobile punycode, and root MSW config; root build now targets ./apps/* only.

Consolidation: provider drivers share withInstanceIdentity; expandHomePathWith replaces local ~ expansion in workspace/source-control paths; isProcessAlive is shared from serverRuntimeState; ACP tool kinds map via exported canonicalItemTypeFromAcpToolKind; git null-separated stdout parsing is centralized.

Call-site shifts: mobile file preview checks call @t3tools/shared/filePreview directly; composer path search imports state/queries; server integration tests use ProviderSessionRuntime.layer and direct AnalyticsService import. CLI authLocationFlags replaces the old sharedServerLocationFlags alias; several formerly public CLI/boot-service exports are module-private. Preview MCP tools no longer mark “safe” browser tools as destructive.

^{Reviewed by Cursor Bugbot for commit 2dcffed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Delete dead code, unused deps, and duplicate helpers across the codebase

• Consolidates duplicate utilities into shared modules: withInstanceIdentity in instanceIdentity.ts, isProcessAlive in serverRuntimeState.ts, and expandHomePathWith in pathExpansion.ts; driver, CLI, and workspace callers now import these instead of local copies.
• Removes dozens of unused exports across web, mobile, server, shared, and client-runtime packages, including hooks, schema codecs, test reset helpers, layer factories, and model utilities.
• Drops direct dependencies (msw, punycode, yaml, @t3tools/contracts, @effect/platform-node) and removes package export subpaths in effect-acp and effect-codex-app-server.
• Narrows the root build script to ./apps/* only, and removes build:contracts and connect:announce-ga scripts.
• Risk: removed named exports (e.g. web.entities hooks, shared.resolveModelSlugForProvider, ipc.Desktop*Schema codecs, provider.mergeProviderSnapshots, bootService utilities) will break any remaining consumers not updated in this PR; out-of-tree callers importing removed subpaths or test helpers will fail at build time.

^{Macroscope summarized 2dcffed.}

[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	13.3 KiB	13.4 KiB	+154 B (+1.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.6 KiB	+151 B (+2.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	57.0 KiB	+1.4 KiB (+2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.2 KiB	13.3 KiB	+136 B (+1.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.3 KiB	6.4 KiB	+134 B (+2.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	55.5 KiB	56.4 KiB	+884 B (+1.6%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 04efa79 · PR result: 2dcffed · 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: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

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

[macroscopeapp]

Approvability

Verdict: Approved at 2dcffed

Macroscope's review found this PR approvable — This is primarily a dead-code and dependency cleanup with mechanical helper consolidation; the touched production paths retain equivalent behavior, and the remaining edits are test, build, or private-module surface cleanup. No product defaults, deployment targets, or new runtime capabilities are introduced.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

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