Skip to content

fix: bridge smoke waits for node position stability before clicking (goal 0107 CI follow-up) - #243

Merged
alicoding merged 1 commit into
mainfrom
smoke-click-stability
Aug 18, 2026
Merged

fix: bridge smoke waits for node position stability before clicking (goal 0107 CI follow-up)#243
alicoding merged 1 commit into
mainfrom
smoke-click-stability

Conversation

@alicoding

Copy link
Copy Markdown
Owner

First CI outcome of the calibrated registry (goal 0107 scope item 2): 5/6 green on the hosted macOS runner — badge, window shape, board render, selection ring, and sticky flip all pass. The one failure was note-card-commit-interaction's select click racing the board's fitView animation — the click lands where the card was mid-animation and selects nothing, the same stale-coordinate class the Playwright suite's clickFrameGutter helper exists for (it also explained the 1-in-3 local flake observed during calibration).

waitForNodeStable polls the card's bounding rect until two consecutive samples agree before the commit and ring checks dispatch their clicks. 3/3 consecutive local runs green; scripted-fake unit tests updated (fast-pathed so negative subtests don't wait out the poll).

Job stays non-required until a CI track record accrues on this.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FW5GkkAG8du7tNdYLk2zSd

…goal 0107 CI follow-up)

First CI run of the calibrated registry: 5/6 green on the hosted
runner; the one failure was the commit check's select click landing
mid-fitView-animation -- the click dispatches to where the card WAS
and selects nothing (the same stale-coordinate class the e2e suite's
clickFrameGutter helper exists for). waitForNodeStable polls the
card's bounding rect across consecutive samples before the commit and
ring checks click; scripted fakes updated. 3/3 local runs green.

Claude-Session: https://claude.ai/code/session_01FW5GkkAG8du7tNdYLk2zSd

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@alicoding
alicoding enabled auto-merge (squash) August 18, 2026 19:01
@alicoding
alicoding merged commit 53b0036 into main Aug 18, 2026
26 of 27 checks passed
@alicoding
alicoding deleted the smoke-click-stability branch August 18, 2026 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant