feat: add GitHub contributors graph #1445

Tobbe · 2026-02-12T20:26:33Z

I started working on this new graph. WDYT?
I wanted to be able to see how "healthy", from a number-of-contibutors perspective a package is.
Like, even if a package has hundreds of contributors, maybe those are all from years past, and the package is now basically only kept alive by one person, which isn't too good from a reliability/bus factor point of view
It uses the GitHub API, so only works for packages that have a github repo linked in the package metadata

vercel · 2026-02-12T20:26:39Z

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Feb 13, 2026 0:21am

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Feb 13, 2026 0:21am
npmx-lunaria	Ignored		Feb 13, 2026 0:21am

github-actions · 2026-02-12T20:27:20Z

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/en-GB.json	Localization changed, will be marked as complete. 🔄️
lunaria/files/en-US.json	Source changed, localizations will be marked as outdated.

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-02-12T20:28:48Z

Codecov Report

❌ Patch coverage is 25.48077% with 155 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
app/composables/useCharts.ts	4.09%	90 Missing and 27 partials ⚠️
app/components/Package/TrendsChart.vue	56.47%	26 Missing and 11 partials ⚠️
app/pages/package/[[org]]/[name].vue	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

coderabbitai · 2026-02-12T20:35:49Z

📝 Walkthrough

Walkthrough

This PR adds per-package repository references and a new "contributors" metric to the trends chart. It introduces a repoRef prop, repoRefsByPackage state, and loadRepoRefsForPackages to fetch repo refs from package-meta. The Metric system now uses MetricContext (packageName, repoRef) and includes a contributors metric when a GitHub repo is available. Multi-package flows and granularity handling were updated to use per-package repo refs. Server and fixture support for GitHub contributors stats and a new contributors-evolution API were added, alongside client-side contributor evolution fetching and caching and localization entries for "Contributors".

Possibly related PRs

test: use fixtures for server side api fetches #828: Modifies server fixture/cache plumbing and adds GitHub contributors fixture handling that overlaps the contributors stats paths touched here.
refactor: create useSearch and extract npm logic #1236: Adds package-meta API and related types/fixtures which are used by the new loadRepoRefsForPackages logic in this PR.

Suggested reviewers

danielroe
graphieros

🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Merge Conflict Detection	⚠️ Warning	❌ Merge conflicts detected (41 files): ⚔️ `.gitignore` (content) ⚔️ `CONTRIBUTING.md` (content) ⚔️ `app/components/AppHeader.vue` (content) ⚔️ `app/components/Compare/FacetSelector.vue` (content) ⚔️ `app/components/Header/SearchBox.vue` (content) ⚔️ `app/components/Package/Card.vue` (content) ⚔️ `app/components/Package/Keywords.vue` (content) ⚔️ `app/components/Package/Maintainers.vue` (content) ⚔️ `app/components/Package/ManagerSelect.vue` (content) ⚔️ `app/components/Package/TrendsChart.vue` (content) ⚔️ `app/components/Package/VersionDistribution.vue` (content) ⚔️ `app/components/Package/Versions.vue` (content) ⚔️ `app/components/Package/WeeklyDownloadStats.vue` (content) ⚔️ `app/components/Readme.vue` (content) ⚔️ `app/composables/useCharts.ts` (content) ⚔️ `app/composables/useMarkdown.ts` (content) ⚔️ `app/composables/useStructuredFilters.ts` (content) ⚔️ `app/pages/index.vue` (content) ⚔️ `app/pages/package/[[org]]/[name].vue` (content) ⚔️ `app/pages/search.vue` (content) ⚔️ `app/utils/frameworks.ts` (content) ⚔️ `app/utils/prehydrate.ts` (content) ⚔️ `i18n/locales/en.json` (content) ⚔️ `i18n/locales/es.json` (content) ⚔️ `i18n/locales/pl-PL.json` (content) ⚔️ `i18n/locales/zh-CN.json` (content) ⚔️ `i18n/schema.json` (content) ⚔️ `lunaria/files/en-GB.json` (content) ⚔️ `lunaria/files/en-US.json` (content) ⚔️ `lunaria/files/es-419.json` (content) ⚔️ `lunaria/files/es-ES.json` (content) ⚔️ `lunaria/files/pl-PL.json` (content) ⚔️ `lunaria/files/zh-CN.json` (content) ⚔️ `modules/runtime/server/cache.ts` (content) ⚔️ `nuxt.config.ts` (content) ⚔️ `package.json` (content) ⚔️ `server/middleware/canonical-redirects.global.ts` (content) ⚔️ `shared/utils/package-analysis.ts` (content) ⚔️ `test/nuxt/a11y.spec.ts` (content) ⚔️ `test/nuxt/composables/use-markdown.spec.ts` (content) ⚔️ `vercel.json` (content) These conflicts must be resolved before merging into `main`.	Resolve conflicts locally and push changes to this branch.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description check	✅ Passed	The pull request description is directly related to the changeset, detailing the addition of a new GitHub contributors graph feature to visualise package health based on contributor activity trends.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

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

