fix(cli): add --format option to actor remove command #6742

Merged
HAL9000 merged 3 commits from fix/issue-6491-actor-remove-format-option into master 2026-05-06 01:07:16 +00:00

3 Commits

Author SHA1 Message Date
HAL9000 03ed12cf0d fix(cli): address reviewer feedback on actor remove --format option
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 53s
CI / quality (pull_request) Successful in 47s
CI / typecheck (pull_request) Successful in 1m3s
CI / security (pull_request) Successful in 1m1s
CI / build (pull_request) Successful in 35s
CI / push-validation (pull_request) Successful in 28s
CI / helm (pull_request) Successful in 37s
CI / e2e_tests (pull_request) Successful in 3m32s
CI / integration_tests (pull_request) Successful in 5m6s
CI / unit_tests (pull_request) Successful in 5m37s
CI / docker (pull_request) Successful in 1m35s
CI / coverage (pull_request) Successful in 9m46s
CI / status-check (pull_request) Successful in 4s
CI / benchmark-regression (pull_request) Successful in 58m7s
- Validate --format argument before any side effects; raise typer.BadParameter
  with a clear message for unsupported format values (fail-fast principle)
- Pass normalised fmt_value (lowercased) to format_output instead of raw fmt
  to ensure consistent behaviour regardless of input casing
- Rewrite robot/helper_actor_remove_cli.py to exercise the real CLI end-to-end
  via subprocess (no mocking); seeds a test actor via agents actor add, then
  removes it with --format json and validates the JSON envelope

ISSUES CLOSED: #6491
2026-05-05 19:09:40 +00:00
HAL9000 bed3993cca test(cli): cover actor remove format regression (#6491)
CI / quality (pull_request) Successful in 42s
CI / push-validation (pull_request) Successful in 29s
CI / lint (pull_request) Successful in 44s
CI / build (pull_request) Successful in 30s
CI / helm (pull_request) Successful in 44s
CI / typecheck (pull_request) Successful in 1m5s
CI / security (pull_request) Successful in 1m7s
CI / e2e_tests (pull_request) Successful in 3m50s
CI / integration_tests (pull_request) Successful in 4m11s
CI / unit_tests (pull_request) Successful in 5m34s
CI / docker (pull_request) Successful in 1m19s
CI / coverage (pull_request) Successful in 13m3s
CI / status-check (pull_request) Successful in 2s
CI / benchmark-publish (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
Add Robot regression coverage for `actor remove --format json`, document the flag in the CLI synopsis, and record the fix in the changelog.\n\nISSUES CLOSED: #6491
2026-04-11 01:01:58 +00:00
HAL9000 43847e7506 fix(cli): add --format option to actor remove command (#6491)
ISSUES CLOSED: #6491
2026-04-11 01:01:58 +00:00