commandmate 0.46.0 → 0.47.0

package/dist/server/src/lib/polling/response-checker.js

@@ -49,6 +49,7 @@ const session_name_1 = require("../../lib/cli-tools/session-name");
 const geometry_delegation_1 = require("../../lib/tmux/geometry-delegation");
 const tmux_session_surface_1 = require("../../lib/session/tmux-session-surface");
 const response_poller_core_1 = require("./response-poller-core");
+const poller_owner_1 = require("./poller-owner");
 const push_1 = require("../../lib/push");
 // Issue #1790: imported by deep path, not through `@/lib/push`. Suites that
 // replace the barrel to count notifications (e.g. the #1547 escalation test)
@@ -227,6 +228,15 @@ function detectPromptOnCleanFrame(cleanOutput, cliToolId, precomputedLines, rawF
                 ? { isPrompt: false, cleanContent: cleanOutput.trim() }
                 : dialog;
         }
+        // Issue #3808: a numbered agy screen the reader declines — the login-method
+        // screen, a permission dialog whose `tab Amend` footer a narrow pane cut or
+        // left below the pane — is not handed to the generic parser either, which
+        // reads `> 1. Google OAuth` as an answerable option. Judged without the
+        // footer, so a missing footer cannot reopen the generic path. The status
+        // verdict publishes the same frame as unclassified (`tools/antigravity/detect.ts`).
+        if ((0, dialog_1.isAntigravityUnvouchedNumberedScreen)(cleanOutput)) {
+            return { isPrompt: false, cleanContent: cleanOutput.trim() };
+        }
     }
     // Issue #2522: Command Code's footer-less `AskUserQuestion`, read by the same
     // module `tools/command-code/detect.ts` publishes its status from — so the
@@ -449,6 +459,48 @@ function markOrSettleStructuredHistoryRecheck(pollerKey, recorded) {
         (0, response_dedup_1.markStructuredHistoryRecheckPending)(pollerKey);
     }
 }
+/**
+ * Whether this tick may still act for `pollerKey` after the `await` it has just
+ * resumed from, logging the refusal when it may not (Issue #3705).
+ *
+ * #3663, #3684 and #3701 asked {@link mayActForPollerKey} at the writes that had
+ * been reported, one at a time. A tick resumes from an `await` with no idea
+ * whether its chain was stopped, cut loose past `STALLED_TICK_TIMEOUT` or moved
+ * by a worktree-ID migration in the meantime, and every write after that
+ * `await` — the prompt sweep, the waiting edge, the completion push, the cursor,
+ * the fault notifier, the TUI accumulator, the recheck and skip-streak state —
+ * lands under a key it no longer owns, on whatever the successor has put there.
+ * So the question is asked after every `await` of the tick, named by the point
+ * it resumed from, and the writes below each point run only on a yes.
+ *
+ * Not asked where a self-stop hands back its verdict (`return await
+ * recordCompletedResponse(…)`): `runPollTick` settles the pause from that
+ * value, and a stop the tick raised itself must not lose what it recorded.
+ *
+ * @param ctx - What this tick is keyed by
+ * @param after - The `await` the tick has just resumed from, for the log line
+ */
+function mayStillActAfter(ctx, after) {
+    if ((0, poller_owner_1.mayActForPollerKey)(ctx.pollerKey))
+        return true;
+    // One line per retired tick, at the first point that turned it away: a tick
+    // refused at the capture is asked again after the reader and the log.
+    const owner = (0, poller_owner_1.currentPollerOwner)();
+    if (owner === undefined || !refusalLogged.has(owner)) {
+        if (owner)
+            refusalLogged.add(owner);
+        logger.info('retired-tick-refused', {
+            worktreeId: ctx.worktreeId,
+            cliToolId: ctx.cliToolId,
+            instanceId: ctx.resolvedInstanceId,
+            generation: owner?.generation,
+            after,
+        });
+    }
+    return false;
+}
+/** The retired chains {@link mayStillActAfter} has already logged (Issue #3705). */
+const refusalLogged = new WeakSet();
 /**
  * Read the frame this tick captured: is there a finished turn on it that the
  * cursor has not already counted?
@@ -469,17 +521,25 @@ function markOrSettleStructuredHistoryRecheck(pollerKey, recorded) {
  * @param ctx - What this tick is keyed by
  * @param output - The capture this tick made
  * @param lastCapturedLine - Its cursor, 0 when there is none
+ * @param owns - Whether the tick still owns its key (Issue #3705); false reads
+ *   the frame and writes nothing
  * @returns What the tick goes on with, or `false` when the tick ends here
  */
