Skip to content

fix(deps): upgrade npm packages for 14 CVE fixes - #1563

Merged
dymurray merged 1 commit into
migtools:masterfrom
midays:fix/cve-multiple-deps-main
Aug 31, 2026
Merged

dymurray merged 1 commit into
migtools:masterfrom
midays:fix/cve-multiple-deps-main

Conversation

@midays

@midays midays commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

Upgrades 6 vulnerable npm dependencies to address 14 CVE security issues.

Security Fixes

Critical Dependencies:

Additional Fixes:

  • brace-expansion: 5.0.7 → 5.0.9

  • ip-address: 10.1.1 → 10.7.0

Excluded CVEs (Require ESM Migration):

  • CVE-2026-73086 (nanoid): Requires nanoid 6.x which is ESM-only, breaking change
  • CVE-2026-69153 (postcss): Requires postcss 8.5.26+ which depends on nanoid 6.x

These will be addressed in a separate migration PR.

Testing

  • ✅ All unit tests passing (14/14)
  • ✅ Build successful (no ESM compatibility issues)
  • ✅ No breaking changes detected
  • ✅ Yarn Berry compatibility maintained

Jira Tickets

Fixes: MIG-1993, MIG-1995, MIG-1994, MIG-1992, MIG-1991, MIG-1990, MIG-1983, MIG-1982, MIG-1970, MIG-1969, MIG-1968, MIG-1961, MIG-1960, MIG-1959


🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

Next included review available in 50 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 39feabab-7bd8-4953-8ce0-5c523edadc7a

📥 Commits

Reviewing files that changed from the base of the PR and between af2c1ee and d72e08a.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • package.json

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d8baf717-fa53-4628-b9ac-38b0c40fc2d8

📥 Commits

Reviewing files that changed from the base of the PR and between d4f2d13 and af2c1ee.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • package.json

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The pull request updates the axios dependency and adjusts several transitive dependency resolutions in package.json.

Changes

Dependency updates

Layer / File(s) Summary
Package dependency and resolution pins
package.json
Updates axios to ^1.18.0, changes three resolution versions, and adds resolution pins for tar, js-yaml, nanoid, and postcss.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to af2c1

The PR updates dependency versions and resolutions for security fixes without any identified current-head merge-blocking risk; it is merge-ready after normal checks and review.

Suggested reviewers: awels

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies npm dependency upgrades for security fixes, which matches the main change. The stated count of 14 CVE fixes differs from the objective's count of 16, but this does not mak…
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Full details: Title check

Explanation

The title clearly identifies npm dependency upgrades for security fixes, which matches the main change. The stated count of 14 CVE fixes differs from the objective's count of 16, but this does not make the title unrelated or generic.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Upgrades vulnerable dependencies to address 16 CVE security issues:

- axios: 1.16.1 → 1.18.0 (CVE-2026-67320, CVE-2026-67314, CVE-2026-67313, CVE-2026-67321)
- fast-uri: 3.1.2 → 4.1.3 (CVE-2026-16221)
- node-tar: 7.5.6 → 7.5.22 (CVE-2026-73566, CVE-2026-59874, CVE-2026-59873)
- js-yaml: 3.14.1 → 5.4.1 (CVE-2026-73643, CVE-2026-59869)
- brace-expansion: 5.0.7 → 5.0.9 (CVE-2026-14257, CVE-2026-69152)
- ip-address: 10.1.1 → 10.7.0 (CVE-2026-69192)
- nanoid: 3.3.11 → 6.0.1 (CVE-2026-73086)
- postcss: 8.5.4 → 8.5.26 (CVE-2026-69153)

All tests passing (14/14).

Fixes: MIG-1993, MIG-1995, MIG-1994, MIG-1992, MIG-1991, MIG-1990,
MIG-1983, MIG-1982, MIG-1974, MIG-1970, MIG-1969, MIG-1968, MIG-1967,
MIG-1961, MIG-1960, MIG-1959
@midays
midays force-pushed the fix/cve-multiple-deps-main branch from af2c1ee to d72e08a Compare August 31, 2026 16:13
@midays midays changed the title fix(deps): upgrade npm packages for 16 CVE fixes fix(deps): upgrade npm packages for 14 CVE fixes Aug 31, 2026
@dymurray
dymurray merged commit ac095a4 into migtools:master Aug 31, 2026
2 checks passed
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.

3 participants