@rmdes/indiekit-endpoint-github 1.2.4 → 1.2.6

package/lib/controllers/activity.js

@@ -109,16 +109,16 @@ export const activityController = {
           activity = utils.extractRepoActivity(events, username);
         }
       } catch (apiError) {
-        return response
-          .status(apiError.status || 500)
-          .json({ error: apiError.message });
+        console.error("[Activity] API error:", apiError.message);
+        return response.json({ activity: [], error: "GitHub API temporarily unavailable" });
       }
 
       activity = activity.slice(0, limits.activity);
 
       response.json({ activity });
     } catch (error) {
-      next(error);
+      console.error("[Activity] Controller error:", error.message);
+      response.json({ activity: [], error: "GitHub API temporarily unavailable" });
     }
   },
 };

package/lib/controllers/changelog.js

@@ -206,7 +206,16 @@ export const changelogController = {
         generatedAt: new Date().toISOString(),
       });
     } catch (error) {
-      next(error);
+      console.error("[Changelog] API error:", error.message);
+      response.json({
+        commits: [],
+        categories: {},
+        commitCategories: {},
+        totalCommits: 0,
+        days: daysValue || 30,
+        generatedAt: new Date().toISOString(),
+        error: "GitHub API temporarily unavailable",
+      });
     }
   },
 };

package/lib/controllers/stars.js

@@ -69,16 +69,16 @@ export const starsController = {
       try {
         starred = await client.getUserStarred(username, limits.stars);
       } catch (apiError) {
-        return response
-          .status(apiError.status || 500)
-          .json({ error: apiError.message });
+        console.error("[Stars] API error:", apiError.message);
+        return response.json({ stars: [], error: "GitHub API temporarily unavailable" });
       }
 
       const stars = utils.formatStarred(starred);
 
       response.json({ stars });
     } catch (error) {
-      next(error);
+      console.error("[Stars] Controller error:", error.message);
+      response.json({ stars: [], error: "GitHub API temporarily unavailable" });
     }
   },
 };

package/lib/github-client.js

@@ -22,7 +22,7 @@ export class GitHubClient {
   async fetch(endpoint) {
     const url = `${BASE_URL}${endpoint}`;
 
-    // Check cache first
+    // Check cache first — return fresh data immediately
     const cached = this.cache.get(url);
     if (cached && Date.now() - cached.timestamp < this.cacheTtl) {
       return cached.data;
@@ -37,28 +37,40 @@ export class GitHubClient {
       headers.Authorization = `Bearer ${this.token}`;
     }
 
-    const response = await fetch(url, { headers });
-
-    if (!response.ok) {
-      // Only use fromFetch for JSON error responses; GitHub sometimes returns
-      // HTML error pages (e.g., 502 Bad Gateway) which cause SyntaxError noise
-      const contentType = response.headers.get("content-type") || "";
-      if (contentType.includes("json")) {
-        throw await IndiekitError.fromFetch(response);
+    try {
+      const response = await fetch(url, { headers });
+
+      if (!response.ok) {
+        // Only use fromFetch for JSON error responses; GitHub sometimes returns
+        // HTML error pages (e.g., 502 Bad Gateway) which cause SyntaxError noise
+        const contentType = response.headers.get("content-type") || "";
+        if (contentType.includes("json")) {
+          throw await IndiekitError.fromFetch(response);
+        }
+
+        throw new IndiekitError(response.statusText, {
+          status: response.status,
+          code: response.statusText,
+        });
       }
 
-      throw new IndiekitError(response.statusText, {
-        status: response.status,
-        code: response.statusText,
-      });
-    }
+      const data = await response.json();
 
-    const data = await response.json();
+      // Cache result
+      this.cache.set(url, { data, timestamp: Date.now() });
 
-    // Cache result
-    this.cache.set(url, { data, timestamp: Date.now() });
+      return data;
+    } catch (error) {
+      // Stale-while-error: if we have stale cached data, return it
+      if (cached) {
+        console.warn(
+          `[GitHub] API error for ${endpoint}: ${error.message}. Serving stale cache (age: ${Math.round((Date.now() - cached.timestamp) / 60_000)}min)`,
+        );
+        return cached.data;
+      }
 
-    return data;
+      throw error;
+    }
   }
 
   /**

package/lib/starred-cache.js

@@ -86,21 +86,20 @@ export async function annotateWithLists(db, lists) {
     }
   }
 
-  // Single ordered bulkWrite: reset all, then set per-repo lists
-  const ops = [
-    { updateMany: { filter: {}, update: { $set: { lists: [] } } } },
-    ...[...repoToLists.entries()].map(([fullName, slugs]) => ({
-      updateOne: {
-        filter: { fullName },
-        update: { $set: { lists: slugs } },
-      },
-    })),
-  ];
+  // Reset all repos' lists first, then annotate per-repo
+  await collection.updateMany({}, { $set: { lists: [] } });
+
+  const annotateOps = [...repoToLists.entries()].map(([fullName, slugs]) => ({
+    updateOne: {
+      filter: { fullName },
+      update: { $set: { lists: slugs } },
+    },
+  }));
 
   // Process in batches of 1000 to avoid MongoDB limits
   let reposAnnotated = 0;
-  for (let i = 0; i < ops.length; i += 1000) {
-    const batch = ops.slice(i, i + 1000);
+  for (let i = 0; i < annotateOps.length; i += 1000) {
+    const batch = annotateOps.slice(i, i + 1000);
     const result = await collection.bulkWrite(batch, { ordered: true });
     reposAnnotated += result.modifiedCount;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rmdes/indiekit-endpoint-github",
-  "version": "1.2.4",
+  "version": "1.2.6",
   "description": "GitHub activity endpoint for Indiekit. Display commits, stars, contributions, and featured repositories.",
   "keywords": [
     "indiekit",