PromptSharp › Prompt Library › Dev & Engineering › Pre-review sweep: your own PR through a security-and-edge-case lens

Pre-review sweep: your own PR through a security-and-edge-case lens

The PR is 'done'. Run the pre-review sweep so human reviewers spend their attention on design — not on nits and the missed null check.

The prompt — copy and run it

You are a staff engineer reviewing a pull request. I will paste the diff and its context. Produce:

A) REVIEW TABLE — columns: file/line reference, severity (blocker / major / nit), category (correctness, security, performance, readability, tests), the issue in one sentence, and a suggested fix as a concrete code change.

B) EDGE-CASE LIST — inputs and states the diff does not handle: empty, null, concurrent access, oversized input, malformed input, permission-denied.

C) VERDICT — approve or request-changes, plus the 2 highest-risk lines in the diff and why.

Inputs: [PASTE DIFF] · [WHAT THE CHANGE DOES + WHY] · [LANGUAGE/FRAMEWORK + TEAM CONVENTIONS]

Rules: Do not invent code that is not in the diff — reference only pasted lines. Any claim about behavior you cannot see (callers, config, upstream state) must be marked "verify in repo". Never echo secrets or keys, and do not include proprietary code beyond what I pasted.

Why this prompt works

LLM review works best as a filter before human review — the machine catches the mechanical 80% so humans argue about design. Severity plus category forces triage instead of a wall of nits, and the 'verify in repo' rule prevents confident-but-wrong claims about code the model cannot see.

Frequently asked

Related Dev & Engineering prompts