Skip to content

Update vulnerable brace-expansion dependencies - #136

Draft
Raymond Zhao (rzhao271) with Copilot wants to merge 5 commits into
mainfrom
copilot/update-vulnerable-npm-dependencies
Draft

Raymond Zhao (rzhao271) with Copilot wants to merge 5 commits into
mainfrom
copilot/update-vulnerable-npm-dependencies

Conversation

Copilot AI commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Component Governance reported vulnerable brace-expansion versions in distinct npm dependency paths.

  • Dependency lockfile updates

    • Updated affected brace-expansion transitive entries in package-lock.json.
    • Preserved separate dependency paths without overrides, resolutions, or force flags.
  • Additional vulnerable path

    • Updated the existing 2.x brace-expansion path as well, since the current advisory also flags that installed range.

Copilot AI and others added 2 commits September 14, 2026 22:17
Co-authored-by: rzhao271 <7199958+rzhao271@users.noreply.github.com>
Co-authored-by: rzhao271 <7199958+rzhao271@users.noreply.github.com>
Copilot AI changed the title [WIP] Update vulnerable npm dependencies to latest versions Update vulnerable brace-expansion dependencies Sep 14, 2026
@rzhao271

Copy link
Copy Markdown
Collaborator

Copilot Please first revert all dependency and lockfile changes made in this PR so the branch is restored to its original dependency state. Then regenerate the update from that clean baseline while observing the repository’s 7-day npm release hold:

npm update brace-expansion --package-lock-only --min-release-age=7
npm install --min-release-age=7
npm test

There are two distinct Component Governance dependency paths, and both must remain on their respective major lines:

  • brace-expansion@1.1.16 must resolve to at least 1.1.17, using a release at least seven days old.
  • brace-expansion@5.0.7 must resolve to at least 5.0.8, using a release at least seven days old.

min-release-age is measured in days and should be applied while npm reconstructs the dependency tree. If either path has no non-vulnerable release old enough, leave that vulnerable version unchanged and report that the hold blocks remediation; do not use overrides, resolutions, --force, --legacy-peer-deps, or equivalent bypasses.

Copilot AI and others added 2 commits September 14, 2026 23:06
Co-authored-by: rzhao271 <7199958+rzhao271@users.noreply.github.com>
Co-authored-by: rzhao271 <7199958+rzhao271@users.noreply.github.com>

Copilot AI commented Sep 14, 2026

Copy link
Copy Markdown
Contributor Author

Copilot Please first revert all dependency and lockfile changes made in this PR so the branch is restored to its original dependency state. Then regenerate the update from that clean baseline while o...

Addressed in d5e5921. I restored the lockfile to the pre-PR baseline first, then regenerated with npm update brace-expansion --package-lock-only --min-release-age=7, npm install --min-release-age=7, and validated with npm test.

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.

Update vulnerable npm dependencies

2 participants