Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

test/e2e/lib/slack-api-proof.sh (1)

214-221: ⚠️ Potential issue | 🟠 Major

Filter nonexistent find roots before invoking execFileSync.

This fallback still passes /usr/local, /tmp/npm-global, and /sandbox unconditionally. If one start directory is missing, GNU find can print a valid match from another root and still exit non-zero; execFileSync then throws, and the empty catch {} drops the discovered path. That means the flaky M-S17 failure can still survive in the new fallback.

Suggested fix

   try {
-    const discovered = execFileSync(
-      "find",
-      ["/usr/local", "/tmp/npm-global", "/sandbox", "-path", "*/dist/extensions/slack/test-api.js", "-print", "-quit"],
-      { encoding: "utf8" },
-    ).trim();
+    const searchRoots = ["/usr/local", "/tmp/npm-global", "/sandbox"].filter((root) => fs.existsSync(root));
+    const discovered = searchRoots.length
+      ? execFileSync(
+          "find",
+          [...searchRoots, "-path", "*/dist/extensions/slack/test-api.js", "-print", "-quit"],
+          { encoding: "utf8" },
+        ).trim()
+      : "";
     if (discovered) addCandidate(path.resolve(discovered, "../../../.."));
   } catch {}

Run this to verify the failure mode still exists and that the current code at Line 214-Line 221 is using unconditional roots:

#!/bin/bash
set -euo pipefail

python3 - <<'PY'
from pathlib import Path
p = Path("test/e2e/lib/slack-api-proof.sh")
lines = p.read_text(encoding="utf-8", errors="replace").splitlines()
for i in range(214, 222):
    print(f"{i:4d}: {lines[i-1]}")
PY

tmpdir="$(mktemp -d)"
mkdir -p "$tmpdir/openclaw/dist/extensions/slack"
touch "$tmpdir/openclaw/dist/extensions/slack/test-api.js"
missing="$tmpdir/does-not-exist"

set +e
find "$missing" "$tmpdir" -path '*/dist/extensions/slack/test-api.js' -print -quit > /tmp/find.out 2> /tmp/find.err
status=$?
set -e

echo "find_exit_status=$status"
echo "--- stdout ---"
cat /tmp/find.out
echo "--- stderr ---"
cat /tmp/find.err

rm -rf "$tmpdir"

Expected result: stdout contains a matching test-api.js path while find_exit_status is non-zero. In that case, execFileSync(...).trim() would throw instead of returning the discovered path.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@test/e2e/lib/slack-api-proof.sh` around lines 214 - 221, The fallback
unconditionally passes roots to execFileSync which can cause find to exit
non-zero if any root is missing; update the try block around execFileSync to
first filter the roots array (e.g. ["/usr/local","/tmp/npm-global","/sandbox"])
using fs.existsSync or fs.statSync so only existing directories are passed, and
only call execFileSync when the filtered list is non-empty; keep using
discovered and addCandidate(path.resolve(...)) as before and preserve the
existing try/catch behavior but avoid swallowing a valid discovered path by
ensuring execFileSync isn't invoked with missing roots.