Skip to content

fix: implement missing CLI flags — --worker, --on-failure (#89)#16

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

fix: implement missing CLI flags — --worker, --on-failure (#89)#16
govindkavaturi-art merged 1 commit intomainfrom
fix/cli-missing-flags

Conversation

@govindkavaturi-art
Copy link
Copy Markdown
Member

Summary

Implements missing CLI flags identified by Argus in issue #89:

Flag Status Detail
--worker Added Sets transport="worker", makes --url optional
--payload Already existed No change
--on-failure Added Accepts JSON config e.g. '{"email": false}'
key regenerate --yes Already existed No change

Tests added

  • test_create_worker_flag_in_help--worker appears in help
  • test_create_requires_url_or_worker — error if neither --url nor --worker
  • test_create_on_failure_in_help--on-failure appears in help
  • test_key_regenerate_help--yes/-y appears in help

All 16 tests pass locally (0.18s).

🤖 Generated with Claude Code

- --worker flag on create: sets transport="worker", makes --url optional
- --on-failure flag on create: accepts JSON config for failure escalation
- --url is now optional when --worker is set (error if neither provided)
- Added 4 new tests: worker flag in help, url-or-worker validation,
  on-failure in help, key regenerate --yes in help
- Total: 16 tests passing

Fixes Argus issue #89.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@govindkavaturi-art govindkavaturi-art enabled auto-merge (squash) April 4, 2026 21:41
@govindkavaturi-art govindkavaturi-art merged commit 958c9c3 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