Skip to content

ci: package and upload extension zip artifact#3

Merged
twschiller merged 1 commit into
mainfrom
ci/package-extension-zip
May 30, 2026
Merged

ci: package and upload extension zip artifact#3
twschiller merged 1 commit into
mainfrom
ci/package-extension-zip

Conversation

@twschiller
Copy link
Copy Markdown
Contributor

Summary

  • Run bun run package (which invokes scripts/package-extension.ts) after the extension build in CI
  • Upload the resulting output/extension.zip as the extension-zip artifact on every run
  • Zip is 1-layer with manifest.json at the root, matching Browserbase's required layout

Test plan

  • CI passes on this PR
  • extension-zip artifact is downloadable from the workflow run
  • Downloaded zip extracts with manifest.json at the top level (no nested folder)

🤖 Generated with Claude Code

Run the existing package-extension.ts script after the extension build
and upload the resulting 1-layer zip (manifest.json at the root) as a
CI artifact, so each build produces a Browserbase-ready archive.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 30, 2026

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

Project Deployment Actions Updated (UTC)
agent-browser-shield-demo-site Ready Ready Preview, Comment May 30, 2026 3:45am

Request Review

@twschiller twschiller merged commit b0e8ff1 into main May 30, 2026
6 checks passed
@twschiller twschiller deleted the ci/package-extension-zip branch May 30, 2026 03:46
twschiller added a commit that referenced this pull request Jun 7, 2026
Adds a capture-phase `change` listener that removes `[data-abs-cleared]`
on trusted (real user / WebDriver / CDP) interactions. Without this, a
controlled React/Vue checkbox would visually flicker on a real click —
native activation toggles the box to true, the framework reconciles
`node.checked = true` from new component state, and the prototype patch
would revert that reconcile because the marker is still present.
Capture-phase placement guarantees the marker is gone before any
framework handler schedules the reconcile.

Synthetic events (`isTrusted === false`, including page-script
`element.click()` and our own `uncheck` dispatch) do not release the
lock — only genuine user gestures do.

Doc updates:
- `docs/src/content/docs/rules.md`: note that the cleared state is held
  against framework re-renders but yields to real user / WebDriver
  clicks.
- `skills/agent-browser-shield/SKILL.md` item #3: tell agents to drive
  the toggle through a click (the standard Playwright/CDP/`.click()`
  path), not `input.checked = true`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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