From 570f60b55791cccd90d9b41bd9aa371fb67f3bd2 Mon Sep 17 00:00:00 2001 From: metal Date: Wed, 29 Apr 2026 00:02:52 +0000 Subject: [PATCH] fix(sandbox): narrow 'no such file or directory' heuristic to Docker socket paths Greptile review feedback: the bare string is a generic POSIX error. Now requires it to appear alongside '/var/run/docker' or 'docker.sock' to avoid false positives. --- src/agents/sandbox/docker.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/agents/sandbox/docker.ts b/src/agents/sandbox/docker.ts index 11e6a763e0a..01bac1401cd 100644 --- a/src/agents/sandbox/docker.ts +++ b/src/agents/sandbox/docker.ts @@ -276,10 +276,12 @@ export function isDockerDaemonUnavailable(stderr: string): boolean { const lower = stderr.toLowerCase(); return ( lower.includes("cannot connect to the docker daemon") || - lower.includes("no such file or directory") || lower.includes("dial unix") || lower.includes("docker daemon is not running") || - lower.includes("connection refused") + 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")) ); }