Skip to content

feat: dependencies page#686

Closed
stordahl wants to merge 7 commits intonpmx-dev:mainfrom
stordahl:main
Closed

feat: dependencies page#686
stordahl wants to merge 7 commits intonpmx-dev:mainfrom
stordahl:main

Conversation

@stordahl
Copy link
Copy Markdown

@stordahl stordahl commented Feb 1, 2026

Closes #29 by making the existing "Dependencies" header in the sidebar clickable, and adding a /deps route structured similar to /code.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 1, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 1, 2026 11:35pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 1, 2026 11:35pm
npmx-lunaria Ignored Ignored Feb 1, 2026 11:35pm

Request Review

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 1, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
lunaria/files/ar-EG.json Localization changed, will be marked as complete. 🔄️
lunaria/files/cs-CZ.json Localization changed, will be marked as complete. 🔄️
lunaria/files/de-DE.json Localization changed, will be marked as complete. 🔄️
lunaria/files/en-US.json Source changed, localizations will be marked as outdated.
lunaria/files/es-419.json Localization changed, will be marked as complete. 🔄️
lunaria/files/es-ES.json Localization changed, will be marked as complete. 🔄️
lunaria/files/fr-FR.json Localization changed, will be marked as complete. 🔄️
lunaria/files/hu-HU.json Localization changed, will be marked as complete. 🔄️
lunaria/files/id-ID.json Localization changed, will be marked as complete. 🔄️
lunaria/files/it-IT.json Localization changed, will be marked as complete. 🔄️
lunaria/files/ja-JP.json Localization changed, will be marked as complete. 🔄️
lunaria/files/ne-NP.json Localization changed, will be marked as complete. 🔄️
lunaria/files/pl-PL.json Localization changed, will be marked as complete. 🔄️
lunaria/files/pt-BR.json Localization changed, will be marked as complete. 🔄️
lunaria/files/ru-RU.json Localization changed, will be marked as complete. 🔄️
lunaria/files/uk-UA.json Localization changed, will be marked as complete. 🔄️
lunaria/files/zh-CN.json Localization changed, will be marked as complete. 🔄️
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@mikouaji
Copy link
Copy Markdown
Contributor

mikouaji commented Feb 1, 2026

Could you move it to /package-deps/{path} so it will go nicely with #674 ?

@Molnes
Copy link
Copy Markdown

Molnes commented Feb 2, 2026

Looking at the vercel builds for this, and between the two last builds it fails to properly get some of the information?
Deployment before f8fe55f:
https://npmx-5i5hdxeyz-poetry.vercel.app/is-even
image

And after f8fe55f:
https://npmx-3b1seggis-poetry.vercel.app/is-even
image
The info does show up if i explicitly select the version https://npmx-3b1seggis-poetry.vercel.app/is-even/v/1.0.0
I don't see why that commit would stop it from showing up tho. Or might just be me misunderstanding ¯\(ツ)

Comment on lines +94 to +99
const canonicalUrl = computed(() => {
if (packageName.value && version.value) {
return `https://npmx.dev/package-deps/${packageName.value}/v/${version.value}`
}
return 'https://npmx.dev/package-deps'
})
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we could resolve the route here instead of hardcoding it.

@serhalp serhalp added the blocked label Feb 6, 2026
@serhalp serhalp added the stale This has become stale and may be closed soon label Mar 15, 2026
@github-actions github-actions Bot removed the stale This has become stale and may be closed soon label Apr 18, 2026
@github-actions
Copy link
Copy Markdown

Thank you for your contribution! 🙏

Unfortunately, this PR has been marked as stale for 14 days without any updates and we're now closing it to keep our repository tidy.

If you'd like to continue working on this, feel free to:

  • Reopen this PR
  • Leave a comment explaining your progress

We appreciate your interest in npmx and hope you'll continue contributing! 💙

@github-actions github-actions Bot added the stale This has become stale and may be closed soon label Apr 18, 2026
@github-actions github-actions Bot closed this Apr 18, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 18, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)
  • release

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0daf5791-d032-4914-a06b-168ed522fcf2

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked stale This has become stale and may be closed soon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add package dependencies tab with grouped view

5 participants