@aparte/core 0.19.2 → 0.19.3

package/dist/{dialog-triggers-CEazY76J.js → dialog-triggers-ChnPFG9M.js}

@@ -84,7 +84,7 @@ const APARTE_DEFAULT_LOCALE = {
   copy: "Copy",
   copied: "Copied!",
   retry: "Retry",
-  thinking: "Thinking...",
+  thinking: "Thinking",
   elicitationOther: "Other…",
   elicitationOtherPlaceholder: "Type your answer…",
   elicitationOtherLabel: "Custom answer",
@@ -124,7 +124,7 @@ const APARTE_DEFAULT_LOCALE = {
   code: "Code",
   generating: "Generating…",
   rebuildingPreview: "Rebuilding preview…",
-  typing: "Typing...",
+  typing: "Typing",
   error: "Error",
   running: "Running...",
   run: "Run",
@@ -148,6 +148,7 @@ const APARTE_DEFAULT_LOCALE = {
   messageInfo: "Details",
   newChat: "New Chat",
   loadingConversations: "Loading conversations",
+  conversationList: "Conversations",
   loadingConversation: "Loading the conversation",
   deleteConversation: "Delete",
   archiveConversation: "Archive",
@@ -698,8 +699,7 @@ class AparteStreamParser {
     return {
       id: this._options.autoGenerateIds ? this._generateId() : "",
       type: "thinking",
-      content,
-      label: "Thinking"
+      content
     };
   }
   _isPartialMatch(buffer, pattern) {
@@ -1062,8 +1062,7 @@ function createStreamAdapter(opts) {
             id: `think-${uuid()}`,
             type: "thinking",
             content: thinkingContent,
-            collapsed: true,
-            label: "Thinking"
+            collapsed: true
           };
           thinkingId = seg.id;
           streaming.add(seg.id);
@@ -2054,7 +2053,7 @@ class AparteConfig {
    * @param sanitizer A sanitizer function (e.g. a DOMPurify wrapper) for
    *   hardened coverage, or `null` to DISABLE sanitization. Disabling exposes
    *   you to XSS from LLM-authored content — only do so for content you fully
-   *   trust and have already sanitized upstream.
+   *   trust and have already sanitized upstream. `null` prints a console warning, once.
    * @example aparteGlobalConfig.setHtmlSanitizer((html) => DOMPurify.sanitize(html));
    */
   setHtmlSanitizer(sanitizer) {
@@ -2275,26 +2274,16 @@ class AparteConfig {
     this._keyProvider = provider;
   }
   /**
-   * Add a key source, without replacing the one `setKeyProvider` holds. Returns a
-   * teardown.
-   *
-   * This is how `AparteClient({ keyResolver })` — the channel the docs teach as
-   * primary — reaches everything that needs a key rather than the chat alone. It
-   * used to reach the chat ONLY, so the model selector, whose one data path is
-   * `refreshProviderModels`, showed an empty list on every cloud provider while
-   * the chat worked: `fetchModels` returns `[]` without a key, so there was
-   * nothing to see and nothing said.
-   *
-   * The direction matters (decision #9): the client registers a source ON the
-   * config, so the capability is never hostage to the client — a page that only
-   * ever calls `setKeyProvider` is unchanged, and a page that constructs no
-   * client at all still lists models.
-   *
-   * Every call is its own source, even when the same function is handed in
-   * twice: two panes sharing one module-level resolver are two mounts, and the
-   * first one to unmount must not take the second one's key with it. A `Set`
-   * keyed by the function itself made them one entry, which either teardown
-   * removed.
+   * Add a key source beside the one {@link AparteConfig.setKeyProvider | setKeyProvider()}
+   * holds, and return the function that removes it. The sources are asked in the order they
+   * were added, then `setKeyProvider`'s, and the first answer wins. `AparteClient`'s
+   * `keyResolver` is registered this way, so the model selector can list a cloud provider's
+   * models too. Each call is a source of its own, even with the same function.
+   * @example
+   * const release = aparteGlobalConfig.registerKeyProvider((providerId) =>
+   *     providerId === 'openrouter' ? localStorage.getItem('openrouter-key') : undefined);
+   * // when what owns the key goes away:
+   * release();
    */
   registerKeyProvider(provider) {
     const entry = (providerId) => provider(providerId);
@@ -2330,9 +2319,10 @@ class AparteConfig {
     return void 0;
   }
   /**
-   * Refresh models for a specific provider
-   * Orchestrates: Key Retrieval -> Fetch -> Return
-   * This keeps UI components unaware of keys.
+   * Fetch a provider's models, with the key and endpoint its key sources answer, so a
+   * component that lists models never handles a key. The list is kept, and
+   * `getCurrentModel()` reads the selected model's capabilities from it. Answers `[]` for an
+   * unknown provider, one with no `fetchModels`, or a fetch that fails (with a console warning).
    */
   async refreshProviderModels(providerId) {
     const provider = this._aiProviders.get(providerId);
@@ -2351,7 +2341,13 @@ class AparteConfig {
   // AI Provider Management (BYORK - Bring Your Own Key)
   // ─────────────────────────────────────────────────────────────────────────
   /**
-   * Register one or more AI providers (e.g., OpenRouter, Gemini, Anthropic)
+   * Register one or more AI providers (e.g., OpenRouter, Gemini, Anthropic).
+   *
+   * When this leaves exactly one provider registered, offering exactly one model it knows
+   * without a request (a scripted or an in-browser provider), and no model is selected,
+   * that model is selected. A provider that lists its models over the network selects
+   * nothing: name one with {@link AparteConfig.setModelConfig | setModelConfig()}, or let the
+   * person pick in `<aparte-model-selector>`.
    * @param providers AparteAIProvider implementations
    * @example
    * aparteGlobalConfig.registerAIProvider(OpenRouterProvider);
@@ -2469,7 +2465,8 @@ class AparteConfig {
   /**
    * Restore previously saved model preference via the registered provider.
    * Should be called once at app startup, before any component mounts.
-   * No-op if no provider is registered or nothing was saved.
+   * No-op if no provider is registered or nothing was saved. The saved selection is
+   * applied without being saved again.
    */
   restoreModelPreference() {
     if (!this._modelPreferenceProvider) return null;
@@ -2482,7 +2479,8 @@ class AparteConfig {
     return null;
   }
   /**
-   * Set model selection configuration
+   * Select the default provider and model, merged over the current selection. With a
+   * model-preference provider registered and both set, the selection is saved through it.
    */
   setModelConfig(config) {
     this._modelConfig = { ...this._modelConfig, ...config };
@@ -2526,12 +2524,6 @@ class AparteConfig {
   hasSelectedModel() {
     return !!(this._modelConfig.defaultProvider && this._modelConfig.defaultModel);
   }
-  /**
-   * Opt-in UX: when enabled, `<aparte-composer>` blocks sending and greys out
-   * until {@link hasSelectedModel} is true (e.g. while the model selector is
-   * still fetching its list). Off by default so single-model / backend setups
-   * that never select a model are unaffected.
-   */
   /**
    * Elicitation policy: whether a choice offers a free-text "Other…" escape.
    *
@@ -2590,6 +2582,12 @@ class AparteConfig {
   getElicitationFieldRenderer() {
     return this._elicitationFieldRenderer;
   }
+  /**
+   * Opt-in UX: when enabled, `<aparte-composer>` blocks sending and greys out
+   * until {@link hasSelectedModel} is true (e.g. while the model selector is
+   * still fetching its list). Off by default so single-model / backend setups
+   * that never select a model are unaffected.
+   */
   setRequireModelSelection(required) {
     if (this._requireModelSelection === required) return;
     this._requireModelSelection = required;
@@ -2625,7 +2623,8 @@ class AparteConfig {
   // Public Rendering Methods (with Fallbacks)
   // ─────────────────────────────────────────────────────────────────────────
   /**
-   * Get translated string by key
+   * The locale's string for `key`, or the default (English) locale's when the locale
+   * leaves it out; `''` for a key neither declares.
    */
   t(key) {
     const val = this._locale[key] || APARTE_DEFAULT_LOCALE[key];
@@ -2665,7 +2664,7 @@ class AparteConfig {
   }
   /**
    * Highlight code block
-   * Fallback: Returns raw code wrapped in <pre><code>
+   * Fallback: Returns raw code wrapped in `<pre><code>`
    * Supports Promise if provider is async/streaming
    */
   async highlightCode(code, lang) {
@@ -2701,18 +2700,11 @@ class AparteConfig {
     return Array.from(this._tools.values()).map((e) => e.tool);
   }
   /**
-   * The registered tools' own `systemPrompt`s, joined, or `null` if none set one.
-   *
-   * `AparteTool.systemPrompt` has always been documented as "injected automatically when
-   * this tool is registered", on the type and in the tools guide — and nothing read it.
-   * A field that is documented and dead is worse than one that does not exist: a caller
-   * writes it, sees the tool work (the model has the schema either way), and never learns
-   * that the sentence explaining WHEN to reach for it was dropped. `@aparte/plugin-ask-user`
-   * sets one, so a shipped plugin was losing its instructions.
-   *
-   * Registration order, blank line between, because two tools' instructions are two
-   * paragraphs and not one run-on sentence. The app's own template stays a separate
-   * message — see `resolveSystemPrompt`, which is about the app and not about the tools.
+   * The registered tools' own `systemPrompt`s, in registration order with a blank line
+   * between, or `null` if none sets one. `AparteClient` sends them with every request, as a
+   * system message after the app's own ({@link AparteConfig.resolveSystemPrompt | resolveSystemPrompt()});
+   * a request you assemble yourself should send them the same way, or the model loses the
+   * tools' instructions.
    */
   resolveToolSystemPrompts() {
     const prompts = Array.from(this._tools.values()).map((entry) => entry.tool.systemPrompt?.trim()).filter((prompt) => Boolean(prompt));
@@ -2876,6 +2868,12 @@ class AparteConfig {
    * confirmation, a text field, a form) and resolves with the user's answer.
    * The `<aparte-elicitation>` Web Component registers itself here; an app can
    * override with its own framework-native presenter. Pass `null` to clear.
+   *
+   * Several can be registered, one per chat: `owner` is an element of the chat the
+   * presenter belongs to (`<aparte-elicitation>` passes itself). A request whose `target`
+   * is inside that chat reaches its presenter; a request with no `target`, or from a chat
+   * with no presenter of its own, reaches the most recently registered one. Registering a
+   * presenter again makes it the most recent.
    */
   setElicitationPresenter(presenter, owner) {
     if (presenter === null) {
@@ -2927,35 +2925,21 @@ class AparteConfig {
     return stack[stack.length - 1]?.fn;
   }
   /**
-   * Ask the user for typed input mid-run and await their response. This is the
-   * generic primitive behind `ask_user` and tool approval — the KIND of
-   * question is the schema, not a bespoke tool.
-   *
-   * **Resolves** `accept` with the value, or `decline` when the user declines.
-   *
-   * **Rejects** — it does not resolve — when the request ends without an answer, with
-   * an {@link AparteElicitationAbortError} whose `name` is `'AbortError'` and whose
-   * `reason` is `'aborted'` (the turn was stopped, the signal fired, or another request
-   * took the question away) or `'no-presenter'` (nothing was mounted to ask it). So
-   * `await` it inside a `try`, or attach a `.catch()`.
+   * Ask the person for typed input in the middle of a run, and wait for the answer. The
+   * primitive behind `ask_user` and tool approval: the kind of question is its schema.
    *
-   * That is a change, and this block described the old behaviour for a whole release:
-   * "resolves `cancel` when the turn is cancelled", and "with NO presenter registered it
-   * resolves `cancel` rather than hanging". Both were false the moment `cancel` was
-   * removed — and the second is the one that bites, because a developer who reads
-   * "rather than hanging" writes no `.catch()` and gets an unhandled rejection. It ships
-   * in the `.d.ts` and in the custom-elements manifest, which is the API surface the
-   * docs site renders, so it was the most-read wrong sentence in the library.
+   * **Resolves** `accept` with the value, or `decline` when the person declines.
    *
-   * A value was removed rather than renamed on purpose: `cancel` was easy to handle as
-   * though it were an answer, and the approval gate did exactly that — it read a stopped
-   * turn as the user refusing a tool they were never shown.
+   * **Rejects**, with no answer, with an {@link AparteElicitationAbortError} (its `name` is
+   * `'AbortError'`) whose `reason` is `'aborted'` — the turn was stopped, the signal fired,
+   * or another request took the question away — or `'no-presenter'`, when nothing is mounted
+   * to ask it (with a console warning, once). `await` it in a `try`, or attach a `.catch()`.
    *
-   * With no presenter it rejects `'no-presenter'` and warns once, because a request
-   * nobody could see is a setup mistake only the developer can fix. The default
-   * presenter is `<aparte-elicitation>`, which registers itself from its
-   * `connectedCallback` — so it has to be IN THE DOM. A docs page of ours claimed it
-   * "installs itself — nothing to register", which is how that was found.
+   * The question goes to the presenter of the chat that holds `request.target` (see
+   * {@link AparteConfig.setElicitationPresenter | setElicitationPresenter()});
+   * `<aparte-elicitation>` is the default one, and registers itself once it is in the DOM.
+   * A chat shows one question at a time: a second request in the same chat waits until the
+   * first is answered or ends, and requests with no `target` share one queue.
    */
   requestUserInput(request) {
     const queue = this._elicitationQueueKey(request);
@@ -3000,10 +2984,13 @@ class AparteConfig {
     return presenter(request);
   }
   /**
-   * Reset ALL configuration back to defaults — providers, registries, model
-   * selection and bubble actions. Previously left `_aiProviders` / `_tools` /
-   * `_toolRenderers` / `_modelConfig` behind, which leaked across SPA
-   * navigations (registries only ever grew). Now a full reset.
+   * Put the configuration back to its defaults: the providers and the selected model,
+   * the tools with their renderers and summaries, the renderers and render hooks, the
+   * Markdown and highlight providers, the sanitizer, the system prompt, the locale, icons,
+   * avatars, actions and host handlers, the conversation manager, the question presenters
+   * and their queue. Three things stay: the transport, the key sources added with
+   * {@link AparteConfig.registerKeyProvider | registerKeyProvider()} (an `AparteClient`'s
+   * `keyResolver` among them), and the {@link AparteConfig.subscribe | subscribe()} listeners.
    */
   reset() {
     this._markdownProvider = void 0;
@@ -3192,6 +3179,9 @@ const textRenderer = {
   getStyles: () => ``
 };
 const THINKING_ANCHOR_SLACK = 24;
+function labelOf(segment) {
+  return segment.label && segment.label !== "Thinking" ? segment.label : contextConfig().t("thinking");
+}
 const thinkingRenderer = {
   type: "thinking",
   /*
@@ -3222,13 +3212,13 @@ 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><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>`,
+  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(labelOf(segment))}</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(labelOf(segment))}">${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 text = labelOf(segment);
     const label = el2.querySelector(".aparte-thinking-label");
     if (label) label.textContent = text;
     el2.querySelector(".aparte-thinking-content")?.setAttribute("aria-label", text);
@@ -5063,6 +5053,14 @@ class AparteChatHost {
       }
       ac.signal.addEventListener("abort", () => resolve("aborted"), { once: true });
     });
+    const { host, hostId } = this.binding;
+    const onAbort = (e) => {
+      const target = e.detail?.targetId;
+      if ((!target || target === hostId) && this._streamAbort === ac) this.stopTokenStream();
+    };
+    window.addEventListener("aparte-abort", onAbort);
+    const replaced = () => this._streamAbort !== void 0 && this._streamAbort !== ac;
+    dispatchLifecycleEvent(host, "aparte-message-start", { messageId, role: "assistant" }, hostId);
     try {
       for (; ; ) {
         const res = await Promise.race([iterator.next(), aborted]);
@@ -5072,12 +5070,24 @@ class AparteChatHost {
       }
       this._flushStreamState();
       this._vp()?.completeMessage?.(messageId);
-      this._setStreamingId(null);
+      if (!replaced()) {
+        this._setStreamingId(null);
+        dispatchLifecycleEvent(
+          host,
+          ac.signal.aborted ? "aparte-message-aborted" : "aparte-message-done",
+          ac.signal.aborted ? { messageId } : { messageId, role: "assistant" },
+          hostId
+        );
+      }
     } catch (err) {
       this._flushStreamState();
-      this._setStreamingId(null);
+      if (!replaced()) {
+        this._setStreamingId(null);
+        dispatchLifecycleEvent(host, "aparte-message-error", { messageId, error: err }, hostId);
+      }
       throw err;
     } finally {
+      window.removeEventListener("aparte-abort", onAbort);
       if (ac.signal.aborted) {
         try {
           await iterator.return?.();
@@ -5814,6 +5824,7 @@ function el(tag, className, text) {
   if (text != null) node.textContent = text;
   return node;
 }
+let askSeq = 0;
 function buildApprovalPanel(message, options, onChange, details) {
   const cfg = contextConfig();
   const panel = el("div", "aparte-elic-panel aparte-approval-panel");
@@ -5843,6 +5854,10 @@ function buildApprovalPanel(message, options, onChange, details) {
   const row = el("div", "aparte-approval-options");
   row.setAttribute("role", "group");
   row.setAttribute("aria-label", cfg.t("approvalOptionsLabel"));
+  if (ask) {
+    ask.id ||= `aparte-approval-ask-${++askSeq}`;
+    row.setAttribute("aria-describedby", ask.id);
+  }
   const labelled = [];
   const textOf = (option) => typeof option.label === "function" ? option.label() : option.label;
   const descriptionOf = (option) => typeof option.description === "function" ? option.description() : option.description;
@@ -5943,6 +5958,11 @@ class AparteClient {
   options;
   /** Config read by this client — an instance config, or the global default. */
   _config;
+  /**
+   * A `keyResolver` becomes a key source on the client's config at once, before `start()`,
+   * so everything that needs a key can use it — the model selector's list included.
+   * `autoRegister` (on by default) installs core's built-in renderers on that config.
+   */
   constructor(options = {}) {
     this.options = {
       autoRegister: true,
@@ -6004,7 +6024,9 @@ class AparteClient {
     };
   }
   /**
-   * Start listening for aparte-send events on the window.
+   * Answer the chats: listen on `window` for `aparte-send`, `aparte-retry`, `aparte-edit`,
+   * `aparte-abort` and `aparte-conversation-loaded`. Nothing streams before this is
+   * called. Starting a stopped client puts its `keyResolver` back on the config.
    */
   start() {
     this._registerKeyResolver();
@@ -6047,7 +6069,8 @@ class AparteClient {
     window.addEventListener("aparte-conversation-loaded", this._boundLoadedHandler);
   }
   /**
-   * Stop listening.
+   * Stop answering: abort the reply in flight, take this client's `keyResolver` off the
+   * config, and remove the listeners.
    */
   stop() {
     this.abort();
@@ -7728,4 +7751,4 @@ export {
   collectRendererStyles as y,
   contentToText as z
 };
-//# sourceMappingURL=dialog-triggers-CEazY76J.js.map
+//# sourceMappingURL=dialog-triggers-ChnPFG9M.js.map