fix(compactRoutes): strip stale locale param in by-path resolver

[Togetic]

🔗 Linked issue

Fixes #4002

📚 Description

With experimental.compactRoutes, the route table is asymmetric: the default-locale route has no :locale param, while the compact route for the other locales does. There are two resolvers in runtime/utils.ts:

• the by-name resolver correctly does delete route.params.locale when retargeting to the default-locale route;
• the by-path resolver swaps route.name to the default-locale route but does not delete params.locale.

So resolving a prefixed path to the default locale from a non-default-locale URL — e.g. localeRoute(route.path, defaultLocale), common in SEO head composables that build x-default / hreflang links — hands vue-router a named location carrying a locale param the target route doesn't declare:

[Vue Router warn]: Discarded invalid param(s) "locale" when navigating.

…once per SSR request for every non-default-locale URL (default-locale URLs are silent). It's dev-only and functionally harmless — vue-router's pickParams drops the stray param, so canonical / hreflang / hrefs stay correct — but it's per-request console noise.

This is a follow-up to #3975, which fixed several compact-route router warnings but missed this code path.

Fix: mirror the existing delete route.params.locale from the by-name branch into the by-path branch, under the same __I18N_COMPACT_ROUTES__ guard.

✅ Verification

• pnpm test:types, pnpm lint, and pnpm test:unit all pass.
• Verified locally: the warning drops to 0 across non-default-locale routes, and generated canonical / hreflang are unchanged.

Summary by CodeRabbit

• Bug Fixes
  • Fixed localized routes so they no longer carry an outdated locale parameter when compact routing is enabled.

[coderabbitai]

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: 79db34cc-5160-4626-8cda-e1ec199888d4

📥 Commits

Reviewing files that changed from the base of the PR and between bd54cb9 and b0cf535.

📒 Files selected for processing (1)

• src/runtime/utils.ts

---

Walkthrough

In resolveLocalizedRouteByPath within src/runtime/utils.ts, a new conditional block removes the locale key from route.params (cast to RouteLikeWithName) when the resolved route name corresponds to an existing per-locale route and the __I18N_COMPACT_ROUTES__ flag is enabled. This prevents per-locale routes from inheriting a locale param that originates from compact-route resolution.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main fix: removing a stale locale param in the by-path resolver for compact routes.
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.

✨ 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

• X
• Mastodon
• Reddit
• LinkedIn

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