From 9aceb0ef52315e54e43e47e93ff14518eb25cdfd Mon Sep 17 00:00:00 2001 From: Ammar Date: Fri, 17 Oct 2025 18:53:54 -0500 Subject: [PATCH] =?UTF-8?q?=F0=9F=A4=96=20Refine=20bash=20sleep=20error=20?= =?UTF-8?q?to=20encourage=20busy=20loops?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/services/tools/bash.test.ts | 4 ++-- src/services/tools/bash.ts | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/services/tools/bash.test.ts b/src/services/tools/bash.test.ts index 49a6a0a4c..f1da9ec75 100644 --- a/src/services/tools/bash.test.ts +++ b/src/services/tools/bash.test.ts @@ -894,8 +894,8 @@ describe("bash tool", () => { expect(result.success).toBe(false); if (!result.success) { - expect(result.error).toContain("sleep commands are blocked"); - expect(result.error).toContain("polling loops"); + expect(result.error).toContain("do not start commands with sleep"); + expect(result.error).toContain("prefer <10s sleeps in busy loops"); expect(result.error).toContain("while ! condition"); expect(result.exitCode).toBe(-1); expect(result.wall_duration_ms).toBe(0); diff --git a/src/services/tools/bash.ts b/src/services/tools/bash.ts index 54a2e7189..8e5143244 100644 --- a/src/services/tools/bash.ts +++ b/src/services/tools/bash.ts @@ -79,7 +79,7 @@ export const createBashTool: ToolFactory = (config: ToolConfiguration) => { return { success: false, error: - "sleep commands are blocked to minimize waiting time. Instead, use polling loops to check conditions repeatedly (e.g., 'while ! condition; do sleep 1; done' or 'until condition; do sleep 1; done').", + "do not start commands with sleep; prefer <10s sleeps in busy loops (e.g., 'while ! condition; do sleep 1; done' or 'until condition; do sleep 1; done').", exitCode: -1, wall_duration_ms: 0, };