@oss-autopilot/core 0.48.0 → 0.49.0

package/dist/commands/daily.d.ts

@@ -8,7 +8,7 @@
  */
 import { type DailyDigest, type CommentedIssue, type PRCheckFailure, type RepoGroup, type AgentState, type StarFilter } 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';
+export { applyStatusOverrides, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatBriefSummary, formatSummary, printDigest, CRITICAL_STATUSES, } from '../core/index.js';
 /**
  * Build a star filter from state for use in fetchUserPRCounts.
  * Returns undefined if no star data is available (first run).

package/dist/commands/daily.js

@@ -6,7 +6,7 @@
  * 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 { getStateManager, PRMonitor, IssueConversationMonitor, requireGitHubToken, CRITICAL_STATUSES, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatBriefSummary, formatSummary, } from '../core/index.js';
+import { getStateManager, PRMonitor, IssueConversationMonitor, requireGitHubToken, CRITICAL_STATUSES, applyStatusOverrides, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatBriefSummary, formatSummary, } from '../core/index.js';
 import { errorMessage, isRateLimitOrAuthError } from '../core/errors.js';
 import { warn } from '../core/logger.js';
 import { emptyPRCountsResult } from '../core/github-stats.js';
@@ -15,7 +15,7 @@ import { deduplicateDigest, compactActionableIssues, compactRepoGroups, } from '
 const MODULE = 'daily';
 // Re-export domain functions so existing consumers (tests, dashboard, startup)
 // can continue importing from './daily.js' without changes.
-export { computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatBriefSummary, formatSummary, printDigest, CRITICAL_STATUSES, } from '../core/index.js';
+export { applyStatusOverrides, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatBriefSummary, formatSummary, printDigest, CRITICAL_STATUSES, } from '../core/index.js';
 /**
  * Build a star filter from state for use in fetchUserPRCounts.
  * Returns undefined if no star data is available (first run).
@@ -249,11 +249,15 @@ function partitionPRs(prMonitor, prs, recentlyClosedPRs, recentlyMergedPRs) {
     catch (error) {
         warn(MODULE, `Failed to expire/persist snoozes: ${errorMessage(error)}`);
     }
+    // Apply dashboard/CLI status overrides before partitioning.
+    // This ensures PRs reclassified in the dashboard (e.g., "Need Attention" → "Waiting")
+    // are respected by the CLI pipeline.
+    const overriddenPRs = applyStatusOverrides(prs, stateManager.getState());
     // Partition PRs into active vs shelved, auto-unshelving when maintainers engage
     const shelvedPRs = [];
     const autoUnshelvedPRs = [];
     const activePRs = [];
-    for (const pr of prs) {
+    for (const pr of overriddenPRs) {
         if (stateManager.isPRShelved(pr.url)) {
             if (CRITICAL_STATUSES.has(pr.status)) {
                 stateManager.unshelvePR(pr.url);
@@ -273,10 +277,10 @@ function partitionPRs(prMonitor, prs, recentlyClosedPRs, recentlyMergedPRs) {
             activePRs.push(pr);
         }
     }
-    // Generate digest from fresh data.
+    // Generate digest from override-applied PRs so status categories are correct.
     // Note: digest.openPRs contains ALL fetched PRs (including shelved).
     // We override summary fields below to reflect active-only counts.
-    const digest = prMonitor.generateDigest(prs, recentlyClosedPRs, recentlyMergedPRs);
+    const digest = prMonitor.generateDigest(overriddenPRs, recentlyClosedPRs, recentlyMergedPRs);
     // Attach shelve info to digest
     digest.shelvedPRs = shelvedPRs;
     digest.autoUnshelvedPRs = autoUnshelvedPRs;
@@ -292,7 +296,7 @@ function partitionPRs(prMonitor, prs, recentlyClosedPRs, recentlyMergedPRs) {
  * Assesses capacity, filters dismissed issues, computes actionable items,
  * and assembles the action menu.
  */
-function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, failures) {
+function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, failures, previousLastDigestAt) {
     const stateManager = getStateManager();
     // Assess capacity from active PRs only (shelved PRs excluded)
     const capacity = assessCapacity(activePRs, stateManager.getState().config.maxActivePRs, shelvedPRs.length);
@@ -357,7 +361,7 @@ function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, fa
             warn(MODULE, `Failed to persist auto-undismissed state: ${errorMessage(error)}`);
         }
     }
