@evident-ai/cli 3.0.1-dev.d06ae3c → 3.0.1-dev.d0ec1bd

package/dist/index.js

@@ -471,12 +471,6 @@ import chalk6 from "chalk";
 import ora3 from "ora";
 import { select as select3 } from "@inquirer/prompts";
 
-// ../../packages/types/src/opencode/index.ts
-function opencodeMessageIdFor(queuedMessageId) {
-  const sanitized = queuedMessageId.replace(/[^a-zA-Z0-9]/g, "_");
-  return `msg_${sanitized}`;
-}
-
 // ../../packages/types/src/telemetry/index.ts
 var TelemetryEventTypes = {
   // Agent activity events (shown in web UI activity log)
@@ -515,7 +509,10 @@ function stripQuery(url) {
 }
 
 // src/lib/telemetry.ts
-var CLI_VERSION = process.env.npm_package_version || "unknown";
+var CLI_VERSION = (true ? "3.0.0" : void 0) ?? process.env.npm_package_version ?? "unknown";
+function getCliVersion() {
+  return CLI_VERSION;
+}
 var eventBuffer = [];
 var flushTimeout = null;
 var isShuttingDown = false;
@@ -715,7 +712,7 @@ async function waitForOpenCodeHealth(port, timeoutMs = 3e4) {
 }
 
 // src/lib/opencode/opencode-version-gate.ts
-var QUEUE_VALIDATED_OPENCODE_VERSIONS = ["1.17.11"];
+var QUEUE_VALIDATED_OPENCODE_VERSIONS = ["1.17.11", "1.18.3"];
 function isQueueValidatedVersion(version2) {
   if (!version2) return false;
   return QUEUE_VALIDATED_OPENCODE_VERSIONS.includes(version2);
@@ -1039,7 +1036,11 @@ function roleOf(m) {
 }
 function completedOf(m) {
   if (!m || typeof m !== "object") return void 0;
-  return m.info?.time?.completed;
+  return m.info?.time?.completed ?? m.time?.completed;
+}
+function createdOf(m) {
+  if (!m || typeof m !== "object") return void 0;
+  return m.info?.time?.created ?? m.time?.created;
 }
 function idOf(m) {
   if (!m || typeof m !== "object") return void 0;
@@ -1059,10 +1060,24 @@ function finishOf(m) {
   const infoFinish = m.info?.finish;
   return typeof infoFinish === "string" ? infoFinish : void 0;
 }
+function errorOf(m) {
+  if (!m || typeof m !== "object") return void 0;
+  return m.info?.error ?? m.error;
+}
 function isAssistantInFlight(m) {
   if (completedOf(m) == null) return true;
   return finishOf(m) === "tool-calls";
 }
+async function getSessionMessages(port, sessionId) {
+  try {
+    const res = await fetch(`${opencodeBase(port)}/session/${sessionId}/message`);
+    if (!res.ok) return null;
+    const body = await res.json();
+    return Array.isArray(body) ? body : null;
+  } catch {
+    return null;
+  }
+}
 async function createOpenCodeSession(port, directory) {
   const url = new URL(`${opencodeBase(port)}/session`);
   if (directory && directory.trim()) {
@@ -1080,9 +1095,16 @@ async function createOpenCodeSession(port, directory) {
   const data = await response.json();
   return data.id;
 }
-async function sendPromptAsync(port, sessionId, content, options, messageId) {
+function messageText(m) {
+  if (!m || !Array.isArray(m.parts)) return "";
+  return m.parts.filter((p) => p.type === "text" && typeof p.text === "string").map((p) => p.text).join("");
+}
+async function sendPromptAsync(port, sessionId, content, options) {
+  const before = await getSessionMessages(port, sessionId);
+  const knownUserIds = new Set(
+    (before ?? []).filter((m) => roleOf(m) === "user").map((m) => idOf(m)).filter((id) => typeof id === "string")
+  );
   const body = {
-    messageID: messageId,
     parts: [{ type: "text", text: content }]
   };
   if (options?.agent) {
@@ -1106,6 +1128,29 @@ async function sendPromptAsync(port, sessionId, content, options, messageId) {
     const text = await res.text().catch(() => "");
     throw new Error(`OpenCode prompt_async failed: HTTP ${res.status}${text ? `: ${text}` : ""}`);
   }
+  const READ_BACK_ATTEMPTS = 5;
+  const READ_BACK_DELAY_MS = 150;
+  for (let attempt = 0; attempt < READ_BACK_ATTEMPTS; attempt++) {
+    const after = await getSessionMessages(port, sessionId);
+    if (after) {
+      let best = null;
+      for (const m of after) {
+        if (roleOf(m) !== "user") continue;
+        const id = idOf(m);
+        if (typeof id !== "string" || knownUserIds.has(id)) continue;
+        if (messageText(m) !== content) continue;
+        const created = createdOf(m) ?? 0;
+        if (best === null || created > best.created) {
+          best = { id, created };
+        }
+      }
+      if (best) return best.id;
+    }
+    if (attempt < READ_BACK_ATTEMPTS - 1) {
+      await new Promise((resolve) => setTimeout(resolve, READ_BACK_DELAY_MS));
+    }
+  }
+  return null;
 }
 function findAssistantReplyAfter(messages, userMessageId) {
   if (!messages || messages.length === 0) return null;
@@ -1122,19 +1167,31 @@ function findAssistantReplyAfter(messages, userMessageId) {
 }
 function findLastAssistantReplyFor(messages, userMessageId) {
   if (!messages || messages.length === 0) return null;
+  let lastCorrelated = null;
+  let lastNonErrored = null;
   for (let i = messages.length - 1; i >= 0; i--) {
     const m = messages[i];
-    if (roleOf(m) === "assistant" && parentIdOf(m) === userMessageId) return m;
+    if (roleOf(m) !== "assistant" || parentIdOf(m) !== userMessageId) continue;
+    if (lastCorrelated === null) lastCorrelated = m;
+    if (errorOf(m) == null) {
+      lastNonErrored = m;
+      break;
+    }
   }
+  if (lastCorrelated) return lastNonErrored ?? lastCorrelated;
   const userIndex = messages.findIndex((m) => idOf(m) === userMessageId);
   if (userIndex === -1) return null;
   let last = null;
+  let lastOk = null;
   for (let i = userIndex + 1; i < messages.length; i++) {
     const role = roleOf(messages[i]);
     if (role === "user") break;
-    if (role === "assistant") last = messages[i];
+    if (role === "assistant") {
+      last = messages[i];
+      if (errorOf(messages[i]) == null) lastOk = messages[i];
+    }
   }
-  return last;
+  return lastOk ?? last;
 }
 function messageRunState(messages, userMessageId) {
   if (!messages || messages.length === 0) return "unknown";
@@ -1144,7 +1201,21 @@ function messageRunState(messages, userMessageId) {
     if (!reply) return "unknown";
   }
   if (!reply) return "queued";
-  return isAssistantInFlight(reply) ? "running" : "done";
+  if (isAssistantInFlight(reply)) return "running";
+  return errorOf(reply) != null ? "failed" : "done";
+}
+function messageError(messages, userMessageId) {
+  const reply = findLastAssistantReplyFor(messages, userMessageId);
+  const error2 = errorOf(reply);
+  if (error2 == null) return null;
+  if (typeof error2 === "string") return error2;
+  if (typeof error2 === "object") {
+    const e = error2;
+    const dataMessage = e.data?.message;
+    if (typeof dataMessage === "string") return dataMessage;
+    if (typeof e.message === "string") return e.message;
+  }
+  return "The agent run failed.";
 }
 function hasRunningAssistantExcept(messages, exceptUserMessageId) {
   if (!messages || messages.length === 0) return false;
@@ -1152,9 +1223,6 @@ function hasRunningAssistantExcept(messages, exceptUserMessageId) {
     (m) => roleOf(m) === "assistant" && parentIdOf(m) !== exceptUserMessageId && isAssistantInFlight(m)
   );
 }
-function opencodeMessageIdFor2(queuedMessageId) {
-  return opencodeMessageIdFor(queuedMessageId);
-}
 
 // src/lib/tunnel/connection.ts
 import WebSocket2 from "ws";
@@ -1559,7 +1627,6 @@ var DEFAULT_RETRY_POLICY = {
 };
 var DEFAULT_PAUSED_POLL_INTERVAL_MS = 2e3;
 var DEFAULT_PAUSED_MAX_WAIT_MS = 10 * 60 * 1e3;
-var DEFAULT_DISPATCH_CONFIRM_MS = 6e3;
 var DEFAULT_STUCK_QUEUED_MS = 6e4;
 var ChannelAuthError = class extends Error {
   constructor(message) {
@@ -1595,11 +1662,18 @@ var ChannelDriver = class {
   sleep;
   pausedPollIntervalMs;
   pausedMaxWaitMs;
-  dispatchConfirmMs;
   stuckQueuedMs;
   now;
   /** Cache of conversationId → opencode sessionId. */
   sessions = /* @__PURE__ */ new Map();
+  /**
+   * Per-opencode-session dispatch lock (Task 2.1a). `sendPromptAsync` is no
+   * longer idempotent (no caller-supplied `messageID`), and its read-back picks
+   * "the one new user row" — which is only unambiguous if no OTHER dispatch into
+   * the SAME session interleaves its snapshot→POST→read-back. This map chains each
+   * session's dispatches so they run serially; distinct sessions stay concurrent.
+   */
+  sessionDispatchLocks = /* @__PURE__ */ new Map();
   /**
    * Per-SESSION watchers (WI-3), keyed by opencode sessionId. Single-flight per
    * session: one polling loop services all of that session's in-flight messages.
@@ -1650,6 +1724,20 @@ var ChannelDriver = class {
    * the row leaves the processing list, exactly like `dontRedispatch`.
    */
   doneUndeliverable = /* @__PURE__ */ new Set();
+  /**
+   * "A null-id re-adopt re-dispatch is in flight, awaiting its read-back" (WI-5
+   * Task 5.4, High-2). Since we no longer send a caller-supplied id, a re-dispatch
+   * is NOT idempotent: if `forceReadoptRun` dispatches on tick N but the read-back +
+   * persist hasn't landed before tick N+1 re-reads the still-null
+   * `row.opencode_message_id`, tick N+1 would dispatch AGAIN → duplicate user turns.
+   * A row is added here right before its `sendPromptAsync` and `forceReadoptRun`
+   * short-circuits while it is present, so a null-id row is re-dispatched AT MOST
+   * ONCE per outstanding read-back. Cleared on a SUCCESSFUL dispatch+read-back (the
+   * row is then tracked in `dispatched`, so `readoptOne`'s early skip prevents
+   * re-entry) OR on a failed/unresolved dispatch (the message is genuinely un-sent,
+   * so the NEXT tick may retry exactly once more).
+   */
+  awaitingReadopt = /* @__PURE__ */ new Set();
   /**
    * Cache of the opencode root directory (from `GET /path`). Resolved lazily on
    * first session creation so drain-created sessions are rooted at the project
@@ -1672,7 +1760,6 @@ var ChannelDriver = class {
     this.sleep = config2.sleep ?? ((ms) => new Promise((r) => setTimeout(r, ms)));
     this.pausedPollIntervalMs = config2.pausedPollIntervalMs ?? DEFAULT_PAUSED_POLL_INTERVAL_MS;
     this.pausedMaxWaitMs = config2.pausedMaxWaitMs ?? DEFAULT_PAUSED_MAX_WAIT_MS;
-    this.dispatchConfirmMs = config2.dispatchConfirmMs ?? DEFAULT_DISPATCH_CONFIRM_MS;
     this.stuckQueuedMs = config2.stuckQueuedMs ?? DEFAULT_STUCK_QUEUED_MS;
     this.now = config2.now ?? (() => Date.now());
   }
@@ -1764,11 +1851,11 @@ var ChannelDriver = class {
         skippedAlreadyDispatched += 1;
         continue;
       }
-      const opencodeMessageId = opencodeMessageIdFor2(message.id);
       const options = {
         agent: message.opencode_agent ?? void 0,
         model: message.opencode_model ?? void 0
       };
+      let opencodeMessageId;
       try {
         this.log({
           level: "info",
@@ -1776,7 +1863,10 @@ var ChannelDriver = class {
           conversation_id: conv.id,
           message_id: message.id
         });
-        await sendPromptAsync(this.port, sessionId, message.content, options, opencodeMessageId);
+        opencodeMessageId = await this.dispatchLocked(
+          sessionId,
+          () => sendPromptAsync(this.port, sessionId, message.content, options)
+        );
       } catch (err) {
         if (err instanceof ChannelAuthError) throw err;
         this.dispatched.delete(message.id);
@@ -1790,6 +1880,15 @@ var ChannelDriver = class {
         });
         continue;
       }
+      if (opencodeMessageId === null) {
+        this.log({
+          level: "error",
+          message: `Message ${message.id.slice(0, 8)} dispatched but its opencode id could not be read back \u2014 leaving un-tracked to retry next tick`,
+          conversation_id: conv.id,
+          message_id: message.id
+        });
+        continue;
+      }
       this.dispatched.add(message.id);
       this.registerInFlight(conv, sessionId, message, opencodeMessageId);
       dispatched += 1;
@@ -1838,6 +1937,25 @@ var ChannelDriver = class {
   // -------------------------------------------------------------------------
   // Per-session watcher (WI-3)
   // -------------------------------------------------------------------------
+  /**
+   * Run one dispatch (`sendPromptAsync` snapshot→POST→read-back) serialized per
+   * opencode session (Task 2.1a), so two dispatches into the SAME session can
+   * never interleave and mis-correlate their read-backs. Distinct sessions run
+   * concurrently. The chained tail intentionally ignores the prior result/error
+   * (each dispatch reports its own outcome to its caller).
+   */
+  dispatchLocked(sessionId, fn) {
+    const prior = this.sessionDispatchLocks.get(sessionId) ?? Promise.resolve();
+    const run2 = prior.then(fn, fn);
+    this.sessionDispatchLocks.set(
+      sessionId,
+      run2.then(
+        () => void 0,
+        () => void 0
+      )
+    );
+    return run2;
+  }
   /** Register a freshly-dispatched message with its session's watcher state. */
   registerInFlight(conv, sessionId, message, opencodeMessageId) {
     let watcher = this.watchers.get(sessionId);
@@ -1902,10 +2020,10 @@ var ChannelDriver = class {
       started: true,
       done: false,
       // Not yet reported stuck-queued. The once-guard (`stuckReported`) applies,
-      // AND the stuck-queued observer now INCLUDES re-adopted queued wedges: it
-      // gates on `state === 'queued'` (turn produced no reply), not on `started`,
-      // so a re-adopted row left wedged in `queued` still emits the signal once
-      // (queued-followup-redrive, #210).
+      // AND the stuck-queued observer INCLUDES re-adopted queued wedges: it gates
+      // on `state === 'queued'` (turn produced no reply), not on `started`, so a
+      // re-adopted row left wedged in `queued` still emits the signal once
+      // (#210/#220 observability).
       stuckReported: false
     });
   }
@@ -1993,10 +2111,15 @@ var ChannelDriver = class {
   async serviceInFlightMessage(sessionId, watcher, inFlight, messages) {
     const conv = watcher.conv;
     const state = messageRunState(messages, inFlight.opencodeMessageId);
-    if ((state === "running" || state === "done") && !inFlight.started) {
+    if ((state === "running" || state === "done" || state === "failed") && !inFlight.started) {
       let claimed;
       try {
-        claimed = await this.markProcessing(conv.id, inFlight.evidentMessageId, sessionId);
+        claimed = await this.markProcessing(
+          conv.id,
+          inFlight.evidentMessageId,
+          sessionId,
+          inFlight.opencodeMessageId
+        );
       } catch (err) {
         if (err instanceof ChannelAuthError) throw err;
         this.log({
@@ -2026,7 +2149,12 @@ var ChannelDriver = class {
           message_id: inFlight.evidentMessageId
         });
         try {
-          await this.markDone(conv.id, inFlight.evidentMessageId, sessionId);
+          await this.markDone(
+            conv.id,
+            inFlight.evidentMessageId,
+            sessionId,
+            inFlight.opencodeMessageId
+          );
         } catch (err) {
           if (err instanceof ChannelAuthError) throw err;
           if (err instanceof ChannelTerminalError) {
@@ -2062,12 +2190,55 @@ var ChannelDriver = class {
       this.removeInFlight(watcher, inFlight.evidentMessageId);
       return;
     }
-    if (state === "unknown") {
-      if (this.now() - inFlight.dispatchedAt >= this.dispatchConfirmMs) {
-        await this.redispatchInFlight(sessionId, inFlight);
+    if (state === "failed") {
+      if (!inFlight.done) {
+        const error2 = messageError(messages, inFlight.opencodeMessageId) ?? void 0;
+        this.log({
+          level: "error",
+          message: `Message ${inFlight.evidentMessageId.slice(0, 8)} errored \u2014 marking failed: ${error2 ?? "(no error text)"}`,
+          conversation_id: conv.id,
+          message_id: inFlight.evidentMessageId
+        });
+        try {
+          await this.markFailed(conv.id, inFlight.evidentMessageId, sessionId, error2);
+        } catch (err) {
+          if (err instanceof ChannelAuthError) throw err;
+          if (err instanceof ChannelTerminalError) {
+            this.log({
+              level: "error",
+              message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} failed (terminal HTTP ${err.status}) \u2014 leaving for the cron safety net: ${err.message}`,
+              conversation_id: conv.id,
+              message_id: inFlight.evidentMessageId
+            });
+            this.removeInFlight(watcher, inFlight.evidentMessageId);
+            return;
+          }
+          if (this.now() >= inFlight.deadline) {
+            this.log({
+              level: "error",
+              message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} failed within the watch window \u2014 leaving for the cron safety net: ${err instanceof Error ? err.message : String(err)}`,
+              conversation_id: conv.id,
+              message_id: inFlight.evidentMessageId
+            });
+            this.removeInFlight(watcher, inFlight.evidentMessageId);
+            return;
+          }
+          this.log({
+            level: "error",
+            message: `Failed to mark message ${inFlight.evidentMessageId.slice(0, 8)} failed (will retry next tick): ${err instanceof Error ? err.message : String(err)}`,
+            conversation_id: conv.id,
+            message_id: inFlight.evidentMessageId
+          });
+          return;
+        }
+        inFlight.done = true;
       }
+      this.removeInFlight(watcher, inFlight.evidentMessageId);
+      return;
     }
-    if (state === "queued" && !inFlight.stuckReported && this.now() - inFlight.dispatchedAt >= this.stuckQueuedMs && !hasRunningAssistantExcept(messages, inFlight.opencodeMessageId)) {
+    const pastStuckBound = this.now() - inFlight.dispatchedAt >= this.stuckQueuedMs;
+    const sessionIdle = state === "queued" && !hasRunningAssistantExcept(messages, inFlight.opencodeMessageId);
+    if (state === "queued" && pastStuckBound && sessionIdle && !inFlight.stuckReported) {
       inFlight.stuckReported = true;
       void this.postSignal(conv.id, inFlight.evidentMessageId, "stuck_queued", {
         stuck_for_ms: this.now() - inFlight.dispatchedAt
@@ -2080,41 +2251,11 @@ var ChannelDriver = class {
         conversation_id: conv.id,
         message_id: inFlight.evidentMessageId
       });
-      this.removeInFlight(watcher, inFlight.evidentMessageId);
-    }
-  }
-  /**
-   * Re-dispatch a message whose user row never appeared (idle-path guard). Safe:
-   * opencode treats a duplicate caller-supplied `messageID` as idempotent (PoC
-   * fact 9) — one user message + one reply even if the original DID land. Resets
-   * the dispatch timestamp so the guard doesn't immediately fire again.
-   */
-  async redispatchInFlight(sessionId, inFlight) {
-    const options = {
-      agent: inFlight.message.opencode_agent ?? void 0,
-      model: inFlight.message.opencode_model ?? void 0
-    };
-    this.log({
-      level: "info",
-      message: `Message ${inFlight.evidentMessageId.slice(0, 8)} not observed after dispatch \u2014 re-dispatching (idle-path guard)`,
-      message_id: inFlight.evidentMessageId
-    });
-    try {
-      await sendPromptAsync(
-        this.port,
-        sessionId,
-        inFlight.message.content,
-        options,
-        inFlight.opencodeMessageId
-      );
-    } catch (err) {
-      this.log({
-        level: "error",
-        message: `Re-dispatch failed for message ${inFlight.evidentMessageId.slice(0, 8)}: ${err instanceof Error ? err.message : String(err)}`,
-        message_id: inFlight.evidentMessageId
+      void this.postSignal(conv.id, inFlight.evidentMessageId, "gave_up", {
+        watched_for_ms: this.now() - inFlight.dispatchedAt
       });
+      this.removeInFlight(watcher, inFlight.evidentMessageId);
     }
-    inFlight.dispatchedAt = this.now();
   }
   // -------------------------------------------------------------------------
   // Restart recovery: re-adopt `processing` messages (ADR-0046, WI-3/4/5)
@@ -2203,10 +2344,17 @@ var ChannelDriver = class {
    * Re-adopt ONE `processing` row against the tick's session message snapshot
    * (ADR-0046 Decision §1/§2). Idempotent: skips a row already being driven.
    *
-   * Branches on `messageRunState(messages, opencodeMessageIdFor(row.id))`:
+   * Branches on `messageRunState(messages, row.opencode_message_id)` — the
+   * opencode-assigned user-message id persisted on the first `processing` PATCH
+   * (#218). A row with a NULL stored id (dispatched but the read-back never landed
+   * before the restart) has no id to correlate → treated as an orphan and
+   * re-dispatched (at most once, see `forceReadoptRun`):
    *   - `done`             → `markDone` now (guarded like the watcher's done branch);
-   *   - `running`/`queued` → re-attach a watcher via `registerReadopted` (no re-dispatch);
-   *   - `unknown`          → re-dispatch the STABLE id + attach a watcher (orphan).
+   *   - `failed`           → `markFailed` with the surfaced error (issue #182), so an
+   *                          errored turn is reported failed on restart, NOT re-dispatched;
+   *   - `running`/`queued` → re-attach a watcher via `registerReadopted` (no re-dispatch),
+   *                          tracking the stored id so the reply correlates by it;
+   *   - `unknown`/null id  → re-dispatch (opencode assigns a fresh id) + attach a watcher.
    *
    * Only `ChannelAuthError` propagates.
    */
@@ -2220,8 +2368,8 @@ var ChannelDriver = class {
       });
       return;
     }
-    const ocId = opencodeMessageIdFor2(row.id);
-    const state = messageRunState(messages, ocId);
+    const ocId = row.opencode_message_id;
+    const state = messageRunState(messages, ocId ?? "");
     if (state === "done") {
       if (this.doneUndeliverable.has(row.id)) {
         this.log({
@@ -2239,7 +2387,7 @@ var ChannelDriver = class {
         message_id: row.id
       });
       try {
-        await this.markDone(row.conversation_id, row.id, sessionId);
+        await this.markDone(row.conversation_id, row.id, sessionId, ocId);
       } catch (err) {
         if (err instanceof ChannelAuthError) throw err;
         if (err instanceof ChannelTerminalError) {
@@ -2263,6 +2411,39 @@ var ChannelDriver = class {
       this.dontRedispatch.delete(row.id);
       return;
     }
+    if (state === "failed") {
+      const error2 = messageError(messages, ocId ?? "") ?? void 0;
+      this.log({
+        level: "error",
+        message: `Re-adopt: message ${row.id.slice(0, 8)} errored while unwatched \u2014 marking failed: ${error2 ?? "(no error text)"}`,
+        conversation_id: row.conversation_id,
+        message_id: row.id
+      });
+      try {
+        await this.markFailed(row.conversation_id, row.id, sessionId, error2);
+      } catch (err) {
+        if (err instanceof ChannelAuthError) throw err;
+        if (err instanceof ChannelTerminalError) {
+          this.doneUndeliverable.add(row.id);
+          this.log({
+            level: "error",
+            message: `Re-adopt: failed to mark message ${row.id.slice(0, 8)} failed (terminal HTTP ${err.status}) \u2014 parking until it leaves processing; leaving for the cron safety net: ${err.message}`,
+            conversation_id: row.conversation_id,
+            message_id: row.id
+          });
+          return;
+        }
+        this.log({
+          level: "error",
+          message: `Re-adopt: failed to mark message ${row.id.slice(0, 8)} failed (will retry next drain): ${err instanceof Error ? err.message : String(err)}`,
+          conversation_id: row.conversation_id,
+          message_id: row.id
+        });
+        return;
+      }
+      this.dontRedispatch.delete(row.id);
+      return;
+    }
     if (this.dontRedispatch.has(row.id)) {
       this.log({
         level: "info",
@@ -2272,7 +2453,7 @@ var ChannelDriver = class {
       });
       return;
     }
-    if (state === "running" || state === "queued") {
+    if ((state === "running" || state === "queued") && ocId) {
       const conv = this.convForRow(sessionId, row);
       const message = this.queuedMessageForRow(row);
       this.registerReadopted(conv, sessionId, message, ocId, this.processedAtMs(row));
@@ -2281,7 +2462,7 @@ var ChannelDriver = class {
       this.ensureWatcherRunning(sessionId);
       this.log({
         level: "info",
-        message: `Re-adopt: message ${row.id.slice(0, 8)} ${state} \u2014 re-attached watcher (stable id, no re-dispatch)`,
+        message: `Re-adopt: message ${row.id.slice(0, 8)} ${state} \u2014 re-attached watcher (stored id, no re-dispatch)`,
         conversation_id: row.conversation_id,
         message_id: row.id
       });
@@ -2290,23 +2471,36 @@ var ChannelDriver = class {
     await this.forceReadoptRun(sessionId, row);
   }
   /**
-   * Re-dispatch an orphaned (`unknown`) `processing` row (ADR-0046 Decision §2).
+   * Re-dispatch an orphaned (`unknown`/null-id) `processing` row (ADR-0046 §2).
    *
-   * The stable-id user message is absent from the session, so we (re-)dispatch with
-   * the STABLE id (`opencodeMessageIdFor(row.id)`) — NOT a divergent per-attempt id.
-   * This is what keeps the reply correlatable: the server's completion
-   * notification looks for the reply under the stable id, so the fresh turn's reply
-   * (which hangs off the stable id) is found and delivered. The residual
-   * duplicate-incomplete-turn semantics (ADR §2, `.harness/restart-recovery-orphan-finding.md`)
-   * are unchanged and, for an ABSENT id, cannot bite — there is no existing turn
-   * to swallow the duplicate.
+   * #218/WI-5: the row's user message is absent (never kept, or a null stored id),
+   * so we re-`prompt_async` WITHOUT a caller id (opencode assigns a monotonic one),
+   * read it back, and register the watcher under the assigned id so the reply
+   * correlates server-side.
    *
-   * `evidentMessageId = row.id` addresses the SERVER row; the stable
-   * `opencodeMessageId` is what the watcher polls. Deadline anchored to
+   * ⚠️ AT-MOST-ONCE (High-2): dispatch is no longer idempotent (no caller-supplied
+   * id). Without a guard, if this dispatches on tick N but the read-back+persist
+   * hasn't landed before tick N+1 re-reads the still-null `opencode_message_id`,
+   * tick N+1 would dispatch AGAIN → duplicate user turns. The `awaitingReadopt`
+   * latch makes a null-id row re-dispatched AT MOST ONCE per outstanding read-back:
+   * short-circuit while the row is latched; clear it on a successful dispatch (the
+   * row is then tracked in `dispatched`, so `readoptOne`'s early skip prevents
+   * re-entry) OR on a failed/unresolved dispatch (genuinely un-sent → the next tick
+   * may retry exactly once more).
+   *
+   * `evidentMessageId = row.id` addresses the SERVER row. Deadline anchored to
    * `processed_at` (Invariant 1).
    */
   async forceReadoptRun(sessionId, row) {
-    const ocId = opencodeMessageIdFor2(row.id);
+    if (this.awaitingReadopt.has(row.id)) {
+      this.log({
+        level: "info",
+        message: `Re-adopt: message ${row.id.slice(0, 8)} already has a re-dispatch awaiting read-back \u2014 skipping (at most once)`,
+        conversation_id: row.conversation_id,
+        message_id: row.id
+      });
+      return;
+    }
     if (this.processedAtMs(row) + this.pausedMaxWaitMs <= this.now()) {
       this.dontRedispatch.add(row.id);
       this.log({
@@ -2323,13 +2517,19 @@ var ChannelDriver = class {
     };
     this.log({
       level: "info",
-      message: `Re-adopt: message ${row.id.slice(0, 8)} orphaned (user message absent) \u2014 re-dispatching with the stable id`,
+      message: `Re-adopt: message ${row.id.slice(0, 8)} orphaned (user message absent) \u2014 re-dispatching (opencode assigns a fresh id)`,
       conversation_id: row.conversation_id,
       message_id: row.id
     });
+    this.awaitingReadopt.add(row.id);
+    let ocId;
     try {
-      await sendPromptAsync(this.port, sessionId, row.content, options, ocId);
+      ocId = await this.dispatchLocked(
+        sessionId,
+        () => sendPromptAsync(this.port, sessionId, row.content, options)
+      );
     } catch (err) {
+      this.awaitingReadopt.delete(row.id);
       if (err instanceof ChannelAuthError) throw err;
       this.log({
         level: "error",
@@ -2339,11 +2539,22 @@ var ChannelDriver = class {
       });
       return;
     }
+    if (ocId === null) {
+      this.awaitingReadopt.delete(row.id);
+      this.log({
+        level: "error",
+        message: `Re-adopt: message ${row.id.slice(0, 8)} re-dispatched but its opencode id could not be read back \u2014 leaving un-tracked to retry next drain`,
+        conversation_id: row.conversation_id,
+        message_id: row.id
+      });
+      return;
+    }
     const conv = this.convForRow(sessionId, row);
     const message = this.queuedMessageForRow(row);
     this.registerReadopted(conv, sessionId, message, ocId, this.processedAtMs(row));
     this.dispatched.add(row.id);
     this.readopted.add(row.id);
+    this.awaitingReadopt.delete(row.id);
     this.ensureWatcherRunning(sessionId);
   }
   /**
@@ -2613,13 +2824,17 @@ var ChannelDriver = class {
    * A single attempt (no internal retry): the watcher's per-tick loop is the
    * retry vehicle for the swap-to-running.
    */
-  async markProcessing(conversationId, messageId, sessionId) {
+  async markProcessing(conversationId, messageId, sessionId, opencodeMessageId) {
     const res = await this.fetchImpl(
       `${this.apiUrl}/agents/${this.agentId}/threads/${conversationId}/messages/${messageId}`,
       {
         method: "PATCH",
         headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
-        body: JSON.stringify({ status: "processing", opencode_session_id: sessionId })
+        body: JSON.stringify({
+          status: "processing",
+          opencode_session_id: sessionId,
+          ...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {}
+        })
       }
     );
     this.assertAuth(res, "marking message as processing");
@@ -2657,13 +2872,17 @@ var ChannelDriver = class {
    *                                    watcher retries next tick within the
    *                                    deadline, Finding 4).
    */
-  async markDone(conversationId, messageId, sessionId) {
+  async markDone(conversationId, messageId, sessionId, opencodeMessageId) {
     const res = await this.fetchImpl(
       `${this.apiUrl}/agents/${this.agentId}/threads/${conversationId}/messages/${messageId}`,
       {
         method: "PATCH",
         headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
-        body: JSON.stringify({ status: "done", opencode_session_id: sessionId })
+        body: JSON.stringify({
+          status: "done",
+          opencode_session_id: sessionId,
+          ...opencodeMessageId ? { opencode_message_id: opencodeMessageId } : {}
+        })
       }
     );
     this.assertAuth(res, "marking message as done");
@@ -2673,7 +2892,17 @@ var ChannelDriver = class {
     }
     throw new ChannelTerminalError(`marking message as done: HTTP ${res.status}`, res.status);
   }
-  async markFailed(conversationId, messageId) {
+  /**
+   * Mark a message `failed`. `sessionId` / `error` are threaded to the API ONLY
+   * when provided (issue #182): a bare `markFailed(conv, msg)` sends
+   * `{status:'failed'}` unchanged (the dispatch-failure path), while an errored
+   * OpenCode turn sends `{status:'failed', opencode_session_id, error}` so the
+   * failure reason reaches the channel.
+   */
+  async markFailed(conversationId, messageId, sessionId, error2) {
+    const body = { status: "failed" };
+    if (sessionId !== void 0) body.opencode_session_id = sessionId;
+    if (error2 !== void 0) body.error = error2;
     await this.callWithRetry(
       "marking message as failed",
       () => this.fetchImpl(
@@ -2681,7 +2910,7 @@ var ChannelDriver = class {
         {
           method: "PATCH",
           headers: { Authorization: this.getAuthHeader(), "Content-Type": "application/json" },
-          body: JSON.stringify({ status: "failed" })
+          body: JSON.stringify(body)
         }
       )
     );
@@ -3037,6 +3266,7 @@ async function getAgentInfo(agentId, authHeader) {
 // src/commands/run.ts
 var MAX_ACTIVITY_LOG_ENTRIES = 10;
 var CHANNEL_POLL_INTERVAL_MS = Number(process.env.EVIDENT_CHANNEL_POLL_INTERVAL_MS) || 2e3;
+var CHANNEL_STUCK_QUEUED_MS = Number(process.env.EVIDENT_STUCK_QUEUED_MS) || void 0;
 function log2(state, message, isError = false) {
   if (state.json) {
     console.log(
@@ -3374,6 +3604,7 @@ async function run(options) {
       apiUrl: getApiUrlConfig(),
       getAuthHeader: () => state.authHeader,
       conversationFilter: state.conversationFilter,
+      stuckQueuedMs: CHANNEL_STUCK_QUEUED_MS,
       log: (entry) => logActivity(state, {
         type: entry.level === "error" ? "error" : "info",
         message: entry.message,
@@ -3393,7 +3624,11 @@ async function run(options) {
             type: "info",
             message: `Tunnel ${isReconnect ? "reconnected" : "connected"} (agent: ${agentId})`
           });
-          emitAgentConnected(state.agentId, { port: state.port });
+          emitAgentConnected(state.agentId, {
+            port: state.port,
+            cli_version: getCliVersion(),
+            opencode_version: state.opencodeVersion
+          });
           if (!isReconnect) tunnelSpinner?.succeed("Tunnel connected");
           if (state.interactive) displayStatus(state);
           channelDriver.drainPending().then((processed) => {