@oss-autopilot/core 0.44.2 → 0.44.15

package/dist/core/github-stats.js

@@ -3,6 +3,7 @@
  * Extracted from PRMonitor to isolate statistics-gathering API calls (#263).
  */
 import { extractOwnerRepo, parseGitHubUrl, isOwnRepo } from './utils.js';
+import { isBelowMinStars } from './types.js';
 import { debug, warn } from './logger.js';
 import { getHttpCache } from './http-cache.js';
 const MODULE = 'github-stats';
@@ -39,13 +40,14 @@ function isCachedPRCounts(v) {
  * string (e.g. mergedAt or closedAt) used for monthly counts and daily activity.
  * Return an empty string to skip histogram tracking for that item.
  */
-async function fetchUserPRCounts(octokit, githubUsername, query, label, accumulateRepo) {
+async function fetchUserPRCounts(octokit, githubUsername, query, label, accumulateRepo, starFilter) {
     if (!githubUsername) {
         return emptyPRCountsResult();
     }
     // Check for a fresh cached result (avoids 10-20 paginated API calls)
     const cache = getHttpCache();
-    const cacheKey = `pr-counts:${label}:${githubUsername}`;
+    const minStarsSuffix = starFilter ? `:stars${starFilter.minStars}` : '';
+    const cacheKey = `pr-counts:v3:${label}:${githubUsername}${minStarsSuffix}`;
     const cached = cache.getIfFresh(cacheKey, PR_COUNTS_CACHE_TTL_MS);
     if (cached && isCachedPRCounts(cached)) {
         debug(MODULE, `Using cached ${label} PR counts for @${githubUsername}`);
@@ -66,7 +68,7 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
     let totalCount;
     while (true) {
         const { data } = await octokit.search.issuesAndPullRequests({
-            q: `is:pr ${query} author:${githubUsername}`,
+            q: `is:pr ${query} author:${githubUsername} -user:${githubUsername}`,
             sort: 'updated',
             order: 'desc',
             per_page: 100,
@@ -86,6 +88,12 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
                 continue;
             // Note: excludeRepos/excludeOrgs are intentionally NOT filtered here.
             // Those filters control issue discovery/search, not historical statistics.
+            // Skip repos below the minimum star threshold (#576).
+            // Repos with unknown star counts (not yet fetched) are included — they'll be
+            // filtered on the next run once star data is cached in repoScores.
+            if (starFilter && isBelowMinStars(starFilter.knownStarCounts.get(repo), starFilter.minStars)) {
+                continue;
+            }
             // Per-repo accumulation + get primary date for histograms
             const primaryDate = accumulateRepo(repos, repo, item);
             // Monthly histogram for primary date (merged/closed)
@@ -135,7 +143,7 @@ async function fetchUserPRCounts(octokit, githubUsername, query, label, accumula
  * Fetch merged PR counts and latest merge dates per repository for the configured user.
  * Also builds a monthly histogram of all merges for the contribution timeline.
  */
-export function fetchUserMergedPRCounts(octokit, githubUsername) {
+export function fetchUserMergedPRCounts(octokit, githubUsername, starFilter) {
     return fetchUserPRCounts(octokit, githubUsername, 'is:merged', 'merged', (repos, repo, item) => {
         if (!item.pull_request?.merged_at) {
             warn(MODULE, `merged_at missing for merged PR ${item.html_url}${item.closed_at ? ', falling back to closed_at' : ', no date available'}`);
@@ -152,17 +160,17 @@ export function fetchUserMergedPRCounts(octokit, githubUsername) {
             repos.set(repo, { count: 1, lastMergedAt: mergedAt });
         }
         return mergedAt;
-    });
+    }, starFilter);
 }
 /**
  * Fetch closed-without-merge PR counts per repository for the configured user.
  * Used to populate closedWithoutMergeCount in repo scores for accurate merge rate.
  */
-export function fetchUserClosedPRCounts(octokit, githubUsername) {
+export function fetchUserClosedPRCounts(octokit, githubUsername, starFilter) {
     return fetchUserPRCounts(octokit, githubUsername, 'is:closed is:unmerged', 'closed', (repos, repo, item) => {
         repos.set(repo, (repos.get(repo) || 0) + 1);
         return item.closed_at || '';
-    });
+    }, starFilter);
 }
 /**
  * Shared helper: search for recent PRs and filter out own repos, excluded repos/orgs.

package/dist/core/index.d.ts

@@ -8,8 +8,8 @@ export { IssueDiscovery, type IssueCandidate, type SearchPriority, isDocOnlyIssu
 export { IssueConversationMonitor } from './issue-conversation.js';
 export { isBotAuthor, isAcknowledgmentComment } from './comment-utils.js';
 export { getOctokit, checkRateLimit, type RateLimitInfo } 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, type CacheEntry } from './http-cache.js';
 export { CRITICAL_STATUSES, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatActionHint, formatBriefSummary, formatSummary, printDigest, } from './daily-logic.js';

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}`);
                 }
             }
         }
@@ -523,7 +509,7 @@ export class IssueVetter {
         if (!body || body.length < 50)
             return false;
         // Check for clear structure
-        const hasSteps = /\d+\.|[-*]\s/.test(body);
+        const hasSteps = /\d\.|[-*]\s/.test(body);
         const hasCodeBlock = /```/.test(body);
         const hasExpectedBehavior = /expect|should|must|want/i.test(body);
         // Must have at least two indicators of clarity

package/dist/core/pr-monitor.d.ts

@@ -11,7 +11,7 @@
  * - display-utils.ts: Display label computation
  * - github-stats.ts: Merged/closed PR counts and star fetching
  */
-import { FetchedPR, DailyDigest, ClosedPR, MergedPR } from './types.js';
+import { FetchedPR, DailyDigest, ClosedPR, MergedPR, StarFilter } from './types.js';
 import { type PRCountsResult } from './github-stats.js';
 export { computeDisplayLabel } from './display-utils.js';
 export { classifyCICheck, classifyFailingChecks } from './ci-analysis.js';
@@ -46,6 +46,29 @@ export declare class PRMonitor {
      * Determine the overall status of a PR
      */
     private determineStatus;
+    /**
+     * CI-fix bots that push commits as a direct result of the contributor's push (#568).
+     * Their commits represent contributor work and should count as addressing feedback.
+     * This is intentionally an allowlist — not all `[bot]` accounts are CI-fix bots
+     * (e.g. dependabot[bot] and renovate[bot] open their own PRs).
+     * Values must be lowercase — lookup uses .toLowerCase() for case-insensitive matching.
+     */
+    private static readonly CI_FIX_BOTS;
+    /**
+     * Check whether the HEAD commit was authored by the contributor (#547).
+     * Returns true when the author matches, when the author is a known CI-fix
+     * bot (#568), or when author info is unavailable (graceful degradation).
+     */
+    private isContributorCommit;
+    /** Minimum gap (ms) between maintainer comment and contributor commit for
+     *  the commit to count as "addressing" the feedback (#547). Prevents false
+     *  positives from race conditions, clock skew, and in-flight pushes. */
+    private static readonly MIN_RESPONSE_GAP_MS;
+    /**
+     * Check whether the contributor's commit is meaningfully after the maintainer's
+     * comment — i.e. the commit timestamp is at least MIN_RESPONSE_GAP_MS later (#547).
+     */
+    private isCommitAfterComment;
     /**
      * Check if PR has merge conflict
      */
@@ -60,7 +83,7 @@ export declare class PRMonitor {
      * Fetch merged PR counts and latest merge dates per repository for the configured user.
      * Delegates to github-stats module.
      */
-    fetchUserMergedPRCounts(): Promise<PRCountsResult<{
+    fetchUserMergedPRCounts(starFilter?: StarFilter): Promise<PRCountsResult<{
         count: number;
         lastMergedAt: string;
     }>>;
@@ -68,7 +91,7 @@ export declare class PRMonitor {
      * Fetch closed-without-merge PR counts per repository for the configured user.
      * Delegates to github-stats module.
      */
-    fetchUserClosedPRCounts(): Promise<PRCountsResult<number>>;
+    fetchUserClosedPRCounts(starFilter?: StarFilter): Promise<PRCountsResult<number>>;
     /**
      * Fetch GitHub star counts for a list of repositories.
      * Delegates to github-stats module.