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
🔧 Implemented parallel fetching of contribution data across all years for performance
📊 Deduplicated repository counting (counts each repo only once, even with multiple contribution types)
⚡ Added timeout protection (9s) with fallback to standard last-year count
🕒 Increased cache duration for all-time stats (6 hours vs 4 hours for standard)
🌐 Added translation key statcard.contribs-alltime for proper label display
🛠️ 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

Technical Details

Uses GitHub GraphQL API to fetch contribution years and yearly data
Fetches all years in parallel using Promise.all() for speed
Counts unique repositories using Set data structure
falls back to standard count on timeout or error
Requires PAT_1 environment variable with GitHub token in vercel's env variables

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.

Copilot · 2025-12-18T06:59:30Z

src/common/envs.js

  ? process.env.GIST_WHITELIST.split(",")
  : undefined;

+const ALL_TIME_CONTRIBS=process.env.ALL_TIME_CONTRIBS == "true";


Use strict equality (===) instead of loose equality (==) for comparing strings. Loose equality can lead to unexpected type coercion. Change to process.env.ALL_TIME_CONTRIBS === "true".

Suggested change

const ALL_TIME_CONTRIBS=process.env.ALL_TIME_CONTRIBS == "true";

const ALL_TIME_CONTRIBS=process.env.ALL_TIME_CONTRIBS === "true";

Copilot · 2025-12-18T06:59:30Z

