From 8b8c6b9309f21ab881f7710f0dd54e7469d7f0c2 Mon Sep 17 00:00:00 2001
From: richardkeep <r.kipsang@gmail.com>
Date: Tue, 20 Oct 2026 17:22:56 +0300
Subject: [PATCH 1/2] Load About page contributors dynamically from GitHub

---
 app/Http/Controllers/AboutController.php      |  5 +-
 app/Queries/RepositoryContributorsQuery.php   | 51 +++++++++++++++++++
 resources/js/lib/about.ts                     | 50 ------------------
 resources/js/pages/about/show.tsx             | 22 ++++++--
 .../Controllers/AboutControllerTest.php       | 14 +++++
 .../RepositoryContributorsQueryTest.php       | 45 ++++++++++++++++
 6 files changed, 130 insertions(+), 57 deletions(-)
 create mode 100644 app/Queries/RepositoryContributorsQuery.php
 create mode 100644 tests/Feature/RepositoryContributorsQueryTest.php

diff --git a/app/Http/Controllers/AboutController.php b/app/Http/Controllers/AboutController.php
index 98b461175..083cc3333 100644
--- a/app/Http/Controllers/AboutController.php
+++ b/app/Http/Controllers/AboutController.php
@@ -4,15 +4,16 @@
 
 namespace App\Http\Controllers;
 
+use App\Queries\RepositoryContributorsQuery;
 use App\Services\PageMetadata;
 use Inertia\Inertia;
 use Inertia\Response;
 
 final readonly class AboutController
 {
-    public function show(PageMetadata $metadata): Response
+    public function show(PageMetadata $metadata, RepositoryContributorsQuery $contributors): Response
     {
-        return Inertia::render('about/show')->withViewData(['preview' => $metadata->page(
+        return Inertia::render('about/show', ['contributors' => $contributors->get(...)])->withViewData(['preview' => $metadata->page(
             'About - /dev/null',
             'How /dev/null was built in the open: from the first commit on stream to the developers who joined and the people who shipped it.',
             route('about.show'),
diff --git a/app/Queries/RepositoryContributorsQuery.php b/app/Queries/RepositoryContributorsQuery.php
new file mode 100644
index 000000000..909ab990f
--- /dev/null
+++ b/app/Queries/RepositoryContributorsQuery.php
@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Queries;
+
+use App\Services\GitHub;
+use Illuminate\Http\Client\HttpClientException;
+use Illuminate\Support\Collection;
+use Illuminate\Support\Facades\Cache;
+use Illuminate\Support\Fluent;
+
+final readonly class RepositoryContributorsQuery
+{
+    public function __construct(private GitHub $github)
+    {
+        //
+    }
+
+    /**
+     * GitHub counts contributions on the default branch (production). Cache across all visitors for one hour.
+     *
+     * @return Collection<int, array{login: string, commits: int}>
+     */
+    public function get(): Collection
+    {
+        $repository = config()->string('services.github.repository');
+
+        return Cache::remember("repository-contributors:{$repository}", 3600, function () use ($repository): Collection {
+            $contributors = collect();
+            try {
+                for ($page = 1; ; $page++) {
+                    $response = $this->github->request()->get("repos/{$repository}/contributors", ['per_page' => 100, 'page' => $page])->throw();
+                    $accounts = $response->collect();
+                    $contributors = $contributors->concat($accounts->where('type', 'User')->mapInto(Fluent::class)->map(fn (Fluent $account): array => [
+                        'login' => $account->string('login')->value(),
+                        'commits' => $account->integer('contributions'),
+                    ]));
+
+                    if ($accounts->count() < 100) {
+                        break;
+                    }
+                }
+            } catch (HttpClientException) {
+                return collect();
+            }
+
+            return $contributors->sortByDesc('commits')->values();
+        });
+    }
+}
diff --git a/resources/js/lib/about.ts b/resources/js/lib/about.ts
index c6bbc3621..faaeb6e56 100644
--- a/resources/js/lib/about.ts
+++ b/resources/js/lib/about.ts
@@ -181,53 +181,3 @@ export const storyAfter: StoryEvent[] = [
         },
     },
 ];
-
-/**
- * Everyone with a commit on production, most commits first. Bots and agents are left out.
- */
-export const storyContributors: StoryContributor[] = [
-    { login: 'xHeaven', commits: 199 },
-    { login: 'nunomaduro', commits: 176 },
-    { login: 'mnapoli', commits: 61 },
-    { login: 'brendt', commits: 42 },
-    { login: 'simonhamp', commits: 18 },
-    { login: 'paulinevos', commits: 16 },
-    { login: 'jubianchi', commits: 16 },
-    { login: 'MrPunyapal', commits: 15 },
-    { login: 'sadiqk2', commits: 8 },
-    { login: 'camilleislasse', commits: 6 },
-    { login: 'hbenabdallah', commits: 5 },
-    { login: 'vtlxx', commits: 5 },
-    { login: 'robelsust', commits: 4 },
-    { login: 'HelgeSverre', commits: 4 },
-    { login: 'SanderMuller', commits: 3 },
-    { login: 'MaximeGoncalves', commits: 3 },
-    { login: 'mathiasonea', commits: 3 },
-    { login: 'fernandofreamunde', commits: 3 },
-    { login: 'mon4ssi', commits: 2 },
-    { login: 'hbenjacquet', commits: 2 },
-    { login: 'Akiyo974', commits: 2 },
-    { login: 'Chemaclass', commits: 2 },
-    { login: 'eznix86', commits: 2 },
-    { login: 'BENNIEV-ZA', commits: 2 },
-    { login: 'ahosker', commits: 2 },
-    { login: 'eramitgupta', commits: 1 },
-    { login: 'utsavsomaiya', commits: 1 },
-    { login: 'tooinfinity', commits: 1 },
-    { login: 'murrant', commits: 1 },
-    { login: 'RobinMalfait', commits: 1 },
-    { login: 'prateekbhujel', commits: 1 },
-    { login: 'ondrejmirtes', commits: 1 },
-    { login: 'lazarospsa', commits: 1 },
-    { login: 'kachelle', commits: 1 },
-    { login: 'kadirtikil', commits: 1 },
-    { login: 'jjaybrown', commits: 1 },
-    { login: 'GTS-MEGHNI', commits: 1 },
-    { login: 'faissaloux', commits: 1 },
-    { login: 'fabpl', commits: 1 },
-    { login: 'danielh-official', commits: 1 },
-    { login: 'chrisgocode', commits: 1 },
-    { login: 'AmanuelLegese', commits: 1 },
-    { login: 'alaminfirdows', commits: 1 },
-    { login: 'aaqidmasoodi', commits: 1 },
-];
diff --git a/resources/js/pages/about/show.tsx b/resources/js/pages/about/show.tsx
index 409804079..5d38a2949 100644
--- a/resources/js/pages/about/show.tsx
+++ b/resources/js/pages/about/show.tsx
@@ -16,6 +16,7 @@ import {
 import AppLayout from '@/layouts/app-layout';
 import type {
     StoryCommit,
+    StoryContributor,
     StoryEvent,
     StoryEventKind,
     StoryStream,
@@ -24,7 +25,6 @@ import {
     storyAfter,
     storyAfterLane,
     storyBefore,
-    storyContributors,
     storyFounders,
     storyStreams,
 } from '@/lib/about';
@@ -264,10 +264,18 @@ function Stream({ stream }: { stream: StoryStream }) {
     );
 }
 
-function Contributors() {
+function Contributors({ contributors }: { contributors: StoryContributor[] }) {
+    if (contributors.length === 0) {
+        return (
+            <p className="text-sm text-muted-foreground">
+                The contributor list is temporarily unavailable. Please check
+                back later.
+            </p>
+        );
+    }
     return (
         <ul className="grid grid-cols-[repeat(auto-fill,minmax(11rem,1fr))] gap-2">
-            {storyContributors.map((contributor) => (
+            {contributors.map((contributor) => (
                 <li key={contributor.login}>
                     <a
                         href={`https://github.com/${contributor.login}`}
@@ -301,7 +309,11 @@ function Contributors() {
     );
 }
 
-export default function Show() {
+export default function Show({
+    contributors,
+}: {
+    contributors: StoryContributor[];
+}) {
     const { repository } = usePage().props;
     return (
         <>
@@ -392,7 +404,7 @@ export default function Show() {
 
                 <DocumentSection id="contributors" title="Contributors">
                     <p>Everyone who has a commit on production. Thank you.</p>
-                    <Contributors />
+                    <Contributors contributors={contributors} />
                     <p className="text-muted-foreground">
                         Want to be on this list?{' '}
                         <a
diff --git a/tests/Feature/Controllers/AboutControllerTest.php b/tests/Feature/Controllers/AboutControllerTest.php
index f507a7170..3d2be8371 100644
--- a/tests/Feature/Controllers/AboutControllerTest.php
+++ b/tests/Feature/Controllers/AboutControllerTest.php
@@ -3,6 +3,13 @@
 declare(strict_types=1);
 
 use App\Models\User;
+use Illuminate\Support\Facades\Http;
+
+beforeEach(function (): void {
+    Http::fake(['api.github.com/repos/*/contributors*' => Http::response([
+        ['login' => 'contributor', 'type' => 'User', 'contributions' => 12],
+    ])]);
+});
 
 it('renders the about page for guests', function (): void {
     $this->get(route('about.show'))
@@ -24,3 +31,10 @@
         ->assertSee('<link rel="canonical" href="'.route('about.show').'">', false)
         ->assertSee('"@type":"WebPage"', false);
 });
+
+it('passes GitHub contributors to the about page', function (): void {
+    $this->get(route('about.show'))->assertInertia(fn ($page) => $page
+        ->has('contributors', 1)
+        ->where('contributors.0.login', 'contributor')
+        ->where('contributors.0.commits', 12));
+});
diff --git a/tests/Feature/RepositoryContributorsQueryTest.php b/tests/Feature/RepositoryContributorsQueryTest.php
new file mode 100644
index 000000000..7f383574a
--- /dev/null
+++ b/tests/Feature/RepositoryContributorsQueryTest.php
@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+use App\Queries\RepositoryContributorsQuery;
+use Illuminate\Http\Client\Request;
+use Illuminate\Support\Facades\Http;
+
+it('paginates contributors, excludes bots, and sorts by commit count', function (): void {
+    Http::fake(['api.github.com/repos/*/contributors*' => fn (Request $request) => Http::response($request['page'] === 1
+        ? array_fill(0, 100, ['login' => 'automation[bot]', 'type' => 'Bot', 'contributions' => 1000])
+        : [
+            ['login' => 'second', 'type' => 'User', 'contributions' => 2],
+            ['login' => 'first', 'type' => 'User', 'contributions' => 10],
+        ])]);
+
+    expect(resolve(RepositoryContributorsQuery::class)->get()->all())->toBe([
+        ['login' => 'first', 'commits' => 10],
+        ['login' => 'second', 'commits' => 2],
+    ]);
+    Http::assertSentCount(2);
+});
+
+it('shares the cache between visitors and refreshes after one hour', function (): void {
+    Http::fake(['api.github.com/repos/*/contributors*' => Http::sequence()
+        ->push([['login' => 'first', 'type' => 'User', 'contributions' => 1]])
+        ->push([['login' => 'first', 'type' => 'User', 'contributions' => 2]])]);
+    $query = resolve(RepositoryContributorsQuery::class);
+
+    expect($query->get()->first()['commits'])->toBe(1);
+    $this->travel(59)->minutes();
+    expect($query->get()->first()['commits'])->toBe(1);
+    Http::assertSentCount(1);
+    $this->travel(2)->minutes();
+    expect($query->get()->first()['commits'])->toBe(2);
+    Http::assertSentCount(2);
+});
+
+it('keeps the page available during a GitHub outage and caches the failed attempt', function (): void {
+    Http::fake(['api.github.com/repos/*/contributors*' => Http::failedConnection()]);
+    $query = resolve(RepositoryContributorsQuery::class);
+
+    expect($query->get())->toBeEmpty()->and($query->get())->toBeEmpty();
+    Http::assertSentCount(1);
+});

From a6ce0c33f0edd4a00c74239a4cc35e94556a22ea Mon Sep 17 00:00:00 2001
From: richardkeep <r.kipsang@gmail.com>
Date: Tue, 20 Oct 2026 17:27:09 +0300
Subject: [PATCH 2/2] fix types

---
 app/Queries/RepositoryContributorsQuery.php | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/app/Queries/RepositoryContributorsQuery.php b/app/Queries/RepositoryContributorsQuery.php
index 909ab990f..f53a3dd1a 100644
--- a/app/Queries/RepositoryContributorsQuery.php
+++ b/app/Queries/RepositoryContributorsQuery.php
@@ -26,7 +26,7 @@ public function get(): Collection
     {
         $repository = config()->string('services.github.repository');
 
-        return Cache::remember("repository-contributors:{$repository}", 3600, function () use ($repository): Collection {
+        $result = Cache::remember("repository-contributors:{$repository}", 3600, function () use ($repository): array {
             $contributors = collect();
             try {
                 for ($page = 1; ; $page++) {
@@ -42,10 +42,12 @@ public function get(): Collection
                     }
                 }
             } catch (HttpClientException) {
-                return collect();
+                return [];
             }
 
-            return $contributors->sortByDesc('commits')->values();
+            return $contributors->sortByDesc('commits')->values()->all();
         });
+
+        return collect($result);
     }
 }