Run Docker image through unified auto-review binary #131

Merged

jwilger merged 1 commit from issue-119-unified-docker-binary into main

2026-05-07 06:36:31 -07:00

jwilger commented

2026-05-07 06:32:07 -07:00

Owner

Summary

update the gateway image command to launch /bin/auto-review gateway
add an image contract check for unified command, no stale ar-gateway entrypoint, and /bin/auto-review plus /bin/git contents
update the existing CLI flake contract to expect the public image command

Verification

nix develop -c cargo fmt --all -- --check
nix build .#checks.x86_64-linux.ar-gateway-docker-image-unified-binary-contract
nix develop -c cargo test -p ar-cli cli::tests::flake_publishes_auto_review_as_the_only_operator_binary -- --nocapture
nix flake check

Closes #119

## Summary - update the gateway image command to launch `/bin/auto-review gateway` - add an image contract check for unified command, no stale `ar-gateway` entrypoint, and `/bin/auto-review` plus `/bin/git` contents - update the existing CLI flake contract to expect the public image command ## Verification - `nix develop -c cargo fmt --all -- --check` - `nix build .#checks.x86_64-linux.ar-gateway-docker-image-unified-binary-contract` - `nix develop -c cargo test -p ar-cli cli::tests::flake_publishes_auto_review_as_the_only_operator_binary -- --nocapture` - `nix flake check` Closes #119

jwilger added 1 commit

2026-05-07 06:32:07 -07:00

feat(gateway): run image through unified binary

CI / Nix flake check (pull_request) Successful in 3m19s

Details

CI / auto_review semantic review (pull_request) Successful in 2s

Details

auto_review auto_review: no findings

Publish release / release-publish (pull_request) Has been skipped

Details

9a069bfc92

auto-review approved these changes

2026-05-07 06:35:44 -07:00

auto-review left a comment

Walkthrough

crates/ar-cli/src/cli.rs:
- Updated the command to launch the gateway using /bin/auto-review instead of ${ar-cli}/bin/auto-review.
flake.nix:
- Modified the Docker image configuration to use /bin/auto-review for the command.
- Added a new contract check to ensure the Docker image uses the unified CLI and does not contain stale entrypoints.
- The contract check verifies the presence of /bin/auto-review and /bin/git in the Docker image layers.

The PR updates the Docker image command to use a unified auto-review binary and adds a contract check for the image. The changes appear safe to merge, but verify the new binary path is correct in all deployment environments. ## Walkthrough - **crates/ar-cli/src/cli.rs**: - Updated the command to launch the gateway using `/bin/auto-review` instead of `${ar-cli}/bin/auto-review`. - **flake.nix**: - Modified the Docker image configuration to use `/bin/auto-review` for the command. - Added a new contract check to ensure the Docker image uses the unified CLI and does not contain stale entrypoints. - The contract check verifies the presence of `/bin/auto-review` and `/bin/git` in the Docker image layers.

jwilger merged commit 67eef8596b into main

2026-05-07 06:36:31 -07:00

jwilger referenced this pull request from a commit

2026-05-07 06:36:32 -07:00

Run Docker image through unified auto-review binary (#131)

jwilger referenced this pull request

2026-05-07 06:40:35 -07:00

Run the Docker image through the unified auto-review binary #119