Skip to content

Disable API-controlled auto approval - #69

Merged
Alajmah merged 2 commits into
mainfrom
api63-disable-auto-approve
Jun 7, 2026
Merged

Alajmah merged 2 commits into
mainfrom
api63-disable-auto-approve

Conversation

@Alajmah

@Alajmah Alajmah commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes API-controlled auto approval from /commands.

Changes

  • Removes auto_approve from CommandRequest.
  • Forces API command execution through LocalOrchestrator with auto_approve=False.
  • Adds a regression test proving API input cannot request auto approval.

Closes #63.

Validation

  • Existing API tests updated.
  • CI should run on this PR.

@gitwire-hq gitwire-hq Bot added the size/S label Jun 7, 2026
@Alajmah
Alajmah merged commit 0aafd3f into main Jun 7, 2026
2 checks passed
@Alajmah
Alajmah deleted the api63-disable-auto-approve branch June 7, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disable API-controlled auto approval

1 participant