Hands-on · step by step
Tutorial: a diff-scoped AI reviewer that doesn't spam your PR
by @samuel — AI code review · developer tooling
Four steps to AI code review with GitHub Actions and tree-sitter — checkpoints, tuning knobs and the failure to guard first.
Before you start — about 45 minutes · GitHub Actions · tree-sitter · a hosted model
Ship this in an afternoon. GitHub Actions does the work — the discipline is in the limits you set around it.
Step 1 — Decide what "working" means
Before touching code, write down three numbers: worst acceptable latency, cost per request, and what should happen when the upstream is unavailable. Everything after this is implementation.
Step 2 — Wire GitHub Actions
The whole flow is one path. No special branches for "large" inputs — express those as different limits, not different code.
# only send the changed hunks plus call sites the model needs
- name: Review
run: |
git diff --unified=0 $BASE_SHA $HEAD_SHA -- '*.ts' > diff.patch
npx review-bot --diff diff.patch --max-comments 5 --no-style-rules
Why this works
The limit is enforced before the expensive call, not after it. Reserving budget up front means a burst degrades the caller that caused it instead of everyone downstream.
Step 3 — Guard the failure path
whole-file prompts invite the model to rewrite your architecture
Write that guard before you write the feature. It costs an hour and saves a postmortem.
Step 4 — Verify it
gh pr view --json comments | jq '.comments | length' # expect ≤ 5
- Lower the limit and confirm the command fails loudly
- Kill the upstream and confirm you get a typed error, not a timeout
- Check the metric you emit per outcome: ok, throttled, upstream_error
Tuning knobs
| Knob | Default | What we run | Why |
|---|---|---|---|
| concurrency | 8 | 24 | Most of the wait is network, not compute |
| retry budget | infinite | 2, with jitter | Unbounded retries turn an outage into an avalanche |
| timeout | 30s | 8s, inherited from the caller | Nobody waits 30s for a chat reply |
Copy it, change the limits, put the verification in CI, move on.
Comments (2)
Log in to join the discussion.