Add Hall of Fame by h3xxit · Pull Request #39 · universal-tool-calling-protocol/utcp-specification

h3xxit · 2025-09-03T15:18:47Z

Summary by cubic

Adds a Hall of Fame page that ranks contributors across the org by recent activity and updates automatically. Includes a GitHub Action, data-fetch script, UI, and docs, with links in the navbar and About page.

New Features
- New Hall of Fame at /hall-of-fame with a leaderboard UI and badges.
- Ranks by a simplified recent-activity score (weighted commits in the last 6 months) across all org repos.
- Data fetched via scripts/fetch-contributors.js → src/data/contributors.json, with optional manual entries in src/data/contributors-manual.json.
- GitHub Action runs daily at 6 AM UTC (and on demand) to refresh data and auto-commit if changed.
- Added nav item and About page link; new docs page (hall-of-fame-docs.md).
- package.json adds fetch-contributors and build:fast scripts; added types and JSON module declarations.
Migration
- No changes required. For local refresh: npm run fetch-contributors (set GITHUB_TOKEN to avoid rate limits).

cubic-dev-ai

24 issues found across 12 files

_{React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.}

cubic-dev-ai · 2025-09-03T15:36:45Z

+    // Get recent commits (last 6 months) for recency calculation
+    const sixMonthsAgo = new Date();
+    sixMonthsAgo.setMonth(sixMonthsAgo.getMonth() - 6);
+    const commits = await githubApi(`/repos/${ORG_NAME}/${repoName}/commits?author=${username}&since=${sixMonthsAgo.toISOString()}&per_page=100`);


Commit activity is not paginated; counts and lastActivity can be truncated for users with >100 recent commits.

Prompt for AI agents

Address the following comment on scripts/fetch-contributors.js at line 61: <comment>Commit activity is not paginated; counts and lastActivity can be truncated for users with >100 recent commits.</comment> <file context> @@ -0,0 +1,340 @@ + // Get recent commits (last 6 months) for recency calculation + const sixMonthsAgo = new Date(); + sixMonthsAgo.setMonth(sixMonthsAgo.getMonth() - 6); + const commits = await githubApi(`/repos/${ORG_NAME}/${repoName}/commits?author=${username}&since=${sixMonthsAgo.toISOString()}&per_page=100`); + + // Get PR reviews by this user </file context>

cubic-dev-ai · 2025-09-03T15:36:45Z

