Skip to Content
Core WorkflowsCode Review

Code Review

Code review is a verification workflow that is independent of plan completion. It is not an automatic byproduct of completion reports, co-actions, or post-mortems — you run it when it’s explicitly requested.

Code review detailCode review detail

The detail view shows the review target (target, branch, commit range), the linked plan and report, the reviewer context, the diff and test summaries, and the findings (severity, file, and resolution status).

When to use it

Recommended for changes with one or more of these signals:

  • Cross-workspace or shared-contract changes
  • Database schema, migration, authentication, permission, billing, quota, or deployment logic changes
  • Large diffs, broad refactors, or changes touching critical user workflows
  • Failed or skipped verification that still needs independent inspection

Don’t create a code review merely because a plan was finished or a completion report was written.

Review types

You can target a review in several ways.

TypeReview target
GITHUB_PRA GitHub pull request
GITLAB_MRA GitLab merge request
BRANCH_DIFFThe diff between two branches
COMMIT_RANGEA specific commit range
LOCAL_DIFFLocal (uncommitted) changes
UPLOADED_DIFFAn uploaded diff file

Reviewer context

Prefer reviewing from a different session, agent, or context than the original implementation — reusing the same assumptions tends to miss design and regression risks. A review request separates the original context (plan, report, commit range, diff, test results) from the reviewer execution context (reviewer agent/model and instructions).

Severity

LevelMeaning
P0Must fix before merge/release. Data loss, security, irreversible operation, or broken core flow
P1Should fix before merge. Likely regression, incorrect behavior, permission gap, migration risk, or missing required validation
P2Fix soon. Maintainability, missing edge case, incomplete test coverage, or confusing API/UI
P3Optional. Naming, readability, non-blocking cleanup, or polish

Finding format

Each finding includes:

  • Severity (P0P3)
  • File path (with line range when available)
  • Title — a short summary
  • Problem — what is wrong
  • Impact — why it matters
  • Suggestion — a concrete fix direction

Include only items verifiable from the diff, repository context, or stated requirements — avoid broad commentary.

Review status

StatusMeaning
PENDINGThe reviewer hasn’t produced a result yet
COMPLETEDReview done (including zero findings — “no issues” is still complete)

Flow

After a completion report, review whether a code review is warranted and, if so, present it as a separate explicit action. When converting findings into a follow-up plan, include only the findings you select, preserving each one’s severity, file, problem, impact, and suggestion.

Last updated on