shadow-claw 1.40.0 → 1.41.0

package/src/core/orchestrator/utils/invokeAgent.ts

@@ -50,6 +50,7 @@ import {
 } from "./operations/provider.js";
 
 import type { ShadowClawDatabase } from "../../../db/db.js";
+import type { ToolDefinition } from "../../../subsystems/tools/types.js";
 import type { SubagentInvokeContext } from "../../../worker/tools/spawn-subagent/spawn-subagent.js";
 import type { Orchestrator } from "../orchestrator.js";
 
@@ -197,10 +198,7 @@ export async function invokeAgent(
   const contextLimit = getContextLimit(modelForTokenLimits);
   const systemPromptTokens =
     estimateTokens(systemPrompt) +
-    (activeTools?.reduce(
-      (acc, t) => acc + estimateTokens(JSON.stringify(t)),
-      0,
-    ) ?? 0);
+    activeTools.reduce((acc, t) => acc + estimateTokens(JSON.stringify(t)), 0);
   let allMessages = await buildConversationMessages(groupId, 200);
   if (freshContext) {
     if (allMessages.length > 0) {
@@ -265,12 +263,12 @@ export async function invokeAgent(
       apiKey: "",
       assistantName: o.assistantName,
       db,
-      enabledTools: activeTools as any,
+      enabledTools: activeTools as unknown as ToolDefinition[],
       invokeSubagent: async (subPayload) => {
         await dispatchSubagentInvoke(db, subPayload, controller.signal);
       },
       maxTokens: effectiveMaxTokens,
-      memory: memory ?? "",
+      memory,
       model: effectiveModel,
       provider: effectiveProviderId,
       providerHeaders: getProviderRuntimeHeaders(
@@ -345,12 +343,12 @@ export async function invokeAgent(
       apiKey: "",
       assistantName: o.assistantName,
       db,
-      enabledTools: activeTools as any,
+      enabledTools: activeTools as unknown as ToolDefinition[],
       invokeSubagent: async (subPayload) => {
         await dispatchSubagentInvoke(db, subPayload, controller.signal);
       },
       maxTokens: effectiveMaxTokens,
-      memory: memory ?? "",
+      memory,
       model: effectiveModel,
       provider: effectiveProviderId,
       providerHeaders: getProviderRuntimeHeaders(
@@ -421,12 +419,12 @@ export async function invokeAgent(
       apiKey: "",
       assistantName: o.assistantName,
       db,
-      enabledTools: activeTools as any,
+      enabledTools: activeTools as unknown as ToolDefinition[],
       invokeSubagent: async (subPayload) => {
         await dispatchSubagentInvoke(db, subPayload, controller.signal);
       },
       maxTokens: effectiveMaxTokens,
-      memory: memory ?? "",
+      memory,
       model: effectiveModel,
       provider: effectiveProviderId,
       providerHeaders: getProviderRuntimeHeaders(

package/src/core/orchestrator/utils/loadChannelConfigurations.test.ts

@@ -196,4 +196,54 @@ describe("loadChannelConfigurations", () => {
 
     consoleWarn.mockRestore();
   });
+
+  it("should configure empty peerjs server config when host is empty", async () => {
+    mockGetConfig.mockImplementation(async (_db: any, key: string) => {
+      if (key === "PEERJS_SERVER_HOST") return "";
+      if (key === "PEERJS_SERVER_PORT") return "not-a-number";
+      if (key === "PEERJS_PEER_ALIASES") return null;
+      return null;
+    });
+
+    await loadChannelConfigurations(mockOrchestrator, mockDb);
+
+    expect(mockOrchestrator.peerjsServerPort).toBe(0);
+    expect(mockOrchestrator.peerjsPeerAliases).toEqual({});
+    expect(mockOrchestrator.peerjs.configure).toHaveBeenCalledWith(
+      "",
+      ["id1", "id2"],
+      {},
+    );
+  });
+
+  it("should omit port and path when they are zero or empty and host is set", async () => {
+    mockGetConfig.mockImplementation(async (_db: any, key: string) => {
+      if (key === "PEERJS_SERVER_HOST") return "custom-host";
+      if (key === "PEERJS_SERVER_PORT") return "0";
+      if (key === "PEERJS_SERVER_PATH") return "";
+      return null;
+    });
+
+    await loadChannelConfigurations(mockOrchestrator, mockDb);
+
+    expect(mockOrchestrator.peerjs.configure).toHaveBeenCalledWith(
+      "",
+      ["id1", "id2"],
+      {
+        host: "custom-host",
+        port: undefined,
+        path: undefined,
+        secure: true,
+      },
+    );
+  });
+
+  it("should evaluate fallback path in fileReader when path is empty string", async () => {
+    await loadChannelConfigurations(mockOrchestrator, mockDb);
+
+    const fileReader = mockOrchestrator.telegram.fileReader;
+    const blob = await fileReader("group1", "");
+
+    expect(blob).toBeInstanceOf(Blob);
+  });
 });