From ba708e752898660099161fd406e00942285a2141 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Fri, 9 Jan 2026 18:31:22 +0000 Subject: [PATCH] Fix type safety hack in memoization test Refactored `trackingExecute` in `src/runner/memoization.test.ts` to match the `executeLlmStep` signature, allowing the removal of a `@ts-ignore` suppression. --- src/runner/memoization.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/runner/memoization.test.ts b/src/runner/memoization.test.ts index 86b04be..5c23ab8 100644 --- a/src/runner/memoization.test.ts +++ b/src/runner/memoization.test.ts @@ -86,13 +86,13 @@ describe('Workflow Memoization (Auto-Hashing)', () => { // We can check if `executeLlmStep` was called. let called = false; - const trackingExecute = async (s: any, c: any) => { + // Match signature of executeLlmStep (at least the required args) + const trackingExecute = async (s: any, c: any, _execFn: any, ..._args: any[]) => { called = true; return mockExecuteLlmStep(s, c); }; // Override the executor for runner2 to track calls - // @ts-ignore - hacking private property or constructor option // Actually we passed it in constructor option. const runner2Tracked = new WorkflowRunner(workflow, { dbPath,