Skip to content

chore: Remediate 1 Dependabot security alert (extract-zip via override) - #251

Merged
amanabiy merged 4 commits into
mainfrom
dependabot-remediate/alert-gap-20260821
Aug 21, 2026
Merged

chore: Remediate 1 Dependabot security alert (extract-zip via override)#251
amanabiy merged 4 commits into
mainfrom
dependabot-remediate/alert-gap-20260821

Conversation

@amanabiy

Copy link
Copy Markdown
Member

Summary

This PR addresses 1 of 4 open Dependabot security alerts in this repo (the only one fixable without a React major version bump).

This is an Outcome B gap PR (below consolidate_threshold of 5, no existing Dependabot PRs).


Alert Addressed

  • GHSA-jmr9-qjv8-65gv / CVE-2026-56876extract-zip, high, <= 2.0.1 → eliminated from tree
    Direct alert URL: https://github.com/cloudscape-design/component-toolkit/security/dependabot/217
    Fix: added overrides: { '@puppeteer/browsers': '^3.2.0' }. @wdio/utils requires @puppeteer/browsers@^2.2.0 which depends on extract-zip@2.0.1. @puppeteer/browsers 3.x replaced extract-zip with modern-tar.
    Rung 4 (overrides — no patched extract-zip exists; first_patched_version: null; rung 1/2/3 cannot fix a package with no patched release). Precedented across this org.

Alerts Left UNRESOLVED

The following 3 alerts require react-router-dom 7.x, which in turn requires react@>=18. This repo uses react@^16.14.0, making a direct bump a breaking change out of scope for a lockfile-only security fix:

These are addressed by the open PR #240 (chore/upgrade-react-router-dom by @jperals), which upgrades React alongside react-router-dom.

NPMPM Availability

@puppeteer/browsers@3.2.1 and all its sub-transitive deps are dev: true — excluded from NPMPM check per reviewer script scope. No non-dev packages changed. The reviewer's NPMPM check script should show GREEN.

Merging this PR

Merging clears 1 of 4 open alerts (the extract-zip alert). The 3 react-router alerts remain open until PR #240 is merged.

Add overrides: { '@puppeteer/browsers': '^3.2.0' } to force
@wdio/utils off @puppeteer/browsers 2.x (which requires extract-zip
2.0.1). @puppeteer/browsers 3.x replaced extract-zip with modern-tar.

No patched version of extract-zip exists (first_patched_version: null),
so elimination via @puppeteer/browsers upgrade is the only available
fix. Rung 4 (overrides) — rung 1/2/3 cannot resolve a package with no
patched release. This pattern is precedented across this org.

The react-router alerts (#197, #198, #199) are NOT addressed here
because react-router-dom 7.x requires react@>=18 while this repo uses
react@^16.14.0. Bumping both is a breaking change handled separately
by the open PR #240.

Alert addressed:
- GHSA-jmr9-qjv8-65gv / CVE-2026-56876 (extract-zip <= 2.0.1, high)
  extract-zip: 2.0.1 -> eliminated from tree
@amanabiy
amanabiy requested a review from a team as a code owner August 21, 2026 14:08
@amanabiy
amanabiy requested review from NathanZlion and removed request for a team August 21, 2026 14:08
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.09%. Comparing base (db3d274) to head (6ebf237).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #251   +/-   ##
=======================================
  Coverage   98.09%   98.09%           
=======================================
  Files          56       56           
  Lines        1787     1787           
  Branches      520      495   -25     
=======================================
  Hits         1753     1753           
- Misses         33       34    +1     
+ Partials        1        0    -1     

☔ 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.

@amanabiy
amanabiy requested review from NathanZlion and ywyyu17 and removed request for NathanZlion August 21, 2026 14:13
Add overrides: { '@puppeteer/browsers': '^3.2.0' } to force
@wdio/utils off @puppeteer/browsers 2.x (which requires extract-zip
2.0.1). @puppeteer/browsers 3.x replaced extract-zip with modern-tar.

No patched version of extract-zip exists (first_patched_version: null),
so elimination via @puppeteer/browsers upgrade is the only available
fix. Rung 4 (overrides) — rung 1/2/3 cannot resolve a package with no
patched release. This pattern is precedented across this org.

The react-router alerts (#197, #198, #199) are NOT addressed here
because react-router-dom 7.x requires react@>=18 while this repo uses
react@^16.14.0. Bumping both is a breaking change handled separately
by the open PR #240.

Alert addressed:
- GHSA-jmr9-qjv8-65gv / CVE-2026-56876 (extract-zip <= 2.0.1, high)
  extract-zip: 2.0.1 -> eliminated from tree
Python json.dump reformatted inline arrays to multi-line. Restore
original compact format for typesVersions entries.
@amanabiy
amanabiy enabled auto-merge August 21, 2026 14:19
@amanabiy
amanabiy added this pull request to the merge queue Aug 21, 2026
Merged via the queue into main with commit 452f759 Aug 21, 2026
78 of 79 checks passed
@amanabiy
amanabiy deleted the dependabot-remediate/alert-gap-20260821 branch August 21, 2026 16:27
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.

2 participants