Skip to content

chore(deps): upgrade to vue-router v5#3941

Open
dargmuesli wants to merge 1 commit intonuxt-modules:mainfrom
dargmuesli:chore/deps/vue-router
Open

chore(deps): upgrade to vue-router v5#3941
dargmuesli wants to merge 1 commit intonuxt-modules:mainfrom
dargmuesli:chore/deps/vue-router

Conversation

@dargmuesli
Copy link
Collaborator

@dargmuesli dargmuesli commented Feb 20, 2026

🔗 Linked issue

#3942

📚 Description

Upgrade to vue-router v5, see nuxt/nuxt#34181.

Summary by CodeRabbit

  • Dependency Updates

    • Upgraded Vue Router from v4.6.4 to v5.0.3
    • Removed unplugin-vue-router dependency; functionality integrated into Vue Router 5
  • Chores

    • Updated build and routing setup to align with Vue Router 5
    • Normalized route alias handling for more consistent behavior
  • Documentation

    • Updated internal comments to reference Vue Router 5

@dargmuesli dargmuesli force-pushed the chore/deps/vue-router branch from ca21b0c to 3f4a2fc Compare March 12, 2026 18:14
@dargmuesli dargmuesli marked this pull request as ready for review March 12, 2026 18:14
@dargmuesli dargmuesli requested a review from BobbieGoede as a code owner March 12, 2026 18:14
@dargmuesli
Copy link
Collaborator Author

@BobbieGoede Nuxt 4.2 now uses vue-router v5 and for typed pages to work with Nuxt i18n, the changes proposed here need a release I think

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 855568c6-691c-456a-b792-79a17b1358dc

📥 Commits

Reviewing files that changed from the base of the PR and between 3f4a2fc and ce3f825.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (4)
  • build.config.ts
  • package.json
  • src/pages.ts
  • src/transform/macros.ts
💤 Files with no reviewable changes (1)
  • build.config.ts
🚧 Files skipped from review as they are similar to previous changes (3)
  • src/transform/macros.ts
  • package.json
  • src/pages.ts

Walkthrough

Removed the unplugin-vue-router dependency and switched imports to vue-router/unplugin in source files. Updated package versions (including Vue Router to 5.0.3) and removed externals entries for the old plugin in build.config.ts. Normalized runtime alias handling by coercing single alias values to arrays when constructing routes. Minor documentation comments updated to reference the new plugin source. No exported/public API signatures were changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: upgrading vue-router from v4.6.4 to v5.0.3 across the codebase and related dependencies.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

You can generate walkthrough in a markdown collapsible section to save space.

Enable the reviews.collapse_walkthrough setting to generate walkthrough in a markdown collapsible section.

@dargmuesli dargmuesli force-pushed the chore/deps/vue-router branch from 3f4a2fc to ce3f825 Compare March 12, 2026 18:26
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