@oss-autopilot/core 0.41.0 → 0.42.1

package/dist/commands/check-integration.d.ts

@@ -2,9 +2,9 @@
  * Check integration command (#83)
  * Detects new files in the current branch that aren't referenced elsewhere
  */
+import type { CheckIntegrationOutput, NewFileInfo } from '../formatters/json.js';
 interface CheckIntegrationOptions {
     base: string;
-    json?: boolean;
 }
-export declare function runCheckIntegration(options: CheckIntegrationOptions): Promise<void>;
-export {};
+export type { CheckIntegrationOutput, NewFileInfo };
+export declare function runCheckIntegration(options: CheckIntegrationOptions): Promise<CheckIntegrationOutput>;

package/dist/commands/check-integration.js

@@ -4,8 +4,8 @@
  */
 import * as path from 'path';
 import { execFileSync } from 'child_process';
-import { outputJson, outputJsonError } from '../formatters/json.js';
 import { debug } from '../core/index.js';
+import { errorMessage } from '../core/errors.js';
 /** File extensions we consider "code" that should be imported/referenced */
 const CODE_EXTENSIONS = new Set([
     '.ts',
@@ -77,14 +77,8 @@ export async function runCheckIntegration(options) {
         newFiles = output ? output.split('\n').filter(Boolean) : [];
     }
     catch (error) {
-        const msg = error instanceof Error ? error.message : String(error);
-        if (options.json) {
-            outputJsonError(`Failed to run git diff: ${msg}`);
-        }
-        else {
-            console.error(`Error: Failed to run git diff: ${msg}`);
-        }
-        process.exit(1);
+        const msg = errorMessage(error);
+        throw new Error(`Failed to run git diff: ${msg}`, { cause: error });
     }
     // Filter to code files, excluding tests, configs, etc.
     const codeFiles = newFiles.filter((f) => {
@@ -94,14 +88,7 @@ export async function runCheckIntegration(options) {
         return !IGNORED_PATTERNS.some((p) => p.test(f));
     });
     if (codeFiles.length === 0) {
-        const result = { newFiles: [], unreferencedCount: 0 };
-        if (options.json) {
-            outputJson(result);
-        }
-        else {
-            console.log('\nNo new code files to check.');
-        }
-        return;
+        return { newFiles: [], unreferencedCount: 0 };
     }
     // Get all tracked files in the repo for reference checking
     let allFiles;
@@ -115,7 +102,7 @@ export async function runCheckIntegration(options) {
             .filter(Boolean);
     }
     catch (err) {
-        // git ls-files failed (e.g. not a git repo) — proceed without reference list
+        // git ls-files failed (e.g. not a git repo) -- proceed without reference list
         debug('check-integration', 'git ls-files failed, reference checking will be skipped', err);
         allFiles = [];
     }
@@ -147,10 +134,10 @@ export async function runCheckIntegration(options) {
             }
             catch (error) {
                 // git grep exit code 1 = no matches (expected), exit code 2+ = real error
-                const exitCode = error && typeof error === 'object' && 'status' in error ? error.status : null;
-                if (exitCode !== null && exitCode !== 1) {
-                    const msg = error instanceof Error ? error.message : String(error);
-                    console.error(`Warning: git grep failed for "${pattern}": ${msg}`);
+                const exitCode = error && typeof error === 'object' && 'status' in error ? error.status : undefined;
+                if (exitCode !== undefined && exitCode !== 1) {
+                    const msg = errorMessage(error);
+                    debug('check-integration', `git grep failed for "${pattern}": ${msg}`);
                 }
             }
         }
@@ -168,25 +155,5 @@ export async function runCheckIntegration(options) {
         results.push(info);
     }
     const unreferencedCount = results.filter((r) => !r.isIntegrated).length;
-    const output = { newFiles: results, unreferencedCount };
-    if (options.json) {
-        outputJson(output);
-    }
-    else {
-        console.log(`\n🔍 Integration Check (base: ${base})\n`);
-        console.log(`New files: ${results.length} | Unreferenced: ${unreferencedCount}\n`);
-        for (const file of results) {
-            const status = file.isIntegrated ? '✅' : '⚠️';
-            console.log(`${status} ${file.path}`);
-            if (file.isIntegrated) {
-                console.log(`   Referenced by: ${file.referencedBy.join(', ')}`);
-            }
-            else {
-                console.log('   Not referenced by any file');
-                if (file.suggestedEntryPoints && file.suggestedEntryPoints.length > 0) {
-                    console.log(`   Suggested entry points: ${file.suggestedEntryPoints.join(', ')}`);
-                }
-            }
-        }
-    }
+    return { newFiles: results, unreferencedCount };
 }

package/dist/commands/comments.d.ts

@@ -2,23 +2,20 @@
  * Comments, Post, and Claim commands
  * Handles GitHub comment interactions
  */
+import { type CommentsOutput, type PostOutput, type ClaimOutput } from '../formatters/json.js';
+export { type CommentsOutput, type PostOutput, type ClaimOutput } from '../formatters/json.js';
 interface CommentsOptions {
     prUrl: string;
     showBots?: boolean;
-    json?: boolean;
 }
 interface PostOptions {
     url: string;
-    message?: string;
-    stdin?: boolean;
-    json?: boolean;
+    message: string;
 }
 interface ClaimOptions {
     issueUrl: string;
     message?: string;
-    json?: boolean;
 }
-export declare function runComments(options: CommentsOptions): Promise<void>;
-export declare function runPost(options: PostOptions): Promise<void>;
-export declare function runClaim(options: ClaimOptions): Promise<void>;
-export {};
+export declare function runComments(options: CommentsOptions): Promise<CommentsOutput>;
+export declare function runPost(options: PostOptions): Promise<PostOutput>;
+export declare function runClaim(options: ClaimOptions): Promise<ClaimOutput>;

package/dist/commands/comments.js

@@ -2,54 +2,46 @@
  * Comments, Post, and Claim commands
  * Handles GitHub comment interactions
  */
-import { getStateManager, getOctokit, parseGitHubUrl, formatRelativeTime, getGitHubToken } from '../core/index.js';
+import { getStateManager, getOctokit, parseGitHubUrl, requireGitHubToken } from '../core/index.js';
 import { paginateAll } from '../core/pagination.js';
-import { outputJson, outputJsonError } from '../formatters/json.js';
 import { validateUrl, validateMessage } from './validation.js';
 export async function runComments(options) {
     validateUrl(options.prUrl);
-    // Token is guaranteed by the preAction hook in cli.ts for non-LOCAL_ONLY_COMMANDS.
-    const token = getGitHubToken();
+    const token = requireGitHubToken();
     const stateManager = getStateManager();
     const octokit = getOctokit(token);
     // Parse PR URL
     const parsed = parseGitHubUrl(options.prUrl);
     if (!parsed || parsed.type !== 'pull') {
-        if (options.json) {
-            outputJsonError('Invalid PR URL format');
-        }
-        else {
-            console.error('Invalid PR URL format');
-        }
-        process.exit(1);
+        throw new Error('Invalid PR URL format');
     }
     const { owner, repo, number: pull_number } = parsed;
     // Get PR details
     const { data: pr } = await octokit.pulls.get({ owner, repo, pull_number });
-    // Get review comments (inline code comments)
-    const reviewComments = await paginateAll((page) => octokit.pulls.listReviewComments({
-        owner,
-        repo,
-        pull_number,
-        per_page: 100,
-        page,
-    }));
-    // Get issue comments (general PR discussion)
-    const issueComments = await paginateAll((page) => octokit.issues.listComments({
-        owner,
-        repo,
-        issue_number: pull_number,
-        per_page: 100,
-        page,
-    }));
-    // Get reviews
-    const reviews = await paginateAll((page) => octokit.pulls.listReviews({
-        owner,
-        repo,
-        pull_number,
-        per_page: 100,
-        page,
-    }));
+    // Fetch review comments, issue comments, and reviews in parallel
+    const [reviewComments, issueComments, reviews] = await Promise.all([
+        paginateAll((page) => octokit.pulls.listReviewComments({
+            owner,
+            repo,
+            pull_number,
+            per_page: 100,
+            page,
+        })),
+        paginateAll((page) => octokit.issues.listComments({
+            owner,
+            repo,
+            issue_number: pull_number,
+            per_page: 100,
+            page,
+        })),
+        paginateAll((page) => octokit.pulls.listReviews({
+            owner,
+            repo,
+            pull_number,
+            per_page: 100,
+            page,
+        })),
+    ]);
     // Filter out own comments, optionally show bots
     const username = stateManager.getState().config.githubUsername;
     const filterComment = (c) => {
@@ -70,242 +62,100 @@ 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());
-    if (options.json) {
-        outputJson({
-            pr: {
-                title: pr.title,
-                state: pr.state,
-                mergeable: pr.mergeable,
-                head: pr.head.ref,
-                base: pr.base.ref,
-                url: pr.html_url,
-            },
-            reviews: relevantReviews.map((r) => ({
-                user: r.user?.login,
-                state: r.state,
-                body: r.body,
-                submittedAt: r.submitted_at,
-            })),
-            reviewComments: relevantReviewComments.map((c) => ({
-                user: c.user?.login,
-                body: c.body,
-                path: c.path,
-                createdAt: c.created_at,
-            })),
-            issueComments: relevantIssueComments.map((c) => ({
-                user: c.user?.login,
-                body: c.body,
-                createdAt: c.created_at,
-            })),
-            summary: {
-                reviewCount: relevantReviews.length,
-                inlineCommentCount: relevantReviewComments.length,
-                discussionCommentCount: relevantIssueComments.length,
-            },
-        });
-        return;
-    }
-    // Text output
-    console.log(`\n💬 Fetching comments for: ${options.prUrl}\n`);
-    console.log(`## ${pr.title}\n`);
-    console.log(`**Status:** ${pr.state} | **Mergeable:** ${pr.mergeable ?? 'checking...'}`);
-    console.log(`**Branch:** ${pr.head.ref} → ${pr.base.ref}`);
-    console.log(`**URL:** ${pr.html_url}\n`);
-    if (relevantReviews.length > 0) {
-        console.log('### Reviews (newest first)\n');
-        for (const review of relevantReviews) {
-            const state = review.state === 'APPROVED' ? '✅' : review.state === 'CHANGES_REQUESTED' ? '❌' : '💬';
-            const time = review.submitted_at ? formatRelativeTime(review.submitted_at) : '';
-            console.log(`${state} **@${review.user?.login}** (${review.state}) - ${time}`);
-            if (review.body) {
-                console.log(`> ${review.body.split('\n').join('\n> ')}\n`);
-            }
-        }
-    }
-    if (relevantReviewComments.length > 0) {
-        console.log('### Inline Comments (newest first)\n');
-        for (const comment of relevantReviewComments) {
-            const time = formatRelativeTime(comment.created_at);
-            console.log(`**@${comment.user?.login}** on \`${comment.path}\` - ${time}`);
-            console.log(`> ${comment.body.split('\n').join('\n> ')}`);
-            if (comment.diff_hunk) {
-                console.log(`\`\`\`diff\n${comment.diff_hunk.slice(-500)}\n\`\`\``);
-            }
-            console.log('');
-        }
-    }
-    if (relevantIssueComments.length > 0) {
-        console.log('### Discussion (newest first)\n');
-        for (const comment of relevantIssueComments) {
-            const time = formatRelativeTime(comment.created_at);
-            console.log(`**@${comment.user?.login}** - ${time}`);
-            console.log(`> ${comment.body?.split('\n').join('\n> ')}\n`);
-        }
-    }
-    if (relevantReviewComments.length === 0 && relevantIssueComments.length === 0 && relevantReviews.length === 0) {
-        console.log('No comments from other users.\n');
-    }
-    console.log('---');
-    console.log(`**Summary:** ${relevantReviews.length} reviews, ${relevantReviewComments.length} inline comments, ${relevantIssueComments.length} discussion comments`);
+    return {
+        pr: {
+            title: pr.title,
+            state: pr.state,
+            mergeable: pr.mergeable,
+            head: pr.head.ref,
+            base: pr.base.ref,
+            url: pr.html_url,
+        },
+        reviews: relevantReviews.map((r) => ({
+            user: r.user?.login,
+            state: r.state,
+            body: r.body ?? null,
+            submittedAt: r.submitted_at ?? null,
+        })),
+        reviewComments: relevantReviewComments.map((c) => ({
+            user: c.user?.login,
+            body: c.body,
+            path: c.path,
+            createdAt: c.created_at,
+        })),
+        issueComments: relevantIssueComments.map((c) => ({
+            user: c.user?.login,
+            body: c.body,
+            createdAt: c.created_at,
+        })),
+        summary: {
+            reviewCount: relevantReviews.length,
+            inlineCommentCount: relevantReviewComments.length,
+            discussionCommentCount: relevantIssueComments.length,
+        },
+    };
 }
 export async function runPost(options) {
     validateUrl(options.url);
-    // Token is guaranteed by the preAction hook in cli.ts for non-LOCAL_ONLY_COMMANDS.
-    const token = getGitHubToken();
-    let message = options.message;
-    // Read from stdin if specified
-    if (options.stdin) {
-        const chunks = [];
-        for await (const chunk of process.stdin) {
-            chunks.push(chunk);
-        }
-        message = Buffer.concat(chunks).toString('utf-8').trim();
-    }
-    if (!message) {
-        if (options.json) {
-            outputJsonError('No message provided');
-        }
-        else {
-            console.error('Error: No message provided');
-        }
-        process.exit(1);
-    }
-    try {
-        validateMessage(message);
-    }
-    catch (error) {
-        if (options.json) {
-            outputJsonError(error instanceof Error ? error.message : 'Invalid message');
-        }
-        else {
-            console.error(`Error: ${error instanceof Error ? error.message : 'Invalid message'}`);
-        }
-        process.exit(1);
+    if (!options.message.trim()) {
+        throw new Error('No message provided');
     }
+    validateMessage(options.message);
+    const token = requireGitHubToken();
     // Parse URL
     const parsed = parseGitHubUrl(options.url);
     if (!parsed) {
-        if (options.json) {
-            outputJsonError('Invalid GitHub URL format');
-        }
-        else {
-            console.error('Invalid GitHub URL format');
-        }
-        process.exit(1);
+        throw new Error('Invalid GitHub URL format');
     }
     const { owner, repo, number } = parsed;
     const octokit = getOctokit(token);
-    if (!options.json) {
-        console.log('\n📝 Posting comment to:', options.url);
-        console.log('---');
-        console.log(message);
-        console.log('---\n');
-    }
-    try {
-        const { data: comment } = await octokit.issues.createComment({
-            owner,
-            repo,
-            issue_number: number,
-            body: message,
-        });
-        if (options.json) {
-            outputJson({
-                commentUrl: comment.html_url,
-                url: options.url,
-            });
-        }
-        else {
-            console.log('✅ Comment posted successfully!');
-            console.log(`   ${comment.html_url}`);
-        }
-    }
-    catch (error) {
-        if (options.json) {
-            outputJsonError(error instanceof Error ? error.message : 'Unknown error');
-        }
-        else {
-            console.error('❌ Failed to post comment:', error instanceof Error ? error.message : error);
-        }
-        process.exit(1);
-    }
+    const { data: comment } = await octokit.issues.createComment({
+        owner,
+        repo,
+        issue_number: number,
+        body: options.message,
+    });
+    return {
+        commentUrl: comment.html_url,
+        url: options.url,
+    };
 }
 export async function runClaim(options) {
     validateUrl(options.issueUrl);
-    // Token is guaranteed by the preAction hook in cli.ts for non-LOCAL_ONLY_COMMANDS.
-    const token = getGitHubToken();
+    const token = requireGitHubToken();
     // Default claim message or custom
     const message = options.message || "Hi! I'd like to work on this issue. Could you assign it to me?";
-    try {
-        validateMessage(message);
-    }
-    catch (error) {
-        if (options.json) {
-            outputJsonError(error instanceof Error ? error.message : 'Invalid message');
-        }
-        else {
-            console.error(`Error: ${error instanceof Error ? error.message : 'Invalid message'}`);
-        }
-        process.exit(1);
-    }
+    validateMessage(message);
     // Parse URL
     const parsed = parseGitHubUrl(options.issueUrl);
     if (!parsed || parsed.type !== 'issues') {
-        if (options.json) {
-            outputJsonError('Invalid issue URL format (must be an issue, not a PR)');
-        }
-        else {
-            console.error('Invalid issue URL format (must be an issue, not a PR)');
-        }
-        process.exit(1);
+        throw new Error('Invalid issue URL format (must be an issue, not a PR)');
     }
     const { owner, repo, number } = parsed;
-    if (!options.json) {
-        console.log('\n🙋 Claiming issue:', options.issueUrl);
-        console.log('---');
-        console.log(message);
-        console.log('---\n');
-    }
     const octokit = getOctokit(token);
-    try {
-        const { data: comment } = await octokit.issues.createComment({
-            owner,
-            repo,
-            issue_number: number,
-            body: message,
-        });
-        // Add to tracked issues
-        const stateManager = getStateManager();
-        stateManager.addIssue({
-            id: number,
-            url: options.issueUrl,
-            repo: `${owner}/${repo}`,
-            number,
-            title: '(claimed)',
-            status: 'claimed',
-            labels: [],
-            createdAt: new Date().toISOString(),
-            updatedAt: new Date().toISOString(),
-            vetted: false,
-        });
-        stateManager.save();
-        if (options.json) {
-            outputJson({
-                commentUrl: comment.html_url,
-                issueUrl: options.issueUrl,
-            });
-        }
-        else {
-            console.log('✅ Issue claimed!');
-            console.log(`   ${comment.html_url}`);
-        }
-    }
-    catch (error) {
-        if (options.json) {
-            outputJsonError(error instanceof Error ? error.message : 'Unknown error');
-        }
-        else {
-            console.error('❌ Failed to claim issue:', error instanceof Error ? error.message : error);
-        }
-        process.exit(1);
-    }
+    const { data: comment } = await octokit.issues.createComment({
+        owner,
+        repo,
+        issue_number: number,
+        body: message,
+    });
+    // Add to tracked issues
+    const stateManager = getStateManager();
+    stateManager.addIssue({
+        id: number,
+        url: options.issueUrl,
+        repo: `${owner}/${repo}`,
+        number,
+        title: '(claimed)',
+        status: 'claimed',
+        labels: [],
+        createdAt: new Date().toISOString(),
+        updatedAt: new Date().toISOString(),
+        vetted: false,
+    });
+    stateManager.save();
+    return {
+        commentUrl: comment.html_url,
+        issueUrl: options.issueUrl,
+    };
 }

package/dist/commands/config.d.ts

@@ -2,10 +2,16 @@
  * Config command
  * Shows or updates configuration
  */
+import type { ConfigOutput } from '../formatters/json.js';
 interface ConfigOptions {
     key?: string;
     value?: string;
-    json?: boolean;
 }
-export declare function runConfig(options: ConfigOptions): Promise<void>;
+export interface ConfigSetOutput {
+    success: true;
+    key: string;
+    value: string;
+}
+export type ConfigCommandOutput = ConfigOutput | ConfigSetOutput;
+export declare function runConfig(options: ConfigOptions): Promise<ConfigCommandOutput>;
 export {};

package/dist/commands/config.js

@@ -3,32 +3,15 @@
  * Shows or updates configuration
  */
 import { getStateManager } from '../core/index.js';
-import { outputJson, outputJsonError } from '../formatters/json.js';
-function exitWithError(msg, json) {
-    if (json) {
-        outputJsonError(msg);
-    }
-    else {
-        console.error(msg);
-    }
-    process.exit(1);
-}
 export async function runConfig(options) {
     const stateManager = getStateManager();
     const currentConfig = stateManager.getState().config;
     if (!options.key) {
         // Show current config
-        if (options.json) {
-            outputJson({ config: currentConfig });
-        }
-        else {
-            console.log('\n⚙️ Current Configuration:\n');
-            console.log(JSON.stringify(currentConfig, null, 2));
-        }
-        return;
+        return { config: currentConfig };
     }
     if (!options.value) {
-        exitWithError('Value required', options.json);
+        throw new Error('Value required');
     }
     const value = options.value;
     // Handle specific config keys
@@ -49,7 +32,7 @@ export async function runConfig(options) {
         case 'exclude-repo': {
             const parts = value.split('/');
             if (parts.length !== 2 || !parts[0] || !parts[1]) {
-                exitWithError(`Invalid repo format "${value}". Use "owner/repo" format. To exclude an entire org, use: config exclude-org ${value}`, options.json);
+                throw new Error(`Invalid repo format "${value}". Use "owner/repo" format. To exclude an entire org, use: config exclude-org ${value}`);
             }
             const valueLower = value.toLowerCase();
             if (!currentConfig.excludeRepos.some((r) => r.toLowerCase() === valueLower)) {
@@ -60,7 +43,7 @@ export async function runConfig(options) {
         }
         case 'exclude-org': {
             if (value.includes('/')) {
-                exitWithError(`Invalid org name "${value}". Use just the org name (e.g., "facebook"), not "owner/repo" format. To exclude a specific repo, use: config exclude-repo ${value}`, options.json);
+                throw new Error(`Invalid org name "${value}". Use just the org name (e.g., "facebook"), not "owner/repo" format. To exclude a specific repo, use: config exclude-repo ${value}`);
             }
             const currentOrgs = currentConfig.excludeOrgs ?? [];
             if (!currentOrgs.some((o) => o.toLowerCase() === value.toLowerCase())) {
@@ -70,13 +53,8 @@ export async function runConfig(options) {
             break;
         }
         default:
-            exitWithError(`Unknown config key: ${options.key}`, options.json);
+            throw new Error(`Unknown config key: ${options.key}`);
     }
     stateManager.save();
-    if (options.json) {
-        outputJson({ success: true, key: options.key, value });
-    }
-    else {
-        console.log(`Set ${options.key} to: ${value}`);
-    }
+    return { success: true, key: options.key, value };
 }

package/dist/commands/daily.d.ts

@@ -6,12 +6,26 @@
  * Domain logic lives in src/core/daily-logic.ts; this file is a thin
  * orchestration layer that wires up the phases and handles I/O.
  */
-import { type DailyOutput } from '../formatters/json.js';
+import { type DailyDigest, type CommentedIssue, type PRCheckFailure, type RepoGroup } from '../core/index.js';
+import { type DailyOutput, type CapacityAssessment, type ActionableIssue, type ActionMenu } from '../formatters/json.js';
 export { computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatBriefSummary, formatSummary, printDigest, CRITICAL_STATUSES, } from '../core/index.js';
-interface DailyOptions {
-    json?: boolean;
+/**
+ * Internal result of the daily check, using full (non-deduplicated) types.
+ * Consumed by printDigest() (text mode) and converted to DailyOutput (JSON mode)
+ * via toDailyOutput() which deduplicates PR objects.
+ */
+export interface DailyCheckResult {
+    digest: DailyDigest;
+    updates: unknown[];
+    capacity: CapacityAssessment;
+    summary: string;
+    briefSummary: string;
+    actionableIssues: ActionableIssue[];
+    actionMenu: ActionMenu;
+    commentedIssues: CommentedIssue[];
+    repoGroups: RepoGroup[];
+    failures: PRCheckFailure[];
 }
-export declare function runDaily(options: DailyOptions): Promise<void>;
 /**
  * Core daily check logic, extracted for reuse by the startup command.
  * Fetches all open PRs, updates state, and returns structured output.
@@ -27,3 +41,13 @@ export declare function runDaily(options: DailyOptions): Promise<void>;
  *   5. generateDigestOutput — capacity, dismiss filter, action menu assembly
  */
 export declare function executeDailyCheck(token: string): Promise<DailyOutput>;
+/**
+ * Run the daily check and return deduplicated DailyOutput.
+ * Errors propagate to the caller.
+ */
+export declare function runDaily(): Promise<DailyOutput>;
+/**
+ * Run the daily check and return the full (non-deduplicated) result.
+ * Used by CLI text mode where printDigest() needs full PR objects.
+ */
+export declare function runDailyForDisplay(): Promise<DailyCheckResult>;