fix: add changelog shortcut to the keyboard shortcuts menu by BittuBarnwal7479 · Pull Request #2873 · npmx-dev/npmx.dev

BittuBarnwal7479 · 2026-06-07T18:55:17Z

🔗 Linked issue

🧭 Context

The - shortcut for opening the changelog tab on package is shown in the UI but missing from the Keyboard Shortcuts search section.

📚 Description

The package page shows a - keyboard shortcut for opening the changelog, but this shortcut is not listed on the Keyboard shortcuts search section.

…languages

vercel · 2026-06-07T18:55:22Z

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

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ready	Preview, Comment	Jul 4, 2026 3:09am
npmx.dev	Ready	Preview, Comment	Jul 4, 2026 3:09am

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
npmx-lunaria	Ignored		Jul 4, 2026 3:09am

coderabbitai · 2026-06-07T18:55:29Z

📝 Walkthrough

Walkthrough

This PR adds a new keyboard shortcut entry for opening the changelog. The AppFooter.vue shortcuts modal gains a - key row, the i18n schema defines a new shortcuts.section.changelog property, and translation strings are added in Arabic, English, Hindi, and Dutch locale files.

Changes

Changelog keyboard shortcut

Layer / File(s)	Summary
i18n schema definition `i18n/schema.json`	Adds a `changelog` string property under `shortcuts.section` in the schema.
AppFooter UI component `app/components/AppFooter.vue`	Adds a new `<li>` entry rendering `-` as the key and `$t('shortcuts.changelog')` as the label in the shortcuts modal.
Locale translations `i18n/locales/ar.json`, `i18n/locales/en.json`, `i18n/locales/hi-IN.json`, `i18n/locales/nl.json`	Adds the changelog shortcut translation string in each locale file.

Possibly related PRs

npmx-dev/npmx.dev#2757: Both PRs touch i18n keyboard-shortcut/changelog-related translation strings.
npmx-dev/npmx.dev#2866: Both PRs update keyboard-shortcut UI text for the same changelog shortcut mapping.
npmx-dev/npmx.dev#2946: Both PRs extend i18n changelog keyboard-shortcut translation keys.

Suggested reviewers: ghostdevv

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
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.
Title check	✅ Passed	The title clearly matches the main change: adding the changelog shortcut to the keyboard shortcuts menu.
Description check	✅ Passed	The description is directly related to the change and explains the missing changelog shortcut in the UI.

✨ 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.}

github-actions · 2026-06-07T18:56:18Z

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
i18n/locales/ar.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/en.json	Source changed, localizations will be marked as outdated.
i18n/locales/hi-IN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/nl.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.

codecov · 2026-06-07T18:56:34Z

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!

ghostdevv

I assume you used AI to do the translations? Usually we prefer it if you only translate the languages you're familiar with as it's easier for people to add missing translations and fix broken ones (plus gives others a good way to contribute). Otherwise LGTM

BittuBarnwal7479 · 2026-06-07T22:47:27Z

I assume you used AI to do the translations? Usually we prefer it if you only translate the languages you're familiar with as it's easier for people to add missing translations and fix broken ones (plus gives others a good way to contribute). Otherwise LGTM

Yes, I used AI to help with the translations. The change was very small, just replacing a few English text.

I agree that native speakers are best suited for translation work, and I appreciate you pointing it out. I'll keep that in mind for future translation contributions.
Thanks!

WilcoSp

Looking good, but like what ghostdev said about the translations, at least in Dutch I localise it differently

WilcoSp · 2026-06-08T19:09:02Z

    "settings": "Open instellingen",
    "compare": "Open vergelijken",
    "compare_from_package": "Vergelijking openen (vooraf ingevuld met het huidige pakket)",
+    "changelog": "Open changelog",


For Dutch this should be "Open wijzigingenoverzicht"

Also for languages you don't know I would suggest leaving them empty instead of "{open} changelog" as it can be that some languages will localise it differently compared to translating directly.

Thanks!
I agree, I'll keep that in mind for future translation contributions.

…utch locale

WilcoSp

at least for Dutch looking good now, further reviewing I'll leave up to ghostdevv.

I'll also share this pr in the i18n channel in discord, maybe some from there will look at the translations

…context for new packages