-    const actionableIssues = collectActionableIssues(nonDismissedPRs, snoozedUrls);
+    const actionableIssues = collectActionableIssues(nonDismissedPRs, snoozedUrls, previousLastDigestAt);
     digest.summary.totalNeedingAttention = actionableIssues.length;
     const briefSummary = formatBriefSummary(digest, actionableIssues.length, issueResponses.length);
     const actionMenu = computeActionMenu(actionableIssues, capacity, filteredCommentedIssues);
@@ -425,10 +429,13 @@ async function executeDailyCheckInternal(token) {
     await updateRepoScores(prMonitor, prs, mergedCounts, closedCounts);
     // Phase 3: Persist monthly analytics
     updateMonthlyAnalytics(prs, monthlyCounts, monthlyClosedCounts, openedFromMerged, openedFromClosed);
+    // Capture lastDigestAt BEFORE Phase 4 overwrites it with the current run's timestamp.
+    // Used by collectActionableIssues to determine which PRs are "new" (created since last digest).
+    const previousLastDigestAt = getStateManager().getState().lastDigestAt;
     // Phase 4: Expire snoozes, partition PRs, generate and save digest
     const { activePRs, shelvedPRs, digest } = partitionPRs(prMonitor, prs, recentlyClosedPRs, recentlyMergedPRs);
     // Phase 5: Build structured output (capacity, dismiss filter, action menu)
-    return generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, failures);
+    return generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, failures, previousLastDigestAt);
 }
 /**
  * Run the daily check and return deduplicated DailyOutput.

package/dist/commands/dashboard-server.js

@@ -8,7 +8,7 @@
 import * as http from 'http';
 import * as fs from 'fs';
 import * as path from 'path';
-import { getStateManager, getGitHubToken, getCLIVersion } from '../core/index.js';
+import { getStateManager, getGitHubToken, getCLIVersion, applyStatusOverrides } from '../core/index.js';
 import { errorMessage, ValidationError } from '../core/errors.js';
 import { warn } from '../core/logger.js';
 import { validateUrl, validateGitHubUrl, PR_URL_PATTERN } from './validation.js';
@@ -33,40 +33,6 @@ const MIME_TYPES = {
     '.png': 'image/png',
     '.ico': 'image/x-icon',
 };
-/**
- * Apply status overrides from state to the PR list.
- * Overrides are auto-cleared if the PR has new activity since the override was set.
- */
-function applyStatusOverrides(prs, state) {
-    const overrides = state.config.statusOverrides;
-    if (!overrides || Object.keys(overrides).length === 0)
-        return prs;
-    const stateManager = getStateManager();
-    // Snapshot keys before iteration — clearStatusOverride mutates the same object
-    const overrideUrls = new Set(Object.keys(overrides));
-    let didAutoClear = false;
-    const result = prs.map((pr) => {
-        const override = stateManager.getStatusOverride(pr.url, pr.updatedAt);
-        if (!override) {
-            if (overrideUrls.has(pr.url))
-                didAutoClear = true;
-            return pr;
-        }
-        if (override.status === pr.status)
-            return pr;
-        return { ...pr, status: override.status };
-    });
-    // Persist any auto-cleared overrides so they don't resurrect on restart
-    if (didAutoClear) {
-        try {
-            stateManager.save();
-        }
-        catch (err) {
-            warn(MODULE, `Failed to persist auto-cleared overrides — they may reappear on restart: ${errorMessage(err)}`);
-        }
-    }
-    return result;
-}
 // ── Helpers ────────────────────────────────────────────────────────────────────
 /**
  * Build the JSON payload that the SPA expects from GET /api/data.

package/dist/core/daily-logic.d.ts

@@ -10,7 +10,7 @@
  *   - formatActionHint — human-readable maintainer action hint label
  *   - formatBriefSummary / formatSummary / printDigest — rendering
  */
