@oss-autopilot/core 0.42.1 → 0.42.3

package/dist/commands/dismiss.d.ts

@@ -1,9 +1,8 @@
 /**
  * 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';
 export interface DismissOutput {
     dismissed: boolean;
     url: string;
@@ -12,10 +11,9 @@ export interface UndismissOutput {
     undismissed: boolean;
     url: string;
 }
-export { ISSUE_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,27 @@
 /**
  * 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';
 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/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 ValidationError(`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.

package/dist/core/index.d.ts

@@ -10,7 +10,7 @@ export { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 export { getOctokit, checkRateLimit, type RateLimitInfo } from './github.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 { 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

@@ -10,7 +10,7 @@ export { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 export { getOctokit, checkRateLimit } from './github.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 { 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-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

@@ -13,7 +13,7 @@ import { getStateManager } from './state.js';
 import { daysBetween, getDataDir } from './utils.js';
 import { DEFAULT_CONFIG } from './types.js';
 import { ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
-import { warn } from './logger.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,11 +68,11 @@ 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;
         }
@@ -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;
@@ -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,7 +308,7 @@ 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 errMsg = errorMessage(error);
@@ -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,7 +350,7 @@ 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 errMsg = errorMessage(error);
@@ -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/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.js

@@ -283,6 +283,11 @@ export class PRMonitor {
             // If the contributor pushed a commit after the maintainer's comment,
             // the changes have been addressed — waiting for maintainer re-review
             if (latestCommitDate && lastMaintainerCommentDate && latestCommitDate > lastMaintainerCommentDate) {
+                // Safety net (#431): if a CHANGES_REQUESTED review was submitted after
+                // the commit, the maintainer still expects changes — don't mask it
+                if (latestChangesRequestedDate && latestCommitDate < latestChangesRequestedDate) {
+                    return 'needs_response';
+                }
                 if (ciStatus === 'failing')
                     return 'failing_ci';
                 return 'changes_addressed';

package/dist/core/review-analysis.js

@@ -101,14 +101,15 @@ export function checkUnrespondedComments(comments, reviews, reviewComments, user
         if (!review.submitted_at)
             continue;
         const body = (review.body || '').trim();
-        // Include COMMENTED reviews even without body text — they indicate
-        // inline review comments were posted and may need a response (#151).
-        // Skip other empty-body reviews (APPROVED, CHANGES_REQUESTED, DISMISSED)
-        // as those are state changes without comment text.
-        if (!body && review.state !== 'COMMENTED')
+        // Include COMMENTED and CHANGES_REQUESTED reviews even without body text —
+        // they indicate inline review comments were posted and need a response (#151, #431).
+        // CHANGES_REQUESTED with only inline comments is actionable maintainer feedback.
+        // Skip other empty-body reviews (APPROVED, DISMISSED) as those are state changes.
+        if (!body && review.state !== 'COMMENTED' && review.state !== 'CHANGES_REQUESTED')
             continue;
         const author = review.user?.login || 'unknown';
-        // For inline-only COMMENTED reviews, skip pure self-replies (#199)
+        // For inline-only COMMENTED reviews, skip pure self-replies (#199).
+        // CHANGES_REQUESTED reviews are always actionable regardless of self-replies.
         if (!body && review.state === 'COMMENTED' && review.id != null) {
             if (isAllSelfReplies(review.id, reviewComments)) {
                 continue;
@@ -117,7 +118,9 @@ export function checkUnrespondedComments(comments, reviews, reviewComments, user
         // Resolve body: prefer actual text, then inline comment text, then synthetic placeholder
         const resolvedBody = body ||
             (review.id != null ? getInlineCommentBody(review.id, reviewComments) : undefined) ||
-            '(posted inline review comments)';
+            (review.state === 'CHANGES_REQUESTED'
+                ? '(requested changes via inline review comments)'
+                : '(posted inline review comments)');
         timeline.push({
             author,
             body: resolvedBody,

package/dist/core/state.d.ts

@@ -206,22 +206,22 @@ export declare class StateManager {
      */
     isPRShelved(url: string): boolean;
     /**
-     * Dismiss an issue by URL. Dismissed issues are excluded from `new_response` notifications
+     * Dismiss an issue or PR by URL. Dismissed URLs are excluded from `new_response` notifications
      * until new activity occurs after the dismiss timestamp.
-     * @param url - The full GitHub issue URL.
-     * @param timestamp - ISO timestamp of when the issue was dismissed.
+     * @param url - The full GitHub issue or PR URL.
+     * @param timestamp - ISO timestamp of when the issue/PR was dismissed.
      * @returns true if newly dismissed, false if already dismissed.
      */
     dismissIssue(url: string, timestamp: string): boolean;
     /**
-     * Undismiss an issue by URL.
-     * @param url - The full GitHub issue URL.
+     * Undismiss an issue or PR by URL.
+     * @param url - The full GitHub issue or PR URL.
      * @returns true if found and removed, false if not dismissed.
      */
     undismissIssue(url: string): boolean;
     /**
-     * Get the timestamp when an issue was dismissed.
-     * @param url - The full GitHub issue URL.
+     * Get the timestamp when an issue or PR was dismissed.
+     * @param url - The full GitHub issue or PR URL.
      * @returns The ISO dismiss timestamp, or undefined if not dismissed.
      */
     getIssueDismissedAt(url: string): string | undefined;

package/dist/core/state.js

@@ -721,10 +721,10 @@ export class StateManager {
     }
     // === Dismiss / Undismiss Issues ===
     /**
-     * Dismiss an issue by URL. Dismissed issues are excluded from `new_response` notifications
+     * Dismiss an issue or PR by URL. Dismissed URLs are excluded from `new_response` notifications
      * until new activity occurs after the dismiss timestamp.
-     * @param url - The full GitHub issue URL.
-     * @param timestamp - ISO timestamp of when the issue was dismissed.
+     * @param url - The full GitHub issue or PR URL.
+     * @param timestamp - ISO timestamp of when the issue/PR was dismissed.
      * @returns true if newly dismissed, false if already dismissed.
      */
     dismissIssue(url, timestamp) {
@@ -738,8 +738,8 @@ export class StateManager {
         return true;
     }
     /**
-     * Undismiss an issue by URL.
-     * @param url - The full GitHub issue URL.
+     * Undismiss an issue or PR by URL.
+     * @param url - The full GitHub issue or PR URL.
      * @returns true if found and removed, false if not dismissed.
      */
     undismissIssue(url) {
@@ -750,8 +750,8 @@ export class StateManager {
         return true;
     }
     /**
-     * Get the timestamp when an issue was dismissed.
-     * @param url - The full GitHub issue URL.
+     * Get the timestamp when an issue or PR was dismissed.
+     * @param url - The full GitHub issue or PR URL.
      * @returns The ISO dismiss timestamp, or undefined if not dismissed.
      */
     getIssueDismissedAt(url) {

package/dist/core/test-utils.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Shared test factories for oss-autopilot.
+ *
+ * Centralises mock object construction so that when types gain new required
+ * fields we only update one place. Every factory accepts a `Partial<T>`
+ * override bag — callers only specify the fields relevant to their test.
+ */
+import type { FetchedPR, DailyDigest, ShelvedPRRef, AgentState } from './types.js';
+import type { CapacityAssessment } from '../formatters/json.js';
+export declare function makeFetchedPR(overrides?: Partial<FetchedPR>): FetchedPR;
+export declare function makeDailyDigest(overrides?: Partial<DailyDigest>): DailyDigest;
+export declare function makeShelvedPRRef(overrides?: Partial<ShelvedPRRef>): ShelvedPRRef;
+export declare function makeCapacityAssessment(overrides?: Partial<CapacityAssessment>): CapacityAssessment;
+export declare function makeAgentState(overrides?: Partial<AgentState>): AgentState;