Skip to content

fix(deps): resolve multiple high/moderate severity vulnerabilities#45

Merged
muneebs merged 3 commits intomainfrom
fix/vulnerabile_deps
Mar 31, 2026
Merged

fix(deps): resolve multiple high/moderate severity vulnerabilities#45
muneebs merged 3 commits intomainfrom
fix/vulnerabile_deps

Conversation

@muneebs
Copy link
Copy Markdown
Owner

@muneebs muneebs commented Mar 31, 2026

  • Update vitest/coverage-v8/ui to 4.1.2 (fixes flatted prototype pollution and DoS)
  • Update nuxt to 4.4.2 in examples and packages/nuxt (fixes h3, nitropack, srvx)
  • Update h3 devDep to ^1.15.9 (minimum patched version)
  • Update express example to ^5.0.0 (eliminates path-to-regexp 0.1.x entirely)
  • Add pnpm overrides: yaml >=2.8.3, node-forge >=1.4.0, picomatch 2.x/4.x, path-to-regexp 0.1.13
  • Update existing overrides: brace-expansion ^2.0.3, serialize-javascript >=7.0.5

Summary by CodeRabbit

  • Chores
    • Upgraded Express example app to major version 5.x
    • Updated Nuxt example application to version 4.4.2
    • Upgraded testing tooling (vitest) and related coverage/ui packages
    • Added and updated package overrides for improved security and maintenance

- Update vitest/coverage-v8/ui to 4.1.2 (fixes flatted prototype pollution and DoS)
- Update nuxt to 4.4.2 in examples and packages/nuxt (fixes h3, nitropack, srvx)
- Update h3 devDep to ^1.15.9 (minimum patched version)
- Update express example to ^5.0.0 (eliminates path-to-regexp 0.1.x entirely)
- Add pnpm overrides: yaml >=2.8.3, node-forge >=1.4.0, picomatch 2.x/4.x, path-to-regexp 0.1.13
- Update existing overrides: brace-expansion ^2.0.3, serialize-javascript >=7.0.5
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 31, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 60d64636-024e-4b7c-842d-0174d8fd6098

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

A set of dependency and package manifest updates: Express in examples moved to ^5.0.0, Nuxt bumped to ^4.4.2, vitest tooling bumped to ^4.1.2, pnpm.overrides expanded, and packages/express exports switched from ./dist/index.js to ./dist/index.mjs with peer range widened.

Changes

Cohort / File(s) Summary
Example App Dependencies
examples/express-app/package.json, examples/nuxt-app/package.json
Bumped example app dependencies: express ^4.22.1 → ^5.0.0; nuxt ^4.3.1 → ^4.4.2.
Root Package & Override Registry
package.json
Updated devDependencies: @vitest/coverage-v8, @vitest/ui, vitest → ^4.1.2; expanded pnpm.overrides with entries for yaml, node-forge, picomatch, path-to-regexp; adjusted brace-expansion and serialize-javascript ranges.
Express Package Export Structure
packages/express/package.json
Removed main/module fields; changed exports["."] target from ./dist/index.js to ./dist/index.mjs; broadened peerDependencies.express to `^4.0.0
Nuxt Package Development Dependencies
packages/nuxt/package.json
Bumped @nuxt/kit, @nuxt/schema, nuxt to ^4.4.2; h3 to ^1.15.9; updated vitest to ^4.1.2.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly Related PRs

Poem

"Lo, versions rise upon the hill,
Express dons five with steady will,
Nuxt and Vitest march in array,
.mjs now lights the midnight way,
Changes made — the code moves still."

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly describes the main purpose of the pull request—resolving security vulnerabilities through dependency updates—which aligns with all the changes across multiple package.json files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/vulnerabile_deps

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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@packages/express/package.json`:
- Around line 42-44: Update the package.json peerDependencies entry for
"express" to accept both v4 and v5 ranges so consumers using Express 5 won't get
conflicts; modify the "peerDependencies" key (the current "express": "^4.x"
declaration) to a widened range such as ">=4 <6" or a caret union like ">=4.0.0
<6.0.0" (or a semver-compatible union) so the adapter declares compatibility
with both Express v4 and v5.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1ff1201a-c885-468f-bd94-62c16d581f2c

📥 Commits

Reviewing files that changed from the base of the PR and between 0cf2ddc and b718c60.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (5)
  • examples/express-app/package.json
  • examples/nuxt-app/package.json
  • package.json
  • packages/express/package.json
  • packages/nuxt/package.json

Comment thread packages/express/package.json
@muneebs muneebs merged commit 9c8ce47 into main Mar 31, 2026
7 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.

1 participant