agentleFS
ambroselittleambroselittle/agent-skills/skills/plan-review/SKILL.md

plan-review

Skill0 starsChanged 15 days agoMITView on GitHub

Review an implementation plan with specialized parallel agents. Accepts a work slug, a full path, or no argument (auto-discovers from current branch). Use after /plan-work produces a plan, or on any plan document you want expert review of.

43 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use plan-execute, pr-comment and 9 more.

Other files people use alongside this one.

Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
  • longranger2Skill

    plan-execute

    Use when the user says "/plan-execute", "plan execute", "implement plan", or "execute plan" and provides a finalized plan file path to carry out. Claude orchestrates, Codex writes code, Claude reviews, and Codex fixes issues until the quality bar is met.

    Also in 3 projects that use this

  • BenjaminGSkill

    pr-comment

    This skill should be used when reviewing someone else's pull request and you want drafted review comments posted to GitHub — either gate / gate-wf findings (tier-marked) or pr-challenge's reviewer questions (unmarked) — each drafted through humanizer and posted as a standalone inline comment at its file:line, after one confirmation. Triggers on "post these as review comments", "comment the gate findings on the PR", "draft and post my review", "leave review comments", after a gate / gate-wf run or a pr-challenge pass on a PR you're reviewing.

    Also in 3 projects that use this

  • MicrosoftSkill

    spec-review

    Independent, multi-model review of a design or spec **before** any code is written, for the microsoft/winappcli repo. Activate when a contributor asks to "review this spec", "review my design", "review this design doc", "validate this approach", "should we build this", "spec review", "design review", or "feature review". Fans out parallel sub-agents — each doing its OWN research against the real codebase and ecosystem rather than trusting the spec — covering necessity & scope, approach & alternatives, feasibility vs reality, risks & unknowns, DX & user impact, and a different-model-family cross-check. Emits a decision-oriented recommendation (proceed / proceed-with-changes / reconsider) to stdout. This is the PRE-CODE companion to the pr-review skill (which reviews code already written); use spec-review at the design/spec stage, not on an implemented diff. Does NOT write code or edit the spec.

    Also in 3 projects that use this

  • AnthropicSkill

    office-hours

    Set up and run Claude Code office hours, including the calendar event when the champion's own calendar connector allows it. Use when a champion wants a regular time for teammates to bring problems.

    Also in 3 projects that use this

  • AWSSkill

    retro

    Conduct a retrospective on a coding session.

    Also in 5 projects that use this

  • BlockSkill

    plan

    Draft a TDD, PRD, or lightweight plan in the conversation without committing it.

    Also in 5 projects that use this

The file

Show its rules and text

We split this skill into the rules it gives an agent

19 rules. 0 are common, 13 are distinctive, and 6 are very specific to this project.

  • 0

    Common

    Most of the 35 edited versions we read keep these.

  • 13

    Distinctive

    This author's own choices. Worth a look.

  • 6

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Arguments: Use the arguments supplied with this skill invocation or the user's current request.

    line 11
  2. Very specific:

    Read ~/.agent-skills/skills/shared/references/harness-capabilities.md before proceeding.

    line 13
  3. Distinctive:

    Context to load explicitly: Run each command below before interpreting its output.

    line 15
  4. Very specific:

    Setup check: If Work folder shows needs-setup, stop: "Run /setup-agent-skills first to configure your work folder, then come back."

    line 21
  5. Distinctive:

    Read the resolved plan. If it's empty or doesn't look like an implementation plan (no phases, no tasks), say so and stop.

    line 37
  6. Distinctive:

    Use the loaded "Plan reviewer agents" context.

    line 47
  7. Very specific:

    All built-in plan reviewers are always-on.

    line 51
  8. Very specific:

    Read ~/.agent-skills/skills/plan-review/references/plan-finding-format.md — include its full content in each agent prompt.

    line 55
  9. Very specific:

    Read applicable AGENTS.md, legacy CLAUDE.md, and .agents/rules/ / .claude/rules/.

    line 59
  10. Distinctive:

    Wait for all reviewers to complete.

    line 84
  11. Distinctive:

    If any MISMATCH findings exist, present them:

    line 96
  12. Distinctive:

    Wait for the user to address each mismatch.

    line 105
  13. Distinctive:

    If no mismatches: "Mismatches: clean — no conflicts between the plan and codebase."

    line 107
  14. Distinctive:

    If no improvements: "Improvements: none — the plan's approach looks solid."

    line 121
  15. Very specific:

    Add to the plan's ## Alternatives Considered section regardless of user decision.

    line 132
  16. Distinctive:

    If no alternatives: "Alternatives: none flagged."

    line 134
  17. Distinctive:

    If all three categories come back clean: "All reviewers returned clean — no mismatches, improvements, or alternatives flagged.

    line 138
  18. Distinctive:

    Read before passing: Always read agent files and the plan before spawning reviewers.

    line 147
  19. Distinctive:

    Don't manufacture findings: If the plan is clean, say so.

    line 150

“Common” is measured: kept by at least 6 in 10 of the 35 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.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Plan Review: Parallel Expert Review of Implementation Plans

You are a senior engineer coordinating a parallel review of an implementation plan. Your job is to run specialized reviewers and synthesize their findings into actionable feedback.

Arguments: Use the arguments supplied with this skill invocation or the user's current request.