src/fetchers/stats.js

 const totalCommitsFetcher = async (username) => {
  if (!githubUsernameRegex.test(username)) {
-    logger.log("Invalid username provided.");
    throw new Error("Invalid username provided.");


Removed logger statements provide useful debugging information when errors occur. While the error is still thrown, removing the logging makes it harder to debug issues in production. Consider keeping these logger calls for operational visibility, especially since they were previously present for error tracking purposes.

Copilot · 2025-12-18T06:59:30Z

src/cards/stats.js

+    label: all_time_contribs
+    ? i18n.t("statcard.contribs-alltime")


Missing space after all_time_contribs in the ternary expression. The code should be formatted as label: all_time_contribs ? with proper spacing for consistency with the project's code style.

Suggested change

label: all_time_contribs

? i18n.t("statcard.contribs-alltime")

label: all_time_contribs ?

i18n.t("statcard.contribs-alltime")

Copilot · 2025-12-18T06:59:31Z

src/fetchers/all-time-contributions.js

+  const res = await retryer(fetcher, { login });
+
+  if (res.data.errors) {
+    throw new Error("Failed to fetch contribution years");


Generic error message "Failed to fetch contribution years" doesn't include the underlying error details. This makes debugging difficult when API calls fail. Consider including the original error message or logging it for better error diagnostics.

Suggested change

throw new Error("Failed to fetch contribution years");

const errorDetails = Array.isArray(res.data.errors)

? res.data.errors

.map((e) => e && e.message ? e.message : JSON.stringify(e))

.join("; ")

: JSON.stringify(res.data.errors);

logger.error(

`Failed to fetch contribution years for login '${login}': ${errorDetails}`,

);

throw new Error(`Failed to fetch contribution years: ${errorDetails}`);

Copilot · 2025-12-18T06:59:31Z

src/fetchers/stats.js

+
+      const allTimePromise = fetchAllTimeContributions(
+        username,
+        process.env.PAT_1,
+      );
+


Hardcoding the use of process.env.PAT_1 makes this feature inflexible and potentially insecure. If PAT_1 is not set or expires, the feature silently falls back without any indication. Consider either: 1) using the token rotation mechanism that the rest of the codebase uses, or 2) accepting a token parameter from the caller to maintain consistency with other fetcher functions.

Suggested change

const allTimePromise = fetchAllTimeContributions(

username,

process.env.PAT_1,

);

const tokenEnvName =

process.env.ALL_TIME_CONTRIB_TOKEN_NAME || "PAT_1";

const token = process.env[tokenEnvName];

const allTimePromise = fetchAllTimeContributions(

username,

token,

);

Copilot · 2025-12-18T06:59:33Z

src/fetchers/all-time-contributions.js

+ */
+const fetchYearContributions = async (login, year, token) => {
+  const from = `${year}-01-01T00:00:00Z`;
+  const to = `${year}-12-31T23:59:59Z`;


The end timestamp uses 23:59:59Z which could miss contributions made in the last second of the year (between 23:59:59 and 24:00:00). Consider using the next year's start time (e.g., ${year + 1}-01-01T00:00:00Z) as the end boundary to ensure all contributions for the year are captured inclusively.

Suggested change

const to = `${year}-12-31T23:59:59Z`;

const to = `${year + 1}-01-01T00:00:00Z`;

Copilot · 2025-12-18T06:59:33Z

api/index.js

+    const cacheSeconds = parseBoolean(all_time_contribs)
+      ? clampValue(
+          parseInt(cache_seconds || SIX_HOURS, 10),
+          SIX_HOURS,
+          ONE_DAY,
+        )
+      : clampValue(
+          parseInt(cache_seconds || FOUR_HOURS, 10),
+          FOUR_HOURS,
+          ONE_DAY,
+        );


The cache configuration for all-time contributions (min: 6 hours, max: 24 hours) is less restrictive than the standard STATS_CARD configuration (min: 12 hours, max: 48 hours). Since all-time data changes even more slowly than last-year data, the cache duration should be longer, not shorter. Consider using higher minimum values (e.g., 12-24 hours minimum) to reduce unnecessary API load.

Copilot · 2025-12-18T06:59:33Z

api/index.js

+    const cacheSeconds = parseBoolean(all_time_contribs)
+      ? clampValue(
+          parseInt(cache_seconds || SIX_HOURS, 10),
+          SIX_HOURS,
+          ONE_DAY,
+        )
+      : clampValue(
+          parseInt(cache_seconds || FOUR_HOURS, 10),
+          FOUR_HOURS,
+          ONE_DAY,
+        );
+


The custom cache logic using clampValue doesn't respect the CACHE_SECONDS environment variable, unlike the removed resolveCacheSeconds function which checks for this override. This breaks existing functionality where administrators can globally override cache durations via environment variables. Consider using resolveCacheSeconds with appropriate parameters or replicating its env var check logic.

Suggested change

const cacheSeconds = parseBoolean(all_time_contribs)

? clampValue(

parseInt(cache_seconds || SIX_HOURS, 10),

SIX_HOURS,

ONE_DAY,

)

: clampValue(

parseInt(cache_seconds || FOUR_HOURS, 10),

FOUR_HOURS,

ONE_DAY,

);

const isAllTimeContribs = parseBoolean(all_time_contribs);

const minCacheSeconds = isAllTimeContribs ? SIX_HOURS : FOUR_HOURS;

// Baseline cache value from request parameter (or default), clamped

const requestedCacheSeconds = parseInt(

cache_seconds || (isAllTimeContribs ? SIX_HOURS : FOUR_HOURS),

10,

);

let cacheSeconds = clampValue(

requestedCacheSeconds,

minCacheSeconds,

ONE_DAY,

);

// Global override via CACHE_SECONDS environment variable, if set

const envCacheSecondsRaw = process.env.CACHE_SECONDS;

if (envCacheSecondsRaw !== undefined) {

const envCacheSeconds = parseInt(envCacheSecondsRaw, 10);

if (Number.isFinite(envCacheSeconds) && envCacheSeconds > 0) {

cacheSeconds = clampValue(envCacheSeconds, minCacheSeconds, ONE_DAY);

}

}

Copilot · 2025-12-18T06:59:34Z

src/fetchers/all-time-contributions.js

+// @ts-check
+
+import { retryer } from "../common/retryer.js";
+import { MissingParamError, CustomError } from "../common/error.js";


Unused import CustomError.

Suggested change

import { MissingParamError, CustomError } from "../common/error.js";

import { MissingParamError } from "../common/error.js";

Copilot · 2025-12-18T06:59:34Z

src/fetchers/all-time-contributions.js

+import { retryer } from "../common/retryer.js";
+import { MissingParamError, CustomError } from "../common/error.js";
+import { request } from "../common/http.js";
+import { logger } from "../common/log.js";


Unused import logger.

Suggested change

import { logger } from "../common/log.js";

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

	const ALL_TIME_CONTRIBS=process.env.ALL_TIME_CONTRIBS == "true";
	const ALL_TIME_CONTRIBS=process.env.ALL_TIME_CONTRIBS === "true";

		label: all_time_contribs
		? i18n.t("statcard.contribs-alltime")

-    throw new Error("Failed to fetch contribution years");
+    const errorDetails = Array.isArray(res.data.errors)
+      ? res.data.errors
+          .map((e) => e && e.message ? e.message : JSON.stringify(e))
+          .join("; ")
+      : JSON.stringify(res.data.errors);
+    logger.error(
+      `Failed to fetch contribution years for login '${login}': ${errorDetails}`,
+    );
+    throw new Error(`Failed to fetch contribution years: ${errorDetails}`);

	const to = `${year}-12-31T23:59:59Z`;
	const to = `${year + 1}-01-01T00:00:00Z`;

-    const cacheSeconds = parseBoolean(all_time_contribs)
-      ? clampValue(
-          parseInt(cache_seconds || SIX_HOURS, 10),
-          SIX_HOURS,
-          ONE_DAY,
-        )
-      : clampValue(
-          parseInt(cache_seconds || FOUR_HOURS, 10),
-          FOUR_HOURS,
-          ONE_DAY,
-        );
+    const isAllTimeContribs = parseBoolean(all_time_contribs);
+    const minCacheSeconds = isAllTimeContribs ? SIX_HOURS : FOUR_HOURS;
+    // Baseline cache value from request parameter (or default), clamped
+    const requestedCacheSeconds = parseInt(
+      cache_seconds || (isAllTimeContribs ? SIX_HOURS : FOUR_HOURS),
+,
+    );
+    let cacheSeconds = clampValue(
+      requestedCacheSeconds,
+      minCacheSeconds,
+      ONE_DAY,
+    );
+    // Global override via CACHE_SECONDS environment variable, if set
+    const envCacheSecondsRaw = process.env.CACHE_SECONDS;
+    if (envCacheSecondsRaw !== undefined) {
+      const envCacheSeconds = parseInt(envCacheSecondsRaw, 10);
+      if (Number.isFinite(envCacheSeconds) && envCacheSeconds > 0) {
+        cacheSeconds = clampValue(envCacheSeconds, minCacheSeconds, ONE_DAY);
+      }
+    }

	import { MissingParamError, CustomError } from "../common/error.js";
	import { MissingParamError } from "../common/error.js";

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

Conversation

banu4prasad commented Nov 8, 2025

Summary

Changes

Usage

Technical Details

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!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant