test(gallery): agent-pane entry cases open each entry's own frame (base red: check:browser-lanes)
Closed
TE
16d ago+5+5+0123456789-3-3-0123456789
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 basee07b30ae690(the two lines above); passes on head.CMUX_BROWSER_TESTS=1 bun test test/gallery-agent-pane-entries.test.tson 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.shpasses.prepush-gate.py: no new violations.
No capture: test-only change.
Agent session: huge-blue
🤖 Generated with Claude Code
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
galleryFrameinstead of every gallery file. warmGalleryFramewarms each entry's frame, ending the cold 100 s warm-up timeouts this file hit.