From nw
Runs six validation checks on roadmaps for architecture reviews: external validity, AC coupling, decomposition ratio, no impl code, concision, hexagonal test boundaries.
npx claudepluginhub nwave-ai/nwave --plugin nwThis skill uses the workspace's default tool permissions.
Six mandatory checks for every roadmap review. Each produces a finding block.
Provides structured critique dimensions for peer reviewing code and tests: detects implementation bias (YAGNI, premature optimization), validates test quality (coupling, flakiness), checks completeness and priorities.
Interactively reviews implementation plans before coding, challenging scope creep, architecture, quality, tests, and performance with mandatory user checkpoints and opinionated recommendations.
Validates implementation plans against codebase reality, architecture, quality, risks, and conventions before execution using four parallel specialized reviewers.
Share bugs, ideas, or general feedback.
Six mandatory checks for every roadmap review. Each produces a finding block.
Verify completing all steps produces an INVOCABLE feature, not just existing code.
Criteria: at least one step targets entry point integration | acceptance tests invoke through driving ports | clear user invocation path after completion
Severity: BLOCKER if no integration step or invocation path. HIGH if tests at wrong boundary.
EXTERNAL VALIDITY CHECK: PASSED|FAILED
Issue: {missing wiring}
Consequence: {what happens without fix}
Required Action: {integration step to add}
Scan AC for implementation details locking crafter into predetermined structure.
Detection: underscore-prefixed identifiers | method signatures with params | internal class references | specific return types
Severity: HIGH. Flag each coupled AC with rewrite.
_install_des_module() copies src/des/DES module importable from installation target after installRationale: AC = WHAT (observable behavior), never HOW (internal structure). Crafter decides during GREEN + REFACTOR.
Calculation: implementation steps / unique production files in files_to_modify
Thresholds: acceptable <= 2.0 | warning 2.0-2.5 | reject > 2.5
Identical pattern check: 3+ steps with identical AC differing by substitution -> require batching.
Verify no implementation code. BLOCKER.
Detection: code snippets/algorithms in descriptions | method implementations in AC | pseudocode/logic | variable names, loops, conditionals
Architect defines WHAT; crafter decides HOW. Roadmap code prevents better solutions via TDD.
Quantitative thresholds: total words 500 (1-3 steps) / 1500 (4-8) / 3000 (9-15) | step description max 50 words | AC max 5 per step, 30 words each | notes max 100 words
Verbosity detection: multi-sentence blocks -> bullets | qualifiers (comprehensive, robust) -> delete | motivational language -> delete | redundancy -> state once | tutorials -> delete, assume expertise
Precision: every AC single interpretation | testable outcome | business over technical jargon
Severity: BLOCKER if word count exceeds threshold.
Verify unit tests respect hexagonal boundaries.
Rules: invoke through driving ports only | no domain entity unit tests | behavior focus | never import internal/private modules
Tests verify TemplateValidator._validate_schema() handles edge casesWhen invalid template provided to render_prompt(), ValidationError raised with clear messageSeverity: HIGH.