fix(cli): exit codes, option handling and Ctrl+C for the accounts and dry-run commands; the log follows GCLO_DATA_DIR #159

New Issue

2026-11-10T22:16:42Z

james.maes commented

2026-11-10 22:16:42 +00:00

Found in the 1.1.1 review. A batch of small command-line defects, each with a regression test.

gclo sync --dry-run that is canceled while the repositories are being listed exits 0 with an empty result, as if the organization had none.
gclo accounts add|edit|remove: a failure of the accounts file or of Windows Credential Manager ends with exit code 70 ("unexpected error, please report it"), where the help promises 2.
gclo accounts -h is reported as an unknown subcommand.
gclo accounts edit accepts a blank --rename, --org, or --target.
A relative --target is stored as typed, so the account means a different folder depending on where it is later used from (the desktop app, another working directory).
--description= is rejected although --description "" works.
The activity log of the CLI ignores GCLO_DATA_DIR: it always goes to the default data folder, while accounts and settings follow the variable.
Ctrl+C: commands that do not observe cancellation (accounts) and the interactive token prompt swallow the first Ctrl+C ("Canceling...") and then carry on.

Found in the 1.1.1 review. A batch of small command-line defects, each with a regression test. - `gclo sync --dry-run` that is canceled while the repositories are being listed exits 0 with an empty result, as if the organization had none. - `gclo accounts add|edit|remove`: a failure of the accounts file or of Windows Credential Manager ends with exit code 70 ("unexpected error, please report it"), where the help promises 2. - `gclo accounts -h` is reported as an unknown subcommand. - `gclo accounts edit` accepts a blank `--rename`, `--org`, or `--target`. - A relative `--target` is stored as typed, so the account means a different folder depending on where it is later used from (the desktop app, another working directory). - `--description=` is rejected although `--description ""` works. - The activity log of the CLI ignores `GCLO_DATA_DIR`: it always goes to the default data folder, while accounts and settings follow the variable. - Ctrl+C: commands that do not observe cancellation (`accounts`) and the interactive token prompt swallow the first Ctrl+C ("Canceling...") and then carry on.

james.maes referenced this issue from a commit

2026-11-10 22:34:47 +00:00

fix(cli): exit codes, option handling and Ctrl+C for accounts and dry runs; the log follows GCLO_DATA_DIR

james.maes referenced a pull request that will close this issue

2026-11-10 22:34:48 +00:00

fix(cli): exit codes, option handling and Ctrl+C for accounts and dry runs; the log follows GCLO_DATA_DIR #160

james.maes referenced this issue from a commit

2026-11-10 22:53:03 +00:00

fix(cli): address the independent review of the command-line fixes

james.maes referenced this issue from a commit

2026-11-10 22:53:06 +00:00

fix(cli): exit codes, option handling and Ctrl+C for accounts and dry runs; the log follows GCLO_DATA_DIR (#160)