@oss-autopilot/core 3.2.0 → 3.3.0

package/dist/commands/check-integration.js

@@ -2,8 +2,8 @@
  * Check integration command (#83)
  * Detects new files in the current branch that aren't referenced elsewhere
  */
-import * as path from 'path';
-import { execFileSync } from 'child_process';
+import * as path from 'node:path';
+import { execFileSync } from 'node:child_process';
 import { debug } from '../core/index.js';
 import { errorMessage } from '../core/errors.js';
 /** File extensions we consider "code" that should be imported/referenced */
@@ -71,8 +71,8 @@ export async function runCheckIntegration(options) {
     let newFiles;
     try {
         const output = execFileSync('git', ['diff', '--name-only', '--diff-filter=A', `${base}...HEAD`], {
-            encoding: 'utf-8',
-            timeout: 10000,
+            encoding: 'utf8',
+            timeout: 10_000,
         }).trim();
         newFiles = output ? output.split('\n').filter(Boolean) : [];
     }
@@ -94,8 +94,8 @@ export async function runCheckIntegration(options) {
     let allFiles;
     try {
         allFiles = execFileSync('git', ['ls-files'], {
-            encoding: 'utf-8',
-            timeout: 10000,
+            encoding: 'utf8',
+            timeout: 10_000,
         })
             .trim()
             .split('\n')
@@ -124,8 +124,8 @@ export async function runCheckIntegration(options) {
         for (const pattern of patterns) {
             try {
                 const grepOutput = execFileSync('git', ['grep', '-l', '--', pattern], {
-                    encoding: 'utf-8',
-                    timeout: 10000,
+                    encoding: 'utf8',
+                    timeout: 10_000,
                 }).trim();
                 if (grepOutput) {
                     const matches = grepOutput.split('\n').filter((f) => f !== newFile);

package/dist/commands/comments.js

@@ -7,6 +7,7 @@ import { ValidationError } from '../core/errors.js';
 import { warn } from '../core/logger.js';
 const MODULE = 'comments';
 import { paginateAll } from '../core/pagination.js';
+import { buildStalenessWarning } from '../formatters/json.js';
 import { validateUrl, validateMessage, validateGitHubUrl, PR_URL_PATTERN, ISSUE_OR_PR_URL_PATTERN, ISSUE_URL_PATTERN, } from './validation.js';
 /**
  * Fetch all comments, reviews, and inline review comments for a PR.
@@ -76,6 +77,7 @@ export async function runComments(options) {
     const relevantReviews = reviews
         .filter((r) => filterComment(r) && r.body && r.body.trim())
         .sort((a, b) => new Date(b.submitted_at || 0).getTime() - new Date(a.submitted_at || 0).getTime());
+    const staleness = stateManager.getStateStaleness();
     return {
         pr: {
             title: pr.title,
@@ -107,6 +109,7 @@ export async function runComments(options) {
             inlineCommentCount: relevantReviewComments.length,
             discussionCommentCount: relevantIssueComments.length,
         },
+        ...(staleness ? { warnings: [buildStalenessWarning(staleness)] } : {}),
     };
 }
 /**

package/dist/commands/config.js

@@ -44,25 +44,29 @@ export async function runConfig(options) {
     const value = options.value;
     // Handle specific config keys
     switch (options.key) {
-        case 'username':
+        case 'username': {
             stateManager.updateConfig({ githubUsername: validateGitHubUsername(value) });
             break;
-        case 'add-language':
+        }
+        case 'add-language': {
             if (!currentConfig.languages.includes(value)) {
                 stateManager.updateConfig({ languages: [...currentConfig.languages, value] });
             }
             break;
-        case 'add-label':
+        }
+        case 'add-label': {
             if (!currentConfig.labels.includes(value)) {
                 stateManager.updateConfig({ labels: [...currentConfig.labels, value] });
             }
             break;
-        case 'remove-label':
+        }
+        case 'remove-label': {
             if (!currentConfig.labels.includes(value)) {
                 throw new Error(`Label "${value}" is not currently configured. Current labels: ${currentConfig.labels.join(', ')}`);
             }
             stateManager.updateConfig({ labels: currentConfig.labels.filter((l) => l !== value) });
             break;
+        }
         case 'add-scope': {
             const scope = validateScope(value);
             const currentScopes = currentConfig.scope ?? [];
@@ -111,9 +115,10 @@ export async function runConfig(options) {
             }
             break;
         }
-        case 'issueListPath':
+        case 'issueListPath': {
             stateManager.updateConfig({ issueListPath: value || undefined });
             break;
+        }
         case 'diffTool': {
             if (!DIFF_TOOLS.includes(value)) {
                 throw new Error(`Invalid diffTool "${value}". Valid options: ${DIFF_TOOLS.join(', ')}`);
@@ -121,13 +126,15 @@ export async function runConfig(options) {
             stateManager.updateConfig({ diffTool: value });
             break;
         }
-        case 'diffToolCustomCommand':
+        case 'diffToolCustomCommand': {
             stateManager.updateConfig({
                 diffToolCustomCommand: value || undefined,
             });
             break;
-        default:
+        }
+        default: {
             throw new ValidationError(formatUnknownKeyError(options.key, 'config'));
+        }
     }
     return { success: true, key: options.key, value };
 }

package/dist/commands/daily-render.js

@@ -20,16 +20,21 @@ import { formatRelativeTime } from '../core/dates.js';
  */
 export function formatActionHint(hint) {
     switch (hint) {
-        case 'demo_requested':
+        case 'demo_requested': {
             return 'demo/screenshot requested';
-        case 'tests_requested':
+        }
+        case 'tests_requested': {
             return 'tests requested';
-        case 'changes_requested':
+        }
+        case 'changes_requested': {
             return 'code changes requested';
-        case 'docs_requested':
+        }
+        case 'docs_requested': {
             return 'documentation requested';
-        case 'rebase_requested':
+        }
+        case 'rebase_requested': {
             return 'rebase requested';
+        }
     }
 }
 /**

package/dist/commands/daily.js

@@ -12,7 +12,7 @@ import { warn } from '../core/logger.js';
 import { emptyPRCountsResult } from '../core/github-stats.js';
 import { createAutopilotScout } from './scout-bridge.js';
 import { updateMonthlyAnalytics } from './dashboard-data.js';
-import { deduplicateDigest, compactActionableIssues, compactRepoGroups, } from '../formatters/json.js';
+import { deduplicateDigest, compactActionableIssues, compactRepoGroups, buildStalenessWarning, } from '../formatters/json.js';
 const MODULE = 'daily';
 /**
  * Record a non-fatal failure: push a structured entry into the run's warnings
@@ -504,6 +504,11 @@ async function executeDailyCheckInternal(token) {
     // One collector shared by every phase — threaded through explicitly so the
     // callgraph documents which phases can produce non-fatal warnings. See #1042.
     const warnings = [];
+    // Surface Gist staleness up-front so consumers see it even if Phase 1 fails (#1193).
+    const staleness = getStateManager().getStateStaleness();
+    if (staleness) {
+        warnings.push(buildStalenessWarning(staleness));
+    }
     // Phase 1: Fetch all PR data from GitHub
     const { prs, failures, mergedCounts, closedCounts, monthlyCounts, monthlyClosedCounts, openedFromMerged, openedFromClosed, recentlyClosedPRs, recentlyMergedPRs, commentedIssues, } = await fetchPRData(prMonitor, token, warnings);
     // Phase 2: Update repo scores (signals, star counts, trust sync)

package/dist/commands/dashboard-lifecycle.js

@@ -3,7 +3,7 @@
  * Handles launching the interactive SPA dashboard as a background process
  * and detecting whether a server is already running.
  */
-import { spawn } from 'child_process';
+import { spawn } from 'node:child_process';
 import { findRunningDashboardServer, isDashboardServerRunning, readDashboardServerInfo, removeDashboardServerInfo, } from './dashboard-process.js';
 import { resolveAssetsDir } from './dashboard.js';
 import { getCLIVersion } from '../core/index.js';

package/dist/commands/dashboard-process.js

@@ -2,9 +2,9 @@
  * Dashboard server process management.
  * PID file operations, health probes, and running server detection.
  */
-import * as http from 'http';
-import * as fs from 'fs';
-import * as path from 'path';
+import * as http from 'node:http';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
 import { getDataDir } from '../core/index.js';
 import { warn } from '../core/logger.js';
 const MODULE = 'dashboard-server';
@@ -17,7 +17,7 @@ export function writeDashboardServerInfo(info) {
 }
 export function readDashboardServerInfo() {
     try {
-        const content = fs.readFileSync(getDashboardPidPath(), 'utf-8');
+        const content = fs.readFileSync(getDashboardPidPath(), 'utf8');
         const parsed = JSON.parse(content);
         if (typeof parsed !== 'object' ||
             parsed === null ||

package/dist/commands/dashboard-server.js

@@ -5,10 +5,10 @@
  *
  * Uses Node's built-in http module — no Express/Fastify.
  */
-import * as http from 'http';
-import * as fs from 'fs';
-import * as path from 'path';
-import * as crypto from 'crypto';
+import * as http from 'node:http';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import * as crypto from 'node:crypto';
 import { getStateManager, getGitHubToken, getCLIVersion, applyStatusOverrides } from '../core/index.js';
 import { errorMessage, ValidationError } from '../core/errors.js';
 import { warn } from '../core/logger.js';
@@ -44,7 +44,7 @@ function readVettedIssues() {
         const info = detectIssueList();
         if (!info)
             return null;
-        const content = fs.readFileSync(info.path, 'utf-8');
+        const content = fs.readFileSync(info.path, 'utf8');
         return parseIssueList(content);
     }
     catch (error) {
@@ -148,7 +148,7 @@ function readBody(req, maxBytes = MAX_BODY_BYTES) {
         });
         req.on('end', () => {
             if (!aborted)
-                resolve(Buffer.concat(chunks).toString('utf-8'));
+                resolve(Buffer.concat(chunks).toString('utf8'));
         });
         req.on('error', (err) => {
             if (!aborted)
@@ -599,6 +599,7 @@ export async function startDashboardServer(options) {
             cachedJsonData = buildDashboardJson(cachedDigest, stateManager.getState(), cachedCommentedIssues, result.allMergedPRs, result.allClosedPRs, cachedPartialFailures);
             cachedIssueListMtimeMs = getIssueListMtimeMs();
             warn(MODULE, 'Background data refresh complete');
+            return;
         })
             .catch((error) => {
             warn(MODULE, `Background data refresh failed (serving cached data): ${errorMessage(error)}`);

package/dist/commands/dashboard.js

@@ -1,8 +1,8 @@
 /**
  * Dashboard command — serves the interactive Preact SPA dashboard.
  */
-import * as fs from 'fs';
-import * as path from 'path';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
 import { getGitHubToken } from '../core/index.js';
 /**
  * Resolve the SPA assets directory from packages/dashboard/dist/.

package/dist/commands/detect-formatters.js

@@ -3,8 +3,8 @@
  * Scans a local repository for configured formatters/linters.
  * Optionally diagnoses CI log output for formatting failures.
  */
-import * as fs from 'fs';
-import * as path from 'path';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
 import { detectFormatters, diagnoseCIFormatterFailure } from '../core/formatter-detection.js';
 import { errorMessage } from '../core/errors.js';
 export async function runDetectFormatters(options) {
@@ -13,7 +13,7 @@ export async function runDetectFormatters(options) {
     if (options.ciLog) {
         let logContent;
         try {
-            logContent = fs.readFileSync(path.resolve(options.ciLog), 'utf-8');
+            logContent = fs.readFileSync(path.resolve(options.ciLog), 'utf8');
         }
         catch (err) {
             throw new Error(`Failed to read CI log file: ${errorMessage(err)}`, { cause: err });

package/dist/commands/doctor.js

@@ -9,10 +9,10 @@
  * Each `check*` function is exported individually so tests can mock its
  * external dependencies in isolation.
  */
-import * as fs from 'fs';
-import * as path from 'path';
-import { execFile } from 'child_process';
-import { promisify } from 'util';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import { execFile } from 'node:child_process';
+import { promisify } from 'node:util';
 import { getGitHubTokenAsync, getOctokit, getStatePath } from '../core/index.js';
 import { AgentStateSchema } from '../core/state-schema.js';
 import { errorMessage } from '../core/errors.js';
@@ -195,7 +195,7 @@ export function checkStateFile(options) {
         };
     }
     try {
-        const raw = fs.readFileSync(statePath, 'utf-8');
+        const raw = fs.readFileSync(statePath, 'utf8');
         const parsed = JSON.parse(raw);
         const result = AgentStateSchema.safeParse(parsed);
         if (!result.success) {

package/dist/commands/guidelines.js

@@ -11,7 +11,7 @@
  * Standalone-mode users see "not available" on store/reset/fetch-corpus
  * because per-repo guidelines require Gist persistence to be useful.
  */
-import { getStateManager, requireGitHubToken, getOctokit, GuidelinesNotAvailableError } from '../core/index.js';
+import { getStateManager, requireGitHubToken, getOctokit, GuidelinesNotAvailableError, maybeCheckpoint, } from '../core/index.js';
 import { fetchPRCommentBundlesBatch } from '../core/pr-comments-fetcher.js';
 import { warn } from '../core/logger.js';
 const MODULE = 'guidelines';
@@ -36,7 +36,7 @@ export async function runGuidelinesView(options) {
     return {
         repo: options.repo,
         content,
-        byteSize: content === null ? 0 : Buffer.byteLength(content, 'utf-8'),
+        byteSize: content === null ? 0 : Buffer.byteLength(content, 'utf8'),
         exists: content !== null,
         storageMode: sm.isGuidelinesAvailable() ? 'gist' : 'local-unavailable',
     };
@@ -54,9 +54,13 @@ export async function runGuidelinesStore(options) {
     const sm = getStateManager();
     // Throws GuidelinesNotAvailableError or GuidelinesTooLargeError on failure.
     sm.setGuidelines(options.repo, options.content);
+    // Push to Gist — autoSave only writes the local state-cache mirror in Gist
+    // mode, so without this checkpoint the change never propagates across
+    // machines (#1200).
+    await maybeCheckpoint(sm, MODULE);
     return {
         repo: options.repo,
-        byteSize: Buffer.byteLength(options.content, 'utf-8'),
+        byteSize: Buffer.byteLength(options.content, 'utf8'),
         stored: true,
     };
 }
@@ -70,6 +74,8 @@ export async function runGuidelinesReset(options) {
     const existed = sm.getGuidelines(options.repo) !== null;
     if (existed) {
         sm.deleteGuidelines(options.repo);
+        // Push to Gist — see runGuidelinesStore note (#1200).
+        await maybeCheckpoint(sm, MODULE);
     }
     return { repo: options.repo, deleted: existed };
 }
@@ -131,6 +137,12 @@ export async function runFetchCorpus(options) {
     for (const bundle of bundles) {
         sm.markPRCommentsFetched(bundle.prUrl, now);
     }
+    // Push commentsFetchedAt stamps to Gist so other machines don't re-fetch
+    // the same PRs forever. autoSave only writes the local mirror in Gist
+    // mode (#1200).
+    if (bundles.length > 0) {
+        await maybeCheckpoint(sm, MODULE);
+    }
     return {
         repo: options.repo,
         bundles,

package/dist/commands/list-move-tier.js

@@ -10,8 +10,8 @@
  *
  * No GitHub calls — pure read/transform/write of a local file.
  */
-import * as fs from 'fs';
-import * as path from 'path';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
 import { errorMessage } from '../core/errors.js';
 const TIER_HEADERS = {
     pursue: '## Pursue',
@@ -47,7 +47,7 @@ function findIssueBlocks(lines, issueUrl) {
     for (let i = 0; i < lines.length; i++) {
         const line = lines[i];
         // Top-level list item — `- `, `* `, `+ `, or `1.` at the start (with no leading whitespace).
-        const isTopLevelListItem = /^[-*+]\s|^\d+\.\s/.test(line);
+        const isTopLevelListItem = /^[*+-]\s|^\d+\.\s/.test(line);
         if (!isTopLevelListItem || !line.includes(issueUrl))
             continue;
         // Capture indented sub-bullets that follow this line.
@@ -166,7 +166,7 @@ export async function runListMoveTier(options) {
     }
     let content;
     try {
-        content = fs.readFileSync(filePath, 'utf-8');
+        content = fs.readFileSync(filePath, 'utf8');
     }
     catch (error) {
         throw new Error(`Failed to read file: ${errorMessage(error)}`, { cause: error });
@@ -174,7 +174,7 @@ export async function runListMoveTier(options) {
     const result = moveIssueToTier(content, options.issueUrl, options.tier);
     if (result.moved) {
         try {
-            fs.writeFileSync(filePath, result.content, 'utf-8');
+            fs.writeFileSync(filePath, result.content, 'utf8');
         }
         catch (error) {
             throw new Error(`Failed to write file: ${errorMessage(error)}`, { cause: error });

package/dist/commands/local-repos.js

@@ -2,10 +2,10 @@
  * Local repos command (#84)
  * Scans configurable directories for local git clones and caches results
  */
-import * as fs from 'fs';
-import * as path from 'path';
-import * as os from 'os';
-import { execFileSync } from 'child_process';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import * as os from 'node:os';
+import { execFileSync } from 'node:child_process';
 import { getStateManager, debug } from '../core/index.js';
 import { errorMessage } from '../core/errors.js';
 /** Default directories to scan for local clones */
@@ -21,7 +21,7 @@ const DEFAULT_SCAN_PATHS = [
 function getGitHubRemote(repoPath) {
     try {
         const remoteUrl = execFileSync('git', ['-C', repoPath, 'remote', 'get-url', 'origin'], {
-            encoding: 'utf-8',
+            encoding: 'utf8',
             timeout: 5000,
             stdio: ['pipe', 'pipe', 'pipe'],
         }).trim();
@@ -30,7 +30,7 @@ function getGitHubRemote(repoPath) {
         if (httpsMatch)
             return httpsMatch[1];
         // Match SSH: git@github.com:owner/repo.git
-        const sshMatch = remoteUrl.match(/github\.com[:/]([^/]+\/[^/]+?)(?:\.git)?$/);
+        const sshMatch = remoteUrl.match(/github\.com[/:]([^/]+\/[^/]+?)(?:\.git)?$/);
         if (sshMatch)
             return sshMatch[1];
         return null;
@@ -45,7 +45,7 @@ function getGitHubRemote(repoPath) {
 function getCurrentBranch(repoPath) {
     try {
         return (execFileSync('git', ['-C', repoPath, 'branch', '--show-current'], {
-            encoding: 'utf-8',
+            encoding: 'utf8',
             timeout: 5000,
             stdio: ['pipe', 'pipe', 'pipe'],
         }).trim() || null);
@@ -66,8 +66,8 @@ export function scanForRepos(scanPaths) {
         let gitDirs;
         try {
             const output = execFileSync('find', [scanPath, '-maxdepth', '4', '-name', '.git', '-type', 'd'], {
-                encoding: 'utf-8',
-                timeout: 30000,
+                encoding: 'utf8',
+                timeout: 30_000,
                 stdio: ['pipe', 'pipe', 'pipe'],
             }).trim();
             gitDirs = output ? output.split('\n').filter(Boolean) : [];

package/dist/commands/parse-list.js

@@ -2,8 +2,8 @@
  * Parse issue list command (#82)
  * Parses markdown issue lists into structured JSON with tier classification
  */
-import * as fs from 'fs';
-import * as path from 'path';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
 import { errorMessage } from '../core/errors.js';
 /** Extract GitHub issue/PR URLs from a markdown line */
 function extractGitHubUrl(line) {
@@ -26,11 +26,11 @@ function extractTitle(line) {
     // Remove list markers (-, *, +, numbered)
     cleaned = cleaned.replace(/^\s*[-*+]\s*/, '').replace(/^\s*\d+\.\s*/, '');
     // Remove checkboxes
-    cleaned = cleaned.replace(/\[[ xX]\]\s*/, '');
+    cleaned = cleaned.replace(/\[[ x]\]\s*/i, '');
     // Remove strikethrough markers
     cleaned = cleaned.replace(/~~/g, '');
     // Remove "Done" markers
-    cleaned = cleaned.replace(/\b(Done|DONE|done)\b/g, '');
+    cleaned = cleaned.replace(/\bdone\b/gi, '');
     // Remove leading/trailing punctuation and whitespace
     cleaned = cleaned.replace(/^[\s\-\u2013\u2014:]+/, '').replace(/[\s\-\u2013\u2014:]+$/, '');
     return cleaned.trim();
@@ -41,7 +41,7 @@ function isCompleted(line) {
     if (/~~.+~~/.test(line))
         return true;
     // Checked checkbox: [x] or [X]
-    if (/\[[xX]\]/.test(line))
+    if (/\[x\]/i.test(line))
         return true;
     // "Done" marker (standalone word, case insensitive)
     if (/\bdone\b/i.test(line))
@@ -55,11 +55,11 @@ function extractScore(line) {
 }
 /** Check if a sub-bullet indicates the item is terminal (completed/abandoned — safe to prune) */
 function isSubBulletTerminal(line) {
-    return /\*\*(Skip|Done|Dropped|Merged|Closed)\*\*/i.test(line);
+    return /\*\*(?:Skip|Done|Dropped|Merged|Closed)\*\*/i.test(line);
 }
 /** Check if a sub-bullet indicates the item is in-progress (not available, but NOT safe to prune) */
 function isSubBulletInProgress(line) {
-    return /\*\*(In Progress|Wait|Waiting)\*\*/i.test(line);
+    return /\*\*(?:In Progress|Wait|Waiting)\*\*/i.test(line);
 }
 /** Parse a markdown string into structured issue items */
 export function parseIssueList(content) {
@@ -79,7 +79,7 @@ export function parseIssueList(content) {
             continue;
         }
         // Skip empty lines and non-list items
-        if (!line.trim() || !/^\s*[-*+]|\s*\d+\.|\s*\[[ xX]\]/.test(line)) {
+        if (!line.trim() || !/^\s*[-*+]|\s*\d+\.|\s*\[[ x]\]/i.test(line)) {
             continue;
         }
         // Extract GitHub URL -- skip lines without one
@@ -193,7 +193,7 @@ export function pruneIssueList(content, minScore = 6) {
         if (/^---\s*$/.test(line)) {
             continue;
         }
-        if (/^\s*(###?\s*)?(Removed|Previously dropped)/i.test(line)) {
+        if (/^\s*(?:###?\s*)?(?:Removed|Previously dropped)/i.test(line)) {
             continue;
         }
         // Skip blockquote metadata lines ("> Sources: ...", "> Prioritized ...")
@@ -242,7 +242,7 @@ export async function runParseList(options) {
     }
     let content;
     try {
-        content = fs.readFileSync(filePath, 'utf-8');
+        content = fs.readFileSync(filePath, 'utf8');
     }
     catch (error) {
         const msg = errorMessage(error);

package/dist/commands/scout-bridge.js

@@ -44,8 +44,8 @@ export function buildScoutState() {
             maxIssueAgeDays: config.maxIssueAgeDays,
             includeDocIssues: config.includeDocIssues,
             minRepoScoreThreshold: config.minRepoScoreThreshold,
-            interPhaseDelayMs: 30000,
-            broadPhaseDelayMs: 90000,
+            interPhaseDelayMs: 30_000,
+            broadPhaseDelayMs: 90_000,
             skipBroadWhenSufficientResults: 15,
             persistence: config.persistence,
             slmTriageModel: config.slmTriageModel,