-function extractCompletedTurn(ctx, output, lastCapturedLine) {
+function extractCompletedTurn(ctx, output, lastCapturedLine, owns) {
     const { db, worktreeId, cliToolId, instanceId, resolvedInstanceId, pollerKey } = ctx;
     // Issue #2000: the frame is in hand, so this is the cheapest place to ask
     // whether the model API has stalled the session. Level in, edge out — see
     // the helper.
-    observeUpstreamFaultForPush(worktreeId, cliToolId, instanceId, output);
+    //
+    // Issue #3705: every write in this function is the caller's `owns` — asked
+    // right after the capture, with no `await` since. A retired tick's frame is
+    // read, and none of it lands on the fault episode, the accumulator, the skip
+    // streaks or the pending prompts of the key it lost.
+    if (owns)
+        observeUpstreamFaultForPush(worktreeId, cliToolId, instanceId, output);
     // Layer 2: Accumulate TUI content for full-screen TUI tools, so a turn that
     // outgrows the alternate-screen pane keeps the head that has scrolled away.
-    if (cliToolId === 'opencode' || cliToolId === 'copilot') {
+    if (owns && (cliToolId === 'opencode' || cliToolId === 'copilot')) {
         // Issue #1911: opencode is accumulated from the CURRENT TURN'S REGION
         // rather than the whole frame. Feeding the raw pane seeded the accumulator
         // with the previous turn's transcript, the echoed prompt and the bottom
@@ -501,7 +561,7 @@ function extractCompletedTurn(ctx, output, lastCapturedLine) {
         //
         // Issue #3552: the same frame ends a run of duplicate response ticks for
         // the same reason — an unfinished frame is not the saved reply.
-        if (result) {
+        if (owns && result) {
             (0, prompt_dedup_1.resetDuplicatePromptSkipStreak)(pollerKey);
             (0, response_dedup_1.resetDuplicateResponseSkipStreak)(pollerKey);
         }
@@ -509,7 +569,7 @@ function extractCompletedTurn(ctx, output, lastCapturedLine) {
         const { thinkingPattern } = (0, cli_patterns_1.getCliToolPatterns)(cliToolId);
         const cleanOutput = (0, cli_patterns_1.stripAnsi)(output);
         const tailLines = cleanOutput.split('\n').slice(-thinking_constants_1.THINKING_TAIL_LINE_COUNT).join('\n');
-        if (thinkingPattern.test(tailLines)) {
+        if (owns && thinkingPattern.test(tailLines)) {
             // Intended twin of the sweep in recordCompletedResponse: this one runs when
             // working is visible on an unfinished frame, that one when a response is
             // recorded (Issue #31).
@@ -570,6 +630,21 @@ function extractCompletedTurn(ctx, output, lastCapturedLine) {
  */
 function savePromptMessage(ctx, promptDetection, result, isFullScreenTui) {
     const { db, worktree, worktreeId, cliToolId, instanceId, resolvedInstanceId, pollerKey } = ctx;
+    // Issue #3701: the question #3663 / #3684 ask, for the prompt a tick retired
+    // during the capture's `await` then finds on the pane — no row, broadcast,
+    // task event, push or waiting episode under the key it lost. Asked BEFORE the
+    // dedup below, which registers the hash before the save: a refused tick that
+    // registered it would make the successor drop the same prompt as a
+    // duplicate. No `await` from here to the end of this function.
+    if (!(0, poller_owner_1.mayActForPollerKey)(pollerKey)) {
+        logger.info('prompt-save-refused', {
+            worktreeId,
+            cliToolId,
+            instanceId: resolvedInstanceId,
+            generation: (0, poller_owner_1.currentPollerOwner)()?.generation,
+        });
+        return false;
+    }
     // Issue #565: Content hash-based duplicate prompt prevention
     const promptContent = promptDetection.rawContent || promptDetection.cleanContent;
     const normalizedForDedup = (0, prompt_dedup_1.normalizePromptForDedup)(promptContent, cliToolId);
@@ -686,9 +761,10 @@ function savePromptMessage(ctx, promptDetection, result, isFullScreenTui) {
  * @param cliToolId - CLI tool identifier
  * @param result - The complete extraction this tick made
  * @param pollerKey - Poller key ("worktreeId:instanceId")
+ * @param owns - Whether the tick still owns its key (Issue #3705)
  * @returns The cleaned response
  */
-function cleanCompletedResponse(cliToolId, result, pollerKey) {
+function cleanCompletedResponse(cliToolId, result, pollerKey, owns) {
     // Clean up responses
     let cleanedResponse = result.response;
     if (cliToolId === 'gemini') {
@@ -702,7 +778,10 @@ function cleanCompletedResponse(cliToolId, result, pollerKey) {
         const sourceContent = accumulatedContent || result.response;
         cleanedResponse = (0, response_cleaner_1.cleanCopilotResponse)(sourceContent);
         cleanedResponse = (0, response_cleaner_1.truncateMessage)(cleanedResponse, copilot_constants_1.COPILOT_MAX_MESSAGE_LENGTH, copilot_constants_1.COPILOT_TRUNCATION_MARKER);
-        (0, tui_accumulator_1.clearTuiAccumulator)(pollerKey);
+        // Issue #3705: read, but not cleared, by a tick that lost the key — the
+        // successor may already be accumulating its own turn under it.
+        if (owns)
+            (0, tui_accumulator_1.clearTuiAccumulator)(pollerKey);
     }
     else if (cliToolId === 'opencode') {
         // Issue #1911 defect 3: opencode wrote to the Layer-2 accumulator but never
@@ -724,7 +803,8 @@ function cleanCompletedResponse(cliToolId, result, pollerKey) {
             ? accumulatedContent
             : result.response;
         cleanedResponse = (0, response_cleaner_1.cleanOpenCodeResponse)(sourceContent);
-        (0, tui_accumulator_1.clearTuiAccumulator)(pollerKey);
+        if (owns)
+            (0, tui_accumulator_1.clearTuiAccumulator)(pollerKey);
     }
     return cleanedResponse;
 }
@@ -744,6 +824,15 @@ function cleanCompletedResponse(cliToolId, result, pollerKey) {
  */
 async function recheckDuplicateResponse(ctx, result, claudeMetadata) {
     const { db, worktree, worktreeId, cliToolId, instanceId, resolvedInstanceId, pollerKey } = ctx;
+    // Issue #3705 (review): a retired tick reaches this function too. The dedup
+    // answers "duplicate" on a hash match BEFORE it asks about ownership (it only
+    // refuses to REGISTER a hash), and the hash it matches may be the successor's
+    // — the same reply, saved by the chain that replaced this one while this
+    // tick's capture was in flight. Everything below writes under the key (the
+    // skip-log claim, the cursor, the recheck claim, then the reader), so the
+    // question is asked first, with no `await` between it and those writes.
+    if (!mayStillActAfter(ctx, 'capture'))
+        return false;
     // Issue #1695: this branch used to drop the response silently — the
     // prompt-side guard above has logged its skip since #565, this one
     // logged nothing at all, so a reply that never reached History left no
@@ -821,6 +910,11 @@ async function recheckDuplicateResponse(ctx, result, claudeMetadata) {
             worktreePath: worktree.path,
             transcriptPathHint: claudeMetadata?.logFilePath ?? null,
         });
+        // Issue #3705: the recheck the successor is owed is not settled, and its
+        // hold not dropped, by a tick retired while the reader ran. What the reader
+        // answered is still the answer (it asks the same question before it writes).
+        if (!mayStillActAfter(ctx, 'recheck-reader'))
+            return recaptured;
         if (recaptured) {
             (0, response_dedup_1.settleStructuredHistoryRecheck)(pollerKey);
             // Issue #2436: the turn is now the agent's own Markdown, so a
@@ -868,7 +962,10 @@ function recordOrHoldScrapedResponse(ctx, cleanedResponse, claudeMetadata, decis
     // width and gutter-prefixed, so no content check could ever recognise them
     // as the same reply.
     if (holdScrapedHistory) {
-        (0, pending_scraped_response_1.holdScrapedResponse)(pollerKey, {
+        // Issue #3663: refused (false) to a tick whose chain was retired while it
+        // awaited the reader — a stop, a stalled-tick restart or a worktree-ID
+        // migration. The store logs that, so this tick does not claim a hold.
+        const held = (0, pending_scraped_response_1.holdScrapedResponse)(pollerKey, {
             worktreeId,
             cliToolId,
             instanceId: resolvedInstanceId,
@@ -884,19 +981,37 @@ function recordOrHoldScrapedResponse(ctx, cleanedResponse, claudeMetadata, decis
             transcriptPathHint: claudeMetadata?.logFilePath ?? null,
             expiresAt: Date.now() + pending_scraped_response_1.PENDING_SCRAPE_HOLD_MS,
         });
-        logger.info('structured-history-scrape-held', {
+        if (held) {
+            logger.info('structured-history-scrape-held', {
+                worktreeId,
+                cliToolId,
+                instanceId: resolvedInstanceId,
+                scrapedLength: cleanedResponse.length,
+                holdMs: pending_scraped_response_1.PENDING_SCRAPE_HOLD_MS,
+            });
+            // The completion edge is announced HERE and not at the flush, because the
+            // turn finished now. A relay waiting on this session gets #2401's grace
+            // window to find a turn-keyed row, which is exactly the row the hold is
+            // waiting for; delaying the announcement by the hold would delay every
+            // delivery by it too. Issue #3684: only for a hold that was taken — a
+            // refused one is a retired tick, which announces nothing under the key
+            // it lost, as in the immediate-save branch below.
+            (0, relay_triggers_1.onRelayTurnCompleted)(worktreeId, cliToolId, resolvedInstanceId, false);
+        }
+    }
+    else if (!suppressScrapedHistory && !(0, poller_owner_1.mayActForPollerKey)(pollerKey)) {
+        // Issue #3684: the same question the hold asks (#3663), for the reply saved
+        // at once — a tick retired while it awaited the reader or the log writes no
+        // row, broadcast or relay edge under the key it lost. No `await` between
+        // this check and the three below. Issue #3705: the caller now asks the
+        // same question for the rest of the tick's writes and the Markdown log for
+        // its own, so a retired tick reaches this branch only to be refused here.
+        logger.info('scraped-response-save-refused', {
             worktreeId,
             cliToolId,
             instanceId: resolvedInstanceId,
-            scrapedLength: cleanedResponse.length,
-            holdMs: pending_scraped_response_1.PENDING_SCRAPE_HOLD_MS,
+            generation: (0, poller_owner_1.currentPollerOwner)()?.generation,
         });
-        // The completion edge is announced HERE and not at the flush, because the
-        // turn finished now. A relay waiting on this session gets #2401's grace
-        // window to find a turn-keyed row, which is exactly the row the hold is
-        // waiting for; delaying the announcement by the hold would delay every
-        // delivery by it too.
-        (0, relay_triggers_1.onRelayTurnCompleted)(worktreeId, cliToolId, resolvedInstanceId, false);
     }
     else if (!suppressScrapedHistory) {
         // Create new CLI tool message in database
@@ -999,7 +1114,13 @@ async function recordCompletedResponse(ctx, turn, claudeMetadata, cleanedRespons
     // a return — so unless the fact is written down now, the ask never happens
     // again. A `true` settles it: the turn is recorded and there is nothing left
     // to re-ask about.
-    markOrSettleStructuredHistoryRecheck(pollerKey, structuredHistoryLive);
+    //
+    // Issue #3705: by a tick that still owns the key. A retired one goes on to
+    // the scrape decision below only so that the store's and the save branch's
+    // own refusals (#3663, #3684) speak for it; nothing else it would write does.
+    if (mayStillActAfter(ctx, 'reader')) {
+        markOrSettleStructuredHistoryRecheck(pollerKey, structuredHistoryLive);
+    }
     // Issue #2317 Phase D: the scrape is dropped while the geometry is
     // delegated, and the transcript capture above is what makes that safe.
     //
@@ -1030,8 +1151,29 @@ async function recordCompletedResponse(ctx, turn, claudeMetadata, cleanedRespons
     // out. Only the two writes that RECORD THE REPLY wait.
     const holdScrapedHistory = !suppressScrapedHistory && captureReport.outcome === 'not_yet_closed';
     // Create Markdown log file for the conversation pair
+    //
+    // Issue #3705: keyed, so the log asks before its own write (after its own
+    // awaits) whether this tick still owns the key. A held reply confirmed by a
+    // stop is logged by the store with no key (`pending-scraped-response`), and
+    // that log is kept.
     if (cleanedResponse && !suppressScrapedHistory && !holdScrapedHistory) {
-        await (0, conversation_logger_1.recordClaudeConversation)(db, worktreeId, cleanedResponse, cliToolId);
+        await (0, conversation_logger_1.recordClaudeConversation)(db, worktreeId, cleanedResponse, cliToolId, { pollerKey });
+    }
+    // Issue #3705: everything from here to the end of the function runs without
+    // an `await`, so this is the tick's last chance to find out that it was
+    // retired while the reader or the log ran. A retired tick sweeps no prompt to
+    // "answered", closes no waiting episode, sends no completion push and moves
+    // no cursor under the key it lost — the successor's, possibly. The scrape
+    // decision still runs: every branch of it asks the same question itself
+    // (#3663 the hold, #3684 the immediate save, the store the discard), and its
+    // refusal lines are what the earlier suites read.
+    if (!mayStillActAfter(ctx, 'conversation-log')) {
+        recordOrHoldScrapedResponse(ctx, cleanedResponse, claudeMetadata, {
+            structuredHistoryLive,
+            suppressScrapedHistory,
+            holdScrapedHistory,
+        });
+        return false;
     }
     // Mark any pending prompts as answered. Intended twin of the sweep in
     // extractCompletedTurn (working visible on an unfinished frame); this one runs
@@ -1103,8 +1245,23 @@ async function checkForResponse(worktreeId, cliToolId, instanceId) {
             (0, response_poller_core_1.stopPolling)(worktreeId, cliToolId, instanceId);
             return false;
         }
+        const pollerKey = (0, response_poller_core_1.getPollerKey)(worktreeId, cliToolId, instanceId);
+        // Issue #3213: what the steps split out of this function are keyed by.
+        const ctx = {
+            db,
+            worktree,
+            worktreeId,
+            cliToolId,
+            instanceId,
+            resolvedInstanceId,
+            pollerKey,
+        };
         // Check if CLI tool session is running
         const running = await (0, cli_session_1.isSessionRunning)(worktreeId, cliToolId, instanceId);
+        // Issue #3705: the first of the tick's awaits. Every one of them is followed
+        // by the same question; see `mayStillActAfter`.
+        if (!mayStillActAfter(ctx, 'session-check'))
+            return false;
         if (!running) {
             logger.info('session-not-running');
             // `stopPolling` is what confirms a held scrape here; see
@@ -1116,23 +1273,14 @@ async function checkForResponse(worktreeId, cliToolId, instanceId) {
             (0, response_poller_core_1.stopPolling)(worktreeId, cliToolId, instanceId);
             return false;
         }
-        const pollerKey = (0, response_poller_core_1.getPollerKey)(worktreeId, cliToolId, instanceId);
-        // Issue #3213: what the steps split out of this function are keyed by.
-        const ctx = {
-            db,
-            worktree,
-            worktreeId,
-            cliToolId,
-            instanceId,
-            resolvedInstanceId,
-            pollerKey,
-        };
         // Issue #2436: a scrape held by an earlier tick, whose transcript has now
         // had its whole budget to close. Ahead of everything below because most
         // ticks of a finished turn never reach the save path — a static frame is a
         // duplicate for an alternate-screen tool and yields no new lines for a
         // scrollback one, and both of those return early.
         await (0, pending_scraped_response_1.settleExpiredPendingScrapedResponse)(pollerKey);
+        if (!mayStillActAfter(ctx, 'pending-scrape-settle'))
+            return false;
         // Issue #2317 Phase D: is a human reading this pane at their own terminal
         // size right now, and did they just stop?
         //
@@ -1144,6 +1292,8 @@ async function checkForResponse(worktreeId, cliToolId, instanceId) {
         const delegation = (0, tmux_session_surface_1.isLiveAttachEligibleSession)(sessionName)
             ? await (0, geometry_delegation_1.probeGeometryDelegation)(sessionName)
             : { delegated: false, released: false };
+        if (!mayStillActAfter(ctx, 'geometry-probe'))
+            return false;
         if (delegation.released) {
             // The canvas is 1000 rows again, so a cursor recorded against a 44-row
             // frame indexes into a pane that no longer exists. Zero is the only value
@@ -1170,7 +1320,16 @@ async function checkForResponse(worktreeId, cliToolId, instanceId) {
         // taken from the same constant extractResponse() measures saturation against
         // (Issue #1670) — a literal here would silently decouple the two.
         const output = await (0, cli_session_1.captureSessionOutput)(worktreeId, cliToolId, tmux_capture_cache_1.CACHE_MAX_CAPTURE_LINES, instanceId);
-        const turn = extractCompletedTurn(ctx, output, lastCapturedLine);
+        // Issue #3705: there is no `await` from here to the transcript reader
+        // inside the save path, so this one answer covers every write in between —
+        // the fault notifier, the TUI accumulator, the prompt sweep, the skip
+        // streaks, the cursor, the in-progress id. A retired tick still READS the
+        // frame and goes on, because the writes that already ask for themselves
+        // (the prompt row #3701, the response hash #3663, the reader and the
+        // scrape decision) are where its refusal is recorded; it writes none of
+        // the rest.
+        const owns = mayStillActAfter(ctx, 'capture');
+        const turn = extractCompletedTurn(ctx, output, lastCapturedLine, owns);
         if (!turn) {
             return false;
         }
@@ -1197,29 +1356,38 @@ async function checkForResponse(worktreeId, cliToolId, instanceId) {
             // Issue #3552: a live prompt is not the saved reply, so a run of
             // duplicate response ticks is over; the response hash is kept (a resume
             // after the answer must not re-save the reply underneath, #2230).
-            (0, response_dedup_1.resetDuplicateResponseSkipStreak)(pollerKey);
+            if (owns)
+                (0, response_dedup_1.resetDuplicateResponseSkipStreak)(pollerKey);
+            // Asks for itself (#3701), before the prompt dedup.
             return savePromptMessage(ctx, promptDetection, result, isFullScreenTui);
         }
         // Issue #3538: a finished frame without a live prompt ends the run of
         // duplicate prompt ticks, as an unfinished one does in `extractCompletedTurn`.
-        (0, prompt_dedup_1.resetDuplicatePromptSkipStreak)(pollerKey);
+        if (owns)
+            (0, prompt_dedup_1.resetDuplicatePromptSkipStreak)(pollerKey);
         // Validate response content is not empty
         if (!result.response || result.response.trim() === '') {
             // Issue #3552: an empty response is not the saved reply either.
-            (0, response_dedup_1.resetDuplicateResponseSkipStreak)(pollerKey);
-            (0, db_1.updateSessionState)(db, worktreeId, cliToolId, result.lineCount, resolvedInstanceId);
+            if (owns) {
+                (0, response_dedup_1.resetDuplicateResponseSkipStreak)(pollerKey);
+                (0, db_1.updateSessionState)(db, worktreeId, cliToolId, result.lineCount, resolvedInstanceId);
+            }
             return false;
         }
         // Parse Claude-specific metadata
         const claudeMetadata = cliToolId === 'claude'
             ? (0, claude_output_1.parseClaudeOutput)(result.response)
             : undefined;
-        const cleanedResponse = cleanCompletedResponse(cliToolId, result, pollerKey);
+        const cleanedResponse = cleanCompletedResponse(cliToolId, result, pollerKey, owns);
         // If cleaned response is empty or just "[No content]", skip saving
         if (!cleanedResponse || cleanedResponse.trim() === '' || cleanedResponse === '[No content]') {
             // Issue #3552: nor is one that cleans down to nothing.
-            (0, response_dedup_1.resetDuplicateResponseSkipStreak)(pollerKey);
-            (0, db_1.updateSessionState)(db, worktreeId, cliToolId, result.lineCount, resolvedInstanceId);
+            if (owns) {
+                (0, response_dedup_1.resetDuplicateResponseSkipStreak)(pollerKey);
+                (0, db_1.updateSessionState)(db, worktreeId, cliToolId, result.lineCount, resolvedInstanceId);
+            }
+            // Left as it was (Issue #3705 sets it aside as harmless): nothing sets the
+            // in-progress id to a non-null value or reads it today.
             (0, db_1.clearInProgressMessageId)(db, worktreeId, cliToolId, resolvedInstanceId);
             return false;
         }
@@ -1234,6 +1402,10 @@ async function checkForResponse(worktreeId, cliToolId, instanceId) {
         // Without this the disabled cursor would leave nothing suppressing re-saves and
         // the poller would append the same finished reply every 2 s.
         if (!lineCountIsCursor) {
+            // A retired tick registers no hash here (#3663, `./response-dedup`), but
+            // it can still MATCH one — the successor's, for the same reply. The
+            // recheck below therefore opens on the ownership question itself before
+            // its claim, its cursor and its read (Issue #3705 review).
             if ((0, response_dedup_1.isDuplicateResponse)(pollerKey, cleanedResponse)) {
                 // `await`, so that a rejection still lands in the catch below.
                 return await recheckDuplicateResponse(ctx, result, claudeMetadata);

package/dist/server/src/lib/polling/response-dedup.js

@@ -64,6 +64,7 @@ exports.renameResponseHashCacheKey = renameResponseHashCacheKey;
 exports.renameResponseHashCacheKeys = renameResponseHashCacheKeys;
 const crypto_1 = require("crypto");
 const global_state_1 = require("../global-state");
+const poller_owner_1 = require("./poller-owner");
 const responseHashCache = (0, global_state_1.getOrInitGlobal)('__responseHashCache', () => new Map());
 /**
  * In-memory cache: pollerKey -> ticks still to wait before the transcript reader
@@ -127,6 +128,13 @@ function isDuplicateResponse(pollerKey, content) {
     if (responseHashCache.get(pollerKey) === hash) {
         return true;
     }
+    // Issue #3663: a tick of a stopped chain no longer owns its key, so it does
+    // not register a hash there — the same line the held scrape draws. Left
+    // behind, the hash of a reply that tick never saves (its hold is refused)
+    // would be inherited by the next chain on the key, or moved by a migration,
+    // and make the successor skip it.
+    if (!(0, poller_owner_1.mayActForPollerKey)(pollerKey))
+        return false;
     responseHashCache.set(pollerKey, hash);
     // Issue #3519: a new response starts a new run of duplicate ticks.
     duplicateResponseSkipStreak.delete(pollerKey);