@oss-autopilot/core 0.42.0 → 0.42.2

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,8 +8,9 @@ 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 { enableDebug, isDebugEnabled, debug, warn, timed } from './logger.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, info, 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';
 export * from './types.js';

package/dist/core/index.js

@@ -8,8 +8,9 @@ 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 { enableDebug, isDebugEnabled, debug, warn, timed } from './logger.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, info, 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';
 export * from './types.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.d.ts

@@ -29,6 +29,11 @@ export declare class IssueDiscovery {
      * Get starred repos, fetching from GitHub if cache is stale
      */
     getStarredReposWithRefresh(): Promise<string[]>;
+    /**
+     * Shared pipeline for Phases 2 and 3: spam-filter, repo-exclusion, vetting, and star-count filter.
+     * Extracts the common logic so each phase only needs to supply search results and context.
+     */
+    private filterVetAndScore;
     /**
      * Search for issues matching our criteria.
      * Searches in priority order: merged-PR repos first (no label filter), then starred repos,

package/dist/core/issue-discovery.js

@@ -12,8 +12,8 @@ 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 { warn } from './logger.js';
+import { ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
+import { debug, info, warn } from './logger.js';
 import { isDocOnlyIssue, detectLabelFarmingRepos, applyPerRepoCap } from './issue-filtering.js';
 import { IssueVetter } from './issue-vetting.js';
 import { calculateViabilityScore as calcViabilityScore } from './issue-scoring.js';
@@ -40,7 +40,7 @@ export class IssueDiscovery {
      * Updates the state manager with the list and timestamp.
      */
     async fetchStarredRepos() {
-        console.log('Fetching starred repositories...');
+        info(MODULE, 'Fetching starred repositories...');
         const starredRepos = [];
         try {
             // Paginate through all starred repos (up to 500 to avoid excessive API calls)
@@ -68,26 +68,26 @@ export class IssueDiscovery {
                 pageCount++;
                 // Limit to 5 pages (500 repos) to avoid excessive API usage
                 if (pageCount >= 5) {
-                    console.log('Reached pagination limit for starred repos (500)');
+                    info(MODULE, 'Reached pagination limit for starred repos (500)');
                     break;
                 }
             }
-            console.log(`Fetched ${starredRepos.length} starred repositories`);
+            info(MODULE, `Fetched ${starredRepos.length} starred repositories`);
             this.stateManager.setStarredRepos(starredRepos);
             return starredRepos;
         }
         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;
         }
@@ -101,6 +101,41 @@ export class IssueDiscovery {
         }
         return this.stateManager.getStarredRepos();
     }
