Skip to content

fix: use git ls-files for sources zip to respect .gitignore#2212

Open
creeperkatze wants to merge 1 commit intowxt-dev:mainfrom
creeperkatze:fix/sources-zip-gitignore
Open

fix: use git ls-files for sources zip to respect .gitignore#2212
creeperkatze wants to merge 1 commit intowxt-dev:mainfrom
creeperkatze:fix/sources-zip-gitignore

Conversation

@creeperkatze
Copy link
Copy Markdown

Overview

Sources ZIP uses tinyglobby which traverses the full directory tree before filtering, causing it to hang or balloon when gitignored directories like Rust's target/ exist locally. Switch to git ls-files --recurse-submodules for the sources ZIP since it should only contain what git tracks anyway. Falls back to the existing glob behavior if git is unavailable.

Manual Testing

  1. Have a project with large gitignored directories (e.g. a built Rust submodule with a target/ dir)
  2. Run wxt zip -b firefox
  3. Before: Produces a bloated ZIP. After: Completes quickly with only git-tracked files.

@creeperkatze creeperkatze requested a review from aklinker1 as a code owner March 25, 2026 16:15
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 25, 2026

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit ab1674b
🔍 Latest deploy log https://app.netlify.com/projects/creative-fairy-df92c4/deploys/69c40a193975f600084dbf23
😎 Deploy Preview https://deploy-preview-2212--creative-fairy-df92c4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

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