From a736b6eededaafdcfa3e4d30fe216fa5fc449593 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Fri, 10 Apr 2026 22:03:59 +0100
Subject: [PATCH] test: remove redundant subagent allowlist conversions

---
 ...ls.subagents.sessions-spawn.allowlist.test.ts | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/src/agents/openclaw-tools.subagents.sessions-spawn.allowlist.test.ts b/src/agents/openclaw-tools.subagents.sessions-spawn.allowlist.test.ts
index b20b13db1c2..bd9c330e904 100644
--- a/src/agents/openclaw-tools.subagents.sessions-spawn.allowlist.test.ts
+++ b/src/agents/openclaw-tools.subagents.sessions-spawn.allowlist.test.ts
@@ -182,9 +182,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
     });
     const result = await spawn({ agentId: "research" });
     expect(result).toMatchObject({ status: "forbidden" });
-    expect(String(result.error ?? "")).toContain(
-      "Sandboxed sessions cannot spawn unsandboxed subagents.",
-    );
+    expect(result.error ?? "").toContain("Sandboxed sessions cannot spawn unsandboxed subagents.");
     expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
   });
 
@@ -199,7 +197,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
     });
     const result = await spawn({ agentId: "research", sandbox: "require" });
     expect(result).toMatchObject({ status: "forbidden" });
-    expect(String(result.error ?? "")).toContain('sandbox="require"');
+    expect(result.error ?? "").toContain('sandbox="require"');
     expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
   });
 
@@ -212,7 +210,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
     });
     const result = await spawn({});
     expect(result).toMatchObject({ status: "forbidden" });
-    expect(String(result.error ?? "")).toContain("sessions_spawn requires explicit agentId");
+    expect(result.error ?? "").toContain("sessions_spawn requires explicit agentId");
     expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
   });
 
@@ -248,8 +246,8 @@ describe("subagent spawn allowlist + sandbox guards", () => {
     });
     const result = await spawn({ agentId: "Agent not found: xyz" });
     expect(result).toMatchObject({ status: "error" });
-    expect(String(result.error ?? "")).toContain("Invalid agentId");
-    expect(String(result.error ?? "")).toContain("agents_list");
+    expect(result.error ?? "").toContain("Invalid agentId");
+    expect(result.error ?? "").toContain("agents_list");
     expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
   });
 
@@ -261,7 +259,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
     });
     const result = await spawn({ agentId: "../../../etc/passwd" });
     expect(result).toMatchObject({ status: "error" });
-    expect(String(result.error ?? "")).toContain("Invalid agentId");
+    expect(result.error ?? "").toContain("Invalid agentId");
     expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
   });
 
@@ -273,7 +271,7 @@ describe("subagent spawn allowlist + sandbox guards", () => {
     });
     const result = await spawn({ agentId: "a".repeat(65) });
     expect(result).toMatchObject({ status: "error" });
-    expect(String(result.error ?? "")).toContain("Invalid agentId");
+    expect(result.error ?? "").toContain("Invalid agentId");
     expect(hoisted.callGatewayMock).not.toHaveBeenCalled();
   });