@oss-autopilot/core 0.42.0 → 0.42.2

package/dist/commands/dashboard.js

@@ -7,6 +7,7 @@ import * as fs from 'fs';
 import * as path from 'path';
 import { execFile } from 'child_process';
 import { getStateManager, getDashboardPath, getGitHubToken } from '../core/index.js';
+import { errorMessage } from '../core/errors.js';
 import { outputJson } from '../formatters/json.js';
 import { fetchDashboardData, computePRsByRepo, computeTopRepos, getMonthlyData } from './dashboard-data.js';
 import { buildDashboardStats, generateDashboardHtml } from './dashboard-templates.js';
@@ -39,7 +40,7 @@ export async function runDashboard(options) {
             commentedIssues = result.commentedIssues;
         }
         catch (error) {
-            console.error('Failed to fetch fresh data:', error instanceof Error ? error.message : error);
+            console.error('Failed to fetch fresh data:', errorMessage(error));
             console.error('Falling back to cached data (issue conversations unavailable)...');
             digest = stateManager.getState().lastDigest;
         }

package/dist/commands/dismiss.d.ts

@@ -1,9 +1,9 @@
 /**
  * Dismiss/Undismiss commands
- * Manages dismissing issue reply notifications without posting a comment.
- * Dismissed issues resurface automatically when new responses arrive after the dismiss timestamp.
+ * Manages dismissing issue and PR notifications without posting a comment.
+ * Dismissed URLs resurface automatically when new responses arrive after the dismiss timestamp.
  */
