@oss-autopilot/core 0.44.2 → 0.44.3

package/dist/core/index.js

@@ -8,8 +8,8 @@ export { IssueDiscovery, isDocOnlyIssue, applyPerRepoCap, DOC_ONLY_LABELS, } fro
 export { IssueConversationMonitor } from './issue-conversation.js';
 export { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 export { getOctokit, checkRateLimit } from './github.js';
-export { parseGitHubUrl, daysBetween, splitRepo, isOwnRepo, getCLIVersion, getDataDir, getStatePath, getBackupDir, getCacheDir, getDashboardPath, formatRelativeTime, byDateDescending, getGitHubToken, getGitHubTokenAsync, requireGitHubToken, resetGitHubTokenCache, } from './utils.js';
-export { OssAutopilotError, ConfigurationError, ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
+export { parseGitHubUrl, daysBetween, splitRepo, isOwnRepo, getCLIVersion, getDataDir, getStatePath, getBackupDir, getCacheDir, getDashboardPath, formatRelativeTime, byDateDescending, getGitHubToken, getGitHubTokenAsync, requireGitHubToken, resetGitHubTokenCache, DEFAULT_CONCURRENCY, } from './utils.js';
+export { OssAutopilotError, ConfigurationError, ValidationError, errorMessage, getHttpStatusCode, isRateLimitError, isRateLimitOrAuthError, } from './errors.js';
 export { enableDebug, isDebugEnabled, debug, info, warn, timed } from './logger.js';
 export { HttpCache, getHttpCache, cachedRequest } from './http-cache.js';
 export { CRITICAL_STATUSES, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatActionHint, formatBriefSummary, formatSummary, printDigest, } from './daily-logic.js';

package/dist/core/issue-conversation.js

@@ -9,12 +9,12 @@ import { getOctokit } from './github.js';
 import { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 import { paginateAll } from './pagination.js';
 import { getStateManager } from './state.js';
-import { daysBetween, splitRepo, extractOwnerRepo, isOwnRepo } from './utils.js';
+import { daysBetween, splitRepo, extractOwnerRepo, isOwnRepo, DEFAULT_CONCURRENCY } from './utils.js';
 import { runWorkerPool } from './concurrency.js';
 import { ConfigurationError, errorMessage } from './errors.js';
 import { debug, warn } from './logger.js';
 const MODULE = 'issue-conversation';
-const MAX_CONCURRENT_REQUESTS = 5;
+const MAX_CONCURRENT_REQUESTS = DEFAULT_CONCURRENCY;
 /** Associations that indicate someone with repo-level permissions. */
 const MAINTAINER_ASSOCIATIONS = new Set(['OWNER', 'MEMBER', 'COLLABORATOR']);
 export class IssueConversationMonitor {

package/dist/core/issue-discovery.d.ts

@@ -65,11 +65,6 @@ export declare class IssueDiscovery {
      * Split repos into batches of the specified size.
      */
     private batchRepos;
-    /**
-     * Check if an error is a GitHub rate limit error (429 or rate-limit 403).
-     * Static proxy kept for backward compatibility with tests.
-     */
-    static isRateLimitError(error: unknown): boolean;
     /**
      * Vet a specific issue (delegates to IssueVetter).
      */

package/dist/core/issue-discovery.js

@@ -12,7 +12,7 @@ import { getOctokit, checkRateLimit } from './github.js';
 import { getStateManager } from './state.js';
 import { daysBetween, getDataDir } from './utils.js';
 import { DEFAULT_CONFIG } from './types.js';
-import { ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
+import { ValidationError, errorMessage, getHttpStatusCode, isRateLimitError } from './errors.js';
 import { debug, info, warn } from './logger.js';
 import { getHttpCache, cachedTimeBased } from './http-cache.js';
 import { isDocOnlyIssue, detectLabelFarmingRepos, applyPerRepoCap } from './issue-filtering.js';
@@ -328,7 +328,7 @@ export class IssueDiscovery {
             catch (error) {
                 const errMsg = errorMessage(error);
                 phase2Error = errMsg;
-                if (IssueVetter.isRateLimitError(error)) {
+                if (isRateLimitError(error)) {
                     rateLimitHitDuringSearch = true;
                 }
                 warn(MODULE, `Error in general issue search: ${errMsg}`);
@@ -370,7 +370,7 @@ export class IssueDiscovery {
             catch (error) {
                 const errMsg = errorMessage(error);
                 phase3Error = errMsg;
-                if (IssueVetter.isRateLimitError(error)) {
+                if (isRateLimitError(error)) {
                     rateLimitHitDuringSearch = true;
                 }
                 warn(MODULE, `Error in maintained-repo search: ${errMsg}`);
@@ -464,7 +464,7 @@ export class IssueDiscovery {
             }
             catch (error) {
                 failedBatches++;
-                if (IssueVetter.isRateLimitError(error)) {
+                if (isRateLimitError(error)) {
                     rateLimitFailures++;
                 }
                 const batchRepos = batch.join(', ');
@@ -489,13 +489,6 @@ export class IssueDiscovery {
         }
         return batches;
     }
-    /**
-     * Check if an error is a GitHub rate limit error (429 or rate-limit 403).
-     * Static proxy kept for backward compatibility with tests.
-     */
-    static isRateLimitError(error) {
-        return IssueVetter.isRateLimitError(error);
-    }
     /**
      * Vet a specific issue (delegates to IssueVetter).
      */

package/dist/core/issue-vetting.d.ts

@@ -29,8 +29,6 @@ export declare class IssueVetter {
         allFailed: boolean;
         rateLimitHit: boolean;
     }>;
-    /** Check if an error is a GitHub rate limit error (429 or rate-limit 403). */
-    static isRateLimitError(error: unknown): boolean;
     checkNoExistingPR(owner: string, repo: string, issueNumber: number): Promise<CheckResult>;
     /**
      * Check how many merged PRs the authenticated user has in a repo.

package/dist/core/issue-vetting.js

@@ -5,14 +5,13 @@
  * Extracted from issue-discovery.ts (#356) to isolate vetting logic.
  */
 import { paginateAll } from './pagination.js';
-import { parseGitHubUrl, daysBetween } from './utils.js';
-import { ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
+import { parseGitHubUrl, daysBetween, DEFAULT_CONCURRENCY } from './utils.js';
+import { ValidationError, errorMessage, isRateLimitError } from './errors.js';
 import { warn } from './logger.js';
 import { getHttpCache, cachedRequest, cachedTimeBased } from './http-cache.js';
 import { calculateRepoQualityBonus, calculateViabilityScore } from './issue-scoring.js';
 const MODULE = 'issue-vetting';
-// Concurrency limit for parallel API calls
-const MAX_CONCURRENT_REQUESTS = 5;
+const MAX_CONCURRENT_REQUESTS = DEFAULT_CONCURRENCY;
 // Cache for contribution guidelines (expires after 1 hour, max 100 entries)
 const guidelinesCache = new Map();
 const CACHE_TTL_MS = 60 * 60 * 1000; // 1 hour
@@ -233,7 +232,7 @@ export class IssueVetter {
      */
     async vetIssuesParallel(urls, maxResults, priority) {
         const candidates = [];
-        const pending = [];
+        const pending = new Map();
         let failedVettingCount = 0;
         let rateLimitFailures = 0;
         let attemptedCount = 0;
@@ -253,21 +252,20 @@ export class IssueVetter {
             })
                 .catch((error) => {
                 failedVettingCount++;
-                if (IssueVetter.isRateLimitError(error)) {
+                if (isRateLimitError(error)) {
                     rateLimitFailures++;
                 }
                 warn(MODULE, `Error vetting issue ${url}:`, errorMessage(error));
-            });
-            pending.push(task);
-            // Limit concurrency
-            if (pending.length >= MAX_CONCURRENT_REQUESTS) {
-                // Wait for at least one to complete, then remove it
-                const completed = await Promise.race(pending.map((p, i) => p.then(() => i)));
-                pending.splice(completed, 1);
+            })
+                .finally(() => pending.delete(url));
+            pending.set(url, task);
+            // Limit concurrency — wait for at least one to complete before launching more
+            if (pending.size >= MAX_CONCURRENT_REQUESTS) {
+                await Promise.race(pending.values());
             }
         }
         // Wait for remaining
-        await Promise.allSettled(pending);
+        await Promise.allSettled(pending.values());
         const allFailed = failedVettingCount === attemptedCount && attemptedCount > 0;
         if (allFailed) {
             warn(MODULE, `All ${attemptedCount} issue(s) failed vetting. ` +
@@ -275,17 +273,6 @@ export class IssueVetter {
         }
         return { candidates: candidates.slice(0, maxResults), allFailed, rateLimitHit: rateLimitFailures > 0 };
     }
-    /** Check if an error is a GitHub rate limit error (429 or rate-limit 403). */
-    static isRateLimitError(error) {
-        const status = getHttpStatusCode(error);
-        if (status === 429)
-            return true;
-        if (status === 403) {
-            const msg = errorMessage(error).toLowerCase();
-            return msg.includes('rate limit');
-        }
-        return false;
-    }
     async checkNoExistingPR(owner, repo, issueNumber) {
         try {
             // Search for PRs that mention this issue
@@ -447,26 +434,25 @@ export class IssueVetter {
             return cached.guidelines;
         }
         const filesToCheck = ['CONTRIBUTING.md', '.github/CONTRIBUTING.md', 'docs/CONTRIBUTING.md', 'contributing.md'];
-        for (const file of filesToCheck) {
-            try {
-                const { data } = await this.octokit.repos.getContent({
-                    owner,
-                    repo,
-                    path: file,
-                });
-                if ('content' in data) {
-                    const content = Buffer.from(data.content, 'base64').toString('utf-8');
-                    const guidelines = this.parseContributionGuidelines(content);
-                    // Cache the result and prune if needed
-                    guidelinesCache.set(cacheKey, { guidelines, fetchedAt: Date.now() });
-                    pruneCache();
-                    return guidelines;
-                }
+        // Probe all paths in parallel — take the first success in priority order
+        const results = await Promise.allSettled(filesToCheck.map((file) => this.octokit.repos.getContent({ owner, repo, path: file }).then(({ data }) => {
+            if ('content' in data) {
+                return Buffer.from(data.content, 'base64').toString('utf-8');
+            }
+            return null;
+        })));
+        for (let i = 0; i < results.length; i++) {
+            const result = results[i];
+            if (result.status === 'fulfilled' && result.value) {
+                const guidelines = this.parseContributionGuidelines(result.value);
+                guidelinesCache.set(cacheKey, { guidelines, fetchedAt: Date.now() });
+                pruneCache();
+                return guidelines;
             }
-            catch (error) {
-                // File not found is expected; only log unexpected errors
-                if (error instanceof Error && !error.message.includes('404') && !error.message.includes('Not Found')) {
-                    warn(MODULE, `Unexpected error fetching ${file} from ${owner}/${repo}: ${error.message}`);
+            if (result.status === 'rejected') {
+                const msg = result.reason instanceof Error ? result.reason.message : String(result.reason);
+                if (!msg.includes('404') && !msg.includes('Not Found')) {
+                    warn(MODULE, `Unexpected error fetching ${filesToCheck[i]} from ${owner}/${repo}: ${msg}`);
                 }
             }
         }

package/dist/core/pr-monitor.js

@@ -13,7 +13,7 @@
  */
 import { getOctokit } from './github.js';
 import { getStateManager } from './state.js';
-import { daysBetween, parseGitHubUrl, extractOwnerRepo } from './utils.js';
+import { daysBetween, parseGitHubUrl, extractOwnerRepo, DEFAULT_CONCURRENCY } from './utils.js';
 import { runWorkerPool } from './concurrency.js';
 import { ConfigurationError, ValidationError, errorMessage, getHttpStatusCode } from './errors.js';
 import { paginateAll } from './pagination.js';
@@ -31,8 +31,7 @@ export { computeDisplayLabel } from './display-utils.js';
 export { classifyCICheck, classifyFailingChecks } from './ci-analysis.js';
 export { isConditionalChecklistItem } from './checklist-analysis.js';
 const MODULE = 'pr-monitor';
-// Concurrency limit for parallel API calls
-const MAX_CONCURRENT_REQUESTS = 5;
+const MAX_CONCURRENT_REQUESTS = DEFAULT_CONCURRENCY;
 export class PRMonitor {
     octokit;
     stateManager;
@@ -482,7 +481,7 @@ export class PRMonitor {
             }
             // If entire chunk failed, likely a systemic issue (rate limit, auth, outage) — abort remaining
             if (chunkFailures === chunk.length && chunk.length > 0) {
-                const remaining = repos.length - i - chunkSize;
+                const remaining = uniqueRepos.length - i - chunkSize;
                 if (remaining > 0) {
                     warn(MODULE, `Entire chunk failed, aborting remaining ${remaining} repos`);
                 }

package/dist/core/utils.d.ts

@@ -1,6 +1,8 @@
 /**
  * Shared utility functions
  */
+/** Default concurrency limit for parallel GitHub API requests. */
+export declare const DEFAULT_CONCURRENCY = 5;
 /**
  * Returns the oss-autopilot data directory path, creating it if it does not exist.
  *

package/dist/core/utils.js

@@ -1,6 +1,8 @@
 /**
  * Shared utility functions
  */
+/** Default concurrency limit for parallel GitHub API requests. */
+export const DEFAULT_CONCURRENCY = 5;
 import * as fs from 'fs';
 import * as path from 'path';
 import * as os from 'os';
@@ -207,7 +209,7 @@ export function extractOwnerRepo(url) {
  * // -9
  */
 export function daysBetween(from, to = new Date()) {
-    return Math.floor((to.getTime() - from.getTime()) / (1000 * 60 * 60 * 24));
+    return Math.max(0, Math.floor((to.getTime() - from.getTime()) / (1000 * 60 * 60 * 24)));
 }
 /**
  * Splits an `"owner/repo"` string into its owner and repo components.
@@ -268,6 +270,8 @@ export function getCLIVersion() {
 export function formatRelativeTime(dateStr) {
     const date = new Date(dateStr);
     const diffMs = Date.now() - date.getTime();
+    if (diffMs < 0)
+        return 'just now';
     const diffMins = Math.floor(diffMs / 60000);
     const diffHours = Math.floor(diffMs / 3600000);
     const diffDays = Math.floor(diffMs / 86400000);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oss-autopilot/core",
-  "version": "0.44.2",
+  "version": "0.44.3",
   "description": "CLI and core library for managing open source contributions",
   "type": "module",
   "bin": {