fix(auto-reply): resolve scp from path for media staging

steipete · steipete · commit 56c2e20b6fdb · 2026-05-08T04:04:10.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -163,6 +163,7 @@ Docs: https://docs.openclaw.ai
 
 ### Fixes
 
+- Auto-reply/media: resolve `scp` from `PATH` when staging sandbox media so nonstandard OpenSSH installs can copy remote attachments.
 - Gateway/watch: leave `OPENCLAW_TRACE_SYNC_IO` disabled by default in `pnpm gateway:watch:raw` so watch mode avoids noisy Node sync-I/O stack traces unless explicitly requested.
 - Codex app-server: close stdio stdin before force-killing the managed app-server, matching Codex single-client shutdown behavior and avoiding unsettled CLI exits after successful runs.
 - CLI/Codex: dispose registered agent harnesses during short-lived CLI shutdown so successful Codex-backed `agent --local` runs do not leave app-server child processes alive.
diff --git a/src/auto-reply/reply.stage-sandbox-media.scp-remote-path.test.ts b/src/auto-reply/reply.stage-sandbox-media.scp-remote-path.test.ts
@@ -107,6 +107,7 @@ describe("stageSandboxMedia scp remote paths", () => {
         workspaceDir,
       });
 
+      expect(childProcessMocks.spawn.mock.calls[0]?.[0]).toBe("scp");
       const remoteCacheRoot = join(CONFIG_DIR, "media", "remote-cache");
       const expectedSafeDir = join(remoteCacheRoot, slugifySessionKey(sessionKey));
       try {
diff --git a/src/auto-reply/reply/stage-sandbox-media.ts b/src/auto-reply/reply/stage-sandbox-media.ts
@@ -319,7 +319,7 @@ async function scpFile(remoteHost: string, remotePath: string, localPath: string
   }
   return new Promise((resolve, reject) => {
     const child = spawn(
-      "/usr/bin/scp",
+      "scp",
       [
         "-o",
         "BatchMode=yes",

Original file line number	Diff line number	Diff line change
`@@ -319,7 +319,7 @@ async function scpFile(remoteHost: string, remotePath: string, localPath: string`
`319`	`319`	`}`
`320`	`320`	`return new Promise((resolve, reject) => {`
`321`	`321`	`const child = spawn(`
`322`		`- "/usr/bin/scp",`
	`322`	`+ "scp",`
`323`	`323`	`[`
`324`	`324`	`"-o",`
`325`	`325`	`"BatchMode=yes",`