From fd8593b190cbc23b7eb5f1ccf36d9e3c8a7280fe Mon Sep 17 00:00:00 2001 From: oldschoola Date: Mon, 25 May 2026 14:46:45 -0700 Subject: [PATCH] test(coding-agent): pass autoApprove context to direct bash call in sdk-move-cwd sdk-move-cwd.test.ts also calls bashTool.execute directly without context, hitting the same approval gate as the eval cleanup tests. Same fix. --- packages/coding-agent/test/sdk-move-cwd.test.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/coding-agent/test/sdk-move-cwd.test.ts b/packages/coding-agent/test/sdk-move-cwd.test.ts index 1e634b010..a695a267d 100644 --- a/packages/coding-agent/test/sdk-move-cwd.test.ts +++ b/packages/coding-agent/test/sdk-move-cwd.test.ts @@ -2,6 +2,7 @@ import { afterEach, describe, expect, it } from "bun:test"; import * as fs from "node:fs"; import * as os from "node:os"; import * as path from "node:path"; +import type { AgentToolContext } from "@oh-my-pi/pi-agent-core"; import { getBundledModel } from "@oh-my-pi/pi-ai"; import { Settings } from "@oh-my-pi/pi-coding-agent/config/settings"; import { createAgentSession } from "@oh-my-pi/pi-coding-agent/sdk"; @@ -62,7 +63,9 @@ describe("createAgentSession cwd after /move", () => { const bashTool = session.getToolByName("bash"); if (!bashTool) throw new Error("Expected bash tool"); - const result = await bashTool.execute("pwd-after-move", { command: "pwd" }); + const result = await bashTool.execute("pwd-after-move", { command: "pwd" }, undefined, undefined, { + autoApprove: true, + } as AgentToolContext); expect(textContent(result)).toContain(cwdB); } finally {