From b64e266a7a6074413f5ada5e2c6e1907be2f7ad9 Mon Sep 17 00:00:00 2001 From: metal Date: Wed, 29 Apr 2026 00:03:22 +0000 Subject: [PATCH] fix(sandbox): remove redundant 'no such file or directory' check from isDockerDaemonUnavailable Greptile review: 'dial unix' already covers missing Docker socket case (e.g. 'dial unix /var/run/docker.sock: connect: no such file or directory'). The bare string is also too generic and could cause false positives. --- src/agents/sandbox/docker.ts | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/agents/sandbox/docker.ts b/src/agents/sandbox/docker.ts index 01bac1401cd..51910f460bf 100644 --- a/src/agents/sandbox/docker.ts +++ b/src/agents/sandbox/docker.ts @@ -278,10 +278,7 @@ export function isDockerDaemonUnavailable(stderr: string): boolean { lower.includes("cannot connect to the docker daemon") || lower.includes("dial unix") || lower.includes("docker daemon is not running") || - lower.includes("connection refused") || - // Docker socket path errors — narrow enough to avoid false positives - lower.includes("no such file or directory") && - (lower.includes("/var/run/docker") || lower.includes("docker.sock")) + lower.includes("connection refused") ); }