From 246062c86f12620776a825b1d3308f4c169beec9 Mon Sep 17 00:00:00 2001 From: cte Date: Tue, 6 Jan 2026 21:30:59 -0800 Subject: [PATCH] Fix tsc errors --- .../__tests__/consolidateApiRequests.spec.ts | 18 +++++------ .../__tests__/consolidateCommands.spec.ts | 30 +++++++++---------- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/packages/core/src/message-utils/__tests__/consolidateApiRequests.spec.ts b/packages/core/src/message-utils/__tests__/consolidateApiRequests.spec.ts index 363a0d0985..1ee5cf3b7d 100644 --- a/packages/core/src/message-utils/__tests__/consolidateApiRequests.spec.ts +++ b/packages/core/src/message-utils/__tests__/consolidateApiRequests.spec.ts @@ -38,9 +38,9 @@ describe("consolidateApiRequests", () => { const result = consolidateApiRequests(messages) expect(result.length).toBe(1) - expect(result[0].say).toBe("api_req_started") + expect(result[0]!.say).toBe("api_req_started") - const parsedText = JSON.parse(result[0].text || "{}") + const parsedText = JSON.parse(result[0]!.text || "{}") expect(parsedText.request).toBe("GET /api/data") expect(parsedText.cost).toBe(0.005) }) @@ -75,9 +75,9 @@ describe("consolidateApiRequests", () => { const result = consolidateApiRequests(messages) expect(result.length).toBe(3) - expect(result[0].text).toBe("Before") - expect(result[1].say).toBe("api_req_started") - expect(result[2].text).toBe("After") + expect(result[0]!.text).toBe("Before") + expect(result[1]!.say).toBe("api_req_started") + expect(result[2]!.text).toBe("After") }) it("should handle multiple api_req pairs", () => { @@ -91,8 +91,8 @@ describe("consolidateApiRequests", () => { const result = consolidateApiRequests(messages) expect(result.length).toBe(2) - expect(JSON.parse(result[0].text || "{}").request).toBe("first") - expect(JSON.parse(result[1].text || "{}").request).toBe("second") + expect(JSON.parse(result[0]!.text || "{}").request).toBe("first") + expect(JSON.parse(result[1]!.text || "{}").request).toBe("second") }) it("should handle orphan api_req_started without finish", () => { @@ -104,8 +104,8 @@ describe("consolidateApiRequests", () => { const result = consolidateApiRequests(messages) expect(result.length).toBe(2) - expect(result[0].say).toBe("api_req_started") - expect(JSON.parse(result[0].text || "{}").request).toBe("orphan") + expect(result[0]!.say).toBe("api_req_started") + expect(JSON.parse(result[0]!.text || "{}").request).toBe("orphan") }) it("should handle invalid JSON in message text", () => { diff --git a/packages/core/src/message-utils/__tests__/consolidateCommands.spec.ts b/packages/core/src/message-utils/__tests__/consolidateCommands.spec.ts index 8b11a858de..ae73dc8979 100644 --- a/packages/core/src/message-utils/__tests__/consolidateCommands.spec.ts +++ b/packages/core/src/message-utils/__tests__/consolidateCommands.spec.ts @@ -16,8 +16,8 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(1) - expect(result[0].ask).toBe("command") - expect(result[0].text).toBe(`ls\n${COMMAND_OUTPUT_STRING}file1.txt\nfile2.txt`) + expect(result[0]!.ask).toBe("command") + expect(result[0]!.text).toBe(`ls\n${COMMAND_OUTPUT_STRING}file1.txt\nfile2.txt`) }) it("should handle multiple command sequences", () => { @@ -31,8 +31,8 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(2) - expect(result[0].text).toBe(`ls\n${COMMAND_OUTPUT_STRING}output1`) - expect(result[1].text).toBe(`pwd\n${COMMAND_OUTPUT_STRING}output2`) + expect(result[0]!.text).toBe(`ls\n${COMMAND_OUTPUT_STRING}output1`) + expect(result[1]!.text).toBe(`pwd\n${COMMAND_OUTPUT_STRING}output2`) }) it("should handle command without output", () => { @@ -44,9 +44,9 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(2) - expect(result[0].ask).toBe("command") - expect(result[0].text).toBe("ls") - expect(result[1].say).toBe("text") + expect(result[0]!.ask).toBe("command") + expect(result[0]!.text).toBe("ls") + expect(result[1]!.say).toBe("text") }) it("should handle duplicate outputs (ask and say with same text)", () => { @@ -59,7 +59,7 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(1) - expect(result[0].text).toBe(`ls\n${COMMAND_OUTPUT_STRING}same output`) + expect(result[0]!.text).toBe(`ls\n${COMMAND_OUTPUT_STRING}same output`) }) }) @@ -78,8 +78,8 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(1) - expect(result[0].ask).toBe("use_mcp_server") - const parsed = JSON.parse(result[0].text || "{}") + expect(result[0]!.ask).toBe("use_mcp_server") + const parsed = JSON.parse(result[0]!.text || "{}") expect(parsed.server).toBe("test") expect(parsed.response).toBe("response data") }) @@ -97,7 +97,7 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(1) - expect(result[0].ask).toBe("use_mcp_server") + expect(result[0]!.ask).toBe("use_mcp_server") }) it("should handle multiple MCP responses", () => { @@ -115,7 +115,7 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(1) - const parsed = JSON.parse(result[0].text || "{}") + const parsed = JSON.parse(result[0]!.text || "{}") expect(parsed.response).toBe("response1\nresponse2") }) }) @@ -132,9 +132,9 @@ describe("consolidateCommands", () => { const result = consolidateCommands(messages) expect(result.length).toBe(3) - expect(result[0].text).toBe("before") - expect(result[1].text).toBe(`ls\n${COMMAND_OUTPUT_STRING}output`) - expect(result[2].text).toBe("after") + expect(result[0]!.text).toBe("before") + expect(result[1]!.text).toBe(`ls\n${COMMAND_OUTPUT_STRING}output`) + expect(result[2]!.text).toBe("after") }) it("should handle empty array", () => {