test(web): remove static markup-only component tests by t3-code[bot] · Pull Request #9364 · pingdotgg/t3code

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

t3-code · 2026-09-03T06:12:52Z

problem

six web component test files rendered components to static html and asserted copied class names, serialized props, or text. they did not exercise layout, tooltips, browser input state, or user interaction, so they could pass while the behavior named by the tests was broken.

change

remove command.test.tsx and ProjectScriptsControl.test.tsx, which only pinned tailwind strings in server-rendered markup
remove PreviewChromeRow.test.tsx and PanelLayoutControls.test.tsx, which only confirmed that input props were serialized into html and could not exercise the focus or tooltip behavior they claimed
remove ComposerPendingTerminalContexts.test.tsx, whose only non-style assertions duplicate terminalContext.test.ts coverage for expiration and label formatting
remove ComposerPromptLengthValidation.test.tsx, whose message and rejection path are already covered by the stronger composerSubmission.test.ts

production code is unchanged.

evidence

before removal:

vp test run on the six removed files plus composerSubmission.test.ts and terminalContext.test.ts: 8 files passed, 34 tests passed

after removal:

vp test run apps/web/src/components/chat/composerSubmission.test.ts apps/web/src/lib/terminalContext.test.ts apps/web/src/components/preview/PreviewView.test.tsx: 3 files passed, 36 tests passed
vp run --filter @t3tools/web typecheck: passed
git diff --check: passed
diff review: 6 test files deleted, 199 lines removed, no production changes

created by gpt-5.6-sol with hermes agent.

Note

Low Risk
Test-only deletions with no runtime changes; risk is reduced brittle coverage, not user-facing regressions if the cited unit tests stay green.

Overview
Removes six web component tests that only used renderToStaticMarkup and asserted Tailwind classes, serialized HTML attributes, or duplicated strings—without covering focus, tooltips, layout, or interaction.

Deleted files: ProjectScriptsControl.test.tsx, ComposerPendingTerminalContexts.test.tsx, ComposerPromptLengthValidation.test.tsx, PanelLayoutControls.test.tsx, PreviewChromeRow.test.tsx, and ui/command.test.tsx. No production code changes.

Behavior those tests hinted at remains covered elsewhere (e.g. composerSubmission.test.ts for prompt length limits, terminalContext.test.ts for terminal context labels/formatting).

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

Note

Remove static markup-only component tests from web app

Deletes six test files that asserted static markup and CSS classes without exercising real component behavior. These covered ProjectScriptsControl, ComposerPendingTerminalContextChip, ComposerPromptLengthValidation, PanelLayoutControls, PreviewChromeRow, and command shell geometry.
The removed tests only checked rendered output structure and class names, not interactions or runtime logic.

^{Macroscope summarized f4fa531.}

github-actions · 2026-09-03T06:15:02Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.7 KiB	13.7 KiB	+3 B (+0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+1 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	58.5 KiB	58.5 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.8 KiB	13.8 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	+2 B (+0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.7 KiB	6.7 KiB	+9 B (+0.1%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	59.3 KiB	59.3 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: b5f4e81 · PR result: f4fa531 · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

macroscopeapp · 2026-09-03T06:15:53Z

Approvability

Verdict: Approved at f4fa531

Macroscope's review found this PR approvable — This PR only removes six static-markup web unit tests; production components, runtime paths, and product defaults remain unchanged. The impact is limited to reduced brittle test coverage in CI, with no customer-facing behavior change.

Notes:

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

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

test(web): remove static markup-only component tests
f4fa531

github-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Sep 3, 2026

macroscopeappBot approved these changes Sep 3, 2026
View reviewed changes