test: bumping coverage

Author: edelauna

src/api/providers/__tests__/native-ollama.spec.ts

@@ -81,6 +81,54 @@ describe("NativeOllamaHandler", () => {
 			expect(results[2]).toEqual({ type: "usage", inputTokens: 10, outputTokens: 2 })
 		})
 
+		it("should map tool_result array content to a concatenated string (text kept, others dropped)", async () => {
+			mockChat.mockImplementation(async function* () {
+				yield { message: { content: "ok" } }
+			})
+
+			const messages = [
+				{
+					role: "user" as const,
+					content: [
+						{
+							type: "tool_result" as const,
+							tool_use_id: "tool-1",
+							content: [
+								{ type: "text" as const, text: "line one" },
+								// Non-text block (document) should be dropped to ""
+								{
+									type: "document" as const,
+									source: {
+										type: "base64" as const,
+										media_type: "application/pdf",
+										data: "abc",
+									} as const,
+								},
+								{ type: "text" as const, text: "line two" },
+							],
+						},
+					],
+				},
+			]
+
+			const stream = handler.createMessage("System", messages)
+			for await (const _ of stream) {
+				// consume stream
+			}
+
+			// Text blocks are joined with "\n"; the non-text block contributes ""
+			expect(mockChat).toHaveBeenCalledWith(
+				expect.objectContaining({
+					messages: expect.arrayContaining([
+						expect.objectContaining({
+							role: "user",
+							content: "line one\n\nline two",
+						}),
+					]),
+				}),
+			)
+		})
+
 		it("should not include num_ctx by default", async () => {
 			// Mock the chat response
 			mockChat.mockImplementation(async function* () {

src/api/transform/__tests__/openai-format.spec.ts

@@ -1,4 +1,4 @@
-// npx vitest run api/transform/__tests__/openai-format.spec.ts
+// pnpm exec vitest run api/transform/__tests__/openai-format.spec.ts
 
 import { Anthropic } from "@anthropic-ai/sdk"
 import OpenAI from "openai"
@@ -143,7 +143,33 @@ describe("convertToOpenAiMessages", () => {
 		expect((openAiMessages[0] as any).content).toBe("some text\n")
 	})
 
-	it("should handle base64 image in tool result with placeholder and skip URL images", () => {
+	it("should handle base64 image in tool result with placeholder", () => {
+		const anthropicMessages: Anthropic.Messages.MessageParam[] = [
+			{
+				role: "user",
+				content: [
+					{
+						type: "tool_result",
+						tool_use_id: "tool-1",
+						content: [
+							{
+								type: "image",
+								source: { type: "base64", media_type: "image/png", data: "base64data" },
+							},
+						],
+					},
+				],
+			},
+		]
+
+		const openAiMessages = convertToOpenAiMessages(anthropicMessages)
+		expect(openAiMessages[0].role).toBe("tool")
+		// base64 images in tool results emit a placeholder; the image itself is
+		// flushed in a separate user message (see comment block in openai-format.ts)
+		expect((openAiMessages[0] as any).content).toBe("(see following user message for image)")
+	})
+
+	it("should render [Image] placeholder for URL image in tool result", () => {
 		const anthropicMessages: Anthropic.Messages.MessageParam[] = [
 			{
 				role: "user",

src/api/transform/__tests__/vscode-lm-format.spec.ts

@@ -1,4 +1,4 @@
-// npx vitest run src/api/transform/__tests__/vscode-lm-format.spec.ts
+// pnpm exec vitest run api/transform/__tests__/vscode-lm-format.spec.ts
 
 import { Anthropic } from "@anthropic-ai/sdk"
 import * as vscode from "vscode"

src/api/transform/__tests__/zai-format.spec.ts

@@ -1,4 +1,4 @@
-// npx vitest run api/transform/__tests__/zai-format.spec.ts
+// pnpm exec vitest run api/transform/__tests__/zai-format.spec.ts
 
 import { Anthropic } from "@anthropic-ai/sdk"
 

src/integrations/misc/__tests__/export-markdown.spec.ts

@@ -76,8 +76,8 @@ describe("export-markdown", () => {
 		it("should format document blocks", () => {
 			const block = {
 				type: "document",
-				source: { type: "base64", media_type: "application/pdf", data: "abc" },
-			} as any as ExtendedContentBlock
+				source: { type: "base64", media_type: "application/pdf", data: "abc" } as const,
+			} satisfies ExtendedContentBlock
 			expect(formatContentBlockToMarkdown(block)).toBe("[Document]")
 		})
 
@@ -86,17 +86,16 @@ describe("export-markdown", () => {
 				type: "search_result",
 				source: "https://example.com",
 				title: "Example",
-				content: [],
-			} as any as ExtendedContentBlock
+				content: [{ type: "text", text: "result text" }],
+			} satisfies ExtendedContentBlock
 			expect(formatContentBlockToMarkdown(block)).toBe("[Search Result]")
 		})
 
 		it("should format tool_reference blocks", () => {
 			const block = {
 				type: "tool_reference",
-				id: "tool-1",
-				name: "read_file",
-			} as any as ExtendedContentBlock
+				tool_name: "read_file",
+			} satisfies ExtendedContentBlock
 			expect(formatContentBlockToMarkdown(block)).toBe("[Tool Reference]")
 		})
 	})