@athenaintel/react 0.9.16 → 0.9.18

package/dist/index.d.ts

@@ -48,11 +48,8 @@ export declare interface AssetPanelState {
     autoOpenedAssets: Map<string, number>;
     registerAssetPanelHost: () => void;
     unregisterAssetPanelHost: () => void;
-    openAsset: (assetId: string, meta?: {
-        name?: string;
-        type?: AssetType;
-        embedSearchParams?: Record<string, string> | null;
-    }) => void;
+    openAsset: (assetId: string, meta?: AssetTabMeta) => void;
+    updateTabMeta: (assetId: string, meta: AssetTabMeta) => void;
     closeTab: (assetId: string) => void;
     setActiveTab: (assetId: string) => void;
     setViewMode: (mode: ViewMode) => void;
@@ -71,6 +68,12 @@ export declare interface AssetTab {
     embedSearchParams: Record<string, string> | null;
 }
 
+declare interface AssetTabMeta {
+    name?: string | null;
+    type?: AssetType;
+    embedSearchParams?: Record<string, string> | null;
+}
+
 export declare type AssetType = 'presentation' | 'spreadsheet' | 'document' | 'notebook' | 'unknown';
 
 export declare const AthenaAssistantActionBar: FC<AthenaAssistantActionBarProps>;

package/dist/index.js

@@ -19849,14 +19849,20 @@ const joinExternalMessages = (messages) => {
               )
             );
         }
+        const { messages: existingMessages, ...toolCallRest } = toolCall;
         const updatedToolCall = {
-          ...toolCall,
+          ...toolCallRest,
           result: output.result,
           artifact: output.artifact,
           isError: output.isError,
-          messages: output.messages
+          ...(output.messages ?? existingMessages) !== void 0 && {
+            messages: output.messages ?? existingMessages
+          }
         };
