@raidiant/notifai 0.3.0 → 0.4.0

package/dist/hooks.js

@@ -1,9 +1,12 @@
 import { randomBytes } from 'node:crypto';
 import { closeSync, existsSync, mkdirSync, openSync, readFileSync, readdirSync, rmSync, statSync, writeFileSync, } from 'node:fs';
 import path from 'node:path';
+import { ApiCallError, isRetryableReplyPollError, } from './client.js';
 import { loadConfig, projectSessionPointerPath, sanitizeSessionId, sessionConfigPath, stateDir, } from './config.js';
 import { buildDraft } from './send.js';
 import { atomicWriteFileSync } from './atomic-file.js';
+import { withFileLock } from './file-lock.js';
+import { HARNESS_CAPABILITIES, HARNESSES, HOOK_TIMING } from './harnesses.js';
 /**
  * Past this, the question's reply window (3600s) has long expired server-side
  * and the companion shows it as dead on next open anyway; pushing a retirement
@@ -13,14 +16,20 @@ import { atomicWriteFileSync } from './atomic-file.js';
 const ORPHAN_TTL_MS = 24 * 3600 * 1000;
 /** More orphans than this means something is looping; keep the newest. */
 const ORPHAN_QUEUE_CAP = 50;
+function summarizeRequestIds(entries) {
+    const ids = entries.flatMap((entry) => entry.request_id === undefined ? [] : [entry.request_id]);
+    return { ids, display: ids.join(', ') };
+}
 /** How often the grace window rechecks whether the user has come back. */
 const GRACE_POLL_MS = 5_000;
 /**
- * Total seconds a Stop hook may spend blocking. Both harnesses kill a command
- * hook at 600s, and a killed hook loses an answer the user has already given,
- * so the grace window yields to the reply wait rather than the other way round.
+ * Total seconds the complete Stop invocation may own. Generated adapters grant
+ * another minute of host headroom, because a killed hook can lose an answer
+ * the user has already given. Grace and reply waiting share this budget while
+ * the reply retains a small actionable window and finalization reserve.
  */
-const STOP_BUDGET_SECONDS = 480;
+const STOP_BUDGET_SECONDS = HOOK_TIMING.totalSeconds;
+const MIN_REPLY_WAIT_SECONDS = HOOK_TIMING.minimumReplySeconds;
 /**
  * The terminal-first wait: the question sits in the terminal for
  * `ask_grace_seconds` from when it was sent, and only then reaches companion
@@ -42,8 +51,8 @@ const STOP_BUDGET_SECONDS = 480;
  */
 export async function awaitGrace(ctx, askedAt) {
     const threshold = ctx.config.away_after_seconds.value;
-    // Never let the grace window eat the reply wait's share of the hook budget.
-    const graceSeconds = Math.min(ctx.config.ask_grace_seconds.value, Math.max(0, STOP_BUDGET_SECONDS - ctx.config.hook_reply_timeout_seconds.value));
+    // Keep a real device-answer window even when a user selects maximum grace.
+    const graceSeconds = Math.min(ctx.config.ask_grace_seconds.value, HOOK_TIMING.maxGraceSeconds);
     // `asked_at` is wall-clock too, and the same jumps apply: a stamp from the
     // future would wait far past the hook's budget, one from the distant past
     // would skip the terminal-first window the user asked for entirely. Anything
@@ -105,63 +114,6 @@ function updateSessionState(sessionId, env, update) {
         return next;
     });
 }
-/**
- * Serialize a short read-modify-write without ever exposing partial JSON.
- * A crashed holder is recoverable after 30 seconds; releases compare their
- * random token under the same cooperating lock discipline before unlinking.
- */
-const FILE_LOCK_STALE_MS = 30_000;
-const FILE_LOCK_WAIT_MS = 1_000;
-const FILE_LOCK_POLL_MS = 5;
-const lockSleep = new Int32Array(new SharedArrayBuffer(4));
-function withFileLock(file, action) {
-    mkdirSync(path.dirname(file), { recursive: true });
-    const token = randomBytes(12).toString('base64url');
-    const deadline = Date.now() + FILE_LOCK_WAIT_MS;
-    for (;;) {
-        try {
-            const handle = openSync(file, 'wx', 0o600);
-            try {
-                writeFileSync(handle, `${JSON.stringify({ token, at: Date.now() })}\n`);
-            }
-            finally {
-                closeSync(handle);
-            }
-            break;
-        }
-        catch (err) {
-            if (err.code !== 'EEXIST')
-                throw err;
-            let stale = false;
-            try {
-                const held = JSON.parse(readFileSync(file, 'utf8'));
-                stale = typeof held.at !== 'number' || Date.now() - held.at >= FILE_LOCK_STALE_MS;
-            }
-            catch {
-                stale = true;
-            }
-            if (stale)
-                rmSync(file, { force: true });
-            else if (Date.now() >= deadline)
-                throw new Error(`timed out waiting for state lock ${file}`);
-            else
-                Atomics.wait(lockSleep, 0, 0, FILE_LOCK_POLL_MS);
-        }
-    }
-    try {
-        return action();
-    }
-    finally {
-        try {
-            const held = JSON.parse(readFileSync(file, 'utf8'));
-            if (held.token === token)
-                rmSync(file, { force: true });
-        }
-        catch {
-            // A replaced or externally removed lock is no longer ours to release.
-        }
-    }
-}
 /**
  * Session state a crashed harness left behind.
  *
@@ -223,8 +175,9 @@ export function pruneAbandonedSessions(env, now = Date.now(), maxAgeMs = SESSION
  * Exclusive create is atomic on POSIX, so exactly one process gets the claim
  * and the other steps aside. A short guard serializes stale replacement with
  * contenders and releases; random ownership tokens keep an old holder from
- * unlinking its replacement. A claim older than the hook budget is a crashed
- * process rather than a live one and is broken, avoiding permanent suppression.
+ * unlinking its replacement. A live PID owns the claim regardless of age; a
+ * known-dead PID is recoverable immediately. Only legacy/corrupt claims with
+ * no trustworthy PID fall back to an age limit.
  */
 const CLAIM_TTL_MS = STOP_BUDGET_SECONDS * 1000;
 const heldClaims = new Map();
