feat: Add all-time contributions support with deduplicated repository count #4644

banu4prasad · 2025-11-08T11:23:04Z

Summary

Adds support for displaying all-time contributions across all years on GitHub, with automatic deduplication of repositories to show accurate unique contribution counts.

Changes

Added all_time_contribs parameter to toggle between last year and all-time contributions
Deduplicated repository counting (counts each repo only once, even with multiple contribution types)
Increased cache duration for all-time stats.
Added ALL_TIME_CONTRIBS environment variable to enable/disable feature globally

Usage

Standard (last year only)
?username=YOUR_USERNAME

All-time contributions (deduplicated)
?username=YOUR_USERNAME&all_time_contribs=true

Environment Variables

ALL_TIME_CONTRIBS=true - Enable all-time contributions feature
PAT_1=<github_token> - GitHub Personal Access Token with read:user scope

fixes #2282

vercel · 2025-11-08T11:23:07Z

@banu4prasad is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

Copilot

Pull request overview

This PR adds support for displaying all-time contribution statistics across a user's entire GitHub history, with automatic deduplication to count unique repositories only once even when a user has multiple contribution types (commits, issues, PRs, reviews) to the same repository.

Key Changes:

Added all_time_contribs query parameter to enable fetching contributions across all years instead of just the last year
Implemented parallel fetching of contribution data using Promise.all() with a 9-second timeout and graceful fallback
Extended cache duration for all-time stats (6 hours default vs 4 hours for standard stats)

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 20 comments.

Show a summary per file

File	Description
`src/fetchers/all-time-contributions.js`	New module that fetches contribution years and yearly data in parallel, deduplicating repositories across all contribution types
`src/fetchers/stats.js`	Integrated all-time contributions feature with timeout protection and fallback logic; removed some logger statements
`api/index.js`	Added parameter handling for `all_time_contribs` and custom cache logic for longer TTL
`src/common/envs.js`	Added `ALL_TIME_CONTRIBS` environment variable to enable/disable the feature globally
`src/cards/stats.js`	Added conditional label rendering to show "all time" vs "last year" based on parameter
`src/cards/types.d.ts`	Added TypeScript type definition for the new `all_time_contribs` option
`src/translations.js`	Added `statcard.contribs-alltime` translation key in 48 languages
`.gitignore`	Added `.DS_Store` to ignore macOS system files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/common/envs.js

src/fetchers/stats.js

src/cards/stats.js

src/fetchers/all-time-contributions.js

src/fetchers/stats.js

src/fetchers/all-time-contributions.js

api/index.js

src/fetchers/all-time-contributions.js

Copilot

Pull request overview

Copilot reviewed 7 out of 9 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-25T15:09:39Z

src/fetchers/all-time-contributions.js

+export const fetchAllTimeContributions = async (login) => {
+  if (!login) {
+    throw new MissingParamError(["login"]);
+  }
+
+  // Fetch all contribution years (uses retryer with token rotation)
+  const years = await fetchContributionYears(login);
+
+  // Count unique repositories across ALL years
+  const allRepos = new Set();
+
+  // Fetch all years in PARALLEL for speed (each uses retryer with token rotation)
+  const yearDataPromises = years.map(year => fetchYearContributions(login, year));
+  const yearDataResults = await Promise.all(yearDataPromises);
+
+  yearDataResults.forEach((yearData) => {
+    const addRepos = (contributions) => {
+      contributions?.forEach((contrib) => {
+        if (contrib.repository?.nameWithOwner) {
+          allRepos.add(contrib.repository.nameWithOwner);
+        }
+      });
+    };
+
+    addRepos(yearData.commitContributionsByRepository);
+    addRepos(yearData.issueContributionsByRepository);
+    addRepos(yearData.pullRequestContributionsByRepository);
+    addRepos(yearData.pullRequestReviewContributionsByRepository);
+  });
+
+  return {
+    totalRepositoriesContributedTo: allRepos.size,
+    yearsAnalyzed: years.length,
+  };
+};


The new fetchAllTimeContributions function and the entire all-time-contributions.js module lack test coverage. Given that this repository has comprehensive test coverage for other fetchers, tests should be added to verify the deduplication logic, error handling, and parallel fetching behavior.

src/common/cache.js

src/cards/stats.js

api/index.js

Copilot

Pull request overview

Copilot reviewed 8 out of 10 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

api/index.js

Copilot · 2025-12-25T15:39:06Z

src/fetchers/stats.js

