@oss-autopilot/core 0.54.0 → 0.56.0

package/dist/commands/move.js

@@ -21,15 +21,17 @@ export async function runMove(options) {
             // Use current time — the CLI doesn't have cached PR data. The override
             // will auto-clear on the next daily run if the PR has new activity after this.
             const lastActivityAt = new Date().toISOString();
-            stateManager.setStatusOverride(options.prUrl, status, lastActivityAt);
-            stateManager.unshelvePR(options.prUrl);
-            stateManager.save();
+            stateManager.batch(() => {
+                stateManager.setStatusOverride(options.prUrl, status, lastActivityAt);
+                stateManager.unshelvePR(options.prUrl);
+            });
             return { url: options.prUrl, target, description: `Moved to ${label}` };
         }
         case 'shelved': {
-            stateManager.shelvePR(options.prUrl);
-            stateManager.clearStatusOverride(options.prUrl);
-            stateManager.save();
+            stateManager.batch(() => {
+                stateManager.shelvePR(options.prUrl);
+                stateManager.clearStatusOverride(options.prUrl);
+            });
             return {
                 url: options.prUrl,
                 target,
@@ -37,11 +39,10 @@ export async function runMove(options) {
             };
         }
         case 'auto': {
-            const clearedOverride = stateManager.clearStatusOverride(options.prUrl);
-            const unshelved = stateManager.unshelvePR(options.prUrl);
-            if (clearedOverride || unshelved) {
-                stateManager.save();
-            }
+            stateManager.batch(() => {
+                stateManager.clearStatusOverride(options.prUrl);
+                stateManager.unshelvePR(options.prUrl);
+            });
             return {
                 url: options.prUrl,
                 target,

package/dist/commands/setup.d.ts

@@ -2,7 +2,7 @@
  * Setup command
  * Interactive setup / configuration
  */
-import { type ProjectCategory } from '../core/types.js';
+import { type ProjectCategory, type IssueScope } from '../core/types.js';
 interface SetupOptions {
     reset?: boolean;
     set?: string[];
@@ -23,6 +23,7 @@ export interface SetupCompleteOutput {
         labels: string[];
         projectCategories: ProjectCategory[];
         preferredOrgs: string[];
+        scope: IssueScope[];
     };
 }
 export interface SetupPrompt {

package/dist/commands/setup.js

@@ -5,7 +5,7 @@
 import { getStateManager, DEFAULT_CONFIG } from '../core/index.js';
 import { ValidationError } from '../core/errors.js';
 import { validateGitHubUsername } from './validation.js';
-import { PROJECT_CATEGORIES } from '../core/types.js';
+import { PROJECT_CATEGORIES, ISSUE_SCOPES } from '../core/types.js';
 /** Parse and validate a positive integer setting value. */
 function parsePositiveInt(value, settingName) {
     const parsed = Number(value);
@@ -21,153 +21,181 @@ export async function runSetup(options) {
     if (options.set && options.set.length > 0) {
         const results = {};
         const warnings = [];
-        for (const setting of options.set) {
-            const [key, ...valueParts] = setting.split('=');
-            const value = valueParts.join('=');
-            switch (key) {
-                case 'username':
-                    validateGitHubUsername(value);
-                    stateManager.updateConfig({ githubUsername: value });
-                    results[key] = value;
-                    break;
-                case 'maxActivePRs': {
-                    const maxPRs = parsePositiveInt(value, 'maxActivePRs');
-                    stateManager.updateConfig({ maxActivePRs: maxPRs });
-                    results[key] = String(maxPRs);
-                    break;
-                }
-                case 'dormantDays': {
-                    const dormant = parsePositiveInt(value, 'dormantDays');
-                    stateManager.updateConfig({ dormantThresholdDays: dormant });
-                    results[key] = String(dormant);
-                    break;
-                }
-                case 'approachingDays': {
-                    const approaching = parsePositiveInt(value, 'approachingDays');
-                    stateManager.updateConfig({ approachingDormantDays: approaching });
-                    results[key] = String(approaching);
-                    break;
-                }
-                case 'languages':
-                    stateManager.updateConfig({ languages: value.split(',').map((l) => l.trim()) });
-                    results[key] = value;
-                    break;
-                case 'labels':
-                    stateManager.updateConfig({ labels: value.split(',').map((l) => l.trim()) });
-                    results[key] = value;
-                    break;
-                case 'showHealthCheck':
-                    stateManager.updateConfig({ showHealthCheck: value !== 'false' });
-                    results[key] = value !== 'false' ? 'true' : 'false';
-                    break;
-                case 'squashByDefault':
-                    if (value === 'ask') {
-                        stateManager.updateConfig({ squashByDefault: 'ask' });
-                        results[key] = 'ask';
+        stateManager.batch(() => {
+            for (const setting of options.set) {
+                const [key, ...valueParts] = setting.split('=');
+                const value = valueParts.join('=');
+                switch (key) {
+                    case 'username':
+                        validateGitHubUsername(value);
+                        stateManager.updateConfig({ githubUsername: value });
+                        results[key] = value;
+                        break;
+                    case 'maxActivePRs': {
+                        const maxPRs = parsePositiveInt(value, 'maxActivePRs');
+                        stateManager.updateConfig({ maxActivePRs: maxPRs });
+                        results[key] = String(maxPRs);
+                        break;
                     }
-                    else {
-                        stateManager.updateConfig({ squashByDefault: value !== 'false' });
-                        results[key] = value !== 'false' ? 'true' : 'false';
+                    case 'dormantDays': {
+                        const dormant = parsePositiveInt(value, 'dormantDays');
+                        stateManager.updateConfig({ dormantThresholdDays: dormant });
+                        results[key] = String(dormant);
+                        break;
                     }
-                    break;
-                case 'minStars': {
-                    const stars = Number(value);
-                    if (!Number.isFinite(stars) || !Number.isInteger(stars) || stars < 0) {
-                        throw new ValidationError(`Invalid value for minStars: "${value}". Must be a non-negative integer.`);
+                    case 'approachingDays': {
+                        const approaching = parsePositiveInt(value, 'approachingDays');
+                        stateManager.updateConfig({ approachingDormantDays: approaching });
+                        results[key] = String(approaching);
+                        break;
                     }
-                    stateManager.updateConfig({ minStars: stars });
-                    results[key] = String(stars);
-                    break;
-                }
-                case 'includeDocIssues':
-                    stateManager.updateConfig({ includeDocIssues: value === 'true' });
-                    results[key] = value === 'true' ? 'true' : 'false';
-                    break;
-                case 'aiPolicyBlocklist': {
-                    const entries = value
-                        .split(',')
-                        .map((r) => r.trim())
-                        .filter(Boolean);
-                    const valid = [];
-                    const invalid = [];
-                    for (const entry of entries) {
-                        const normalized = entry.replace(/\s+/g, '');
-                        if (/^[a-zA-Z0-9._-]+\/[a-zA-Z0-9._-]+$/.test(normalized)) {
-                            valid.push(normalized);
+                    case 'languages':
+                        stateManager.updateConfig({ languages: value.split(',').map((l) => l.trim()) });
+                        results[key] = value;
+                        break;
+                    case 'labels':
+                        stateManager.updateConfig({ labels: value.split(',').map((l) => l.trim()) });
+                        results[key] = value;
+                        break;
+                    case 'showHealthCheck':
+                        stateManager.updateConfig({ showHealthCheck: value !== 'false' });
+                        results[key] = value !== 'false' ? 'true' : 'false';
+                        break;
+                    case 'squashByDefault':
+                        if (value === 'ask') {
+                            stateManager.updateConfig({ squashByDefault: 'ask' });
+                            results[key] = 'ask';
                         }
                         else {
-                            invalid.push(entry);
+                            stateManager.updateConfig({ squashByDefault: value !== 'false' });
+                            results[key] = value !== 'false' ? 'true' : 'false';
                         }
-                    }
-                    if (invalid.length > 0) {
-                        warnings.push(`Warning: Skipping invalid entries (expected "owner/repo" format): ${invalid.join(', ')}`);
-                        results['aiPolicyBlocklist_invalidEntries'] = invalid.join(', ');
-                    }
-                    if (valid.length === 0 && entries.length > 0) {
-                        warnings.push('Warning: All entries were invalid. Blocklist not updated.');
-                        results[key] = '(all entries invalid)';
+                        break;
+                    case 'minStars': {
+                        const stars = Number(value);
+                        if (!Number.isFinite(stars) || !Number.isInteger(stars) || stars < 0) {
+                            throw new ValidationError(`Invalid value for minStars: "${value}". Must be a non-negative integer.`);
+                        }
+                        stateManager.updateConfig({ minStars: stars });
+                        results[key] = String(stars);
                         break;
                     }
-                    stateManager.updateConfig({ aiPolicyBlocklist: valid });
-                    results[key] = valid.length > 0 ? valid.join(', ') : '(empty)';
-                    break;
-                }
-                case 'projectCategories': {
-                    const categories = value
-                        .split(',')
-                        .map((c) => c.trim())
-                        .filter(Boolean);
-                    const validCategories = [];
-                    const invalidCategories = [];
-                    for (const cat of categories) {
-                        if (PROJECT_CATEGORIES.includes(cat)) {
-                            validCategories.push(cat);
+                    case 'includeDocIssues':
+                        stateManager.updateConfig({ includeDocIssues: value === 'true' });
+                        results[key] = value === 'true' ? 'true' : 'false';
+                        break;
+                    case 'aiPolicyBlocklist': {
+                        const entries = value
+                            .split(',')
+                            .map((r) => r.trim())
+                            .filter(Boolean);
+                        const valid = [];
+                        const invalid = [];
+                        for (const entry of entries) {
+                            const normalized = entry.replace(/\s+/g, '');
+                            if (/^[a-zA-Z0-9._-]+\/[a-zA-Z0-9._-]+$/.test(normalized)) {
+                                valid.push(normalized);
+                            }
+                            else {
+                                invalid.push(entry);
+                            }
                         }
-                        else {
-                            invalidCategories.push(cat);
+                        if (invalid.length > 0) {
+                            warnings.push(`Warning: Skipping invalid entries (expected "owner/repo" format): ${invalid.join(', ')}`);
+                            results['aiPolicyBlocklist_invalidEntries'] = invalid.join(', ');
+                        }
+                        if (valid.length === 0 && entries.length > 0) {
+                            warnings.push('Warning: All entries were invalid. Blocklist not updated.');
+                            results[key] = '(all entries invalid)';
+                            break;
                         }
+                        stateManager.updateConfig({ aiPolicyBlocklist: valid });
+                        results[key] = valid.length > 0 ? valid.join(', ') : '(empty)';
+                        break;
                     }
-                    if (invalidCategories.length > 0) {
-                        warnings.push(`Unknown project categories: ${invalidCategories.join(', ')}. Valid: ${PROJECT_CATEGORIES.join(', ')}`);
+                    case 'projectCategories': {
+                        const categories = value
+                            .split(',')
+                            .map((c) => c.trim())
+                            .filter(Boolean);
+                        const validCategories = [];
+                        const invalidCategories = [];
+                        for (const cat of categories) {
+                            if (PROJECT_CATEGORIES.includes(cat)) {
+                                validCategories.push(cat);
+                            }
+                            else {
+                                invalidCategories.push(cat);
+                            }
+                        }
+                        if (invalidCategories.length > 0) {
+                            warnings.push(`Unknown project categories: ${invalidCategories.join(', ')}. Valid: ${PROJECT_CATEGORIES.join(', ')}`);
+                        }
+                        const dedupedCategories = [...new Set(validCategories)];
+                        stateManager.updateConfig({ projectCategories: dedupedCategories });
+                        results[key] = dedupedCategories.length > 0 ? dedupedCategories.join(', ') : '(empty)';
+                        break;
                     }
-                    const dedupedCategories = [...new Set(validCategories)];
-                    stateManager.updateConfig({ projectCategories: dedupedCategories });
-                    results[key] = dedupedCategories.length > 0 ? dedupedCategories.join(', ') : '(empty)';
-                    break;
-                }
-                case 'preferredOrgs': {
-                    const orgs = value
-                        .split(',')
-                        .map((o) => o.trim())
-                        .filter(Boolean);
-                    const validOrgs = [];
-                    for (const org of orgs) {
-                        if (org.includes('/')) {
-                            warnings.push(`"${org}" looks like a repo path. Use org name only (e.g., "vercel" not "vercel/next.js").`);
+                    case 'preferredOrgs': {
+                        const orgs = value
+                            .split(',')
+                            .map((o) => o.trim())
+                            .filter(Boolean);
+                        const validOrgs = [];
+                        for (const org of orgs) {
+                            if (org.includes('/')) {
+                                warnings.push(`"${org}" looks like a repo path. Use org name only (e.g., "vercel" not "vercel/next.js").`);
+                            }
+                            else if (!/^[a-zA-Z0-9]([a-zA-Z0-9-]*[a-zA-Z0-9])?$/.test(org)) {
+                                warnings.push(`"${org}" is not a valid GitHub organization name. Skipping.`);
+                            }
+                            else {
+                                validOrgs.push(org.toLowerCase());
+                            }
                         }
-                        else if (!/^[a-zA-Z0-9]([a-zA-Z0-9-]*[a-zA-Z0-9])?$/.test(org)) {
-                            warnings.push(`"${org}" is not a valid GitHub organization name. Skipping.`);
+                        const dedupedOrgs = [...new Set(validOrgs)];
+                        stateManager.updateConfig({ preferredOrgs: dedupedOrgs });
+                        results[key] = dedupedOrgs.length > 0 ? dedupedOrgs.join(', ') : '(empty)';
+                        break;
+                    }
+                    case 'scope': {
+                        const scopeValues = value
+                            .split(',')
+                            .map((s) => s.trim())
+                            .filter(Boolean);
+                        const validScopes = [];
+                        const invalidScopes = [];
+                        for (const s of scopeValues) {
+                            if (ISSUE_SCOPES.includes(s)) {
+                                validScopes.push(s);
+                            }
+                            else {
+                                invalidScopes.push(s);
+                            }
                         }
-                        else {
-                            validOrgs.push(org.toLowerCase());
+                        if (invalidScopes.length > 0) {
+                            warnings.push(`Unknown issue scopes: ${invalidScopes.join(', ')}. Valid: ${ISSUE_SCOPES.join(', ')}`);
                         }
+                        const dedupedScopes = [...new Set(validScopes)];
+                        stateManager.updateConfig({ scope: dedupedScopes.length > 0 ? dedupedScopes : undefined });
+                        results[key] = dedupedScopes.length > 0 ? dedupedScopes.join(', ') : '(empty — using labels only)';
+                        break;
                     }
-                    const dedupedOrgs = [...new Set(validOrgs)];
-                    stateManager.updateConfig({ preferredOrgs: dedupedOrgs });
-                    results[key] = dedupedOrgs.length > 0 ? dedupedOrgs.join(', ') : '(empty)';
-                    break;
+                    case 'issueListPath':
+                        stateManager.updateConfig({ issueListPath: value || undefined });
+                        results[key] = value || '(cleared)';
+                        break;
+                    case 'complete':
+                        if (value === 'true') {
+                            stateManager.markSetupComplete();
+                            results[key] = 'true';
+                        }
+                        break;
+                    default:
+                        warnings.push(`Unknown setting: ${key}`);
                 }
-                case 'complete':
-                    if (value === 'true') {
-                        stateManager.markSetupComplete();
-                        results[key] = 'true';
-                    }
-                    break;
-                default:
-                    warnings.push(`Unknown setting: ${key}`);
             }
-        }
-        stateManager.save();
+        });
         return { success: true, settings: results, warnings: warnings.length > 0 ? warnings : undefined };
     }
     // Show setup status
@@ -183,6 +211,7 @@ export async function runSetup(options) {
                 labels: config.labels,
                 projectCategories: config.projectCategories ?? [],
                 preferredOrgs: config.preferredOrgs ?? [],
+                scope: config.scope ?? [],
             },
         };
     }
@@ -232,6 +261,13 @@ export async function runSetup(options) {
                 default: ['good first issue', 'help wanted'],
                 type: 'list',
             },
+            {
+                setting: 'scope',
+                prompt: 'What scope of issues do you want to discover? (beginner, intermediate, advanced — leave empty for default labels only)',
+                current: config.scope ?? [],
+                default: [],
+                type: 'list',
+            },
             {
                 setting: 'aiPolicyBlocklist',
                 prompt: 'Repos with anti-AI contribution policies to block (owner/repo, comma-separated)?',

package/dist/commands/shelve.js

@@ -15,21 +15,21 @@ export async function runShelve(options) {
     validateUrl(options.prUrl);
     validateGitHubUrl(options.prUrl, PR_URL_PATTERN, 'PR');
     const stateManager = getStateManager();
-    const added = stateManager.shelvePR(options.prUrl);
-    const clearedOverride = stateManager.clearStatusOverride(options.prUrl);
-    if (added || clearedOverride) {
-        stateManager.save();
-    }
+    let added = false;
+    stateManager.batch(() => {
+        added = stateManager.shelvePR(options.prUrl);
+        stateManager.clearStatusOverride(options.prUrl);
+    });
     return { shelved: added, url: options.prUrl };
 }
 export async function runUnshelve(options) {
     validateUrl(options.prUrl);
     validateGitHubUrl(options.prUrl, PR_URL_PATTERN, 'PR');
     const stateManager = getStateManager();
-    const removed = stateManager.unshelvePR(options.prUrl);
-    const clearedOverride = stateManager.clearStatusOverride(options.prUrl);
-    if (removed || clearedOverride) {
-        stateManager.save();
-    }
+    let removed = false;
+    stateManager.batch(() => {
+        removed = stateManager.unshelvePR(options.prUrl);
+        stateManager.clearStatusOverride(options.prUrl);
+    });
     return { unshelved: removed, url: options.prUrl };
 }

package/dist/commands/startup.js

@@ -10,6 +10,7 @@ import * as fs from 'fs';
 import { execFile } from 'child_process';
 import { getStateManager, getGitHubToken, getCLIVersion, detectGitHubUsername } from '../core/index.js';
 import { errorMessage } from '../core/errors.js';
+import { warn } from '../core/logger.js';
 import { executeDailyCheck } from './daily.js';
 import { launchDashboardServer } from './dashboard-lifecycle.js';
 /**
@@ -53,22 +54,37 @@ export function countIssueListItems(content) {
 export function detectIssueList() {
     let issueListPath = '';
     let source = 'auto-detected';
-    // 1. Check config file for configured path
-    const configPath = '.claude/oss-autopilot/config.md';
-    if (fs.existsSync(configPath)) {
-        try {
-            const configContent = fs.readFileSync(configPath, 'utf-8');
-            const configuredPath = parseIssueListPathFromConfig(configContent);
-            if (configuredPath && fs.existsSync(configuredPath)) {
-                issueListPath = configuredPath;
-                source = 'configured';
-            }
+    // 1. Check state.json config (primary)
+    try {
+        const stateManager = getStateManager();
+        const configuredPath = stateManager.getState().config.issueListPath;
+        if (configuredPath && fs.existsSync(configuredPath)) {
+            issueListPath = configuredPath;
+            source = 'configured';
         }
-        catch (error) {
-            console.error('[STARTUP] Failed to read config:', errorMessage(error));
+    }
+    catch (error) {
+        // State manager may not be initialized yet — fall through to legacy config.md
+        warn('startup', `Could not read issueListPath from state: ${errorMessage(error)}`);
+    }
+    // 2. Fallback: config.md (legacy — will be removed in future)
+    if (!issueListPath) {
+        const configPath = '.claude/oss-autopilot/config.md';
+        if (fs.existsSync(configPath)) {
+            try {
+                const configContent = fs.readFileSync(configPath, 'utf-8');
+                const configuredPath = parseIssueListPathFromConfig(configContent);
+                if (configuredPath && fs.existsSync(configuredPath)) {
+                    issueListPath = configuredPath;
+                    source = 'configured';
+                }
+            }
+            catch (error) {
+                console.error('[STARTUP] Failed to read config:', errorMessage(error));
+            }
         }
     }
-    // 2. Probe known paths
+    // 3. Probe known paths
     if (!issueListPath) {
         const probes = ['open-source/potential-issue-list.md', 'oss/issue-list.md', 'issues.md'];
         for (const probe of probes) {
@@ -81,7 +97,7 @@ export function detectIssueList() {
     }
     if (!issueListPath)
         return undefined;
-    // 3. Count available/completed items
+    // 4. Count available/completed items
     try {
         const content = fs.readFileSync(issueListPath, 'utf-8');
         const { availableCount, completedCount } = countIssueListItems(content);

package/dist/core/ci-analysis.d.ts

@@ -2,6 +2,7 @@
  * CI Analysis - Classification and analysis of CI check runs and combined statuses.
  * Extracted from PRMonitor to isolate CI-related logic (#263).
  */
+import type { Octokit } from '@octokit/rest';
 import { CIFailureCategory, ClassifiedCheck, CIStatusResult } from './types.js';
 /**
  * Classify a failing CI check as actionable, fork_limitation, auth_gate, or infrastructure (#81, #145).
@@ -61,4 +62,9 @@ export declare function analyzeCombinedStatus(combinedStatus: {
  * Priority: failing > pending > passing > unknown.
  */
 export declare function mergeStatuses(checkRunAnalysis: CheckRunAnalysis, combinedAnalysis: CombinedStatusAnalysis, checkRunCount: number): CIStatusResult;
+/**
+ * Get CI status for a commit SHA by querying both the combined status API and check runs API.
+ * Returns the merged status and names of any failing checks for diagnostics.
+ */
+export declare function getCIStatus(octokit: Octokit, owner: string, repo: string, sha: string): Promise<CIStatusResult>;
 export {};

package/dist/core/ci-analysis.js

@@ -2,6 +2,12 @@
  * CI Analysis - Classification and analysis of CI check runs and combined statuses.
  * Extracted from PRMonitor to isolate CI-related logic (#263).
  */
+import { getHttpStatusCode, errorMessage } from './errors.js';
+import { debug, warn } from './logger.js';
+/** Return a fresh unknown CI status (avoids shared mutable state between callers). */
+function unknownCIStatus() {
+    return { status: 'unknown', failingCheckNames: [], failingCheckConclusions: new Map() };
+}
 /**
  * Known CI check name patterns that indicate fork limitations rather than real failures (#81).
  * These are deployment/preview services that require repo-level secrets unavailable in forks.
@@ -119,15 +125,23 @@ export function analyzeCombinedStatus(combinedStatus) {
     const hasRealFailure = realStatuses.some((s) => s.state === 'failure' || s.state === 'error');
     const hasRealPending = realStatuses.some((s) => s.state === 'pending');
     const hasRealSuccess = realStatuses.some((s) => s.state === 'success');
-    const effectiveCombinedState = hasRealFailure
-        ? 'failure'
-        : hasRealPending
-            ? 'pending'
-            : hasRealSuccess
-                ? 'success'
-                : realStatuses.length === 0
-                    ? 'success' // All statuses were auth gates; don't inherit original failure
-                    : combinedStatus.state;
+    let effectiveCombinedState;
+    if (hasRealFailure) {
+        effectiveCombinedState = 'failure';
+    }
+    else if (hasRealPending) {
+        effectiveCombinedState = 'pending';
+    }
+    else if (hasRealSuccess) {
+        effectiveCombinedState = 'success';
+    }
+    else if (realStatuses.length === 0) {
+        // All statuses were auth gates; don't inherit original failure
+        effectiveCombinedState = 'success';
+    }
+    else {
+        effectiveCombinedState = combinedStatus.state;
+    }
     const hasStatuses = combinedStatus.statuses.length > 0;
     // Collect failing status names from combined status API
     const failingStatusNames = [];
@@ -157,9 +171,72 @@ export function mergeStatuses(checkRunAnalysis, combinedAnalysis, checkRunCount)
     if (hasSuccessfulChecks || effectiveCombinedState === 'success') {
         return { status: 'passing', failingCheckNames: [], failingCheckConclusions: new Map() };
     }
-    // No checks found at all - this is common for repos without CI
+    // No checks found at all — common for repos without CI
     if (!hasStatuses && checkRunCount === 0) {
-        return { status: 'unknown', failingCheckNames: [], failingCheckConclusions: new Map() };
+        return unknownCIStatus();
+    }
+    return unknownCIStatus();
+}
+/**
+ * Get CI status for a commit SHA by querying both the combined status API and check runs API.
+ * Returns the merged status and names of any failing checks for diagnostics.
+ */
+export async function getCIStatus(octokit, owner, repo, sha) {
+    if (!sha)
+        return unknownCIStatus();
+    try {
+        // Fetch both combined status and check runs in parallel
+        const [statusResponse, checksResponse] = await Promise.all([
+            octokit.repos.getCombinedStatusForRef({ owner, repo, ref: sha }),
+            // 404 is expected for repos without check runs configured; log other errors for debugging
+            octokit.checks.listForRef({ owner, repo, ref: sha }).catch((err) => {
+                const status = getHttpStatusCode(err);
+                // Rate limit errors must propagate — matches listReviewComments pattern (#481)
+                if (status === 429)
+                    throw err;
+                if (status === 403) {
+                    const msg = errorMessage(err).toLowerCase();
+                    if (msg.includes('rate limit') || msg.includes('abuse detection'))
+                        throw err;
+                }
+                if (status === 404) {
+                    debug('pr-monitor', `Check runs 404 for ${owner}/${repo}@${sha.slice(0, 7)} (no checks configured)`);
+                }
+                else {
+                    warn('pr-monitor', `Non-404 error fetching check runs for ${owner}/${repo}@${sha.slice(0, 7)}: ${status ?? err}`);
+                }
+                return null;
+            }),
+        ]);
+        const combinedStatus = statusResponse.data;
+        const allCheckRuns = checksResponse?.data?.check_runs || [];
+        // Deduplicate check runs by name, keeping only the most recent run per unique name.
+        // GitHub returns all historical runs (including re-runs), so without deduplication
+        // a superseded failure will incorrectly flag the PR as failing even after a re-run passes.
+        const latestCheckRunsByName = new Map();
+        for (const check of allCheckRuns) {
+            const existing = latestCheckRunsByName.get(check.name);
+            if (!existing || new Date(check.started_at ?? 0) > new Date(existing.started_at ?? 0)) {
+                latestCheckRunsByName.set(check.name, check);
+            }
+        }
+        const checkRuns = [...latestCheckRunsByName.values()];
+        const checkRunAnalysis = analyzeCheckRuns(checkRuns);
+        const combinedAnalysis = analyzeCombinedStatus(combinedStatus);
+        return mergeStatuses(checkRunAnalysis, combinedAnalysis, checkRuns.length);
+    }
+    catch (error) {
+        const statusCode = getHttpStatusCode(error);
+        if (statusCode === 401 || statusCode === 403 || statusCode === 429) {
+            throw error;
+        }
+        else if (statusCode === 404) {
+            // Repo might not have CI configured, this is normal
+            debug('pr-monitor', `CI check 404 for ${owner}/${repo} (no CI configured)`);
+        }
+        else {
+            warn('pr-monitor', `Failed to check CI for ${owner}/${repo}@${sha.slice(0, 7)}: ${errorMessage(error)}`);
+        }
+        return unknownCIStatus();
     }
-    return { status: 'unknown', failingCheckNames: [], failingCheckConclusions: new Map() };
 }