PR #60923: fix(cli): set non-zero exit code on argument errors

• Repository: openclaw/openclaw
• Author: Linux2010
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/60923

Description (problem / solution / changelog)

Summary

Fixes #60905

When CLI commands receive invalid arguments (e.g., openclaw sessions list), Commander.js prints an error but was returning exit code 0. This broke scripts and monitoring tools that rely on exit codes to detect failures.

Solution

Call program.exitOverride() with a callback that sets process.exitCode = 1 for all errors except help/version displays.

Before

$ openclaw sessions list 2>&1
error: too many arguments for 'sessions'. Expected 0 arguments but got 1.
$ echo $?
0

After

$ openclaw sessions list 2>&1
error: too many arguments for 'sessions'. Expected 0 arguments but got 1.
$ echo $?
1

Test Coverage

Added 3 new unit tests to verify exit code behavior:

• ✅ Argument errors set exitCode = 1
• ✅ --help sets exitCode = 0 (handles commander.helpDisplayed)
• ✅ --version sets exitCode = 0

Test Plan

• Unit tests pass: pnpm test src/cli/program/build-program.test.ts
• Manual test confirms exit code is now 1 on argument errors
• Help (--help) still exits with code 0
• Version (--version) still exits with code 0

Changed files

• src/cli/program/build-program.test.ts (modified, +52/-1)
• src/cli/program/build-program.ts (modified, +14/-0)

PR #60949: fix(cli): show friendly error for sessions list instead of generic too many arguments

• Repository: openclaw/openclaw
• Author: merlinmiao
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/60949

Description (problem / solution / changelog)

Summary

When users run openclaw sessions list, Commander rejects it as "too many arguments for sessions. Expected 0 arguments" without explaining that sessions itself is the list command and list is not a valid subcommand.

This PR adds a dedicated route for sessions list that produces a clear error message guiding users to the correct usage (openclaw sessions without list). The route exits with code 1 to properly signal an error condition.

Fixes #60905

Test plan

• sessions list shows friendly error and exits with code 1
• sessions (bare) still works correctly and exits with code 0
• All existing tests pass

Changed files

• pnpm-lock.yaml (modified, +2/-0)
• src/cli/program/routes.ts (modified, +13/-0)