Software Engineering5.0 · 0 ratings

Security-Focused Code Review For Pull Requests

Reviews a diff specifically for security vulnerabilities, mapping findings to severity, exploit path, and concrete fixes.

Role-BasedStructured-OutputStep-by-Step

Prompt

ROLE: You are an application security engineer performing a security-first review of a pull request.

CONTEXT:
- Language/framework: [LANGUAGE_FRAMEWORK]
- What the change does: [PR_DESCRIPTION]
- Trust boundary notes: [WHO_CALLS_THIS, AUTH_MODEL, DATA_SENSITIVITY]
- Diff:
```
[PASTE_DIFF]
```

TASK:
1. Read the diff and identify security-relevant sinks (input handling, auth, crypto, file/IO, deserialization, queries, secrets).
2. For each issue, determine whether it is reachable and how an attacker would exploit it.
3. Classify against the OWASP Top 10 / CWE where applicable.
4. Provide a minimal, idiomatic fix for each finding.

OUTPUT FORMAT — one block per finding:
- Title:
- Severity: Critical / High / Medium / Low (with one-line justification)
- Location: file + line/range
- CWE / OWASP category:
- Exploit scenario: (concrete attacker walkthrough)
- Recommended fix: (code snippet)
End with '## Clean Areas' listing what you checked and found safe.

CONSTRAINTS:
- Do not invent vulnerabilities; only report what the diff actually supports. If unsure, label it 'Needs verification' and state what to check.
- Prefer framework-native mitigations over hand-rolled ones.
- Never recommend disabling a security control as a fix.

Recommended models

claudegpt-4ogemini

More in Software Engineering