Skip to content

fix(computer): no thumbnail of a window that may show a GTK4 password field - #852

Open
OAraLabs wants to merge 1 commit into
mainfrom
fix/computer-gtk4-password-no-thumbnail
Open

OAraLabs wants to merge 1 commit into
mainfrom
fix/computer-gtk4-password-no-thumbnail

Conversation

@OAraLabs

Copy link
Copy Markdown
Owner

Why

The cockpit's per-step thumbnail is skipped when a password field is in the walk, judged by a role containing "password" (AT-SPI's password text). GTK4 reports a PasswordEntry as an unnamed text box, exactly like an empty entry, so a GTK4 sign-in window was captured and its thumbnail sent to Beacon ("thumbnail": "sent" on the test rig).

What

WindowCapture counts the walk's unnamed text fields, and decide_capture skips the picture (password_field) when there is any: GTK4 gives no way to tell the two apart. The capture is built in one module function, _window_capture(), so the count is tested from SDK-shaped output.

Verification

New tests/test_computer_gtk4_password_thumbnail.py (all red before). Computer-use suites 634 passed; ruff clean. On the rig the same sign-in step then reported thumbnail: skipped, password_field.

🤖 Generated with Claude Code

… field

The cockpit's per-step thumbnail is skipped when a password field is in the
walk, judged by a role containing "password" (AT-SPI's "password text").
GTK4 reports a PasswordEntry as an unnamed "text box", exactly like an empty
entry, so a GTK4 sign-in window was captured and sent to Beacon
("thumbnail": "sent" on the computer-use rig).

WindowCapture now counts the walk's unnamed text fields, and decide_capture
skips the picture ("password_field") when there is any: GTK4 gives no way to
tell the two apart, and any doubt skips. The capture is built in one module
function, _window_capture(), so the count is tested from SDK-shaped output.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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