-import type { FetchedPR, FetchedPRStatus, StalenessTier, ActionReason, DailyDigest, ShelvedPRRef, MaintainerActionHint, ComputedRepoSignals, RepoGroup, CommentedIssue, CommentedIssueWithResponse } from './types.js';
+import type { FetchedPR, FetchedPRStatus, StalenessTier, ActionReason, DailyDigest, AgentState, ShelvedPRRef, MaintainerActionHint, ComputedRepoSignals, RepoGroup, CommentedIssue, CommentedIssueWithResponse } from './types.js';
 import type { CapacityAssessment, ActionableIssue, ActionMenu } from '../formatters/json.js';
 /**
  * Statuses indicating action needed from the contributor.
@@ -25,6 +25,7 @@ export declare const CRITICAL_STATUSES: ReadonlySet<FetchedPRStatus>;
 export declare const CRITICAL_ACTION_REASONS: ReadonlySet<ActionReason>;
 /** Staleness tiers indicating staleness — maintainer comments during these tiers don't count as responsive. */
 export declare const STALE_STATUSES: ReadonlySet<StalenessTier>;
+export declare function applyStatusOverrides(prs: FetchedPR[], state: Readonly<AgentState>): FetchedPR[];
 /**
  * Map a full FetchedPR to a lightweight ShelvedPRRef for digest output.
  * Only the fields needed for display are retained, reducing JSON payload size.
@@ -53,7 +54,7 @@ export declare function assessCapacity(activePRs: FetchedPR[], maxActivePRs: num
  * Note: Recently closed PRs are informational only and excluded from this list.
  * They are available separately in digest.recentlyClosedPRs (#156).
  */
-export declare function collectActionableIssues(prs: FetchedPR[], snoozedUrls?: Set<string>): ActionableIssue[];
+export declare function collectActionableIssues(prs: FetchedPR[], snoozedUrls?: Set<string>, lastDigestAt?: string): ActionableIssue[];
 /**
  * Format a maintainer action hint as a human-readable label
  */

package/dist/core/daily-logic.js

@@ -12,6 +12,8 @@
  */
 import { formatRelativeTime } from './utils.js';
 import { warn } from './logger.js';
+import { errorMessage } from './errors.js';
+import { getStateManager } from './state.js';
 // ---------------------------------------------------------------------------
 // Constants
 // ---------------------------------------------------------------------------
