deep-code-review
Skill2 starsChanged 27 days agoMITView on GitHub
Use when reviewing, auditing, hardening, red-teaming or quality-gating a repo, PR, branch or diff, even without the word review. Evidence-grounded review: correctness, security, data, performance, reliability, tests, CI, infra, docs, a11y, privacy. Severity-ranked file:line report.
13 other projects use a version of this.
Adopted by
Most recently taken up by
- bhoon716/skill-forgetheir own version, first seen Nov 2, 2026
- phntm7/skillstheir own version, first seen Nov 1, 2026
- neysofu/skillstheir own version, first seen Oct 30, 2026
- Luo-root/deep-code-reviewtheir own version, first seen Oct 25, 2026
- remigiusz-antczak/deep-code-reviewword for word, first seen Oct 16, 2026
- pmarreck/llm_skillstheir own version, first seen Oct 15, 2026
Other versions13 projects use a version of this.
Word for word · 1
- remigiusz-antczak/deep-code-review★ 2100% same
Edited · 0
None found.
Wrote their own version · 12
- expo/expo★ 53ksame name
- felipeArnold/filament-ai-skills★ 9same name
- ducafecat/skills★ 6same name
- bhoon716/skill-forge★ 4same name
- M4n5ter/skills★ 4same name
- AthArvA-188/top-10-claude-skills★ 3same name
- coryparrry/codex-skills★ 3same name
- Luo-root/deep-code-review★ 2same name
- pmarreck/llm_skills★ 2same name
- MichaelLeeHobbs/skills★ 1same name
2 more not listed
How matches are found. Word for word: the exact same text. Edited: shares passages with this file (“99% same” means 99% of its five-word phrases also appear here; 10% or more counts). Their own version: similar meaning, or, weakest, the same name. We can’t tell which project had it first. Projects are counted among the 200 most-starred candidates, and mentions are links to the repository we found, up to 50 per site, so both counts are a floor.
In this repository12 other files it gives its agents.
- AGENTS.mdAGENTS.md
- CLAUDE.mdCLAUDE.md
- agentic-ceoSkill
- agentic-deliverySkill
- business-opsSkill
- communication-structureSkill
- contributionSkill
- growth-analyticsSkill
- idea-criticSkill
- positioningSkill
- product-discoverySkill
- product-output-safetySkill
The file
Files
Deep Code Review
You are a senior reviewer hunting defects a change introduces. For a small DIFF (at most 300 changed lines, no blast-radius flag such as a removed guard, widened route or CORS origin, a check moved client-side, or a producer wired across a trust boundary) this file is the whole method: read nothing else, make one patch read, answer next turn. Anything else (FILE, FULL, large or flagged, or touching auth, crypto, payments, data deletion or another high-stakes gate): read references/full-map.md first; it holds scope modes, hard rules, load map, domain map, severity, routing; references/full-map-principles.md: principles 4-6, 10, 11, stage.
- Read every hunk. For each changed line ask what input, state or caller breaks: inverted or off-by-one conditions, nil/empty/zero inputs, swallowed or fail-open errors, state not restored on exit, removed or renamed API, weakened validation, injection, resource leaks, data loss, output corruption, ordering bugs, unchecked assumptions about other files.
- A changed line that looks deliberate can still be the bug: compare what the new code does with what its name, comment, neighbouring branches and removed lines imply, and test each branch with empty, missing, zero, repeated and nested inputs.
- Keep a finding when tracing a plausible caller input through the cited lines gives wrong behavior. Verify by re-reading the cited lines and tracing the failing input; report only at confidence 80 or more. Drop pre-existing code and style. Each finding names file, new-file line (hunk
+startplus offset, never the patch text's own line number), mechanism and one failing input. Severity: High = wrong result, crash, data loss or security hole on a normal path; Medium = plausible edge case; Low = unlikely. - One finding per distinct mechanism, most severe first, at most 8.
- Guarantees: for each guarantee-bearing comment, docstring or UI string in the diff (never, always, exactly once), name the test that forces the adversarial case it denies; an unpinned guarantee is a finding, High if it guards privacy, consent, money or data.
Whole-app security audit: read only references/security-review-mode.md. Security review of a small DIFF: read only references/appsec-verify.md (section 0 first), not full-map. Paths are in this skill's base dir, not the repo.
BUDGET (small DIFF): read references/small-diff-read-budget.md first.
OUTPUT. Severity-ranked findings, each file:line, severity, mechanism and failing input in 1-2 sentences. Template: references/full-map-report.md.
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

