Skip to content

Browser toolbox D: approval screenshots - #27

Draft
josephschorr wants to merge 2 commits into
pr/browser-cfrom
pr/browser-d
Draft

josephschorr wants to merge 2 commits into
pr/browser-cfrom
pr/browser-d

Conversation

@josephschorr

Copy link
Copy Markdown
Member

Captures a viewport screenshot with fail-closed redaction of credential-filled fields, stores it as an ArtifactRender, and renders it on the approval surfaces — the web chat interaction card, the Slack approval thread, and a link in the local TUI.

This is part of the stacked browser toolbox series (A→E), to be reviewed and merged bottom-up. Stacked on C.

Draft: the integration + e2e execution legs (envtest + SpiceDB) are pending — the local box is under sustained load. Full build, go vet -tags=integration/e2e, manifest-drift, and all changed packages under -race are green. The gate will run and this will be marked ready before merge.

@vercel

vercel Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
openagentprimitives Ready Ready Preview Oct 11, 2026 4:02am UTC

Request Review

Adds the optional Screenshot reference on the interaction request payload, viewport capture with fail-closed redaction of credential-filled fields, storage as an ArtifactRender, and attachment to browser interactions and the CAPTCHA notice.
Renders the approval screenshot in the web chat interaction card, uploads it into the Slack approval thread, and shows a link in the local TUI — with e2e coverage and docs.

This branch was successfully deployed

1 active deployment
Preview — 8b3deda9 Deployed Oct 11, 2026 by vercel[bot]
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