chore(web): keep the thread details split surface class module-private - #13570
Conversation
99d76f2 exported THREAD_DETAILS_PANEL_SPLIT_BUTTON_SURFACE_CLASS, but only this module uses it, so knip fails the Check job on every PR against the V2 branch. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: unavailable · PR result: Scenario and decoded snapshot size10 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.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This one-line change makes an unused styling constant module-private while preserving its value and all consuming class strings. It is a contained encapsulation cleanup with no product-default, runtime behavior, schema, deployment, or static-analysis suppression impact. You can add or adjust custom eligibility rules. Learn more. |
e116684
into
t3code/codex-turn-mapping
#13570) Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
vp run knip:check, the first step of CI's Check job, fails on the V2 branch tip:99d76f2 added the constant as an export, but only other constants in the same module use it. This keeps it module-private.
Verification
vp run knip:check: clean (exit 0).apps/webtypecheck: no errors or warnings.Model: Claude Opus 5.5 (Claude Code)
🤖 Generated with Claude Code