@@ -249,8 +202,13 @@ export function claimQuestionPush(sessionId, env, now = Date.now(), beforeStaleR
             try {
                 const held = JSON.parse(readFileSync(file, 'utf8'));
                 const age = typeof held.at === 'number' ? now - held.at : Number.POSITIVE_INFINITY;
-                if (age >= 0 && age < CLAIM_TTL_MS)
+                if (typeof held.pid === 'number') {
+                    if (processIsAlive(held.pid))
+                        return false;
+                }
+                else if (age >= 0 && age < CLAIM_TTL_MS) {
                     return false;
+                }
             }
             catch {
                 // Unreadable or corrupt: treat as abandoned.
@@ -274,6 +232,15 @@ export function claimQuestionPush(sessionId, env, now = Date.now(), beforeStaleR
         rmSync(guard, { force: true });
     }
 }
+function processIsAlive(pid) {
+    try {
+        process.kill(pid, 0);
+        return true;
+    }
+    catch (err) {
+        return err.code === 'EPERM';
+    }
+}
 export function releaseQuestionPush(sessionId, env) {
     const file = claimPath(sessionId, env);
     const token = heldClaims.get(file);
@@ -332,13 +299,53 @@ function claimPath(sessionId, env) {
  * meantime, and keeps one runaway agent from writing megabytes per session.
  */
 const MAX_STORED_QUESTION_CHARS = 2000;
-/** Records which session is working in a directory, for `notifai ask`. */
+/** Records every live session working in a directory, for `notifai ask`. */
 export function writeProjectSession(cwd, env, sessionId, now, harness) {
     const file = projectSessionPointerPath(cwd, env);
     withFileLock(`${file}.lock`, () => {
-        atomicWriteFileSync(file, `${JSON.stringify({ session_id: sessionId, updated_at: now, harness })}\n`);
+        const existing = readStoredProjectSessionPointers(file).filter((entry) => entry.sessionId !== sessionId && now - entry.updatedAt <= 24 * 3600 * 1000);
+        atomicWriteFileSync(file, `${JSON.stringify({
+            sessions: [
+                ...existing.map((entry) => ({
+                    session_id: entry.sessionId,
+                    updated_at: entry.updatedAt,
+                    harness: entry.harness,
+                })),
+                { session_id: sessionId, updated_at: now, harness },
+            ],
+        })}\n`);
     });
 }
+function readStoredProjectSessionPointers(file) {
+    if (!existsSync(file))
+        return [];
+    try {
+        const parsed = JSON.parse(readFileSync(file, 'utf8'));
+        if (!Array.isArray(parsed.sessions))
+            return [];
+        return parsed.sessions.flatMap((candidate) => {
+            if (typeof candidate !== 'object' || candidate === null || Array.isArray(candidate))
+                return [];
+            const entry = candidate;
+            if (typeof entry.session_id !== 'string' || entry.session_id === '')
+                return [];
+            if (typeof entry.updated_at !== 'number')
+                return [];
+            if (!HARNESSES.includes(entry.harness))
+                return [];
+            return [
+                {
+                    sessionId: entry.session_id,
+                    updatedAt: entry.updated_at,
+                    harness: entry.harness,
+                },
+            ];
+        });
+    }
+    catch {
+        return [];
+    }
+}
 /**
  * Resolves the session working in `cwd`. Stale pointers are ignored rather than
  * trusted: an id left by a session that ended days ago would send the question
@@ -349,43 +356,65 @@ export function readProjectSession(cwd, env, now, maxAgeMs = 24 * 3600 * 1000) {
 }
 export function readProjectSessionPointer(cwd, env, now, maxAgeMs = 24 * 3600 * 1000) {
     const file = projectSessionPointerPath(cwd, env);
-    if (!existsSync(file))
+    const live = readStoredProjectSessionPointers(file)
+        .filter((entry) => now - entry.updatedAt <= maxAgeMs)
+        .filter((entry) => {
+        try {
+            // A pointer is routing evidence only while its session state still
+            // exists and parses. SessionEnd and explicit cleanup therefore
+            // invalidate it even when a crash prevented removal from this index.
+            const sessionFile = sessionStatePath(entry.sessionId, env);
+            if (!existsSync(sessionFile))
+                return false;
+            const state = JSON.parse(readFileSync(sessionFile, 'utf8'));
+            return typeof state === 'object' && state !== null && !Array.isArray(state);
+        }
+        catch {
+            return false;
+        }
+    })
+        .sort((left, right) => right.updatedAt - left.updatedAt);
+    const latest = live[0];
+    return latest === undefined ? null : { sessionId: latest.sessionId, harness: latest.harness };
+}
+export function readMatchingProjectSessionPointer(cwd, env, now, sessionId, harness, maxAgeMs = 24 * 3600 * 1000) {
+    const file = projectSessionPointerPath(cwd, env);
+    const match = readStoredProjectSessionPointers(file).find((entry) => entry.sessionId === sessionId &&
+        entry.harness === harness &&
+        now - entry.updatedAt <= maxAgeMs);
+    if (match === undefined)
         return null;
     try {
-        const parsed = JSON.parse(readFileSync(file, 'utf8'));
-        if (typeof parsed.session_id !== 'string' || parsed.session_id === '')
-            return null;
-        if (typeof parsed.updated_at !== 'number' || now - parsed.updated_at > maxAgeMs)
-            return null;
-        if (!HOOK_HARNESSES.includes(parsed.harness))
-            return null;
         // A pointer is routing evidence only while its session state still exists
         // and parses. SessionEnd and explicit cleanup therefore invalidate it even
         // when a crash prevented the pointer file itself from being removed.
-        const sessionFile = sessionStatePath(parsed.session_id, env);
+        const sessionFile = sessionStatePath(match.sessionId, env);
         if (!existsSync(sessionFile))
             return null;
         const state = JSON.parse(readFileSync(sessionFile, 'utf8'));
         if (typeof state !== 'object' || state === null || Array.isArray(state))
             return null;
-        return { sessionId: parsed.session_id, harness: parsed.harness };
+        return { sessionId: match.sessionId, harness: match.harness };
     }
     catch {
         return null;
     }
 }
-const HOOK_HARNESSES = ['claude-code', 'codex', 'cursor', 'opencode'];
 function clearMatchingProjectSession(cwd, env, sessionId) {
     const file = projectSessionPointerPath(cwd, env);
     withFileLock(`${file}.lock`, () => {
-        try {
-            const parsed = JSON.parse(readFileSync(file, 'utf8'));
-            if (parsed.session_id === sessionId)
-                rmSync(file, { force: true });
-        }
-        catch {
-            // Missing/corrupt is already not a live pointer.
+        const remaining = readStoredProjectSessionPointers(file).filter((entry) => entry.sessionId !== sessionId);
+        if (remaining.length === 0) {
+            rmSync(file, { force: true });
+            return;
         }
+        atomicWriteFileSync(file, `${JSON.stringify({
+            sessions: remaining.map((entry) => ({
+                session_id: entry.sessionId,
+                updated_at: entry.updatedAt,
+                harness: entry.harness,
+            })),
+        })}\n`);
     });
 }
 /**
@@ -438,6 +467,9 @@ export function isUserAway(state, config, now, idleSeconds) {
  * than about the user. Matches the project pointer's staleness horizon.
  */
 const MAX_PLAUSIBLE_SILENCE_MS = 24 * 3600 * 1000;
+function gate(ctx, verdict, reason, data = {}) {
+    ctx.log?.info('hook.gate', { verdict, reason, ...data });
+}
 /** Re-check local presence at this cadence while a pushed question is waiting. */
 const REPLY_PRESENCE_POLL_SECONDS = 5;
 /**
@@ -455,7 +487,7 @@ const REPLY_PRESENCE_POLL_SECONDS = 5;
  * Put one registered question on the user's devices. Submission only — the
  * wait happens once, across every live question, in the caller.
  */
-async function submitQuestion(ctx, options) {
+async function prepareQuestionSubmission(ctx, options) {
     const collapseKey = `notifai-hook-${randomBytes(8).toString('base64url')}`;
     // A draft carrying `reply` is rejected outright if it targets a device that
     // cannot answer, so resolve the healthy companion platforms explicitly.
@@ -478,8 +510,25 @@ async function submitQuestion(ctx, options) {
     });
     if (!build.ok)
         return { error: build.error };
-    const receipt = await ctx.client.submit({ idempotency_key: `hook-${randomBytes(12).toString('base64url')}`, draft: build.draft }, 0);
-    return { requestId: receipt.request_id, collapseKey, devices: answerable };
+    return {
+        request_id: `req_${randomBytes(18).toString('base64url')}`,
+        idempotency_key: `hook-${randomBytes(12).toString('base64url')}`,
+        collapse_key: collapseKey,
+        device_ids: answerable,
+        draft: build.draft,
+        owner_deadline_at: options.ownerDeadlineAt,
+    };
+}
+async function submitQuestion(ctx, intent) {
+    const receipt = await ctx.client.submit({
+        request_id: intent.request_id,
+        idempotency_key: intent.idempotency_key,
+        draft: intent.draft,
+    }, 0);
+    if (receipt.request_id !== intent.request_id) {
+        throw new Error(`server replay returned ${receipt.request_id}, expected reserved ${intent.request_id}`);
+    }
+    return receipt;
 }
 /**
  * One blocking wait across every live question. Each round polls all of them
@@ -487,41 +536,68 @@ async function submitQuestion(ctx, options) {
  * first round that finds any reply returns everything found in that round, and
  * whatever was not answered stays registered.
  */
-async function waitForAnyReplyWhileAway(ctx, requestIds, timeoutSeconds) {
+async function waitForAnyReplyWhileAway(ctx, requestIds, timeoutSeconds, monitorPresence = true) {
     const deadline = ctx.now() + timeoutSeconds * 1000;
     let degraded = false;
     let firstPoll = true;
+    const permanentFailures = new Map();
     for (;;) {
-        if (ctx.config.require_idle.value) {
+        if (monitorPresence && ctx.config.require_idle.value) {
             const idle = ctx.idleSeconds();
             if (idle !== null && idle < ctx.config.away_after_seconds.value) {
-                return { byRequest: new Map(), degraded, userReturned: true };
+                return { byRequest: new Map(), degraded, userReturned: true, permanentFailures };
             }
         }
         const remainingMs = Math.max(0, deadline - ctx.now());
         if (!firstPoll && remainingMs === 0) {
-            return { byRequest: new Map(), degraded, userReturned: false };
+            return { byRequest: new Map(), degraded, userReturned: false, permanentFailures };
         }
         firstPoll = false;
         const pollSeconds = Math.min(REPLY_PRESENCE_POLL_SECONDS, Math.max(0, Math.ceil(remainingMs / 1000)));
-        const results = await Promise.all(requestIds.map(async (requestId) => {
+        const results = await Promise.all(requestIds
+            .filter((requestId) => !permanentFailures.has(requestId))
+            .map(async (requestId) => {
             try {
                 return { requestId, ...(await ctx.waitForFirstReply(requestId, pollSeconds)) };
             }
-            catch {
-                return { requestId, replies: [], degraded: true };
+            catch (err) {
+                if (isRetryableReplyPollError(err)) {
+                    return { requestId, replies: [], degraded: true };
+                }
+                return {
+                    requestId,
+                    replies: [],
+                    degraded: false,
+                    permanentFailure: replyPollFailure(err),
+                };
             }
         }));
         const byRequest = new Map();
         for (const result of results) {
             degraded ||= result.degraded === true;
+            if (result.permanentFailure !== undefined) {
+                permanentFailures.set(result.requestId, result.permanentFailure);
+            }
             if (result.replies.length > 0)
                 byRequest.set(result.requestId, result.replies);
         }
-        if (byRequest.size > 0)
-            return { byRequest, degraded, userReturned: false };
+        if (byRequest.size > 0 || permanentFailures.size === requestIds.length) {
+            return { byRequest, degraded, userReturned: false, permanentFailures };
+        }
     }
 }
+/** A non-retryable replies fault, with enough server identity to act on it. */
+function replyPollFailure(err) {
+    return err instanceof ApiCallError
+        ? `${err.code} (HTTP ${err.status}): ${err.message}`
+        : String(err);
+}
+function permanentReplyFailureNote(failures) {
+    if (failures.size === 0)
+        return null;
+    const details = [...failures].map(([requestId, failure]) => `${requestId}: ${failure}`).join('; ');
+    return `reply polling stopped after a permanent server rejection (${details}); the affected question will be retired before its continuation owner exits`;
+}
 /**
  * Healthy companion devices that implement replies. Both the iOS app and the
  * macOS app register reply categories and submit answers directly.
@@ -552,15 +628,24 @@ function sessionLabel(ctx, envelope) {
         return explicit;
     return envelope.session_id;
 }
-/** Best effort: a question that outlives its hook is a nuisance, not a failure. */
-async function closeQuietly(ctx, requestId) {
+/**
+ * Close is a server-side serialization fence: a successful response contains
+ * every reply that committed before the window closed. `null` is deliberately
+ * different from an empty reply set — it means ownership could not be proven
+ * final and the local record must stay recoverable.
+ */
+async function finalizeReplies(ctx, requestId) {
     try {
-        await ctx.client.closeReplies(requestId);
+        return await ctx.client.closeReplies(requestId);
     }
     catch {
-        // The window expires on its own; nothing here is worth failing a hook for.
+        return null;
     }
 }
+/** Best effort only for already-journaled retirement debt. */
+async function closeQuietly(ctx, requestId) {
+    await finalizeReplies(ctx, requestId);
+}
 /**
  * Retire the question on every device it reached. A state change is not news
  * (D-B): the retirement rides as a background state sync — no alert, no
@@ -759,7 +844,7 @@ const MAX_CONTINUATION_COUNT = 3;
 /** One bounded recovery poll for a question that outlived its original wait. */
 async function pollPendingReply(ctx, pending, timeoutSeconds) {
     if (pending.request_id === undefined) {
-        return { reply: null, replies: [], degraded: false, failed: false };
+        return { reply: null, replies: [], degraded: false, permanentFailure: null };
     }
     try {
         const result = await ctx.waitForFirstReply(pending.request_id, timeoutSeconds);
@@ -767,13 +852,39 @@ async function pollPendingReply(ctx, pending, timeoutSeconds) {
             reply: result.replies.at(-1) ?? null,
             replies: result.replies,
             degraded: result.degraded === true,
-            failed: false,
+            permanentFailure: null,
         };
     }
-    catch {
-        return { reply: null, replies: [], degraded: true, failed: true };
+    catch (err) {
+        return isRetryableReplyPollError(err)
+            ? { reply: null, replies: [], degraded: true, permanentFailure: null }
+            : { reply: null, replies: [], degraded: false, permanentFailure: replyPollFailure(err) };
     }
 }
+/** Close several windows concurrently without confusing failure with silence. */
+async function finalizePendings(ctx, pending) {
+    return Promise.all(pending.map(async (entry) => ({
+        pending: entry,
+        response: await finalizeReplies(ctx, entry.request_id),
+    })));
+}
+/** Convert a fenced final reply set into the exact answer handed to the model. */
+function finalizedAnswer(finalized) {
+    const replies = finalized.response?.replies ?? [];
+    const reply = replies.at(-1);
+    return reply === undefined ? null : { pending: finalized.pending, reply, replies };
+}
+/** Persist and surface one answer without letting the two representations drift. */
+function reportAnswer(ctx, notes, answered, late) {
+    ctx.log?.info('hook.answer', {
+        answered: true,
+        ...(late ? { late: true } : {}),
+        request_id: answered.pending.request_id,
+        device: answered.reply.device_name,
+        text: answered.reply.text,
+    });
+    notes.push(`${late ? 'late ' : ''}answer from ${answered.reply.device_name}: ${answered.reply.text}`);
+}
 /**
  * One bounded recovery poll across every delivered question, concurrently —
  * N questions must not multiply the hook's latency budget by N.
@@ -781,15 +892,19 @@ async function pollPendingReply(ctx, pending, timeoutSeconds) {
 async function pollPendingReplies(ctx, live, timeoutSeconds) {
     const polls = await Promise.all(live.map((entry) => pollPendingReply(ctx, entry, timeoutSeconds)));
     const answered = [];
-    let troubled = false;
+    let transientTrouble = false;
+    const permanentFailures = new Map();
     for (const [index, poll] of polls.entries()) {
         if (poll.reply !== null) {
             answered.push({ pending: live[index], reply: poll.reply, replies: poll.replies });
         }
-        if (poll.failed || poll.degraded)
-            troubled = true;
+        if (poll.degraded)
+            transientTrouble = true;
+        if (poll.permanentFailure !== null) {
+            permanentFailures.set(live[index].request_id, poll.permanentFailure);
+        }
     }
-    return { answered, troubled };
+    return { answered, transientTrouble, permanentFailures };
 }
 /** The registered-question queue. Anything but the current shape reads as empty. */
 function pendingList(state) {
@@ -797,6 +912,9 @@ function pendingList(state) {
 }
 /** Registration identity — the convention every racing writer compares by. */
 function isSamePending(a, b) {
+    if (a.question_id !== undefined || b.question_id !== undefined) {
+        return a.question_id !== undefined && a.question_id === b.question_id;
+    }
     return a.question === b.question && a.asked_at === b.asked_at;
 }
 /** The question set this pending record pushes, however it was registered. */
@@ -849,25 +967,39 @@ function answersContext(answered, remaining) {
     });
     return `Notifai — the user answered ${answered.length} questions:\n${lines.join('\n')}\nContinue with these answers.${tail}`;
 }
-function userPromptAnswerOutput(context) {
+function stopAnswerOutput(context) {
     return JSON.stringify({
-        hookSpecificOutput: {
-            hookEventName: 'UserPromptSubmit',
-            additionalContext: context,
-        },
+        decision: 'block',
+        reason: context,
     });
 }
-function stopAnswerOutput(context) {
-    return JSON.stringify({ decision: 'block', reason: context });
-}
 /**
  * Close, truthfully retire, and forget every answered pending question in one
  * state write, without dropping any other retirement debt or any question
  * still waiting. Stop answers also open one bounded continuation generation;
  * UserPromptSubmit answers ride the user's new turn.
  */
-async function finishAnsweredPendings(ctx, envelope, sessionId, answered, continuation) {
-    const retirements = answered
+function stageAcceptedAnswers(ctx, sessionId, answered, remaining) {
+    updateSessionState(sessionId, ctx.env, (current) => {
+        const pendingRemaining = pendingList(current).filter((entry) => !answered.some(({ pending }) => isSamePending(entry, pending)));
+        const next = {
+            ...current,
+            accepted: {
+                answers: answered,
+                remaining,
+                recorded_at: ctx.now(),
+            },
+        };
+        if (pendingRemaining.length > 0)
+            next.pending = pendingRemaining;
+        else
+            delete next.pending;
+        return next;
+    });
+}
+/** Settle only after a successor hook proves the harness received the answer. */
+function settleAcceptedAnswers(ctx, sessionId, accepted) {
+    const retirements = accepted.answers
         .map(({ pending }) => retiringQuestion(pending, 'answered'))
         .filter((entry) => entry !== null);
     updateSessionState(sessionId, ctx.env, (current) => {
@@ -879,24 +1011,39 @@ async function finishAnsweredPendings(ctx, envelope, sessionId, answered, contin
             else
                 retiring[existing] = retirement;
         }
-        const remaining = pendingList(current).filter((entry) => !answered.some(({ pending }) => isSamePending(entry, pending)));
         const next = { ...current, retiring };
-        if (remaining.length > 0)
-            next.pending = remaining;
+        delete next.accepted;
+        next.continuation = {
+            answered_at: accepted.recorded_at,
+            count: accepted.continuation_count ?? 1,
+        };
+        return next;
+    });
+}
+async function retirePendings(ctx, envelope, sessionId, entries, state) {
+    updateSessionState(sessionId, ctx.env, (current) => {
+        const retiring = [...(current.retiring ?? [])];
+        for (const entry of entries) {
+            const retirement = retiringQuestion(entry, state);
+            if (retirement !== null && !retiring.some((item) => item.request_id === retirement.request_id)) {
+                retiring.push(retirement);
+            }
+        }
+        const pending = pendingList(current).filter((candidate) => !entries.some((entry) => isSamePending(candidate, entry)));
+        const next = { ...current };
+        if (pending.length > 0)
+            next.pending = pending;
         else
             delete next.pending;
-        if (continuation) {
-            next.continuation = {
-                answered_at: ctx.now(),
-                count: (envelope.stop_hook_active === true ? current.continuation?.count ?? 0 : 0) + 1,
-            };
-        }
-        else {
-            next.last_prompt_at = ctx.now();
-        }
+        if (retiring.length > 0)
+            next.retiring = retiring;
+        else
+            delete next.retiring;
         return next;
     });
-    await drainRetirements(ctx, sessionId, ctx.env, sessionLabel(ctx, envelope));
+    // Network retirement is deliberately deferred to a later no-question hook.
+    // The state write above is the durable operation; blocking here creates a
+    // crash window before an answer can reach harness stdout.
 }
 // ---------------------------------------------------------------------------
 // UserPromptSubmit — the user is at the keyboard
@@ -913,34 +1060,63 @@ export async function handleUserPromptSubmit(ctx, envelope) {
     if (!sessionId)
         return { notes };
     const state = readSessionState(sessionId, ctx.env);
+    if (state.accepted !== undefined) {
+        updateSessionState(sessionId, ctx.env, (current) => ({
+            ...current,
+            last_prompt_at: ctx.now(),
+        }));
+        if (envelope.cwd !== undefined && ctx.harness !== undefined) {
+            writeProjectSession(envelope.cwd, ctx.env, sessionId, ctx.now(), ctx.harness);
+        }
+        notes.push('a device answer is safely journaled; the next Stop will deliver it');
+        return { notes };
+    }
     const live = pendingList(state).filter((entry) => entry.request_id !== undefined);
     let lateAnswers = [];
     if (live.length > 0) {
-        const { answered, troubled } = await pollPendingReplies(ctx, live, LATE_PROMPT_POLL_SECONDS);
-        lateAnswers = answered;
-        if (answered.length > 0) {
-            await finishAnsweredPendings(ctx, envelope, sessionId, answered, false);
-            for (const { reply } of answered) {
-                notes.push(`late answer from ${reply.device_name}: ${reply.text}`);
+        const { answered, transientTrouble, permanentFailures } = await pollPendingReplies(ctx, live, LATE_PROMPT_POLL_SECONDS);
+        const finalized = await finalizePendings(ctx, answered.map((entry) => entry.pending));
+        lateAnswers = answered.map((entry) => {
+            const response = finalized.find((candidate) => isSamePending(candidate.pending, entry.pending))?.response;
+            const replies = response?.replies.length ? response.replies : entry.replies;
+            if (response === null) {
+                notes.push(`could not confirm the close fence for ${entry.pending.request_id}; preserving the durable answer already observed`);
             }
+            return { ...entry, replies, reply: replies.at(-1) };
+        });
+        if (answered.length > 0) {
+            stageAcceptedAnswers(ctx, sessionId, lateAnswers, 0);
+            for (const answer of lateAnswers)
+                reportAnswer(ctx, notes, answer, true);
         }
-        if (troubled) {
+        if (transientTrouble) {
             notes.push('could not check every pending question for a late answer before the prompt');
         }
+        const permanentFailure = permanentReplyFailureNote(permanentFailures);
+        if (permanentFailure !== null)
+            notes.push(permanentFailure);
     }
     // Park before dropping `pending`. If the process dies between these writes,
     // the next hook sees both copies and dedupes them; the old order could die in
     // the gap after erasing the only request/collapse/device identifiers.
     updateSessionState(sessionId, ctx.env, (current) => {
         const retiring = [...(current.retiring ?? [])];
-        for (const entry of pendingList(current)) {
+        const unasked = pendingList(current).filter((entry) => entry.request_id === undefined);
+        for (const entry of pendingList(current).filter((entry) => entry.request_id !== undefined)) {
             const retirement = retiringQuestion(entry, 'answered_elsewhere');
             if (retirement !== null &&
                 !retiring.some((parked) => parked.request_id === retirement.request_id)) {
                 retiring.push(retirement);
             }
         }
-        return { last_prompt_at: ctx.now(), ...(retiring.length > 0 ? { retiring } : {}) };
+        return {
+            last_prompt_at: ctx.now(),
+            ...(current.last_stop_at === undefined ? {} : { last_stop_at: current.last_stop_at }),
+            ...(unasked.length > 0 ? { pending: unasked } : {}),
+            ...(retiring.length > 0 ? { retiring } : {}),
+            ...(current.accepted === undefined ? {} : { accepted: current.accepted }),
+            ...(current.continuation === undefined ? {} : { continuation: current.continuation }),
+        };
     });
     // The bridge that lets a plain `notifai ask` find the hook's canonical
     // session: an agent shell command gets no hook payload, and not every
@@ -948,78 +1124,162 @@ export async function handleUserPromptSubmit(ctx, envelope) {
     if (envelope.cwd !== undefined && ctx.harness !== undefined) {
         writeProjectSession(envelope.cwd, ctx.env, sessionId, ctx.now(), ctx.harness);
     }
+    if (lateAnswers.length > 0) {
+        // UserPromptSubmit has no later hook field proving its stdout reached the
+        // model. Keep the journal and let the next Stop use its acknowledged
+        // continuation channel instead of recreating the crash-before-stdout gap.
+        notes.push('the late device answer will continue the agent at this turn’s Stop');
+        return { notes };
+    }
     const retired = await drainRetirements(ctx, sessionId, ctx.env, sessionLabel(ctx, envelope));
     const orphaned = await drainOrphanRetirements(ctx, ctx.env, ctx.now());
     const swept = [...retired, ...orphaned];
     if (swept.length > 0) {
         notes.push(`retired question${swept.length > 1 ? 's' : ''} ${swept.join(', ')}`);
     }
-    if (lateAnswers.length > 0) {
-        // Whatever the user did not answer was just retired above — they are at
-        // the keyboard now, so nothing is "still waiting" from their side.
-        return { stdout: userPromptAnswerOutput(answersContext(lateAnswers, 0)), notes };
-    }
     return { notes };
 }
 // ---------------------------------------------------------------------------
 // Stop — the turn ended; escalate a registered question
 // ---------------------------------------------------------------------------
-/**
- * How long an escalated question keeps accepting an answer after the turn
- * ended. Long enough to survive a walk away from the desk, short enough that a
- * forgotten question does not resurface days later as a live prompt.
- */
-const QUESTION_WINDOW_SECONDS = 3600;
 /**
  * Only engages for a question the agent explicitly registered with
  * `notifai ask`. Guessing from the last assistant message was the alternative
  * and it is not worth it: a false positive here hijacks the terminal.
  */
-export async function handleStop(ctx, envelope) {
+export async function handleStop(ctx, envelope, processDeadlineAt = ctx.now() + STOP_BUDGET_SECONDS * 1000) {
     const notes = [];
+    const hardDeadlineAt = processDeadlineAt;
     const sessionId = envelope.session_id;
-    if (!sessionId)
+    if (!sessionId) {
+        gate(ctx, 'held', 'no-session');
         return { notes };
-    // Anything retired since the last hook is still live on the devices. This
-    // runs before every early return below, including the nagging guard: a queued
-    // retirement has nothing to do with whether *this* turn has a question to
-    // escalate, and the turn that supersedes a question is very often the one
-    // continuing from the previous answer.
-    const swept = [
-        ...(await drainRetirements(ctx, sessionId, ctx.env, sessionLabel(ctx, envelope))),
-        ...(await drainOrphanRetirements(ctx, ctx.env, ctx.now())),
-    ];
-    if (swept.length > 0) {
-        notes.push(`retired superseded question${swept.length > 1 ? 's' : ''} ${swept.join(', ')}`);
     }
-    const state = readSessionState(sessionId, ctx.env);
-    const pending = pendingList(state);
-    if (pending.length === 0)
+    // This event marker is diagnostic evidence in its own right. Record it
+    // before every early return so doctor can distinguish a working Stop route
+    // from the prompt-only state that previously looked healthy.
+    updateSessionState(sessionId, ctx.env, (current) => ({
+        ...current,
+        last_stop_at: ctx.now(),
+    }));
+    // Claim before any reply poll, not only before the grace window. Two racing
+    // Stops can both observe the same accepted journal or live request. An
+    // atomic state update cannot retract duplicate block output a loser already
+    // built, so one session claim owns the complete Stop decision: accepted
+    // replay/ack, late-answer collection, cleanup, and any new escalation.
+    //
+    // Real clock, deliberately, not `ctx.now` — the claim answers "is another
+    // process alive right now", which the injectable clock cannot speak to. It
+    // is also the only clock the *other* process shares.
+    if (!claimQuestionPush(sessionId, ctx.env)) {
+        gate(ctx, 'held', 'claimed-elsewhere');
+        notes.push('another hook is already handling this question');
         return { notes };
+    }
+    try {
+        let state = readSessionState(sessionId, ctx.env);
+        if (state.accepted !== undefined) {
+            const accepted = state.accepted;
+            // A recursive Stop is emitted only after the previous Stop's blocking
+            // output was admitted and the continued model turn ran. That event is
+            // the acknowledgment: persisting a separate "offered" bit before stdout
+            // would recreate the crash gap this journal exists to close.
+            const deliveryProven = envelope.stop_hook_active === true;
+            if (!deliveryProven) {
+                updateSessionState(sessionId, ctx.env, (current) => {
+                    if (current.accepted === undefined)
+                        return current;
+                    return {
+                        ...current,
+                        accepted: {
+                            ...current.accepted,
+                            continuation_count: (current.continuation?.count ?? 0) + 1,
+                        },
+                    };
+                });
+                for (const answer of accepted.answers)
+                    reportAnswer(ctx, notes, answer, true);
+                return {
+                    stdout: stopAnswerOutput(answersContext(accepted.answers, accepted.remaining)),
+                    notes,
+                };
+            }
+            settleAcceptedAnswers(ctx, sessionId, accepted);
+            state = readSessionState(sessionId, ctx.env);
+        }
+        const pending = pendingList(state);
+        if (pending.length === 0) {
+            const swept = [
+                ...(await drainRetirements(ctx, sessionId, ctx.env, sessionLabel(ctx, envelope))),
+                ...(await drainOrphanRetirements(ctx, ctx.env, ctx.now())),
+            ];
+            if (swept.length > 0) {
+                notes.push(`retired question${swept.length > 1 ? 's' : ''} ${swept.join(', ')}`);
+            }
+            gate(ctx, 'held', 'no-question');
+            return { notes };
+        }
+        return await handleClaimedStop(ctx, envelope, sessionId, state, pending, notes, hardDeadlineAt);
+    }
+    finally {
+        releaseQuestionPush(sessionId, ctx.env);
+    }
+}
+/** The complete per-session Stop decision, after this process owns the claim. */
+async function handleClaimedStop(ctx, envelope, sessionId, state, pending, notes, hardDeadlineAt) {
     const live = pending.filter((entry) => entry.request_id !== undefined);
     const unasked = pending.filter((entry) => entry.request_id === undefined);
+    let liveToEscalate = live;
+    if (ctx.harness !== undefined &&
+        HARNESS_CAPABILITIES[ctx.harness].stopContinuation === 'unsupported') {
+        for (const entry of live)
+            await closeQuietly(ctx, entry.request_id);
+        await retirePendings(ctx, envelope, sessionId, pending, 'expired');
+        gate(ctx, 'held', 'no-question');
+        notes.push(`${HARNESS_CAPABILITIES[ctx.harness].deliveryContract}; use a blocking \`notifai send --reply\` question`);
+        return { notes };
+    }
     if (live.length > 0) {
-        const { answered, troubled } = await pollPendingReplies(ctx, live, LATE_STOP_POLL_SECONDS);
+        const { answered, permanentFailures } = await pollPendingReplies(ctx, live, LATE_STOP_POLL_SECONDS);
+        const permanentFailure = permanentReplyFailureNote(permanentFailures);
+        if (permanentFailure !== null)
+            notes.push(permanentFailure);
+        const permanentlyRejected = live.filter((entry) => permanentFailures.has(entry.request_id));
+        await Promise.all(permanentlyRejected.map((entry) => closeQuietly(ctx, entry.request_id)));
+        if (permanentlyRejected.length > 0) {
+            await retirePendings(ctx, envelope, sessionId, permanentlyRejected, 'expired');
+        }
         if (answered.length > 0) {
-            await finishAnsweredPendings(ctx, envelope, sessionId, answered, true);
-            for (const { reply } of answered) {
-                notes.push(`late answer from ${reply.device_name}: ${reply.text}`);
-            }
+            const finalized = await finalizePendings(ctx, answered.map((entry) => entry.pending));
+            const authoritative = answered.map((entry) => {
+                const response = finalized.find((candidate) => isSamePending(candidate.pending, entry.pending))?.response;
+                const replies = response?.replies.length ? response.replies : entry.replies;
+                if (response === null) {
+                    notes.push(`could not confirm the close fence for ${entry.pending.request_id}; delivering the durable answer already observed`);
+                }
+                return { ...entry, replies, reply: replies.at(-1) };
+            });
+            stageAcceptedAnswers(ctx, sessionId, authoritative, pending.length - authoritative.length - permanentlyRejected.length);
+            gate(ctx, 'proceeding', 'answered', { answers: authoritative.length });
+            for (const answer of authoritative)
+                reportAnswer(ctx, notes, answer, true);
             // Anything still unasked rides the next Stop: the agent is being resumed
             // with answers right now, and may not even need the rest afterwards.
             return {
-                stdout: stopAnswerOutput(answersContext(answered, pending.length - answered.length)),
+                stdout: stopAnswerOutput(answersContext(authoritative, pending.length - authoritative.length - permanentlyRejected.length)),
                 notes,
             };
         }
+        const recoverableLive = live.filter((entry) => !permanentFailures.has(entry.request_id));
+        liveToEscalate = recoverableLive;
         if (unasked.length === 0) {
-            // Already live on the user's devices from an earlier Stop; asking twice
-            // for one question is the nagging failure this feature exists to avoid.
-            const ids = live.map((entry) => entry.request_id).join(', ');
-            notes.push(troubled
-                ? `already asked (${ids}); could not check whether ${live.length === 1 ? 'its answer' : 'their answers'} arrived`
-                : `already asked (${ids}); waiting for ${live.length === 1 ? 'that answer' : 'those answers'}`);
-            return { notes };
+            if (recoverableLive.length === 0)
+                return { notes };
+            // A previous answer may have resumed the model while independent
+            // questions remain. The next Stop is their successor owner: keep waiting
+            // to the original absolute deadline instead of polling for four seconds
+            // and abandoning them.
+            return await escalate(ctx, envelope, sessionId, [], recoverableLive, notes, hardDeadlineAt);
         }
         // Questions registered after the earlier push still owe the user their
         // notification; fall through and escalate just those.
@@ -1032,147 +1292,339 @@ export async function handleStop(ctx, envelope) {
         const isNew = continuation !== undefined &&
             unasked.some((entry) => entry.asked_at !== undefined && entry.asked_at > continuation.answered_at);
         if (!isNew) {
+            gate(ctx, 'held', 'continuation-repeat');
             notes.push('already continuing from an answer; not asking again this turn');
             return { notes };
         }
         if (continuation.count >= MAX_CONTINUATION_COUNT) {
+            gate(ctx, 'held', 'continuation-limit', {
+                count: continuation.count,
+                limit: MAX_CONTINUATION_COUNT,
+            });
             notes.push(`answer continuation limit (${MAX_CONTINUATION_COUNT}) reached; leaving the question in the terminal`);
             return { notes };
         }
     }
-    if (!ctx.config.ask_notifications.value)
-        return { notes };
-    if (!isUserAway(state, ctx.config, ctx.now(), ctx.idleSeconds())) {
-        notes.push('you are at the keyboard; leaving the question in the terminal');
+    // Silent to the user by design — they switched routing off, so saying so on
+    // every turn would be nagging about their own setting. That silence is also
+    // why it belongs in the log: from outside, "ask_notifications = false" and "a
+    // bug ate my question" look exactly the same.
+    if (!ctx.config.ask_notifications.value) {
+        gate(ctx, 'held', 'notifications-off', { source: ctx.config.ask_notifications.source });
         return { notes };
     }
-    // Claim before the grace window, not after: two racing hooks would otherwise
-    // both wait, both find the user still absent, and both push.
-    // Real clock, deliberately, not `ctx.now` — the claim answers "is another
-    // process alive right now", which the injectable clock cannot speak to. It
-    // is also the only clock the *other* process shares.
-    if (!claimQuestionPush(sessionId, ctx.env)) {
-        notes.push('another hook is already handling this question');
+    const idle = ctx.idleSeconds();
+    if (!isUserAway(state, ctx.config, ctx.now(), idle)) {
+        gate(ctx, 'held', 'user-present', {
+            idle_seconds: idle,
+            away_after_seconds: ctx.config.away_after_seconds.value,
+            require_idle: ctx.config.require_idle.value,
+            last_prompt_at: state.last_prompt_at ?? null,
+        });
+        notes.push('you are at the keyboard; leaving the question in the terminal');
         return { notes };
     }
-    try {
-        return await escalate(ctx, envelope, sessionId, unasked, live, notes);
-    }
-    finally {
-        releaseQuestionPush(sessionId, ctx.env);
-    }
+    gate(ctx, 'proceeding', 'proceeding', {
+        unasked: unasked.length,
+        already_live: liveToEscalate.length,
+        idle_seconds: idle,
+        grace_seconds: ctx.config.ask_grace_seconds.value,
+    });
+    return await escalate(ctx, envelope, sessionId, unasked, liveToEscalate, notes, hardDeadlineAt);
 }
 /** The escalation itself, split out so the claim is released on every path. */
-async function escalate(ctx, envelope, sessionId, unasked, alreadyLive, notes) {
+async function escalate(ctx, envelope, sessionId, unasked, alreadyLive, notes, hardDeadlineAt) {
     // Away right now, but the questions still owe the user their terminal-first
     // window before anything reaches their devices — measured from the oldest
     // registration, because that is the question that has waited longest.
-    const oldest = Math.min(...unasked.map((entry) => entry.asked_at ?? ctx.now()));
-    const grace = await awaitGrace(ctx, oldest);
-    if (grace === 'user-returned') {
-        notes.push('you came back before the wait elapsed; leaving the questions in the terminal');
-        return { notes };
-    }
-    if (grace === 'no-signal') {
-        notes.push('no idle signal on this machine; asking now rather than holding the terminal');
+    const budgetStartedAt = ctx.now();
+    const existingDeadlines = alreadyLive.flatMap((entry) => entry.reply_deadline_at === undefined || entry.reply_deadline_at <= budgetStartedAt
+        ? []
+        : [entry.reply_deadline_at]);
+    const budgetDeadlineAt = Math.min(hardDeadlineAt - HOOK_TIMING.finalizationReserveSeconds * 1000, ...(existingDeadlines.length === 0 ? [] : existingDeadlines));
+    if (unasked.length > 0 && alreadyLive.length === 0) {
+        const oldest = Math.min(...unasked.map((entry) => entry.asked_at ?? ctx.now()));
+        const grace = await awaitGrace(ctx, oldest);
+        ctx.log?.debug('hook.gate', { verdict: 'grace', reason: grace, waited_from: oldest });
+        if (grace === 'user-returned') {
+            gate(ctx, 'held', 'user-returned', { grace_seconds: ctx.config.ask_grace_seconds.value });
+            notes.push('you came back before the wait elapsed; leaving the questions in the terminal');
+            return { notes };
+        }
+        if (grace === 'no-signal') {
+            notes.push('no idle signal on this machine; asking now rather than holding the terminal');
+        }
     }
     // Phase one: every registered question reaches the user's devices, each as
     // its own notification — one ask never stands in for another.
     const submitted = [];
     for (const entry of unasked) {
+        const remainingSeconds = Math.floor((budgetDeadlineAt - ctx.now()) / 1000);
+        if (remainingSeconds <
+            MIN_REPLY_WAIT_SECONDS + HOOK_TIMING.submissionReserveSeconds) {
+            notes.push('the Stop budget cannot fit device resolution, submission, and a full reply window; leaving this question in the terminal');
+            continue;
+        }
+        const replyWindowSeconds = Math.min(ctx.config.hook_reply_timeout_seconds.value, remainingSeconds - HOOK_TIMING.submissionReserveSeconds);
+        const ownerDeadlineAt = Math.min(budgetDeadlineAt, ctx.now() +
+            (replyWindowSeconds + HOOK_TIMING.submissionReserveSeconds) * 1000);
         const questions = pendingQuestions(entry);
-        const sent = await submitQuestion(ctx, {
-            title: ctx.config.project.value === null
-                ? 'Question'
-                : `Question · ${ctx.config.project.value}`,
-            // A set is answered on the expanded card; the banner leads with the
-            // first question and says how much more is waiting behind it.
-            body: questions.length > 1
-                ? `${questions[0].text} (+${questions.length - 1} more)`
-                : entry.question,
-            questions,
-            ...(entry.detail !== undefined ? { detail: entry.detail } : {}),
-            event: 'agent_question',
-            session: sessionLabel(ctx, envelope),
-            // Outlives the block, and stays open on purpose: the answer is still
-            // useful to the next turn, which collects it with `notifai replies`.
-            windowSeconds: QUESTION_WINDOW_SECONDS,
-        });
-        if ('error' in sent) {
-            notes.push(sent.error);
+        let intent = entry.submission;
+        if (intent === undefined) {
+            const prepared = await prepareQuestionSubmission(ctx, {
+                title: ctx.config.project.value === null
+                    ? 'Question'
+                    : `Question · ${ctx.config.project.value}`,
+                // A set is answered on the expanded card; the banner leads with the
+                // first question and says how much more is waiting behind it.
+                body: questions.length > 1
+                    ? `${questions[0].text} (+${questions.length - 1} more)`
+                    : entry.question,
+                questions,
+                ...(entry.detail !== undefined ? { detail: entry.detail } : {}),
+                event: 'agent_question',
+                session: sessionLabel(ctx, envelope),
+                windowSeconds: replyWindowSeconds,
+                ownerDeadlineAt,
+            });
+            if ('error' in prepared) {
+                ctx.log?.error('hook.pushed', { ok: false, message: prepared.error });
+                notes.push(prepared.error);
+                continue;
+            }
+            intent = prepared;
+            // Durable before submit. If the server commits and the response is lost,
+            // the reserved request id still lets this owner poll and finalize the
+            // exact card; a successor can also replay the frozen draft/key.
+            updateSessionState(sessionId, ctx.env, (current) => {
+                const list = pendingList(current);
+                const index = list.findIndex((candidate) => isSamePending(candidate, entry));
+                if (index < 0)
+                    return current;
+                const next = [...list];
+                next[index] = { ...next[index], submission: prepared };
+                return { ...current, pending: next };
+            });
+        }
+        else if (intent.owner_deadline_at <= ctx.now()) {
+            // The frozen wire identity survives a crashed owner, but its local owner
+            // lease does not. Re-arm only the local deadline; request id, key,
+            // targets, and draft remain byte-identical for idempotent replay.
+            const frozenReplyWindow = intent.draft.reply?.expires_in_seconds ?? replyWindowSeconds;
+            intent = {
+                ...intent,
+                owner_deadline_at: Math.min(budgetDeadlineAt, ctx.now() +
+                    (frozenReplyWindow + HOOK_TIMING.submissionReserveSeconds) * 1000),
+            };
+            const rearmed = intent;
+            updateSessionState(sessionId, ctx.env, (current) => {
+                const list = pendingList(current);
+                const index = list.findIndex((candidate) => isSamePending(candidate, entry));
+                if (index < 0)
+                    return current;
+                const next = [...list];
+                next[index] = { ...next[index], submission: rearmed };
+                return { ...current, pending: next };
+            });
+        }
+        if (ctx.now() >= intent.owner_deadline_at) {
+            notes.push('the Stop budget ended before submission; preserving the frozen intent');
             continue;
         }
+        const beforeSubmitSeconds = Math.floor((intent.owner_deadline_at - ctx.now()) / 1000);
+        if (beforeSubmitSeconds <
+            MIN_REPLY_WAIT_SECONDS + HOOK_TIMING.submissionReserveSeconds / 2) {
+            notes.push('device resolution consumed the submission reserve; preserving the frozen intent instead of shrinking its reply window');
+            continue;
+        }
+        let receipt;
+        let admissionConfirmed = false;
+        try {
+            receipt = await submitQuestion(ctx, intent);
+            admissionConfirmed = true;
+        }
+        catch (err) {
+            if (err instanceof ApiCallError && err.status < 500 && err.status !== 408) {
+                // A concrete client response is an admission rejection, not an
+                // ambiguous commit. Keep the frozen intent for a later retry and leave
+                // the question in the terminal; polling a request the server says it
+                // did not accept only burns the owner window and hides the real fault.
+                notes.push(`question submission was rejected (${err.code}, HTTP ${err.status}); preserving it for recovery`);
+                ctx.log?.error('hook.pushed', {
+                    ok: false,
+                    request_id: intent.request_id,
+                    status: err.status,
+                    code: err.code,
+                    message: err.message,
+                });
+                continue;
+            }
+            // This is deliberately not a fail-open return. A transport failure is
+            // ambiguous: the server may already have dispatched the card. The
+            // client-generated request id keeps polling/finalization possible.
+            notes.push(`question submission response was ambiguous; recovering ${intent.request_id}`);
+            ctx.log?.error('hook.pushed', { ok: false, request_id: intent.request_id, message: String(err) });
+        }
+        if (admissionConfirmed) {
+            ctx.log?.info('hook.pushed', {
+                ok: true,
+                request_id: intent.request_id,
+                devices: intent.device_ids.length,
+                questions: questions.length,
+                text: questions[0].text,
+            });
+        }
+        const committedReplyDeadline = receipt?.reply_expires_at === null || receipt?.reply_expires_at === undefined
+            ? intent.owner_deadline_at
+            : Date.parse(receipt.reply_expires_at);
         const live = {
             ...entry,
-            request_id: sent.requestId,
-            collapse_key: sent.collapseKey,
-            device_ids: sent.devices,
+            request_id: intent.request_id,
+            collapse_key: intent.collapse_key,
+            device_ids: intent.device_ids,
+            // Submission and every later poll share one wall-clock owner. The
+            // server may start its relative window a little later while the submit
+            // response is in flight; closing at this deadline is therefore the
+            // authoritative backstop against accepting an answer into a void.
+            reply_deadline_at: Math.min(intent.owner_deadline_at, Number.isFinite(committedReplyDeadline)
+                ? committedReplyDeadline
+                : intent.owner_deadline_at),
         };
+        delete live.submission;
         submitted.push(live);
         // Record what is now live on the user's devices BEFORE any wait. If we
         // only learned these ids afterwards, a question that timed out would
         // leave no trace, and the user returning to the terminal could never
         // retire it — the notification would stay answerable for an hour with
         // nobody listening.
-        updateSessionState(sessionId, ctx.env, (current) => {
-            const list = pendingList(current);
-            const index = list.findIndex((candidate) => isSamePending(candidate, entry) && candidate.request_id === undefined);
-            if (index >= 0) {
-                const next = [...list];
-                next[index] = live;
-                return { ...current, pending: next };
-            }
-            // The entry vanished while the submit was in flight (the user's prompt
-            // wiped the queue). The delivered notification must still be retirable,
-            // so park it rather than lose its only identifiers.
-            const retirement = retiringQuestion(live, 'answered_elsewhere');
-            const retiring = [...(current.retiring ?? [])];
-            if (!retiring.some((parked) => parked.request_id === retirement.request_id)) {
-                retiring.push(retirement);
-            }
-            return { ...current, retiring };
-        });
+        if (admissionConfirmed) {
+            updateSessionState(sessionId, ctx.env, (current) => {
+                const list = pendingList(current);
+                const index = list.findIndex((candidate) => isSamePending(candidate, entry) && candidate.request_id === undefined);
+                if (index >= 0) {
+                    const next = [...list];
+                    next[index] = live;
+                    return { ...current, pending: next };
+                }
+                // The entry vanished while the submit was in flight (the user's prompt
+                // wiped the queue). The delivered notification must still be retirable,
+                // so park it rather than lose its only identifiers.
+                const retirement = retiringQuestion(live, 'answered_elsewhere');
+                const retiring = [...(current.retiring ?? [])];
+                if (!retiring.some((parked) => parked.request_id === retirement.request_id)) {
+                    retiring.push(retirement);
+                }
+                return { ...current, retiring };
+            });
+        }
+    }
+    const staleLive = alreadyLive.filter((entry) => entry.reply_deadline_at === undefined || entry.reply_deadline_at <= ctx.now());
+    const finalizedStale = await finalizePendings(ctx, staleLive);
+    const staleAnswers = finalizedStale
+        .map(finalizedAnswer)
+        .filter((entry) => entry !== null);
+    const staleSilence = finalizedStale
+        .filter((entry) => entry.response !== null && entry.response.replies.length === 0)
+        .map((entry) => entry.pending);
+    const staleUnproven = finalizedStale.filter((entry) => entry.response === null);
+    if (staleSilence.length > 0) {
+        await retirePendings(ctx, envelope, sessionId, staleSilence, 'expired');
+        notes.push(`expired ${staleSilence.length} question${staleSilence.length === 1 ? '' : 's'} whose continuation owner had ended`);
+    }
+    if (staleUnproven.length > 0) {
+        notes.push(`could not finalize ${staleUnproven.length} expired question${staleUnproven.length === 1 ? '' : 's'}; preserving ownership for recovery`);
+    }
+    if (staleAnswers.length > 0) {
+        const remaining = Math.max(0, pendingList(readSessionState(sessionId, ctx.env)).length - staleAnswers.length);
+        stageAcceptedAnswers(ctx, sessionId, staleAnswers, remaining);
+        for (const answer of staleAnswers)
+            reportAnswer(ctx, notes, answer, true);
+        return {
+            stdout: stopAnswerOutput(answersContext(staleAnswers, remaining)),
+            notes,
+        };
     }
-    const waitingOn = [...alreadyLive, ...submitted];
+    const waitingOn = [
+        ...alreadyLive.filter((entry) => entry.reply_deadline_at !== undefined && entry.reply_deadline_at > ctx.now()),
+        ...submitted,
+    ];
     if (waitingOn.length === 0)
         return { notes };
-    if (ctx.harness === 'opencode') {
-        notes.push(`question${submitted.length === 1 ? '' : 's'} sent without blocking OpenCode; retrieve answers on the next prompt or with: notifai replies --pending`);
-        return { notes };
-    }
     // Phase two: one blocking wait across everything live, old and new alike.
-    const timeoutSeconds = ctx.config.hook_reply_timeout_seconds.value;
-    const waited = await waitForAnyReplyWhileAway(ctx, waitingOn.map((entry) => entry.request_id), timeoutSeconds);
+    const ownerDeadline = Math.min(budgetDeadlineAt, Math.max(...waitingOn.map((entry) => entry.reply_deadline_at)));
+    const timeoutSeconds = Math.max(0, Math.ceil((ownerDeadline - ctx.now()) / 1000));
+    const waited = await waitForAnyReplyWhileAway(ctx, waitingOn.map((entry) => entry.request_id), timeoutSeconds, ctx.harness !== 'opencode');
+    const permanentFailure = permanentReplyFailureNote(waited.permanentFailures);
+    if (permanentFailure !== null)
+        notes.push(permanentFailure);
     if (waited.byRequest.size === 0) {
-        // Keep the pending records so a returning user's UserPromptSubmit can
-        // retire the notifications still live on their devices. `request_id`
-        // being set is also what stops the next Stop pushing them again.
-        const ids = waitingOn.map((entry) => entry.request_id).join(', ');
+        // Once this owner returns, no shipped command-hook harness can guarantee
+        // a later answer reaches the same agent turn. Close first, then retire the
+        // local records, so the Companion never accepts an answer into a void.
+        const finalized = await finalizePendings(ctx, waitingOn);
+        const finalAnswers = finalized
+            .map(finalizedAnswer)
+            .filter((entry) => entry !== null);
+        const confirmedSilent = finalized
+            .filter((entry) => entry.response !== null && entry.response.replies.length === 0)
+            .map((entry) => entry.pending);
+        const unproven = finalized.filter((entry) => entry.response === null);
+        if (confirmedSilent.length > 0) {
+            await retirePendings(ctx, envelope, sessionId, confirmedSilent, waited.userReturned ? 'answered_elsewhere' : 'expired');
+        }
+        if (unproven.length > 0) {
+            notes.push(`could not finalize ${unproven.length} question${unproven.length === 1 ? '' : 's'}; preserving ownership rather than risking a lost late answer`);
+        }
+        if (finalAnswers.length > 0) {
+            const remaining = Math.max(0, pendingList(readSessionState(sessionId, ctx.env)).length - finalAnswers.length);
+            stageAcceptedAnswers(ctx, sessionId, finalAnswers, remaining);
+            for (const answer of finalAnswers)
+                reportAnswer(ctx, notes, answer, false);
+            return {
+                stdout: stopAnswerOutput(answersContext(finalAnswers, remaining)),
+                notes,
+            };
+        }
+        const requestIdSummary = summarizeRequestIds(waitingOn);
+        ctx.log?.info('hook.answer', {
+            answered: false,
+            request_ids: requestIdSummary.ids,
+            user_returned: waited.userReturned,
+            degraded: waited.degraded,
+            permanent_failures: waited.permanentFailures.size,
+            waited_seconds: timeoutSeconds,
+        });
         if (waited.userReturned) {
-            notes.push(`you came back after the question${waitingOn.length === 1 ? ' was' : 's were'} sent; returning the terminal while ${waitingOn.length === 1 ? 'it stays' : 'they stay'} answerable (${ids}). ` +
-                'Retrieve answers with: notifai replies --pending');
+            notes.push(`you came back after the question${waitingOn.length === 1 ? ' was' : 's were'} sent; ${waitingOn.length === 1 ? 'it was' : 'they were'} retired before returning the terminal (${requestIdSummary.display})`);
         }
-        else {
+        else if (waited.permanentFailures.size === 0) {
             notes.push(waited.degraded
-                ? 'could not reach the server to find out whether you answered; check with: notifai replies --pending'
-                : 'no answer in time; retrieve it later with: notifai replies --pending');
+                ? 'could not reach the server before the owner deadline; the question was retired so no answer can be lost later'
+                : 'no answer in time; the question expired with its continuation owner');
         }
         return { notes };
     }
+    const polledAnswered = waitingOn.filter((entry) => waited.byRequest.has(entry.request_id));
+    const finalizedAnswered = await finalizePendings(ctx, polledAnswered);
     const answered = [];
-    for (const entry of waitingOn) {
+    for (const finalized of finalizedAnswered) {
+        const entry = finalized.pending;
         const replies = waited.byRequest.get(entry.request_id);
         if (replies === undefined)
             continue;
-        // First answer claims the question across devices; the close is what
-        // enforces that, and the latest reply within the window is the answer.
-        await closeQuietly(ctx, entry.request_id);
-        answered.push({ pending: entry, reply: replies.at(-1), replies });
-    }
-    await finishAnsweredPendings(ctx, envelope, sessionId, answered, true);
-    for (const { reply } of answered) {
-        notes.push(`answer from ${reply.device_name}: ${reply.text}`);
+        // The fenced set wins because it includes corrections that committed
+        // while close was waiting for earlier writers. If finalization itself was
+        // unreachable, never discard the durable reply already observed.
+        const finalReplies = finalized.response?.replies ?? [];
+        const authoritative = finalReplies.length > 0 ? finalReplies : replies;
+        answered.push({ pending: entry, reply: authoritative.at(-1), replies: authoritative });
+        if (finalized.response === null) {
+            notes.push(`could not confirm the close fence for ${entry.request_id}; delivering the durable answer already observed`);
+        }
     }
+    stageAcceptedAnswers(ctx, sessionId, answered, waitingOn.length - answered.length);
+    for (const answer of answered)
+        reportAnswer(ctx, notes, answer, false);
     return {
         stdout: stopAnswerOutput(answersContext(answered, waitingOn.length - answered.length)),
         notes,
@@ -1193,7 +1645,7 @@ export function handleSessionEnd(env, envelope, now = Date.now()) {
     const notes = [];
     const sessionId = envelope.session_id;
     if (!sessionId)
-        return { notes };
+        return { notes, log: { outcome: 'ignored', reason: 'missing-session-id' } };
     if (envelope.cwd !== undefined)
         clearMatchingProjectSession(envelope.cwd, env, sessionId);
     const state = readSessionState(sessionId, env);
@@ -1208,7 +1660,10 @@ export function handleSessionEnd(env, envelope, now = Date.now()) {
             notes.push(err instanceof Error ? err.message : String(err));
             // Preserve the only identifiers instead of turning an explicit corrupt
             // state into an unretirable question during SessionEnd cleanup.
-            return { notes };
+            return {
+                notes,
+                log: { outcome: 'preserved', reason: 'incomplete-question-identifiers' },
+            };
         }
     }
     if (orphans.length > 0) {
@@ -1216,8 +1671,27 @@ export function handleSessionEnd(env, envelope, now = Date.now()) {
         orphanRetirements(env, orphans, label !== undefined && label !== '' ? label : sessionId, now);
         notes.push(`queued ${orphans.length} question${orphans.length > 1 ? 's' : ''} for retirement on the next hook`);
     }
+    if (state.accepted !== undefined) {
+        const preserved = { accepted: state.accepted };
+        if (state.last_prompt_at !== undefined)
+            preserved.last_prompt_at = state.last_prompt_at;
+        if (state.last_stop_at !== undefined)
+            preserved.last_stop_at = state.last_stop_at;
+        if (state.continuation !== undefined)
+            preserved.continuation = state.continuation;
+        writeSessionState(sessionId, env, preserved);
+        notes.push('preserved an accepted device answer for this exact session to resume');
+        return {
+            notes,
+            log: {
+                outcome: 'answer-preserved',
+                queued_retirements: orphans.length,
+                accepted_answers: state.accepted.answers.length,
+            },
+        };
+    }
     clearSessionState(sessionId, env);
-    return { notes };
+    return { notes, log: { outcome: 'cleaned', queued_retirements: orphans.length } };
 }
 // ---------------------------------------------------------------------------
 // ask — register a question for the turn's end
@@ -1254,6 +1728,7 @@ export function registerQuestion(sessionId, env, question, now = Date.now()) {
             pending: [
                 ...pending,
                 {
+                    question_id: `q_${randomBytes(12).toString('base64url')}`,
                     asked_at: now,
                     ...question,
                     question: question.question.slice(0, MAX_STORED_QUESTION_CHARS),