@jambonz/llm 0.6.2 → 0.6.4

package/dist/test-kit/index.d.cts

@@ -1,4 +1,4 @@
-import { b as AdapterFactory, e as AuthKind, A as AuthSpec, L as LlmAdapter, d as ApiKeyAuth, h as ClientOptions, P as PromptRequest, m as LlmEvent, M as Message, p as ToolCallEvent, o as ModelInfo, c as AdapterManifest } from '../types-Y0o6rN3B.cjs';
+import { b as AdapterFactory, e as AuthKind, A as AuthSpec, L as LlmAdapter, d as ApiKeyAuth, h as ClientOptions, P as PromptRequest, m as LlmEvent, M as Message, p as ToolCallEvent, o as ModelInfo, c as AdapterManifest } from '../types-dbgrfviJ.cjs';
 
 /**
  * Named scenarios the harness must be able to stage. Each scenario configures
@@ -129,7 +129,7 @@ declare class FakeAdapter implements LlmAdapter<ApiKeyAuth> {
     constructor(state: FakeState);
     init(auth: ApiKeyAuth, _client?: ClientOptions): void;
     stream(req: PromptRequest): AsyncIterable<LlmEvent>;
-    appendAssistantToolCall(history: Message[], toolCalls: ReadonlyArray<ToolCallEvent>): Message[];
+    appendAssistantToolCall(history: Message[], toolCalls: ReadonlyArray<ToolCallEvent>, assistantText?: string): Message[];
     appendToolResult(history: Message[], toolCallId: string, result: unknown): Message[];
     listAvailableModels(): Promise<ModelInfo[]>;
     testCredential(): Promise<void>;

package/dist/test-kit/index.d.ts

@@ -1,4 +1,4 @@
-import { b as AdapterFactory, e as AuthKind, A as AuthSpec, L as LlmAdapter, d as ApiKeyAuth, h as ClientOptions, P as PromptRequest, m as LlmEvent, M as Message, p as ToolCallEvent, o as ModelInfo, c as AdapterManifest } from '../types-Y0o6rN3B.js';
+import { b as AdapterFactory, e as AuthKind, A as AuthSpec, L as LlmAdapter, d as ApiKeyAuth, h as ClientOptions, P as PromptRequest, m as LlmEvent, M as Message, p as ToolCallEvent, o as ModelInfo, c as AdapterManifest } from '../types-dbgrfviJ.js';
 
 /**
  * Named scenarios the harness must be able to stage. Each scenario configures
@@ -129,7 +129,7 @@ declare class FakeAdapter implements LlmAdapter<ApiKeyAuth> {
     constructor(state: FakeState);
     init(auth: ApiKeyAuth, _client?: ClientOptions): void;
     stream(req: PromptRequest): AsyncIterable<LlmEvent>;
-    appendAssistantToolCall(history: Message[], toolCalls: ReadonlyArray<ToolCallEvent>): Message[];
+    appendAssistantToolCall(history: Message[], toolCalls: ReadonlyArray<ToolCallEvent>, assistantText?: string): Message[];
     appendToolResult(history: Message[], toolCallId: string, result: unknown): Message[];
     listAvailableModels(): Promise<ModelInfo[]>;
     testCredential(): Promise<void>;

package/dist/test-kit/index.js

@@ -232,10 +232,11 @@ async function* streamFromOpenAI(client, req, options) {
   }
   yield endEvent;
 }
-function appendOpenAIAssistantToolCall(history, toolCalls) {
+function appendOpenAIAssistantToolCall(history, toolCalls, assistantText) {
+  const text = assistantText?.trim() ? assistantText : null;
   const wireMessage = {
     role: "assistant",
-    content: null,
+    content: text,
     tool_calls: toolCalls.map((tc) => ({
       id: tc.id,
       type: "function",
@@ -249,7 +250,7 @@ function appendOpenAIAssistantToolCall(history, toolCalls) {
     ...history,
     {
       role: "assistant",
-      content: "",
+      content: text ?? "",
       vendorRaw: wireMessage
     }
   ];
@@ -520,8 +521,8 @@ var OpenAIAdapter = class {
       includeCacheKey: this.forwardCacheKey()
     });
   }
-  appendAssistantToolCall(history, toolCalls) {
-    return appendOpenAIAssistantToolCall(history, toolCalls);
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    return appendOpenAIAssistantToolCall(history, toolCalls, assistantText);
   }
   appendToolResult(history, toolCallId, result) {
     return appendOpenAIToolResult(history, toolCallId, result);
@@ -1487,21 +1488,25 @@ var AnthropicAdapter = class {
     }
     yield endEvent;
   }
-  appendAssistantToolCall(history, toolCalls) {
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    const text = assistantText?.trim() ? assistantText : null;
     const wireMessage = {
       role: "assistant",
-      content: toolCalls.map((tc) => ({
-        type: "tool_use",
-        id: tc.id,
-        name: tc.name,
-        input: tc.arguments ?? {}
-      }))
+      content: [
+        ...text ? [{ type: "text", text }] : [],
+        ...toolCalls.map((tc) => ({
+          type: "tool_use",
+          id: tc.id,
+          name: tc.name,
+          input: tc.arguments ?? {}
+        }))
+      ]
     };
     return [
       ...history,
       {
         role: "assistant",
-        content: "",
+        content: text ?? "",
         vendorRaw: wireMessage
       }
     ];
@@ -1753,21 +1758,25 @@ async function* streamFromGemini(client, req) {
   };
   yield endEvent;
 }
-function appendGeminiAssistantToolCall(history, toolCalls) {
+function appendGeminiAssistantToolCall(history, toolCalls, assistantText) {
+  const text = assistantText?.trim() ? assistantText : null;
   const wireMessage = {
     role: "model",
-    parts: toolCalls.map((tc) => ({
-      functionCall: {
-        name: parseToolName(tc.id),
-        args: tc.arguments ?? {}
-      }
-    }))
+    parts: [
+      ...text ? [{ text }] : [],
+      ...toolCalls.map((tc) => ({
+        functionCall: {
+          name: parseToolName(tc.id),
+          args: tc.arguments ?? {}
+        }
+      }))
+    ]
   };
   return [
     ...history,
     {
       role: "assistant",
-      content: "",
+      content: text ?? "",
       vendorRaw: wireMessage
     }
   ];
@@ -2027,8 +2036,8 @@ var GoogleAdapter = class {
   stream(req) {
     return streamFromGemini(this.ensureClient(), req);
   }
-  appendAssistantToolCall(history, toolCalls) {
-    return appendGeminiAssistantToolCall(history, toolCalls);
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    return appendGeminiAssistantToolCall(history, toolCalls, assistantText);
   }
   appendToolResult(history, toolCallId, result) {
     return appendGeminiToolResult(history, toolCallId, result);
@@ -2233,8 +2242,8 @@ var VertexGeminiAdapter = class {
   stream(req) {
     return streamFromGemini(this.ensureClient(), req);
   }
-  appendAssistantToolCall(history, toolCalls) {
-    return appendGeminiAssistantToolCall(history, toolCalls);
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    return appendGeminiAssistantToolCall(history, toolCalls, assistantText);
   }
   appendToolResult(history, toolCallId, result) {
     return appendGeminiToolResult(history, toolCallId, result);
@@ -2431,8 +2440,8 @@ var VertexOpenAIAdapter = class {
       vendorMetadataExtractor: VERTEX_OPENAI_METADATA_EXTRACTOR
     });
   }
-  appendAssistantToolCall(history, toolCalls) {
-    return appendOpenAIAssistantToolCall(history, toolCalls);
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    return appendOpenAIAssistantToolCall(history, toolCalls, assistantText);
   }
   appendToolResult(history, toolCallId, result) {
     return appendOpenAIToolResult(history, toolCallId, result);
@@ -2783,11 +2792,12 @@ var BedrockAdapter = class {
     if (Object.keys(inferenceConfig).length > 0) {
       commandInput.inferenceConfig = inferenceConfig;
     }
-    if (req.cacheKey) {
+    const caching = req.cacheKey ? cachePointSupport(req.model) : void 0;
+    if (caching) {
       if (commandInput.system) {
         commandInput.system.push({ cachePoint: { type: "default" } });
       }
-      if (tools) {
+      if (tools && caching.tools) {
         tools.tools.push({ cachePoint: { type: "default" } });
       }
       markLastMessageForCaching2(messages);
@@ -2924,22 +2934,26 @@ var BedrockAdapter = class {
     }
     yield endEvent;
   }
-  appendAssistantToolCall(history, toolCalls) {
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    const text = assistantText?.trim() ? assistantText : null;
     const wireMessage = {
       role: "assistant",
-      content: toolCalls.map((tc) => ({
-        toolUse: {
-          toolUseId: tc.id,
-          name: tc.name,
-          input: tc.arguments ?? {}
-        }
-      }))
+      content: [
+        ...text ? [{ text }] : [],
+        ...toolCalls.map((tc) => ({
+          toolUse: {
+            toolUseId: tc.id,
+            name: tc.name,
+            input: tc.arguments ?? {}
+          }
+        }))
+      ]
     };
     return [
       ...history,
       {
         role: "assistant",
-        content: "",
+        content: text ?? "",
         vendorRaw: wireMessage
       }
     ];
@@ -3007,6 +3021,11 @@ function buildWireMessages3(req) {
   }
   return out;
 }
+function cachePointSupport(model) {
+  if (model.includes("anthropic.claude")) return { tools: true };
+  if (model.includes("amazon.nova")) return { tools: false };
+  return void 0;
+}
 function markLastMessageForCaching2(messages) {
   if (messages.length === 0) return;
   const last = messages[messages.length - 1];
@@ -3229,8 +3248,8 @@ var AzureOpenAIAdapter = class {
       vendorMetadataExtractor: AZURE_OPENAI_METADATA_EXTRACTOR
     });
   }
-  appendAssistantToolCall(history, toolCalls) {
-    return appendOpenAIAssistantToolCall(history, toolCalls);
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    return appendOpenAIAssistantToolCall(history, toolCalls, assistantText);
   }
   appendToolResult(history, toolCallId, result) {
     return appendOpenAIToolResult(history, toolCallId, result);
@@ -3885,6 +3904,55 @@ function runContractTests(harness) {
       expect(appended.role).toBe("assistant");
       expect(appended.vendorRaw).toBeDefined();
     });
+    it("[22] appendAssistantToolCall carries pre-tool assistant text onto the wire", async () => {
+      await harness.mockScenario("tool-call");
+      const adapter = await init(harness);
+      const first = await drainStream(adapter, {
+        model: harness.toolCapableModel,
+        messages: [{ role: "user", content: "go" }],
+        tools: [
+          { name: "test_tool", description: "x", parameters: { type: "object" } }
+        ]
+      });
+      const tc = first.find(
+        (e) => e.type === "toolCall"
+      );
+      expect(tc).toBeDefined();
+      const preToolText = "Checking that now.";
+      const history = adapter.appendAssistantToolCall(
+        [{ role: "user", content: "go" }],
+        [tc],
+        preToolText
+      );
+      const appended = history[history.length - 1];
+      expect(appended.content).toBe(preToolText);
+      expect(JSON.stringify(appended.vendorRaw)).toContain(preToolText);
+    });
+    it("[23] appendAssistantToolCall omits empty pre-tool text from the wire", async () => {
+      await harness.mockScenario("tool-call");
+      const adapter = await init(harness);
+      const first = await drainStream(adapter, {
+        model: harness.toolCapableModel,
+        messages: [{ role: "user", content: "go" }],
+        tools: [
+          { name: "test_tool", description: "x", parameters: { type: "object" } }
+        ]
+      });
+      const tc = first.find(
+        (e) => e.type === "toolCall"
+      );
+      expect(tc).toBeDefined();
+      for (const blank of [void 0, "", "   "]) {
+        const history = adapter.appendAssistantToolCall(
+          [{ role: "user", content: "go" }],
+          [tc],
+          blank
+        );
+        const appended = history[history.length - 1];
+        expect(appended.content).toBe("");
+        expect(JSON.stringify(appended.vendorRaw)).not.toContain('"text":""');
+      }
+    });
   });
 }
 async function init(harness) {
@@ -3949,13 +4017,15 @@ var FakeAdapter = class {
         throw new Error(`FakeAdapter: unknown scenario '${scenario}'`);
     }
   }
-  appendAssistantToolCall(history, toolCalls) {
+  appendAssistantToolCall(history, toolCalls, assistantText) {
+    const text = assistantText?.trim() ? assistantText : null;
     return [
       ...history,
       {
         role: "assistant",
-        content: "",
+        content: text ?? "",
         vendorRaw: {
+          ...text ? { text } : {},
           role: "assistant",
           toolCalls: toolCalls.map((tc) => ({
             id: tc.id,