fix: set org to empty string to make sure its route param is omitted

[KazariEX]

fix#1121

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Feb 7, 2026 5:40am

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Feb 7, 2026 5:40am
npmx-lunaria	Ignored		Feb 7, 2026 5:40am

[coderabbitai]

📝 Walkthrough

Walkthrough

An explicit import type { RouteLocationRaw } was added and packageRoute now returns RouteLocationRaw. Unscoped package handling was changed so the tuple default is ['', packageName] and the org param defaults to '' in that branch. Scoped package handling keeps its split logic with the name default set to '' in destructuring. Route name literals no longer use as const and are plain strings ('package-version' and 'package').

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	The PR description 'fix #1121' is minimal but directly references the linked issue #1121, relating to fixing package links without scope on org pages.
Linked Issues check	✅ Passed	The code changes set org to empty string for unscoped packages, ensuring the org route parameter is omitted and package links resolve to /package/[name] instead of /package/[org]/[name], directly addressing issue #1121.
Out of Scope Changes check	✅ Passed	All changes in app/utils/router.ts are scoped to fixing the packageRoute function to handle unscoped packages correctly, which is directly required by issue #1121.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

app/utils/router.ts (1)

8-24: ⚠️ Potential issue | 🔴 Critical

Remove empty org parameter to avoid Vue Router warnings.

Vue Router does not omit empty string params—it treats them as invalid and produces warnings like "missing param… but received ''". The current code passes org: '' for non-scoped packages, which will cause this error. Conditionally omit org from params when it is empty:

Suggested fix

 if (version) {
return {
name: 'package-version',
params: {
- org,+ ...(org ? { org } : {}),
name,
version,
},
}
}
return {
name: 'package',
params: {
- org,+ ...(org ? { org } : {}),
name,
},
}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!