Skip to content

feat(cli): turn auto-approval on or off with project update - #414

Merged
gregberge merged 1 commit into
mainfrom
claude/arg-518-cli-auto-approval
Nov 6, 2026
Merged

gregberge merged 1 commit into
mainfrom
claude/arg-518-cli-auto-approval

Conversation

@gregberge

Copy link
Copy Markdown
Member

Adds --auto-approval true|false to argos project update, for the per-project toggle from argos-ci/argos#2754 (ARG-518).

argos project update --project acme/web --auto-approval false   # every build needs a review
argos project update --project acme/web --auto-approval true

Release order

Release after argos-ci/argos#2754 is deployed. Before that, the API's zod body schema strips the unknown field, so the flag does nothing: the project comes back unchanged.

Verification

  • turbo run build check-types lint check-format test passes for @argos-ci/cli and @argos-ci/api-client. format.test.ts now covers both outputs.
  • I ran the built CLI against a local backend on the feat(project): allow turning off auto-approved branches argos#2754 branch, with a throwaway token:
    • --auto-approval false printed Auto-approved branches: none, and the database had autoApprovalEnabled = false;
    • setting --auto-approved-branch-glob 'release/**' while off kept auto-approval off;
    • --auto-approval true printed Auto-approved branches: release/**;
    • --auto-approval maybe was rejected with the allowed choices.

🤖 Generated with Claude Code

`argos project update --auto-approval false` turns off auto-approved
branches, so every build needs a review, base branch included; `true`
turns it back on. The glob is kept either way. `project get` and
`project update` print "Auto-approved branches: none" while it is off.

The API client's Project schema and updateProject body gain
`autoApprovalEnabled`, added by hand like #386 since the field is not in
the published OpenAPI spec until argos-ci/argos#2754 deploys. Until
then the API ignores the flag.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@linear-code

linear-code Bot commented Nov 5, 2026

Copy link
Copy Markdown

ARG-518

@gregberge
gregberge merged commit 58c7cf7 into main Nov 6, 2026
101 of 104 checks passed
@gregberge
gregberge deleted the claude/arg-518-cli-auto-approval branch November 6, 2026 11:05
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.

1 participant