Implement unified `auto-review` CLI with grouped commands #116

New issue

Closed

opened 2026-05-06 11:57:45 -07:00 by jwilger · 0 comments

jwilger commented

2026-05-06 11:57:45 -07:00

Owner

Follow-up implementation slice from #115.

Decision context

The accepted ADRs will make a breaking switch to a single public auto-review executable with grouped domain commands. The gateway service becomes auto-review gateway.

Scope

Add the public auto-review binary target.
Compose the gateway service and operator CLI into that binary.
Replace the current flat operator command surface with grouped domain namespaces:
- gateway
- auth
- webhook
- config
- review
- bench
- ops
- history
- learnings
Extract/reuse the gateway startup path so auto-review gateway does not duplicate env parsing, validation, or defaults.
Treat this as a breaking switch: no ar-gateway, auto_review, or old top-level subcommand compatibility shims are required.

Acceptance criteria

auto-review gateway starts the same gateway behavior that ar-gateway starts today, before isolation launcher integration.
Existing operator functionality is reachable only through the accepted grouped command taxonomy.
Command help documents the grouped layout.
Public docs and crate README references to command names are updated in the implementation PR.

Required tests

Clap parsing tests for the grouped command surface.
README/help contract tests for every public grouped command.
Gateway startup/config tests proving shared env parsing and validation.

Follow-up implementation slice from #115. ## Decision context The accepted ADRs will make a breaking switch to a single public `auto-review` executable with grouped domain commands. The gateway service becomes `auto-review gateway`. ## Scope - Add the public `auto-review` binary target. - Compose the gateway service and operator CLI into that binary. - Replace the current flat operator command surface with grouped domain namespaces: - `gateway` - `auth` - `webhook` - `config` - `review` - `bench` - `ops` - `history` - `learnings` - Extract/reuse the gateway startup path so `auto-review gateway` does not duplicate env parsing, validation, or defaults. - Treat this as a breaking switch: no `ar-gateway`, `auto_review`, or old top-level subcommand compatibility shims are required. ## Acceptance criteria - `auto-review gateway` starts the same gateway behavior that `ar-gateway` starts today, before isolation launcher integration. - Existing operator functionality is reachable only through the accepted grouped command taxonomy. - Command help documents the grouped layout. - Public docs and crate README references to command names are updated in the implementation PR. ## Required tests - Clap parsing tests for the grouped command surface. - README/help contract tests for every public grouped command. - Gateway startup/config tests proving shared env parsing and validation.

jwilger added a new dependency

2026-05-06 12:13:08 -07:00

#118 Build embedded minimal OCI rootfs for auto-review gateway

jwilger added this to the 1.0 milestone

2026-05-06 12:17:49 -07:00

jwilger self-assigned this

2026-05-06 12:34:59 -07:00

jwilger added a new dependency

2026-05-06 12:35:19 -07:00

#124 chore: release v0.2.1

jwilger removed a dependency

2026-05-06 12:35:26 -07:00