coderabbitai

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@i18n/locales/cs-CZ.json`:
- Line 39: The cs-CZ translation for the "changelog" key is misleading ("Otevřít
změny nového balíčku"); update the value for the "changelog" entry in
i18n/locales/cs-CZ.json (key: "changelog") to a neutral phrase that reflects
opening the changelog for the current package, e.g., "Otevřít changelog" or
"Otevřít změny balíčku", ensuring it no longer implies the package is new.

In `@i18n/locales/zh-TW.json`:
- Line 39: The "changelog" translation value currently reads "開啟變更日誌（針對新套件）",
which implies the shortcut is only for new packages; update the value for the
"changelog" key to remove the misleading qualifier and use "開啟變更日誌" so it
correctly reflects opening the changelog for the current package; locate the
"changelog" key in i18n/locales/zh-TW.json and replace its value accordingly.

🪄 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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e56b936e-6688-486f-ac4c-b92f6f747ff2

📥 Commits

Reviewing files that changed from the base of the PR and between 5ff06e6 and 710e3a6.

📒 Files selected for processing (9)

i18n/locales/cs-CZ.json
i18n/locales/hu-HU.json
i18n/locales/it-IT.json
i18n/locales/ja-JP.json
i18n/locales/pl-PL.json
i18n/locales/ta-IN.json
i18n/locales/tr-TR.json
i18n/locales/zh-CN.json
i18n/locales/zh-TW.json

✅ Files skipped from review due to trivial changes (5)

i18n/locales/zh-CN.json
i18n/locales/hu-HU.json
i18n/locales/pl-PL.json
i18n/locales/tr-TR.json
i18n/locales/ja-JP.json

🚧 Files skipped from review as they are similar to previous changes (2)

i18n/locales/ta-IN.json
i18n/locales/it-IT.json

gameroman

I'd recommend to not adding translations for the languages you are not familiar with and wait until someone who does adds them later

… familiar)

BittuBarnwal7479 · 2026-06-08T19:53:57Z

I'd recommend to not adding translations for the languages you are not familiar with and wait until someone who does adds them later

I've removed the unverified translations and kept the change focused on the original fix.

feat: add changelog shortcut in search help and localize in multiple …

48582ac

…languages

vercel Bot deployed to Preview – npmx.dev June 7, 2026 18:57 View deployment

ghostdevv requested changes Jun 7, 2026

View reviewed changes

ghostdevv changed the title ~~feat: add Missing "Open changelog" shortcut in Keyboard Shortcuts search section~~ fix: add missing changelog entry in keyboard shortcuts menu Jun 7, 2026

WilcoSp suggested changes Jun 8, 2026

View reviewed changes

fix(i18n): update changelog label to "Open wijzigingenoverzicht" in D…

5ff06e6

…utch locale

vercel Bot deployed to Preview – npmx.dev June 8, 2026 19:20 View deployment

WilcoSp approved these changes Jun 8, 2026

View reviewed changes

fix(i18n): update changelog labels for multiple languages to clarify …

710e3a6

…context for new packages

coderabbitai Bot reviewed Jun 8, 2026

View reviewed changes

Comment thread i18n/locales/cs-CZ.json Outdated

Comment thread i18n/locales/zh-TW.json Outdated

gameroman requested changes Jun 8, 2026

View reviewed changes

fix(i18n): remove changelog label from multiple language locales (not…

a248e80

… familiar)

vercel Bot deployed to Preview – npmx.dev June 8, 2026 19:53 View deployment

ghostdevv approved these changes Jun 16, 2026

View reviewed changes

ghostdevv requested a review from gameroman June 16, 2026 23:53

gameroman approved these changes Jun 17, 2026

View reviewed changes

Merge branch 'main' into search-help/add-changelog-shortcut

d3a4dcd

ghostdevv changed the title ~~fix: add missing changelog entry in keyboard shortcuts menu~~ fix: add changelog shortcut to the keyboard shortcuts menu Jul 4, 2026

ghostdevv enabled auto-merge July 4, 2026 03:07

vercel Bot deployed to Preview – docs.npmx.dev July 4, 2026 03:08 View deployment

vercel Bot deployed to Preview – npmx.dev July 4, 2026 03:09 View deployment

ghostdevv added this pull request to the merge queue Jul 4, 2026

Merged via the queue into npmx-dev:main with commit fb67ebb Jul 4, 2026
25 checks passed

github-actions Bot mentioned this pull request Jul 4, 2026

chore: release v0.17.0 #2992

Open

Uh oh!

Uh oh!

Conversation

BittuBarnwal7479 commented Jun 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Linked issue

🧭 Context

📚 Description

Uh oh!

vercel Bot commented Jun 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented Jun 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

github-actions Bot commented Jun 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Lunaria Status Overview

Tracked Files

Uh oh!

codecov Bot commented Jun 7, 2026

Codecov Report

Uh oh!

ghostdevv left a comment

Choose a reason for hiding this comment

Uh oh!

BittuBarnwal7479 commented Jun 7, 2026

Uh oh!

WilcoSp left a comment

Choose a reason for hiding this comment

Uh oh!

WilcoSp Jun 8, 2026

Choose a reason for hiding this comment

Uh oh!

BittuBarnwal7479 Jun 8, 2026

Choose a reason for hiding this comment

Uh oh!

WilcoSp left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

gameroman left a comment

Choose a reason for hiding this comment

Uh oh!

BittuBarnwal7479 commented Jun 8, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

BittuBarnwal7479 commented Jun 7, 2026 •

edited

Loading

vercel Bot commented Jun 7, 2026 •

edited

Loading

coderabbitai Bot commented Jun 7, 2026 •

edited

Loading

github-actions Bot commented Jun 7, 2026 •

edited

Loading