fix: remove ambiguous community wording from replacement suggestions by TheAlexLichter · Pull Request #2875 · npmx-dev/npmx.dev

TheAlexLichter · 2026-06-07T20:00:37Z

🔗 Linked issue

None (simple change)

🧭 Context

"The community" feels quite ambiguous/unclear at the moment, and two links (leading to the same main site) in a single warning draw focus away from the important part.

📚 Description

This PR slightly changes the wording, removes the "community" part, uses passive voice and makes the statement les ambiguous.

vercel · 2026-06-07T20:00:43Z

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Jun 10, 2026 10:02pm

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Jun 10, 2026 10:02pm
npmx-lunaria	Ignored		Jun 10, 2026 10:02pm

coderabbitai · 2026-06-07T20:00:49Z

📝 Walkthrough

Walkthrough

Vue components for rendering package replacement types are corrected and simplified (simple/documented branches). i18n schema adds a new learn_more_above key. Locale files are updated to remove {community} placeholders and the community key from package.replacement across languages.

Changes

Replacement messaging refactor

Layer / File(s)	Summary
i18n schema extension for learn_more_above `i18n/schema.json`	Adds `learn_more_above` as a validated string property under `package.replacement`.
Vue component refactoring `app/components/Compare/ReplacementSuggestion.vue`, `app/components/Package/Replacement.vue`	Fixes template closure in ReplacementSuggestion.vue for the `simple` branch; simplifies the `documented` branch in Replacement.vue to render `$t('package.replacement.documented')` instead of the prior `i18n-t` template containing module code and a community link.
Locale file updates for all languages `i18n/locales/*` (multiple files)	Updates `package.replacement` blocks in locale files to remove `{community}` interpolation and the `community` key, rewrites or simplifies `simple`/`documented` messages to package‑neutral wording, and retains/adjusts `learn_more` and `learn_more_above` entries.

Possibly related PRs

npmx-dev/npmx.dev#2068: Touches the same replacement UI paths (ReplacementSuggestion.vue / Replacement.vue) and i18n selection for documented/simple rendering.

Suggested reviewers

ghostdevv
serhalp
alexdln

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description adequately explains the motivation (removing ambiguous 'community' phrasing) and the approach (passive voice, reduced ambiguity), relating directly to the changeset of updated i18n translations and component rendering logic.
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 and accurately describes the main change: removing ambiguous community-related wording from replacement suggestion messages across the codebase.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch wording-warning

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

github-actions · 2026-06-07T20:01:42Z

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/az-AZ.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/bg-BG.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/bn-IN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/cs-CZ.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/de.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/en.json	Source changed, localizations will be marked as outdated.
i18n/locales/es.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/fr-FR.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/hi-IN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/hu-HU.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/id-ID.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/it-IT.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/ja-JP.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/kn-IN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/mr-IN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/nb-NO.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/ne-NP.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/nl.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/pl-PL.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/pt-BR.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/pt-PT.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/ro-RO.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/ru-RU.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/sr-Latn-RS.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/ta-IN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/te-IN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/tr-TR.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/uk-UA.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/vi-VN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/zh-CN.json	Localization changed, will be marked as complete. 🔄️
i18n/locales/zh-TW.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-07T20:02:17Z

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!

danielroe

this seems good to me, with the link doing the heavy lifting to point users to e18e

patak-cat

I also think this is a better way to word the warnings. Nice one!

gameroman

iirc, locales for other languages shouldn't be touched

WilcoSp

For Dutch "documented" can be improved.

Co-authored-by: Wilco <17604138+WilcoSp@users.noreply.github.com>

ghostdevv · 2026-06-09T20:26:50Z

iirc, locales for other languages shouldn't be touched

+1 it's better to remove translations that need changes rather than use AI 🙏

ghostdevv

~~The only other thing is that now there won't be any link to e18e on non-documented replacements~~ edit: there already isn't

…pmx-dev#2875) Co-authored-by: Wilco <17604138+WilcoSp@users.noreply.github.com> Co-authored-by: Willow (GHOST) <git@willow.sh>

fix(i18n): remove ambiguous community wording
29ed545

vercelBotdeployed to Preview – npmx.dev June 7, 2026 20:01 View deployment

gameroman added the i18n Related to internationalization label Jun 7, 2026

gameroman added the e18e Relates to incorporating, linking to, or collaborating with https://e18e.dev label Jun 7, 2026

TheAlexLichter requested a review from danielroe June 7, 2026 20:46

danielroe approved these changes Jun 7, 2026
View reviewed changes

danielroe requested review from 43081j and patak-cat June 7, 2026 20:48

gameroman self-requested a review June 7, 2026 21:10

ghostdevv added the needs review This PR is waiting for a review from a maintainer label Jun 7, 2026

patak-cat approved these changes Jun 8, 2026
View reviewed changes

gameroman requested changes Jun 8, 2026
View reviewed changes

WilcoSp suggested changes Jun 8, 2026
View reviewed changes

Comment threadi18n/locales/nl.json Outdated

43081j approved these changes Jun 9, 2026
View reviewed changes

Update i18n/locales/nl.json
5df3acf
Co-authored-by: Wilco <17604138+WilcoSp@users.noreply.github.com>

vercelBotdeployed to Preview – npmx.dev June 9, 2026 08:28 View deployment

WilcoSp approved these changes Jun 9, 2026
View reviewed changes

ghostdevv added 2 commits June 10, 2026 22:57

Merge remote-tracking branch 'origin/main' into wording-warning
de66bfc

chore: remove extra i18n changes
b392404

vercelBotdeployed to Preview – npmx.dev June 10, 2026 22:02 View deployment

ghostdevv changed the title ~~fix(i18n): remove ambiguous community wording~~fix: remove ambiguous community wording from replacement suggestionsJun 10, 2026

ghostdevv removed the needs review This PR is waiting for a review from a maintainer label Jun 10, 2026

ghostdevv requested changes Jun 10, 2026
View reviewed changes

ghostdevv approved these changes Jun 10, 2026
View reviewed changes

ghostdevv requested a review from gameroman June 10, 2026 22:09

gameroman approved these changes Jun 11, 2026
View reviewed changes

gameroman added this pull request to the merge queue Jun 11, 2026

Merged via the queue into main with commit 1c48d0cJun 11, 2026
28 checks passed

gameroman deleted the wording-warning branch June 11, 2026 08:12

github-actionsBot mentioned this pull request Jun 11, 2026
chore: release v0.16.0 #2856
Merged

Uh oh!

Uh oh!

Conversation

TheAlexLichter commented Jun 7, 2026

🔗 Linked issue

🧭 Context

📚 Description

Uh oh!

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

Uh oh!

Uh oh!

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

Uh oh!

Walkthrough

Changes

Possibly related PRs

Suggested reviewers

Uh oh!

github-actionsBot 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!

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

Uh oh!

Codecov Report

Uh oh!

danielroe left a comment

Choose a reason for hiding this comment

Uh oh!

patak-cat left a comment

Choose a reason for hiding this comment

Uh oh!

gameroman left a comment

Choose a reason for hiding this comment

Uh oh!

WilcoSp left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ghostdevv commented Jun 9, 2026

Uh oh!

ghostdevv left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

vercelBot commented Jun 7, 2026•
edited
Loading

coderabbitaiBot commented Jun 7, 2026•
edited
Loading

github-actionsBot commented Jun 7, 2026•
edited
Loading

codecovBot commented Jun 7, 2026•
edited
Loading

ghostdevv left a comment •
edited
Loading