@oss-scout/core 1.2.2 → 1.2.3

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

@@ -108,6 +108,19 @@ export interface RecommendationInput {
      * instead of a competing-PR penalty.
      */
     ownPR: boolean;
+    /**
+     * The linked PR was already merged (#249 part B). The issue is effectively
+     * resolved, so it is a hard skip — surfacing it as a contribution
+     * opportunity is noise. Distinct from an open competing PR (which may be
+     * revivable). Defaults to false.
+     */
+    linkedPRMerged?: boolean;
+    /**
+     * The linked PR was closed without merging (#249 part B). The previous
+     * attempt was abandoned or rejected, so default to skip rather than
+     * re-surfacing it. Defaults to false.
+     */
+    linkedPRClosed?: boolean;
     notClaimed: boolean;
     clearRequirements: boolean;
     contributionGuidelinesFound: boolean;

package/dist/core/issue-vetting.js

@@ -37,7 +37,11 @@ export function deriveRecommendation(input) {
     if (!input.noExistingPR)
         notes.push(input.ownPR
             ? "Your PR is already in flight for this issue"
-            : "Existing PR found for this issue");
+            : input.linkedPRMerged
+                ? "A PR for this issue was already merged"
+                : input.linkedPRClosed
+                    ? "A PR for this issue was closed without merging"
+                    : "Existing PR found for this issue");
     if (!input.notClaimed)
         notes.push("Issue appears to be claimed by someone");
     if (input.existingPRInconclusive) {
@@ -57,8 +61,16 @@ export function deriveRecommendation(input) {
     if (!input.contributionGuidelinesFound)
         notes.push("No CONTRIBUTING.md found");
     // Reasons to skip / approve.
-    if (!input.noExistingPR)
-        reasonsToSkip.push(input.ownPR ? "You already have a PR in flight" : "Has existing PR");
+    if (!input.noExistingPR) {
+        if (input.ownPR)
+            reasonsToSkip.push("You already have a PR in flight");
+        else if (input.linkedPRMerged)
+            reasonsToSkip.push("Linked PR already merged");
+        else if (input.linkedPRClosed)
+            reasonsToSkip.push("Linked PR closed without merge");
+        else
+            reasonsToSkip.push("Has existing PR");
+    }
     if (!input.notClaimed)
         reasonsToSkip.push("Already claimed");
     if (!input.projectIsActive && !input.projectCheckFailed)
@@ -92,6 +104,13 @@ export function deriveRecommendation(input) {
     if (input.issueClosed) {
         recommendation = "skip";
     }
+    else if (input.linkedPRMerged || input.linkedPRClosed) {
+        // The issue is resolved (merged) or its attempt was abandoned/rejected
+        // (closed) — a hard skip, not a revive opportunity (#249 part B). An OPEN
+        // competing PR is deliberately NOT caught here; it falls through to the
+        // existing competing-PR handling below.
+        recommendation = "skip";
+    }
     else if (input.ownPR) {
         // You're already working on this; don't re-surface it as competition.
         recommendation = "skip";
@@ -201,6 +220,12 @@ export class IssueVetter {
             !linkedPR.merged &&
             username !== "" &&
             linkedPR.author.toLowerCase() === username.toLowerCase();
+        // Linked-PR lifecycle gate (#249 part B): a merged linked PR means the
+        // issue is resolved; a closed-unmerged one means the attempt was
+        // abandoned/rejected. Both are hard skips. (state === "closed" && merged
+        // is how buildLinkedPRFromTimelineEvent encodes a merged PR.)
+        const linkedPRMerged = !!linkedPR && linkedPR.merged;
+        const linkedPRClosed = !!linkedPR && linkedPR.state === "closed" && !linkedPR.merged;
         // Analyze issue quality
         const clearRequirements = analyzeRequirements(core.body);
         // When the health check itself failed (API error), use a neutral default:
@@ -263,6 +288,8 @@ export class IssueVetter {
         const { notes, reasonsToApprove, reasonsToSkip, recommendation } = deriveRecommendation({
             noExistingPR,
             ownPR,
+            linkedPRMerged,
+            linkedPRClosed,
             notClaimed,
             clearRequirements,
             contributionGuidelinesFound: !!contributionGuidelines,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oss-scout/core",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "Personalized GitHub issue finder with multi-strategy search, deep vetting, and viability scoring — CLI, library, MCP server, and Claude Code plugin",
   "type": "module",
   "bin": {