-        bindExternalStoreMessage(updatedToolCall, [...getExternalStoreMessages(toolCall), output]);
+        bindExternalStoreMessage(updatedToolCall, [
+          ...getExternalStoreMessages(toolCallRest),
+          output
+        ]);
         assistantMessage.content[toolCallIdx] = updatedToolCall;
       } else {
         console.warn(
@@ -19865,10 +19871,14 @@ const joinExternalMessages = (messages) => {
       }
     } else {
       const role = output.role;
-      const content = (typeof output.content === "string" ? [{ type: "text", text: output.content }] : output.content).map((c) => {
+      const rawContent = typeof output.content === "string" ? [{ type: "text", text: output.content }] : Array.isArray(output.content) ? output.content : [];
+      const content = rawContent.flatMap((c) => {
+        if (!c || typeof c !== "object") {
+          return [];
+        }
         const mapped = { ...c };
         bindExternalStoreMessage(mapped, output);
-        return mapped;
+        return [mapped];
       });
       switch (role) {
         case "system":
@@ -19999,7 +20009,13 @@ const convertExternalMessages = (messages, callback, isRunning, metadata) => {
   const callbackResults = [];
   for (const message of messages) {
     const output = callback(message, metadata);
-    const outputs = Array.isArray(output) ? output : [output];
+    const rawOutputs = Array.isArray(output) ? output : [output];
+    const outputs = rawOutputs.filter(
+      (candidate) => candidate != null
+    );
+    if (outputs.length === 0) {
+      continue;
+    }
     const result = { input: message, outputs };
     callbackResults.push(result);
   }
@@ -20022,37 +20038,40 @@ const warnForUnknownMessagePartType = (type) => {
 };
 const contentToParts = (content) => {
   if (typeof content === "string") return [{ type: "text", text: content }];
-  return content.map((part) => {
+  return content.flatMap((part) => {
+    if (!part || typeof part !== "object" || !("type" in part)) {
+      return [];
+    }
     const type = part.type;
     switch (type) {
       case "text":
-        return { type: "text", text: part.text };
+        return [{ type: "text", text: part.text }];
       case "text_delta":
-        return { type: "text", text: part.text };
+        return [{ type: "text", text: part.text }];
       case "image_url":
         if (typeof part.image_url === "string") {
-          return { type: "image", image: part.image_url };
+          return [{ type: "image", image: part.image_url }];
         }
-        return {
+        return [{
           type: "image",
           image: part.image_url.url
-        };
+        }];
       case "thinking":
-        return { type: "reasoning", text: part.thinking };
+        return [{ type: "reasoning", text: part.thinking }];
       case "reasoning":
-        return {
+        return [{
           type: "reasoning",
           text: part.summary.map((s) => s.text).join("\n\n\n")
-        };
+        }];
       case "tool_use":
-        return null;
+        return [];
       case "input_json_delta":
-        return null;
+        return [];
       default:
         warnForUnknownMessagePartType(type);
-        return null;
+        return [];
     }
-  }).filter((a) => a !== null);
+  });
 };
 const getNumberAtPath = (value, path) => {
   let current = value;
@@ -20157,7 +20176,7 @@ const convertLangChainMessage = (message) => {
         (_b = message.additional_kwargs) == null ? void 0 : _b.reasoning,
         ...normalizedContent,
         ...((_c = message.additional_kwargs) == null ? void 0 : _c.tool_outputs) ?? []
-      ].filter((c) => c !== void 0);
+      ].filter((c) => c != null);
       return {
         role: "assistant",
         id: message.id,
@@ -20180,6 +20199,8 @@ const convertLangChainMessage = (message) => {
         isError: message.status === "error"
       };
     }
+    default:
+      return null;
   }
 };
 const convertLangChainToThreadMessages = (messages, isRunning = false, metadata = {}) => {
@@ -58874,6 +58895,22 @@ const useAssetPanelStore = create()(
         };
         return { isOpen: true, tabs: [...s.tabs, newTab], activeTabId: assetId };
       }),
+      updateTabMeta: (assetId, meta) => set2((s) => {
+        const existing = s.tabs.find((t) => t.id === assetId);
+        if (!existing) {
+          return {};
+        }
+        return {
+          tabs: s.tabs.map(
+            (t) => t.id === assetId ? {
+              ...t,
+              name: meta.name ?? t.name,
+              type: meta.type ?? t.type,
+              embedSearchParams: meta.embedSearchParams ?? t.embedSearchParams
+            } : t
+          )
+        };
+      }),
       closeTab: (assetId) => set2((s) => {
         var _a2;
         const tabs = s.tabs.filter((t) => t.id !== assetId);
@@ -65254,6 +65291,15 @@ const normalizeAssetType = (value) => {
   const normalizedValue = value.trim().replace(/^athena\//, "").toLowerCase().replace(/-/g, "_");
   return EMBED_ASSET_TYPE_ALIASES[normalizedValue] ?? null;
 };
+const getEmbedSearchParamsCacheKey = (embedSearchParams) => {
+  if (!embedSearchParams || Object.keys(embedSearchParams).length === 0) {
+    return "{}";
+  }
+  const sortedEntries = Object.entries(embedSearchParams).sort(
+    ([left], [right]) => left.localeCompare(right)
+  );
+  return JSON.stringify(Object.fromEntries(sortedEntries));
+};
 const appendEmbedSearchParams = ({
   embedUrl,
   embedSearchParams
@@ -65297,7 +65343,7 @@ function useAssetEmbed(assetId, options = {
       setAssetType(null);
       return;
     }
-    const cacheKey = `${assetId}:${readOnly}:${token ?? apiKey ?? "anon"}:${JSON.stringify(embedSearchParams ?? {})}`;
+    const cacheKey = `${assetId}:${readOnly}:${token ?? apiKey ?? "anon"}:${getEmbedSearchParamsCacheKey(embedSearchParams)}`;
     const cached = embedCache.get(cacheKey);
     if (cached && cached.expiresAt > Date.now() / 1e3) {
       setEmbedUrl(cached.url);
@@ -65390,7 +65436,7 @@ const AssetIframe = memo(
       if (nextName === tabName && nextType === tabType) {
         return;
       }
-      useAssetPanelStore.getState().openAsset(tabId, {
+      useAssetPanelStore.getState().updateTabMeta(tabId, {
         name: nextName ?? void 0,
         type: nextType
       });
@@ -65456,7 +65502,7 @@ const PanelContent = ({
   return /* @__PURE__ */ jsx(
     "div",
     {
-      className: `flex-1 overflow-hidden ${isTiled ? `${gridClass} gap-px bg-border/60` : ""}`,
+      className: cn("flex-1 overflow-hidden", isTiled && gridClass, isTiled && "gap-px bg-border/60"),
       children: tabs.map((tab) => {
         const isActive2 = tab.id === activeTabId;
         const config2 = ASSET_TYPE_CONFIG[tab.type];
@@ -65523,7 +65569,10 @@ const TabBar = () => {
     return /* @__PURE__ */ jsxs(
       "div",
       {
-        className: `group flex max-w-[180px] shrink-0 cursor-pointer items-center gap-1.5 border-r border-border/40 px-3 py-1.5 text-xs ${isActive2 ? "border-b-2 border-b-primary bg-background text-foreground" : "text-muted-foreground hover:bg-background/50 hover:text-foreground"}`,
+        className: cn(
+          "group flex max-w-[180px] shrink-0 cursor-pointer items-center gap-1.5 border-r border-border/40 px-3 py-1.5 text-xs",
+          isActive2 ? "border-b-2 border-b-primary bg-background text-foreground" : "text-muted-foreground hover:bg-background/50 hover:text-foreground"
+        ),
         onClick: () => setActiveTab(tab.id),
         children: [
           /* @__PURE__ */ jsx(TypeIcon, { className: "size-3 shrink-0" }),
@@ -65565,7 +65614,10 @@ const PanelHeader = ({ fullscreen }) => {
       "button",
       {
         onClick: () => setViewMode(isTiled ? "tabs" : "tiled"),
-        className: `flex size-7 items-center justify-center rounded-lg transition-colors ${isTiled ? "bg-primary/10 text-primary" : "text-muted-foreground hover:bg-accent hover:text-foreground"}`,
+        className: cn(
+          "flex size-7 items-center justify-center rounded-lg transition-colors",
+          isTiled ? "bg-primary/10 text-primary" : "text-muted-foreground hover:bg-accent hover:text-foreground"
+        ),
         title: isTiled ? "Switch to tabs" : "Tile all assets",
         children: isTiled ? /* @__PURE__ */ jsx(Layers, { className: "size-3.5" }) : /* @__PURE__ */ jsx(LayoutGrid, { className: "size-3.5" })
       }