+
+const fetchContributorsForRepo = async (repoName) => {
+  try {
+    const contributors = await githubApi(`/repos/${ORG_NAME}/${repoName}/contributors?per_page=100`);


Contributors list is not paginated; results are limited to 100 per repository, causing missing contributors.

Prompt for AI agents

Address the following comment on scripts/fetch-contributors.js at line 42: <comment>Contributors list is not paginated; results are limited to 100 per repository, causing missing contributors.</comment> <file context> @@ -0,0 +1,340 @@ + +const fetchContributorsForRepo = async (repoName) => { + try { + const contributors = await githubApi(`/repos/${ORG_NAME}/${repoName}/contributors?per_page=100`); + return contributors.map(contributor => ({ + ...contributor, </file context>

cubic-dev-ai · 2025-09-03T15:36:47Z

@@ -0,0 +1,65 @@
+name: Update Contributors Data


Missing explicit permissions; declare permissions: contents: write to ensure push can succeed and follow least-privilege.

Prompt for AI agents

Address the following comment on .github/workflows/update-contributors.yml at line 1: <comment>Missing explicit permissions; declare permissions: contents: write to ensure push can succeed and follow least-privilege.</comment> <file context> @@ -0,0 +1,65 @@ +name: Update Contributors Data + +on: </file context>

cubic-dev-ai · 2025-09-03T15:36:47Z

+  contributors: GitHubContributor[];
+}
+
+export interface DisplayContributor {


Inconsistent naming (snake_case and camelCase) within DisplayContributor increases cognitive load and mapping errors; use a consistent convention per interface (preferably snake_case for API-shaped data or camelCase for UI models) and rename fields accordingly.

Prompt for AI agents

Address the following comment on src/types/contributors.ts at line 39: <comment>Inconsistent naming (snake_case and camelCase) within DisplayContributor increases cognitive load and mapping errors; use a consistent convention per interface (preferably snake_case for API-shaped data or camelCase for UI models) and rename fields accordingly.</comment> <file context> @@ -0,0 +1,60 @@ + contributors: GitHubContributor[]; +} + +export interface DisplayContributor { + id: number; + name: string; </file context>

cubic-dev-ai · 2025-09-03T15:36:47Z

+export interface DisplayContributor {
+  id: number;
+  name: string;
+  username: string;


Having both username and githubUsername is redundant and ambiguous; consolidate to a single, clearly named field aligned with the source data (e.g., login) to avoid confusion and mapping errors.

Prompt for AI agents

Address the following comment on src/types/contributors.ts at line 42: <comment>Having both username and githubUsername is redundant and ambiguous; consolidate to a single, clearly named field aligned with the source data (e.g., login) to avoid confusion and mapping errors.</comment> <file context> @@ -0,0 +1,60 @@ +export interface DisplayContributor { + id: number; + name: string; + username: string; + githubUsername: string; + role: string; </file context>

cubic-dev-ai · 2025-09-03T15:36:47Z

+  total_contributions: number;
+  total_impact_score: number;
+  total_recent_activity: number;
+  scoring_method: string;


Narrow scoring_method to the known literal value to improve type safety and prevent accidental misspellings.

Prompt for AI agents

Address the following comment on src/types/contributors.ts at line 35: <comment>Narrow scoring_method to the known literal value to improve type safety and prevent accidental misspellings.</comment> <file context> @@ -0,0 +1,60 @@ + total_contributions: number; + total_impact_score: number; + total_recent_activity: number; + scoring_method: string; + contributors: GitHubContributor[]; +} </file context>

cubic-dev-ai · 2025-09-03T15:36:48Z

+  username: string;
+  githubUsername: string;
+  role: string;
+  status: string;


Constrain status to a union of documented values to improve type safety and prevent invalid states.

Prompt for AI agents

Address the following comment on src/types/contributors.ts at line 45: <comment>Constrain status to a union of documented values to improve type safety and prevent invalid states.</comment> <file context> @@ -0,0 +1,60 @@ + username: string; + githubUsername: string; + role: string; + status: string; + contributions: number; + impact_score: number; </file context>

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

* made lines of code instead * Update scripts/fetch-contributors.js Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> * Fix 0 id --------- Co-authored-by: Razvan Radulescu <43811028+h3xxit@users.noreply.github.com> Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

cubic-dev-ai · 2025-09-04T11:03:28Z

@cubic-dev-ai review

@h3xxit I've started the AI code review. It'll take a few minutes to complete.

cubic-dev-ai · 2025-09-04T11:04:06Z

@cubic-dev-ai review

@h3xxit I've started the AI code review. It'll take a few minutes to complete.

cubic-dev-ai

8 issues found across 12 files

_{React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.}

cubic-dev-ai · 2025-09-04T11:20:42Z

+  }
+
+  .linesChangedLabel {
+    font-size: 0.4rem;


Font size is too small for readability on mobile; increase to at least ~0.75rem for legibility.

Prompt for AI agents

Address the following comment on src/pages/hall-of-fame.module.css at line 844: <comment>Font size is too small for readability on mobile; increase to at least ~0.75rem for legibility.</comment> <file context> @@ -0,0 +1,846 @@ + } + + .linesChangedLabel { + font-size: 0.4rem; + } +} </file context>

cubic-dev-ai · 2025-09-04T11:20:42Z

+
+.contributorStatus {
+  font-size: 0.75rem;
+  color: #666666;


Text color is low-contrast against the dark background; use a lighter gray to improve readability.

Prompt for AI agents

Address the following comment on src/pages/hall-of-fame.module.css at line 343: <comment>Text color is low-contrast against the dark background; use a lighter gray to improve readability.</comment> <file context> @@ -0,0 +1,846 @@ + +.contributorStatus { + font-size: 0.75rem; + color: #666666; + margin: 0; + font-style: italic; </file context>

cubic-dev-ai · 2025-09-04T11:20:43Z

+   - **Hybrid impact scoring** combining recent activity, overall contributions, code quality, and multi-project involvement
+   - Status determination based on impact scores and activity recency
+   - Real GitHub profile images with emoji fallbacks for errors
+   - Badge assignment for notable contributors based on multiple metrics


Claims badges are assigned based on multiple metrics, but only a single impact-score-based badge is implemented.

Prompt for AI agents

Address the following comment on hall-of-fame-docs.md at line 20: <comment>Claims badges are assigned based on multiple metrics, but only a single impact-score-based badge is implemented.</comment> <file context> @@ -0,0 +1,185 @@ + - **Hybrid impact scoring** combining recent activity, overall contributions, code quality, and multi-project involvement + - Status determination based on impact scores and activity recency + - Real GitHub profile images with emoji fallbacks for errors + - Badge assignment for notable contributors based on multiple metrics + +3. **Automatic Updates**: GitHub Actions automatically updates the contributors data: </file context>

cubic-dev-ai · 2025-09-04T11:20:43Z

+  id: number;
+  name: string;
+  username: string;
+  githubUsername: string;


Duplicate fields username and githubUsername store the same data; consolidate to a single property to avoid drift and simplify usage.

Prompt for AI agents

Address the following comment on src/types/contributors.ts at line 54: <comment>Duplicate fields username and githubUsername store the same data; consolidate to a single property to avoid drift and simplify usage.</comment> <file context> @@ -0,0 +1,86 @@ + id: number; + name: string; + username: string; + githubUsername: string; + role: string; + status: string; </file context>

cubic-dev-ai · 2025-09-04T11:20:43Z

    "docusaurus": "docusaurus",
    "start": "docusaurus start",
    "build": "docusaurus build",
+    "build:fast": "docusaurus build",


Script name suggests a faster build but runs the same command as build, which can mislead contributors and CI.

Prompt for AI agents

Address the following comment on package.json at line 9: <comment>Script name suggests a faster build but runs the same command as build, which can mislead contributors and CI.</comment> <file context> @@ -6,13 +6,15 @@ "docusaurus": "docusaurus", "start": "docusaurus start", "build": "docusaurus build", + "build:fast": "docusaurus build", "swizzle": "docusaurus swizzle", "deploy": "docusaurus deploy", </file context>

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

h3xxit and others added 5 commits September 2, 2025 15:17

Add Langchain adapter maintainters

580e6ee

hall of fame v1

cbd1b2b

Merge branch 'hall-of-fame' into dev

9adbf2d

hall-of-fame v2

7e46f3e

Update about-us.md

c37ce14

cubic-dev-ai Bot reviewed Sep 3, 2025

View reviewed changes

h3xxit and others added 2 commits September 3, 2025 17:40

Update about/about-us.md

7c7cb8f

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

cubic-dev-ai Bot reviewed Sep 4, 2025

View reviewed changes

h3xxit and others added 3 commits September 4, 2025 13:52

Update hall-of-fame-docs.md

0d5a7ce

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

Update package.json

c1439e7

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

Update hall-of-fame-docs.md

5b27dff

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

h3xxit merged commit f56cf36 into main Sep 4, 2025
1 check passed

Uh oh!

Conversation

h3xxit commented Sep 3, 2025 • edited by cubic-dev-ai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by cubic

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cubic-dev-ai Bot Sep 3, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 3, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 3, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 3, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 3, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 3, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 3, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot commented Sep 4, 2025

Uh oh!

cubic-dev-ai Bot commented Sep 4, 2025

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

cubic-dev-ai Bot Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cubic-dev-ai Bot Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

h3xxit commented Sep 3, 2025 •

edited by cubic-dev-ai Bot

Loading