feat: Add Cloudflare Workers support + follow-up fixes by BBleae · Pull Request #4823 · anuraghazra/github-readme-stats

BBleae · 2026-02-28T16:32:10Z

Summary

include Cloudflare Workers support from feat: Add Cloudflare Workers support #4714
fix runtime HTTP behavior to use native fetch in non-test environments
normalize non-GraphQL HTTP errors into GraphQL-like errors shape
lazily load worker routes after env injection
preserve NODE_ENV=test in pat-info tests

Context

This branch contains commit 1c4a4b5 from #4714 plus follow-up commit 727d211.

I could not push to the original PR head branch (biplavbarua:master) due permissions, so this PR is opened as a replacement path for review.

Related: #4714

vercel · 2026-02-28T16:32:14Z

@BBleae 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

Adds a Cloudflare Workers deployment path for github-readme-stats while adjusting runtime HTTP behavior and error normalization so the existing API handlers can run across environments more consistently.

Changes:

Introduces a Workers entrypoint (src/worker.js) plus an Express/Vercel-style adapter (src/common/adapter.js) and Wrangler config (wrangler.toml).
Switches GraphQL HTTP requests to native fetch at runtime (keeping axios for tests) and normalizes some non-GraphQL HTTP error payloads into a GraphQL-like { errors: [...] } shape.
Updates repo fetcher error handling and adjusts tests to accommodate the new runtime behavior.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
wrangler.toml	Adds Wrangler configuration for deploying the Worker.
package.json / package-lock.json	Adds `wrangler` dev dependency and lockfile updates.
src/worker.js	Worker fetch handler with lazy route loading after env injection.
src/common/adapter.js	Adapter that bridges Request/Response to existing `(req, res)` handlers.
src/common/http.js	Uses native `fetch` outside tests; preserves axios semantics for tests.
src/common/retryer.js	Normalizes certain HTTP error payloads into GraphQL-like `errors`.
src/fetchers/repo.js	Adds explicit `errors` handling with logging + CustomError throwing.
src/cards/gist.js / src/cards/wakatime.js	Switches language colors loading approach (JSON import).
tests/pat-info.test.js	Preserves `NODE_ENV=test` so tests keep using axios path.
tests/calculateRank.test.js	Makes rank percentile assertion less brittle.

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

Copilot · 2026-02-28T16:43:27Z

+// import { createRequire } from "module";
+// const require = createRequire(import.meta.url);
+// const languageColors = require("../common/languageColors.json"); // now works
+import languageColors from "../common/languageColors.json";


import languageColors from "../common/languageColors.json"; will fail under Node ESM unless JSON import attributes/assertions are provided (or the file is converted to a JS module). This also makes the surrounding comment misleading (it describes a createRequire workaround that is now commented out). Please switch to a Node-compatible approach (e.g., JS module export or JSON import with attributes) and update/remove the outdated comment block.

Copilot · 2026-02-28T16:43:27Z

+// import { createRequire } from "module";
+// const require = createRequire(import.meta.url);
+// const languageColors = require("../common/languageColors.json"); // now works
+import languageColors from "../common/languageColors.json";


import languageColors from "../common/languageColors.json"; is not valid in Node ESM without JSON import attributes/assertions, so this can throw at runtime/tests. Also the comment block still refers to a createRequire workaround that is now commented out. Please use a Node-compatible way to load this data (e.g., convert the JSON to a JS module export, or use JSON import attributes) and align the comment with the implementation.

Copilot · 2026-02-28T16:43:27Z