⚔️ Resolve merge conflicts (beta)

Auto-commit resolved conflicts to branch tobbe-feat-gh-contrib-graph
Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

app/components/Package/TrendsChart.vue (1)
764-766: Consider defensive check for array access.

Accessing METRICS.value[0] without a guard could technically return undefined if the array were empty. While METRICS always contains at least two metrics in practice, adding a defensive check aligns with strict type-safety guidelines.
🛡️ Optional defensive improvement
 const activeMetricDef = computed(
-  () => METRICS.value.find(m => m.id === selectedMetric.value) ?? METRICS.value[0],
+  () => METRICS.value.find(m => m.id === selectedMetric.value) ?? METRICS.value[0] ?? METRICS.value.find(() => true),
 )
Alternatively, ensure the type signature accounts for this:
 const activeMetricDef = computed(
-  () => METRICS.value.find(m => m.id === selectedMetric.value) ?? METRICS.value[0],
+  () => {
+    const found = METRICS.value.find(m => m.id === selectedMetric.value)
+    if (found) return found
+    const first = METRICS.value[0]
+    if (first) return first
+    throw new Error('No metrics available')
+  },
 )
As per coding guidelines: "ensure you always check when accessing an array value by index".

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

coderabbitai

Actionable comments posted: 3

app/components/Package/TrendsChart.vue

server/api/github/contributors-evolution/[owner]/[repo].get.ts

coderabbitai

Actionable comments posted: 1

coderabbitai · 2026-02-12T20:42:46Z

app/pages/package/[[org]]/[name].vue

+          <PackageWeeklyDownloadStats
+            :packageName
+            :createdIso="pkg?.time?.created ?? null"
+            :repoRef="repoRef"
+          />


⚠️ Potential issue | 🔴 Critical

Fix repoRef nullability for PackageWeeklyDownloadStats.

repoRef can be null from useRepoMeta, but the child prop expects RepoRef | undefined, causing the reported type check failure. Normalise null to undefined (or widen the prop type).

🛠️ Suggested fix

- :repoRef="repoRef" + :repoRef="repoRef ?? undefined"

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

<PackageWeeklyDownloadStats

:packageName

:createdIso="pkg?.time?.created ?? null"

:repoRef="repoRef"

/>

<PackageWeeklyDownloadStats

:packageName

:createdIso="pkg?.time?.created ?? null"

:repoRef="repoRef ?? undefined"

/>

🧰 Tools

🪛 GitHub Check: 💪 Type check

[failure] 1371-1371:
Type 'RepoRef | null' is not assignable to type 'RepoRef | undefined'.

graphieros · 2026-02-12T21:50:56Z

When switching to monthly or yearly, the component is designed to display estimations for incomplete periods.
It was designed originally to chart downloads. However this KPI is not compatible with estimations. The component must be adapted to show estimations for compatible facets.

The facet does not appear to be working on the compare page.

I would also move the fetching to the useCharts composable

Tobbe · 2026-02-12T21:57:14Z

When switching to monthly or yearly, the component is designed to display estimations for incomplete periods. It was designed originally to chart downloads. However this KPI is not compatible with estimations. The component must be adapted to show estimations for compatible facets.

Yes, I did find it a bit weird that it was estimating how many contributors a package will have 😀 How should we handle the incomplete period though? 🤔

graphieros · 2026-02-12T22:01:30Z

