Fix Android node shell wrapper

Xila · Xila · commit 432adcd6c992 · 2026-04-29T07:51:09.000+08:00
diff --git a/docs/cli/node.md b/docs/cli/node.md
@@ -67,6 +67,7 @@ Options:
 
 `openclaw node run` and `openclaw node install` resolve gateway auth from config/env (no `--token`/`--password` flags on node commands):
 
+- Android/Termux node hosts use `/data/data/com.termux/files/usr/bin/sh` as the shell wrapper for `system.run`.
 - `OPENCLAW_GATEWAY_TOKEN` / `OPENCLAW_GATEWAY_PASSWORD` are checked first.
 - Then local config fallback: `gateway.auth.token` / `gateway.auth.password`.
 - In local mode, node host intentionally does not inherit `gateway.remote.token` / `gateway.remote.password`.
diff --git a/docs/nodes/index.md b/docs/nodes/index.md
@@ -419,6 +419,7 @@ Notes:
 - The node host stores its node id, token, display name, and gateway connection info in `~/.openclaw/node.json`.
 - Exec approvals are enforced locally via `~/.openclaw/exec-approvals.json`
   (see [Exec approvals](/tools/exec-approvals)).
+- Android/Termux node hosts use `/data/data/com.termux/files/usr/bin/sh` as the shell wrapper for `system.run`.
 - On macOS, the headless node host executes `system.run` locally by default. Set
   `OPENCLAW_NODE_EXEC_HOST=app` to route `system.run` through the companion app exec host; add
   `OPENCLAW_NODE_EXEC_FALLBACK=0` to require the app host and fail closed if it is unavailable.
diff --git a/src/infra/node-shell.test.ts b/src/infra/node-shell.test.ts
@@ -1,5 +1,5 @@
 import { describe, expect, it } from "vitest";
-import { buildNodeShellCommand } from "./node-shell.js";
+import { ANDROID_TERMUX_SHELL, buildNodeShellCommand } from "./node-shell.js";
 
 describe("buildNodeShellCommand", () => {
   it("uses cmd.exe for win-prefixed platform labels", () => {
@@ -26,7 +26,20 @@ describe("buildNodeShellCommand", () => {
     ]);
   });
 
-  it("uses /bin/sh for non-windows and missing platform values", () => {
+  it("uses the Termux shell for Android nodes", () => {
+    expect(buildNodeShellCommand("echo hi", "android")).toEqual([
+      ANDROID_TERMUX_SHELL,
+      "-lc",
+      "echo hi",
+    ]);
+    expect(buildNodeShellCommand("echo hi", " Android ")).toEqual([
+      ANDROID_TERMUX_SHELL,
+      "-lc",
+      "echo hi",
+    ]);
+  });
+
+  it("uses /bin/sh for non-windows, non-android, and missing platform values", () => {
     expect(buildNodeShellCommand("echo hi", "darwin")).toEqual(["/bin/sh", "-lc", "echo hi"]);
     expect(buildNodeShellCommand("echo hi", "linux")).toEqual(["/bin/sh", "-lc", "echo hi"]);
     expect(buildNodeShellCommand("echo hi")).toEqual(["/bin/sh", "-lc", "echo hi"]);
diff --git a/src/infra/node-shell.ts b/src/infra/node-shell.ts
@@ -1,9 +1,14 @@
 import { normalizeLowercaseStringOrEmpty } from "../shared/string-coerce.js";
 
+export const ANDROID_TERMUX_SHELL = "/data/data/com.termux/files/usr/bin/sh";
+
 export function buildNodeShellCommand(command: string, platform?: string | null) {
   const normalized = normalizeLowercaseStringOrEmpty((platform ?? "").trim());
   if (normalized.startsWith("win")) {
     return ["cmd.exe", "/d", "/s", "/c", command];
   }
+  if (normalized === "android") {
+    return [ANDROID_TERMUX_SHELL, "-lc", command];
+  }
   return ["/bin/sh", "-lc", command];
 }

Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,14 @@`
`1`	`1`	`import { normalizeLowercaseStringOrEmpty } from "../shared/string-coerce.js";`
`2`	`2`
	`3`	`+export const ANDROID_TERMUX_SHELL = "/data/data/com.termux/files/usr/bin/sh";`
	`4`	`+`
`3`	`5`	`export function buildNodeShellCommand(command: string, platform?: string \| null) {`
`4`	`6`	`const normalized = normalizeLowercaseStringOrEmpty((platform ?? "").trim());`
`5`	`7`	`if (normalized.startsWith("win")) {`
`6`	`8`	`return ["cmd.exe", "/d", "/s", "/c", command];`
`7`	`9`	`}`
	`10`	`+ if (normalized === "android") {`
	`11`	`+ return [ANDROID_TERMUX_SHELL, "-lc", command];`
	`12`	`+ }`
`8`	`13`	`return ["/bin/sh", "-lc", command];`
`9`	`14`	`}`