From 243293e5b033bf2d41e1bbd8544c4ff67ff86c8c Mon Sep 17 00:00:00 2001 From: Lex Christopherson Date: Sat, 21 Mar 2026 09:59:38 -0600 Subject: [PATCH] fix: resolve extension typecheck errors in test files - await-tool.test.ts: widen getTextFromResult param to accept ImageContent (text optional) - auto-loop.test.ts: add missing rebuildState and resolveModelId to LoopDeps mock Co-Authored-By: Claude Opus 4.6 (1M context) --- src/resources/extensions/async-jobs/await-tool.test.ts | 4 ++-- src/resources/extensions/gsd/tests/auto-loop.test.ts | 2 ++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/resources/extensions/async-jobs/await-tool.test.ts b/src/resources/extensions/async-jobs/await-tool.test.ts index 524b54048..3a93c4569 100644 --- a/src/resources/extensions/async-jobs/await-tool.test.ts +++ b/src/resources/extensions/async-jobs/await-tool.test.ts @@ -7,8 +7,8 @@ import assert from "node:assert/strict"; import { AsyncJobManager } from "./job-manager.ts"; import { createAwaitTool } from "./await-tool.ts"; -function getTextFromResult(result: { content: Array<{ type: string; text: string }> }): string { - return result.content.map((c) => c.text).join("\n"); +function getTextFromResult(result: { content: Array<{ type: string; text?: string }> }): string { + return result.content.map((c) => c.text ?? "").join("\n"); } const noopSignal = new AbortController().signal; diff --git a/src/resources/extensions/gsd/tests/auto-loop.test.ts b/src/resources/extensions/gsd/tests/auto-loop.test.ts index 42e96393f..49805d22c 100644 --- a/src/resources/extensions/gsd/tests/auto-loop.test.ts +++ b/src/resources/extensions/gsd/tests/auto-loop.test.ts @@ -414,6 +414,8 @@ function makeMockDeps( return "continue" as const; }, getSessionFile: () => "/tmp/session.json", + rebuildState: async () => {}, + resolveModelId: (id: string, models: any[]) => models.find((m: any) => m.id === id), }; const merged = { ...baseDeps, ...overrides, callLog };