Review Workflow

---
name: review-workflow
description: Complete quality assurance workflow orchestrating validation, comprehensive review, and functional testing. Sequential workflow from quality gating through multi-dimensional review to scenario testing. Use when conducting complete skill quality assurance, pre-deployment validation, or comprehensive quality checks combining multiple review approaches.
allowed-tools: Read, Write, Edit, Glob, Grep, Bash, WebSearch, WebFetch
---

# Review Workflow

## Overview

review-workflow provides complete quality assurance by orchestrating three complementary review skills into a comprehensive validation workflow.

**Purpose**: Complete quality assurance from quality gating through comprehensive review to functional testing

**Component Skills**:
1. **skill-validator** - Pass/fail quality gate (minimum standards)
2. **review-multi** - Comprehensive multi-dimensional review (1-5 scoring)
3. **skill-tester** - Functional testing in real scenarios

**Workflow Pattern**: Sequential (validator → review → testing)

**Result**: Complete quality assessment with deployment decision, quality scores, and functional validation

## When to Use

- Pre-deployment quality assurance (complete validation before release)
- Comprehensive quality checks (all aspects validated)
- Certification process (validate skill meets all standards)
- Major update validation (ensure quality maintained after changes)
- Quality audit (periodic comprehensive checks)

## Review Workflow

### Step 1: Quality Gate Validation (skill-validator)

**Purpose**: Fast pass/fail check - ensures minimum standards met before deeper review

**Component Skill**: skill-validator

**Process**:
1. Run skill-validator on skill
2. Check all 4 validation operations:
   - Structure validation
   - Content validation
   - Pattern validation
   - Production readiness
3. Result: PASS or FAIL

**Decision Point**:
- **PASS** → Proceed to Step 2 (comprehensive review)
- **FAIL** → Stop, fix critical issues, re-run Step 1

**Outputs**:
- Validation report (pass/fail)
- Critical issues list (if failed)
- Go/no-go decision

**Time Estimate**: 30-45 minutes

---

### Step 2: Comprehensive Review (review-multi)

**Purpose**: Multi-dimensional quality assessment with 1-5 scoring

**Component Skill**: review-multi

**Process**:
1. Run all 5 review operations:
   - Operation 1: Structure Review (5-10 min, automated)
   - Operation 2: Content Review (15-30 min, manual)
   - Operation 3: Quality Review (20-40 min, mixed)
   - Operation 4: Usability Review (30-60 min, manual testing)
   - Operation 5: Integration Review (15-25 min, manual)
2. Calculate weighted overall score
3. Map to grade (A/B/C/D/F)
4. Assess production readiness
5. Generate improvement recommendations

**Outputs**:
- Overall score (1.0-5.0)
- Grade (A/B/C/D/F)
- Per-dimension scores
- Production readiness assessment
- Prioritized improvement recommendations

**Time Estimate**: 1.5-2.5 hours

---

### Step 3: Functional Testing (skill-tester)

**Purpose**: Validate skill works correctly in real scenarios

**Component Skill**: skill-tester

**Process**:
1. Run testing operations:
   - Scenario Testing (test in realistic use case)
   - Example Validation (verify examples execute)
   - Integration Testing (test with other skills if applicable)
   - Usability Testing (assess ease of use)
2. Document test results
3. Identify any functional issues

**Outputs**:
- Scenario test results (success/failure)
- Example validation results (all pass/some fail)
- Integration test results
- Usability assessment
- Functional issues list (if any)

**Time Estimate**: 45-90 minutes

---

## Post-Workflow: Deployment Decision

After completing all 3 steps:

### Aggregate Results

**Quality Gate** (Step 1):
- PASS/FAIL status
- Critical issues (if any)

**Comprehensive Review** (Step 2):
- Overall score: X.X/5.0
- Grade: A/B/C/D/F
- Production readiness assessment
- Improvement recommendations

**Functional Testing** (Step 3):
- Scenario tests: Pass/Fail
- Examples: All working/Some broken
- Integration: Working/Issues
- Usability: Good/Needs improvement

### Make Deployment Decision

**Deploy Immediately**:
- Quality Gate: PASS
- Overall Score: ≥4.5 (Grade A)
- Functional Tests: All passing
- **Action**: Deploy to production

**Deploy with Notes**:
- Quality Gate: PASS
- Overall Score: 4.0-4.4 (Grade B+)
- Functional Tests: Passing
- **Action**: Deploy, note improvements for next iteration

**Hold for Improvements**:
- Quality Gate: PASS
- Overall Score: 3.5-3.9 (Grade B-)
- OR Functional Tests: Some failures
- **Action**: Fix identified issues, re-run workflow

**Do Not Deploy**:
- Quality Gate: FAIL
- OR Overall Score: <3.5 (Grade C-F)
- OR Functional Tests: Major failures
- **Action**: Significant rework required

---

## Best Practices

### 1. Always Run Complete Workflow
**Practice**: Run all 3 steps before deployment (don't skip)

**Rationale**: Each step catches different issues - comprehensive coverage

### 2. Fix Critical Issues Immediately
**Practice**: If Step 1 fails, fix issues before proceeding

**Rationale**: Wastes time to do comprehensive review on skill failing basic validation

### 3. Document All Findings
**Practice**: Record results from each step for complete record

**Rationale**: Enables tracking improvements over time, learning from patterns

### 4. Apply Improvements Iteratively
**Practice**: Use Step 2 recommendations to plan next version

**Rationale**: Continuous improvement - each iteration gets better

### 5. Re-Run After Major Changes
**Practice**: Re-run workflow after applying significant improvements

**Rationale**: Validates improvements effective, no regressions introduced

---

## Quick Reference

### The 3-Step Review Workflow

| Step | Skill | Purpose | Time | Output |
|------|-------|---------|------|--------|
| 1 | skill-validator | Quality gate (pass/fail) | 30-45m | PASS/FAIL, deploy decision |
| 2 | review-multi | Comprehensive review (1-5) | 1.5-2.5h | Score, grade, recommendations |
| 3 | skill-tester | Functional testing | 45-90m | Test results, functional validation |

**Total Time**: 3-4 hours for complete QA

### Workflow Pattern

```
skill-validator (Quality Gate)
    ↓
  PASS? ──No──> Fix issues, retry
    ↓ Yes
review-multi (Comprehensive Review)
    ↓
Overall Score + Grade + Recommendations
    ↓
skill-tester (Functional Testing)
    ↓
Test Results + Usability Assessment
    ↓
Deployment Decision (Deploy/Hold/Rework)
```

### Deployment Decision Matrix

| Validator | Review Score | Tests | Decision |
|-----------|--------------|-------|----------|
| PASS | ≥4.5 (A) | All Pass | ✅ Deploy Now |
| PASS | 4.0-4.4 (B+) | All Pass | ✅ Deploy + Note Improvements |
| PASS | 3.5-3.9 (B-) | Pass/Some Fail | ⚠️ Hold - Fix Issues |
| PASS | <3.5 (C-F) | Any | ❌ Rework Required |
| FAIL | Any | Any | ❌ Fix Critical, Retry |

### Integration with Development

```
Build skill (development-workflow) →
Review workflow (this skill) →
Apply improvements (skill-updater) →
Re-validate (review-workflow) →
Deploy
```

---

**review-workflow provides complete quality assurance through orchestrated validation, review, and testing.**