From 47aada6afcee483a0361009a3b31cdc56e602f3c Mon Sep 17 00:00:00 2001 From: danshapiro Date: Wed, 26 Nov 2025 14:29:28 -0800 Subject: [PATCH 1/2] Add optional wait before auto screenshots From 498e60d3ca15a125bb81d9ee09609d4c53a9023a Mon Sep 17 00:00:00 2001 From: danshapiro Date: Wed, 26 Nov 2025 14:31:15 -0800 Subject: [PATCH 2/2] Add proposal for wait-before-screenshot hooks --- PROPOSAL_wait-before-screenshot.md | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 PROPOSAL_wait-before-screenshot.md diff --git a/PROPOSAL_wait-before-screenshot.md b/PROPOSAL_wait-before-screenshot.md new file mode 100644 index 0000000..43e5484 --- /dev/null +++ b/PROPOSAL_wait-before-screenshot.md @@ -0,0 +1,6 @@ +## Proposal: wait hooks for reliable auto screenshots + +- Problem: auto screenshots fire immediately after actions; animated dialogs are still hidden (opacity/transform), so captures can miss them even though they exist in the DOM. +- Impact: screenshots may show the underlying page instead of the UI the user just opened. +- Fix: allow an optional post-action delay (`waitAfterActionMs`) and a screenshot `waitForVisible` on a selector before capture. Defaults stay lightweight to avoid regressions. +- Identified and prepared by Codex CLI (GPT-5) while debugging missing controls in captured images.