lavish-axi 0.1.33 → 0.1.35

package/README.md

@@ -125,6 +125,7 @@ pnpm link
 ┌────────────────────────┐
 │ lavish-axi poll waits  │
 │ and returns prompts    │
+│ or layout warnings     │
 └────────────────────────┘
 ```
 
@@ -138,9 +139,10 @@ pnpm link
 - **Open-time layout gate** - The browser chrome masks each artifact until the real in-iframe layout audit reports no error-severity findings.
   Warning-only artifacts reveal normally; error findings notify the agent through the same `layout_warnings` poll path and keep the curtain up until a clean reload.
   The user can click **Show anyway**, and a bounded safety timeout reveals with a persistent layout-issues banner so review is never blocked indefinitely.
-- **Layout warnings** - After fonts load and layout settles, the injected SDK audits the real browser render for page horizontal overflow, element overflow, clipped text, and overlapping text.
-  Intentional horizontal scrollers using `overflow-x: auto` or `scroll` are excluded.
-  Fresh warnings are returned from `lavish-axi poll` as `layout_warnings` with `selector`, `kind`, `overflowPx`, `viewportWidth`, and `severity`, so agents can fix unreadable layouts before asking the human to review.
+- **Layout warnings** - After fonts load and layout settles, the injected SDK audits the real browser render for page horizontal overflow, element overflow, clipped or visibly spilling text, and overlapping text.
+  Intentional horizontal scrollers using `overflow-x: auto` or `scroll` are excluded from horizontal checks, and `overflow-y: auto` or `scroll` is treated as intentional for vertical overflow.
+  Current findings are returned from `lavish-axi poll` as `layout_warnings` with `selector`, `kind`, `overflowPx`, `viewportWidth`, `severity`, and `persistent`.
+  Fresh error-severity findings should be fixed and rechecked before asking the human to review; repeated or warning-only findings can be surfaced to the human with a note when the cause is not obvious.
 - **Local assets** - Copy local images, CSS, fonts, and scripts next to the HTML artifact and reference them with relative paths from that directory; root-prefixed paths such as `/assets/logo.png` will not resolve through Lavish's artifact route.
 - **Export and sharing** - `lavish-axi export` writes `<name>.export.html` by inlining local assets only, stripping the annotation SDK, and leaving remote CDN/font references as links that still need network access.
   `lavish-axi share` publishes the same local-inlined HTML to [ht-ml.app](https://ht-ml.app), a third-party hosting service not part of Lavish.
@@ -153,10 +155,14 @@ pnpm link
 - **Feedback controls** - Native controls (radios, checkboxes, inputs, selects, buttons, labels, disclosure summaries, contenteditable) are interactive automatically, so they do not need `data-lavish-action`.
   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.
-  The browser chrome keeps editing actions in the overflow menu (copy path, reload artifact, copy DOM snapshot, export standalone HTML, publish link, end session) and can submit queued prompts with **Send & end session**, which delivers the prompts before ending the session.
+  The browser chrome keeps editing actions in the overflow menu (copy path, reload artifact, copy DOM snapshot, export standalone HTML, publish link, end session) and can submit queued prompts with **Send & end session**, which sends the prompts and user-ended attribution together.
 - **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.
 - **Agent presence** - The browser shows when no agent is listening, keeps queued feedback and fresh layout warnings for the next successful `lavish-axi poll` send even across reloads, and only blocks human sends while the agent is working on delivered feedback. The no-timeout poll writes an immediate stderr banner and periodic stderr heartbeats while stdout stays reserved for the final response; if the poll is interrupted or times out, re-run it because queued feedback is never lost.
+- **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.
+  A plain `lavish-axi <html-file>` after a user-initiated end refuses to reopen the browser and returns guidance instead; pass `--reopen` only when the user asks for further review or something important needs their visual attention.
+  Agent-initiated ends keep reopening normally, same as before.
+  `lavish-axi poll`'s `ended` response and the `feedback` response for the final batch before an end both carry `next_step` guidance telling the agent to stop polling and deliver remaining updates in chat instead of reopening.
 - **Precise targets** - Text annotations include selected text plus range anchors, so agents are not limited to whole-element selectors.
 - **Server cleanup** - The detached server stops after the last session ends when nothing is connected, or after `LAVISH_AXI_IDLE_TIMEOUT_MS` (default 30 minutes) with no browser or poll connections.
   Set `LAVISH_AXI_IDLE_TIMEOUT_MS=0` or `off` to disable idle self-shutdown.
@@ -169,9 +175,9 @@ 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.                                                                                                                                                                                |
-| `lavish-axi poll <html-file>`   | Long-poll until the user sends feedback, ends the session, or the browser reports fresh `layout_warnings`; leave no-timeout polls running, or re-run them if interrupted.                                                                                                 |
-| `lavish-axi end <html-file>`    | End a session.                                                                                                                                                                                                                                                            |
+| `lavish-axi <html-file>`        | Open or resume a Lavish Editor session, with the open-time layout gate enabled by default. 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, ends the session, or the browser reports fresh `layout_warnings`; leave no-timeout polls running, or re-run them if interrupted. On `status: 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, and `--password` makes viewers enter the password before viewing. |
 | `lavish-axi stop`               | Shut down the background server.                                                                                                                                                                                                                                          |
@@ -190,6 +196,7 @@ For flows, architecture, state, or sequence diagrams, open the diagram playbook
 | ------------------------ | --------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
 | `lavish-axi <html-file>` | `--no-open`           | Ensure the server/session exists without opening another browser window.                                                                                                                                                            |
 | `lavish-axi <html-file>` | `--no-gate`           | Skip the open-time layout curtain for this browser open.                                                                                                                                                                            |
+| `lavish-axi <html-file>` | `--reopen`            | Reopen a session the user explicitly ended from the browser; without it, a plain open refuses and explains why instead of reopening uninvited.                                                                                      |
 | `lavish-axi update`      | `--check`             | Report current vs latest npm version without installing an update.                                                                                                                                                                  |
 | `lavish-axi export`      | `--out <path>`        | Write the export to a specific path instead of `<name>.export.html` next to the source.                                                                                                                                             |
 | `lavish-axi share`       | `--password <pw>`     | Make the third-party ht-ml.app page private; viewers must supply the password.                                                                                                                                                      |

