Skip to content

fix: resolve 6 CLI gaps — update command, key regen header, --callback alias#17

Merged
govindkavaturi-art merged 1 commit intomainfrom
fix/cli-flags-v2
Apr 4, 2026
Merged

fix: resolve 6 CLI gaps — update command, key regen header, --callback alias#17
govindkavaturi-art merged 1 commit intomainfrom
fix/cli-flags-v2

Conversation

@govindkavaturi-art
Copy link
Copy Markdown
Member

Summary

Fixes all 6 CLI gaps from Argus integration tests (#89):

Issue Root Cause Fix
--worker not working Was working — test environment issue No code change
--payload not implemented Was working No code change
--on-failure not implemented Was working No code change
key regenerate --yes HTTP 400 Missing X-Confirm-Destructive: true header Added header in auth.py
update command missing Never existed Added full update command
--callback not recognized Only --url existed Added --callback as alias

Tests

19/19 passing (3 new tests added).

🤖 Generated with Claude Code

1. --worker: already worked, no change needed
2. --payload: already worked, no change needed
3. --on-failure: already worked, no change needed
4. key regenerate --yes: was returning HTTP 400 because API requires
   X-Confirm-Destructive: true header — now sends it
5. update command: did not exist — added with --name, --cron, --url,
   --payload, --description, --on-failure flags
6. --callback flag: added as alias for --url on both create and update

19 tests passing (3 new: callback alias, update help, update validation).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@govindkavaturi-art govindkavaturi-art enabled auto-merge (squash) April 4, 2026 21:53
@govindkavaturi-art govindkavaturi-art merged commit 9902b7f into main Apr 4, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants