Repository navigation
fix(computer): keep the probe's fixture above an app's startup windows on X11 - #528
Merged
Merged
Conversation
…s on X11 An app that keeps a window above others at startup, as Typora's licence dialog does for about two seconds, stayed over the full-screen fixture on xfwm4, which won't put a full-screen window above it, and took the probe's first clicks: the probe then failed with "P4 a click in frame space hits its cell". On X11 the fixture is now maximized and kept on top, raising itself again, at most twice a second, when another window covers it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… fixture takes the top back Raising the fixture focused its root, so a window an app opened during a typing check would leave the probe's keys with the root instead of the text box it had just clicked. The widget that last had the focus now gets it back. The covered integration test also requires every check to have run: a fixture that never started leaves its checks unverified, not failed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
6 of 22 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The computer probe's fixture could lose to an app's startup window on X11, and the probe then failed with
P4 a click in frame space hits its cell.The fixture relied on full screen to be on top. xfwm4 won't put a full-screen window above a window an app has put above. One app keeps a licence dialog above for about two seconds at startup. If the fixture appeared within those two seconds:
This is a race, so it fails some boots and not others. A stacking capture during a natural failure shows the dialog
ABOVEand taking the click. The same cause fits an app that opens maximized at boot.On X11 the fixture is now:
-topmost;<Visibility>says another window covers it, at most once every 0.5 s, so it can't fight a window that raises itself.macOS and Windows keep full screen. The probe aims at the boxes the fixture reports, so the smaller, maximized area changes nothing for the checks.
Test plan
mainand passes here. All 37 conformance tests pass.tst/integration/computer/covered_test.py: an app whose boot window stays on top over the middle of the screen, run withagent-env run --no-emulate. It must reachreadywith no failed check.mainit fails with P4's frame, norm, double and crop clicks.main, 5 of 5 here.main, 3 of 3 pass here.main(cua driver, a GitHub clone test that needs credentials, a Windows disk recipe).🤖 Generated with Claude Code


Confidence Score: 5/5
The PR appears safe to merge.Summary
The PR keeps the X11 fixture above app windows and adds a covered-window regression test.
Reviews (7) · Last reviewed commit: "fix(computer): give the focus back to th..." · Reviewed by Greptile