+      );
+    } catch (err) {
+      // Clear timeout on error as well
+      clearTimeout(timeoutId);


The variable timeoutId may be undefined when clearTimeout(timeoutId) is called in the catch block. If an error occurs before the timeout is set (e.g., in the Promise.race setup), clearTimeout will be called with undefined. While this doesn't cause an error in JavaScript, it's better to check if timeoutId is defined before clearing it for clarity and best practices.

Copilot · 2025-12-25T15:39:06Z

api/index.js

+      hide_rank: parseBoolean(hide_rank),
+      include_all_commits: parseBoolean(include_all_commits),
+      all_time_contribs: parseBoolean(all_time_contribs),
+      commits_year: parseInt(commits_year, 10),


When commits_year is undefined or not provided, parseInt(commits_year, 10) will return NaN. This should be handled the same way as in line 100 where it's conditionally parsed: commits_year ? parseInt(commits_year, 10) : undefined. The NaN value could cause unexpected behavior in the rendering logic.

Suggested change

commits_year: parseInt(commits_year, 10),

commits_year: commits_year ? parseInt(commits_year, 10) : undefined,

Copilot · 2025-12-25T15:39:06Z

api/index.js

    const stats = await fetchStats(
      username,
      parseBoolean(include_all_commits),
+      parseBoolean(all_time_contribs),
      parseArray(exclude_repo),
      showStats.includes("prs_merged") ||
        showStats.includes("prs_merged_percentage"),
      showStats.includes("discussions_started"),
      showStats.includes("discussions_answered"),
-      parseInt(commits_year, 10),
+      commits_year ? parseInt(commits_year, 10) : undefined,
    );


The integration of the all_time_contribs parameter with the existing API endpoint and stats fetcher lacks test coverage. Consider adding tests to tests/api.test.js and tests/fetchStats.test.js that verify the parameter is correctly passed through and the feature works as expected when enabled or disabled.

Copilot · 2025-12-25T15:39:06Z

tests/fetchAllTimeContributions.test.js

@@ -0,0 +1,485 @@
+import { afterEach, beforeEach, describe, expect, it, jest } from "@jest/globals";


Unused imports beforeEach, jest.

Suggested change

import { afterEach, beforeEach, describe, expect, it, jest } from "@jest/globals";

import { afterEach, describe, expect, it } from "@jest/globals";

banu4prasad and others added 14 commits November 5, 2025 18:12

gitignore

2629b1c

[feat] all_time_contribs

72561d7

quick fix

d2418bc

[fix] allContrib

8c54693

[fix] timeout error

09ae38f

[fix] logger.log

87c9bd6

minor changes

fbd784d

changes 2

0f360d3

fixx

f0b65db

force=F

dcd8159

trying

89f64f7

changes

c1ce1f3

remove deduplicate

758310d

Merge branch 'anuraghazra:master' into master

2816400

github-actions bot added stats-card Feature, Enhancement, Fixes related to stats the stats card. card-i18n Card text translations. labels Nov 8, 2025

Merge branch 'anuraghazra:master' into master

718263c

Copilot AI review requested due to automatic review settings December 18, 2025 06:44

Copilot started reviewing on behalf of banu4prasad December 18, 2025 06:44 View session

Copilot AI reviewed Dec 18, 2025

View reviewed changes

banu4prasad and others added 2 commits December 25, 2025 20:29

changes as per the review

ed9e12a

Merge branch 'anuraghazra:master' into master

e652efa

banu4prasad requested a review from Copilot December 25, 2025 15:04

Copilot started reviewing on behalf of banu4prasad December 25, 2025 15:05 View session

Copilot AI reviewed Dec 25, 2025

View reviewed changes

banu4prasad added 4 commits December 25, 2025 20:46

removed unnecessary import

20ed7b7

minor fix

86e6a8e

cache duration

1455495

add envs

78f49cc

banu4prasad added 2 commits December 25, 2025 21:02

test file

25f78f8

changes

bf59aba

banu4prasad requested a review from Copilot December 25, 2025 15:35

Copilot started reviewing on behalf of banu4prasad December 25, 2025 15:35 View session

Copilot AI reviewed Dec 25, 2025

View reviewed changes

	commits_year: parseInt(commits_year, 10),
	commits_year: commits_year ? parseInt(commits_year, 10) : undefined,

		@@ -0,0 +1,485 @@
		import { afterEach, beforeEach, describe, expect, it, jest } from "@jest/globals";

	import { afterEach, beforeEach, describe, expect, it, jest } from "@jest/globals";
	import { afterEach, describe, expect, it } from "@jest/globals";

Uh oh!

feat: Add all-time contributions support with deduplicated repository count #4644

Are you sure you want to change the base?

feat: Add all-time contributions support with deduplicated repository count #4644

Uh oh!

Conversation

banu4prasad commented Nov 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Usage

Environment Variables

Uh oh!

vercel bot commented Nov 8, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 25, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI Dec 25, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 25, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 25, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 25, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

banu4prasad commented Nov 8, 2025 •

edited

Loading