@noodleseed/assistant 1.23.0 → 1.25.0

package/README.md

@@ -35,8 +35,8 @@ troubleshooting) lives at <https://docs.noodleseed.dev/guides/embedded-assistant
 the server itself is `@noodleseed/one` (`npm install -g @noodleseed/one`).
 
 If your page sends a `Content-Security-Policy`, allow the Noodle service origin in `connect-src` (turns and
-event streams) **and** `frame-src` (app widgets render inside a hosted sandbox document served from the
-service origin, `endpoints.sandbox`, so your `script-src` can stay strict). Details are in the guide's
+event streams) and the exact session-advertised `endpoints.sandbox` origin in `frame-src` (it may be a
+dedicated sandbox origin, so your `script-src` can stay strict). Details are in the guide's
 "Content-Security-Policy on your page" section.
 
 ## Quick start
@@ -410,13 +410,8 @@ import "@noodleseed/assistant/app-view";
 </template>
 ```
 
-Configure Vue's `isCustomElement` for `noodle-app-view`. Angular uses the same element with `[client]`,
-`[view]`, and `[theme]` property bindings. Do not serialize `client` or `view` into attributes.
-
-`<noodle-app-view>` is the canonical host; `NoodleAppView` delegates to it. Both use the
-service-advertised sandbox URL, route App calls through the supplied client, publish theme changes without
-replacing the iframe, and request standard teardown on semantic replacement, disconnect, or an App teardown
-request.
+Framework setup, lifecycle ownership, sandbox/CSP behavior, and staged diagnostics are owned by the
+[Bring your own UI guide](https://docs.noodleseed.dev/docs/guides/bring-your-own-ui).
 
 The App document owns its action intent: it calls standard `tools/call` after connecting and never relies on
 native form navigation. If that call needs input or confirmation, the same `client` publishes the normal

package/dist/{app-view-element-7ZQZLYbL.d.ts → app-view-element-9ojWWYov.d.cts}

@@ -1,9 +1,10 @@
-import { A as AssistantClient, a as AssistantViewAvailableDetail, e as AssistantErrorDetail } from './client-Bp_VKpMg.js';
+import { A as AssistantClient, a as AssistantViewAvailableDetail, e as AssistantErrorDetail } from './client-C07ppTU_.cjs';
 
 declare const APP_VIEW_TAG_NAME = "noodle-app-view";
 interface AssistantAppViewErrorDetail {
-    readonly code: 'view_render_timeout';
+    readonly code: 'view_render_timeout' | 'view_bridge_error';
     readonly retryable: false;
+    readonly stage: 'sandbox_load' | 'sandbox_proxy' | 'app_initialize';
 }
 declare global {
     interface HTMLElementTagNameMap {

package/dist/{app-view-element-1i_w6FDf.d.cts → app-view-element-BM4lral-.d.ts}

@@ -1,9 +1,10 @@
-import { A as AssistantClient, a as AssistantViewAvailableDetail, e as AssistantErrorDetail } from './client-uo57Swlm.cjs';
+import { A as AssistantClient, a as AssistantViewAvailableDetail, e as AssistantErrorDetail } from './client-BkogON2A.js';
 
 declare const APP_VIEW_TAG_NAME = "noodle-app-view";
 interface AssistantAppViewErrorDetail {
-    readonly code: 'view_render_timeout';
+    readonly code: 'view_render_timeout' | 'view_bridge_error';
     readonly retryable: false;
+    readonly stage: 'sandbox_load' | 'sandbox_proxy' | 'app_initialize';
 }
 declare global {
     interface HTMLElementTagNameMap {

package/dist/app-view.cjs

@@ -15197,7 +15197,7 @@ __export(app_view_exports, {
 });
 module.exports = __toCommonJS(app_view_exports);
 
-// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/types.js
+// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_supports-color@7.2.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/types.js
 init_v4();
 var RELATED_TASK_META_KEY = "io.modelcontextprotocol/related-task";
 var JSONRPC_VERSION = "2.0";
@@ -16718,7 +16718,7 @@ var UrlElicitationRequiredError = class extends McpError {
 // ../../node_modules/.pnpm/zod@4.4.3/node_modules/zod/v4/mini/parse.js
 init_core2();
 
-// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/server/zod-compat.js
+// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_supports-color@7.2.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/server/zod-compat.js
 function isZ4Schema(s2) {
   const schema = s2;
   return !!schema._zod;
@@ -16781,7 +16781,7 @@ function getLiteralValue(schema) {
   return void 0;
 }
 
-// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/experimental/tasks/interfaces.js
+// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_supports-color@7.2.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/experimental/tasks/interfaces.js
 function isTerminal(status) {
   return status === "completed" || status === "failed" || status === "cancelled";
 }
@@ -16789,7 +16789,7 @@ function isTerminal(status) {
 // ../../node_modules/.pnpm/zod-to-json-schema@3.25.2_zod@4.4.3/node_modules/zod-to-json-schema/dist/esm/parsers/string.js
 var ALPHA_NUMERIC = new Set("ABCDEFGHIJKLMNOPQRSTUVXYZabcdefghijklmnopqrstuvxyz0123456789");
 
-// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/server/zod-json-schema-compat.js
+// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_supports-color@7.2.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/server/zod-json-schema-compat.js
 function getMethodLiteral(schema) {
   const shape = getObjectShape(schema);
   const methodSchema = shape?.method;
@@ -16810,7 +16810,7 @@ function parseWithCompat(schema, data) {
   return result.data;
 }
 
-// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/shared/protocol.js
+// ../../node_modules/.pnpm/@modelcontextprotocol+sdk@1.29.0_supports-color@7.2.0_zod@4.4.3/node_modules/@modelcontextprotocol/sdk/dist/esm/shared/protocol.js
 var DEFAULT_REQUEST_TIMEOUT_MSEC = 6e4;
 var Protocol = class {
   constructor(_options) {
@@ -17922,6 +17922,15 @@ var E = class {
   sessionId;
   setProtocolVersion;
 };
+function AZ(X) {
+  if (!X) return "";
+  let Y = [];
+  if (X.camera) Y.push("camera");
+  if (X.microphone) Y.push("microphone");
+  if (X.geolocation) Y.push("geolocation");
+  if (X.clipboardWrite) Y.push("clipboard-write");
+  return Y.join("; ");
+}
 var KX = [j];
 var DX = class extends _ {
   _client;
@@ -18192,34 +18201,131 @@ function GX(X, Y) {
   return JSON.stringify(X) === JSON.stringify(Y);
 }
 
+// src/app-sandbox-policy.ts
+var MAX_DOMAINS_PER_DIRECTIVE = 32;
+function isRecord(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function isLoopback(hostname3) {
+  return hostname3 === "localhost" || hostname3 === "127.0.0.1" || hostname3 === "[::1]";
+}
+function normalizeDomain(value, connect) {
+  if (value.length > 512 || /[;\r\n'"\s]/u.test(value)) return void 0;
+  try {
+    const parsed = new URL(value);
+    const secure = parsed.protocol === "https:" || connect && parsed.protocol === "wss:";
+    const local = isLoopback(parsed.hostname) && (parsed.protocol === "http:" || connect && parsed.protocol === "ws:");
+    if (!secure && !local) return void 0;
+    if (parsed.username || parsed.password || parsed.pathname !== "/" || parsed.search || parsed.hash) {
+      return void 0;
+    }
+    return parsed.origin;
+  } catch {
+    return void 0;
+  }
+}
+function normalizeDomains(values, connect) {
+  const normalized = [
+    ...new Set(
+      values.slice(0, MAX_DOMAINS_PER_DIRECTIVE).map((value) => typeof value === "string" ? normalizeDomain(value, connect) : void 0).filter((value) => value !== void 0)
+    )
+  ];
+  return normalized;
+}
+function resolveAssistantAppSandboxPolicy(resourceMeta) {
+  if (!isRecord(resourceMeta)) return {};
+  const rawCsp = resourceMeta.csp;
+  const csp = isRecord(rawCsp) ? {
+    ...Array.isArray(rawCsp.connectDomains) ? { connectDomains: normalizeDomains(rawCsp.connectDomains, true) } : {},
+    ...Array.isArray(rawCsp.resourceDomains) ? { resourceDomains: normalizeDomains(rawCsp.resourceDomains, false) } : {},
+    ...Array.isArray(rawCsp.frameDomains) ? { frameDomains: normalizeDomains(rawCsp.frameDomains, false) } : {},
+    ...Array.isArray(rawCsp.baseUriDomains) ? { baseUriDomains: normalizeDomains(rawCsp.baseUriDomains, false) } : {}
+  } : void 0;
+  const rawPermissions = resourceMeta.permissions;
+  const permissions = isRecord(rawPermissions) ? {
+    ...isRecord(rawPermissions.camera) ? { camera: {} } : {},
+    ...isRecord(rawPermissions.microphone) ? { microphone: {} } : {},
+    ...isRecord(rawPermissions.geolocation) ? { geolocation: {} } : {},
+    ...isRecord(rawPermissions.clipboardWrite) ? { clipboardWrite: {} } : {}
+  } : void 0;
+  const allowAttribute = AZ(permissions);
+  return {
+    ...csp === void 0 ? {} : { csp },
+    ...permissions === void 0 ? {} : { permissions },
+    ...allowAttribute ? { allowAttribute } : {}
+  };
+}
+function assistantAppSandboxUrl(sandboxUrl, policy) {
+  const parsed = new URL(sandboxUrl);
+  if (policy.csp) parsed.searchParams.set("csp", JSON.stringify(policy.csp));
+  const parentOrigin = globalThis.location?.origin;
+  if (parentOrigin && parentOrigin !== "null") {
+    parsed.hash = new URLSearchParams({ parentOrigin }).toString();
+  }
+  return parsed.href;
+}
+
 // src/app-host.ts
-var PROXY_DOCUMENT = `<!doctype html><meta charset="utf-8"><style>html,body,iframe{border:0;margin:0;width:100%;height:100%;overflow:hidden}body{background:transparent}</style><script>
-let inner;
+var PROXY_DOCUMENT = `<!doctype html><meta charset="utf-8"><style>html,body,iframe{border:0;margin:0;width:100%;height:100%;overflow:hidden}body{background:transparent}</style><body><script>
+let inner,loaded=false;
+if(self===top)throw new Error('The assistant sandbox relay must be embedded');
+const standardSandbox='allow-scripts allow-same-origin allow-forms';
+const rawParentOrigin=new URLSearchParams(location.hash.slice(1)).get('parentOrigin');
+let expectedParentOrigin;
+try{const parsed=new URL(rawParentOrigin||'');if(parsed.origin===rawParentOrigin&&/^https?:$/.test(parsed.protocol))expectedParentOrigin=parsed.origin}catch{}
+const parentTarget=expectedParentOrigin||'*';
+const ownOrigin=self.origin;
 const ready={jsonrpc:'2.0',method:'ui/notifications/sandbox-proxy-ready',params:{}};
 // Re-announce until the host acknowledges with the resource: when this document is hosted
 // cross-origin, the first announcement can arrive before the host's load handler is listening.
-const announce=setInterval(()=>parent.postMessage(ready,'*'),120);
+const announce=setInterval(()=>parent.postMessage(ready,parentTarget),120);
 setTimeout(()=>clearInterval(announce),15000);
 addEventListener('message',(event)=>{
   if(event.source===parent){
+    if(expectedParentOrigin&&event.origin!==expectedParentOrigin)return;
     const message=event.data;
     if(message?.method==='ui/notifications/sandbox-resource-ready'){
       clearInterval(announce);
-      if(inner)return;
+      if(loaded)return;
+      loaded=true;
+      const requestedSandbox=message.params?.sandbox;
       inner=document.createElement('iframe');
-      inner.setAttribute('sandbox',message.params?.sandbox||'allow-scripts');
+      inner.setAttribute('sandbox',requestedSandbox===standardSandbox?standardSandbox:'allow-scripts');
       inner.setAttribute('referrerpolicy','no-referrer');
-      inner.srcdoc=String(message.params?.html||'');
-      document.body.replaceChildren(inner);
+      const permissions=message.params?.permissions;
+      const allow=[];
+      if(permissions?.camera)allow.push('camera');
+      if(permissions?.microphone)allow.push('microphone');
+      if(permissions?.geolocation)allow.push('geolocation');
+      if(permissions?.clipboardWrite)allow.push('clipboard-write');
+      if(allow.length)inner.setAttribute('allow',allow.join('; '));
+      document.body.append(inner);
+      let html=String(message.params?.html||'');
+      const csp=message.params?.csp;
+      if(csp&&typeof csp==='object'){
+        const list=(value,fallback)=>Array.isArray(value)&&value.length?value.join(' '):fallback;
+        const resources=Array.isArray(csp.resourceDomains)?csp.resourceDomains:[];
+        const suffix=resources.length?' '+resources.join(' '):'';
+        const policy=["default-src 'none'","script-src 'unsafe-inline'"+suffix,"style-src 'unsafe-inline'"+suffix,'img-src data: blob:'+suffix,'font-src data: blob:'+suffix,'media-src data: blob:'+suffix,'worker-src blob:'+suffix,'connect-src '+list(csp.connectDomains,"'none'"),'frame-src '+list(csp.frameDomains,"'none'"),"object-src 'none'","form-action 'none'",'base-uri '+list(csp.baseUriDomains,"'none'")].join('; ');
+        const meta='<meta http-equiv="Content-Security-Policy" content="'+policy.replace(/&/g,'&amp;').replace(/"/g,'&quot;')+'">';
+        const doctype=html.match(/^\\s*<!doctype[^>]*>/i);
+        html=doctype?html.slice(0,doctype[0].length)+meta+html.slice(doctype[0].length):meta+html;
+      }
+      try{
+        const doc=inner.contentDocument||inner.contentWindow?.document;
+        if(!doc)throw new Error('document unavailable');
+        doc.open();doc.write(html);doc.close();
+      }catch{inner.srcdoc=html}
       return;
     }
     inner?.contentWindow?.postMessage(message,'*');
   } else if(inner && event.source===inner.contentWindow) {
-    parent.postMessage(event.data,'*');
+    if(event.origin!==ownOrigin)return;
+    parent.postMessage(event.data,parentTarget);
   }
 });
-parent.postMessage(ready,'*');
-</script>`;
+parent.postMessage(ready,parentTarget);
+</script></body>`;
 function isAllowedAppLink(value, allowedDomains, baseUrl) {
   try {
     const parsed = new URL(value, baseUrl);
@@ -18246,16 +18352,42 @@ function createAssistantAppHostContext(theme, allowFullscreen = false, displayMo
 }
 var APP_RENDER_TIMEOUT_MS = 1e4;
 var APP_TEARDOWN_TIMEOUT_MS = 1e3;
+var STANDARD_SANDBOX = "allow-scripts allow-same-origin allow-forms";
 var mountedApps = /* @__PURE__ */ new WeakMap();
 function resolveSandboxUrl(value) {
   if (!value) return void 0;
   try {
     const parsed = new URL(value);
-    return parsed.protocol === "https:" || parsed.protocol === "http:" ? parsed.href : void 0;
+    const loopback = parsed.hostname === "localhost" || parsed.hostname === "127.0.0.1" || parsed.hostname === "[::1]";
+    const permittedProtocol = parsed.protocol === "https:" || parsed.protocol === "http:" && loopback;
+    const embedderOrigin = globalThis.location?.origin;
+    if (!permittedProtocol || embedderOrigin && embedderOrigin !== "null" && parsed.origin === embedderOrigin) {
+      return void 0;
+    }
+    return parsed.href;
   } catch {
     return void 0;
   }
 }
+function copyAppJson(value, depth = 0) {
+  if (depth > 64) throw new Error("App payload nesting exceeds the host limit");
+  if (value === null || typeof value === "string" || typeof value === "boolean") return value;
+  if (typeof value === "number") {
+    if (!Number.isFinite(value)) throw new Error("App payload numbers must be finite");
+    return value;
+  }
+  if (Array.isArray(value)) return value.map((entry) => copyAppJson(entry, depth + 1));
+  const copied = {};
+  for (const [key, entry] of Object.entries(value)) {
+    Object.defineProperty(copied, key, {
+      value: copyAppJson(entry, depth + 1),
+      enumerable: true,
+      configurable: true,
+      writable: true
+    });
+  }
+  return copied;
+}
 function mountAssistantApp(detail, actions, options = {}) {
   if (!detail.html) return void 0;
   const card = document.createElement("section");
@@ -18276,16 +18408,25 @@ function mountAssistantApp(detail, actions, options = {}) {
   const frame = document.createElement("iframe");
   frame.className = "noodle-app-frame";
   frame.title = detail.title ?? detail.tool;
-  frame.setAttribute("sandbox", "allow-scripts");
+  const policy = resolveAssistantAppSandboxPolicy(detail.resourceMeta);
   frame.setAttribute("referrerpolicy", "no-referrer");
   const sandboxUrl = resolveSandboxUrl(options.sandboxUrl);
-  if (sandboxUrl) frame.src = sandboxUrl;
-  else frame.srcdoc = PROXY_DOCUMENT;
+  const appliedPolicy = sandboxUrl ? policy : { csp: policy.csp ?? {} };
+  if (sandboxUrl) {
+    frame.setAttribute("sandbox", STANDARD_SANDBOX);
+    if (policy.allowAttribute) frame.setAttribute("allow", policy.allowAttribute);
+    frame.src = assistantAppSandboxUrl(sandboxUrl, policy);
+  } else {
+    frame.setAttribute("sandbox", "allow-scripts");
+    frame.srcdoc = PROXY_DOCUMENT;
+  }
   card.append(frame);
   let mountedBridge;
   let mountedTransport;
   let destroyed = false;
   let initialized = false;
+  let renderFailed = false;
+  let renderStage = "sandbox_load";
   let currentTheme = actions.theme;
   let displayMode = "inline";
   let hostContext = createAssistantAppHostContext(
@@ -18294,6 +18435,15 @@ function mountAssistantApp(detail, actions, options = {}) {
     displayMode
   );
   let destroyPromise;
+  let transportClosePromise;
+  let renderTimer;
+  const closeTransport = () => {
+    if (transportClosePromise) return transportClosePromise;
+    const closing = mountedTransport?.close().catch(() => {
+    }) ?? Promise.resolve();
+    transportClosePromise = closing;
+    return closing;
+  };
   const applyDisplayMode = (requestedMode) => {
     displayMode = requestedMode === "fullscreen" && options.allowFullscreen === true ? "fullscreen" : "inline";
     card.toggleAttribute("data-fullscreen", displayMode === "fullscreen");
@@ -18311,18 +18461,26 @@ function mountAssistantApp(detail, actions, options = {}) {
     applyDisplayMode("inline");
     frame.focus();
   });
-  const renderTimer = setTimeout(() => {
-    if (initialized || !frame.isConnected) return;
+  const showRenderFailure = (code) => {
+    if (code === "view_render_timeout" && initialized || destroyed || renderFailed || !frame.isConnected)
+      return;
+    renderFailed = true;
+    clearTimeout(renderTimer);
     const note = document.createElement("div");
     note.className = "noodle-app-fallback";
     note.textContent = "This app view could not be displayed.";
     frame.replaceWith(note);
-    options.onRenderFailure?.({ code: "view_render_timeout", retryable: false });
+    options.onRenderFailure?.({ code, retryable: false, stage: renderStage });
+    void closeTransport();
+  };
+  renderTimer = setTimeout(() => {
+    showRenderFailure("view_render_timeout");
   }, APP_RENDER_TIMEOUT_MS);
   frame.addEventListener(
     "load",
     () => {
-      if (destroyed) return;
+      if (destroyed || renderFailed) return;
+      renderStage = "sandbox_proxy";
       const target = frame.contentWindow;
       if (!target) return;
       const bridge = new DX(
@@ -18333,7 +18491,11 @@ function mountAssistantApp(detail, actions, options = {}) {
           serverTools: {},
           serverResources: {},
           updateModelContext: { text: {}, structuredContent: {} },
-          message: { text: {} }
+          message: { text: {} },
+          sandbox: {
+            ...appliedPolicy.csp === void 0 ? {} : { csp: appliedPolicy.csp },
+            ...appliedPolicy.permissions === void 0 ? {} : { permissions: appliedPolicy.permissions }
+          }
         },
         {
           hostContext
@@ -18341,13 +18503,15 @@ function mountAssistantApp(detail, actions, options = {}) {
       );
       mountedBridge = bridge;
       bridge.onsandboxready = () => {
-        if (destroyed) return;
+        if (destroyed || renderFailed) return;
+        renderStage = "app_initialize";
         const current = options.getDetail?.() ?? detail;
         void bridge.sendSandboxResourceReady({
           html: current.html,
-          sandbox: "allow-scripts",
-          ...current.resourceMeta ?? {}
-        });
+          sandbox: sandboxUrl ? STANDARD_SANDBOX : "allow-scripts",
+          ...appliedPolicy.csp === void 0 ? {} : { csp: appliedPolicy.csp },
+          ...appliedPolicy.permissions === void 0 ? {} : { permissions: appliedPolicy.permissions }
+        }).catch(() => showRenderFailure("view_bridge_error"));
       };
       bridge.onsizechange = ({ height }) => {
         if (destroyed) return;
@@ -18411,21 +18575,30 @@ function mountAssistantApp(detail, actions, options = {}) {
         return await actions.client.requestApp("resources/read", params);
       };
       bridge.oninitialized = () => {
-        if (destroyed) return;
-        initialized = true;
-        clearTimeout(renderTimer);
-        bridge.setHostContext(hostContext);
-        const current = options.getDetail?.() ?? detail;
-        void bridge.sendToolInput({ arguments: current.arguments ?? {} });
-        void bridge.sendToolResult({
-          content: [{ type: "text", text: JSON.stringify(current.result) }],
-          ...typeof current.result === "object" && current.result !== null && !Array.isArray(current.result) ? { structuredContent: current.result } : {}
-        });
+        if (destroyed || renderFailed) return;
+        try {
+          const current = options.getDetail?.() ?? detail;
+          const copiedArguments = copyAppJson(current.arguments ?? {});
+          const copiedResult = copyAppJson(current.result);
+          const argumentsRecord = typeof copiedArguments === "object" && copiedArguments !== null && !Array.isArray(copiedArguments) ? copiedArguments : {};
+          initialized = true;
+          clearTimeout(renderTimer);
+          bridge.setHostContext(hostContext);
+          void Promise.all([
+            bridge.sendToolInput({ arguments: argumentsRecord }),
+            bridge.sendToolResult({
+              content: [{ type: "text", text: JSON.stringify(copiedResult) }],
+              ...typeof copiedResult === "object" && copiedResult !== null && !Array.isArray(copiedResult) ? { structuredContent: copiedResult } : {}
+            })
+          ]).catch(() => showRenderFailure("view_bridge_error"));
+        } catch {
+          showRenderFailure("view_bridge_error");
+        }
       };
       const transport = new E(target, target);
       mountedTransport = transport;
       void bridge.connect(transport).catch(() => {
-        void destroy();
+        showRenderFailure("view_bridge_error");
       });
     },
     { once: true }
@@ -18451,8 +18624,7 @@ function mountAssistantApp(detail, actions, options = {}) {
         }
       } catch {
       }
-      await mountedTransport?.close().catch(() => {
-      });
+      await closeTransport();
       card.remove();
       mountedApps.delete(card);
     })();