[facundofarias]

Problem

Every CliError printed twice with a stack trace, and the exit code was always 1:

 ERROR  Not authenticated. Run pixelvault register ...
    at m (dist/index.js:...)
    at runCommand (node_modules/citty/...)
 ...
 ERROR  Not authenticated. Run pixelvault register ...

Cause: citty's runMain handles errors itself. For any non-citty error it logs the error with its stack, logs the message again, and calls process.exit(1). So the .catch after runMain in index.ts never ran, and CliError.exitCode was ignored.

Fix

New src/lib/run.ts (runCli), used by index.ts:

• --help / --version → still runMain (it renders them well).
• Everything else → runCommand, with our own error handling:
  • CliError → one Error: … line on stderr, exit with its exitCode.
  • citty usage errors (missing argument, unknown command) → replayed through runMain for its usage rendering. These are thrown while parsing, before any command body runs, so nothing with side effects is re-executed (covered by a test).
  • Any other error → one line, exit 1.

Test plan

• typecheck, build, 62 tests (new tests/lib/run.test.ts: single line / no stack / exitCode / no re-run / success / usage error)
• Built binary: pixelvault usage with no key prints one line and exits 1; --version, --help unchanged

🤖 Generated with Claude Code