Skip to content

chore(deps): Replace jszip with @aklinker1/zero-zip - #2538

Merged
aklinker1 merged 10 commits into
mainfrom
zero-zip
Jul 31, 2026
Merged

chore(deps): Replace jszip with @aklinker1/zero-zip#2538
aklinker1 merged 10 commits into
mainfrom
zero-zip

Conversation

@aklinker1

@aklinker1 aklinker1 commented Jul 30, 2026

Copy link
Copy Markdown
Member

Overview

Apart of #2523. Reduces WXT's install footprint: 20MB / 139 packages18MB / 127 packages

That's a 10% reduction in bytes, 8.6% reduction in dependencies.

Manual Testing

  • Does chrome accept these zips?
    • Yes, on Linux at least
  • Does it work with a compression level of zero?
    • Yes, demo project is 269.24 kB in build output, zip is 273.1 kB.
  • Are the zip sizes comparable?
    • 97.65 kB on main, 97.1 kB on this PR
  • What about time it takes to build the zip?
    • 53 ms on main, 27 ms on this PR
  • Memory used? /usr/bin/time -v bun zip
    • 415912 on main, 407784 on this PR

@netlify

netlify Bot commented Jul 30, 2026

Copy link
Copy Markdown

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit 9f3e952
🔍 Latest deploy log https://app.netlify.com/projects/creative-fairy-df92c4/deploys/6a6ce338034c180008719a04
😎 Deploy Preview https://deploy-preview-2538--creative-fairy-df92c4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added pkg/storage Includes changes to the `packages/storage` directory pkg/wxt Includes changes to the `packages/wxt` directory labels Jul 30, 2026
@aklinker1 aklinker1 changed the title Zero zip chore(deps): Replace jszip with @aklinker1/zero-zip inline Jul 30, 2026
@pkg-pr-new

pkg-pr-new Bot commented Jul 30, 2026

Copy link
Copy Markdown

Open in StackBlitz

@wxt-dev/analytics

npm i https://pkg.pr.new/@wxt-dev/analytics@2538

@wxt-dev/auto-icons

npm i https://pkg.pr.new/@wxt-dev/auto-icons@2538

@wxt-dev/browser

npm i https://pkg.pr.new/@wxt-dev/browser@2538

@wxt-dev/i18n

npm i https://pkg.pr.new/@wxt-dev/i18n@2538

@wxt-dev/is-background

npm i https://pkg.pr.new/@wxt-dev/is-background@2538

@wxt-dev/module-react

npm i https://pkg.pr.new/@wxt-dev/module-react@2538

@wxt-dev/module-solid

npm i https://pkg.pr.new/@wxt-dev/module-solid@2538

@wxt-dev/module-svelte

npm i https://pkg.pr.new/@wxt-dev/module-svelte@2538

@wxt-dev/module-vue

npm i https://pkg.pr.new/@wxt-dev/module-vue@2538

@wxt-dev/runner

npm i https://pkg.pr.new/@wxt-dev/runner@2538

@wxt-dev/storage

npm i https://pkg.pr.new/@wxt-dev/storage@2538

@wxt-dev/unocss

npm i https://pkg.pr.new/@wxt-dev/unocss@2538

@wxt-dev/webextension-polyfill

npm i https://pkg.pr.new/@wxt-dev/webextension-polyfill@2538

wxt

npm i https://pkg.pr.new/wxt@2538

commit: 9f3e952

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.17%. Comparing base (d9653d7) to head (9f3e952).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2538      +/-   ##
==========================================
- Coverage   79.19%   79.17%   -0.02%     
==========================================
  Files         131      133       +2     
  Lines        3975     3981       +6     
  Branches      913      924      +11     
==========================================
+ Hits         3148     3152       +4     
  Misses        733      733              
- Partials       94       96       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aklinker1

aklinker1 commented Jul 30, 2026

Copy link
Copy Markdown
Member Author

Interestingly, we can't inline dependencies when using --unbundle mode... that's why checks are failing

@aklinker1
aklinker1 marked this pull request as ready for review July 31, 2026 17:58
@aklinker1 aklinker1 changed the title chore(deps): Replace jszip with @aklinker1/zero-zip inline chore(deps): Replace jszip with @aklinker1/zero-zip Jul 31, 2026
@aklinker1 aklinker1 removed the pkg/storage Includes changes to the `packages/storage` directory label Jul 31, 2026
@aklinker1
aklinker1 merged commit 4e5cecd into main Jul 31, 2026
18 checks passed
@aklinker1
aklinker1 deleted the zero-zip branch July 31, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg/wxt Includes changes to the `packages/wxt` directory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant