base red: transcript spacing table rows that #31497 brought to value

Merged
TE
4d ago
+2-5

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 pending on those three rows.
  • Keep step-to-live (base: 25) and live-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.ts failed 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 Autofix with [code]smith
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.