Agent skill · Code Review & Quality

named-persona-adversarial-review

Code review through the lens of real engineers' documented philosophies (Torvalds, Thompson, Carmack, Kent Beck, Jobs, Cagan). Complements abstract-role adversarial review with named, sourced perspectives. Use when automated review findings feel generic, when a PR has architectural or UX impact, or when the author wants pre-submit hardening beyond standard checks.

Alireza Rezvani23,369★ · +428/wk · 1 repos on radarProfile →
claude-codecodexcursorMIT
Install
npx skills add alirezarezvani/claude-skills --skill named-persona-adversarial-review --agent claude-code

Same command for any agent — swap --agent for codex, cursor, copilot.

Facts
Files in the skill folder: 2
SKILL.md size: 11 KB
Bundled scripts: none
Path: engineering-team/skills/named-persona-adversarial-review/SKILL.md
Open the folder on GitHub →
Where it comes from
Stars: 23,791 · +422 this week
Language: Python
Read our review of the source →

Weekly change comes from our own snapshots, not the repository page — it measures attention, not adoption.

From the SKILL.md

# Named-Persona Adversarial Review > **TL;DR:** Abstract roles find abstract problems. Named engineers with *documented, sourced* philosophies find problems you would actually fix — as long as you cite the real principle and never invent the quote. **Triggers:** "review this PR with real engineers" | "named persona review" | "philosophy-grounded review" ## Example Output ``` CRITICAL [Torvalds]: Special-case error handling at auth.ts:47 duplicates the happy path. Torvalds' documented "good taste" principle: restructure so the special case disappears rather than adding a branch. (confidence: high — TED 2016) WARNING [Thompson]: parseConfig() does three unrelated things; the Unix "do one thing well" principle argues to split it. (confidence: high) NOTE [Jobs]: Error "EACCES:13" leaks an errno at the user surface; "start from the customer experience" argues for a human message. (confidence: high — WWDC 1997) Verdict: CONCERNS — fix CRITICAL before merge. ``` ## Problem Abstract adversarial review ("act as a saboteur") produces generic findings — the model imagines what a reviewer *might* say. This skill grounds each lens in a **real, sourced engineering philosophy** documented in [`re

What's inside
Steps it walks through
  1. Example Output
  2. Problem
  3. Attribution discipline (read this first — it is the load-bearing rule)
  4. Rules
  5. Persona Pools
  6. Severity Levels
  7. The Process
  8. Step 0: Read twice
  9. Step 1: Ground the principles first
  10. Step 2: Review (3 independent — 2 engineers + 1 product)
  11. Step 3: Synthesize & post
  12. Integrity Check (Feynman)
  13. Exit Condition
  14. When to Use
Ships with 1 file
  • references/persona_principles.md
More from claude-skills
All skills →
About this skill
What does the named-persona-adversarial-review skill do?

Code review through the lens of real engineers' documented philosophies (Torvalds, Thompson, Carmack, Kent Beck, Jobs, Cagan). Complements abstract-role adversarial review with named, sourced perspectives. Use when automated review findings feel generic, when a PR has architectural or UX impact, or when the author wants pre-submit hardening beyond standard checks.

How do I install it?

Run `npx skills add alirezarezvani/claude-skills --skill named-persona-adversarial-review --agent claude-code` — it drops the skill into your project so the agent can pick it up. Swap the --agent value for codex, cursor or copilot if you use one of those.

Where does this skill come from?

From alirezarezvani/claude-skills, a repository with 23,791 stars. We read it straight from the repository tree rather than a submitted listing, so what you see here is what is actually published.

Is a popular skill a good skill?

Not necessarily. Stars measure attention, not adoption — a repository can trend for a week and be abandoned. That is why we show the weekly change from our own snapshots next to the total, instead of a single flattering number.

Keep going