Repository navigation
Investigate history footer overlap in Electron CI captures #61
Description
Part of #2.
The macOS Electron CI desktop captures for #60 draw the history Fork from here action across the expanded tool card near its Input heading. Local screenshots from the same tests place the action below the card. The CI tests pass even after waiting through animation frames and checking that the footer button's DOM bounds are below the visible Output control.
Investigate whether this is a compositor capture discrepancy or an intermittent virtual transcript layout problem. Check the rendered pixels as well as the DOM bounds before changing production layout. Add a regression and reliable desktop proof once the cause is understood.
Uploaded CI proof with the discrepancy, in test-results/tool-preview/write-bounded-preview.png and edit-bounded-preview.png. The narrow captures show the shortened-preview notice and reachable controls without the footer in view.
Reproduce with mise exec -- bun run test:e2e tests/e2e/tool-preview.test.ts on the macOS CI runner. Local capture alone has not reproduced the overlap. The preview-length, full-input clipboard and Show more / Show less assertions pass on both hosts.
Activity
johnnyhuy commentedon Nov 13, 2026
agent claim: agent/issue-62-native-history-captures @ a25471a. Working on child #62 to verify native window sizing and rendered footer visibility. This issue stays open pending hosted macOS evidence.
johnnyhuy commentedon Nov 13, 2026
Child #62 is ready in #63. The earlier test emulated a tall viewport without resizing the native window. The new capture uses actual native content bounds and verifies the renderer, PNG, footer hit target and painted label. All 53 hosted Electron tests pass. Downloaded desktop, minimum-window and collapsed PNGs were visually reviewed and show the footer below the card. Proof and measured sidecars are linked from #63. The original live overlap did not reproduce locally, and the underlying compositor behaviour has not been isolated further. Keeping this parent open until the child is merged.
johnnyhuy commentedon Nov 13, 2026
All child slices for this investigation have merged. Child #62 landed in #63 as 41d05c8.
The capture harness previously emulated a renderer taller than its native window. It now resizes the native window, verifies matching native, renderer and PNG dimensions, and checks footer geometry, hit testing and painted text through expansion, collapse and resizing. The original live overlap has not reproduced, and the Chromium compositor behaviour has not been isolated further. No production layout change was justified.
All 53 hosted Electron tests passed. The hosted screenshots were downloaded and checked again after merge, with the footer below the tool card. Main also passed lint, typechecking, all 90 unit tests, all 77 integration tests and the production build under the pinned mise toolchain. Fresh local Electron capture was blocked by a missing Electron Framework, as recorded in the PR.
Proof is linked from #63 and available at https://github.com/echohello-dev/pidex/actions/runs/43418696489/artifacts/14032688604 until 20 November 2026. Closing this investigation with its regression merged. The wider tool-card roadmap remains open under #2.