base red: transcript spacing table rows that #31497 brought to value
Red on base
webviews/test/transcript-spacing-table.test.ts (#31614) fails on feat-cmux-next since #31497 (7a9e1ab5) landed. #31497 brought step-pitch (22), run-to-step (22) and step-to-prompt (43.5, target 43 ± 1) to their values. Their rows still had pending, so they ran as test.failing, and a passing test.failing reports a failure. This is the forcing function working as designed: the PR that lands a value drops its mark.
Fix
- Drop
pendingon those three rows. - Keep
step-to-live(base: 25) andlive-to-step(base: 49) pending, since they are not on the 22 pitch yet. Their marks now name grid-system (the live line and the waiting group).
Base after #31497, Chromium light and dark: step-pitch 22, run-to-step 22, step-to-prompt 43.5, step-to-live 25, live-to-step 49, prompt-to-step 42.5, step-to-prose 46, prose-to-step 46.75.
Spec: leo-workspace design/transcript-spacing.md.
Observed / Not run
- Before, with all four #31497 marks dropped,
cmux-ci test -- bun test test/transcript-spacing-table.test.tsfailed on step-to-live, live-to-step and step-to-prompt in both themes. - At this head: the same command (result added below).
Changelog
none
Agent session: agent session
🤖 Generated with Claude Code
![View with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3-3x.png)
![Autofix with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark-3x.png)
Need help on this PR? Tag @codesmith-bot with what you need.
Summary by cubic
Fixes the transcript spacing table's test.failing rows on feat-cmux-next so tests no longer report failures when they pass. Un-pends step-pitch, run-to-step, and step-to-prompt now that #31497 brought them to their target values; step-to-live and live-to-step stay pending because they are not on the 22 pitch yet, with their marks now naming grid-system as the blocker.