test: prune trivial error and layout tests

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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

[t3-code]

this removes seven complete low-signal test files and narrows three source symbols that were public only for those tests.

removals

• deletes error-value tests that repeated tagged-error constructor fields and message getters
• deletes sidebar width tests that mirrored simple clamp expressions and searched raw source for tailwind classes
• deletes direct mirrors of string truncation, session fallback, and catalog error construction
• keeps CatalogDependencyResolutionError, initialConfigOption, and THREAD_SIDEBAR_DEFAULT_WIDTH local now that no production consumer imports them

this removes 15 tests and 213 test lines. production behavior is unchanged.

verification

before deletion:

• vp test run <seven removed files> passed, 7 files and 15 tests

after deletion:

• retained checkpointing, terminal, and client-runtime neighbor suites passed, 7 files and 85 tests
• affected scripts, web, client-runtime, shared, and server package typechecks passed
• typechecks emitted only existing effect suggestion diagnostics
• targeted lint passed
• targeted formatting passed
• git diff --check passed
• all 1,030 open pull requests were checked for path conflicts; none touches these files

generated by gpt-5.6-sol with hermes agent.

request provenance

• requested by: @juliusmarminge

---

Note

Low Risk
Test-only deletions and export-to-module-private refactors with no runtime logic changes.

Overview
Removes seven low-signal test files (~213 lines, 15 tests) that mostly asserted tagged-error fields/messages, mirrored simple clamp math for the thread sidebar, or read source files for Tailwind class strings. Production behavior is unchanged.

Tightens module boundaries by making symbols test-only exports private: THREAD_SIDEBAR_DEFAULT_WIDTH in threadSidebarWidth.ts, initialConfigOption in session.ts, and CatalogDependencyResolutionError in resolve-catalog.ts.

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

Note

Remove trivial error and layout test files and privatize now-unused exports

• Deletes nine test files that validated simple error constructors, schema checks, layout constants, and string utilities.
• Removes the export keyword from THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, and CatalogDependencyResolutionError, making them module-private since their tests no longer reference them externally.
• Risk: any out-of-tree code importing THREAD_SIDEBAR_DEFAULT_WIDTH, initialConfigOption, or CatalogDependencyResolutionError from threadSidebarWidth.ts, session.ts, or resolve-catalog.ts will break.

^{Macroscope summarized 746f1fd.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.8%)	15.1 KiB	✅
Codex	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+26.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+599 B (+10.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	49.7 KiB	55.6 KiB	+5.9 KiB (+11.8%)	66.4 KiB	✅
Codex	Live turn messages	16	11	−5 (−31.3%)	21	✅
Claude	Total thread wire	11.3 KiB	13.3 KiB	+2.0 KiB (+17.7%)	15.1 KiB	✅
Claude	Thread snapshot wire	5.5 KiB	6.9 KiB	+1.4 KiB (+25.9%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	5.9 KiB	6.4 KiB	+604 B (+10.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	50.6 KiB	56.4 KiB	+5.9 KiB (+11.6%)	66.4 KiB	✅
Claude	Live turn messages	16	11	−5 (−31.3%)	21	✅

Baseline: 83d769f · PR result: 746f1fd · 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 746f1fd

Macroscope's review found this PR approvable — The change is limited to deleting test-only files and making three previously test-only exports module-private; the underlying production logic and request-path behavior remain unchanged. The only residual impact is a narrow reduction of undocumented export surface, with no in-repository consumers found.

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