Slipstream/auto_review
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases 27 Packages Activity Actions

feat(cli): unify operator binary #125

Merged
jwilger merged 1 commit from issue-116-unified-auto-review-cli into main 2026-05-06 13:55:31 -07:00
Conversation 1 Commits 1 Files changed 24 +652 -278
jwilger commented 2026-05-06 13:18:31 -07:00
Owner
Copy link

Summary

  • Publishes auto-review as the single operator binary and removes the public ar-gateway executable surface.
  • Rehomes gateway startup behind the shared ar_gateway::run_from_env() seam used by auto-review gateway.
  • Updates Nix packages/container entrypoint, systemd files, and public docs to use grouped auto-review commands.

Verification

  • nix develop -c cargo test -p ar-cli
  • nix develop -c cargo test -p ar-gateway --lib --tests
  • nix develop -c cargo test -p ar-gateway
  • nix develop -c cargo fmt --all -- --check
  • nix develop -c cargo clippy -p ar-cli -p ar-gateway --all-targets -- -D warnings
  • nix build .#ar-cli
  • nix build .#ar-gateway-image

Closes #116

## Summary - Publishes `auto-review` as the single operator binary and removes the public `ar-gateway` executable surface. - Rehomes gateway startup behind the shared `ar_gateway::run_from_env()` seam used by `auto-review gateway`. - Updates Nix packages/container entrypoint, systemd files, and public docs to use grouped `auto-review` commands. ## Verification - `nix develop -c cargo test -p ar-cli` - `nix develop -c cargo test -p ar-gateway --lib --tests` - `nix develop -c cargo test -p ar-gateway` - `nix develop -c cargo fmt --all -- --check` - `nix develop -c cargo clippy -p ar-cli -p ar-gateway --all-targets -- -D warnings` - `nix build .#ar-cli` - `nix build .#ar-gateway-image` Closes #116
feat(cli): unify operator binary
All checks were successful
CI / Nix flake check (pull_request) Successful in 4m40s
Details
CI / auto_review semantic review (pull_request) Successful in 1s
Details
auto_review auto_review: no findings
Publish release / release-publish (pull_request) Has been skipped
Details
4f0ee5280f
auto-review approved these changes 2026-05-06 13:23:35 -07:00
auto-review left a comment
Copy link

This PR unifies the operator binary by publishing auto-review as the single executable, removing ar-gateway, and updating related documentation and scripts. The changes appear safe to merge, with all references to the old command names updated.

Walkthrough

  • CONTRIBUTING.md: Updated command references from auto_review to auto-review.
  • QUICKSTART.md: Updated build and command instructions to reflect the unified auto-review binary.
  • crates/ar-cli: Updated the binary name and command structure to auto-review, grouping subcommands by domain.
  • crates/ar-gateway: Moved main functionality to startup.rs and exposed run_from_env for gateway startup.
  • flake.nix: Updated default package to ar-cli and command to start the gateway to auto-review gateway.
  • Documentation: Updated all references to the old command names to reflect the new auto-review command structure.
This PR unifies the operator binary by publishing `auto-review` as the single executable, removing `ar-gateway`, and updating related documentation and scripts. The changes appear safe to merge, with all references to the old command names updated. ## Walkthrough - **CONTRIBUTING.md**: Updated command references from `auto_review` to `auto-review`. - **QUICKSTART.md**: Updated build and command instructions to reflect the unified `auto-review` binary. - **crates/ar-cli**: Updated the binary name and command structure to `auto-review`, grouping subcommands by domain. - **crates/ar-gateway**: Moved main functionality to `startup.rs` and exposed `run_from_env` for gateway startup. - **flake.nix**: Updated default package to `ar-cli` and command to start the gateway to `auto-review gateway`. - **Documentation**: Updated all references to the old command names to reflect the new `auto-review` command structure.
Sign in to join this conversation.
Reviewers
No reviewers
auto-review
Labels
Clear labels   
bug

   
discussion

   
enhancement
  
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
discussion
enhancement
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Slipstream/auto_review!125
No description provided.