lavish-axi 0.1.66 → 0.1.68

package/README.md

@@ -198,13 +198,14 @@ pnpm link
   For reversible choices, let option clicks update local state, then queue exactly one final answer from a per-question submit or Queue answer button with `window.lavish.queuePrompt()`.
   Mark only custom (non-native) clickable elements with `data-lavish-action` so Lavish does not annotate them, and use `data-lavish-question` or `queueKey` when pre-send updates for the same question should replace each other.
   On wider screens, queued annotation preview pills and chat history share a scrollable Conversation panel above a sticky composer, so long feedback queues do not push the text box or send controls off screen.
-  The browser chrome keeps editing actions in the overflow menu (copy path, reload artifact, copy DOM snapshot, export standalone HTML, publish link, end session), while the composer exposes **Send & End** beside **Send to Agent** to submit queued prompts and user-ended attribution together.
-  Composer feedback is visibly queued before the text box clears and stays queued until acknowledged, with persistent recovery guidance if sending fails or takes longer than 10 seconds.
+  The browser chrome keeps editing actions in the overflow menu (copy path, reload artifact, copy DOM snapshot, export standalone HTML, publish link, end session), while the composer exposes **Send & End** beside **Send to Agent** to submit queued prompts and user-ended attribution together. Once **Send & End** starts, Lavish pauses new feedback, waits up to 5 seconds for feedback preparation already underway, keeps any open annotation draft intact until delivery succeeds, then holds that exact terminal batch. A completed reservation survives a page reload, including when its prompts were already delivered and only the pending end remains; incomplete preparation is not restored. If preparation times out, Lavish keeps the review open and its existing queue editable with a visible error. A transient delivery failure permits only retrying the same terminal batch through **Send & End** until it succeeds. If a snapshot makes the request too large (HTTP 413), Lavish retries that exact batch once without the optional snapshot. An actionable attachment rejection (HTTP 400), recoverable layout-selection conflict (HTTP 409), or unrecoverable 413 instead cancels the terminal reservation without ending the session, keeps the prompts queued, and restores editing, removal, **Send to Agent**, **Send & End**, and **End session** so the reviewer can revise or remove stale feedback before submitting again.
+  Composer feedback is visibly queued before the text box clears and stays queued until acknowledged. Send waits up to 5 seconds for the artifact's DOM snapshot, then delivers without that optional context if the frame no longer answers; a late snapshot cannot submit the batch twice. A failed or still-unacknowledged POST keeps the queue and shows persistent recovery guidance.
 - **Reviewing on a phone** - Below 860px wide, the artifact takes the whole screen above a **Conversation** dock, and the conversation opens as a bottom sheet over it: tap the dock, swipe it up, or press the chevron to raise it; tap the dimmed artifact, swipe the sheet down, press the chevron, or press Escape to lower it. The dock reports what matters while the sheet is down - how many prompts are queued, a reply that arrived while you were reading, or whether the agent is listening - and the sheet stays open across a reload of the review page. The sheet sizes itself to the visible viewport and respects safe-area insets; if the keyboard or attachments leave little room, conversation content yields or scrolls while the send actions remain pinned above the bottom edge. In landscape the sheet covers the top bar as well. Wider screens keep the side-by-side layout.
 - **Keyboard shortcuts** - In the chrome composer, Enter sends queued prompts and Shift+Enter inserts a newline.
   In the annotation card, Enter queues the annotation, Shift+Enter inserts a newline, and Ctrl+Enter (Cmd+Enter on macOS) queues it and sends all queued prompts immediately. Escape closes the card, same as Cancel, but only while it is empty (no text, no attachment); with unsent text or an attachment present, Escape does nothing rather than risk discarding it.
   Cmd+I or Ctrl+I toggles between annotate and explore mode from either the browser chrome or the artifact iframe, including while focus is in a textarea or control.
 - **Agent presence** - The browser shows when no agent is listening, keeps queued feedback for the next successful `lavish-axi poll` send even across reloads, and keeps human feedback actions available while the agent is working because the server queues them for the next poll. The agent's reply (`--agent-reply`) concludes delivered work and returns presence to waiting.
+  Closing the last review tab while a poll is active starts a 10-second reconnect grace period. A reload or quick reconnect keeps an active poll waiting; if every review stays disconnected, the poll returns `browser_disconnected` without ending the resumable session, and the agent asks whether to reopen or end it instead of acting uninvited.
   The no-timeout poll always writes an immediate stderr banner so it is visibly not hung; it adds the periodic stderr wait ticks only in an interactive terminal, so when stderr is piped (as under agent harnesses) the captured output carries no tick noise. Stdout always stays reserved for the final response; if the poll is interrupted or times out before feedback arrives, re-run it because feedback remains queued until delivery. Poll delivery consumes the response, so read the complete response before truncating or filtering it.
   Codex-specific guidance keeps that poll attached to the active turn instead of hiding it in a background task, because completed background tasks may not resume the agent.
 - **Session end etiquette** - Lavish tracks who ended a session: a human clicking **End session** (or **Send & end session**) in the browser is a user-initiated end, while `lavish-axi end <html-file>` is agent-initiated.
@@ -253,7 +254,7 @@ pnpm link
 | `lavish-axi`                    | Show current sessions and usage guidance.                                                                                                                                                                                                                                                                                                                 |
 | `lavish-axi update`             | Check for or apply the latest npm release through the AXI SDK self-updater.                                                                                                                                                                                                                                                                               |
 | `lavish-axi <html-file>`        | Open or resume a Lavish Editor session, with the open-time layout gate enabled by default. Unresolved layout issues from earlier in the session are preserved. Refuses to reopen a session the user explicitly ended from the browser unless `--reopen` is passed.                                                                                        |
