fix(deps): replace extract-zip - #473
Draft
kmbroai wants to merge 1 commit into
Draft
Conversation
mldangelo-oai
approved these changes
Aug 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replace the unpatched
extract-zip@2.0.1dependency affected by GHSA-jmr9-qjv8-65gv. The existing call site already rejected symlink entries beforeextract-zipcould create them, but removing the package clears the flagged dependency and avoids future exposure to its unsafe default behavior.Changes
yauzl@3.4.0directly for lazy ZIP parsing and streaming extraction__MACOSXfilteringTesting
pnpm install --frozen-lockfile— passedpnpm run types— passedpnpm run format— passedpnpm audit --prod --audit-level high— passed with no known vulnerabilitiespnpm packand installed-tarballcheck:package— passedThe full local runtime test file reached 97 passed and 7 skipped; 24 unrelated tests require a trusted temporary-directory ancestry that this sandbox does not provide. All ZIP extraction tests passed, and GitHub Actions will run the complete Linux, macOS, and Windows matrix.
Risk and rollout
The behavior change is limited to plugin ZIP extraction. The replacement keeps the existing fail-closed archive policy and uses the parser that
extract-zippreviously wrapped, upgraded to its current release. No migration is required. Dependabot alerts should close after this lands and GitHub refreshes the dependency graph.Public disclosure review