@oss-autopilot/core 0.42.3 → 0.42.4

package/dist/cli.bundle.cjs

@@ -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);

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/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/package.json

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