@oss-autopilot/core 0.41.0 → 0.42.1

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.
      */

package/dist/core/http-cache.js

@@ -14,6 +14,7 @@ import * as path from 'path';
 import * as crypto from 'crypto';
 import { getCacheDir } from './utils.js';
 import { debug } from './logger.js';
+import { getHttpStatusCode } from './errors.js';
 const MODULE = 'http-cache';
 /**
  * Maximum age (in ms) before a cache entry is considered stale and eligible for
@@ -44,6 +45,20 @@ export class HttpCache {
     pathFor(url) {
         return path.join(this.cacheDir, `${this.keyFor(url)}.json`);
     }
+    /**
+     * 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, maxAgeMs) {
+        const entry = this.get(key);
+        if (!entry)
+            return null;
+        const age = Date.now() - new Date(entry.cachedAt).getTime();
+        if (!Number.isFinite(age) || age < 0 || age > maxAgeMs)
+            return null;
+        return entry.body;
+    }
     /**
      * Look up a cached response. Returns `null` if no cache entry exists.
      */
@@ -262,8 +277,5 @@ export async function cachedRequest(cache, url, fetcher) {
  * Octokit throws a RequestError with status 304 for conditional requests.
  */
 function isNotModifiedError(err) {
-    if (err && typeof err === 'object' && 'status' in err) {
-        return err.status === 304;
-    }
-    return false;
+    return getHttpStatusCode(err) === 304;
 }

package/dist/core/index.d.ts

@@ -8,7 +8,8 @@ export { IssueDiscovery, type IssueCandidate, type SearchPriority, isDocOnlyIssu
 export { IssueConversationMonitor } from './issue-conversation.js';
 export { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 export { getOctokit, checkRateLimit, type RateLimitInfo } from './github.js';
-export { parseGitHubUrl, daysBetween, splitRepo, getDataDir, getStatePath, getBackupDir, getCacheDir, getDashboardPath, formatRelativeTime, byDateDescending, getGitHubToken, getGitHubTokenAsync, requireGitHubToken, resetGitHubTokenCache, } from './utils.js';
+export { parseGitHubUrl, daysBetween, splitRepo, isOwnRepo, getCLIVersion, getDataDir, getStatePath, getBackupDir, getCacheDir, getDashboardPath, formatRelativeTime, byDateDescending, getGitHubToken, getGitHubTokenAsync, requireGitHubToken, resetGitHubTokenCache, } from './utils.js';
+export { OssAutopilotError, ConfigurationError, ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
 export { enableDebug, isDebugEnabled, debug, warn, timed } from './logger.js';
 export { HttpCache, getHttpCache, resetHttpCache, cachedRequest, type CacheEntry } from './http-cache.js';
 export { CRITICAL_STATUSES, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatActionHint, formatBriefSummary, formatSummary, printDigest, } from './daily-logic.js';

package/dist/core/index.js

@@ -8,7 +8,8 @@ export { IssueDiscovery, isDocOnlyIssue, applyPerRepoCap, DOC_ONLY_LABELS, } fro
 export { IssueConversationMonitor } from './issue-conversation.js';
 export { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 export { getOctokit, checkRateLimit } from './github.js';
-export { parseGitHubUrl, daysBetween, splitRepo, getDataDir, getStatePath, getBackupDir, getCacheDir, getDashboardPath, formatRelativeTime, byDateDescending, getGitHubToken, getGitHubTokenAsync, requireGitHubToken, resetGitHubTokenCache, } from './utils.js';
+export { parseGitHubUrl, daysBetween, splitRepo, isOwnRepo, getCLIVersion, getDataDir, getStatePath, getBackupDir, getCacheDir, getDashboardPath, formatRelativeTime, byDateDescending, getGitHubToken, getGitHubTokenAsync, requireGitHubToken, resetGitHubTokenCache, } from './utils.js';
+export { OssAutopilotError, ConfigurationError, ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
 export { enableDebug, isDebugEnabled, debug, warn, timed } from './logger.js';
 export { HttpCache, getHttpCache, resetHttpCache, cachedRequest } from './http-cache.js';
 export { CRITICAL_STATUSES, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatActionHint, formatBriefSummary, formatSummary, printDigest, } from './daily-logic.js';

package/dist/core/issue-conversation.js

@@ -9,9 +9,9 @@ import { getOctokit } from './github.js';
 import { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 import { paginateAll } from './pagination.js';
 import { getStateManager } from './state.js';
-import { daysBetween, splitRepo, extractOwnerRepo } from './utils.js';
+import { daysBetween, splitRepo, extractOwnerRepo, isOwnRepo } from './utils.js';
 import { runWorkerPool } from './concurrency.js';
-import { ConfigurationError } from './errors.js';
+import { ConfigurationError, errorMessage } from './errors.js';
 import { debug, warn } from './logger.js';
 const MODULE = 'issue-conversation';
 const MAX_CONCURRENT_REQUESTS = 5;
@@ -71,7 +71,7 @@ export class IssueConversationMonitor {
             const { owner, repo } = parsed;
             const repoFullName = `${owner}/${repo}`;
             // Skip issues in user-owned repos (we only care about contributing to others' projects)
-            if (owner.toLowerCase() === username.toLowerCase())
+            if (isOwnRepo(owner, username))
                 continue;
             // Skip user-authored issues
             if (item.user?.login?.toLowerCase() === username.toLowerCase())
@@ -107,7 +107,7 @@ export class IssueConversationMonitor {
                 }
             }
             catch (error) {
-                const msg = error instanceof Error ? error.message : String(error);
+                const msg = errorMessage(error);
                 failures.push({ issueUrl: item.html_url, error: msg });
                 warn(MODULE, `Error analyzing issue ${item.html_url}: ${msg}`);
             }

package/dist/core/issue-discovery.js

@@ -12,7 +12,7 @@ import { getOctokit, checkRateLimit } from './github.js';
 import { getStateManager } from './state.js';
 import { daysBetween, getDataDir } from './utils.js';
 import { DEFAULT_CONFIG } from './types.js';
-import { ValidationError } from './errors.js';
+import { ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
 import { warn } from './logger.js';
 import { isDocOnlyIssue, detectLabelFarmingRepos, applyPerRepoCap } from './issue-filtering.js';
 import { IssueVetter } from './issue-vetting.js';
@@ -78,16 +78,16 @@ export class IssueDiscovery {
         }
         catch (error) {
             const cachedRepos = this.stateManager.getStarredRepos();
-            const errorMessage = error instanceof Error ? error.message : String(error);
-            warn(MODULE, 'Error fetching starred repos:', errorMessage);
+            const errMsg = errorMessage(error);
+            warn(MODULE, 'Error fetching starred repos:', errMsg);
             if (cachedRepos.length === 0) {
                 warn(MODULE, `Failed to fetch starred repositories from GitHub API. ` +
-                    `No cached repos available. Error: ${errorMessage}\n` +
+                    `No cached repos available. Error: ${errMsg}\n` +
                     `Tip: Ensure your GITHUB_TOKEN has the 'read:user' scope and try again.`);
             }
             else {
                 warn(MODULE, `Failed to fetch starred repositories from GitHub API. ` +
-                    `Using ${cachedRepos.length} cached repos instead. Error: ${errorMessage}`);
+                    `Using ${cachedRepos.length} cached repos instead. Error: ${errMsg}`);
             }
             return cachedRepos;
         }
@@ -128,11 +128,11 @@ export class IssueDiscovery {
         }
         catch (error) {
             // Fail fast on auth errors — no point searching with a bad token
-            if (error?.status === 401) {
+            if (getHttpStatusCode(error) === 401) {
                 throw error;
             }
             // Non-fatal: proceed with search for transient/network errors
-            warn(MODULE, 'Could not check rate limit:', error instanceof Error ? error.message : error);
+            warn(MODULE, 'Could not check rate limit:', errorMessage(error));
         }
         // Get merged-PR repos (highest merge probability)
         const mergedPRRepos = this.stateManager.getReposWithMergedPRs();
@@ -306,12 +306,12 @@ export class IssueDiscovery {
                 console.log(`Found ${starFiltered.length} candidates from general search`);
             }
             catch (error) {
-                const errorMessage = error instanceof Error ? error.message : String(error);
-                phase2Error = errorMessage;
+                const errMsg = errorMessage(error);
+                phase2Error = errMsg;
                 if (IssueVetter.isRateLimitError(error)) {
                     rateLimitHitDuringSearch = true;
                 }
-                warn(MODULE, `Error in general issue search: ${errorMessage}`);
+                warn(MODULE, `Error in general issue search: ${errMsg}`);
             }
         }
         // Phase 3: Actively maintained repos (#349)
@@ -376,12 +376,12 @@ export class IssueDiscovery {
                 console.log(`Found ${starFiltered.length} candidates from maintained-repo search`);
             }
             catch (error) {
-                const errorMessage = error instanceof Error ? error.message : String(error);
-                phase3Error = errorMessage;
+                const errMsg = errorMessage(error);
+                phase3Error = errMsg;
                 if (IssueVetter.isRateLimitError(error)) {
                     rateLimitHitDuringSearch = true;
                 }
-                warn(MODULE, `Error in maintained-repo search: ${errorMessage}`);
+                warn(MODULE, `Error in maintained-repo search: ${errMsg}`);
             }
         }
         if (allCandidates.length === 0) {
@@ -476,7 +476,7 @@ export class IssueDiscovery {
                     rateLimitFailures++;
                 }
                 const batchRepos = batch.join(', ');
-                warn(MODULE, `Error searching issues in batch [${batchRepos}]:`, error instanceof Error ? error.message : error);
+                warn(MODULE, `Error searching issues in batch [${batchRepos}]:`, errorMessage(error));
             }
         }
         const allBatchesFailed = failedBatches === batches.length && batches.length > 0;

package/dist/core/issue-vetting.js

@@ -6,7 +6,7 @@
  */
 import { paginateAll } from './pagination.js';
 import { parseGitHubUrl, daysBetween } from './utils.js';
-import { ValidationError } from './errors.js';
+import { ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
 import { warn } from './logger.js';
 import { getHttpCache, cachedRequest } from './http-cache.js';
 import { calculateRepoQualityBonus, calculateViabilityScore } from './issue-scoring.js';
@@ -254,7 +254,7 @@ export class IssueVetter {
                 if (IssueVetter.isRateLimitError(error)) {
                     rateLimitFailures++;
                 }
-                warn(MODULE, `Error vetting issue ${url}:`, error instanceof Error ? error.message : error);
+                warn(MODULE, `Error vetting issue ${url}:`, errorMessage(error));
             });
             pending.push(task);
             // Limit concurrency
@@ -275,11 +275,11 @@ export class IssueVetter {
     }
     /** Check if an error is a GitHub rate limit error (429 or rate-limit 403). */
     static isRateLimitError(error) {
-        const status = error?.status;
+        const status = getHttpStatusCode(error);
         if (status === 429)
             return true;
         if (status === 403) {
-            const msg = error instanceof Error ? error.message.toLowerCase() : String(error).toLowerCase();
+            const msg = errorMessage(error).toLowerCase();
             return msg.includes('rate limit');
         }
         return false;
@@ -306,9 +306,9 @@ export class IssueVetter {
             return { passed: data.total_count === 0 && linkedPRs.length === 0 };
         }
         catch (error) {
-            const errorMessage = error instanceof Error ? error.message : String(error);
-            warn(MODULE, `Failed to check for existing PRs on ${owner}/${repo}#${issueNumber}: ${errorMessage}. Assuming no existing PR.`);
-            return { passed: true, inconclusive: true, reason: errorMessage };
+            const errMsg = errorMessage(error);
+            warn(MODULE, `Failed to check for existing PRs on ${owner}/${repo}#${issueNumber}: ${errMsg}. Assuming no existing PR.`);
+            return { passed: true, inconclusive: true, reason: errMsg };
         }
     }
     /**
@@ -325,8 +325,8 @@ export class IssueVetter {
             return data.total_count;
         }
         catch (error) {
-            const errorMessage = error instanceof Error ? error.message : String(error);
-            warn(MODULE, `Could not check merged PRs in ${owner}/${repo}: ${errorMessage}. Defaulting to 0.`);
+            const errMsg = errorMessage(error);
+            warn(MODULE, `Could not check merged PRs in ${owner}/${repo}: ${errMsg}. Defaulting to 0.`);
             return 0;
         }
     }
@@ -370,9 +370,9 @@ export class IssueVetter {
             return { passed: true };
         }
         catch (error) {
-            const errorMessage = error instanceof Error ? error.message : String(error);
-            warn(MODULE, `Failed to check claim status on ${owner}/${repo}#${issueNumber}: ${errorMessage}. Assuming not claimed.`);
-            return { passed: true, inconclusive: true, reason: errorMessage };
+            const errMsg = errorMessage(error);
+            warn(MODULE, `Failed to check claim status on ${owner}/${repo}#${issueNumber}: ${errMsg}. Assuming not claimed.`);
+            return { passed: true, inconclusive: true, reason: errMsg };
         }
     }
     async checkProjectHealth(owner, repo) {
@@ -403,8 +403,8 @@ export class IssueVetter {
                 }
             }
             catch (error) {
-                const errorMessage = error instanceof Error ? error.message : String(error);
-                warn(MODULE, `Failed to check CI status for ${owner}/${repo}: ${errorMessage}. Defaulting to unknown.`);
+                const errMsg = errorMessage(error);
+                warn(MODULE, `Failed to check CI status for ${owner}/${repo}: ${errMsg}. Defaulting to unknown.`);
             }
             return {
                 repo: `${owner}/${repo}`,
@@ -419,8 +419,8 @@ export class IssueVetter {
             };
         }
         catch (error) {
-            const errorMessage = error instanceof Error ? error.message : String(error);
-            warn(MODULE, `Error checking project health for ${owner}/${repo}: ${errorMessage}`);
+            const errMsg = errorMessage(error);
+            warn(MODULE, `Error checking project health for ${owner}/${repo}: ${errMsg}`);
             return {
                 repo: `${owner}/${repo}`,
                 lastCommitAt: '',
@@ -430,7 +430,7 @@ export class IssueVetter {
                 ciStatus: 'unknown',
                 isActive: false,
                 checkFailed: true,
-                failureReason: errorMessage,
+                failureReason: errMsg,
             };
         }
     }

package/dist/core/pr-monitor.d.ts

@@ -12,6 +12,7 @@
  * - github-stats.ts: Merged/closed PR counts and star fetching
  */
 import { FetchedPR, DailyDigest, ClosedPR, MergedPR } from './types.js';
+import { type PRCountsResult } from './github-stats.js';
 export { computeDisplayLabel } from './display-utils.js';
 export { classifyCICheck, classifyFailingChecks } from './ci-analysis.js';
 export { isConditionalChecklistItem } from './checklist-analysis.js';
@@ -59,35 +60,20 @@ export declare class PRMonitor {
      * Fetch merged PR counts and latest merge dates per repository for the configured user.
      * Delegates to github-stats module.
      */
-    fetchUserMergedPRCounts(): Promise<{
-        repos: Map<string, {
-            count: number;
-            lastMergedAt: string;
-        }>;
-        monthlyCounts: Record<string, number>;
-        monthlyOpenedCounts: Record<string, number>;
-        dailyActivityCounts: Record<string, number>;
-    }>;
+    fetchUserMergedPRCounts(): Promise<PRCountsResult<{
+        count: number;
+        lastMergedAt: string;
+    }>>;
     /**
      * Fetch closed-without-merge PR counts per repository for the configured user.
      * Delegates to github-stats module.
      */
-    fetchUserClosedPRCounts(): Promise<{
-        repos: Map<string, number>;
-        monthlyCounts: Record<string, number>;
-        monthlyOpenedCounts: Record<string, number>;
-        dailyActivityCounts: Record<string, number>;
-    }>;
+    fetchUserClosedPRCounts(): Promise<PRCountsResult<number>>;
     /**
      * Fetch GitHub star counts for a list of repositories.
      * Delegates to github-stats module.
      */
     fetchRepoStarCounts(repos: string[]): Promise<Map<string, number>>;
-    /**
-     * Shared helper: search for recent PRs and filter out own repos, excluded repos/orgs.
-     * Returns parsed search results that pass all filters.
-     */
-    private fetchRecentPRs;
     /**
      * Fetch PRs closed without merge in the last N days.
      * Delegates to github-stats module.