Review

Reading a change properly

The review is the part that was never delegable. It is also the part that gets skipped first.

Removed code hides the surprises

Read every removed block and say out loud what used to happen there.

The edges of the change

  1. Dependency manifests and lock files.
  2. Build and CI configuration.
  3. Anything reading an environment variable or a secret.
  4. Generated files committed alongside the source.
  5. Tests that were changed rather than added.

Proof, in that order

Reading first, running second: a passing test on unread code tells you the test passed, not that the change is right.

Other pages

01

The short list of habits

A short list of working habits, each with the failure it prevents.

Read →
02

Two agents, compared honestly

Claude, from Anthropic, and Codex, from OpenAI, are two takes on the same idea: an assistant that lives in your project rather than in a browser tab.

Read →