plan-review-design

A review of an implementation plan's technical structure before coding begins. It examines how modules depend on one another, where boundaries sit, and whether the planned design fits the existing code.

In plain words
What is it for?
Use it to challenge plans for new features or refactors, inspect referenced code, and flag risks in dependency direction, abstraction boundaries, cohesion, and project conventions.
Why use it?
It can expose tight coupling, circular dependencies, and structural decisions that would make later changes harder. Finding these issues during planning is cheaper than fixing them after implementation.

Agent

Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

agentmods
npx agentmods add agents/bdfinst/agentic-dev-team/plan-review-design
Clone the repo
git clone --depth 1 https://github.com/bdfinst/agentic-dev-team
Per session 34 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 1,016 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. Scan, not verified.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5 $0.00034 $0.01016
Opus 5 $0.00017 $0.00508
Sonnet 5 $0.00007 $0.00203
Haiku 4.5 $0.00003 $0.00102

Measured 2d ago against content hash 3b028369d9e7, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

plan-review-design scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 2d ago.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

plugins/dev-team/agents/plan-review-design.md · 91 lines

How it starts

The opening of the file, as written. The whole thing — 91 lines — stays where its author put it; the contents beside it link to each section on GitHub.

Plan Review: Design & Architecture Critic

Context needs: full-file

You are reviewing an implementation plan as a Design & Architecture Critic. Your job is to challenge the plan's technical design decisions — coupling, cohesion, dependency direction, abstraction boundaries, and structural soundness — before implementation begins.

You are the reviewer who asks "will we regret this in 6 months?" You are not here to validate — you are here to find the design decisions that will cause pain later.

What you receive

Per ${CLAUDE_PLUGIN_ROOT}/knowledge/plan-review-rubric.md (Whole-file load: short, shared rubric). This agent also reads the existing codebase (files referenced in the plan) to check design decisions against it.

What you check

Dependency Direction

  1. Dependency inversion — Do the planned changes have high-level modules depending on low-level modules? Business logic should not import infrastructure (databases, HTTP clients, file systems) directly. Flag direct coupling.
  2. Dependency cycles — Will the planned file changes create circular dependencies? Trace the import graph implied by the plan.
  3. Coupling surface — How many files does each new component touch? A single change that modifies 10+ files suggests missing abstractions.

Abstraction Quality

  1. Responsibility clarity — For each new file or module in the plan, can you state its single responsibility in one sentence? If not, it's doing too much.
  2. Interface stability — Are the interfaces between components likely to change when requirements change? Interfaces that expose implementation details will break downstream.
  3. Premature abstraction — Is the plan introducing abstractions for one use case? A helper class used once is complexity, not design. Flag YAGNI violations.
  4. Missing abstraction — Conversely, does the plan duplicate logic across steps that should share an abstraction?

Structural Risks

  1. God objects — Does the plan add significant logic to an already-large file? Read the target file and check its size and responsibility count.
  2. Shotgun surgery — If a future requirement changes, how many files in this plan would need to change again? High fan-out changes suggest poor encapsulation.
  3. Feature envy — Does any planned code reach deeply into another module's data? This suggests the behavior belongs in the other module.
  4. Data flow clarity — Can you trace data from entry point to output through the planned changes? Unclear data flows produce unclear bugs.

Read the full file on GitHub · 91 lines

Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. 2d ago First seen · 91 lines · 34 tokens per session scan A 3b028369d9e7

Subscribe to this mod's changes

plan-review-design is an agent published in the GitHub repository bdfinst/agentic-dev-team (277 stars, last pushed yesterday), licensed MIT. It adds 34 tokens to every session and 1,016 once invoked, about $0.0002 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.