@aparte/core 0.16.7 → 0.16.9

package/dist/{dialog-triggers-Dw_DIlQ7.js → dialog-triggers-CApC7K1I.js}

@@ -1,4 +1,4 @@
-import { A as APARTE_ICON_GLYPHS } from "./glyphs-DQTeOj4w.js";
+import { A as APARTE_ICON_GLYPHS } from "./glyphs-C8z2oJUg.js";
 import { runStreamAgent } from "@aparte/engine";
 function contentToText(content) {
   if (typeof content === "string") return content;
@@ -97,7 +97,7 @@ const APARTE_DEFAULT_LOCALE = {
   elicitationAnswerLabel: "Your answer",
   approvalInstructionPlaceholder: "Or tell it what to do instead…",
   approvalAsk: "Run {tool}?",
-  approvalWaiting: "waiting for you",
+  approvalWaiting: "Waiting",
   scrollToBottom: "Scroll to bottom",
   suggestionsLabel: "Suggested prompts",
   contextLabel: "Context window",
@@ -115,6 +115,7 @@ const APARTE_DEFAULT_LOCALE = {
   removeAttachment: "Remove {name}",
   selectSearchPlaceholder: "Search…",
   selectSearchLabel: "Search options",
+  loading: "Loading…",
   download: "Download",
   previewPending: "Press Preview to run this artifact.",
   sandboxError: "The sandbox failed during generation.",
@@ -2484,14 +2485,16 @@ const thinkingRenderer = {
    * only part that is genuinely about reasoning rather than about disclosure.
    * `.aparte-thinking-label` stays too: `relabel` below queries it.
    */
-  render: (segment) => `<details class="aparte-segment aparte-segment-thinking aparte-accordion__item" data-segment-id="${escapeHtml(segment.id)}" ${segment.collapsed === false ? "open" : ""}><summary class="aparte-accordion__header aparte-thinking-header"><span class="aparte-thinking-label">${escapeHtml(segment.label || contextConfig().t("thinking"))}</span>${contextConfig().getIcon("expand")}</summary><div class="aparte-accordion__panel aparte-thinking-content">${contextConfig().renderMarkdown(segment.content)}</div></details>`,
+  render: (segment) => `<details class="aparte-segment aparte-segment-thinking aparte-accordion__item" data-segment-id="${escapeHtml(segment.id)}" ${segment.collapsed === false ? "open" : ""}><summary class="aparte-accordion__header aparte-thinking-header"><span class="aparte-thinking-label">${escapeHtml(segment.label || contextConfig().t("thinking"))}</span><span class="aparte-accordion__icon">${contextConfig().getIcon("expand")}</span></summary><div class="aparte-accordion__panel aparte-thinking-content" role="region" tabindex="0" aria-label="${escapeAttr(segment.label || contextConfig().t("thinking"))}">${contextConfig().renderMarkdown(segment.content)}</div></details>`,
   /**
    * The default label is the only config-derived text here — and `segment.label`
    * still wins, exactly as in `render`, because that string is the app's.
    */
   relabel: (el2, segment) => {
+    const text = segment.label || contextConfig().t("thinking");
     const label = el2.querySelector(".aparte-thinking-label");
-    if (label) label.textContent = segment.label || contextConfig().t("thinking");
+    if (label) label.textContent = text;
+    el2.querySelector(".aparte-thinking-content")?.setAttribute("aria-label", text);
   },
   update: (el2, segment) => {
     const contentEl = el2.querySelector(".aparte-thinking-content");
@@ -2585,6 +2588,10 @@ function streamHighlight(element, paneSelector, content, lang, segId) {
   }).catch(() => {
   });
 }
+function nameButton(button, label) {
+  button.setAttribute("aria-label", label);
+  button.setAttribute("title", label);
+}
 const codeRenderer = {
   type: "code",
   render: (segment) => `
@@ -2592,7 +2599,7 @@ const codeRenderer = {
             <div class="aparte-code-header">
                 ${segment.filename ? `<span class="aparte-code-filename">${escapeHtml(segment.filename)}</span>` : `<span class="aparte-code-header-filler"></span>`}
                 <span class="aparte-code-language">${escapeHtml(segment.language || "")}</span>
-                <button class="aparte-btn aparte-btn--icon aparte-btn--sm aparte-code-copy" data-action="copy" title="${escapeAttr(contextConfig().t("copy"))}">
+                <button type="button" class="aparte-btn aparte-btn--icon aparte-btn--sm aparte-code-copy" data-action="copy" aria-label="${escapeAttr(contextConfig().t("copy"))}" title="${escapeAttr(contextConfig().t("copy"))}">
                     ${contextConfig().getIcon("copy")}
                 </button>
             </div>
@@ -2605,7 +2612,7 @@ const codeRenderer = {
   relabel: (element) => {
     const copyBtn = element.querySelector(".aparte-code-copy");
     if (!copyBtn || copyBtn.dataset.copied) return;
-    copyBtn.setAttribute("title", contextConfig().t("copy"));
+    nameButton(copyBtn, contextConfig().t("copy"));
     copyBtn.innerHTML = contextConfig().getIcon("copy");
   },
   setup: (element, segment) => {
@@ -2623,11 +2630,11 @@ const codeRenderer = {
         void copyText(source).catch(() => {
         });
         copyBtn.innerHTML = contextConfig(copyBtn).getIcon("check");
-        copyBtn.setAttribute("title", contextConfig(copyBtn).t("copied"));
+        nameButton(copyBtn, contextConfig(copyBtn).t("copied"));
         copyBtn.dataset.copied = "1";
         setTimeout(() => {
           copyBtn.innerHTML = contextConfig(copyBtn).getIcon("copy");
-          copyBtn.setAttribute("title", contextConfig(copyBtn).t("copy"));
+          nameButton(copyBtn, contextConfig(copyBtn).t("copy"));
           delete copyBtn.dataset.copied;
         }, 1500);
       });
@@ -2662,13 +2669,11 @@ const errorRenderer = {
     }
     return `
         <div class="aparte-segment aparte-alert aparte-alert--danger aparte-segment-error" data-segment-id="${escapeHtml(segment.id)}">
-            <div class="aparte-error-icon-wrapper">
-                ${contextConfig().getIcon("error") || "⚠"}
-            </div>
-            <div class="aparte-error-content">
-                <div class="aparte-error-title">${escapeHtml(contextConfig().t("error"))}</div>
-                <div class="aparte-error-message">${escapeHtml(segment.content)}</div>
-                ${segment.details ? `<div class="aparte-error-details">${escapeHtml(segment.details)}</div>` : ""}
+            <span class="aparte-alert__icon">${contextConfig().getIcon("error") || "⚠"}</span>
+            <div class="aparte-alert__body">
+                <div class="aparte-alert__title">${escapeHtml(contextConfig().t("error"))}</div>
+                <div class="aparte-alert__message">${escapeHtml(segment.content)}</div>
+                ${segment.details ? `<pre class="aparte-segment-error__details">${escapeHtml(segment.details)}</pre>` : ""}
             </div>
         </div>
     `;
@@ -2683,9 +2688,9 @@ const errorRenderer = {
    * language it arrived in. Relabelling it would be inventing content.
    */
   relabel: (element) => {
-    const wrap = element.querySelector(".aparte-error-icon-wrapper");
+    const wrap = element.querySelector(".aparte-alert__icon");
     if (wrap) wrap.innerHTML = contextConfig().getIcon("error") || "⚠";
-    const title = element.querySelector(".aparte-error-title");
+    const title = element.querySelector(".aparte-alert__title");
     if (title) title.textContent = contextConfig().t("error");
   },
   getStyles: () => ``
@@ -2705,7 +2710,7 @@ function stateBadge(segment) {
   const cfg = contextConfig();
   const status = segment.status ?? "pending";
   const word = (key) => escapeHtml(cfg.t(key));
-  if (status === "awaiting-approval") return word("approvalWaiting");
+  if (status === "awaiting-approval") return `${cfg.getIcon("pause")}${word("approvalWaiting")}`;
   if (status === "pending") return word("toolRunning");
   if (status === "resolved") return `${cfg.getIcon("check")}${word("toolCompleted")}`;
   if (status === "rejected") return `${cfg.getIcon("close")}${word("toolRejected")}`;
@@ -4380,13 +4385,13 @@ class AparteChatHost {
     });
   }
 }
-function dispatchLifecycleEvent(target, name, detail) {
-  const id = target.id || void 0;
+function dispatchLifecycleEvent(target, name, detail, targetId) {
+  const id = targetId ?? (target.id || void 0);
   const stamped = detail && typeof detail === "object" ? { targetId: id, ...detail } : detail;
   target.dispatchEvent(new CustomEvent(name, { bubbles: true, composed: true, detail: stamped }));
 }
 function createStreamAdapter(opts) {
-  const { target, config, messageId } = opts;
+  const { target, config, messageId, targetId } = opts;
   let parser = new AparteStreamParser({ blocks: config.getStreamBlocks() });
   let streaming = /* @__PURE__ */ new Set();
   let thinkingId = null;
@@ -4474,7 +4479,7 @@ function createStreamAdapter(opts) {
       }
       case "tool-awaiting-approval":
         target.updateSegment?.(`tool-${e.toolCallId}`, { status: "awaiting-approval" });
-        dispatchLifecycleEvent(target, "aparte-tool-approval-request", { toolCallId: e.toolCallId, toolName: e.name, input: e.input });
+        dispatchLifecycleEvent(target, "aparte-tool-approval-request", { toolCallId: e.toolCallId, toolName: e.name, input: e.input }, targetId);
         break;
       case "tool-approved":
         target.updateSegment?.(`tool-${e.toolCallId}`, { status: "pending" });
@@ -4508,7 +4513,7 @@ function createStreamAdapter(opts) {
         }
         break;
       case "run-aborted":
-        dispatchLifecycleEvent(target, "aparte-message-aborted", { messageId });
+        dispatchLifecycleEvent(target, "aparte-message-aborted", { messageId }, targetId);
         break;
       case "run-done":
         target.updateMessage?.(messageId, { status: "completed" });
@@ -4685,12 +4690,20 @@ function buildEnumField(field, onChange, fallbackLabel, settle) {
   wrap.appendChild(list);
   const controls = () => Array.from(list.querySelectorAll(".aparte-elic-control"));
   const other = () => list.querySelector('input[value="__other__"]');
+  let arrowed = false;
+  list.addEventListener("keydown", (e) => {
+    arrowed = e.key.startsWith("Arrow");
+  });
+  list.addEventListener("click", () => {
+    arrowed = false;
+  });
   list.addEventListener("change", () => {
     const o = other();
     if (o && otherText) {
       otherText.style.display = o.checked ? "" : "none";
-      if (o.checked) otherText.focus();
+      if (o.checked && !arrowed) otherText.focus();
     }
+    arrowed = false;
     onChange();
   });
   otherText?.addEventListener("input", onChange);
@@ -5308,6 +5321,7 @@ class AparteClient {
    * sequence, so it can't drift between the three entry points.
    */
   async _streamTurn(targetElement, messageId, provider, messages, modelId, authConfig, opts) {
+    const chatId = this._targetIdOf(targetElement);
     const registeredTools = this._toolsForCurrentModel();
     let baseRequest = {
       messages,
@@ -5323,22 +5337,22 @@ class AparteClient {
       baseRequest = { ...baseRequest, tools: void 0 };
     }
     if (this._isAborted) {
-      dispatchLifecycleEvent(targetElement, "aparte-message-aborted", { messageId });
+      dispatchLifecycleEvent(targetElement, "aparte-message-aborted", { messageId }, chatId);
       this._updateMessage(targetElement, messageId, { status: "completed" });
       return;
     }
-    dispatchLifecycleEvent(targetElement, "aparte-message-start", { messageId, role: "assistant" });
+    dispatchLifecycleEvent(targetElement, "aparte-message-start", { messageId, role: "assistant" }, chatId);
     const streamController = new AbortController();
     try {
       const usage = await this._streamLoop(targetElement, messageId, provider, baseRequest, authConfig, streamController);
       if (this._isAborted || streamController.signal.aborted) {
         this._updateMessage(targetElement, messageId, { status: "completed" });
       } else {
-        dispatchLifecycleEvent(targetElement, "aparte-message-done", { messageId, role: "assistant", usage });
+        dispatchLifecycleEvent(targetElement, "aparte-message-done", { messageId, role: "assistant", usage }, chatId);
       }
     } catch (error) {
       if (this._isAborted || streamController.signal.aborted) {
-        dispatchLifecycleEvent(targetElement, "aparte-message-aborted", { messageId });
+        dispatchLifecycleEvent(targetElement, "aparte-message-aborted", { messageId }, chatId);
         this._updateMessage(targetElement, messageId, { status: "completed" });
         return;
       }
@@ -5526,13 +5540,64 @@ class AparteClient {
     return null;
   }
   /**
-   * Convert AparteMessage[] to AparteChatMessage[] for re-submission.
+   * Which chat a render target BELONGS to — the id the lifecycle events must carry.
+   *
+   * Not `target.id`: the target is whatever renders, and an `<aparte-chat>` shell
+   * delegates that to its `.viewport`, which has no id. So every lifecycle event on
+   * a shell-shaped chat went out unstamped, and the receive side reads a missing id
+   * as "for every chat on the page" — deliberately, so a single-chat page needs no
+   * wiring. On a two-chat page one chat's turn therefore drove every composer.
+   *
+   * The climb is the same rule `aparte-composer._ownTargetId` uses on the receive
+   * side, and it has to be, or the two would disagree about which chat is which:
+   * the nearest `aparte-chat` / `[data-aparte-chat]` ancestor (or the element
+   * itself) that HAS an id. Deliberately not `chatBoundaryOf` — that returns the
+   * boundary whether or not it is identified, and an id-less boundary must fall
+   * through to the next one up rather than answer "no id".
+   */
+  _targetIdOf(el2) {
+    let node = el2 ?? null;
+    while (node) {
+      const tag = node.tagName?.toLowerCase();
+      const isHost = tag === "aparte-chat" || node.hasAttribute?.("data-aparte-chat");
+      if (isHost && node.id) return node.id;
+      node = node.parentElement;
+    }
+    return el2?.id || void 0;
+  }
+  /**
+   * Convert AparteMessage[] to AparteChatMessage[] for re-submission (retry, edit).
+   *
+   * The SLICE is each caller's own business — retry cuts before the reply it is
+   * regenerating, edit cuts after the message being reworded, send cuts at the last
+   * answered turn. What a message CONTRIBUTES to the wire is not: send, retry and
+   * edit must not disagree about it, and they did. This one had no filter at all,
+   * so a failed turn and a turn stopped before its first token both went out as
+   * `{ role: 'assistant', content: '' }` — a claim the assistant said nothing, which
+   * some providers reject outright and the rest read as an empty reply worth
+   * imitating.
    */
   _messagesToChatMessages(messages) {
-    return messages.filter((m) => m.role === "user" || m.role === "assistant").map((m) => ({
+    return this._wireHistory(messages);
+  }
+  /**
+   * The one rule for what a message contributes to the wire.
+   *
+   * Not the slice — the contribution: a role the wire has (`user`/`assistant`), a
+   * turn that neither failed nor is still in flight, and text that is not empty.
+   *
+   * `_wireText` (not `m.content`): assistant replies stream their text into
+   * `segments`, leaving `content` as '', so without flattening every reply would be
+   * empty here and the drop below would swallow the whole transcript.
+   */
+  _wireHistory(messages) {
+    return messages.filter((m) => {
+      if (m.role !== "user" && m.role !== "assistant") return false;
+      return m.status !== "error" && !AparteClient._inFlight(m);
+    }).map((m) => ({
       role: m.role,
       content: this._wireText(m)
-    }));
+    })).filter((m) => m.content.length > 0);
   }
   /** The "no model selected" warning is said once per client, not once per dropped send. */
   _warnedNoModel = false;
@@ -5666,22 +5731,30 @@ class AparteClient {
     messages.push({ role: "user", content: userMsg });
     return messages;
   }
+  /**
+   * A turn that has not finished yet. `status` is OPTIONAL on `AparteMessage`, so
+   * the question the history gates can ask is "is this still in flight?", never
+   * "is this completed?".
+   *
+   * The gates below used to ask the second one, and a host that SEEDS a transcript
+   * — restoring a saved conversation, hydrating a server-rendered one — has no
+   * reason to invent a status for turns that are already over. With none, the
+   * cutoff never advanced past 0 and the whole transcript was sliced away: the
+   * model got the new question with no context, and nothing in the UI said so,
+   * because the viewport still showed every message.
+   */
+  static _inFlight(m) {
+    return m.status === "streaming" || m.status === "pending";
+  }
   _toHistoryMessages(messages) {
     let cutoff = 0;
     for (let i = 0; i < messages.length; i++) {
       const m = messages[i];
-      if (m.role === "assistant" && m.status === "completed") {
+      if (m.role === "assistant" && m.status !== "error" && !AparteClient._inFlight(m)) {
         cutoff = i + 1;
       }
     }
-    return messages.slice(0, cutoff).filter((m) => {
-      if (m.role === "user") return m.status !== "error";
-      if (m.role === "assistant") return m.status === "completed";
-      return false;
-    }).map((m) => ({
-      role: m.role,
-      content: this._wireText(m)
-    })).filter((m) => m.content.length > 0);
+    return this._wireHistory(messages.slice(0, cutoff));
   }
   /**
    * A message's text as the MODEL should read it — the one mapper both history
@@ -5854,10 +5927,12 @@ ${content}
    */
   async _runViaStreamRunner(streamRunner, targetElement, messageId, provider, baseRequest, authConfig, streamController) {
     const signal = streamController.signal;
+    const chatId = this._targetIdOf(targetElement);
     const adapter = createStreamAdapter({
       target: targetElement,
       config: this._config,
-      messageId
+      messageId,
+      ...chatId ? { targetId: chatId } : {}
     });
     const prefixSegments = baseRequest._meta?.["prefixSegments"];
     const emitter = (event) => {
@@ -5950,7 +6025,7 @@ ${content}
         segments: [errorSegment]
       });
     }
-    dispatchLifecycleEvent(target, "aparte-message-error", { messageId, error });
+    dispatchLifecycleEvent(target, "aparte-message-error", { messageId, error }, this._targetIdOf(target));
   }
 }
 function findHead(node) {
@@ -6264,7 +6339,7 @@ function applyElementProps(el2, props, transformValue = (value) => value) {
       if (value === null || value === void 0) el2.style.removeProperty(key);
       else if (typeof value === "object" || typeof value === "function") el2.style.removeProperty(key);
       else el2.style.setProperty(key, String(value));
-    } else if (key.startsWith("on")) {
+    } else if (key.toLowerCase().startsWith("on")) {
       el2.removeAttribute(key);
     } else if (value === null || value === void 0 || value === false) {
       el2.removeAttribute(key);
@@ -6402,4 +6477,4 @@ export {
   collectRendererStyles as y,
   contentToText as z
 };
-//# sourceMappingURL=dialog-triggers-Dw_DIlQ7.js.map
+//# sourceMappingURL=dialog-triggers-CApC7K1I.js.map