-| `lavish-axi poll <html-file>`   | Long-poll until the user sends feedback or ends the session; detected layout issues wait in the user's Layout issues inbox and arrive only when queued. Leave no-timeout polls running, or re-run them if interrupted. Codex guidance keeps polls attached to the active turn. On `status: ended`, stop polling and do not reopen uninvited.              |
+| `lavish-axi poll <html-file>`   | Long-poll until the user sends feedback, ends the session, or leaves every review disconnected past the reconnect grace period; detected layout issues arrive only when queued. On `browser_disconnected`, ask before reopening or ending the still-resumable session. On `ended`, stop polling and do not reopen uninvited.                              |
 | `lavish-axi end <html-file>`    | End a session as the agent; unlike a user-initiated end from the browser, this still allows a plain reopen later.                                                                                                                                                                                                                                         |
 | `lavish-axi export <html-file>` | Write a portable copy of the artifact: one HTML file with its local assets inlined, so it opens with no server and no sibling files. Remote CDN/font references are left as links.                                                                                                                                                                        |
 | `lavish-axi share <html-file>`  | Publish the artifact (local assets inlined) to [ht-ml.app](https://ht-ml.app), a third-party host not part of Lavish, and print a visitable URL plus a secret update key; shares are public by default, `--private` locks one behind a generated password, and the same command republishes or unpublishes an existing page with `--site`/`--update-key`. |

package/dist/chrome-client.js

@@ -5,6 +5,7 @@ const sessionData = JSON.parse(sessionDataElement?.textContent || "{}");
 const key = String(sessionData.key || "");
 const filePath = String(sessionData.file || "");
 const queueStorageKey = "lavish-axi:queued:" + key;
+const terminalStorageKey = "lavish-axi:terminal:" + key;
 // Review-chrome state that must survive a browser refresh. Keyed per session so one review's
 // triage can never leak into another artifact's.
 const warningSelectionStorageKey = "lavish-axi:warning-selection:" + key;
@@ -183,7 +184,6 @@ const queued = loadQueuedPrompts();
 let annotation = true;
 let ended = false;
 let agentPresence = "waiting";
-let pendingSnapshot = "";
 const layoutGateEnabled = sessionData.layoutGateEnabled !== false;
 const configuredLayoutGateMaxHoldMs = Number(sessionData.layoutGateMaxHoldMs);
 const layoutGateMaxHoldMs =
@@ -211,11 +211,24 @@ let layoutGateTimer;
 let layoutWarnings = Array.isArray(sessionData.initialLayoutWarnings) ? sessionData.initialLayoutWarnings : [];
 const selectedWarningIds = new Set(loadJsonState(warningSelectionStorageKey, []));
 let warningsDrawerOpen = false;
-const snapshotRequests = [];
-let endAfterSubmit = false;
+/** @typedef {{ done: Promise<boolean>, finish: (succeeded: boolean) => void }} FeedbackPreparation */
+/** @typedef {{ prompts: any[], inFlight: boolean, order: number }} TerminalSubmission */
+/** @type {Map<string, { action: "copy" | "submit", prompts?: any[], endAfter?: boolean, terminal?: TerminalSubmission | null, acknowledgement?: object, order?: number, timeout?: ReturnType<typeof setTimeout> }>} */
+const snapshotRequests = new Map();
+let nextSnapshotRequestId = 0;
+let nextSendOperationOrder = 0;
 let workingBubble = null;
 let submitQueuedPromise = null;
-let submitQueuedAgain = false;
+const pendingSubmissions = [];
+const deliveredPrompts = new WeakSet();
+const pendingAcknowledgements = new Set();
+/** @type {Set<FeedbackPreparation>} */
+const feedbackPreparations = new Set();
+/** @type {TerminalSubmission | null} */
+let terminalSubmission =
+  loadJsonState(terminalStorageKey, false) === true
+    ? { prompts: queued.slice(), inFlight: false, order: ++nextSendOperationOrder }
+    : null;
 /** @type {ReturnType<typeof setTimeout> | undefined} */
 let sendAcknowledgementTimer;
 let lastScroll = { x: 0, y: 0 };
@@ -249,10 +262,17 @@ const HEALTH_PROBE_TIMEOUT_MS = 4000;
 // batch. Bound that whole wait so a missing SDK response or a browser/network stall can never look
 // like a dead button while the user's queue remains safely stored in this tab.
 const SEND_ACKNOWLEDGEMENT_WARNING_MS = 10_000;
+const TERMINAL_PREPARATION_TIMEOUT_MS = 5000;
+// A DOM snapshot adds useful context, but the reviewer's own words are the payload.
+// If the artifact frame navigated away from the injected SDK (or otherwise stopped
+// answering), deliver those words without a snapshot instead of waiting forever.
+const SNAPSHOT_REQUEST_TIMEOUT_MS = 5000;
 const SEND_STALLED_COPY =
   "Still trying to send. Your feedback is saved in this tab. Keep this tab open while Lavish catches up, and check that the server is running.";
 const SEND_FAILED_COPY =
   "Could not send. Your feedback is still queued in this tab. Check that Lavish is running, then click Send to Agent to retry.";
+const TERMINAL_SEND_FAILED_COPY =
+  "Could not send. Your terminal feedback is still queued in this tab. Check that Lavish is running, then click Send & End to retry the same batch.";
 const HEALTH_NO_ANSWER_TITLE = "Lavish did not answer.";
 const HEALTH_NO_ANSWER_COPY =
   "Lavish did not answer the check, so this page cannot tell whether it is running. Try again in a moment.";
@@ -274,6 +294,9 @@ let copyHintTimer;
 /** @type {ReturnType<typeof setTimeout> | undefined} */
 let sendHintTimer;
 let sendHintPersistent = false;
+/** @type {{ kind: "preparation" | "submission" | "terminal", operation: object, preparationType?: string } | null} */
+let sendFailureOwner = null;
+let sendAcknowledgementWarningVisible = false;
 
 function artifactFrameSrcForLoad(load) {
   const separator = artifactSrc.includes("?") ? "&" : "?";
@@ -380,6 +403,15 @@ function persistQueuedPrompts() {
   }
 }
 
+function persistTerminalReservation(reserved) {
+  try {
+    if (reserved) sessionStorage.setItem(terminalStorageKey, "true");
+    else sessionStorage.removeItem(terminalStorageKey);
+  } catch {
+    // Session storage can be unavailable; the in-memory reservation still protects this page.
+  }
+}
+
 function promptTargetLabel(prompt) {
   if (prompt?.target?.type === "table-cell") {
     const semantic = [prompt.target.rowLabel, prompt.target.columnLabel].filter(Boolean).join(" → ");
@@ -423,6 +455,7 @@ function render() {
 
   for (const button of annotationPills.querySelectorAll(".pill-close")) {
     const closeButton = /** @type {HTMLButtonElement} */ (button);
+    closeButton.disabled = terminalSubmission !== null;
     closeButton.addEventListener("click", (event) => removeQueuedPrompt(Number(closeButton.dataset.index), event));
   }
   updateSendState();
@@ -431,8 +464,17 @@ function render() {
 }
 
 function updateSendState() {
-  sendButton.disabled = ended;
-  sendAndEndButton.disabled = sendButton.disabled;
+  const terminalReserved = terminalSubmission !== null;
+  // A terminal send owns the exact review batch, so freeze interactions inside the
+  // artifact without disabling annotation mode. Disabling annotation mode closes the
+  // SDK card and destroys an unsent draft before delivery has actually succeeded.
+  frame.inert = ended || terminalReserved;
+  sendButton.disabled = ended || terminalReserved;
+  sendAndEndButton.disabled = ended || Boolean(terminalSubmission?.inFlight);
+  annotationSwitch.disabled = ended || terminalReserved;
+  chatInput.disabled = ended || terminalReserved;
+  chatAttachButton.disabled = ended || terminalReserved;
+  endButton.disabled = ended || terminalReserved;
   if (warningsQueueButton) updateWarningSelectionState();
 }
 
@@ -513,24 +555,49 @@ function hideSendHint(force = false) {
   sendHint.textContent = DEFAULT_SEND_HINT;
   sendHint.classList.remove("persistent");
   sendHintPersistent = false;
+  sendAcknowledgementWarningVisible = false;
 }
 
 function armSendAcknowledgementWarning() {
-  if (sendAcknowledgementTimer || !queued.length) return;
+  if (sendAcknowledgementTimer || pendingAcknowledgements.size === 0) return;
   sendAcknowledgementTimer = setTimeout(() => {
     sendAcknowledgementTimer = undefined;
-    if (queued.length) showSendHint(SEND_STALLED_COPY, null, false);
+    if (pendingAcknowledgements.size > 0 && !sendFailureOwner) {
+      sendAcknowledgementWarningVisible = true;
+      showSendHint(SEND_STALLED_COPY, null, false);
+    }
   }, SEND_ACKNOWLEDGEMENT_WARNING_MS);
 }
 
 function clearSendAcknowledgementWarning() {
   clearTimeout(sendAcknowledgementTimer);
   sendAcknowledgementTimer = undefined;
+  sendAcknowledgementWarningVisible = false;
+}
+
+function showQueuedSendFailure(message = SEND_FAILED_COPY, owner = null) {
+  showPersistentSendFailure(message, true, owner);
 }
 
-function showQueuedSendFailure(message = SEND_FAILED_COPY) {
+function showPersistentSendFailure(message, requireQueuedFeedback = false, owner = null) {
   clearSendAcknowledgementWarning();
-  if (queued.length) showSendHint(message, null, false);
+  if (requireQueuedFeedback && !queued.length) return;
+  if (sendFailureOwner?.kind === "preparation") return;
+  const currentOrder = Number(sendFailureOwner?.operation?.order) || 0;
+  const nextOrder = Number(owner?.operation?.order) || 0;
+  if (owner?.kind !== "preparation" && currentOrder > nextOrder) return;
+  sendFailureOwner = owner;
+  showSendHint(message, null, false);
+}
+
+function clearPersistentSendFailure() {
+  sendFailureOwner = null;
+  hideSendHint(true);
+}
+
+function clearPreparationFailure(preparationType) {
+  if (sendFailureOwner?.kind !== "preparation" || sendFailureOwner.preparationType !== preparationType) return;
+  clearPersistentSendFailure();
 }
 
 function setMenuOpen(button, menu, open) {
@@ -939,11 +1006,12 @@ function scrollElementIntoView(el) {
 
 function removeQueuedPrompt(index, event) {
   if (event) event.stopPropagation();
+  if (terminalSubmission) return;
   queued.splice(index, 1);
   persistQueuedPrompts();
   if (!queued.length) {
     clearSendAcknowledgementWarning();
-    hideSendHint(true);
+    if (sendFailureOwner?.kind !== "preparation") clearPersistentSendFailure();
   }
   render();
 }
@@ -952,9 +1020,28 @@ function promptQueueKey(prompt) {
   return prompt && typeof prompt[internalQueueKeyField] === "string" ? prompt[internalQueueKeyField].trim() : "";
 }
 
-function enqueuePrompt(rawPrompt) {
+function beginFeedbackPreparation() {
+  if (ended || terminalSubmission) return null;
+  /** @type {(succeeded: boolean) => void} */
+  let finishPromise = () => {};
+  const done = new Promise((resolve) => {
+    finishPromise = resolve;
+  });
+  const preparation = {
+    done,
+    finish(succeeded) {
+      if (!feedbackPreparations.delete(preparation)) return;
+      finishPromise(succeeded);
+    },
+  };
+  feedbackPreparations.add(preparation);
+  return preparation;
+}
+
+function enqueuePrompt(rawPrompt, /** @type {FeedbackPreparation | null} */ preparation = null) {
+  if ((preparation && !feedbackPreparations.has(preparation)) || (terminalSubmission && !preparation)) return false;
   const prompt = sanitizeQueuedPrompt(rawPrompt);
-  if (!prompt) return;
+  if (!prompt) return false;
 
   const queueKey = promptQueueKey(prompt);
   if (queueKey) {
@@ -970,6 +1057,7 @@ function enqueuePrompt(rawPrompt) {
 
   persistQueuedPrompts();
   render();
+  return true;
 }
 
 function stripInternalPromptFields(prompt) {
@@ -983,9 +1071,46 @@ function postToFrame(message) {
   if (frame.contentWindow) frame.contentWindow.postMessage(message, "*");
 }
 
-function requestSnapshot(action) {
-  snapshotRequests.push(action);
-  postToFrame({ type: "lavish:requestSnapshot" });
+function requestSnapshot(action, prompts = [], endAfter = false, terminal = null) {
+  const requestId = "snapshot-" + ++nextSnapshotRequestId;
+  const request =
+    action === "submit"
+      ? { action, prompts, endAfter, terminal, order: terminal?.order || ++nextSendOperationOrder }
+      : { action };
+  snapshotRequests.set(requestId, request);
+  if (action === "submit") {
+    request.acknowledgement = {};
+    pendingAcknowledgements.add(request.acknowledgement);
+    armSendAcknowledgementWarning();
+    request.timeout = setTimeout(() => completeSnapshotRequest(requestId, ""), SNAPSHOT_REQUEST_TIMEOUT_MS);
+  }
+  postToFrame({ type: "lavish:requestSnapshot", snapshot_request_id: requestId });
+}
+
+function takeSnapshotRequest(requestId) {
+  if (typeof requestId !== "string" || !requestId || !snapshotRequests.has(requestId)) return null;
+  const request = snapshotRequests.get(requestId);
+  snapshotRequests.delete(requestId);
+  if (request?.timeout) clearTimeout(request.timeout);
+  return request || null;
+}
+
+function completeSnapshotRequest(requestId, snapshot) {
+  const request = takeSnapshotRequest(requestId);
+  if (!request) return;
+  if (request.action === "copy") {
+    copyText(snapshot || "");
+    return;
+  }
+
+  submitQueued({
+    prompts: request.prompts || [],
+    domSnapshot: snapshot || "",
+    endAfter: request.endAfter === true,
+    terminal: request.terminal || null,
+    acknowledgement: request.acknowledgement || null,
+    order: request.order || 0,
+  }).catch(() => {});
 }
 
 function createChatAttachmentsController() {
@@ -1188,6 +1313,10 @@ const chatAttachmentController = createChatAttachmentsController();
 
 function sendQueued(endAfter) {
   if (ended) return;
+  if (terminalSubmission) {
+    if (endAfter && !terminalSubmission.inFlight) retryTerminalSubmission();
+    return;
+  }
   closeMenus();
 
   // A pending or failed chip holds back only the COMPOSER message (and an
@@ -1214,50 +1343,138 @@ function sendQueued(endAfter) {
       chatAttachmentController.reset();
     }
   }
-  if (!queued.length) {
-    if (!chipsBlocked) showSendHint();
+  const shouldEnd = Boolean(endAfter && !chipsBlocked);
+  const preparations = shouldEnd ? [...feedbackPreparations] : [];
+  if (!queued.length && preparations.length === 0) {
+    if (!chipsBlocked && !sendFailureOwner) showSendHint();
     return;
   }
-  hideSendHint(true);
-  armSendAcknowledgementWarning();
+  if (!sendFailureOwner) hideSendHint(true);
+  if (shouldEnd) {
+    const terminal = {
+      prompts: [],
+      inFlight: true,
+      order: ++nextSendOperationOrder,
+    };
+    terminalSubmission = terminal;
+    updateSendState();
+    finishTerminalPreparation(terminal, preparations);
+    return;
+  }
+  requestSnapshot("submit", queued.slice(), false, null);
+}
+
+function finishTerminalPreparation(terminal, preparations) {
+  if (preparations.length === 0) {
+    completeTerminalPreparation(terminal, []);
+    return;
+  }
+  const timeout = setTimeout(() => {
+    if (terminalSubmission !== terminal || ended) return;
+    for (const preparation of preparations) preparation.finish(false);
+    showPersistentSendFailure(
+      "Could not finish preparing all feedback within 5 seconds. This review remains open, and existing queued feedback is still editable.",
+      false,
+      { kind: "preparation", operation: terminal, preparationType: "terminal" },
+    );
+    releaseTerminalSubmission(terminal);
+  }, TERMINAL_PREPARATION_TIMEOUT_MS);
+  Promise.all(preparations.map((preparation) => preparation.done)).then((results) => {
+    clearTimeout(timeout);
+    completeTerminalPreparation(terminal, results);
+  });
+}
+
+function completeTerminalPreparation(terminal, results) {
+  if (terminalSubmission !== terminal || ended) return;
+  if (results.some((succeeded) => !succeeded)) {
+    releaseTerminalSubmission(terminal);
+    return;
+  }
+  terminal.prompts = queued.slice();
+  clearPreparationFailure("terminal");
+  if (!terminal.prompts.length) {
+    releaseTerminalSubmission(terminal);
+    if (!sendFailureOwner) showSendHint();
+    return;
+  }
+  // Only make the reservation durable once every preparation has joined the exact
+  // batch. A reload before this point must restore an ordinary editable queue, not
+  // an incomplete terminal submission.
+  persistTerminalReservation(true);
+  requestSnapshot("submit", terminal.prompts, true, terminal);
+}
+
+function retryTerminalSubmission() {
+  if (!terminalSubmission || terminalSubmission.inFlight || ended) return;
+  terminalSubmission.inFlight = true;
+  updateSendState();
+  requestSnapshot("submit", terminalSubmission.prompts, true, terminalSubmission);
+}
 
-  if (endAfter && !chipsBlocked) endAfterSubmit = true;
-  requestSnapshot("submit");
+function markTerminalSubmissionFailed(submission) {
+  if (!terminalSubmission || submission.terminal !== terminalSubmission || ended) return;
+  terminalSubmission.inFlight = false;
+  updateSendState();
+}
+
+function releaseTerminalSubmission(terminal) {
+  if (terminalSubmission !== terminal || ended) return;
+  terminalSubmission = null;
+  persistTerminalReservation(false);
+  render();
 }
 
-async function submitQueued() {
+async function submitQueued(submission) {
+  pendingSubmissions.push(submission);
   if (submitQueuedPromise) {
-    submitQueuedAgain = true;
     return submitQueuedPromise;
   }
 
-  let succeeded = false;
-  submitQueuedPromise = submitQueuedOnce();
+  submitQueuedPromise = (async () => {
+    /** @type {unknown} */
+    let firstError = null;
+    while (pendingSubmissions.length && !ended) {
+      const next = pendingSubmissions.shift();
+      if (!next) continue;
+      try {
+        const result = await submitQueuedOnce(next, firstError !== null);
+        if (result === false) markTerminalSubmissionFailed(next);
+      } catch (error) {
+        markTerminalSubmissionFailed(next);
+        if (firstError === null) firstError = error;
+      } finally {
+        if (next.acknowledgement) pendingAcknowledgements.delete(next.acknowledgement);
+      }
+    }
+    if (firstError !== null) throw firstError;
+  })();
   try {
-    const result = await submitQueuedPromise;
-    succeeded = result !== false;
-    return result;
+    return await submitQueuedPromise;
   } finally {
     submitQueuedPromise = null;
-    const shouldSubmitAgain = submitQueuedAgain;
-    submitQueuedAgain = false;
-    if (!succeeded) {
-      endAfterSubmit = false;
-    } else if (!ended && shouldSubmitAgain) {
-      if (queued.length) {
-        submitQueued().catch(() => {});
-      } else if (endAfterSubmit) {
-        endAfterSubmit = false;
-        endSession();
-      }
-    }
   }
 }
 
-async function submitQueuedOnce() {
-  const prompts = queued.slice();
-  const shouldEndSession = endAfterSubmit;
-  const body = { prompts: prompts.map(stripInternalPromptFields), domSnapshot: pendingSnapshot };
+async function submitQueuedOnce(submission, preserveFailureState = false) {
+  const prompts = submission.prompts.filter((prompt) => !deliveredPrompts.has(prompt));
+  const shouldEndSession = submission.endAfter;
+  if (!prompts.length) {
+    if (shouldEndSession && !ended) {
+      try {
+        await endSession(submission.terminal);
+      } catch (error) {
+        showPersistentSendFailure(TERMINAL_SEND_FAILED_COPY, false, {
+          kind: "terminal",
+          operation: submission.terminal,
+        });
+        throw error;
+      }
+    }
+    settleAcknowledgementGuidance(submission, preserveFailureState);
+    return;
+  }
+  const body = { prompts: prompts.map(stripInternalPromptFields), domSnapshot: submission.domSnapshot };
   if (shouldEndSession) body.endSession = true;
   let response;
   try {
@@ -1266,11 +1483,33 @@ async function submitQueuedOnce() {
       headers: { "content-type": "application/json" },
       body: JSON.stringify(body),
     });
+    // The DOM snapshot is optional context and can be the difference between a
+    // useful terminal batch and Express' request-size limit. Retry exactly once
+    // without it; never mutate or split the user's exact feedback batch.
+    if (response.status === 413 && body.domSnapshot) {
+      body.domSnapshot = "";
+      response = await fetch("/api/" + key + "/prompts", {
+        method: "POST",
+        headers: { "content-type": "application/json" },
+        body: JSON.stringify(body),
+      });
+    }
   } catch (error) {
-    showQueuedSendFailure();
+    showQueuedSendFailure(submission.terminal ? TERMINAL_SEND_FAILED_COPY : SEND_FAILED_COPY, {
+      kind: submission.terminal ? "terminal" : "submission",
+      operation: submission.terminal || submission,
+    });
     throw error;
   }
   if (!response.ok) {
+    if (response.status === 413) {
+      showQueuedSendFailure(
+        "Could not send because this feedback is too large. It is still queued. Remove some feedback or attachments, then try again.",
+        { kind: "submission", operation: submission },
+      );
+      if (submission.terminal) releaseTerminalSubmission(submission.terminal);
+      throw new Error("queued feedback exceeds the request size limit");
+    }
     if (response.status === 409) {
       const data = await response.json().catch(() => null);
       // The session already ended before this batch arrived - most likely this chrome missed the
@@ -1278,16 +1517,16 @@ async function submitQueuedOnce() {
       // for another attempt that will be refused the same way.
       if (data?.status === "ended") {
         clearSendAcknowledgementWarning();
-        endAfterSubmit = false;
         markSessionEnded();
         return false;
       }
       if (Array.isArray(data?.warnings)) setLayoutWarnings(data.warnings);
-      endAfterSubmit = false;
       showQueuedSendFailure(
         "Could not send because the layout issue selection changed. Your feedback is still queued. Review the current issues, then click Send to Agent to retry.",
+        { kind: "submission", operation: submission },
       );
-      return false;
+      if (submission.terminal) releaseTerminalSubmission(submission.terminal);
+      return;
     }
     // C4: the server persisted nothing (atomic reject) - the queue below is left
     // intact because the splice only runs on success. Surface exactly what failed
@@ -1295,31 +1534,64 @@ async function submitQueuedOnce() {
     if (response.status === 400) {
       const detail = await response.json().catch(() => ({}));
       if (Array.isArray(detail.rejected) && detail.rejected.length) {
-        showQueuedSendFailure(describeAttachmentRejection(detail.rejected, detail.caps));
+        showQueuedSendFailure(describeAttachmentRejection(detail.rejected, detail.caps), {
+          kind: "submission",
+          operation: submission,
+        });
+        if (submission.terminal) releaseTerminalSubmission(submission.terminal);
       } else {
-        showQueuedSendFailure();
+        showQueuedSendFailure(submission.terminal ? TERMINAL_SEND_FAILED_COPY : SEND_FAILED_COPY, {
+          kind: submission.terminal ? "terminal" : "submission",
+          operation: submission.terminal || submission,
+        });
       }
     } else {
-      showQueuedSendFailure();
+      showQueuedSendFailure(submission.terminal ? TERMINAL_SEND_FAILED_COPY : SEND_FAILED_COPY, {
+        kind: submission.terminal ? "terminal" : "submission",
+        operation: submission.terminal || submission,
+      });
     }
     throw new Error("failed to submit queued prompts");
   }
   for (const prompt of prompts) {
+    deliveredPrompts.add(prompt);
     const index = queued.indexOf(prompt);
     if (index !== -1) queued.splice(index, 1);
   }
   persistQueuedPrompts();
   render();
-  clearSendAcknowledgementWarning();
-  hideSendHint(true);
-  if (queued.length) armSendAcknowledgementWarning();
+  settleAcknowledgementGuidance(submission, preserveFailureState);
   if (shouldEndSession) {
-    endAfterSubmit = false;
     markSessionEnded();
     return;
   }
 }
 
+function submissionResolvesSendFailure(submission) {
+  if (!sendFailureOwner) return true;
+  if (sendFailureOwner.kind === "preparation") return false;
+  if (sendFailureOwner.kind === "terminal") return sendFailureOwner.operation === submission.terminal;
+  const failedSubmission = sendFailureOwner.operation;
+  return (
+    failedSubmission === submission ||
+    (Array.isArray(failedSubmission.prompts) &&
+      failedSubmission.prompts.every((prompt) => deliveredPrompts.has(prompt)))
+  );
+}
+
+function settleAcknowledgementGuidance(submission, preserveFailureState) {
+  if (!submissionResolvesSendFailure(submission) || (preserveFailureState && queued.length)) return;
+  const hasLaterAcknowledgement = [...pendingAcknowledgements].some(
+    (acknowledgement) => acknowledgement !== submission.acknowledgement,
+  );
+  if (hasLaterAcknowledgement) {
+    if (!sendAcknowledgementTimer && !sendAcknowledgementWarningVisible) armSendAcknowledgementWarning();
+    return;
+  }
+  clearSendAcknowledgementWarning();
+  clearPersistentSendFailure();
+}
+
 function normalizeLayoutFindings(value) {
   return Array.isArray(value)
     ? value.filter((item) => item && typeof item === "object" && String(item.severity || "").toLowerCase() === "error")
@@ -1758,7 +2030,7 @@ function updateWarningSelectionState() {
   warningsSelectAll.checked = selectable.length > 0 && selectedCount === selectable.length;
   warningsSelectAll.indeterminate = selectedCount > 0 && selectedCount < selectable.length;
   warningsSelected.textContent = selectedCount === 0 ? "None selected" : selectedCount + " selected";
-  warningsQueueButton.disabled = selectedCount === 0 || ended;
+  warningsQueueButton.disabled = selectedCount === 0 || ended || terminalSubmission !== null;
 }
 
 function toggleSelectAllWarnings() {
@@ -1815,10 +2087,15 @@ async function dismissWarning(id) {
 // One queued batch = one ordinary queued prompt. The CLI cannot tell it apart from any other
 // feedback, which is exactly the point: no parallel agent protocol.
 async function queueSelectedWarningFixes() {
-  if (ended) return;
+  const preparation = beginFeedbackPreparation();
+  if (!preparation) return;
   const ids = [...selectedWarningIds];
-  if (ids.length === 0) return;
+  if (ids.length === 0) {
+    preparation.finish(true);
+    return;
+  }
   warningsQueueButton.disabled = true;
+  let succeeded = false;
   try {
     const response = await fetch("/api/" + key + "/layout-warnings/queue", {
       method: "POST",
@@ -1828,21 +2105,36 @@ async function queueSelectedWarningFixes() {
     if (!response.ok) throw new Error("failed to queue layout warning fixes");
     const data = await response.json();
     if (data.prompt) {
-      enqueuePrompt({
-        uid: "",
-        prompt: data.prompt.prompt,
-        selector: "",
-        tag: "layout-warnings",
-        text: data.prompt.text,
-        target: data.prompt.target,
-      });
+      if (
+        !enqueuePrompt(
+          {
+            uid: "",
+            prompt: data.prompt.prompt,
+            selector: "",
+            tag: "layout-warnings",
+            text: data.prompt.text,
+            target: data.prompt.target,
+          },
+          preparation,
+        )
+      )
+        throw new Error("failed to retain layout warning fixes");
     }
     selectedWarningIds.clear();
     persistWarningSelection();
     if (Array.isArray(data.warnings)) setLayoutWarnings(data.warnings);
     closeWarningsDrawer({ restoreFocus: true });
+    clearPreparationFailure("layout-warnings");
+    succeeded = true;
   } catch {
+    showPersistentSendFailure(
+      "Could not prepare the selected layout fixes. Review the current issues and try again.",
+      false,
+      { kind: "preparation", operation: preparation, preparationType: "layout-warnings" },
+    );
     updateWarningSelectionState();
+  } finally {
+    preparation.finish(succeeded);
   }
 }
 
@@ -1857,8 +2149,8 @@ async function refreshLayoutWarnings() {
   }
 }
 
-async function endSession() {
-  if (ended) return;
+async function endSession(terminal = null) {
+  if (ended || (terminalSubmission && terminal !== terminalSubmission)) return;
   const response = await fetch("/api/" + key + "/end", { method: "POST" });
   if (!response.ok) throw new Error("failed to end session");
   markSessionEnded();
@@ -1867,6 +2159,10 @@ async function endSession() {
 function markSessionEnded() {
   if (ended) return;
   ended = true;
+  pendingAcknowledgements.clear();
+  clearSendAcknowledgementWarning();
+  terminalSubmission = null;
+  persistTerminalReservation(false);
   cancelArtifactLoadRecovery();
   closeMenus();
   closeShareDialog();
@@ -2619,7 +2915,17 @@ function whiteboardSummaryText(summaryLines) {
 }
 
 async function queueWhiteboardFeedback(index, message, mode) {
+  const preparation = beginFeedbackPreparation();
+  if (!preparation) {
+    postToWhiteboard(index, mode, {
+      type: "lavish-whiteboard:queueResult",
+      ok: false,
+      error: "Feedback delivery is already ending this review.",
+    });
+    return;
+  }
   const diagramId = whiteboardRecord(index).diagramId;
+  let succeeded = false;
   try {
     // Persist the exact reviewed state before queueing, so the paths in the
     // prompt point at what the user actually saw.
@@ -2643,36 +2949,46 @@ async function queueWhiteboardFeedback(index, message, mode) {
       "\n\nEdited scene JSON: " +
       String(files.scene_path || "") +
       (files.preview_path ? "\nPNG preview: " + String(files.preview_path) : "");
-    enqueuePrompt({
-      uid: "",
-      prompt: promptText,
-      selector: "",
-      tag: "whiteboard",
-      text: "Whiteboard: diagram " + (index + 1),
-      target: {
-        type: "excalidraw-scene",
-        diagramIndex: index,
-        diagramId,
-        sourceHash: String(message.sourceHash || ""),
-        scenePath: String(files.scene_path || ""),
-        previewPath: String(files.preview_path || ""),
-        imageFallback: Boolean(message.imageFallback),
-        stats: message.stats && typeof message.stats === "object" ? message.stats : {},
-      },
-      // Re-queueing the same diagram's whiteboard before sending replaces the
-      // earlier unsent prompt instead of stacking duplicates.
-      [internalQueueKeyField]: "whiteboard:" + index,
-    });
+    if (
+      !enqueuePrompt(
+        {
+          uid: "",
+          prompt: promptText,
+          selector: "",
+          tag: "whiteboard",
+          text: "Whiteboard: diagram " + (index + 1),
+          target: {
+            type: "excalidraw-scene",
+            diagramIndex: index,
+            diagramId,
+            sourceHash: String(message.sourceHash || ""),
+            scenePath: String(files.scene_path || ""),
+            previewPath: String(files.preview_path || ""),
+            imageFallback: Boolean(message.imageFallback),
+            stats: message.stats && typeof message.stats === "object" ? message.stats : {},
+          },
+          // Re-queueing the same diagram's whiteboard before sending replaces the
+          // earlier unsent prompt instead of stacking duplicates.
+          [internalQueueKeyField]: "whiteboard:" + index,
+        },
+        preparation,
+      )
+    )
+      throw new Error("failed to retain whiteboard feedback");
     // Queued from the whiteboard inside the artifact, like any other in-artifact prompt.
     pulseSheetDock();
     postToWhiteboard(index, mode, { type: "lavish-whiteboard:queueResult", ok: true });
     if (mode === "overlay") closeWhiteboard();
+    clearPreparationFailure("whiteboard");
+    succeeded = true;
   } catch (error) {
     postToWhiteboard(index, mode, {
       type: "lavish-whiteboard:queueResult",
       ok: false,
       error: error instanceof Error ? error.message : String(error),
     });
+  } finally {
+    preparation.finish(succeeded);
   }
 }
 
@@ -2987,16 +3303,7 @@ window.addEventListener("message", (event) => {
     pulseSheetDock();
   }
   if (msg.type === "lavish:snapshot") {
-    const snapshotAction = snapshotRequests.shift() || "submit";
-    if (snapshotAction === "copy") {
-      copyText(msg.snapshot || "");
-    } else {
-      pendingSnapshot = msg.snapshot || "";
-      // submitQueuedOnce throws to signal "nothing was delivered" - its own
-      // finally already reset the end intent and it left the queue intact for a
-      // retry, so the rejection has no remaining consumer here.
-      submitQueued().catch(() => {});
-    }
+    completeSnapshotRequest(msg.snapshot_request_id, msg.snapshot || "");
   }
   if (msg.type === "lavish:scroll") {
     lastScroll = { x: Number(msg.x) || 0, y: Number(msg.y) || 0 };
@@ -3150,7 +3457,7 @@ async function uploadAttachment(message, reportResult = postToFrame, signal) {
 loadFrame();
 
 function toggleAnnotationMode() {
-  if (ended) return;
+  if (ended || terminalSubmission) return;
   annotation = !annotation;
   annotationSwitch.setAttribute("aria-pressed", String(annotation));
   postToFrame({ type: "lavish:setAnnotationMode", enabled: annotation });

package/dist/cli.mjs

@@ -6689,7 +6689,10 @@ function createArtifactSdk(deriveQueueKey, isNativeInteractive = isNativeInterac
       activeAttachments?.handleResult(msg.localId, msg.ok, msg.id, msg.error);
     }
     if (msg.type === "lavish:requestSnapshot") {
-      postArtifactMessage("lavish:snapshot", { snapshot: snapshot() });
+      postArtifactMessage("lavish:snapshot", {
+        snapshot: snapshot(),
+        snapshot_request_id: typeof msg.snapshot_request_id === "string" ? msg.snapshot_request_id : ""
+      });
     }
     if (msg.type === "lavish:restoreScroll") {
       window.scrollTo(Number(msg.x) || 0, Number(msg.y) || 0);
@@ -8813,6 +8816,7 @@ var WHITEBOARD_CHANNEL_TOKEN_TTL_MS = 5 * 6e4;
 var NETWORK_RECONCILE_CACHE_MS = 1e3;
 var TAILSCALE_BIND_RETRY_DELAYS_MS = [100, 250, 500];
 var WEBSOCKET_CLOSE_GRACE_MS = 250;
+var BROWSER_DISCONNECT_GRACE_MS = 1e4;
 var ARTIFACT_CONTENT_SECURITY_POLICY = "sandbox allow-scripts allow-forms allow-popups allow-popups-to-escape-sandbox allow-downloads";
 var ATTACHMENT_SWEEP_INTERVAL_MS = 60 * 6e4;
 var SHUTDOWN_REASONS = /* @__PURE__ */ new Set(["upgrade", "local-build", "stop"]);
@@ -8910,6 +8914,7 @@ async function serve({
   debug = false,
   log = null,
   pollHeartbeatMs = 15e3,
+  browserDisconnectGraceMs = BROWSER_DISCONNECT_GRACE_MS,
   idleTimeoutMs = resolveIdleTimeoutMs(),
   host = bindHost(env),
   hosts,
@@ -8942,6 +8947,7 @@ async function serve({
   const activePolls = /* @__PURE__ */ new Map();
   const deliveredFeedback = /* @__PURE__ */ new Set();
   const liveEventClients = /* @__PURE__ */ new Map();
+  const browserDisconnectTimers = /* @__PURE__ */ new Map();
   const whiteboardChannelSecret = crypto6.randomBytes(32);
   const outstandingRepairBatches = /* @__PURE__ */ new Set();
   const diagnosticViewportClasses = resolveDiagnosticViewportClasses();
@@ -8954,6 +8960,7 @@ async function serve({
   let serverReady = false;
   let networkReconcileCheckedAt = 0;
   let cachedNetworkStale = false;
+  let shuttingDown = false;
   let networkReconcilePromise = null;
   function broadcastLiveEvent(type, key, data = {}) {
     for (const [client, clientKey] of liveEventClients) {
@@ -8965,7 +8972,30 @@ async function serve({
   events.on("agent-presence", (key, state) => broadcastLiveEvent("agent-presence", key, { state }));
   events.on("layout-warnings", (key, warnings) => broadcastLiveEvent("layout-warnings", key, { warnings }));
   events.on("ended", (key, endedBy) => broadcastLiveEvent("ended", key, { ended_by: endedBy || null }));
+  function hasLiveEventClient(key) {
+    for (const clientKey of liveEventClients.values()) {
+      if (clientKey === key) return true;
+    }
+    return false;
+  }
+  function clearBrowserDisconnectTimer(key) {
+    const timer = browserDisconnectTimers.get(key);
+    if (!timer) return;
+    clearTimeout(timer);
+    browserDisconnectTimers.delete(key);
+  }
+  function scheduleBrowserDisconnect(key) {
+    clearBrowserDisconnectTimer(key);
+    if (shuttingDown || hasLiveEventClient(key) || !activePolls.has(key)) return;
+    const timer = setTimeout(() => {
+      browserDisconnectTimers.delete(key);
+      if (!hasLiveEventClient(key) && activePolls.has(key)) events.emit("browser-disconnected", key);
+    }, browserDisconnectGraceMs);
+    timer.unref?.();
+    browserDisconnectTimers.set(key, timer);
+  }
   function attachLiveEventClient(client, key, onClose) {
+    clearBrowserDisconnectTimer(key);
     liveEventClients.set(client, key);
     refreshIdleTimer();
     let cleanedUp = false;
@@ -8973,6 +9003,7 @@ async function serve({
       if (cleanedUp) return;
       cleanedUp = true;
       liveEventClients.delete(client);
+      scheduleBrowserDisconnect(key);
       refreshIdleTimer();
     };
     onClose(cleanup);
@@ -9252,25 +9283,28 @@ async function serve({
         if (heartbeat) clearInterval(heartbeat);
         events.off("feedback", onFeedback);
         events.off("ended", onFeedback);
+        events.off("browser-disconnected", onBrowserDisconnected);
         setPollActive(key, activePolls, deliveredFeedback, events, false);
+        if (!activePolls.has(key)) clearBrowserDisconnectTimer(key);
         refreshIdleTimer();
         cleanupPoll = null;
         detachRequestClose();
       };
-      const respond = async () => {
+      const respond = async (forcedResult = null) => {
         if (responding || res.writableEnded) return;
         responding = true;
         try {
           const result = await store.takeFeedback(key);
+          const responseResult = forcedResult && result.status === "waiting" ? forcedResult : result;
           if (requestClosed || req.destroyed || res.writableEnded) {
-            await restoreClosedFeedback(key, result);
+            await restoreClosedFeedback(key, responseResult);
             return;
           }
-          finishFeedbackDelivery(key, result);
+          finishFeedbackDelivery(key, responseResult);
           if (streamHeartbeat) {
-            res.end(JSON.stringify(result));
+            res.end(JSON.stringify(responseResult));
           } else {
-            res.json(result);
+            res.json(responseResult);
           }
         } finally {
           cleanup();
@@ -9282,8 +9316,13 @@ async function serve({
         }
         respond().catch(handleRespondError);
       };
+      const onBrowserDisconnected = (changedKey) => {
+        if (changedKey !== key || res.writableEnded) return;
+        respond({ status: "browser_disconnected" }).catch(handleRespondError);
+      };
       events.on("feedback", onFeedback);
       events.on("ended", onFeedback);
+      events.on("browser-disconnected", onBrowserDisconnected);
       cleanupPoll = cleanup;
       if (requestClosed || req.destroyed || res.writableEnded) {
         cleanup();
@@ -10045,7 +10084,6 @@ ${body}`
   }
   publicPort = httpServers[0].address().port;
   serverReady = true;
-  let shuttingDown = false;
   function shutdown(reloadKey = "", reason = "") {
     if (shuttingDown) return;
     shuttingDown = true;
@@ -10070,6 +10108,8 @@ ${body}`
       }
     }
     liveEventClients.clear();
+    for (const timer of browserDisconnectTimers.values()) clearTimeout(timer);
+    browserDisconnectTimers.clear();
     for (const w of watchers.values()) {
       w.close().catch(() => {
       });
@@ -10912,11 +10952,12 @@ var POLL_WAKE_PATH_RULES = Object.freeze([
   "Never use `nohup`, shell `&`, `disown`, redirected fire-and-forget processes, or a detached terminal without an explicit verified callback merely to keep polling alive.",
   "If the harness has no completion-aware background facility, use the foreground poll or first wire a verified wake callback into the surrounding supervisor.",
   "Do not tell the user the artifact is being monitored until that wake path is live.",
-  "If the poll gets killed or times out before feedback arrives, re-run it - feedback remains queued until delivery. Poll delivery consumes the response, so read it completely."
+  "If the poll gets killed or times out before feedback arrives, re-run it - feedback remains queued until delivery. Poll delivery consumes the response, so read it completely.",
+  "If it returns browser_disconnected, the review window stayed disconnected past its reconnect grace period but the session remains resumable; ask the user whether to reopen or end it, and do neither uninvited."
 ]);
 var POLL_SEND_AND_END_RULE = "`Send & End` ends the session. Its final feedback is still delivered once. After that response, polling stops, and the agent must not reopen the session uninvited.";
 var CODEX_POLL_WAKE_PATH_GUIDANCE = "Codex detected: completed background tasks may not resume Codex automatically, so keep the poll attached to the active turn.";
-var VERSION = "0.1.66";
+var VERSION = "0.1.68";
 function detectInvokingAgent(env = process.env) {
   return ["CODEX_SANDBOX", "CODEX_THREAD_ID"].some((key) => Object.hasOwn(env, key)) ? "codex" : "generic";
 }
@@ -11032,7 +11073,7 @@ function createHomeOutput({ bin, sessions, includeSessions = true, agent = "gene
       "Run `lavish-axi <html-file>` to open or resume a Lavish Editor session. If the user explicitly ended the session from the browser, this refuses to reopen it and explains why instead of reopening uninvited - pass `--reopen` only when the user asks for further review or something important needs their visual attention",
       "Unless the user specifies another location, create HTML artifacts in the current working directory under `.lavish/`",
       "Lavish serves the html file through a local express.js server. If your html needs to reference other filesystem assets such as images, CSS, fonts, and local scripts, copy them into the same directory as the HTML file, then reference them with relative paths from that directory. Never prepend `/` to those asset paths - root paths won't work",
-      `Run \`lavish-axi poll <html-file>\` to wait for user feedback. It long-polls and stays silent until the user sends feedback or ends the session, so leave it running - never kill it. Detected layout issues never return this poll: the browser files them in the user's Layout issues inbox in the Lavish top bar, and they arrive as an ordinary tag "layout-warnings" prompt only when the user selects them and queues the fixes. Never edit the artifact to chase a layout issue the user has not queued. The only exception is a fatal artifact_failures response, which means the review surface itself could not be used. ${pollExecutionGuidance({ agent })} ${POLL_SEND_AND_END_RULE}`,
+      `Run \`lavish-axi poll <html-file>\` to wait for user feedback. It long-polls and stays silent until the user sends feedback, ends the session, or leaves every review window disconnected past the reconnect grace period, so leave it running - never kill it. Detected layout issues never return this poll: the browser files them in the user's Layout issues inbox in the Lavish top bar, and they arrive as an ordinary tag "layout-warnings" prompt only when the user selects them and queues the fixes. Never edit the artifact to chase a layout issue the user has not queued. The only exception is a fatal artifact_failures response, which means the review surface itself could not be used. ${pollExecutionGuidance({ agent })} ${POLL_SEND_AND_END_RULE}`,
       'Mermaid is the whiteboard opt-in, not the diagram default: only when the user asks for an editable whiteboard, author that diagram as Mermaid in a `.mermaid` container. Rendered Mermaid diagrams there become embedded, editable Excalidraw whiteboards in the browser (click a diagram to unlock editing; a Fullscreen action opens it over the whole viewport) - flowchart, sequence, class, ER, and state diagrams convert to editable shapes; other types embed as an image to draw on. Scenes autosave locally; an unmodified autosave silently re-converts when a reload changes the Mermaid source. If the reviewer edited the scene, they choose to re-convert and discard saved edits or keep editing the saved scene. Standalone and exported copies still render plain Mermaid. Queue feedback adds a prompt to the Conversation panel; when the user sends it, poll returns a tag "whiteboard" prompt carrying a bounded edit summary plus local scenePath (.excalidraw JSON) and previewPath (PNG) files - read the summary first, open the files only when needed, then apply the edits by updating the Mermaid source in the artifact (never try to write the scene back)',
       "Run `lavish-axi end <html-file>` to end a session as the agent - ending it this way still allows a plain reopen later. When the user ends it from the browser instead, a later `lavish-axi <html-file>` refuses to reopen it without `--reopen`",
       "Run `lavish-axi export <html-file> [--out <path>]` to write a portable copy of the artifact - one HTML file with its LOCAL assets inlined - so it opens with no Lavish server and no sibling files. Remote CDN/font references are left as links, so it needs network to render those. Users can also export from the browser chrome's overflow menu",
@@ -11073,7 +11114,7 @@ function createOpenOutput({
     session: { file, url, status },
     ...networkWarning ? { network_warning: networkWarning } : {},
     ...selfPaintWarning ? { self_paint_warning: selfPaintWarning } : {},
-    next_step: `${selfPaintPrefix}Do not respond to the user just yet. Now you must run \`lavish-axi poll ${file}\`. This command long-polls until the user sends feedback or ends the session, and it stays silent the whole time - that is normal, never kill it. Layout issues the browser detects do not return this poll; they wait in the user's Layout issues inbox until the user queues them, then arrive as an ordinary tag "layout-warnings" prompt. Do not pass --timeout-ms during normal agent use. ${pollExecutionGuidance({ agent })} After applying feedback, run \`lavish-axi poll ${file} --agent-reply "<message for the user>"\` without --timeout-ms to show your response in Lavish Editor and wait for more feedback. If the user ends the session, stop polling and do not reopen it by re-running \`lavish-axi ${file}\` unless the user asks for further review or something genuinely important needs their visual attention - deliver routine updates directly in this conversation instead. When reopening is warranted, run \`lavish-axi ${file} --reopen\`.`
+    next_step: `${selfPaintPrefix}Do not respond to the user just yet. Now you must run \`lavish-axi poll ${file}\`. This command long-polls until the user sends feedback, ends the session, or leaves every review window disconnected past the reconnect grace period, and it stays silent the whole time - that is normal, never kill it. Layout issues the browser detects do not return this poll; they wait in the user's Layout issues inbox until the user queues them, then arrive as an ordinary tag "layout-warnings" prompt. Do not pass --timeout-ms during normal agent use. ${pollExecutionGuidance({ agent })} After applying feedback, run \`lavish-axi poll ${file} --agent-reply "<message for the user>"\` without --timeout-ms to show your response in Lavish Editor and wait for more feedback. If the user ends the session, stop polling and do not reopen it by re-running \`lavish-axi ${file}\` unless the user asks for further review or something genuinely important needs their visual attention - deliver routine updates directly in this conversation instead. When reopening is warranted, run \`lavish-axi ${file} --reopen\`.`
   };
 }
 function createUserEndedOpenOutput({ file, url, networkWarning = void 0 }) {
@@ -11174,11 +11215,11 @@ ${pollInterruptedText(absolute)}
   }
 }
 function pollWaitBannerText(file) {
-  return `[lavish-axi] Long-polling for user feedback on ${file}. This stays silent until the user sends feedback or ends the session - leave it running. Detected layout issues do NOT return this poll: they wait in the user's Layout issues inbox until the user queues them as ordinary feedback. If it gets killed or times out before feedback arrives, re-run \`lavish-axi poll ${file}\` - feedback remains queued until delivery. Poll delivery consumes the response, so read it completely.`;
+  return `[lavish-axi] Long-polling for user feedback on ${file}. This stays silent until the user sends feedback, ends the session, or leaves every review window disconnected past the reconnect grace period - leave it running. Detected layout issues do NOT return this poll: they wait in the user's Layout issues inbox until the user queues them as ordinary feedback. If it gets killed or times out before feedback arrives, re-run \`lavish-axi poll ${file}\` - feedback remains queued until delivery. Poll delivery consumes the response, so read it completely.`;
 }
 function pollWaitTickText(elapsedMs) {
   const minutes = Math.round(elapsedMs / 6e4);
-  return `[lavish-axi] Still waiting for user feedback (${minutes}m). Leave this running until the user sends feedback or ends the session.`;
+  return `[lavish-axi] Still waiting for user feedback (${minutes}m). Leave this running until the user sends feedback, ends the session, or leaves every review window disconnected.`;
 }
 function pollInterruptedText(file) {
   return `[lavish-axi] Poll interrupted before user feedback arrived. The user may still be reviewing - re-run \`lavish-axi poll ${file}\` to keep waiting; feedback remains queued until delivery. Poll delivery consumes the response, so read it completely.`;
@@ -11232,6 +11273,12 @@ function createPollOutput({ file, response, agent = "generic" }) {
       next_step: createEndedNextStep(file, response.ended_by)
     };
   }
+  if (response.status === "browser_disconnected") {
+    return {
+      session: { file, status: "browser_disconnected" },
+      next_step: "The Lavish review window was closed or disconnected. The session remains open and resumable; ask the user whether they want to reopen it or end the session. Do not reopen or end it without their direction."
+    };
+  }
   return {
     session: { file, status: response.status || "waiting" },
     next_step: `No user feedback arrived before the optional timeout. Run \`lavish-axi poll ${file}\` without --timeout-ms to wait indefinitely - feedback remains queued until delivery, so re-running the poll is safe while waiting. Poll delivery consumes the response, so read it completely.`
@@ -11251,7 +11298,7 @@ function createFeedbackNextStep(file, artifactFailures, sessionEnded, endedBy, p
     return `${failureNote}${layoutNote}${whiteboardNote}${attachmentNote}This was the last feedback before the Lavish Editor session ended. Stop polling ${file}. Deliver any remaining updates directly in this conversation, or run \`lavish-axi ${file}\` to open a fresh session if the user needs further visual review.`;
   }
   const prefix = count > 0 ? artifactFailuresPrefix(file, artifactFailures) : `Apply the requested changes to ${file}. `;
-  return `${prefix}${layoutNote}${whiteboardNote}${attachmentNote}Do not respond to the user just yet. Now you must run \`lavish-axi poll ${file} --agent-reply "<message for the user>"\` without --timeout-ms unless the user ended the session. The poll waits silently until the user sends more feedback or ends the session - never kill it. ${pollExecutionGuidance({ agent })}`;
+  return `${prefix}${layoutNote}${whiteboardNote}${attachmentNote}Do not respond to the user just yet. Now you must run \`lavish-axi poll ${file} --agent-reply "<message for the user>"\` without --timeout-ms unless the user ended the session. The poll waits silently until the user sends more feedback, ends the session, or leaves every review window disconnected past the reconnect grace period - never kill it. ${pollExecutionGuidance({ agent })}`;
 }
 function artifactFailuresPrefix(file, artifactFailures) {
   const count = artifactFailures.length;
@@ -12316,7 +12363,7 @@ Usage:
 
 ${DESIGN_SYSTEM_HINT}
 
-Note: poll long-polls indefinitely by default until the user sends feedback or ends the session, staying silent while it waits - never kill it. Layout issues the browser detects are passive: they collect in the user's Layout issues inbox in the Lavish top bar and reach the agent only when the user selects them and queues the fixes, as an ordinary tag "layout-warnings" prompt. Do not pass --timeout-ms during normal agent use; it is for tests and debugging only. ${pollExecutionGuidance({ agent })} ${POLL_SEND_AND_END_RULE}
+Note: poll long-polls until the user sends feedback, ends the session, or leaves every review window disconnected past the reconnect grace period, staying silent while it waits - never kill it. Layout issues the browser detects are passive: they collect in the user's Layout issues inbox in the Lavish top bar and reach the agent only when the user selects them and queues the fixes, as an ordinary tag "layout-warnings" prompt. Do not pass --timeout-ms during normal agent use; it is for tests and debugging only. ${pollExecutionGuidance({ agent })} ${POLL_SEND_AND_END_RULE}
 
 `;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lavish-axi",
-  "version": "0.1.66",
+  "version": "0.1.68",
   "packageManager": "pnpm@11.1.1",
   "description": "HTML is the new markdown. Lavish is the new editor for your HTML artifacts.",
   "type": "module",

package/plugin.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://agent-plugins.org/schemas/1.0.0/plugin.schema.json",
   "name": "lavish-axi",
-  "version": "0.1.66",
+  "version": "0.1.68",
   "description": "HTML is the new markdown. Lavish is the new editor for your HTML artifacts.",
   "author": {
     "name": "Kun Chen",