review-all
Skill0 starsChanged 16 days agoMITView on GitHub
One command for a full code review. Runs five review skills one after another on the same change (your code-review panel, ponytail-review, pstack blast-radius, pstack thermo-nuclear-code-quality-review, pstack interrogate), then merges them into one ranked, deduplicated report. Report only, changes no code. Use for /review-all, "review everything", "run all the reviews", or "full review".
6 other projects use a version of this.
Adopted by
Most recently taken up by
- unlockers-ai/unlockers-skills-librarytheir own version, first seen Nov 8, 2026
- NovruzAliyevDev/skillstheir own version, first seen Nov 1, 2026
- paultyng/skill-issuetheir own version, first seen Oct 13, 2026
- jtenniswood/espcontroltheir own version, first seen Oct 8, 2026
- ncoevoet/claude-review-alltheir own version, first seen May 28, 2026
- clinton3141/dotcopilottheir own version, first seen Apr 13, 2026
Other versions6 projects use a version of this.
Word for word · 0
None found.
Edited · 0
None found.
Wrote their own version · 6
- jtenniswood/espcontrol★ 1.2ksame name
- ncoevoet/claude-review-all★ 45same name
- paultyng/skill-issue★ 8same name
- NovruzAliyevDev/skills★ 4same name
- unlockers-ai/unlockers-skills-library★ 4same name
- clinton3141/dotcopilot★ 0same name
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 repository23 other files it gives its agents.
- CLAUDE.mdCLAUDE.md
- agent-orchestrationSkill
- ask-for-approvalSkill
- claim-verification-loopSkill
- code-councilSkill
- code-reviewSkill
- context-as-imagesSkill
- grillingSkill
- grow-test-coverageSkill
- investigateSkill
- loop-until-verifiedSkill
- plan-and-buildSkill
- prompt-cachingSkill
- proofreadSkill
- review-before-mergeSkill
- review-panelSkill
- to-questionnaireSkill
- track-review-findingsSkill
- trim-contextSkill
- unslopSkill
- verify-findingsSkill
- whatnowSkill
- wizardSkill
The file
We split this skill into the rules it gives an agent
7 rules. 0 are common, 4 are distinctive, and 3 are very specific to this project.
0
Common
Most of the 5 edited versions we read keep these.
4
Distinctive
This author's own choices. Worth a look.
3
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- Distinctive: line 18
Report only: No skill in this run edits code, commits or posts to GitHub.
- Distinctive: line 23
A failed skill is not a clean result: If a skill errors or its agents die, re-run it once.
- Distinctive: line 24
If it fails again, list it under "Did not run" in the report.
- Very specific: line 42
Every skill writes its own report files to <scratchpad>/review-all/, never into the repo.
- Very specific: line 61
After each skill, save its findings to a scratch file (<scratchpad>/review-all/<n>-<skill>.md) before you start the next one.
- Distinctive: line 66
Read the five scratch files.
- Very specific: line 101
Write the merged report to <scratchpad>/review-all/report.md.
“Common” is measured: kept by at least 6 in 10 of the 5 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
Review all
Runs every review skill installed on this machine against one change and hands back one report. Each skill looks from a different angle, so their overlap is a signal: a problem two skills find on their own is more likely real.
Input: the skill args: a PR number, a PR URL, branch with an optional base commit, or
nothing for local uncommitted changes. A caller skill may add caller: plan-and-build; see step 4.
Rules for the whole run
- Report only. No skill in this run edits code, commits or posts to GitHub. Tell each skill so when you invoke it. Why: the reviewer never fixes what it reviewed (global CLAUDE.md, "Verification"). Fixes come after the user reads the report.
- One skill at a time.
code-reviewandinterrogateeach fan out several agents. Running them together breaks the 5-agents-at-once cap and hits rate limits. - A failed skill is not a clean result. If a skill errors or its agents die, re-run it once. If it fails again, list it under "Did not run" in the report. Never read silence as "no findings".
Step 1: pin the target
Work out the target once and pass the same one to every skill, so all five review the same code.
- Args hold a PR number or URL: PR mode. Record the PR number and its head commit
(
gh pr view <n> --json headRefOid). - Args start with
branch: branch mode, for committed work on a feature branch. If a base sha follows,BASEis that sha. Otherwise find the default branch (git symbolic-ref --short refs/remotes/origin/HEAD; exit code 128 means no remote HEAD, so fall back to whichever ofmain,masterexists) and setBASE=$(git merge-base <default> HEAD). No default branch found: ask the user for the base. Reviewgit diff $BASE..HEAD. No commits since the base: say "Nothing to review" and stop. - Otherwise: local mode. Record
git rev-parse HEADandgit diff HEAD --statplus untracked files fromgit status --porcelain. Nothing changed: say "Nothing to review" and stop.
Every skill writes its own report files to <scratchpad>/review-all/, never into the repo.
code-review normally writes .claude/reviews/; tell it to use the scratch folder instead.
With caller: plan-and-build, also tell every skill: "non-interactive: ask no questions,
run no hyperresearch, write only under the scratch folder". After each skill, git status
--porcelain must show nothing new; revert anything a skill wrote into the repo and log it.
Step 2: run the five skills, in this order
Invoke each with the Skill tool. Pass the target from step 1 and the line "report only: do not edit code, commit or post anywhere".
| # | Skill | What it adds |
| 1 | code-review | Typecheck, lint, tests and build first, then 8 lens reviewers with refuters. Runs first so a broken build is known before the rest. Answer "no" to its GitHub-posting question; posting happens in step 4. In branch mode, pass "branch review: use git diff <BASE>..HEAD in place of git diff HEAD; repoRoot is this checkout", because code-review has no branch mode of its own. |
| 2 | ponytail:ponytail-review | Logic, safety, load, missing tests, and every line that can be deleted. |
| 3 | pstack:blast-radius | What the change breaks outside the diff, proven by running real code. Tell it: single pass, no arena, and run only read-only commands or tests against a local or sandbox stack. |
| 4 | pstack:thermo-nuclear-code-quality-review | Strict maintainability: bad abstractions, giant files, condition sprawl. |
| 5 | pstack:interrogate | Adversarial reviewers from ~/.claude/pstack-models.md (interrogate reviewers line). Runs last, so it can also challenge what the first four missed. |
After each skill, save its findings to a scratch file (<scratchpad>/review-all/<n>-<skill>.md)
before you start the next one. That way the merge reads files, not a long context.
Step 3: merge into one report
- Read the five scratch files.
- Refute skills 2 to 5.
code-reviewalready refutes its own findings. The findings from ponytail-review, blast-radius, thermo-nuclear-code-quality-review and interrogate arrive unchecked. Batch them, about 5 per Sonnet agent, grouped by file. Each refuter tries to disprove its findings by a different method than the one that found them (run a command, or read the connected code instead of the diff). Follow code-council's refuter rules (~/.claude/skills/code-council/SKILL.md, "Accuracy first, then cost" section). Medium, high and critical findings also get a second blind refuter: a different agent that never sees the first verdict and must run something (a command or a test), not only read. Verdicts: REFUTED (drop it, and note it in the report), CONFIRMED (keeps its proof), UNVERIFIED (when unsure; never default to REFUTED). At most 5 agents at once. A dead or errored refuter is not a verdict: re-run it once, then list its findings as UNVERIFIED and the refuter under "Did not run". - Merge duplicates: the same defect at the same
file:line, or the same root cause, is one finding. Keep the clearest wording and list every skill that found it. - Severity: map every skill's scale to one: critical, high, medium, low (Critical = critical, Important = high, Must fix = high, Minor or Nice to have = low). Use the highest severity any skill gave it. A finding that two or more skills reached on their own goes up one level (global CLAUDE.md rule).
- Proof: every finding needs
file:lineplus a command, test or query that shows it. A finding that survived step 2 keeps that proof. Any other finding is labelled UNVERIFIED. Keep it in the report and don't drop it. - Conflicts: if one skill flags something and another argues it is fine, keep the finding and show both sides in one line.
Load the unslop skill, then write the report:
- Top: what was reviewed (target and commit), the verdict (ship, fix first, or rethink), and what the user must decide.
- Findings, most severe first. Each one: the problem in plain words, a concrete example,
why it matters, the smallest fix (prefer fixes that delete code), and which skills found it.
Evidence (
file:line, command and output) goes last, under "How we know". - Cost: one line. Count the agents you launched and the refuters yourself. Take tokens from code-review's run-cost line ("Agents: ... cost: ..."). Write "tokens unknown" for the other four skills, because they report no cost.
- Did not run: any skill that failed twice, and why.
Write the merged report to <scratchpad>/review-all/report.md. The session model does the
merge; if the session is not on Opus, hand the five files to one Opus subagent to merge.
Outside caller mode, if the ReportFindings tool is available, report through it instead of
as text.
Step 4: ask before acting
With caller: plan-and-build, skip this step. Return to the caller: the verdict (ship, fix
first, or rethink), the report path, and every critical and high finding, UNVERIFIED ones
included. The caller handles fixes.
Otherwise, end by asking the user:
- which findings to fix (fixes go through a separate review afterwards), and
- in PR mode, whether to post the report to the PR. Post only on a clear yes in this conversation.
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.

