From bc4aa4aaa1eb02ca68b7e73ae2df6036c651bb96 Mon Sep 17 00:00:00 2001 From: Mario Zechner Date: Tue, 2 Dec 2025 14:50:29 +0100 Subject: [PATCH] fix(coding-agent): use describe.skipIf for RPC test with explicit model - Use describe.skipIf pattern matching other tests - Skip when ANTHROPIC_API_KEY and ANTHROPIC_OAUTH_TOKEN are missing - Use explicit --provider anthropic --model claude-sonnet-4-5 --- packages/coding-agent/test/rpc.test.ts | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/packages/coding-agent/test/rpc.test.ts b/packages/coding-agent/test/rpc.test.ts index de45a81b..cd13a469 100644 --- a/packages/coding-agent/test/rpc.test.ts +++ b/packages/coding-agent/test/rpc.test.ts @@ -7,16 +7,13 @@ import { fileURLToPath } from "node:url"; import type { AgentEvent } from "@mariozechner/pi-agent-core"; import { afterEach, beforeEach, describe, expect, test } from "vitest"; -// Skip RPC integration test on CI runners; it depends on external LLM calls and can exit early -const maybeDescribe = process.env.CI ? describe.skip : describe; - const __dirname = dirname(fileURLToPath(import.meta.url)); /** * RPC mode tests. * Regression test for issue #83: https://github.com/badlogic/pi-mono/issues/83 */ -maybeDescribe("RPC mode", () => { +describe.skipIf(!process.env.ANTHROPIC_API_KEY && !process.env.ANTHROPIC_OAUTH_TOKEN)("RPC mode", () => { let agent: ChildProcess; let sessionDir: string; @@ -38,13 +35,17 @@ maybeDescribe("RPC mode", () => { test("should save messages to session file", async () => { // Spawn agent in RPC mode with custom session directory - agent = spawn("node", ["dist/cli.js", "--mode", "rpc"], { - cwd: join(__dirname, ".."), - env: { - ...process.env, - PI_CODING_AGENT_DIR: sessionDir, + agent = spawn( + "node", + ["dist/cli.js", "--mode", "rpc", "--provider", "anthropic", "--model", "claude-sonnet-4-5"], + { + cwd: join(__dirname, ".."), + env: { + ...process.env, + PI_CODING_AGENT_DIR: sessionDir, + }, }, - }); + ); const events: AgentEvent[] = [];