From ffb12e2cdbd628332373f0d76f7928c673bf38c8 Mon Sep 17 00:00:00 2001 From: Priyanshu Choudhary <57816400+Priyanchew@users.noreply.github.com> Date: Tue, 5 May 2026 01:40:46 +0530 Subject: [PATCH] test(core): update orchestrator-prompt test for cross-platform runtime warning The orchestrator system prompt was rewritten in 1d8c8f75 to call out both tmux send-keys (Unix) and the Windows named-pipe write path so the orchestrator agent doesn't try either. The test still asserted the old literal "never use raw \`tmux send-keys\`" string. Update it to assert the new platform-neutral phrasing plus the presence of both runtime mentions. Co-Authored-By: Claude Sonnet 4.6 --- packages/core/src/__tests__/orchestrator-prompt.test.ts | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packages/core/src/__tests__/orchestrator-prompt.test.ts b/packages/core/src/__tests__/orchestrator-prompt.test.ts index 268ce1124..0ba2248b0 100644 --- a/packages/core/src/__tests__/orchestrator-prompt.test.ts +++ b/packages/core/src/__tests__/orchestrator-prompt.test.ts @@ -64,7 +64,7 @@ describe("generateOrchestratorPrompt", () => { expect(prompt).toContain("do not edit repository files or implement fixes"); }); - it("mandates ao send and bans raw tmux access", async () => { + it("mandates ao send and bans raw runtime access", async () => { const generateOrchestratorPrompt = await loadGenerateOrchestratorPrompt(); const prompt = generateOrchestratorPrompt({ config, @@ -73,7 +73,11 @@ describe("generateOrchestratorPrompt", () => { }); expect(prompt).toContain("Always use `ao send`"); - expect(prompt).toContain("never use raw `tmux send-keys`"); + // Platform-neutral runtime warning — must call out both tmux (Unix) and the + // Windows named-pipe path so the orchestrator agent doesn't try either. + expect(prompt).toContain("never bypass it by writing to the runtime layer directly"); + expect(prompt).toContain("tmux send-keys"); + expect(prompt).toContain("named pipe"); expect(prompt).toContain("ao send --no-wait"); });