When switching to monthly or yearly, the component is designed to display estimations for incomplete periods. It was designed originally to chart downloads. However this KPI is not compatible with estimations. The component must be adapted to show estimations for compatible facets.

Yes, I did find it a bit weird that it was estimating how many contributors a package will have 😀 How should we handle the incomplete period though? 🤔

We are dealing with absolutes. n contributors today does not add up with n yesterday. So weekly, monthly, yearly should be averages.

Tobbe · 2026-02-12T22:10:28Z

The facet does not appear to be working on the compare page.

Yeah, I hadn't gotten around to fully testing that one yet.
I've made it work now (locally)

But I also need to find a package that doesn't have a GitHub repo in its package.json to test with.
You don't happen to know of one offhand, do you?

Tobbe · 2026-02-12T22:46:10Z

I faked one not having a github link, just to be able to test how it'd work.
It now prints a small notice for packages not included

Tobbe · 2026-02-12T23:18:56Z

When switching to monthly or yearly, the component is designed to display estimations for incomplete periods. It was designed originally to chart downloads. However this KPI is not compatible with estimations. The component must be adapted to show estimations for compatible facets.

Yes, I did find it a bit weird that it was estimating how many contributors a package will have 😀 How should we handle the incomplete period though? 🤔

We are dealing with absolutes. n contributors today does not add up with n yesterday. So weekly, monthly, yearly should be averages.

Likes also isn't additive, right? So maybe likes and contributors should be handled differently compared to downloads.
But I don't want to touch the logic for existing graphs in this PR. So how about we leave it like it is for all three facets for now, and then have another PR to fix this logic?

vercel bot deployed to Preview – npmx.dev February 12, 2026 20:28 View deployment

feat: add GitHub contributors graph

dc34dc7

Tobbe force-pushed the tobbe-feat-gh-contrib-graph branch from 752f464 to dc34dc7 Compare February 12, 2026 20:30

vercel bot deployed to Preview – npmx.dev February 12, 2026 20:32 View deployment

coderabbitai bot reviewed Feb 12, 2026

View reviewed changes

app/components/Package/TrendsChart.vue Show resolved Hide resolved

app/components/Package/TrendsChart.vue Show resolved Hide resolved

server/api/github/contributors-evolution/[owner]/[repo].get.ts Outdated Show resolved Hide resolved

Fix type issues

f0f4fd5

vercel bot deployed to Preview – npmx.dev February 12, 2026 20:38 View deployment

coderabbitai bot reviewed Feb 12, 2026

View reviewed changes

retry on 202

85778ac

Tobbe changed the title ~~feat: Add GitHub contributors graph~~ feat: add GitHub contributors graph Feb 12, 2026

vercel bot deployed to Preview – npmx.dev February 12, 2026 21:26 View deployment

Go back to handling both null and undefined

b11e4a0

vercel bot deployed to Preview – npmx.dev February 12, 2026 21:36 View deployment

graphieros requested a review from serhalp February 12, 2026 21:51

graphieros marked this pull request as draft February 12, 2026 22:07

Fix review comments

22ce764

vercel bot deployed to Preview – npmx.dev February 12, 2026 22:48 View deployment

Fix hydration missmatch error

ac2df1a

vercel bot deployed to Preview – npmx.dev February 13, 2026 00:21 View deployment

Uh oh!

feat: add GitHub contributors graph #1445

Are you sure you want to change the base?

feat: add GitHub contributors graph #1445

Conversation

Tobbe commented Feb 12, 2026

Uh oh!

vercel bot commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Feb 12, 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 Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

coderabbitai bot commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Possibly related PRs

Suggested reviewers

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

graphieros commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Tobbe commented Feb 12, 2026

Uh oh!

graphieros commented Feb 12, 2026

Uh oh!

Tobbe commented Feb 12, 2026

Uh oh!

Tobbe commented Feb 12, 2026

Uh oh!

Tobbe commented Feb 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel bot commented Feb 12, 2026 •

edited

Loading

github-actions bot commented Feb 12, 2026 •

edited

Loading

codecov bot commented Feb 12, 2026 •

edited

Loading

coderabbitai bot commented Feb 12, 2026 •

edited

Loading

graphieros commented Feb 12, 2026 •

edited

Loading