vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-03-28 18:33:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(agents): use claude-cli backend in tools-disabled regression test

Browse Source 
codex-cli lacks systemPromptArg, so the system prompt is never
serialized into argv — making the not-toContain assertion pass
vacuously even on pre-fix code. Switch to claude-cli which defines
systemPromptArg ("--append-system-prompt") and add a positive
assertion that the user-supplied prompt IS present in argv.

Co-Authored-By: Dhiman's Agentic Suite <dhiman.seal@hotmail.com>
This commit is contained in:
dhi13man
2026-03-15 12:55:13 +05:30
committed by Peter Steinberger
parent 99f0ea8d43
commit 9f8c4efa9b
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/agents/cli-runner.test.ts
View File

@@ -250,8 +250,8 @@ describe("runCliAgent with process supervisor", () => {
sessionFile: "/tmp/session.jsonl",
workspaceDir: "/tmp",
prompt: "Run: node script.mjs",
provider: "codex-cli",
model: "gpt-5.2-codex",
provider: "claude-cli",
model: "sonnet",
timeoutMs: 1_000,
runId: "run-no-tools-disabled",
extraSystemPrompt: "You are a helpful assistant.",
@@ -262,12 +262,14 @@ describe("runCliAgent with process supervisor", () => {
argv?: string[];
input?: string;
};
// The CLI runner must not inject "Tools are disabled" into the system
// prompt passed to CLI backends. CLI backends (e.g., Claude Code CLI)
// manage their own native tools; the injected text caused them to
// refuse using their own tools. See: openclaw/openclaw#44135
// Use claude-cli because it defines systemPromptArg ("--append-system-prompt"),
// so the system prompt is serialized into argv. The codex-cli backend lacks
// systemPromptArg, meaning the prompt is dropped before reaching argv —
// making the assertion vacuous. See: openclaw/openclaw#44135
const allArgs = (input.argv ?? []).join("\n");
expect(allArgs).not.toContain("Tools are disabled in this session");
// Verify the user-supplied system prompt IS present (proves the arg path works)
expect(allArgs).toContain("You are a helpful assistant.");
});
it("runs CLI through supervisor and returns payload", async () => {