@@ -34,6 +36,62 @@ export const CRITICAL_ACTION_REASONS = new Set([
 /** Staleness tiers indicating staleness — maintainer comments during these tiers don't count as responsive. */
 export const STALE_STATUSES = new Set(['dormant', 'approaching_dormant']);
 // ---------------------------------------------------------------------------
+// Status overrides
+// ---------------------------------------------------------------------------
+/**
+ * Apply status overrides from state to the PR list.
+ * Overrides are auto-cleared if the PR has new activity since the override was set.
+ *
+ * When an override changes the status, the contradictory reason field is cleared
+ * and an appropriate default is set so downstream logic (assessCapacity, collectActionableIssues)
+ * works correctly.
+ */
+const VALID_OVERRIDE_STATUSES = new Set(['needs_addressing', 'waiting_on_maintainer']);
+export function applyStatusOverrides(prs, state) {
+    const overrides = state.config.statusOverrides;
+    if (!overrides || Object.keys(overrides).length === 0)
+        return prs;
+    const stateManager = getStateManager();
+    // Snapshot keys before iteration — clearStatusOverride mutates the same object
+    const overrideUrls = new Set(Object.keys(overrides));
+    let didAutoClear = false;
+    const result = prs.map((pr) => {
+        try {
+            const override = stateManager.getStatusOverride(pr.url, pr.updatedAt);
+            if (!override) {
+                if (overrideUrls.has(pr.url))
+                    didAutoClear = true;
+                return pr;
+            }
+            if (!VALID_OVERRIDE_STATUSES.has(override.status)) {
+                warn('daily-logic', `Invalid override status "${override.status}" for ${pr.url} — ignoring`);
+                return pr;
+            }
+            if (override.status === pr.status)
+                return pr;
+            // Clear the contradictory reason field and set an appropriate default
+            if (override.status === 'waiting_on_maintainer') {
+                return { ...pr, status: override.status, actionReason: undefined, waitReason: 'pending_review' };
+            }
+            return { ...pr, status: override.status, waitReason: undefined, actionReason: 'needs_response' };
+        }
+        catch (err) {
+            warn('daily-logic', `Failed to apply status override for ${pr.url}: ${errorMessage(err)}`);
+            return pr;
+        }
+    });
+    // Persist any auto-cleared overrides so they don't resurrect on restart
+    if (didAutoClear) {
+        try {
+            stateManager.save();
+        }
+        catch (err) {
+            warn('daily-logic', `Failed to persist auto-cleared overrides — they may reappear on restart: ${errorMessage(err)}`);
+        }
+    }
+    return result;
+}
+// ---------------------------------------------------------------------------
 // Internal helpers
 // ---------------------------------------------------------------------------
 /**
@@ -140,9 +198,10 @@ export function assessCapacity(activePRs, maxActivePRs, shelvedPRCount) {
  * Note: Recently closed PRs are informational only and excluded from this list.
  * They are available separately in digest.recentlyClosedPRs (#156).
  */
-export function collectActionableIssues(prs, snoozedUrls = new Set()) {
+export function collectActionableIssues(prs, snoozedUrls = new Set(), lastDigestAt) {
     const issues = [];
     const actionPRs = prs.filter((pr) => pr.status === 'needs_addressing');
+    const lastDigestTime = lastDigestAt ? new Date(lastDigestAt).getTime() : NaN;
     const reasonOrder = [
         'needs_response',
         'needs_changes',
@@ -191,7 +250,18 @@ export function collectActionableIssues(prs, snoozedUrls = new Set()) {
                     label = `[${reason}]`;
                     type = 'needs_response';
             }
-            issues.push({ type, pr, label });
+            // A PR is "new" if it was created after the last daily digest (first time seen).
+            // If there's no previous digest (first run) or createdAt is invalid, assume new.
+            const createdTime = new Date(pr.createdAt).getTime();
+            let isNewContribution;
+            if (isNaN(createdTime)) {
+                warn('daily-logic', `Invalid createdAt "${pr.createdAt}" for PR ${pr.url}, assuming new contribution`);
+                isNewContribution = true;
+            }
+            else {
+                isNewContribution = isNaN(lastDigestTime) || createdTime > lastDigestTime;
+            }
+            issues.push({ type, pr, label, isNewContribution });
         }
     }
     return issues;

package/dist/core/index.d.ts

@@ -12,7 +12,7 @@ export { parseGitHubUrl, daysBetween, splitRepo, isOwnRepo, getCLIVersion, getDa
 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';
+export { CRITICAL_STATUSES, applyStatusOverrides, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatActionHint, formatBriefSummary, formatSummary, printDigest, } from './daily-logic.js';
 export { computeContributionStats, type ContributionStats, type ComputeStatsInput } from './stats.js';
 export { fetchPRTemplate, type PRTemplateResult } from './pr-template.js';
 export * from './types.js';

package/dist/core/index.js

@@ -12,7 +12,7 @@ export { parseGitHubUrl, daysBetween, splitRepo, isOwnRepo, getCLIVersion, getDa
 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';
+export { CRITICAL_STATUSES, applyStatusOverrides, computeRepoSignals, groupPRsByRepo, assessCapacity, collectActionableIssues, computeActionMenu, toShelvedPRRef, formatActionHint, formatBriefSummary, formatSummary, printDigest, } from './daily-logic.js';
 export { computeContributionStats } from './stats.js';
 export { fetchPRTemplate } from './pr-template.js';
 export * from './types.js';

package/dist/formatters/json.d.ts

@@ -25,6 +25,8 @@ export interface ActionableIssue {
     type: ActionableIssueType;
     pr: FetchedPR;
     label: string;
+    /** True if the PR was created after the last daily digest (first time seen). */
+    isNewContribution: boolean;
 }
 /**
  * Compact version of ActionableIssue for JSON output.
@@ -36,6 +38,8 @@ export interface CompactActionableIssue {
     type: ActionableIssueType;
     prUrl: string;
     label: string;
+    /** True if the PR was created after the last daily digest (first time seen). */
+    isNewContribution: boolean;
 }
 /**
  * A single action menu item pre-computed by the CLI.

package/dist/formatters/json.js

@@ -30,6 +30,7 @@ export function compactActionableIssues(issues) {
         type: issue.type,
         prUrl: issue.pr.url,
         label: issue.label,
+        isNewContribution: issue.isNewContribution,
     }));
 }
 /**

package/package.json

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