fix: increase coverage of vscode-lm-format.spec.ts

Author: rrewll

src/api/providers/__tests__/vscode-lm.spec.ts

@@ -90,8 +90,8 @@ describe("VsCodeLmHandler", () => {
 	beforeEach(() => {
 		vi.clearAllMocks()
 		// Set up a default successful mock for selectChatModels before creating the handler
-		const mockModel = { ...mockLanguageModelChat }
-		;(vscode.lm.selectChatModels as Mock).mockResolvedValue([mockModel])
+		const mockModels = [{ ...mockLanguageModelChat }]
+		;(vscode.lm.selectChatModels as Mock).mockResolvedValue(mockModels)
 		handler = new VsCodeLmHandler(defaultOptions)
 	})
 

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

@@ -154,6 +154,61 @@ describe("convertToVsCodeLmMessages", () => {
 		expect(toolCall.type).toBe("tool_call")
 	})
 
+	it("should handle tool_use with non-object non-string input", () => {
+		const consoleWarnSpy = vi.spyOn(console, "warn").mockImplementation(() => {})
+
+		const messages: Anthropic.Messages.MessageParam[] = [
+			{
+				role: "assistant",
+				content: [
+					{
+						type: "tool_use",
+						id: "tool-num",
+						name: "numericTool",
+						input: 42 as any, // number is valid JSON
+					},
+				],
+			},
+		]
+
+		const result = convertToVsCodeLmMessages(messages)
+
+		expect(result).toHaveLength(1)
+		expect(result[0].role).toBe("assistant")
+		// asObjectSafe returns {} for non-object/non-string, no console.warn triggered
+		expect(consoleWarnSpy).not.toHaveBeenCalled()
+
+		consoleWarnSpy.mockRestore()
+	})
+
+	it("should log Zoo Code branded warning when asObjectSafe fails to parse invalid JSON string", () => {
+		const consoleWarnSpy = vi.spyOn(console, "warn").mockImplementation(() => {})
+
+		const messages: Anthropic.Messages.MessageParam[] = [
+			{
+				role: "assistant",
+				content: [
+					{
+						type: "tool_use",
+						id: "tool-bad",
+						name: "badJsonTool",
+						input: "not-valid-json{{{",
+					},
+				],
+			},
+		]
+
+		const result = convertToVsCodeLmMessages(messages)
+
+		expect(result).toHaveLength(1)
+		expect(consoleWarnSpy).toHaveBeenCalledWith(
+			"Zoo Code <Language Model API>: Failed to parse object:",
+			expect.any(Error),
+		)
+
+		consoleWarnSpy.mockRestore()
+	})
+
 	it("should handle image blocks with appropriate placeholders", () => {
 		const messages: Anthropic.Messages.MessageParam[] = [
 			{

src/eslint-suppressions.json

@@ -541,7 +541,7 @@
 	},
 	"api/transform/__tests__/vscode-lm-format.spec.ts": {
 		"@typescript-eslint/no-explicit-any": {
-			"count": 23
+			"count": 24
 		}
 	},
 	"api/transform/__tests__/zai-format.spec.ts": {