-import { ISSUE_URL_PATTERN } from './validation.js';
+import { ISSUE_OR_PR_URL_PATTERN } from './validation.js';
 export interface DismissOutput {
     dismissed: boolean;
     url: string;
@@ -12,10 +12,10 @@ export interface UndismissOutput {
     undismissed: boolean;
     url: string;
 }
-export { ISSUE_URL_PATTERN };
+export { ISSUE_OR_PR_URL_PATTERN };
 export declare function runDismiss(options: {
-    issueUrl: string;
+    url: string;
 }): Promise<DismissOutput>;
 export declare function runUndismiss(options: {
-    issueUrl: string;
+    url: string;
 }): Promise<UndismissOutput>;

package/dist/commands/dismiss.js

@@ -1,29 +1,29 @@
 /**
  * Dismiss/Undismiss commands
- * Manages dismissing issue reply notifications without posting a comment.
- * Dismissed issues resurface automatically when new responses arrive after the dismiss timestamp.
+ * Manages dismissing issue and PR notifications without posting a comment.
+ * Dismissed URLs resurface automatically when new responses arrive after the dismiss timestamp.
  */
 import { getStateManager } from '../core/index.js';
-import { ISSUE_URL_PATTERN, validateGitHubUrl, validateUrl } from './validation.js';
-// Re-export for backward compatibility with tests
-export { ISSUE_URL_PATTERN };
+import { ISSUE_OR_PR_URL_PATTERN, validateGitHubUrl, validateUrl } from './validation.js';
+// Re-export for tests
+export { ISSUE_OR_PR_URL_PATTERN };
 export async function runDismiss(options) {
-    validateUrl(options.issueUrl);
-    validateGitHubUrl(options.issueUrl, ISSUE_URL_PATTERN, 'issue');
+    validateUrl(options.url);
+    validateGitHubUrl(options.url, ISSUE_OR_PR_URL_PATTERN, 'issue or PR');
     const stateManager = getStateManager();
-    const added = stateManager.dismissIssue(options.issueUrl, new Date().toISOString());
+    const added = stateManager.dismissIssue(options.url, new Date().toISOString());
     if (added) {
         stateManager.save();
     }
-    return { dismissed: added, url: options.issueUrl };
+    return { dismissed: added, url: options.url };
 }
 export async function runUndismiss(options) {
-    validateUrl(options.issueUrl);
-    validateGitHubUrl(options.issueUrl, ISSUE_URL_PATTERN, 'issue');
+    validateUrl(options.url);
+    validateGitHubUrl(options.url, ISSUE_OR_PR_URL_PATTERN, 'issue or PR');
     const stateManager = getStateManager();
-    const removed = stateManager.undismissIssue(options.issueUrl);
+    const removed = stateManager.undismissIssue(options.url);
     if (removed) {
         stateManager.save();
     }
-    return { undismissed: removed, url: options.issueUrl };
+    return { undismissed: removed, url: options.url };
 }

package/dist/commands/local-repos.js

@@ -7,6 +7,7 @@ import * as path from 'path';
 import * as os from 'os';
 import { execFileSync } from 'child_process';
 import { getStateManager, debug } from '../core/index.js';
+import { errorMessage } from '../core/errors.js';
 /** Default directories to scan for local clones */
 const DEFAULT_SCAN_PATHS = [
     path.join(os.homedir(), 'Documents', 'oss'),
@@ -115,7 +116,7 @@ export async function runLocalRepos(options) {
         stateManager.save();
     }
     catch (error) {
-        const msg = error instanceof Error ? error.message : String(error);
+        const msg = errorMessage(error);
         debug('local-repos', `Failed to cache scan results: ${msg}`);
     }
     return {

package/dist/commands/parse-list.js

@@ -4,6 +4,7 @@
  */
 import * as fs from 'fs';
 import * as path from 'path';
+import { errorMessage } from '../core/errors.js';
 /** Extract GitHub issue/PR URLs from a markdown line */
 function extractGitHubUrl(line) {
     const match = line.match(/https:\/\/github\.com\/([^/]+\/[^/]+)\/issues\/(\d+)/);
@@ -100,7 +101,7 @@ export async function runParseList(options) {
         content = fs.readFileSync(filePath, 'utf-8');
     }
     catch (error) {
-        const msg = error instanceof Error ? error.message : String(error);
+        const msg = errorMessage(error);
         throw new Error(`Failed to read file: ${msg}`, { cause: error });
     }
     return parseIssueList(content);

package/dist/commands/startup.js

@@ -7,21 +7,11 @@
  * `node cli.bundle.cjs startup --json` call, reducing UI noise in Claude Code.
  */
 import * as fs from 'fs';
-import * as path from 'path';
 import { execFile } from 'child_process';
-import { getStateManager, getGitHubToken } from '../core/index.js';
+import { getStateManager, getGitHubToken, getCLIVersion } from '../core/index.js';
+import { errorMessage } from '../core/errors.js';
 import { executeDailyCheck } from './daily.js';
 import { writeDashboardFromState } from './dashboard.js';
-function getVersion() {
-    try {
-        const pkgPath = path.join(path.dirname(process.argv[1]), '..', 'package.json');
-        return JSON.parse(fs.readFileSync(pkgPath, 'utf-8')).version;
-    }
-    catch (error) {
-        console.error('[STARTUP] Failed to detect CLI version:', error instanceof Error ? error.message : error);
-        return '0.0.0';
-    }
-}
 /**
  * Parse issueListPath from a config file's YAML frontmatter.
  * Returns the path string or undefined if not found.
@@ -75,7 +65,7 @@ export function detectIssueList() {
             }
         }
         catch (error) {
-            console.error('[STARTUP] Failed to read config:', error instanceof Error ? error.message : error);
+            console.error('[STARTUP] Failed to read config:', errorMessage(error));
         }
     }
     // 2. Probe known paths
@@ -98,7 +88,7 @@ export function detectIssueList() {
         return { path: issueListPath, source, availableCount, completedCount };
     }
     catch (error) {
-        console.error(`[STARTUP] Failed to read issue list at ${issueListPath}:`, error instanceof Error ? error.message : error);
+        console.error(`[STARTUP] Failed to read issue list at ${issueListPath}:`, errorMessage(error));
         return { path: issueListPath, source, availableCount: 0, completedCount: 0 };
     }
 }
@@ -122,7 +112,7 @@ function openInBrowser(filePath) {
  * Errors from the daily check propagate to the caller.
  */
 export async function runStartup() {
-    const version = getVersion();
+    const version = getCLIVersion();
     const stateManager = getStateManager();
     // 1. Check setup
     if (!stateManager.isSetupComplete()) {
@@ -151,7 +141,7 @@ export async function runStartup() {
         }
     }
     catch (error) {
-        console.error('[STARTUP] Dashboard generation failed:', error instanceof Error ? error.message : error);
+        console.error('[STARTUP] Dashboard generation failed:', errorMessage(error));
     }
     // Append dashboard status to brief summary (only startup opens the browser, not daily)
     if (dashboardOpened) {

package/dist/commands/validation.d.ts

@@ -5,10 +5,12 @@
 export declare const PR_URL_PATTERN: RegExp;
 /** Matches GitHub issue URLs: https://github.com/owner/repo/issues/123 */
 export declare const ISSUE_URL_PATTERN: RegExp;
+/** Matches GitHub issue or PR URLs: /issues/123 or /pull/123 */
+export declare const ISSUE_OR_PR_URL_PATTERN: RegExp;
 /**
  * Validate a GitHub URL against a pattern. Throws if invalid.
  */
-export declare function validateGitHubUrl(url: string, pattern: RegExp, entityType: 'PR' | 'issue'): void;
+export declare function validateGitHubUrl(url: string, pattern: RegExp, entityType: 'PR' | 'issue' | 'issue or PR'): void;
 /**
  * Validate that a URL does not exceed the maximum allowed length.
  * Returns the URL if valid, throws if too long.

package/dist/commands/validation.js

@@ -6,6 +6,8 @@ import { ValidationError } from '../core/errors.js';
 export const PR_URL_PATTERN = /^https:\/\/github\.com\/[^/]+\/[^/]+\/pull\/\d+$/;
 /** Matches GitHub issue URLs: https://github.com/owner/repo/issues/123 */
 export const ISSUE_URL_PATTERN = /^https:\/\/github\.com\/[^/]+\/[^/]+\/issues\/\d+$/;
+/** Matches GitHub issue or PR URLs: /issues/123 or /pull/123 */
+export const ISSUE_OR_PR_URL_PATTERN = /^https:\/\/github\.com\/[^/]+\/[^/]+\/(issues|pull)\/\d+$/;
 /** Maximum allowed URL length */
 const MAX_URL_LENGTH = 2048;
 /** Maximum allowed PR/issue number */
@@ -20,8 +22,12 @@ const REPO_PATTERN = /^[a-zA-Z0-9._-]+\/[a-zA-Z0-9._-]+$/;
 export function validateGitHubUrl(url, pattern, entityType) {
     if (pattern.test(url))
         return;
-    const example = entityType === 'PR' ? 'https://github.com/owner/repo/pull/123' : 'https://github.com/owner/repo/issues/123';
-    throw new Error(`Invalid ${entityType} URL: ${url}. Expected format: ${example}`);
+    const examples = {
+        PR: 'https://github.com/owner/repo/pull/123',
+        issue: 'https://github.com/owner/repo/issues/123',
+        'issue or PR': 'https://github.com/owner/repo/issues/123 or https://github.com/owner/repo/pull/123',
+    };
+    throw new ValidationError(`Invalid ${entityType} URL: ${url}. Expected format: ${examples[entityType]}`);
 }
 /**
  * Validate that a URL does not exceed the maximum allowed length.
@@ -29,7 +35,7 @@ export function validateGitHubUrl(url, pattern, entityType) {
  */
 export function validateUrl(url) {
     if (url.length > MAX_URL_LENGTH) {
-        throw new Error(`URL exceeds maximum length of ${MAX_URL_LENGTH} characters`);
+        throw new ValidationError(`URL exceeds maximum length of ${MAX_URL_LENGTH} characters`);
     }
     return url;
 }
@@ -39,7 +45,7 @@ export function validateUrl(url) {
  */
 export function validatePRNumber(num) {
     if (!Number.isInteger(num) || num < 1 || num > MAX_PR_NUMBER) {
-        throw new Error(`PR number must be a positive integer up to ${MAX_PR_NUMBER}`);
+        throw new ValidationError(`PR number must be a positive integer up to ${MAX_PR_NUMBER}`);
     }
     return num;
 }
@@ -49,7 +55,7 @@ export function validatePRNumber(num) {
  */
 export function validateMessage(message) {
     if (message.length > MAX_MESSAGE_LENGTH) {
-        throw new Error(`Message exceeds maximum length of ${MAX_MESSAGE_LENGTH} characters`);
+        throw new ValidationError(`Message exceeds maximum length of ${MAX_MESSAGE_LENGTH} characters`);
     }
     return message;
 }
@@ -59,7 +65,7 @@ export function validateMessage(message) {
  */
 export function validateRepoIdentifier(repo) {
     if (!REPO_PATTERN.test(repo)) {
-        throw new Error(`Invalid repository format: "${repo}". Expected "owner/repo".`);
+        throw new ValidationError(`Invalid repository format: "${repo}". Expected "owner/repo".`);
     }
     return repo;
 }

package/dist/core/errors.d.ts

@@ -22,3 +22,12 @@ export declare class ConfigurationError extends OssAutopilotError {
 export declare class ValidationError extends OssAutopilotError {
     constructor(message: string);
 }
+/**
+ * Extract a human-readable message from an unknown error value.
+ */
+export declare function errorMessage(e: unknown): string;
+/**
+ * Safely extract an HTTP status code from an unknown error (e.g. Octokit errors).
+ * Returns undefined if the error doesn't have a numeric `status` property.
+ */
+export declare function getHttpStatusCode(error: unknown): number | undefined;

package/dist/core/errors.js

@@ -32,3 +32,20 @@ export class ValidationError extends OssAutopilotError {
         this.name = 'ValidationError';
     }
 }
+/**
+ * Extract a human-readable message from an unknown error value.
+ */
+export function errorMessage(e) {
+    return e instanceof Error ? e.message : String(e);
+}
+/**
+ * Safely extract an HTTP status code from an unknown error (e.g. Octokit errors).
+ * Returns undefined if the error doesn't have a numeric `status` property.
+ */
+export function getHttpStatusCode(error) {
+    if (error && typeof error === 'object' && 'status' in error) {
+        const status = error.status;
+        return typeof status === 'number' && Number.isFinite(status) ? status : undefined;
+    }
+    return undefined;
+}

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

@@ -4,35 +4,28 @@
  */
 import { Octokit } from '@octokit/rest';
 import { ClosedPR, MergedPR } from './types.js';
+/** TTL for cached PR count results (1 hour). */
+export declare const PR_COUNTS_CACHE_TTL_MS: number;
+/** Return type shared by both merged and closed PR count functions. */
+export interface PRCountsResult<R> {
+    repos: Map<string, R>;
+    monthlyCounts: Record<string, number>;
+    monthlyOpenedCounts: Record<string, number>;
+    dailyActivityCounts: Record<string, number>;
+}
 /**
  * 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.
  */
-export declare function fetchUserMergedPRCounts(octokit: Octokit, githubUsername: string): Promise<{
-    repos: Map<string, {
-        count: number;
-        lastMergedAt: string;
-    }>;
-    monthlyCounts: Record<string, number>;
-    monthlyOpenedCounts: Record<string, number>;
-    dailyActivityCounts: Record<string, number>;
-}>;
+export declare function fetchUserMergedPRCounts(octokit: Octokit, githubUsername: string): Promise<PRCountsResult<{
+    count: number;
+    lastMergedAt: string;
+}>>;
 /**
  * Fetch closed-without-merge PR counts per repository for the configured user.
  * Used to populate closedWithoutMergeCount in repo scores for accurate merge rate.
  */
-export declare function fetchUserClosedPRCounts(octokit: Octokit, githubUsername: string): Promise<{
-    repos: Map<string, number>;
-    monthlyCounts: Record<string, number>;
-    monthlyOpenedCounts: Record<string, number>;
-    dailyActivityCounts: Record<string, number>;
-}>;
-/**
- * Fetch GitHub star counts for a list of repositories.
- * Used to populate stargazersCount in repo scores for dashboard filtering by minStars.
- * Fetches concurrently with per-repo error isolation (missing/private repos are skipped).
- */
-export declare function fetchRepoStarCounts(octokit: Octokit, repos: string[]): Promise<Map<string, number>>;
+export declare function fetchUserClosedPRCounts(octokit: Octokit, githubUsername: string): Promise<PRCountsResult<number>>;
 /**
  * Shared helper: search for recent PRs and filter out own repos, excluded repos/orgs.
  * Returns parsed search results that pass all filters.

package/dist/core/github-stats.js

@@ -2,19 +2,51 @@
  * GitHub Stats - Fetching merged/closed PR counts and repository star counts.
  * Extracted from PRMonitor to isolate statistics-gathering API calls (#263).
  */
-import { extractOwnerRepo, parseGitHubUrl } from './utils.js';
-import { ValidationError } from './errors.js';
+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;
+/** Type guard for deserialized cache data — prevents crashes on corrupt/stale cache. */
+function isCachedPRCounts(v) {
+    if (typeof v !== 'object' || v === null)
+        return false;
+    const obj = v;
+    return (Array.isArray(obj.reposEntries) &&
+        typeof obj.monthlyCounts === 'object' &&
+        obj.monthlyCounts !== null &&
+        typeof obj.monthlyOpenedCounts === 'object' &&
+        obj.monthlyOpenedCounts !== null &&
+        typeof obj.dailyActivityCounts === 'object' &&
+        obj.dailyActivityCounts !== null);
+}
 /**
- * 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.
+ * Shared paginated search for user PR counts with histogram tracking.
+ *
+ * Handles: pagination, owner extraction, skip-own-repos, monthly/daily histograms.
+ * The `accumulateRepo` callback handles per-repo data and returns the primary date
+ * string (e.g. mergedAt or closedAt) used for monthly counts and daily activity.
+ * Return an empty string to skip histogram tracking for that item.
  */
-export async function fetchUserMergedPRCounts(octokit, githubUsername) {
+async function fetchUserPRCounts(octokit, githubUsername, query, label, accumulateRepo) {
     if (!githubUsername) {
         return { repos: new Map(), monthlyCounts: {}, monthlyOpenedCounts: {}, dailyActivityCounts: {} };
     }
-    debug(MODULE, `Fetching merged PR counts for @${githubUsername}...`);
+    // Check for a fresh cached result (avoids 10-20 paginated API calls)
+    const cache = getHttpCache();
+    const cacheKey = `pr-counts:${label}:${githubUsername}`;
+    const cached = cache.getIfFresh(cacheKey, PR_COUNTS_CACHE_TTL_MS);
+    if (cached && isCachedPRCounts(cached)) {
+        debug(MODULE, `Using cached ${label} PR counts for @${githubUsername}`);
+        return {
+            repos: new Map(cached.reposEntries),
+            monthlyCounts: cached.monthlyCounts,
+            monthlyOpenedCounts: cached.monthlyOpenedCounts,
+            dailyActivityCounts: cached.dailyActivityCounts,
+        };
+    }
+    debug(MODULE, `Fetching ${label} PR counts for @${githubUsername}...`);
     const repos = new Map();
     const monthlyCounts = {};
     const monthlyOpenedCounts = {};
@@ -23,7 +55,7 @@ export async function fetchUserMergedPRCounts(octokit, githubUsername) {
     let fetched = 0;
     while (true) {
         const { data } = await octokit.search.issuesAndPullRequests({
-            q: `is:pr is:merged author:${githubUsername}`,
+            q: `is:pr ${query} author:${githubUsername}`,
             sort: 'updated',
             order: 'desc',
             per_page: 100,
@@ -32,49 +64,35 @@ export async function fetchUserMergedPRCounts(octokit, githubUsername) {
         for (const item of data.items) {
             const parsed = extractOwnerRepo(item.html_url);
             if (!parsed) {
-                warn(MODULE, `Skipping merged PR with unparseable URL: ${item.html_url}`);
+                warn(MODULE, `Skipping ${label} PR with unparseable URL: ${item.html_url}`);
                 continue;
             }
             const { owner } = parsed;
             const repo = `${owner}/${parsed.repo}`;
             // Skip own repos (PRs to your own repos aren't OSS contributions)
-            if (owner.toLowerCase() === githubUsername.toLowerCase())
+            if (isOwnRepo(owner, githubUsername))
                 continue;
             // Note: excludeRepos/excludeOrgs are intentionally NOT filtered here.
             // Those filters control issue discovery/search, not historical statistics.
-            // A merged PR is a merged PR regardless of current tracking preferences.
-            const mergedAt = item.pull_request?.merged_at || item.closed_at || '';
-            // Per-repo tracking
-            const existing = repos.get(repo);
-            if (existing) {
-                existing.count += 1;
-                if (mergedAt && mergedAt > existing.lastMergedAt) {
-                    existing.lastMergedAt = mergedAt;
-                }
-            }
-            else {
-                repos.set(repo, { count: 1, lastMergedAt: mergedAt });
-            }
-            // Monthly histogram (every PR counted individually)
-            if (mergedAt) {
-                const month = mergedAt.slice(0, 7); // "YYYY-MM"
+            // Per-repo accumulation + get primary date for histograms
+            const primaryDate = accumulateRepo(repos, repo, item);
+            // Monthly histogram for primary date (merged/closed)
+            if (primaryDate) {
+                const month = primaryDate.slice(0, 7); // "YYYY-MM"
                 monthlyCounts[month] = (monthlyCounts[month] || 0) + 1;
+                // Daily activity for primary date
+                const day = primaryDate.slice(0, 10);
+                if (day.length === 10)
+                    dailyActivityCounts[day] = (dailyActivityCounts[day] || 0) + 1;
             }
-            // Track when this PR was opened (for monthly opened histogram)
+            // Track when this PR was opened (for monthly opened histogram + daily activity)
             if (item.created_at) {
                 const openedMonth = item.created_at.slice(0, 7); // "YYYY-MM"
                 monthlyOpenedCounts[openedMonth] = (monthlyOpenedCounts[openedMonth] || 0) + 1;
-                // Daily activity: PR opened
                 const openedDay = item.created_at.slice(0, 10);
                 if (openedDay.length === 10)
                     dailyActivityCounts[openedDay] = (dailyActivityCounts[openedDay] || 0) + 1;
             }
-            // Daily activity: PR merged
-            if (mergedAt) {
-                const mergedDay = mergedAt.slice(0, 10);
-                if (mergedDay.length === 10)
-                    dailyActivityCounts[mergedDay] = (dailyActivityCounts[mergedDay] || 0) + 1;
-            }
         }
         fetched += data.items.length;
         // Stop if we've fetched all results or hit the API limit (1000)
@@ -83,120 +101,48 @@ export async function fetchUserMergedPRCounts(octokit, githubUsername) {
         }
         page++;
     }
-    debug(MODULE, `Found ${fetched} merged PRs across ${repos.size} repos`);
+    debug(MODULE, `Found ${fetched} ${label} PRs across ${repos.size} repos`);
+    // Cache the aggregated result (Map → entries array for JSON serialization)
+    cache.set(cacheKey, '', {
+        reposEntries: Array.from(repos.entries()),
+        monthlyCounts,
+        monthlyOpenedCounts,
+        dailyActivityCounts,
+    });
     return { repos, monthlyCounts, monthlyOpenedCounts, dailyActivityCounts };
 }
 /**
- * Fetch closed-without-merge PR counts per repository for the configured user.
- * Used to populate closedWithoutMergeCount in repo scores for accurate merge rate.
+ * 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.
  */
-export async function fetchUserClosedPRCounts(octokit, githubUsername) {
-    if (!githubUsername) {
-        return { repos: new Map(), monthlyCounts: {}, monthlyOpenedCounts: {}, dailyActivityCounts: {} };
-    }
-    debug(MODULE, `Fetching closed PR counts for @${githubUsername}...`);
-    const repos = new Map();
-    const monthlyCounts = {};
-    const monthlyOpenedCounts = {};
-    const dailyActivityCounts = {};
-    let page = 1;
-    let fetched = 0;
-    while (true) {
-        const { data } = await octokit.search.issuesAndPullRequests({
-            q: `is:pr is:closed is:unmerged author:${githubUsername}`,
-            sort: 'updated',
-            order: 'desc',
-            per_page: 100,
-            page,
-        });
-        for (const item of data.items) {
-            const parsed = extractOwnerRepo(item.html_url);
-            if (!parsed) {
-                warn(MODULE, `Skipping closed PR with unparseable URL: ${item.html_url}`);
-                continue;
-            }
-            const { owner } = parsed;
-            const repo = `${owner}/${parsed.repo}`;
-            // Skip own repos
-            if (owner.toLowerCase() === githubUsername.toLowerCase())
-                continue;
-            // Note: excludeRepos/excludeOrgs are intentionally NOT filtered here.
-            // Those filters control issue discovery/search, not historical statistics.
-            // A closed PR is a closed PR regardless of current tracking preferences.
-            repos.set(repo, (repos.get(repo) || 0) + 1);
-            // Track when this PR was closed (for monthly closed histogram)
-            if (item.closed_at) {
-                const closedMonth = item.closed_at.slice(0, 7); // "YYYY-MM"
-                monthlyCounts[closedMonth] = (monthlyCounts[closedMonth] || 0) + 1;
-                // Daily activity: PR closed
-                const closedDay = item.closed_at.slice(0, 10);
-                if (closedDay.length === 10)
-                    dailyActivityCounts[closedDay] = (dailyActivityCounts[closedDay] || 0) + 1;
-            }
-            // Track when this PR was opened (for monthly opened histogram)
-            if (item.created_at) {
-                const openedMonth = item.created_at.slice(0, 7); // "YYYY-MM"
-                monthlyOpenedCounts[openedMonth] = (monthlyOpenedCounts[openedMonth] || 0) + 1;
-                // Daily activity: PR opened
-                const openedDay = item.created_at.slice(0, 10);
-                if (openedDay.length === 10)
-                    dailyActivityCounts[openedDay] = (dailyActivityCounts[openedDay] || 0) + 1;
+export function fetchUserMergedPRCounts(octokit, githubUsername) {
+    return fetchUserPRCounts(octokit, githubUsername, 'is:merged', 'merged', (repos, repo, item) => {
+        if (!item.pull_request?.merged_at) {
+            warn(MODULE, `merged_at missing for merged PR ${item.html_url}${item.closed_at ? ', falling back to closed_at' : ', no date available'}`);
+        }
+        const mergedAt = item.pull_request?.merged_at || item.closed_at || '';
+        const existing = repos.get(repo);
+        if (existing) {
+            existing.count += 1;
+            if (mergedAt && mergedAt > existing.lastMergedAt) {
+                existing.lastMergedAt = mergedAt;
             }
         }
-        fetched += data.items.length;
-        if (fetched >= data.total_count || fetched >= 1000 || data.items.length === 0) {
-            break;
+        else {
+            repos.set(repo, { count: 1, lastMergedAt: mergedAt });
         }
-        page++;
-    }
-    debug(MODULE, `Found ${fetched} closed (unmerged) PRs across ${repos.size} repos`);
-    return { repos, monthlyCounts, monthlyOpenedCounts, dailyActivityCounts };
+        return mergedAt;
+    });
 }
 /**
- * Fetch GitHub star counts for a list of repositories.
- * Used to populate stargazersCount in repo scores for dashboard filtering by minStars.
- * Fetches concurrently with per-repo error isolation (missing/private repos are skipped).
+ * Fetch closed-without-merge PR counts per repository for the configured user.
+ * Used to populate closedWithoutMergeCount in repo scores for accurate merge rate.
  */
-export async function fetchRepoStarCounts(octokit, repos) {
-    if (repos.length === 0)
-        return new Map();
-    debug(MODULE, `Fetching star counts for ${repos.length} repos...`);
-    const results = new Map();
-    // Fetch in parallel chunks to avoid overwhelming the API
-    const chunkSize = 10;
-    for (let i = 0; i < repos.length; i += chunkSize) {
-        const chunk = repos.slice(i, i + chunkSize);
-        const settled = await Promise.allSettled(chunk.map(async (repo) => {
-            const parts = repo.split('/');
-            if (parts.length !== 2 || !parts[0] || !parts[1]) {
-                throw new ValidationError(`Malformed repo identifier: "${repo}"`);
-            }
-            const [owner, name] = parts;
-            const { data } = await octokit.repos.get({ owner, repo: name });
-            return { repo, stars: data.stargazers_count };
-        }));
-        let chunkFailures = 0;
-        for (let j = 0; j < settled.length; j++) {
-            const result = settled[j];
-            if (result.status === 'fulfilled') {
-                results.set(result.value.repo, result.value.stars);
-            }
-            else {
-                chunkFailures++;
-                warn(MODULE, `Failed to fetch stars for ${chunk[j]}: ${result.reason instanceof Error ? result.reason.message : result.reason}`);
-            }
-        }
-        // If entire chunk failed, likely a systemic issue (rate limit, auth, outage) — abort remaining
-        if (chunkFailures === chunk.length && chunk.length > 0) {
-            const remaining = repos.length - i - chunkSize;
-            if (remaining > 0) {
-                warn(MODULE, `Entire chunk failed, aborting remaining ${remaining} repos`);
-            }
-            break;
-        }
-    }
-    debug(MODULE, `Fetched star counts for ${results.size}/${repos.length} repos`);
-    return results;
+export function fetchUserClosedPRCounts(octokit, githubUsername) {
+    return fetchUserPRCounts(octokit, githubUsername, 'is:closed is:unmerged', 'closed', (repos, repo, item) => {
+        repos.set(repo, (repos.get(repo) || 0) + 1);
+        return item.closed_at || '';
+    });
 }
 /**
  * Shared helper: search for recent PRs and filter out own repos, excluded repos/orgs.
@@ -226,7 +172,7 @@ export async function fetchRecentPRs(octokit, config, query, label, days, mapIte
         }
         const repo = `${parsed.owner}/${parsed.repo}`;
         // Skip own repos
-        if (parsed.owner.toLowerCase() === config.githubUsername.toLowerCase())
+        if (isOwnRepo(parsed.owner, config.githubUsername))
             continue;
         // Skip excluded repos and orgs
         if (config.excludeRepos.includes(repo))

package/dist/core/http-cache.d.ts

@@ -32,6 +32,12 @@ export declare class HttpCache {
     private keyFor;
     /** Full path to the cache file for a given URL. */
     private pathFor;
+    /**
+     * Return the cached body if the entry exists and is younger than `maxAgeMs`.
+     * Useful for time-based caching where ETag validation isn't applicable
+     * (e.g., caching aggregated results from paginated API calls).
+     */
+    getIfFresh(key: string, maxAgeMs: number): unknown | null;
     /**
      * Look up a cached response. Returns `null` if no cache entry exists.
      */