Read ~/.agent-skills/skills/shared/references/harness-capabilities.md before proceeding. Use its active-harness mappings for skills, questions, progress, delegation, model tiers, project instructions, and connected tools.

Context to load explicitly: Run each command below before interpreting its output. These are shell commands to execute, not values supplied automatically by the harness.

  • Work folder: bash ~/.agent-skills/scripts/shared/context.sh work-folder
  • Ticket ID: bash ~/.agent-skills/scripts/shared/context.sh ticket-id
  • Plan reviewer agents: bash ~/.agent-skills/scripts/plan-review/context.sh plan-reviewer-agents

Setup check: If Work folder shows needs-setup, stop: "Run /setup-agent-skills first to configure your work folder, then come back."


Phase 0: Resolve the Plan

Determine what plan to review from the invocation input:

Input detection

  • Bare slug — looks like a slug (lowercase, hyphens, no spaces). Resolve to <work-folder>/plan.md using the configured work root.
  • File path — starts with /, ./, or ~, or ends with .md. Read the file directly.
  • No arguments — use the loaded Work folder. If not "none", resolve to <work-folder>/plan.md. If "none": "No plan found. Provide a work slug or file path, or switch to your feature branch first."

Validate

Read the resolved plan. If it's empty or doesn't look like an implementation plan (no phases, no tasks), say so and stop.

Store the plan content — you'll pass it to every reviewer.


Phase 1: Run Reviewers (Parallel)

Determine which reviewers to run

Use the loaded "Plan reviewer agents" context. This lists:

  • always-on agents — run unconditionally
  • selectable agents — mention them to the user and ask which to include (or skip if none)

All built-in plan reviewers are always-on. For each exact slug, use .agents/plan-reviewers/<name>.md first, then legacy .claude/agents/plan-reviewers/<name>.md, then the built-in persona at ~/.agent-skills/skills/plan-review/agents/<name>.md. Run one winning persona per name; do not move existing repository overrides.

Read the finding format

Read ~/.agent-skills/skills/plan-review/references/plan-finding-format.md — include its full content in each agent prompt.

Read project instructions

Read applicable AGENTS.md, legacy CLAUDE.md, and .agents/rules/ / .claude/rules/; reviewers need project conventions. Deduplicate linked copies.

Spawn all reviewers in parallel using the medium model tier

Each agent prompt:

"You are a plan reviewer. Read your reviewer persona below, then review the implementation plan provided.

Your reviewer persona: [paste full contents of the agent's .md file, excluding the always: true frontmatter line]

Plan finding format (you MUST use this format for all findings): [paste full contents of plan-finding-format.md]

Project conventions: [paste applicable project instructions, or state that none were found]

The plan to review: [paste full plan document]

READ ONLY — no edits, no running tests, typecheck, lint, format, or starting any processes. Budget: if you have not produced a structured finding report after reviewing the plan once, stop and return what you have.

Return findings grouped by type: MISMATCHES, IMPROVEMENTS, ALTERNATIVES. If a category has no findings, say 'None' — do not omit the section."

Wait for all reviewers to complete.


Phase 2: Synthesize and Present (Stepwise, Dialogic)

Collect all findings from all reviewers. Save the full findings to disk (in <work-folder>/reviews/ as <slug>.plan-review.<YYYY-MM-DD>.md, creating the directory with mkdir -p if needed) for reference and session recovery. Then present findings to the user one category at a time — don't dump everything at once.

Step 1: MISMATCHES — Present first, resolve before moving on

If any MISMATCH findings exist, present them:

"Mismatches found — the plan conflicts with what's actually in the code:

M1. [What the plan says] vs. [what the code shows] — flagged by [reviewer] M2. ...

These need to be resolved before we proceed. What's the right direction for each?"

Wait for the user to address each mismatch. Update the plan accordingly.

If no mismatches: "Mismatches: clean — no conflicts between the plan and codebase."

Step 2: IMPROVEMENTS — Present next

"Improvements — the reviewers found better approaches:

I1. [change] — suggested by [reviewer] I2. [change] — suggested by [reviewer]

I'll apply all of these unless you want to skip any. (e.g., 'skip I2')"

Apply approved improvements to the plan. Add a ## Changes from Review section listing what changed and which reviewer suggested it.

If no improvements: "Improvements: none — the plan's approach looks solid."

Step 3: ALTERNATIVES — Present last

"Alternatives — worth considering but not necessarily better:

A1. [approach] — tradeoffs: [vs current]. Flagged by [reviewer]

These are added to the plan's Alternatives Considered section for reference. Want to adopt any of these instead of the current approach?"

Add to the plan's ## Alternatives Considered section regardless of user decision.

If no alternatives: "Alternatives: none flagged."

Clean review

If all three categories come back clean: "All reviewers returned clean — no mismatches, improvements, or alternatives flagged. The plan looks solid."


Next step: Review complete. Continue with /do-work to start implementing.

Guidelines

  • Read before passing. Always read agent files and the plan before spawning reviewers.
  • Parallel, not serial. Spawn all reviewers at once — they're independent.
  • Findings are suggestions, not commands. The user decides what to act on.
  • Don't manufacture findings. If the plan is clean, say so. A clean review is a good outcome.
  • Stay in your lane. This skill reviews plans — it doesn't write code, run tests, or modify anything beyond the plan document itself.

Download now (ZIP)

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
What did you use it for? e.g. Reviewing pull requests in a Rails app

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.