@oss-scout/core 1.2.1 → 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/dist/core/types.d.ts

@@ -281,6 +281,21 @@ export interface SearchOptions {
      * `interPhaseDelayMs` for the rationale (#143).
      */
     broadPhaseDelayMs?: number;
+    /**
+     * Exclude issues already surfaced by a recent search so consecutive
+     * searches rotate to fresh candidates instead of returning the same set
+     * (#249). A result counts as "recently surfaced" when its `lastSeenAt`
+     * (recorded by `saveResults`) is within `recentlySurfacedTtlDays`.
+     * Defaults to `true`. Pass `false` to force-resurface (e.g. an explicit
+     * "search the same pool again" request).
+     */
+    excludeRecentlySurfaced?: boolean;
+    /**
+     * TTL in days for the `excludeRecentlySurfaced` rotation window (#249).
+     * Results last surfaced more than this many days ago are eligible to
+     * resurface. Defaults to 7.
+     */
+    recentlySurfacedTtlDays?: number;
 }
 /** Result of a search operation. */
 export interface SearchResult {

package/dist/scout.js

@@ -170,6 +170,18 @@ export class OssScout {
         // Auto-cull expired skips before searching
         this.cullExpiredSkips();
         const skippedUrls = new Set((this.state.skippedIssues ?? []).map((s) => s.url));
+        // Rotation (#249): also exclude issues surfaced by a recent search so
+        // consecutive searches return fresh candidates instead of the same set.
+        // Folded into the same exclusion set the issue filter already honors.
+        if (options?.excludeRecentlySurfaced ?? true) {
+            const ttlDays = options?.recentlySurfacedTtlDays ?? 7;
+            const cutoff = Date.now() - ttlDays * 24 * 60 * 60 * 1000;
+            for (const r of this.state.savedResults ?? []) {
+                const seen = Date.parse(r.lastSeenAt);
+                if (!Number.isNaN(seen) && seen >= cutoff)
+                    skippedUrls.add(r.issueUrl);
+            }
+        }
         const discovery = new IssueDiscovery(this.githubToken, this.state.preferences, this);
         // Per-call flags override the persisted personalization defaults (#168).
         // An empty preference array reads as "no boost" just like an absent flag.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oss-scout/core",
-  "version": "1.2.1",
+  "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": {