package/dist/chrome-client.js

@@ -328,21 +328,26 @@ async function submitQueued() {
     submitQueuedAgain = false;
     if (!succeeded) {
       endAfterSubmit = false;
-    } else if (shouldSubmitAgain && queued.length) {
-      submitQueued();
-    } else if (endAfterSubmit) {
-      endAfterSubmit = false;
-      await endSession();
+    } else if (!ended && shouldSubmitAgain) {
+      if (queued.length) {
+        submitQueued();
+      } else if (endAfterSubmit) {
+        endAfterSubmit = false;
+        endSession();
+      }
     }
   }
 }
 
 async function submitQueuedOnce() {
   const prompts = queued.slice();
+  const shouldEndSession = endAfterSubmit;
+  const body = { prompts: prompts.map(stripInternalPromptFields), domSnapshot: pendingSnapshot };
+  if (shouldEndSession) body.endSession = true;
   const response = await fetch("/api/" + key + "/prompts", {
     method: "POST",
     headers: { "content-type": "application/json" },
-    body: JSON.stringify({ prompts: prompts.map(stripInternalPromptFields), domSnapshot: pendingSnapshot }),
+    body: JSON.stringify(body),
   });
   if (!response.ok) throw new Error("failed to submit queued prompts");
   for (const prompt of prompts) {
@@ -351,6 +356,11 @@ async function submitQueuedOnce() {
   }
   persistQueuedPrompts();
   render();
+  if (shouldEndSession) {
+    endAfterSubmit = false;
+    markSessionEnded();
+    return;
+  }
   if (agentPresence === "listening") setAgentPresence("working");
 }
 
@@ -474,6 +484,11 @@ async function endSession() {
   if (ended) return;
   const response = await fetch("/api/" + key + "/end", { method: "POST" });
   if (!response.ok) throw new Error("failed to end session");
+  markSessionEnded();
+}
+
+function markSessionEnded() {
+  if (ended) return;
   ended = true;
   closeMenus();
   annotationSwitch.disabled = true;

package/dist/cli.mjs

@@ -4277,6 +4277,57 @@ function isNativeInteractiveControl(el) {
     "button,input,select,textarea,option,optgroup,label,summary,[contenteditable]:not([contenteditable='false'])"
   ));
 }
