@oss-autopilot/core 0.42.2 → 0.42.4

package/dist/cli.bundle.cjs

@@ -4387,10 +4387,10 @@ var init_state = __esm({
       }
       // === Dismiss / Undismiss Issues ===
       /**
-       * Dismiss an issue by URL. Dismissed issues are excluded from `new_response` notifications
+       * Dismiss an issue or PR by URL. Dismissed URLs are excluded from `new_response` notifications
        * until new activity occurs after the dismiss timestamp.
-       * @param url - The full GitHub issue URL.
-       * @param timestamp - ISO timestamp of when the issue was dismissed.
+       * @param url - The full GitHub issue or PR URL.
+       * @param timestamp - ISO timestamp of when the issue/PR was dismissed.
        * @returns true if newly dismissed, false if already dismissed.
        */
       dismissIssue(url, timestamp) {
@@ -4404,8 +4404,8 @@ var init_state = __esm({
         return true;
       }
       /**
-       * Undismiss an issue by URL.
-       * @param url - The full GitHub issue URL.
+       * Undismiss an issue or PR by URL.
+       * @param url - The full GitHub issue or PR URL.
        * @returns true if found and removed, false if not dismissed.
        */
       undismissIssue(url) {
@@ -4416,8 +4416,8 @@ var init_state = __esm({
         return true;
       }
       /**
-       * Get the timestamp when an issue was dismissed.
-       * @param url - The full GitHub issue URL.
+       * Get the timestamp when an issue or PR was dismissed.
+       * @param url - The full GitHub issue or PR URL.
        * @returns The ISO dismiss timestamp, or undefined if not dismissed.
        */
       getIssueDismissedAt(url) {
@@ -11213,7 +11213,24 @@ var init_pr_monitor = __esm({
         );
         const ciPromise = this.getCIStatus(owner, repo, ghPR.head.sha);
         const needCommitDate = hasUnrespondedComment || reviewDecision === "changes_requested";
-        const commitDatePromise = needCommitDate ? this.octokit.repos.getCommit({ owner, repo, ref: ghPR.head.sha }).then((res) => res.data.commit.author?.date).catch(() => void 0) : Promise.resolve(void 0);
+        const commitDatePromise = needCommitDate ? this.octokit.repos.getCommit({ owner, repo, ref: ghPR.head.sha }).then((res) => res.data.commit.author?.date).catch((err) => {
+          const status2 = getHttpStatusCode(err);
+          if (status2 === 429) throw err;
+          if (status2 === 403) {
+            const msg = errorMessage(err).toLowerCase();
+            if (msg.includes("rate limit") || msg.includes("abuse detection")) throw err;
+            warn(
+              "pr-monitor",
+              `403 fetching commit date for ${owner}/${repo}@${ghPR.head.sha.slice(0, 7)}: ${errorMessage(err)}`
+            );
+            return void 0;
+          }
+          warn(
+            "pr-monitor",
+            `Failed to fetch commit date for ${owner}/${repo}@${ghPR.head.sha.slice(0, 7)}: ${errorMessage(err)}`
+          );
+          return void 0;
+        }) : Promise.resolve(void 0);
         const [{ status: ciStatus, failingCheckNames, failingCheckConclusions }, latestCommitDate] = await Promise.all([
           ciPromise,
           commitDatePromise
@@ -13685,11 +13702,12 @@ function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, fa
       const dismissTime = new Date(dismissedAt).getTime();
       if (isNaN(responseTime) || isNaN(dismissTime)) {
         console.error(`[DAILY] Invalid timestamp in dismiss check for ${issue.url}, including issue`);
-        stateManager2.undismissIssue(issue.url);
-        hasAutoUndismissed = true;
         return true;
       }
       if (responseTime > dismissTime) {
+        console.error(
+          `[DAILY] Auto-undismissing issue ${issue.url}: new response at ${issue.lastResponseAt} after dismiss at ${dismissedAt}`
+        );
         stateManager2.undismissIssue(issue.url);
         hasAutoUndismissed = true;
         return true;
@@ -13697,9 +13715,6 @@ function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, fa
     }
     return false;
   });
-  if (hasAutoUndismissed) {
-    stateManager2.save();
-  }
   const issueResponses = filteredCommentedIssues.filter(
     (i) => i.status === "new_response"
   );
@@ -13707,8 +13722,32 @@ function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, fa
   const snoozedUrls = new Set(
     Object.keys(stateManager2.getState().config.snoozedPRs ?? {}).filter((url) => stateManager2.isSnoozed(url))
   );
-  const dismissedUrls = new Set(Object.keys(stateManager2.getState().config.dismissedIssues ?? {}));
-  const nonDismissedPRs = activePRs.filter((pr) => !dismissedUrls.has(pr.url));
+  const nonDismissedPRs = activePRs.filter((pr) => {
+    const dismissedAt = stateManager2.getIssueDismissedAt(pr.url);
+    if (!dismissedAt) return true;
+    const activityTime = new Date(pr.updatedAt).getTime();
+    const dismissTime = new Date(dismissedAt).getTime();
+    if (isNaN(activityTime) || isNaN(dismissTime)) {
+      console.error(`[DAILY] Invalid timestamp in PR dismiss check for ${pr.url}, including PR`);
+      return true;
+    }
+    if (activityTime > dismissTime) {
+      console.error(
+        `[DAILY] Auto-undismissing PR ${pr.url}: new activity at ${pr.updatedAt} after dismiss at ${dismissedAt}`
+      );
+      stateManager2.undismissIssue(pr.url);
+      hasAutoUndismissed = true;
+      return true;
+    }
+    return false;
+  });
+  if (hasAutoUndismissed) {
+    try {
+      stateManager2.save();
+    } catch (error) {
+      console.error("[DAILY] Failed to persist auto-undismissed state:", errorMessage(error));
+    }
+  }
   const actionableIssues = collectActionableIssues(nonDismissedPRs, snoozedUrls);
   digest.summary.totalNeedingAttention = actionableIssues.length;
   const briefSummary = formatBriefSummary(digest, actionableIssues.length, issueResponses.length);
@@ -17199,7 +17238,6 @@ var init_shelve = __esm({
 // src/commands/dismiss.ts
 var dismiss_exports = {};
 __export(dismiss_exports, {
-  ISSUE_OR_PR_URL_PATTERN: () => ISSUE_OR_PR_URL_PATTERN,
   runDismiss: () => runDismiss,
   runUndismiss: () => runUndismiss
 });

package/dist/commands/daily.js

@@ -305,14 +305,14 @@ function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, fa
             const responseTime = new Date(issue.lastResponseAt).getTime();
             const dismissTime = new Date(dismissedAt).getTime();
             if (isNaN(responseTime) || isNaN(dismissTime)) {
-                // Invalid timestamp — fail open (include issue to be safe)
+                // Invalid timestamp — fail open (include issue to be safe) without
+                // permanently removing dismiss record (may be a transient data issue)
                 console.error(`[DAILY] Invalid timestamp in dismiss check for ${issue.url}, including issue`);
-                stateManager.undismissIssue(issue.url);
-                hasAutoUndismissed = true;
                 return true;
             }
             if (responseTime > dismissTime) {
                 // New activity after dismiss — auto-undismiss and resurface
+                console.error(`[DAILY] Auto-undismissing issue ${issue.url}: new response at ${issue.lastResponseAt} after dismiss at ${dismissedAt}`);
                 stateManager.undismissIssue(issue.url);
                 hasAutoUndismissed = true;
                 return true;
@@ -321,15 +321,41 @@ function generateDigestOutput(digest, activePRs, shelvedPRs, commentedIssues, fa
         // Still dismissed (last response is at or before dismiss timestamp)
         return false;
     });
-    if (hasAutoUndismissed) {
-        stateManager.save();
-    }
     const issueResponses = filteredCommentedIssues.filter((i) => i.status === 'new_response');
     const summary = formatSummary(digest, capacity, issueResponses);
     const snoozedUrls = new Set(Object.keys(stateManager.getState().config.snoozedPRs ?? {}).filter((url) => stateManager.isSnoozed(url)));
-    // Filter dismissed PR URLs from actionable issues (#416)
-    const dismissedUrls = new Set(Object.keys(stateManager.getState().config.dismissedIssues ?? {}));
-    const nonDismissedPRs = activePRs.filter((pr) => !dismissedUrls.has(pr.url));
+    // Filter dismissed PRs: suppress if dismissed after last activity, auto-undismiss if new activity (#416, #468)
+    const nonDismissedPRs = activePRs.filter((pr) => {
+        const dismissedAt = stateManager.getIssueDismissedAt(pr.url);
+        if (!dismissedAt)
+            return true; // Not dismissed — include
+        const activityTime = new Date(pr.updatedAt).getTime();
+        const dismissTime = new Date(dismissedAt).getTime();
+        if (isNaN(activityTime) || isNaN(dismissTime)) {
+            // Invalid timestamp — fail open (include PR to be safe) without
+            // permanently removing dismiss record (may be a transient data issue)
+            console.error(`[DAILY] Invalid timestamp in PR dismiss check for ${pr.url}, including PR`);
+            return true;
+        }
+        if (activityTime > dismissTime) {
+            // New activity after dismiss — auto-undismiss and resurface
+            console.error(`[DAILY] Auto-undismissing PR ${pr.url}: new activity at ${pr.updatedAt} after dismiss at ${dismissedAt}`);
+            stateManager.undismissIssue(pr.url);
+            hasAutoUndismissed = true;
+            return true;
+        }
+        // Still dismissed (last activity is at or before dismiss timestamp)
+        return false;
+    });
+    // Persist auto-undismiss state changes (issue + PR combined into one save)
+    if (hasAutoUndismissed) {
+        try {
+            stateManager.save();
+        }
+        catch (error) {
+            console.error('[DAILY] Failed to persist auto-undismissed state:', errorMessage(error));
+        }
+    }
     const actionableIssues = collectActionableIssues(nonDismissedPRs, snoozedUrls);
     digest.summary.totalNeedingAttention = actionableIssues.length;
     const briefSummary = formatBriefSummary(digest, actionableIssues.length, issueResponses.length);

package/dist/commands/dismiss.d.ts

@@ -3,7 +3,6 @@
  * Manages dismissing issue and PR notifications without posting a comment.
  * Dismissed URLs resurface automatically when new responses arrive after the dismiss timestamp.
  */
-import { ISSUE_OR_PR_URL_PATTERN } from './validation.js';
 export interface DismissOutput {
     dismissed: boolean;
     url: string;
@@ -12,7 +11,6 @@ export interface UndismissOutput {
     undismissed: boolean;
     url: string;
 }
-export { ISSUE_OR_PR_URL_PATTERN };
 export declare function runDismiss(options: {
     url: string;
 }): Promise<DismissOutput>;

package/dist/commands/dismiss.js

@@ -5,8 +5,6 @@
  */
 import { getStateManager } from '../core/index.js';
 import { ISSUE_OR_PR_URL_PATTERN, validateGitHubUrl, validateUrl } from './validation.js';
-// Re-export for tests
-export { ISSUE_OR_PR_URL_PATTERN };
 export async function runDismiss(options) {
     validateUrl(options.url);
     validateGitHubUrl(options.url, ISSUE_OR_PR_URL_PATTERN, 'issue or PR');

package/dist/core/pr-monitor.js

@@ -204,7 +204,23 @@ export class PRMonitor {
             ? this.octokit.repos
                 .getCommit({ owner, repo, ref: ghPR.head.sha })
                 .then((res) => res.data.commit.author?.date)
-                .catch(() => undefined)
+                .catch((err) => {
+                // Rate limit errors must propagate — silently swallowing them produces
+                // misleading status (e.g. needs_changes when changes were addressed) (#469).
+                const status = getHttpStatusCode(err);
+                if (status === 429)
+                    throw err;
+                if (status === 403) {
+                    const msg = errorMessage(err).toLowerCase();
+                    if (msg.includes('rate limit') || msg.includes('abuse detection'))
+                        throw err;
+                    // Non-rate-limit 403 (DMCA, private repo, SSO) — degrade gracefully
+                    warn('pr-monitor', `403 fetching commit date for ${owner}/${repo}@${ghPR.head.sha.slice(0, 7)}: ${errorMessage(err)}`);
+                    return undefined;
+                }
+                warn('pr-monitor', `Failed to fetch commit date for ${owner}/${repo}@${ghPR.head.sha.slice(0, 7)}: ${errorMessage(err)}`);
+                return undefined;
+            })
             : Promise.resolve(undefined);
         const [{ status: ciStatus, failingCheckNames, failingCheckConclusions }, latestCommitDate] = await Promise.all([
             ciPromise,

package/dist/core/review-analysis.js

@@ -108,7 +108,8 @@ export function checkUnrespondedComments(comments, reviews, reviewComments, user
         if (!body && review.state !== 'COMMENTED' && review.state !== 'CHANGES_REQUESTED')
             continue;
         const author = review.user?.login || 'unknown';
-        // For inline-only COMMENTED reviews, skip pure self-replies (#199)
+        // For inline-only COMMENTED reviews, skip pure self-replies (#199).
+        // CHANGES_REQUESTED reviews are always actionable regardless of self-replies.
         if (!body && review.state === 'COMMENTED' && review.id != null) {
             if (isAllSelfReplies(review.id, reviewComments)) {
                 continue;

package/dist/core/state.d.ts

@@ -206,22 +206,22 @@ export declare class StateManager {
      */
     isPRShelved(url: string): boolean;
     /**
-     * Dismiss an issue by URL. Dismissed issues are excluded from `new_response` notifications
+     * Dismiss an issue or PR by URL. Dismissed URLs are excluded from `new_response` notifications
      * until new activity occurs after the dismiss timestamp.
-     * @param url - The full GitHub issue URL.
-     * @param timestamp - ISO timestamp of when the issue was dismissed.
+     * @param url - The full GitHub issue or PR URL.
+     * @param timestamp - ISO timestamp of when the issue/PR was dismissed.
      * @returns true if newly dismissed, false if already dismissed.
      */
     dismissIssue(url: string, timestamp: string): boolean;
     /**
-     * Undismiss an issue by URL.
-     * @param url - The full GitHub issue URL.
+     * Undismiss an issue or PR by URL.
+     * @param url - The full GitHub issue or PR URL.
      * @returns true if found and removed, false if not dismissed.
      */
     undismissIssue(url: string): boolean;
     /**
-     * Get the timestamp when an issue was dismissed.
-     * @param url - The full GitHub issue URL.
+     * Get the timestamp when an issue or PR was dismissed.
+     * @param url - The full GitHub issue or PR URL.
      * @returns The ISO dismiss timestamp, or undefined if not dismissed.
      */
     getIssueDismissedAt(url: string): string | undefined;

package/dist/core/state.js

@@ -721,10 +721,10 @@ export class StateManager {
     }
     // === Dismiss / Undismiss Issues ===
     /**
-     * Dismiss an issue by URL. Dismissed issues are excluded from `new_response` notifications
+     * Dismiss an issue or PR by URL. Dismissed URLs are excluded from `new_response` notifications
      * until new activity occurs after the dismiss timestamp.
-     * @param url - The full GitHub issue URL.
-     * @param timestamp - ISO timestamp of when the issue was dismissed.
+     * @param url - The full GitHub issue or PR URL.
+     * @param timestamp - ISO timestamp of when the issue/PR was dismissed.
      * @returns true if newly dismissed, false if already dismissed.
      */
     dismissIssue(url, timestamp) {
@@ -738,8 +738,8 @@ export class StateManager {
         return true;
     }
     /**
-     * Undismiss an issue by URL.
-     * @param url - The full GitHub issue URL.
+     * Undismiss an issue or PR by URL.
+     * @param url - The full GitHub issue or PR URL.
      * @returns true if found and removed, false if not dismissed.
      */
     undismissIssue(url) {
@@ -750,8 +750,8 @@ export class StateManager {
         return true;
     }
     /**
-     * Get the timestamp when an issue was dismissed.
-     * @param url - The full GitHub issue URL.
+     * Get the timestamp when an issue or PR was dismissed.
+     * @param url - The full GitHub issue or PR URL.
      * @returns The ISO dismiss timestamp, or undefined if not dismissed.
      */
     getIssueDismissedAt(url) {

package/dist/core/types.d.ts

@@ -451,7 +451,7 @@ export interface AgentConfig {
     aiPolicyBlocklist?: string[];
     /** PR URLs manually shelved by the user. Shelved PRs are excluded from capacity and actionable issues. Auto-unshelved when maintainers engage. */
     shelvedPRUrls?: string[];
-    /** Issue URLs dismissed by the user, mapped to ISO timestamp of when dismissed. Issues with new responses after the dismiss timestamp resurface automatically. */
+    /** Issue/PR URLs dismissed by the user, mapped to ISO timestamp of when dismissed. Issues with new responses after the dismiss timestamp resurface automatically. Named dismissedIssues for state backward compatibility (#416). */
     dismissedIssues?: Record<string, string>;
     /** PR URLs with snoozed CI failures, mapped to snooze metadata. Snoozed PRs are excluded from actionable CI failure list until expiry. */
     snoozedPRs?: Record<string, SnoozeInfo>;

package/package.json

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