transcript-cohesion: give the file a 480 s limit (it takes about 240 s; shard 1 timed out)

Merged
TE
2d ago
+1-0

What

Base red: test/transcript-cohesion.test.ts fails or times out on feat-cmux-next, and it takes webviews shard 1 past its 30-minute limit. That blocks every PR whose shard 1 runs it, #35312 among them.

  • The claude variant expected no Worked for header. #34774 and #35113 changed that on purpose: a finished Claude turn now folds under one Worked for, closed, like Codex's. The test now waits for that header and checks the shared rhythm, a 22px header with the reply 16px below.
  • The file runs 48 browser cases, about 240 s on a fleet worker. The runner's default per-file limit is 120 s, so the file always timed out. It now declares // cmux-web-test-timeout: 480, like the accent-glass files.

Testing

On a fleet worker with the CI runner (bun run test test/transcript-cohesion.test.ts, no retries):

  • before: 46 pass and 2 fail (claude keeps the activity rhythm, light and dark) in 235 s, which is a TIMEOUT at the 120 s limit;
  • after: PASS, 48 of 48, in 237 s.

Observed / Not run

  • Observed: the run above.
  • Not run: the full webviews suite.

Changelog

none

🤖 Generated with Claude Code

Agent session: agent session


Autofix with [code]smith
Open in [code]smith · Need help on this PR? Tag @codesmith-bot with what you need.


Summary by cubic

Adds the cmux-web-test-timeout: 480 file limit to transcript-cohesion, whose 48 browser cases take about 240 s and kept timing out at the runner's 120 s default, taking webviews shard 1 along with it. The merge from feat-cmux-next also brings in #35410, which fixed the claude variant's assertion to expect the folded Worked for header introduced by #34774 and #35113.