+  if (res.data.errors) {
+    logger.error(res.data.errors);
+    if (res.data.errors[0].message) {
+      throw new CustomError(
+        wrapTextMultiline(res.data.errors[0].message, 90, 1)[0],
+        res.statusText,
+      );
+    }
+    throw new CustomError(
+      "Something went wrong while trying to retrieve the repo data using the GraphQL API.",
+      CustomError.GRAPHQL_ERROR,
+    );


When GraphQL returns errors, the HTTP status is typically 200/"OK". Passing res.statusText as the CustomError type will therefore set secondaryMessage to "OK", which is not a meaningful secondary message for users. Consider using a stable CustomError type (e.g., CustomError.GRAPHQL_ERROR) and/or mapping from res.data.errors[0].type when available.

Copilot · 2026-02-28T16:43:28Z

+    ).toEqual(
+      expect.objectContaining({
+        level: "B-",
+        percentile: expect.closeTo(65.02918514848255, 10),


expect.closeTo(…, 10) uses an extremely tight tolerance (10 decimal digits), which can still cause flaky tests if floating-point behavior changes slightly across Node/Jest versions. Consider using a looser precision (fewer digits) or asserting percentile with toBeCloseTo separately from the level assertion.

Suggested change

percentile: expect.closeTo(65.02918514848255, 10),

percentile: expect.closeTo(65.02918514848255, 5),

…oad worker routes - http.js: use native fetch for non-test envs to avoid axios adapter issues; add normalizeJsonPayload helper for consistent response parsing - retryer.js: normalize non-GraphQL message payloads (e.g. Bad credentials) into GraphQL-like error shape in both success and catch paths - repo.js: add missing imports and handle GraphQL errors from retryer response - worker.js: lazy-load route handlers so process.env is populated before module evaluation; replace switch with route-map lookup; guard process.env from being undefined - tests/pat-info.test.js: preserve NODE_ENV when resetting process.env in beforeAll so http.js continues to use axios mock adapter in tests

…d compat createRequire(import.meta.url) restored in the previous commit fixes Node/Vercel (bare JSON imports throw ERR_IMPORT_ATTRIBUTE_MISSING on Node >=22) but crashes Cloudflare workerd at module evaluation, where import.meta.url is undefined (GET /api/gist|wakatime -> HTTP 500). Use standard import attributes instead: works on Node >=22, Jest (experimental-vm-modules), and wrangler/esbuild, which statically inlines the JSON into the worker bundle. Bump ESLint ecmaVersion to 2025 so espree parses the attribute syntax. Verified: full test suite (27/242/9), eslint --max-warnings 0, wrangler dry-run bundle inlines JSON with no runtime require, and wrangler dev --local serves /api/gist and /api/wakatime with HTTP 200.

harryzcy · 2026-06-17T23:21:14Z

@@ -0,0 +1,8 @@
+name = "github-readme-stats"
+main = "src/worker.js"
+compatibility_date = "2024-09-23"


Use a more recent date: 2026-06-16

harryzcy · 2026-06-17T23:24:33Z

Related issue: #3433

Copilot AI review requested due to automatic review settings February 28, 2026 16:32

BBleae requested a review from qwerty541 as a code owner February 28, 2026 16:32

github-actions Bot added repo-card Issues related to the pin/repo card. wakatime-card Issues related to the wakatime card. dependencies Pull requests that update a dependency file labels Feb 28, 2026

BBleae mentioned this pull request Feb 28, 2026

feat: Add Cloudflare Workers support #4714

Open

github-actions Bot added the gist-card label Feb 28, 2026

Copilot started reviewing on behalf of BBleae February 28, 2026 16:32 View session

Copilot AI reviewed Feb 28, 2026

View reviewed changes

biplavbarua mentioned this pull request Mar 4, 2026

fix: address review comments on PR #4823 BBleae/github-readme-stats#1

Closed

biplavbarua and others added 4 commits June 5, 2026 14:04

feat: Add Cloudflare Workers support

0188dc0

fix(pr): address reviewer feedback for ESM, graphql errors, and tests

1fd5d22

BBleae force-pushed the codex/pr-4714-followup branch from 727d211 to 4d72a71 Compare June 5, 2026 05:44

harryzcy reviewed Jun 17, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Add Cloudflare Workers support + follow-up fixes#4823

feat: Add Cloudflare Workers support + follow-up fixes#4823
BBleae wants to merge 4 commits into
anuraghazra:masterfrom
BBleae:codex/pr-4714-followup

BBleae commented Feb 28, 2026

Uh oh!

vercel Bot commented Feb 28, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

Copilot AI Feb 28, 2026

Uh oh!

harryzcy Jun 17, 2026

Uh oh!

harryzcy commented Jun 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

	percentile: expect.closeTo(65.02918514848255, 10),
	percentile: expect.closeTo(65.02918514848255, 5),

Uh oh!

Conversation

BBleae commented Feb 28, 2026

Summary

Context

Uh oh!

vercel Bot commented Feb 28, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

harryzcy Jun 17, 2026

Choose a reason for hiding this comment

Uh oh!

harryzcy commented Jun 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants