@oss-autopilot/core 0.53.1 → 0.55.0

package/dist/commands/init.js

@@ -9,7 +9,6 @@ export async function runInit(options) {
     const stateManager = getStateManager();
     // Set username in config
     stateManager.updateConfig({ githubUsername: options.username });
-    stateManager.save();
     return {
         username: options.username,
         message: 'Username saved. Run `daily` to fetch your open PRs from GitHub.',

package/dist/commands/local-repos.js

@@ -113,11 +113,10 @@ export async function runLocalRepos(options) {
     const cachedAt = new Date().toISOString();
     try {
         stateManager.setLocalRepoCache({ repos, scanPaths, cachedAt });
-        stateManager.save();
     }
     catch (error) {
         const msg = errorMessage(error);
-        console.error(`Warning: Failed to cache scan results: ${msg}`);
+        console.error(`Warning: Failed to cache scan results to disk: ${msg}`);
     }
     return {
         repos,

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 {};