/roborev:pull-request-reviewer
Pre-review a pull request to anticipate problems and shorten the review cycle.
Usage
/roborev:pull-request-reviewer [pr_number_or_branch]
Description
Analyzes a pull request (or the current branch's diff against main) before formal review. Runs automated checks, inspects the diff for common problem areas derived from historical review patterns, and generates a structured pre-review report with blocking issues, likely findings, and suggestions.
The goal is to catch problems that would fail review before submitting, reducing round-trips.
Instructions
When the user invokes /roborev:pull-request-reviewer [pr_number_or_branch]:
Determine the target: PR number via gh pr view, or current branch diff against main
Gather the diff: Full diff, changed file list, and commit log
Run automated checks: go build, go vet, go test, gofmt
Analyze for common problem areas:
- Security: Prompt injection in agentic prompts, auth/authz gaps on new endpoints, command injection, unbounded input
- Concurrency: Data races, file descriptor races, context/channel misuse
- Error handling: Unchecked return values, partial failure corruption, missing error context
- Test quality: Coverage gaps, flaky patterns, test isolation
- Scope: Unrelated changes mixed in, large files not split, commit hygiene
- Go idioms: Format/vet compliance, deprecated patterns, style consistency
- Dependencies: Unjustified external deps, version consistency, build tags
Check the PR description: Verify Summary, Test Plan, and issue references
Present the report: Blocking issues, likely findings, suggestions, check results
Offer to fix: If blocking issues found, offer to fix them and re-run tests
Example
User: /roborev:pull-request-reviewer
Agent:
- Detects current branch, computes diff against
main
- Runs
go build, go vet, go test -- all pass
- Finds 2 issues: missing error-path test (medium), unformatted file (low)
- Reports: "Pre-review found 2 items. Would you like me to fix them before you submit the PR?"
1---2name: hdb-pull-request-reviewer3description: Hdb Pull Request Reviewer4---5# /roborev:pull-request-reviewer67Pre-review a pull request to anticipate problems and shorten the review cycle.89## Usage1011```12/roborev:pull-request-reviewer [pr_number_or_branch]13```1415## Description1617Analyzes a pull request (or the current branch's diff against main) before formal review. Runs automated checks, inspects the diff for common problem areas derived from historical review patterns, and generates a structured pre-review report with blocking issues, likely findings, and suggestions.1819The goal is to catch problems that would fail review _before_ submitting, reducing round-trips.2021## Instructions2223When the user invokes `/roborev:pull-request-reviewer [pr_number_or_branch]`:24251. **Determine the target**: PR number via `gh pr view`, or current branch diff against `main`26272. **Gather the diff**: Full diff, changed file list, and commit log28293. **Run automated checks**: `go build`, `go vet`, `go test`, `gofmt`30314. **Analyze for common problem areas**:32 - **Security**: Prompt injection in agentic prompts, auth/authz gaps on new endpoints, command injection, unbounded input33 - **Concurrency**: Data races, file descriptor races, context/channel misuse34 - **Error handling**: Unchecked return values, partial failure corruption, missing error context35 - **Test quality**: Coverage gaps, flaky patterns, test isolation36 - **Scope**: Unrelated changes mixed in, large files not split, commit hygiene37 - **Go idioms**: Format/vet compliance, deprecated patterns, style consistency38 - **Dependencies**: Unjustified external deps, version consistency, build tags39405. **Check the PR description**: Verify Summary, Test Plan, and issue references41426. **Present the report**: Blocking issues, likely findings, suggestions, check results43447. **Offer to fix**: If blocking issues found, offer to fix them and re-run tests4546## Example4748User: `/roborev:pull-request-reviewer`4950Agent:511. Detects current branch, computes diff against `main`522. Runs `go build`, `go vet`, `go test` -- all pass533. Finds 2 issues: missing error-path test (medium), unformatted file (low)544. Reports: "Pre-review found 2 items. Would you like me to fix them before you submit the PR?"
hughdbrown