PR #84039: fix(cli): honor --no-prefix-cwd in acp

• Repository: openclaw/openclaw
• Author: extrasmall0
• State: closed | merged: True
• Link: https://github.com/openclaw/openclaw/pull/84039

Description (problem / solution / changelog)

Summary

• Let Commander own the negated option default for --no-prefix-cwd.
• Read the positive prefixCwd option that Commander actually populates.
• Add ACP CLI coverage for both the explicit opt-out and the default behavior.

Closes #83901.

Verification

• node scripts/run-vitest.mjs src/cli/acp-cli.option-collisions.test.ts src/acp/translator.prompt-prefix.test.ts
• Crabbox AWS cbx_1689d0ad78e9, run run_a406418db6fe: regression failed before the source patch, then the focused ACP CLI and translator tests passed after the patch (15/15).

Real behavior proof

Behavior addressed: openclaw acp --no-prefix-cwd now forwards prefixCwd=false into the ACP bridge so prompts are not prefixed with [Working directory: ...]; the default path still forwards prefixCwd=true. Real environment tested: Fresh checkout of this PR branch after pnpm install and pnpm build, using a live ACP translator harness from the checkout; maintainer follow-up also verified focused behavior on AWS Crabbox Linux lease cbx_1689d0ad78e9. Exact steps or command run after this patch: Build the PR checkout, run the ACP translator harness once with the default prefixCwd path and once with the --no-prefix-cwd / prefixCwd=false path, then run node scripts/run-vitest.mjs src/cli/acp-cli.option-collisions.test.ts src/acp/translator.prompt-prefix.test.ts -- --reporter=verbose. Evidence after fix: The live ACP translator harness printed this after-fix output:

default prefixCwd=true
[Working directory: ~/openclaw-test]

hello
---
with --no-prefix-cwd / prefixCwd=false
hello

Observed result after fix: The default path still prefixes the working directory, and the --no-prefix-cwd path suppresses that prefix. Focused tests also passed with 15 passed; the same regression test failed before the source patch with prefixCwd received as true. What was not tested: No long-running interactive ACP server was left running; the live proof used a bounded ACP translator harness and the maintainer proof used focused CLI/translator tests.

Changed files

• src/cli/acp-cli.option-collisions.test.ts (modified, +21/-0)
• src/cli/acp-cli.ts (modified, +2/-2)