---

name: validation-plan-artifacts description: This skill should be used when the user asks to "review research", "review data model", "review contracts", or mentions "plan quality", "phase review", or "design gaps". Provides phase-specific review criteria for planning artifacts with issue classification.

Reviewing Plan Artifacts

Purpose

Find gaps in planning artifacts and generate issues that need resolution before proceeding to the next phase. Focus on design completeness and quality, not implementation details. This skill provides phase-specific review criteria for the Devil's Advocate.

Review Focus by Phase

Each phase has specific checks to execute. The checks identify Critical, Important, and Minor issues.

See PHASE-CHECKLISTS.md for detailed phase-specific checklists and key questions.

Issue Classification

Issues are classified by severity to determine appropriate action:

See ISSUE-TEMPLATES.md for severity classification rules, issue documentation formats, and report templates.

Review Process

Step 1: Gather Context

Read and understand:

• The artifact being reviewed
• The spec requirements it should satisfy
• Previous artifacts (for consistency checks)
• Constitution principles (for compliance)

Step 2: Execute Checks

For each check in the phase-specific checklist:

1. Ask the question
2. Look for evidence in the artifact
3. If issue found, classify severity
4. Document the issue

Step 3: Cross-Reference

• Check traceability (can trace requirement -> artifact)
• Check consistency (artifacts agree with each other)
• Check completeness (nothing obviously missing)

Step 4: Generate Report

• Classify verdict based on issues found
• Document all issues with evidence
• Provide specific, actionable suggestions
• Acknowledge what was done well

Incremental Review Mode

For phases after the first artifact (data-model, contracts), use incremental review to optimize time while preserving rigor.

Full Review (New Artifact Only)

• Execute ALL phase-specific checks from PHASE-CHECKLISTS.md
• Document issues with full evidence
• This is your primary focus—no shortcuts here

Consistency Check (Previous Artifacts)

• Use the cross-artifact checklist at ${CLAUDE_PLUGIN_ROOT}/templates/cross-artifact-checklist.md
• Do NOT re-read previous artifacts in full
• Spot-check: entity names, requirement IDs, decision references
• Flag only inconsistencies between artifacts
• Time budget: 1-2 minutes per previous artifact

When to Escalate to Full Re-Review

• If 2+ consistency issues found → re-read that specific artifact
• If contradictions detected → flag for supervisor
• If unsure → note uncertainty in report, recommend targeted review

Report Format (Incremental Mode)

Copy## Review Summary

| Aspect | Status |
|--------|--------|
| **New Artifact** | {artifact} - FULL REVIEW |
| **Previous Artifacts** | CONSISTENCY CHECK ONLY |

## New Artifact Issues

{Full issue documentation with evidence}

## Cross-Artifact Consistency

| Check | Status | Notes |
|-------|--------|-------|
| Entity names | Pass/Fail | {any mismatches} |
| Requirement IDs | Pass/Fail | {any gaps} |
| Decision alignment | Pass/Fail | {any contradictions} |

## Verdict

{ready / needs-revision / critical-gaps}

Phase Application

---

Verdict Criteria

Quality Checklist

Before finalizing review, verify:

• All phase-specific checks executed
• Issues properly classified by severity
• Evidence cited for each issue
• Suggested fixes are actionable
• Verdict matches issue severity
• Cross-artifact concerns noted
• Strengths acknowledged