Clear sticky salvage hard-block after parallel sibling succeeds

Two concurrent identical-brief task calls can both admit. If one
salvages and the other succeeds, recordOutcome kept the hard-block
lastSalvage sticky through the success, so the failing sibling's
sticky block later refused a brief that already produced a good
report in the same wave. A successful complete now always clears
lastSalvage, since the success itself proves the brief is
re-dispatchable.

Author: TheGreatAxios

src/subagent/brief-dispatch.ts

@@ -178,16 +178,11 @@ export function createBriefDispatchLedger(): BriefDispatchLedger {
         return;
       }
       if (salvage === null) {
-        // Successful complete resets the same-brief retry budget. Hard-block
-        // lastSalvage is sticky for the session and must not be cleared by a
-        // concurrent twin that finishes after thrash was already recorded.
-        if (existing.lastSalvage !== undefined && isHardBlockSalvage(existing.lastSalvage)) {
-          byFingerprint.set(fingerprint, {
-            dispatchCount: existing.dispatchCount,
-            lastSalvage: existing.lastSalvage,
-          });
-          return;
-        }
+        // CL-6710: a successful complete clears the sticky hard-block too.
+        // Two concurrent identical-brief dispatches can both admit; if one
+        // salvages and the other succeeds, the success proves the brief is
+        // re-dispatchable, so it must not leave the sibling's hard-block
+        // standing for the rest of the session.
         byFingerprint.set(fingerprint, { dispatchCount: 0 });
         return;
       }

src/subagent/index.test.ts

@@ -2183,7 +2183,7 @@ describe("brief re-dispatch ledger (CL-4343 / CL-5203)", () => {
     expect(second.dispatchCount).toBe(2);
   });
 
-  test("successful complete resets retry budget; hard-block is sticky", () => {
+  test("successful complete resets retry budget and clears soft salvage", () => {
     const ledger = createBriefDispatchLedger();
     const fp = fingerprintTaskBrief({ prompt: "ok job" });
     expect(ledger.admit(fp).ok).toBe(true);
@@ -2195,13 +2195,24 @@ describe("brief re-dispatch ledger (CL-4343 / CL-5203)", () => {
     expect(afterSuccess.ok).toBe(true);
     if (!afterSuccess.ok) throw new Error("expected admit");
     expect(afterSuccess.dispatchCount).toBe(1);
+  });
+
+  test("CL-6710: a parallel sibling success clears a hard-block salvage on the same fingerprint", () => {
+    const ledger = createBriefDispatchLedger();
+    const fp = fingerprintTaskBrief({ prompt: "parallel identical brief" });
+
+    // Two concurrent identical-brief dispatches both admit before either finishes.
+    expect(ledger.admit(fp).ok).toBe(true);
+    expect(ledger.admit(fp).ok).toBe(true);
 
-    // Hard-block salvage is sticky for the session — success on a concurrent twin must not clear it.
-    const stickyFp = fingerprintTaskBrief({ prompt: "no-progress sticky" });
-    ledger.admit(stickyFp);
-    ledger.recordOutcome(stickyFp, "no-progress");
-    ledger.recordOutcome(stickyFp, null);
-    expect(ledger.admit(stickyFp).ok).toBe(false);
+    // One sibling salvages (hard-block class)...
+    ledger.recordOutcome(fp, "no-progress");
+    // ...but the other sibling succeeds in the same wave.
+    ledger.recordOutcome(fp, null);
+
+    // The brief already produced a good report this wave — it must stay
+    // re-dispatchable, not stuck behind the losing sibling's hard-block.
+    expect(ledger.admit(fp).ok).toBe(true);
   });
 
   test("release undoes admit when run never produces a body", () => {

src/subagent/stop-policy.ts

@@ -452,8 +452,7 @@ const FORCED_STOP_SUMMARIES: Record<ForcedStopReason, string> = {
   deadline: "Stopped: wall-clock deadline reached before finishing.",
   stalled:
     "Stopped after a long silence with no tool activity. The parent can re-dispatch or check the background work directly.",
-  repetition:
-    "Stopped: degenerate repetition in streamed output (same window looping mid-turn).",
+  repetition: "Stopped: degenerate repetition in streamed output (same window looping mid-turn).",
   "incomplete-report": "Stopped: worker narrated instead of writing a report envelope.",
   "turn-budget": "Turn budget reached before finishing.",
 };