+    /**
+     * Shared pipeline for Phases 2 and 3: spam-filter, repo-exclusion, vetting, and star-count filter.
+     * Extracts the common logic so each phase only needs to supply search results and context.
+     */
+    async filterVetAndScore(items, filterIssues, excludedRepoSets, remainingNeeded, minStars, phaseLabel) {
+        const spamRepos = detectLabelFarmingRepos(items);
+        if (spamRepos.size > 0) {
+            const spamCount = items.filter((i) => spamRepos.has(i.repository_url.split('/').slice(-2).join('/'))).length;
+            debug(MODULE, `[SPAM_FILTER] Filtered ${spamCount} issues from ${spamRepos.size} label-farming repos: ${[...spamRepos].join(', ')}`);
+        }
+        const itemsToVet = filterIssues(items)
+            .filter((item) => {
+            const repoFullName = item.repository_url.split('/').slice(-2).join('/');
+            if (spamRepos.has(repoFullName))
+                return false;
+            return excludedRepoSets.every((s) => !s.has(repoFullName));
+        })
+            .slice(0, remainingNeeded * 2);
+        if (itemsToVet.length === 0) {
+            debug(MODULE, `[${phaseLabel}] All ${items.length} items filtered before vetting`);
+            return { candidates: [], allVetFailed: false, rateLimitHit: false };
+        }
+        const { candidates: results, allFailed: allVetFailed, rateLimitHit, } = await this.vetter.vetIssuesParallel(itemsToVet.map((i) => i.html_url), remainingNeeded, 'normal');
+        const starFiltered = results.filter((c) => {
+            if (c.projectHealth.checkFailed)
+                return true;
+            const stars = c.projectHealth.stargazersCount ?? 0;
+            return stars >= minStars;
+        });
+        const starFilteredCount = results.length - starFiltered.length;
+        if (starFilteredCount > 0) {
+            debug(MODULE, `[STAR_FILTER] Filtered ${starFilteredCount} ${phaseLabel} candidates below ${minStars} stars`);
+        }
+        return { candidates: starFiltered, allVetFailed, rateLimitHit };
+    }
     /**
      * Search for issues matching our criteria.
      * Searches in priority order: merged-PR repos first (no label filter), then starred repos,
@@ -112,6 +147,7 @@ export class IssueDiscovery {
         const languages = options.languages || config.languages;
         const labels = options.labels || config.labels;
         const maxResults = options.maxResults || 10;
+        const minStars = config.minStars ?? 50;
         const allCandidates = [];
         let phase0Error = null;
         let phase1Error = null;
@@ -128,11 +164,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();
@@ -162,7 +198,7 @@ export class IssueDiscovery {
         const includeDocIssues = config.includeDocIssues ?? true;
         const aiBlocklisted = new Set(config.aiPolicyBlocklist ?? DEFAULT_CONFIG.aiPolicyBlocklist ?? []);
         if (aiBlocklisted.size > 0) {
-            console.log(`[AI_POLICY_FILTER] Filtering issues from ${aiBlocklisted.size} blocklisted repo(s): ${[...aiBlocklisted].join(', ')}`);
+            debug(MODULE, `[AI_POLICY_FILTER] Filtering issues from ${aiBlocklisted.size} blocklisted repo(s): ${[...aiBlocklisted].join(', ')}`);
         }
         const filterIssues = (items) => {
             return items.filter((item) => {
@@ -196,7 +232,7 @@ export class IssueDiscovery {
         if (phase0Repos.length > 0) {
             const mergedInPhase0 = Math.min(mergedPRRepos.length, phase0Repos.length);
             const openInPhase0 = phase0Repos.length - mergedInPhase0;
-            console.log(`Phase 0: Searching issues in ${phase0Repos.length} repos (${mergedInPhase0} merged-PR, ${openInPhase0} open-PR, no label filter)...`);
+            info(MODULE, `Phase 0: Searching issues in ${phase0Repos.length} repos (${mergedInPhase0} merged-PR, ${openInPhase0} open-PR, no label filter)...`);
             // Phase 0a: merged-PR repos (priority: merged_pr)
             const mergedPhase0Repos = phase0Repos.slice(0, mergedInPhase0);
             if (mergedPhase0Repos.length > 0) {
@@ -210,7 +246,7 @@ export class IssueDiscovery {
                     if (rateLimitHit) {
                         rateLimitHitDuringSearch = true;
                     }
-                    console.log(`Found ${mergedCandidates.length} candidates from merged-PR repos`);
+                    info(MODULE, `Found ${mergedCandidates.length} candidates from merged-PR repos`);
                 }
             }
             // Phase 0b: open-PR repos (priority: starred — intermediate tier)
@@ -227,7 +263,7 @@ export class IssueDiscovery {
                     if (rateLimitHit) {
                         rateLimitHitDuringSearch = true;
                     }
-                    console.log(`Found ${openCandidates.length} candidates from open-PR repos`);
+                    info(MODULE, `Found ${openCandidates.length} candidates from open-PR repos`);
                 }
             }
         }
@@ -235,7 +271,7 @@ export class IssueDiscovery {
         if (allCandidates.length < maxResults && starredRepos.length > 0) {
             const reposToSearch = starredRepos.filter((r) => !phase0RepoSet.has(r));
             if (reposToSearch.length > 0) {
-                console.log(`Phase 1: Searching issues in ${reposToSearch.length} starred repos...`);
+                info(MODULE, `Phase 1: Searching issues in ${reposToSearch.length} starred repos...`);
                 const remainingNeeded = maxResults - allCandidates.length;
                 if (remainingNeeded > 0) {
                     const { candidates: starredCandidates, allBatchesFailed, rateLimitHit, } = await this.searchInRepos(reposToSearch.slice(0, 10), baseQuery, remainingNeeded, 'starred', filterIssues);
@@ -246,14 +282,14 @@ export class IssueDiscovery {
                     if (rateLimitHit) {
                         rateLimitHitDuringSearch = true;
                     }
-                    console.log(`Found ${starredCandidates.length} candidates from starred repos`);
+                    info(MODULE, `Found ${starredCandidates.length} candidates from starred repos`);
                 }
             }
         }
         // Phase 2: General search (if still need more)
         let phase2Error = null;
         if (allCandidates.length < maxResults) {
-            console.log('Phase 2: General issue search...');
+            info(MODULE, 'Phase 2: General issue search...');
             const remainingNeeded = maxResults - allCandidates.length;
             try {
                 const { data } = await this.octokit.search.issuesAndPullRequests({
@@ -262,40 +298,9 @@ export class IssueDiscovery {
                     order: 'desc',
                     per_page: remainingNeeded * 3, // Fetch extra since some will be filtered
                 });
-                console.log(`Found ${data.total_count} issues in general search, processing top ${data.items.length}...`);
-                // Detect and filter label-farming repos (#97)
-                const spamRepos = detectLabelFarmingRepos(data.items);
-                if (spamRepos.size > 0) {
-                    const spamCount = data.items.filter((i) => spamRepos.has(i.repository_url.split('/').slice(-2).join('/'))).length;
-                    console.log(`[SPAM_FILTER] Filtered ${spamCount} issues from ${spamRepos.size} label-farming repos: ${[...spamRepos].join(', ')}`);
-                }
-                // Filter and exclude already-found repos
+                info(MODULE, `Found ${data.total_count} issues in general search, processing top ${data.items.length}...`);
                 const seenRepos = new Set(allCandidates.map((c) => c.issue.repo));
-                const itemsToVet = filterIssues(data.items)
-                    .filter((item) => {
-                    const repoFullName = item.repository_url.split('/').slice(-2).join('/');
-                    return !spamRepos.has(repoFullName);
-                })
-                    .filter((item) => {
-                    const repoFullName = item.repository_url.split('/').slice(-2).join('/');
-                    // Skip if already searched in earlier phases
-                    return (!phase0RepoSet.has(repoFullName) && !starredRepoSet.has(repoFullName) && !seenRepos.has(repoFullName));
-                })
-                    .slice(0, remainingNeeded * 2);
-                const { candidates: results, allFailed: allVetFailed, rateLimitHit: vetRateLimitHit, } = await this.vetter.vetIssuesParallel(itemsToVet.map((i) => i.html_url), remainingNeeded, 'normal');
-                // Apply minStars filter to Phase 2 results (#105)
-                // Phase 0/1 are exempt — if you've already contributed to or starred a repo, star count is irrelevant.
-                const minStars = config.minStars ?? 50;
-                const starFiltered = results.filter((c) => {
-                    if (c.projectHealth.checkFailed)
-                        return true; // Don't penalize repos we couldn't check
-                    const stars = c.projectHealth.stargazersCount ?? 0;
-                    return stars >= minStars;
-                });
-                const starFilteredCount = results.length - starFiltered.length;
-                if (starFilteredCount > 0) {
-                    console.log(`[STAR_FILTER] Filtered ${starFilteredCount} candidates below ${minStars} stars`);
-                }
+                const { candidates: starFiltered, allVetFailed, rateLimitHit: vetRateLimitHit, } = await this.filterVetAndScore(data.items, filterIssues, [phase0RepoSet, starredRepoSet, seenRepos], remainingNeeded, minStars, 'Phase 2');
                 allCandidates.push(...starFiltered);
                 if (allVetFailed) {
                     phase2Error = (phase2Error ? phase2Error + '; ' : '') + 'all vetting failed';
@@ -303,15 +308,15 @@ export class IssueDiscovery {
                 if (vetRateLimitHit) {
                     rateLimitHitDuringSearch = true;
                 }
-                console.log(`Found ${starFiltered.length} candidates from general search`);
+                info(MODULE, `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)
@@ -320,13 +325,12 @@ export class IssueDiscovery {
         // Uses label-free query to cast a wider net focused on repo health.
         let phase3Error = null;
         if (allCandidates.length < maxResults) {
-            console.log('Phase 3: Searching actively maintained repos...');
+            info(MODULE, 'Phase 3: Searching actively maintained repos...');
             const remainingNeeded = maxResults - allCandidates.length;
             const thirtyDaysAgo = new Date();
             thirtyDaysAgo.setDate(thirtyDaysAgo.getDate() - 30);
             const pushedSince = thirtyDaysAgo.toISOString().split('T')[0];
-            const phase3MinStars = config.minStars ?? 50;
-            const phase3Query = `is:issue is:open no:assignee ${langQuery} stars:>=${phase3MinStars} pushed:>=${pushedSince} archived:false`
+            const phase3Query = `is:issue is:open no:assignee ${langQuery} stars:>=${minStars} pushed:>=${pushedSince} archived:false`
                 .replace(/  +/g, ' ')
                 .trim();
             try {
@@ -336,36 +340,9 @@ export class IssueDiscovery {
                     order: 'desc',
                     per_page: remainingNeeded * 3,
                 });
-                console.log(`Found ${data.total_count} issues in maintained-repo search, processing top ${data.items.length}...`);
-                // Filter spam, already-found repos, and already-searched repos
-                const spamRepos = detectLabelFarmingRepos(data.items);
-                if (spamRepos.size > 0) {
-                    const spamCount = data.items.filter((i) => spamRepos.has(i.repository_url.split('/').slice(-2).join('/'))).length;
-                    console.log(`[SPAM_FILTER] Filtered ${spamCount} issues from ${spamRepos.size} label-farming repos: ${[...spamRepos].join(', ')}`);
-                }
+                info(MODULE, `Found ${data.total_count} issues in maintained-repo search, processing top ${data.items.length}...`);
                 const seenRepos = new Set(allCandidates.map((c) => c.issue.repo));
-                const itemsToVet = filterIssues(data.items)
-                    .filter((item) => {
-                    const repoFullName = item.repository_url.split('/').slice(-2).join('/');
-                    return (!spamRepos.has(repoFullName) &&
-                        !phase0RepoSet.has(repoFullName) &&
-                        !starredRepoSet.has(repoFullName) &&
-                        !seenRepos.has(repoFullName));
-                })
-                    .slice(0, remainingNeeded * 2);
-                const { candidates: results, allFailed: allVetFailed, rateLimitHit: vetRateLimitHit, } = await this.vetter.vetIssuesParallel(itemsToVet.map((i) => i.html_url), remainingNeeded, 'normal');
-                // Apply minStars filter (same as Phase 2, #105)
-                const minStars = config.minStars ?? 50;
-                const starFiltered = results.filter((c) => {
-                    if (c.projectHealth.checkFailed)
-                        return true;
-                    const stars = c.projectHealth.stargazersCount ?? 0;
-                    return stars >= minStars;
-                });
-                const starFilteredCount = results.length - starFiltered.length;
-                if (starFilteredCount > 0) {
-                    console.log(`[STAR_FILTER] Filtered ${starFilteredCount} Phase 3 candidates below ${minStars} stars`);
-                }
+                const { candidates: starFiltered, allVetFailed, rateLimitHit: vetRateLimitHit, } = await this.filterVetAndScore(data.items, filterIssues, [phase0RepoSet, starredRepoSet, seenRepos], remainingNeeded, minStars, 'Phase 3');
                 allCandidates.push(...starFiltered);
                 if (allVetFailed) {
                     phase3Error = 'all vetting failed';
@@ -373,15 +350,15 @@ export class IssueDiscovery {
                 if (vetRateLimitHit) {
                     rateLimitHitDuringSearch = true;
                 }
-                console.log(`Found ${starFiltered.length} candidates from maintained-repo search`);
+                info(MODULE, `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 +453,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;
@@ -553,7 +530,7 @@ export class IssueDiscovery {
         content += `- **Score**: Viability score (0-100)\n`;
         content += `- **Recommendation**: Y = approve, N = skip, ? = needs_review\n`;
         fs.writeFileSync(outputFile, content, 'utf-8');
-        console.log(`Saved ${sorted.length} issues to ${outputFile}`);
+        info(MODULE, `Saved ${sorted.length} issues to ${outputFile}`);
         return outputFile;
     }
     /**

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/logger.d.ts

@@ -11,6 +11,11 @@ export declare function isDebugEnabled(): boolean;
  * Log a debug message. Only outputs when --debug is enabled.
  */
 export declare function debug(module: string, message: string, ...args: unknown[]): void;
+/**
+ * Log an informational message. Always outputs to stderr.
+ * Use for user-facing progress indicators during long-running operations.
+ */
+export declare function info(module: string, message: string, ...args: unknown[]): void;
 /**
  * Log a warning. Always outputs.
  */

package/dist/core/logger.js

@@ -21,6 +21,14 @@ export function debug(module, message, ...args) {
     const timestamp = new Date().toISOString();
     console.error(`[${timestamp}] [DEBUG] [${module}] ${message}`, ...args);
 }
+/**
+ * Log an informational message. Always outputs to stderr.
+ * Use for user-facing progress indicators during long-running operations.
+ */
+export function info(module, message, ...args) {
+    const timestamp = new Date().toISOString();
+    console.error(`[${timestamp}] [INFO] [${module}] ${message}`, ...args);
+}
 /**
  * Log a warning. Always outputs.
  */

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.