@oss-autopilot/core 0.42.5 → 0.42.6

package/dist/cli.bundle.cjs

@@ -10847,6 +10847,9 @@ var init_display_utils = __esm({
 });
 
 // src/core/github-stats.ts
+function emptyPRCountsResult() {
+  return { repos: /* @__PURE__ */ new Map(), monthlyCounts: {}, monthlyOpenedCounts: {}, dailyActivityCounts: {} };
+}
 function isCachedPRCounts(v) {
   if (typeof v !== "object" || v === null) return false;
   const obj = v;
@@ -10854,7 +10857,7 @@ function isCachedPRCounts(v) {
 }
 async function fetchUserPRCounts(octokit, githubUsername, query, label, accumulateRepo) {
   if (!githubUsername) {
-    return { repos: /* @__PURE__ */ new Map(), monthlyCounts: {}, monthlyOpenedCounts: {}, dailyActivityCounts: {} };
+    return emptyPRCountsResult();
   }
   const cache = getHttpCache();
   const cacheKey = `pr-counts:${label}:${githubUsername}`;
@@ -10875,6 +10878,7 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
   const dailyActivityCounts = {};
   let page = 1;
   let fetched = 0;
+  let totalCount;
   while (true) {
     const { data } = await octokit.search.issuesAndPullRequests({
       q: `is:pr ${query} author:${githubUsername}`,
@@ -10883,6 +10887,7 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
       per_page: 100,
       page
     });
+    totalCount = data.total_count;
     for (const item of data.items) {
       const parsed = extractOwnerRepo(item.html_url);
       if (!parsed) {
@@ -10907,11 +10912,17 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
       }
     }
     fetched += data.items.length;
-    if (fetched >= data.total_count || fetched >= 1e3 || data.items.length === 0) {
+    if (fetched >= data.total_count || fetched >= 1e3 || data.items.length === 0 || page >= MAX_PAGINATION_PAGES) {
       break;
     }
     page++;
   }
+  if (fetched < totalCount && page >= MAX_PAGINATION_PAGES) {
+    warn(
+      MODULE6,
+      `Pagination capped at ${MAX_PAGINATION_PAGES} pages: fetched ${fetched} of ${totalCount} ${label} PRs. Stats may be incomplete for prolific contributors.`
+    );
+  }
   debug(MODULE6, `Found ${fetched} ${label} PRs across ${repos.size} repos`);
   cache.set(cacheKey, "", {
     reposEntries: Array.from(repos.entries()),
@@ -11020,7 +11031,7 @@ async function fetchRecentlyMergedPRs(octokit, config, days = 7) {
     }
   );
 }
-var MODULE6, PR_COUNTS_CACHE_TTL_MS;
+var MODULE6, PR_COUNTS_CACHE_TTL_MS, MAX_PAGINATION_PAGES;
 var init_github_stats = __esm({
   "src/core/github-stats.ts"() {
     "use strict";
@@ -11028,7 +11039,8 @@ var init_github_stats = __esm({
     init_logger();
     init_http_cache();
     MODULE6 = "github-stats";
-    PR_COUNTS_CACHE_TTL_MS = 60 * 60 * 1e3;
+    PR_COUNTS_CACHE_TTL_MS = 24 * 60 * 60 * 1e3;
+    MAX_PAGINATION_PAGES = 3;
   }
 });
 
@@ -13476,8 +13488,14 @@ async function fetchPRData(prMonitor, token) {
   const issueMonitor = new IssueConversationMonitor(token);
   const [mergedResult, closedResult, recentlyClosedPRs, recentlyMergedPRs, issueConversationResult] = await Promise.all(
     [
-      prMonitor.fetchUserMergedPRCounts(),
-      prMonitor.fetchUserClosedPRCounts(),
+      prMonitor.fetchUserMergedPRCounts().catch((err) => {
+        console.error(`Warning: Failed to fetch merged PR counts: ${errorMessage(err)}`);
+        return emptyPRCountsResult();
+      }),
+      prMonitor.fetchUserClosedPRCounts().catch((err) => {
+        console.error(`Warning: Failed to fetch closed PR counts: ${errorMessage(err)}`);
+        return emptyPRCountsResult();
+      }),
       prMonitor.fetchRecentlyClosedPRs().catch((err) => {
         console.error(`Warning: Failed to fetch recently closed PRs: ${errorMessage(err)}`);
         return [];
@@ -13626,12 +13644,16 @@ async function updateRepoScores(prMonitor, prs, mergedCounts, closedCounts) {
 function updateAnalytics(prs, monthlyCounts, monthlyClosedCounts, openedFromMerged, openedFromClosed) {
   const stateManager2 = getStateManager();
   try {
-    stateManager2.setMonthlyMergedCounts(monthlyCounts);
+    if (Object.keys(monthlyCounts).length > 0) {
+      stateManager2.setMonthlyMergedCounts(monthlyCounts);
+    }
   } catch (error) {
     console.error("[DAILY] Failed to store monthly merged counts:", errorMessage(error));
   }
   try {
-    stateManager2.setMonthlyClosedCounts(monthlyClosedCounts);
+    if (Object.keys(monthlyClosedCounts).length > 0) {
+      stateManager2.setMonthlyClosedCounts(monthlyClosedCounts);
+    }
   } catch (error) {
     console.error("[DAILY] Failed to store monthly closed counts:", errorMessage(error));
   }
@@ -13646,7 +13668,9 @@ function updateAnalytics(prs, monthlyCounts, monthlyClosedCounts, openedFromMerg
         combinedOpenedCounts[month] = (combinedOpenedCounts[month] || 0) + 1;
       }
     }
-    stateManager2.setMonthlyOpenedCounts(combinedOpenedCounts);
+    if (Object.keys(combinedOpenedCounts).length > 0) {
+      stateManager2.setMonthlyOpenedCounts(combinedOpenedCounts);
+    }
   } catch (error) {
     console.error("[DAILY] Failed to compute/store monthly opened counts:", errorMessage(error));
   }
@@ -13816,6 +13840,7 @@ var init_daily = __esm({
     "use strict";
     init_core();
     init_errors();
+    init_github_stats();
     init_json();
     init_core();
   }
@@ -14542,8 +14567,14 @@ async function fetchDashboardData(token) {
       console.error(`Warning: Failed to fetch recently merged PRs: ${errorMessage(err)}`);
       return [];
     }),
-    prMonitor.fetchUserMergedPRCounts(),
-    prMonitor.fetchUserClosedPRCounts(),
+    prMonitor.fetchUserMergedPRCounts().catch((err) => {
+      console.error(`Warning: Failed to fetch merged PR counts: ${errorMessage(err)}`);
+      return emptyPRCountsResult();
+    }),
+    prMonitor.fetchUserClosedPRCounts().catch((err) => {
+      console.error(`Warning: Failed to fetch closed PR counts: ${errorMessage(err)}`);
+      return emptyPRCountsResult();
+    }),
     issueMonitor.fetchCommentedIssues().catch((error) => {
       const msg = errorMessage(error);
       if (msg.includes("No GitHub username configured")) {
@@ -14567,12 +14598,16 @@ async function fetchDashboardData(token) {
   const { monthlyCounts, monthlyOpenedCounts: openedFromMerged } = mergedResult;
   const { monthlyCounts: monthlyClosedCounts, monthlyOpenedCounts: openedFromClosed } = closedResult;
   try {
-    stateManager2.setMonthlyMergedCounts(monthlyCounts);
+    if (Object.keys(monthlyCounts).length > 0) {
+      stateManager2.setMonthlyMergedCounts(monthlyCounts);
+    }
   } catch (error) {
     console.error("[DASHBOARD] Failed to store monthly merged counts:", errorMessage(error));
   }
   try {
-    stateManager2.setMonthlyClosedCounts(monthlyClosedCounts);
+    if (Object.keys(monthlyClosedCounts).length > 0) {
+      stateManager2.setMonthlyClosedCounts(monthlyClosedCounts);
+    }
   } catch (error) {
     console.error("[DASHBOARD] Failed to store monthly closed counts:", errorMessage(error));
   }
@@ -14587,7 +14622,9 @@ async function fetchDashboardData(token) {
         combinedOpenedCounts[month] = (combinedOpenedCounts[month] || 0) + 1;
       }
     }
-    stateManager2.setMonthlyOpenedCounts(combinedOpenedCounts);
+    if (Object.keys(combinedOpenedCounts).length > 0) {
+      stateManager2.setMonthlyOpenedCounts(combinedOpenedCounts);
+    }
   } catch (error) {
     console.error("[DASHBOARD] Failed to store monthly opened counts:", errorMessage(error));
   }
@@ -14634,6 +14671,7 @@ var init_dashboard_data = __esm({
     "use strict";
     init_core();
     init_errors();
+    init_github_stats();
     init_daily();
   }
 });

package/dist/commands/daily.js

@@ -8,6 +8,7 @@
  */
 import { getStateManager, PRMonitor, IssueConversationMonitor, requireGitHubToken, CRITICAL_STATUSES, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatBriefSummary, formatSummary, } from '../core/index.js';
 import { errorMessage } from '../core/errors.js';
+import { emptyPRCountsResult } from '../core/github-stats.js';
 import { deduplicateDigest, compactActionableIssues, compactRepoGroups, } from '../formatters/json.js';
 // Re-export domain functions so existing consumers (tests, dashboard, startup)
 // can continue importing from './daily.js' without changes.
@@ -28,11 +29,17 @@ async function fetchPRData(prMonitor, token) {
         console.error(`Warning: ${failures.length} PR fetch(es) failed`);
     }
     // Fetch merged PR counts, closed PR counts, recently closed PRs, recently merged PRs, and commented issues in parallel
-    // Recently closed/merged are non-critical (cosmetic sections), so isolate their failure
+    // All stats fetches are non-critical (cosmetic/scoring), so isolate their failure
     const issueMonitor = new IssueConversationMonitor(token);
     const [mergedResult, closedResult, recentlyClosedPRs, recentlyMergedPRs, issueConversationResult] = await Promise.all([
-        prMonitor.fetchUserMergedPRCounts(),
-        prMonitor.fetchUserClosedPRCounts(),
+        prMonitor.fetchUserMergedPRCounts().catch((err) => {
+            console.error(`Warning: Failed to fetch merged PR counts: ${errorMessage(err)}`);
+            return emptyPRCountsResult();
+        }),
+        prMonitor.fetchUserClosedPRCounts().catch((err) => {
+            console.error(`Warning: Failed to fetch closed PR counts: ${errorMessage(err)}`);
+            return emptyPRCountsResult();
+        }),
         prMonitor.fetchRecentlyClosedPRs().catch((err) => {
             console.error(`Warning: Failed to fetch recently closed PRs: ${errorMessage(err)}`);
             return [];
@@ -195,15 +202,21 @@ async function updateRepoScores(prMonitor, prs, mergedCounts, closedCounts) {
  */
 function updateAnalytics(prs, monthlyCounts, monthlyClosedCounts, openedFromMerged, openedFromClosed) {
     const stateManager = getStateManager();
-    // Store monthly chart data (non-critical — each metric isolated so partial failures don't leave inconsistent state)
+    // Store monthly chart data (non-critical — each metric isolated so partial failures don't leave inconsistent state).
+    // Guard: skip overwriting when the data is empty to avoid wiping existing chart data on transient API failures.
+    // An empty object means the fetch failed and fell back to emptyPRCountsResult(), so we preserve previous state.
     try {
-        stateManager.setMonthlyMergedCounts(monthlyCounts);
+        if (Object.keys(monthlyCounts).length > 0) {
+            stateManager.setMonthlyMergedCounts(monthlyCounts);
+        }
     }
     catch (error) {
         console.error('[DAILY] Failed to store monthly merged counts:', errorMessage(error));
     }
     try {
-        stateManager.setMonthlyClosedCounts(monthlyClosedCounts);
+        if (Object.keys(monthlyClosedCounts).length > 0) {
+            stateManager.setMonthlyClosedCounts(monthlyClosedCounts);
+        }
     }
     catch (error) {
         console.error('[DAILY] Failed to store monthly closed counts:', errorMessage(error));
@@ -221,7 +234,9 @@ function updateAnalytics(prs, monthlyCounts, monthlyClosedCounts, openedFromMerg
                 combinedOpenedCounts[month] = (combinedOpenedCounts[month] || 0) + 1;
             }
         }
-        stateManager.setMonthlyOpenedCounts(combinedOpenedCounts);
+        if (Object.keys(combinedOpenedCounts).length > 0) {
+            stateManager.setMonthlyOpenedCounts(combinedOpenedCounts);
+        }
     }
     catch (error) {
         console.error('[DAILY] Failed to compute/store monthly opened counts:', errorMessage(error));

package/dist/commands/dashboard-data.js

@@ -5,6 +5,7 @@
  */
 import { getStateManager, PRMonitor, IssueConversationMonitor } from '../core/index.js';
 import { errorMessage } from '../core/errors.js';
+import { emptyPRCountsResult } from '../core/github-stats.js';
 import { toShelvedPRRef } from './daily.js';
 /**
  * Fetch fresh dashboard data from GitHub.
@@ -25,8 +26,14 @@ export async function fetchDashboardData(token) {
             console.error(`Warning: Failed to fetch recently merged PRs: ${errorMessage(err)}`);
             return [];
         }),
-        prMonitor.fetchUserMergedPRCounts(),
-        prMonitor.fetchUserClosedPRCounts(),
+        prMonitor.fetchUserMergedPRCounts().catch((err) => {
+            console.error(`Warning: Failed to fetch merged PR counts: ${errorMessage(err)}`);
+            return emptyPRCountsResult();
+        }),
+        prMonitor.fetchUserClosedPRCounts().catch((err) => {
+            console.error(`Warning: Failed to fetch closed PR counts: ${errorMessage(err)}`);
+            return emptyPRCountsResult();
+        }),
         issueMonitor.fetchCommentedIssues().catch((error) => {
             const msg = errorMessage(error);
             if (msg.includes('No GitHub username configured')) {
@@ -51,14 +58,19 @@ export async function fetchDashboardData(token) {
     // Store monthly chart data (opened/merged/closed) so charts have data
     const { monthlyCounts, monthlyOpenedCounts: openedFromMerged } = mergedResult;
     const { monthlyCounts: monthlyClosedCounts, monthlyOpenedCounts: openedFromClosed } = closedResult;
+    // Guard: skip overwriting when data is empty to avoid wiping chart data on transient API failures.
     try {
-        stateManager.setMonthlyMergedCounts(monthlyCounts);
+        if (Object.keys(monthlyCounts).length > 0) {
+            stateManager.setMonthlyMergedCounts(monthlyCounts);
+        }
     }
     catch (error) {
         console.error('[DASHBOARD] Failed to store monthly merged counts:', errorMessage(error));
     }
     try {
-        stateManager.setMonthlyClosedCounts(monthlyClosedCounts);
+        if (Object.keys(monthlyClosedCounts).length > 0) {
+            stateManager.setMonthlyClosedCounts(monthlyClosedCounts);
+        }
     }
     catch (error) {
         console.error('[DASHBOARD] Failed to store monthly closed counts:', errorMessage(error));
@@ -74,7 +86,9 @@ export async function fetchDashboardData(token) {
                 combinedOpenedCounts[month] = (combinedOpenedCounts[month] || 0) + 1;
             }
         }
-        stateManager.setMonthlyOpenedCounts(combinedOpenedCounts);
+        if (Object.keys(combinedOpenedCounts).length > 0) {
+            stateManager.setMonthlyOpenedCounts(combinedOpenedCounts);
+        }
     }
     catch (error) {
         console.error('[DASHBOARD] Failed to store monthly opened counts:', errorMessage(error));

package/dist/core/github-stats.d.ts

@@ -4,7 +4,7 @@
  */
 import { Octokit } from '@octokit/rest';
 import { ClosedPR, MergedPR } from './types.js';
-/** TTL for cached PR count results (1 hour). */
+/** TTL for cached PR count results (24 hours — these stats change slowly). */
 export declare const PR_COUNTS_CACHE_TTL_MS: number;
 /** Return type shared by both merged and closed PR count functions. */
 export interface PRCountsResult<R> {
@@ -13,6 +13,8 @@ export interface PRCountsResult<R> {
     monthlyOpenedCounts: Record<string, number>;
     dailyActivityCounts: Record<string, number>;
 }
+/** Returns an empty PRCountsResult. Used as the fallback when stats fetches fail or username is empty. */
+export declare function emptyPRCountsResult<R>(): PRCountsResult<R>;
 /**
  * Fetch merged PR counts and latest merge dates per repository for the configured user.
  * Also builds a monthly histogram of all merges for the contribution timeline.

package/dist/core/github-stats.js

@@ -6,8 +6,18 @@ import { extractOwnerRepo, parseGitHubUrl, isOwnRepo } from './utils.js';
 import { debug, warn } from './logger.js';
 import { getHttpCache } from './http-cache.js';
 const MODULE = 'github-stats';
-/** TTL for cached PR count results (1 hour). */
-export const PR_COUNTS_CACHE_TTL_MS = 60 * 60 * 1000;
+/** TTL for cached PR count results (24 hours — these stats change slowly). */
+export const PR_COUNTS_CACHE_TTL_MS = 24 * 60 * 60 * 1000;
+/**
+ * Maximum number of pages to fetch during paginated PR count searches.
+ * GitHub's Search API becomes unreliable (500s, ECONNRESET) at deep pages (4-5+).
+ * 3 pages × 100 per_page = 300 results, which covers the vast majority of users.
+ */
+const MAX_PAGINATION_PAGES = 3;
+/** Returns an empty PRCountsResult. Used as the fallback when stats fetches fail or username is empty. */
+export function emptyPRCountsResult() {
+    return { repos: new Map(), monthlyCounts: {}, monthlyOpenedCounts: {}, dailyActivityCounts: {} };
+}
 /** Type guard for deserialized cache data — prevents crashes on corrupt/stale cache. */
 function isCachedPRCounts(v) {
     if (typeof v !== 'object' || v === null)
@@ -31,7 +41,7 @@ function isCachedPRCounts(v) {
  */
 async function fetchUserPRCounts(octokit, githubUsername, query, label, accumulateRepo) {
     if (!githubUsername) {
-        return { repos: new Map(), monthlyCounts: {}, monthlyOpenedCounts: {}, dailyActivityCounts: {} };
+        return emptyPRCountsResult();
     }
     // Check for a fresh cached result (avoids 10-20 paginated API calls)
     const cache = getHttpCache();
@@ -53,6 +63,7 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
     const dailyActivityCounts = {};
     let page = 1;
     let fetched = 0;
+    let totalCount;
     while (true) {
         const { data } = await octokit.search.issuesAndPullRequests({
             q: `is:pr ${query} author:${githubUsername}`,
@@ -61,6 +72,7 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
             per_page: 100,
             page,
         });
+        totalCount = data.total_count;
         for (const item of data.items) {
             const parsed = extractOwnerRepo(item.html_url);
             if (!parsed) {
@@ -95,14 +107,22 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
             }
         }
         fetched += data.items.length;
-        // Stop if we've fetched all results or hit the API limit (1000)
-        if (fetched >= data.total_count || fetched >= 1000 || data.items.length === 0) {
+        // Stop if we've fetched all results, hit the API limit (1000), or reached max pages
+        // GitHub Search API returns 500/ECONNRESET on deep pagination (page 4-5+),
+        // so we cap at MAX_PAGINATION_PAGES to avoid taking down the entire startup flow.
+        if (fetched >= data.total_count || fetched >= 1000 || data.items.length === 0 || page >= MAX_PAGINATION_PAGES) {
             break;
         }
         page++;
     }
+    if (fetched < totalCount && page >= MAX_PAGINATION_PAGES) {
+        warn(MODULE, `Pagination capped at ${MAX_PAGINATION_PAGES} pages: fetched ${fetched} of ${totalCount} ${label} PRs. Stats may be incomplete for prolific contributors.`);
+    }
     debug(MODULE, `Found ${fetched} ${label} PRs across ${repos.size} repos`);
-    // Cache the aggregated result (Map → entries array for JSON serialization)
+    // Cache the aggregated result (Map → entries array for JSON serialization).
+    // Note: truncated results (from pagination cap) are cached with the same TTL. This is an
+    // accepted tradeoff — prolific contributors (300+ PRs) may see slightly incomplete stats
+    // for up to 24 hours, but these are cosmetic and self-heal on the next cache expiry.
     cache.set(cacheKey, '', {
         reposEntries: Array.from(repos.entries()),
         monthlyCounts,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oss-autopilot/core",
-  "version": "0.42.5",
+  "version": "0.42.6",
   "description": "CLI and core library for managing open source contributions",
   "type": "module",
   "bin": {