Skip to main content

Get your changes reviewed

Three review scopes share one final Verify stage. Select the scope from the change's risk before review; do not run each rung in sequence.

Prerequisites: a session in the repo with the changes (uncommitted, staged, a branch, or a PR number).

Rung 1 โ€” /k-light-review: your own low-risk changesโ€‹

/k-light-review

The agent judges the diff against applicable correctness and hygiene criteria, consuming existing check evidence. Missing planned checks run once in final Verify. On your own uncommitted changes, findings inside the diff's own behavior are fixed in the same pass โ€” write scope, not review, is the authority; scope-expanding findings come back as proposals. Findings do not authorize self-audits or convergence. Output names findings, applied fixes, evidence, and remaining gaps.

It will refuse and point up a rung when the target is a PR, someone else's code, or risky/stateful territory โ€” that's the built-in escalation, not an error.

Rung 2 โ€” full k-review: PRs, others' code, risky changesโ€‹

review PR #4321

or for local work that needs base-branch context: review my branch against main.

Standard review reads the complete primary PR discussion and references needed for named material questions, then traces base behavior and affected consumers. Stateful risks need transition evidence from focused tests or a disposable harness. Write scope, not authorship, grants edit authority: your own branch is fixed in place; someone else's PR gets proposals until you say fix it. Known fixes belong to Produce before final Verify; publication retains exact-payload approval.

Rung 3 โ€” /k-deep-review: independent lanes + adversarial verificationโ€‹

/k-deep-review PR #4321

Deep review retains strong artifact review and adversarial challenge as distinct questions against the same frozen candidate and shared evidence. It prefers a different model family at equal capability. Additional specialist lenses need independent risks; blind fresh-eyes applies only to comprehension risk and receives no narrative, history, or prior findings. Applicable live UI evidence belongs to this same final stage.

Expect anchored findings, unresolved evidence gaps, relevant UI artifacts, and any requested publication draft. There is no findings-auditor, verifier-of-verifier, post-review, or automatic repair chain.

Use it when the change needs independent review plus adversarial challenge (SOP ยง3.7). Workers are leaves; the root owns scope, checks, and the terminal outcome.

Reading any review's outputโ€‹

  • Findings are ordered by severity; each carries evidence (file:line, command output) โ€” no evidence, no finding.
  • Compatibility impact: line tells you if behavior was removed/kept โ€” it must say none unless you asked otherwise.
  • Review alone does not authorize repairs; write scope does โ€” your own changes are fixed in place, others' come back as proposals. In any task with existing authority, the root follows SOP ยง3.5 for scoped recovery and revalidation; a failed check does not require the same permission again.

Pivots from hereโ€‹

  • Review found an architectural smell โ†’ hand it to k-codebase-design ("design a better seam for this").
  • Review found a bug worth its own work โ†’ draft an issue from finding 2.
  • A finding needs a live browser check โ†’ the agent does this itself in rungs 2โ€“3; in rung 1 just ask verify this in the browser.