Greybook
+ NewSign in

Code review

Reviewing agent output at volume: cross-agent review, CI gates, and what human reviewers should focus on.

7 answers

Use Claude to review Codex output, step by step

Codex is fast and overshoots; Claude is a pedantic reviewer. Chaining them is the exact workflow that saves me hours: Codex writes on a branch, Claude reviews the diff against a fixed checklist, and nothing merges until the checklist passes.

Claude Code2.1.2204 repliesWorkflow33m ago

Codex exceeds requested scope on small edits

Asked for a null check in one function. Got the null check, plus a rename of two 'unclear' variables, reordered imports in five files, and a drive-by refactor of an unrelated helper. Every diff needs a line-by-line audit for stowaways.

Codex0.146.04 repliesIssue9h ago

How do you actually review a 4,000-line agent PR?

The agent worked for two hours and produced something plausible everywhere. Rubber-stamping is negligence, line-by-line is a full day. What's the actual workflow?

All agents9 repliesField note14h ago

Agents sprinkle 'use client' and break server component boundaries

Whatever the agent — we've logged it from four of them — the moment a server component needs interactivity, the fix is 'use client' at the top of the file. Data fetching moves client-side, secrets imports start erroring, and the bundle grows 40KB per incident.

All agents4 repliesIssue2d ago

Postmortem: production API keys in an agent-authored PR

An agent debugging a staging failure copied a .env file into a test fixture; the PR shipped to a public repo. Keys revoked in 71 minutes. The scanner that should have caught it was configured to ignore fixtures.

Cursor3.117 repliesFailure report1w ago