Skip to content

docs: route Peekaboo automation through app host - #22

Merged
steipete merged 1 commit into
mainfrom
docs/peekaboo-gui-host
Jul 1, 2026
Merged

docs: route Peekaboo automation through app host#22
steipete merged 1 commit into
mainfrom
docs/peekaboo-gui-host

Conversation

@steipete

@steipete steipete commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • launch Peekaboo.app before live UI automation so app-held TCC grants are available
  • distinguish the GUI bridge.sock host from the reusable CLI daemon.sock
  • verify the GUI host through its explicit socket and reserve --no-remote for caller-local TCC tests

Proof

  • checked the host contract against steipete/Peekaboo main at b0184ad3ef88
  • live open -a Peekaboo plus explicit bridge status --verbose --json --bridge-socket .../Peekaboo/bridge.sock selected hostKind: gui
  • live bridge permissions: Screen Recording, Accessibility, and Event Synthesizing granted
  • scripts/validate-skills — 49 skills validated
  • final autoreview (--mode local) — clean, no accepted/actionable findings; fixed the initial default-routing ambiguity it reported

@steipete
steipete merged commit 9342caf into main Jul 1, 2026
2 checks passed
@steipete
steipete deleted the docs/peekaboo-gui-host branch July 1, 2026 13:19

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0e5a000c03

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread skills/peekaboo/SKILL.md
7. Verify output files with `sips -g pixelWidth -g pixelHeight <path>` or view the image.
2. For live UI work, launch `Peekaboo.app`; verify the GUI bridge and its permissions.
3. Run `permissions status --json`; if missing TCC, report exact missing grant.
4. For screenshots, use `image`; include `--path`, `--json`, and usually `--no-remote` only when deliberately testing caller-local TCC.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Remove --no-remote from the default screenshot path

This newly narrows --no-remote to caller-local TCC tests, but the Common Commands screenshot example above still includes --no-remote, so agents copying the default capture command will bypass the launched app/Bridge path. Peekaboo's Bridge docs state that --no-remote forces local execution, which means normal screenshots will still lose the app-held TCC grants this change is trying to use; please remove it from the default example or label that example as local-TCC-only.

Useful? React with 👍 / 👎.

Comment thread skills/peekaboo/SKILL.md
[ -x "$PB" ] || PB="$(command -v peekaboo)"

open -a Peekaboo
"$PB" bridge status --verbose --json --bridge-socket "$HOME/Library/Application Support/Peekaboo/bridge.sock"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Pin the GUI socket for follow-up live commands

In environments where a healthy reusable daemon already exists, this verifies bridge.sock once but does not make the later image/see/click/type commands use that GUI host. The Peekaboo Bridge docs say default automation commands use the daemon first and only --bridge-socket or PEEKABOO_BRIDGE_SOCKET overrides host discovery, so a Mac whose TCC grants are on Peekaboo.app but not the daemon can still fail after following this block; export PEEKABOO_BRIDGE_SOCKET or pass the socket on the live commands.

Useful? React with 👍 / 👎.

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