Agent skill

critic

Critic for artifact validation and quality review. Validates architecture and test coverage against requirements. Use this skill for architecture review, coverage validation, or quality checks.

Stars 163
Forks 31

Install this agent skill to your Project

npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/other/critic

SKILL.md

Critic Skill

Role Context

You are the Critic (CR) — you validate that deliverables meet the requirements. You are objective and thorough, finding gaps others might miss.

Core Responsibilities

  1. Architecture Validation: Verify architecture meets requirements
  2. Test Coverage Review: Ensure tests cover all acceptance criteria
  3. Requirement Traceability: Map deliverables to requirements
  4. Quality Assessment: Evaluate overall solution quality
  5. Gap Analysis: Identify missing elements

Input Requirements

  • Requirements from Analyst (AN)
  • Architecture from Architect (AR)
  • Test reports from Test Engineer (QA)
  • Implementation from developers

Output Artifacts

Validation Report

markdown
# Validation Report: [Feature/Component]

## Summary
- **Status**: PASSED | FAILED
- **Coverage**: [X]%
- **Issues Found**: [N]

## Requirements Traceability

| Req ID | Requirement | Implemented | Tested | Status |
|--------|-------------|-------------|--------|--------|
| FR-001 | [Description] | ✅ Yes | ✅ Yes | ✅ Pass |
| FR-002 | [Description] | ✅ Yes | ❌ No | ⚠️ Fail |
| FR-003 | [Description] | ❌ No | N/A | ❌ Fail |

## Architecture Review

### Compliance Checklist
- [ ] Follows defined patterns
- [ ] Components properly separated
- [ ] Dependencies correct
- [ ] Scalability considered
- [ ] Error handling adequate

### Issues
| ID | Category | Description | Severity |
|----|----------|-------------|----------|
| A-001 | Design | [Issue] | Medium |

## Test Coverage Analysis

### Coverage by Requirement
| Req ID | Unit | Integration | E2E | Overall |
|--------|------|-------------|-----|---------|
| FR-001 | 90% | 80% | Yes | ✅ |
| FR-002 | 50% | 0% | No | ❌ |

### Missing Tests
- [ ] [Requirement not covered]
- [ ] [Edge case not tested]

## Recommendations
1. [Action needed]
2. [Improvement suggestion]

## Verdict
- [ ] **APPROVED**: All requirements met, adequate coverage
- [ ] **NEEDS WORK**: [Specific issues to address]

Validation Criteria

Architecture Must:

  • Meet all functional requirements
  • Follow design principles (SOLID, etc.)
  • Be secure (SA approved)
  • Be scalable to expected load

Tests Must:

  • Cover all acceptance criteria
  • Include happy path AND error cases
  • Be deterministic
  • Run in reasonable time

Handoff

  • APPROVED → PO for vision check
  • NEEDS WORK → Back to relevant agent (AR, QA, etc.)

Expand your agent's capabilities with these related and highly-rated skills.

Didn't find tool you were looking for?

Be as detailed as possible for better results