Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ci: stop double-zipping extension artifact#7

Merged
twschiller merged 1 commit into
mainfrom
fix-double-zip-artifact
May 30, 2026
Merged

ci: stop double-zipping extension artifact#7
twschiller merged 1 commit into
mainfrom
fix-double-zip-artifact

Conversation

@twschiller

Copy link
Copy Markdown
Contributor

Summary

  • actions/upload-artifact always wraps the uploaded path in a zip, so uploading output/extension.zip produced an extension-zip.zip containing extension.zip — users had to unzip twice before they could upload to Browserbase or load unpacked in Chrome.
  • v7 (just bumped in Bump actions/upload-artifact from 4 to 7 #6) added an archive: false option for single-file uploads that sends the file as-is. Switching to it makes the download a plain extension.zip. The name input is ignored in this mode (artifact name comes from the file), so it's removed.

Test plan

  • CI runs green on this PR
  • Download the extension.zip artifact from the run, confirm it unzips directly to manifest.json + bundle (no nested zip)
  • Spot-check the unzipped folder loads in Chrome via "Load unpacked"

🤖 Generated with Claude Code

actions/upload-artifact wraps uploaded paths in a zip, so uploading
output/extension.zip produced a double-zipped download. v7 added
archive: false for single-file uploads, which sends the file as-is.
The artifact now downloads as extension.zip — ready to upload to
Browserbase or load unpacked in Chrome.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@vercel

vercel Bot commented May 30, 2026

Copy link
Copy Markdown

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 12:27pm

Request Review

@twschiller twschiller merged commit 4a43513 into main May 30, 2026
6 checks passed
@twschiller twschiller deleted the fix-double-zip-artifact branch May 30, 2026 12:28
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