test(gallery): agent-pane entry cases open each entry's own frame (base red: check:browser-lanes)

Closed
TE
16d ago
+5-3

Summary

Base red: check:browser-lanes fails on webviews/test/gallery-agent-pane-entries.test.ts (lines 85 and 94, "opens a gallery frame without entryPath"). #23250 added that guard, and #23237 added this file, at about the same time. The cases now open galleryFrame(entry, { variant }), and warmGalleryFrame warms each entry's frame instead of every gallery file. That also ends the cold 100 s warm-up timeouts this file used to hit.

Changelog

none

Test

  • cd webviews && bun run check:browser-lanes: red on base e07b30ae690 (the two lines above); passes on head.
  • CMUX_BROWSER_TESTS=1 bun test test/gallery-agent-pane-entries.test.ts on a cold server: 12 of 13 passed in 111 s, with no warm-up timeout. The one failure, harness-health-notice#dismissed (layout shift 0.0019 on clicking Update), passed on the rerun. That is the notice's leading spinner moving the words whenever the "updating" frame gets painted, a product bug the Not Now PR fixes.

Observed / Not run

  • Observed: the runs above. scripts/web-lint.sh passes. prepush-gate.py: no new violations.

No capture: test-only change.

Agent session: huge-blue

🤖 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 gallery agent-pane entry tests so each case opens its own entry's frame, satisfying the new browser-lane guard.

  • Each frame now loads only the entry under test via galleryFrame instead of every gallery file.
  • warmGalleryFrame warms each entry's frame, ending the cold 100 s warm-up timeouts this file hit.