fix(assistant): handle nullish state and optional callbacks in assistantSessionState

src/helpers/assistantSessionState.js

@@ -5,15 +5,16 @@
  */
 export function closeAssistantSessionState(state) {
   return {
-    conversationId: state.conversationId ?? null,
+    conversationId: state?.conversationId ?? null,
     pendingCommand: null,
     thinking: false,
     busy: false,
     responseReady: false,
   };
 }
 
-export function resolveAssistantPanelBusy({ agentState, activeToolName, submissionInFlight }) {
+export function resolveAssistantPanelBusy(params) {
+  const { agentState, activeToolName, submissionInFlight } = params || {};
   return Boolean(
     submissionInFlight ||
     activeToolName ||
@@ -36,15 +37,15 @@ export async function restoreAssistantConversation({
   isActive = () => true,
 }) {
   try {
-    await loadConversation(conversationId);
+    await loadConversation?.(conversationId);
     if (!isActive()) return "inactive";
-    onReady();
+    onReady?.();
     return "restored";
   } catch (error) {
     if (!isActive()) return "inactive";
-    onError(error);
-    onReset();
-    onReady();
+    onError?.(error);
+    onReset?.();
+    onReady?.();
     return "reset";
   }
 }

test/helpers/assistantSessionState.test.js

@@ -111,3 +111,47 @@ test("failed Assistant history resets before enabling a fresh conversation", asy
   );
   assert.deepEqual(events, ["error", "reset", "ready"]);
 });
+
+test("assistantSessionState helpers handle nullish inputs safely", async () => {
+  const {
+    closeAssistantSessionState,
+    resolveAssistantPanelBusy,
+    restoreAssistantConversation,
+  } = await import("../../src/helpers/assistantSessionState.js");
+
+  assert.deepEqual(closeAssistantSessionState(null), {
+    conversationId: null,
+    pendingCommand: null,
+    thinking: false,
+    busy: false,
+    responseReady: false,
+  });
+  assert.deepEqual(closeAssistantSessionState(undefined), {
+    conversationId: null,
+    pendingCommand: null,
+    thinking: false,
+    busy: false,
+    responseReady: false,
+  });
+
+  assert.equal(resolveAssistantPanelBusy(null), false);
+  assert.equal(resolveAssistantPanelBusy(undefined), false);
+
+  assert.equal(
+    await restoreAssistantConversation({
+      conversationId: 42,
+      loadConversation: async () => {},
+    }),
+    "restored"
+  );
+
+  assert.equal(
+    await restoreAssistantConversation({
+      conversationId: 42,
+      loadConversation: async () => {
+        throw new Error("fail");
+      },
+    }),
+    "reset"
+  );
+});