+function fragmentsSignificantlyOverlap(fragmentsA, fragmentsB, { minAreaRatio = 0.25, minAreaPx = 24 } = {}) {
+  function rectAreaOf(rect) {
+    return Math.max(0, rect.width) * Math.max(0, rect.height);
+  }
+  function intersectionAreaOf(a, b) {
+    const width = Math.max(0, Math.min(a.right, b.right) - Math.max(a.left, b.left));
+    const height = Math.max(0, Math.min(a.bottom, b.bottom) - Math.max(a.top, b.top));
+    return width * height;
+  }
+  for (const a of fragmentsA) {
+    const threshold = Math.min(rectAreaOf(a) * minAreaRatio, minAreaPx);
+    for (const b of fragmentsB) {
+      if (intersectionAreaOf(a, b) >= threshold) return true;
+    }
+  }
+  return false;
+}
+function classifyHorizontalOverflow({ scrollWidth, clientWidth, overflowX, hasText, isTruncated, epsilon = 1 }) {
+  const overflowPx = clientWidth > 0 ? scrollWidth - clientWidth : 0;
+  if (overflowPx <= epsilon) return null;
+  const clipsText = hasText && (overflowX === "hidden" || overflowX === "clip") && !isTruncated;
+  return { overflowPx, kind: clipsText ? "clipped-text" : "element-scroll-overflow" };
+}
+function classifyVerticalOverflow({ scrollHeight, clientHeight, overflowY, hasText, isTruncated, epsilon = 1 }) {
+  const overflowPx = clientHeight > 0 ? scrollHeight - clientHeight : 0;
+  if (overflowPx <= epsilon) return null;
+  const scrollable = overflowY === "auto" || overflowY === "scroll";
+  if (scrollable || !hasText || isTruncated) return null;
+  const clips = overflowY === "hidden" || overflowY === "clip";
+  return { overflowPx, kind: "clipped-text", clips };
+}
+function resolveVisibleSpillCandidates(spillCandidates, { epsilon = 1 } = {}) {
+  function spillBottomEdge(candidate) {
+    const explicit = Number(candidate.spillBottom);
+    if (Number.isFinite(explicit)) return explicit;
+    const rectBottom = Number(candidate.rect?.bottom);
+    const overflowPx = Number(candidate.overflowPx);
+    if (!Number.isFinite(rectBottom) || !Number.isFinite(overflowPx)) return null;
+    return rectBottom + overflowPx;
+  }
+  function sameSpillEdge(candidate, other) {
+    const candidateBottom = spillBottomEdge(candidate);
+    const otherBottom = spillBottomEdge(other);
+    return candidateBottom !== null && otherBottom !== null && Math.abs(candidateBottom - otherBottom) <= epsilon;
+  }
+  return spillCandidates.filter(
+    (candidate) => !spillCandidates.some(
+      (other) => other.el !== candidate.el && candidate.el.contains(other.el) && sameSpillEdge(candidate, other)
+    )
+  );
+}
 function createArtifactSdk(deriveQueueKey, isNativeInteractive = isNativeInteractiveControl) {
   let annotationMode = true;
   let hovered = null;
@@ -4475,11 +4526,6 @@ function createArtifactSdk(deriveQueueKey, isNativeInteractive = isNativeInterac
   function rectArea(rect) {
     return Math.max(0, rect.width) * Math.max(0, rect.height);
   }
-  function intersectionArea(a, b) {
-    const width = Math.max(0, Math.min(a.right, b.right) - Math.max(a.left, b.left));
-    const height = Math.max(0, Math.min(a.bottom, b.bottom) - Math.max(a.top, b.top));
-    return width * height;
-  }
   function isVisibleForLayoutAudit(el, rect = el.getBoundingClientRect()) {
     if (!el || isLavishUi(el) || rect.width <= 0 || rect.height <= 0) return false;
     const style = getComputedStyle(el);
@@ -4543,31 +4589,56 @@ function createArtifactSdk(deriveQueueKey, isNativeInteractive = isNativeInterac
   function isIntentionalTextTruncation(style) {
     return style.textOverflow === "ellipsis" || Number.parseInt(style.webkitLineClamp || "0", 10) > 0;
   }
-  function auditElementOverflow(el, viewportWidth, findings, seen) {
+  function auditElementOverflow(el, viewportWidth, findings, seen, spillCandidates) {
     if (el === document.body || el === document.documentElement || hasIntentionalHorizontalScrollerAncestor(el)) return;
     const rect = el.getBoundingClientRect();
     if (!isVisibleForLayoutAudit(el, rect)) return;
     const style = getComputedStyle(el);
-    const scrollOverflowPx = el.clientWidth > 0 ? el.scrollWidth - el.clientWidth : 0;
-    if (scrollOverflowPx > layoutAuditOverflowEpsilon) {
-      const clipsText = hasReadableText(el) && (style.overflowX === "hidden" || style.overflowX === "clip") && !isIntentionalTextTruncation(style);
+    const hasText = hasReadableText(el);
+    const isTruncated = isIntentionalTextTruncation(style);
+    const horizontal = classifyHorizontalOverflow({
+      scrollWidth: el.scrollWidth,
+      clientWidth: el.clientWidth,
+      overflowX: style.overflowX,
+      hasText,
+      isTruncated,
+      epsilon: layoutAuditOverflowEpsilon
+    });
+    if (horizontal) {
       pushLayoutFinding(findings, seen, {
         selector: selector(el),
-        kind: clipsText ? "clipped-text" : "element-scroll-overflow",
-        overflowPx: scrollOverflowPx,
+        kind: horizontal.kind,
+        overflowPx: horizontal.overflowPx,
         viewportWidth,
-        severity: clipsText ? "error" : overflowSeverity(scrollOverflowPx)
+        severity: horizontal.kind === "clipped-text" ? "error" : overflowSeverity(horizontal.overflowPx)
       });
     }
-    const verticalClipPx = el.clientHeight > 0 ? el.scrollHeight - el.clientHeight : 0;
-    if (verticalClipPx > layoutAuditOverflowEpsilon && hasReadableText(el) && (style.overflowY === "hidden" || style.overflowY === "clip") && !isIntentionalTextTruncation(style)) {
-      pushLayoutFinding(findings, seen, {
-        selector: selector(el),
-        kind: "clipped-text",
-        overflowPx: verticalClipPx,
-        viewportWidth,
-        severity: "error"
-      });
+    const vertical = classifyVerticalOverflow({
+      scrollHeight: el.scrollHeight,
+      clientHeight: el.clientHeight,
+      overflowY: style.overflowY,
+      hasText,
+      isTruncated,
+      epsilon: layoutAuditOverflowEpsilon
+    });
+    if (vertical) {
+      if (vertical.clips) {
+        pushLayoutFinding(findings, seen, {
+          selector: selector(el),
+          kind: vertical.kind,
+          overflowPx: vertical.overflowPx,
+          viewportWidth,
+          severity: "error"
+        });
+      } else {
+        spillCandidates.push({
+          el,
+          selector: selector(el),
+          overflowPx: vertical.overflowPx,
+          viewportWidth,
+          spillBottom: rect.bottom + vertical.overflowPx
+        });
+      }
     }
     const parent2 = el.parentElement;
     if (!parent2 || parent2 === document.body || parent2 === document.documentElement) return;
@@ -4585,36 +4656,56 @@ function createArtifactSdk(deriveQueueKey, isNativeInteractive = isNativeInterac
       });
     }
   }
+  function resolveSpillCandidates(spillCandidates, findings, seen) {
+    for (const candidate of resolveVisibleSpillCandidates(spillCandidates, { epsilon: layoutAuditOverflowEpsilon })) {
+      pushLayoutFinding(findings, seen, {
+        selector: candidate.selector,
+        kind: "clipped-text",
+        overflowPx: candidate.overflowPx,
+        viewportWidth: candidate.viewportWidth,
+        severity: "error"
+      });
+    }
+  }
+  function elementLineFragments(el) {
+    const rects = [...el.getClientRects()].filter((r) => r.width > 0 && r.height > 0);
+    if (rects.length) return rects;
+    const rect = el.getBoundingClientRect();
+    return rect.width > 0 && rect.height > 0 ? [rect] : [];
+  }
   function auditOverlappingText(elements, viewportWidth, findings, seen) {
-    const candidates = elements.filter((el) => el.children.length === 0 && hasReadableText(el)).filter((el) => isVisibleForLayoutAudit(el)).slice(0, 200);
+    const candidates = elements.filter((el) => el.children.length === 0 && hasReadableText(el)).filter((el) => isVisibleForLayoutAudit(el)).filter((el) => getComputedStyle(el).position === "static").slice(0, 200);
     for (const el of candidates) {
-      const rect = el.getBoundingClientRect();
-      if (rectArea(rect) < 16) continue;
-      const points = [
-        { x: rect.left + rect.width / 2, y: rect.top + rect.height / 2 },
-        { x: rect.left + Math.min(4, rect.width / 2), y: rect.top + Math.min(4, rect.height / 2) },
-        { x: rect.right - Math.min(4, rect.width / 2), y: rect.bottom - Math.min(4, rect.height / 2) }
-      ];
-      for (const point of points) {
-        if (point.x < 0 || point.y < 0 || point.x > viewportWidth || point.y > window.innerHeight) continue;
-        const top = document.elementFromPoint(point.x, point.y);
-        if (!(top instanceof Element) || top === el || el.contains(top) || top.contains(el) || isLavishUi(top))
-          continue;
-        if (hasIntentionalHorizontalScrollerAncestor(top)) continue;
-        const elPosition = getComputedStyle(el).position;
-        const topPosition = getComputedStyle(top).position;
-        if (elPosition !== "static" || topPosition !== "static") continue;
-        const topRect = top.getBoundingClientRect();
-        const overlapArea = intersectionArea(rect, topRect);
-        if (overlapArea < Math.min(rectArea(rect) * 0.25, 24)) continue;
-        pushLayoutFinding(findings, seen, {
-          selector: selector(el),
-          kind: "overlapping-text",
-          overflowPx: 0,
-          viewportWidth,
-          severity: "error"
-        });
-        break;
+      const fragments = elementLineFragments(el);
+      let flagged = false;
+      for (const rect of fragments) {
+        if (flagged) break;
+        if (rectArea(rect) < 16) continue;
+        const points = [
+          { x: rect.left + rect.width / 2, y: rect.top + rect.height / 2 },
+          { x: rect.left + Math.min(4, rect.width / 2), y: rect.top + Math.min(4, rect.height / 2) },
+          { x: rect.right - Math.min(4, rect.width / 2), y: rect.bottom - Math.min(4, rect.height / 2) }
+        ];
+        for (const point of points) {
+          if (point.x < 0 || point.y < 0 || point.x > viewportWidth || point.y > window.innerHeight) continue;
+          const top = document.elementFromPoint(point.x, point.y);
+          if (!(top instanceof Element) || top === el || el.contains(top) || top.contains(el) || isLavishUi(top))
+            continue;
+          if (hasIntentionalHorizontalScrollerAncestor(top)) continue;
+          if (getComputedStyle(top).position !== "static") continue;
+          if (!fragmentsSignificantlyOverlap([rect], elementLineFragments(top))) continue;
+          pushLayoutFinding(findings, seen, {
+            selector: selector(el),
+            kind: "overlapping-text",
+            overflowPx: 0,
+            viewportWidth,
+            // Heuristic and sampling-based even after fragment-aware matching, so it stays a
+            // warning rather than holding the open-time gate the way a real clip/overflow does.
+            severity: "warning"
+          });
+          flagged = true;
+          break;
+        }
       }
     }
   }
@@ -4633,7 +4724,9 @@ function createArtifactSdk(deriveQueueKey, isNativeInteractive = isNativeInterac
       });
     }
     const elements = collectLayoutAuditElements();
-    for (const el of elements) auditElementOverflow(el, viewportWidth, findings, seen);
+    const spillCandidates = [];
+    for (const el of elements) auditElementOverflow(el, viewportWidth, findings, seen, spillCandidates);
+    resolveSpillCandidates(spillCandidates, findings, seen);
     auditOverlappingText(elements, viewportWidth, findings, seen);
     return findings;
   }
@@ -4918,6 +5011,7 @@ var SessionStore = class {
       pending_prompts: existing.pending_prompts || 0,
       prompts: existingPrompts,
       layout_warnings: [],
+      delivered_layout_warning_keys: existing.delivered_layout_warning_keys || [],
       dom_snapshot: existing.dom_snapshot || "",
       chat: existing.chat || [],
       updated_at: (/* @__PURE__ */ new Date()).toISOString()
@@ -4933,13 +5027,16 @@ var SessionStore = class {
       return null;
     }
     const prompts = Array.isArray(payload.prompts) ? payload.prompts : [];
+    const shouldEndSession = Boolean(payload.endSession || payload.end_session);
+    const alreadyEnded = session.status === "ended";
     const normalizedPrompts = prompts.map(normalizePrompt);
     const userMessages = normalizedPrompts.filter((prompt) => prompt.tag === "message" && prompt.prompt).map((prompt) => ({ role: "user", text: prompt.prompt, at: (/* @__PURE__ */ new Date()).toISOString() }));
     session.prompts = [...session.prompts || [], ...normalizedPrompts];
     session.chat = [...session.chat || [], ...userMessages];
     session.pending_prompts = session.prompts.length;
     session.dom_snapshot = String(payload.domSnapshot || payload.dom_snapshot || "");
-    session.status = "feedback";
+    session.status = shouldEndSession || alreadyEnded ? "ended" : "feedback";
+    if (shouldEndSession) session.ended_by = "user";
     session.updated_at = (/* @__PURE__ */ new Date()).toISOString();
     await this.writeState(state);
     return session;
@@ -4950,13 +5047,23 @@ var SessionStore = class {
     if (!session) {
       return null;
     }
-    const layoutWarnings = normalizeLayoutWarnings(payload.layout_warnings || payload.layoutWarnings || []);
+    const deliveredWarningKeys = session.delivered_layout_warning_keys || [];
+    const deliveredKeys = new Set(deliveredWarningKeys);
+    const layoutWarnings = normalizeLayoutWarnings(
+      payload.layout_warnings || payload.layoutWarnings || [],
+      deliveredKeys
+    );
+    const activeWarningKeys = new Set(layoutWarnings.map(layoutWarningKey));
+    const nextDeliveredWarningKeys = deliveredWarningKeys.filter((key2) => activeWarningKeys.has(key2)).slice(-200);
+    const deliveredKeysChanged = nextDeliveredWarningKeys.length !== deliveredWarningKeys.length || nextDeliveredWarningKeys.some((key2, index) => key2 !== deliveredWarningKeys[index]);
     const previousSignature = JSON.stringify(session.layout_warnings || []);
     const nextSignature = JSON.stringify(layoutWarnings);
-    if (previousSignature === nextSignature) {
+    const warningsChanged = previousSignature !== nextSignature;
+    if (!warningsChanged && !deliveredKeysChanged) {
       return { session, changed: false, hasWarnings: layoutWarnings.length > 0 };
     }
     session.layout_warnings = layoutWarnings;
+    session.delivered_layout_warning_keys = nextDeliveredWarningKeys;
     if (layoutWarnings.length > 0 && session.status !== "ended") {
       session.status = "feedback";
     } else if ((session.prompts || []).length === 0 && session.status !== "ended") {
@@ -4964,7 +5071,7 @@ var SessionStore = class {
     }
     session.updated_at = (/* @__PURE__ */ new Date()).toISOString();
     await this.writeState(state);
-    return { session, changed: true, hasWarnings: layoutWarnings.length > 0 };
+    return { session, changed: warningsChanged, hasWarnings: layoutWarnings.length > 0 };
   }
   async takeFeedback(key) {
     const state = await this.readState();
@@ -4974,33 +5081,48 @@ var SessionStore = class {
     }
     const prompts = session.prompts || [];
     const layoutWarnings = session.layout_warnings || [];
+    const alreadyEnded = session.status === "ended";
     if (prompts.length === 0 && layoutWarnings.length === 0) {
-      return session.status === "ended" ? { status: "ended" } : { status: "waiting" };
+      return alreadyEnded ? { status: "ended", ended_by: session.ended_by } : { status: "waiting" };
     }
     const result = {
       status: "feedback",
       dom_snapshot: session.dom_snapshot || "",
       prompts,
-      ...layoutWarnings.length > 0 ? { layout_warnings: layoutWarnings } : {}
+      ...layoutWarnings.length > 0 ? { layout_warnings: layoutWarnings } : {},
+      // This is the final delivery before the session shows as ended - flag it so the agent
+      // knows not to expect (or force) a reopened browser afterward.
+      ...alreadyEnded ? { session_ended: true, ended_by: session.ended_by } : {}
     };
     session.prompts = [];
     session.layout_warnings = [];
     session.pending_prompts = 0;
     session.dom_snapshot = "";
-    if (session.status !== "ended") {
+    if (layoutWarnings.length > 0) {
+      const deliveredKeys = new Set(session.delivered_layout_warning_keys || []);
+      for (const warning of layoutWarnings) deliveredKeys.add(layoutWarningKey(warning));
+      session.delivered_layout_warning_keys = [...deliveredKeys].slice(-200);
+    }
+    if (!alreadyEnded) {
       session.status = "open";
     }
     session.updated_at = (/* @__PURE__ */ new Date()).toISOString();
     await this.writeState(state);
     return result;
   }
-  async endSession(key) {
+  // `endedBy` distinguishes a human ending review from the browser chrome ("user") from an
+  // agent explicitly closing the loop via `lavish-axi end` ("agent"). Only a user-initiated end
+  // blocks a plain reopen - see `SessionStore` callers in server.js.
+  async endSession(key, endedBy = "agent") {
     const state = await this.readState();
     const session = state.sessions[key];
     if (!session) {
       return null;
     }
+    const existingEndedBy = session.status === "ended" ? session.ended_by : void 0;
+    const nextEndedBy = endedBy === "user" || existingEndedBy === "user" ? "user" : "agent";
     session.status = "ended";
+    session.ended_by = nextEndedBy;
     session.updated_at = (/* @__PURE__ */ new Date()).toISOString();
     await this.writeState(state);
     return session;
@@ -5052,15 +5174,23 @@ function normalizePrompt(prompt) {
   if (target) normalized.target = target;
   return normalized;
 }
-function normalizeLayoutWarnings(layoutWarnings) {
+function layoutWarningKey(warning) {
+  return `${warning.kind}:${warning.selector}`;
+}
+function normalizeLayoutWarnings(layoutWarnings, deliveredKeys = /* @__PURE__ */ new Set()) {
   if (!Array.isArray(layoutWarnings)) return [];
-  return layoutWarnings.filter((warning) => warning && typeof warning === "object" && !Array.isArray(warning)).map((warning) => ({
-    selector: String(warning.selector || ""),
-    kind: String(warning.kind || "layout-warning"),
-    overflowPx: normalizeFiniteNumber(warning.overflowPx),
-    viewportWidth: normalizeFiniteNumber(warning.viewportWidth),
-    severity: warning.severity === "warning" ? "warning" : "error"
-  }));
+  return layoutWarnings.filter((warning) => warning && typeof warning === "object" && !Array.isArray(warning)).map((warning) => {
+    const selector = String(warning.selector || "");
+    const kind = String(warning.kind || "layout-warning");
+    return {
+      selector,
+      kind,
+      overflowPx: normalizeFiniteNumber(warning.overflowPx),
+      viewportWidth: normalizeFiniteNumber(warning.viewportWidth),
+      severity: warning.severity === "warning" ? "warning" : "error",
+      persistent: deliveredKeys.has(layoutWarningKey({ kind, selector }))
+    };
+  });
 }
 function normalizeFiniteNumber(value) {
   const number = Number(value);
@@ -5139,9 +5269,15 @@ async function serve({
     try {
       const file = await canonicalFile(req.body.file);
       const key = sessionKey(file);
+      const reopen = Boolean(req.body.reopen);
+      const existing = await store.findByKey(key);
+      if (existing?.status === "ended" && existing.ended_by === "user" && !reopen) {
+        logEvent?.(`session open blocked (user-ended) key=${key} file=${file}`);
+        res.json({ key, file, url: existing.url, status: "user-ended" });
+        return;
+      }
       const sessionUrl = `http://${hostForUrl(linkHostName)}:${publicPort}/session/${key}`;
       const url = shouldDisableLayoutGateOpen(req.body || {}) ? appendNoGateParam(sessionUrl) : sessionUrl;
-      const existing = await store.findByKey(key);
       const session = await store.upsertSession(file, sessionUrl);
       if (existing?.status === "ended") {
         clearFeedbackDelivery(key, activePolls, deliveredFeedback, events);
@@ -5227,13 +5363,16 @@ async function serve({
   });
   app.post("/api/:key/prompts", async (req, res, next) => {
     try {
+      const shouldEndSession = Boolean(req.body?.endSession || req.body?.end_session);
       const session = await store.queuePrompts(req.params.key, req.body || {});
       if (!session) {
         res.status(404).json({ error: "session not found" });
         return;
       }
-      events.emit("feedback", req.params.key);
+      if (shouldEndSession) clearFeedbackDelivery(req.params.key, activePolls, deliveredFeedback, events);
+      events.emit(shouldEndSession ? "ended" : "feedback", req.params.key);
       res.json({ status: "queued", pending_prompts: session.pending_prompts });
+      if (shouldEndSession) await shutdownIfNoLiveSessions();
     } catch (error) {
       next(error);
     }
@@ -5255,7 +5394,7 @@ async function serve({
   });
   app.post("/api/:key/end", async (req, res, next) => {
     try {
-      await store.endSession(req.params.key);
+      await store.endSession(req.params.key, "user");
       clearFeedbackDelivery(req.params.key, activePolls, deliveredFeedback, events);
       events.emit("ended", req.params.key);
       res.json({ status: "ended" });
@@ -5342,7 +5481,7 @@ async function serve({
     try {
       const file = await canonicalFile(req.body.file);
       const key = sessionKey(file);
-      await store.endSession(key);
+      await store.endSession(key, "agent");
       clearFeedbackDelivery(key, activePolls, deliveredFeedback, events);
       events.emit("ended", key);
       res.json({ status: "ended" });
@@ -5818,6 +5957,10 @@ const key=${JSON.stringify(key)};
 void key;
 const deriveQueueKey=${deriveLavishQueueKey.toString()};
 const isNativeInteractiveControl=${isNativeInteractiveControl.toString()};
+const fragmentsSignificantlyOverlap=${fragmentsSignificantlyOverlap.toString()};
+const resolveVisibleSpillCandidates=${resolveVisibleSpillCandidates.toString()};
+const classifyHorizontalOverflow=${classifyHorizontalOverflow.toString()};
+const classifyVerticalOverflow=${classifyVerticalOverflow.toString()};
 (${createArtifactSdk.toString()})(deriveQueueKey, isNativeInteractiveControl);
 })();`;
 }
@@ -6004,7 +6147,7 @@ function normalizePagePath(path6) {
 var COMMANDS = /* @__PURE__ */ new Set(["open", "poll", "end", "stop", "server", "playbook", "design", "setup", "export", "share"]);
 var RESERVED = new Set(RESERVED_COMMANDS);
 var DESCRIPTION = "Lavish Editor helps agents turn rich HTML artifacts into collaborative human review surfaces. Whenever you are about to give user a complex response that will be easier to understand via a rich / interactive page, consider using Lavish Editor. First generate an interactive HTML artifact according to user request, then run `lavish-axi <html-file>` so the user can visually review it, annotate elements or selected text, queue prompts, and send feedback back through `lavish-axi poll`.";
-var VERSION = "0.1.33";
+var VERSION = "0.1.35";
 async function run(argv) {
   await ensureStateDir();
   const normalizedArgv = normalizeArgv(argv);
@@ -6096,11 +6239,11 @@ function createHomeOutput({ bin, sessions, includeSessions = true }) {
     ],
     playbooks: listPlaybooks(),
     help: [
-      "Run `lavish-axi <html-file>` to open or resume a Lavish Editor session",
+      "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 or browser-reported layout_warnings. It long-polls and stays silent until the user sends feedback, ends the session, or the real browser reports fresh layout_warnings, so leave it running - never kill it. Fix layout_warnings before involving the human. If your harness limits how long a foreground command may run, run the poll as a background task; if it gets killed or times out anyway, just re-run it - queued feedback is never lost",
-      "Run `lavish-axi end <html-file>` to end a session",
+      "Run `lavish-axi poll <html-file>` to wait for user feedback or browser-reported layout_warnings. It long-polls and stays silent until the user sends feedback, ends the session, or the real browser reports fresh layout_warnings, so leave it running - never kill it. Fix and re-check fresh error-severity layout_warnings before involving the human; if the poll says every current warning is persistent or low-severity, proceed with a note instead of looping. If your harness limits how long a foreground command may run, run the poll as a background task; if it gets killed or times out anyway, just re-run it - queued feedback is never lost. When it reports the session ended, stop polling and do not reopen it uninvited - deliver remaining updates in this conversation instead",
+      "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",
       "Run `lavish-axi share <html-file> [--password <pw>] [--token <t>]` to publish the artifact on ht-ml.app (https://ht-ml.app), a third-party hosting service not part of Lavish, and get back a visitable URL. Shares are PUBLIC by default, so anyone with the link can open them. Pass --password to publish a PRIVATE password-protected page; viewers must supply the password to view. Local assets are inlined; remote refs load over the network. It returns the url plus a secret update_key for managing the page later. Use --token or LAVISH_AXI_HTML_APP_TOKEN only when you have an optional bearer token; it is never required. Users can also publish from the browser chrome's overflow menu",
       "Run `lavish-axi stop` to shut down the background server (it also self-stops when idle or after the last session ends with nothing connected)",
@@ -6129,7 +6272,13 @@ function createPlaybookOutput(args) {
 function createOpenOutput({ file, url, status }) {
   return {
     session: { file, url, status },
-    next_step: `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 the real browser reports layout_warnings from the in-iframe layout audit, and it stays silent the whole time - that is normal, never kill it. If layout_warnings arrive, fix overflow, clipped text, or overlapping unreadable content and re-check before involving the human. Do not pass --timeout-ms during normal agent use. If your harness limits how long a foreground command may run, run the poll as a background task and wait for it to finish; if the poll still gets killed or times out, just re-run it - queued feedback is never lost. 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.`
+    next_step: `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 the real browser reports layout_warnings from the in-iframe layout audit, and it stays silent the whole time - that is normal, never kill it. If layout_warnings arrive, follow the poll response's next_step: fix and re-check fresh error-severity overflow or clipped-text findings before involving the human, but persistent or low-severity warnings may be surfaced with a note when the cause is not obvious. Do not pass --timeout-ms during normal agent use. If your harness limits how long a foreground command may run, run the poll as a background task and wait for it to finish; if the poll still gets killed or times out, just re-run it - queued feedback is never lost. 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 }) {
+  return {
+    session: { file, url, status: "user-ended" },
+    next_step: `The user explicitly ended this Lavish Editor session from the browser, so \`lavish-axi ${file}\` did not reopen it. Do not reopen 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\`.`
   };
 }
 async function openCommand(args) {
@@ -6140,8 +6289,12 @@ async function openCommand(args) {
   await assertHtmlFile(file);
   const absolute = await canonicalFile(file);
   const noGate = args.includes("--no-gate");
+  const reopen = args.includes("--reopen");
   const baseUrl = await ensureServer({ forceRestart: shouldForceRestartForLocalBuild(process.argv[1] || "") });
-  const response = await postJson(`${baseUrl}/api/sessions`, { file: absolute, noGate });
+  const response = await postJson(`${baseUrl}/api/sessions`, { file: absolute, noGate, reopen });
+  if (response.status === "user-ended") {
+    return createUserEndedOpenOutput({ file: absolute, url: response.url });
+  }
   if (shouldOpenBrowser(args, process.env)) {
     try {
       const open = (await import("open")).default;
@@ -6229,26 +6382,66 @@ function createPollOutput({ file, response }) {
   }
   if (response.status === "feedback") {
     const layoutWarnings = Array.isArray(response.layout_warnings) ? response.layout_warnings : [];
+    const sessionEnded = Boolean(response.session_ended);
+    const endedBy = typeof response.ended_by === "string" ? response.ended_by : void 0;
     return {
-      session: { file, status: "feedback" },
+      session: {
+        file,
+        status: "feedback",
+        ...sessionEnded ? { session_ended: true, ...endedBy ? { ended_by: endedBy } : {} } : {}
+      },
       dom_snapshot: response.dom_snapshot || "",
       prompts: response.prompts || [],
       ...layoutWarnings.length > 0 ? { layout_warnings: layoutWarnings } : {},
-      next_step: createFeedbackNextStep(file, layoutWarnings.length)
+      next_step: createFeedbackNextStep(file, layoutWarnings, sessionEnded, endedBy)
     };
   }
   if (response.status === "ended") {
-    return { session: { file, status: "ended" } };
+    return {
+      session: { file, status: "ended", ...response.ended_by ? { ended_by: response.ended_by } : {} },
+      next_step: createEndedNextStep(file, response.ended_by)
+    };
   }
   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 - queued feedback is never lost, so re-running the poll is always safe.`
   };
 }
-function createFeedbackNextStep(file, layoutWarningCount) {
-  const layoutPrefix = layoutWarningCount > 0 ? `${layoutWarningCount} layout warning${layoutWarningCount === 1 ? "" : "s"} detected - fix horizontal overflow, clipped text, or overlapping unreadable content in ${file}, then reload or re-open the artifact and re-check before involving the human. ` : `Apply the requested changes to ${file}. `;
+function createFeedbackNextStep(file, layoutWarnings, sessionEnded, endedBy) {
+  const count = layoutWarnings.length;
+  if (sessionEnded) {
+    const layoutNote = count > 0 ? `${count} layout warning${count === 1 ? "" : "s"} arrived alongside this final feedback. ` : "";
+    if (endedBy === "user") {
+      return `${layoutNote}This was the last feedback before the user ended the session. Stop polling ${file} and do not reopen it - deliver any remaining updates directly in this conversation instead. Only run \`lavish-axi ${file} --reopen\` if the user explicitly asks for further review or something genuinely important needs their visual attention.`;
+    }
+    return `${layoutNote}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 layoutPrefix = count > 0 ? layoutWarningsPrefix(file, layoutWarnings) : `Apply the requested changes to ${file}. `;
   return `${layoutPrefix}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 reports fresh layout_warnings - never kill it. If your harness limits how long a foreground command may run, run the poll as a background task; if it still gets killed or times out, just re-run it - queued feedback is never lost.`;
 }
+function layoutWarningsPrefix(file, layoutWarnings) {
+  const count = layoutWarnings.length;
+  const plural = count === 1 ? "" : "s";
+  const allPersistent = layoutWarnings.every((warning) => warning.persistent);
+  const allLowSeverity = layoutWarnings.every((warning) => warning.severity !== "error");
+  const allRepeatOrLowSeverity = layoutWarnings.every((warning) => warning.persistent || warning.severity !== "error");
+  if (allPersistent) {
+    return `${count} layout warning${plural} detected, and every one was already reported in a prior poll and is still unresolved - if you already attempted a fix, it is fine to proceed to the human with a short note about what remains instead of looping further edits and reloads. `;
+  }
+  if (allLowSeverity) {
+    return `${count} low-severity layout warning${plural} detected (no error-severity findings) - fix them if the cause is obvious in ${file}, otherwise it is fine to proceed to the human with a note instead of iterating further. `;
+  }
+  if (allRepeatOrLowSeverity) {
+    return `${count} layout warning${plural} detected, with no fresh error-severity findings - fix any obvious low-severity issue in ${file}, otherwise it is fine to proceed to the human with a note instead of iterating further. `;
+  }
+  return `${count} layout warning${plural} detected - fix horizontal overflow or clipped text in ${file}, then re-check in the browser before involving the human. Lavish live-reloads the artifact automatically after you save, so you do not need to re-run \`lavish-axi ${file}\` for this. `;
+}
+function createEndedNextStep(file, endedBy) {
+  if (endedBy === "user") {
+    return `The user ended this Lavish Editor session. Stop polling ${file} - do not run \`lavish-axi ${file}\` to reopen it. Deliver any remaining updates directly in this conversation instead. Only reopen with \`lavish-axi ${file} --reopen\` if the user explicitly asks for further review or something genuinely important needs their visual attention.`;
+  }
+  return `This Lavish Editor session for ${file} has ended. Stop polling. 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.`;
+}
 async function endCommand(args) {
   const file = firstPositionalArg(args);
   if (!file) {
@@ -6771,7 +6964,7 @@ var TOP_LEVEL_HELP = `lavish-axi - Lavish Editor AXI
 
 Usage:
   lavish-axi
-  lavish-axi <html-file> [--no-open] [--no-gate]
+  lavish-axi <html-file> [--no-open] [--no-gate] [--reopen]
   lavish-axi poll <html-file> [--agent-reply "..."]
   lavish-axi end <html-file>
   lavish-axi export <html-file> [--out <path>]
@@ -6783,21 +6976,21 @@ Usage:
 
 ${DESIGN_SYSTEM_HINT}
 
-Note: poll long-polls indefinitely by default until the user sends feedback, ends the session, or the browser reports fresh layout_warnings, staying silent while it waits - never kill it. Fix layout_warnings before involving the human. Do not pass --timeout-ms during normal agent use; it is for tests and debugging only. If your harness limits how long a foreground command may run, run the poll as a background task; if it gets killed or times out anyway, just re-run it - queued feedback is never lost.
+Note: poll long-polls indefinitely by default until the user sends feedback, ends the session, or the browser reports fresh layout_warnings, staying silent while it waits - never kill it. Fix and re-check fresh error-severity layout_warnings before involving the human; persistent or low-severity findings may be surfaced with a note when the cause is not obvious. Do not pass --timeout-ms during normal agent use; it is for tests and debugging only. If your harness limits how long a foreground command may run, run the poll as a background task; if it gets killed or times out anyway, just re-run it - queued feedback is never lost. When the user ends a session from the browser, stop polling and do not reopen it uninvited - pass --reopen to <html-file> only when the user asks for further review or something important needs their visual attention.
 
 `;
 var COMMAND_HELP = {
-  open: `Usage: lavish-axi <html-file> [--no-open] [--no-gate]
+  open: `Usage: lavish-axi <html-file> [--no-open] [--no-gate] [--reopen]
 
-Open or resume a Lavish Editor review session for an HTML artifact. Use --no-open when you need to ensure the server/session exists without opening another browser window. Use --no-gate to skip the open-time layout curtain for this browser open.
+Open or resume a Lavish Editor review session for an HTML artifact. Use --no-open when you need to ensure the server/session exists without opening another browser window. Use --no-gate to skip the open-time layout curtain for this browser open. If the user explicitly ended the session from the browser, this refuses to reopen it and returns guidance instead - pass --reopen to force it open when the user asks for further review or something important needs their visual attention. Sessions ended by the agent (\`lavish-axi end\`) reopen normally without the flag.
 `,
   poll: `Usage: lavish-axi poll <html-file> [--agent-reply "..."]
 
-This command long-polls indefinitely for queued user prompts and browser-reported layout_warnings, then returns them to the agent. It stays silent while it waits - that is normal, never kill it. Fix layout_warnings before involving the human. Do not pass --timeout-ms during normal agent use; it is for tests and debugging only. If your harness limits how long a foreground command may run, run the poll as a background task and wait for it to finish; if it still gets killed or times out, just re-run it - queued feedback is never lost. Use --agent-reply after applying prior feedback to display your response in Lavish Editor before waiting again.
+This command long-polls indefinitely for queued user prompts and browser-reported layout_warnings, then returns them to the agent. It stays silent while it waits - that is normal, never kill it. Fix and re-check fresh error-severity layout_warnings before involving the human; persistent or low-severity findings may be surfaced with a note when the cause is not obvious. Do not pass --timeout-ms during normal agent use; it is for tests and debugging only. If your harness limits how long a foreground command may run, run the poll as a background task and wait for it to finish; if it still gets killed or times out, just re-run it - queued feedback is never lost. Use --agent-reply after applying prior feedback to display your response in Lavish Editor before waiting again. When status is ended, stop polling and do not reopen the session uninvited - deliver remaining updates directly in this conversation instead.
 `,
   end: `Usage: lavish-axi end <html-file>
 
-End a Lavish Editor session.
+End a Lavish Editor session as the agent. A session ended this way still reopens normally on the next \`lavish-axi <html-file>\`, unlike a user ending it from the browser, which requires --reopen.
 `,
   export: `Usage: lavish-axi export <html-file> [--out <path>]
 

package/package.json

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

package/skills/lavish/SKILL.md

@@ -34,9 +34,10 @@ Use lavish-axi when the user asks for a visual artifact, HTML explainer, interac
 3. Run `npx -y lavish-axi poll <html-file>` to long-poll for the user's annotations, queued prompts, and browser-reported `layout_warnings`.
    The poll stays silent until the user acts or the real browser reports fresh layout warnings - leave it running, never kill it.
    If your harness limits how long a foreground command may run, run the poll as a background task; if it gets killed or times out anyway, just re-run it - queued feedback is never lost.
-4. If poll returns `layout_warnings`, fix overflow, clipped text, or overlapping unreadable content and re-check before involving the human.
+4. If poll returns `layout_warnings`, follow the returned `next_step`: fix and re-check fresh error-severity findings, but proceed with a note instead of looping when every current warning is persistent or low-severity.
 5. Apply human feedback, then poll again with `--agent-reply "<message>"` to reply in the browser and keep the loop going.
 6. Run `npx -y lavish-axi end <html-file>` when the review is finished.
+7. If the user ends the session from the browser instead, `npx -y lavish-axi <html-file>` refuses to reopen it and says so - only pass `--reopen` when the user asks for further review or something genuinely important needs their visual attention. Otherwise deliver remaining updates directly in this conversation.
 
 ## Visual guidance
 
@@ -62,11 +63,11 @@ For flows, architecture, state, or sequence diagrams, do not hand-build boxes-an
 
 ## Commands & rules
 
-- Run `npx -y lavish-axi <html-file>` to open or resume a Lavish Editor session
+- Run `npx -y 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 `npx -y lavish-axi poll <html-file>` to wait for user feedback or browser-reported layout_warnings. It long-polls and stays silent until the user sends feedback, ends the session, or the real browser reports fresh layout_warnings, so leave it running - never kill it. Fix layout_warnings before involving the human. If your harness limits how long a foreground command may run, run the poll as a background task; if it gets killed or times out anyway, just re-run it - queued feedback is never lost
-- Run `npx -y lavish-axi end <html-file>` to end a session
+- Run `npx -y lavish-axi poll <html-file>` to wait for user feedback or browser-reported layout_warnings. It long-polls and stays silent until the user sends feedback, ends the session, or the real browser reports fresh layout_warnings, so leave it running - never kill it. Fix and re-check fresh error-severity layout_warnings before involving the human; if the poll says every current warning is persistent or low-severity, proceed with a note instead of looping. If your harness limits how long a foreground command may run, run the poll as a background task; if it gets killed or times out anyway, just re-run it - queued feedback is never lost. When it reports the session ended, stop polling and do not reopen it uninvited - deliver remaining updates in this conversation instead
+- Run `npx -y 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 `npx -y lavish-axi <html-file>` refuses to reopen it without `--reopen`
 - Run `npx -y 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
 - Run `npx -y lavish-axi share <html-file> [--password <pw>] [--token <t>]` to publish the artifact on ht-ml.app (https://ht-ml.app), a third-party hosting service not part of Lavish, and get back a visitable URL. Shares are PUBLIC by default, so anyone with the link can open them. Pass --password to publish a PRIVATE password-protected page; viewers must supply the password to view. Local assets are inlined; remote refs load over the network. It returns the url plus a secret update_key for managing the page later. Use --token or LAVISH_AXI_HTML_APP_TOKEN only when you have an optional bearer token; it is never required. Users can also publish from the browser chrome's overflow menu
 - Run `npx -y lavish-axi stop` to shut down the background server (it also self-stops when idle or after the last session ends with nothing connected)