---

name: superspec:execute description: | Use when executing implementation plans in current session. Dispatches fresh subagent per task with two-stage review. Spec Reviewer validates against Specs, Quality Reviewer checks code quality.

Subagent-Driven Development

🔴 CRITICAL: Phase Protocol (Prevents Context Drift!)

Before starting ANY phase, you MUST read skills/phase-protocol/SKILL.md.

This protocol prevents you from forgetting tasks during long development sessions:

Copy┌────────────────────────────────────────────────────────────────┐
│ PHASE PROTOCOL SUMMARY                                          │
├────────────────────────────────────────────────────────────────┤
│ ENTRY (start of each phase):                                    │
│   1. Read phase-protocol skill (refresh context)                │
│   2. Read tasks.md (get task list)                              │
│   3. CREATE TODO IMMEDIATELY ← Before reading other docs!       │
│   4. Gate: Verify TODO completeness                             │
│   5. Read plan.md, design.md, specs/*.md                        │
│   6. Gate: Output key understanding                             │
│   7. Begin implementation                                       │
├────────────────────────────────────────────────────────────────┤
│ EXIT (end of each phase):                                       │
│   1. Update tasks.md                                            │
│   2. Git commit                                                 │
│   3. Re-read phase-protocol skill ← Loop back!                  │
│   4. Create next phase TODO                                     │
└────────────────────────────────────────────────────────────────┘

Why this works: TODO survives context compression. Exit Gate forces re-read.

---

⚠️ CRITICAL: Document Updates (Read This First!)

You MUST maintain TWO separate progress records:

TodoWrite is NOT a substitute for updating tasks.md!

After EVERY task completion, you MUST:

1. ✅ Mark task [x] in superspec/changes/[id]/tasks.md
2. ✅ Update Status counts at top of tasks.md
3. ✅ Update Completion Tracking table at bottom of tasks.md

Self-check: If you only used TodoWrite without editing tasks.md, you did NOT update progress correctly!

---

Overview

Execute plan by dispatching fresh subagent per task, with two-stage review:

1. Spec Compliance Review - Does implementation match Specs?
2. Code Quality Review - Is the code well-written?

Core principle: Fresh subagent per task + two-stage review = high quality, fast iteration

Announce at start: "I'm using subagent-driven development to execute the plan."

When to Use

CopyHave implementation plan?
    ↓ yes
Tasks mostly independent?
    ↓ yes
Stay in this session?
    ↓ yes
→ Use subagent-development

Prerequisites

• Proposal exists: superspec/changes/[id]/proposal.md
• Design exists: superspec/changes/[id]/design.md - Required
• Specs exist: superspec/changes/[id]/specs/**/*.md
• Plan exists: superspec/changes/[id]/plan.md
• Tasks exist: superspec/changes/[id]/tasks.md
• Specs validated: superspec validate [id] --strict passes

The Process

Copy┌────────────────────────────────────────────────────────────────────┐
│                    Per Task Loop                                    │
├────────────────────────────────────────────────────────────────────┤
│                                                                     │
│   ┌─────────────────────────────────────────────────────────────┐  │
│   │ 1. Dispatch Implementer Subagent                            │  │
│   │    - Provide: Full task text + context + Spec reference     │  │
│   │    - Follows TDD: RED → GREEN → REFACTOR                    │  │
│   │    - Self-reviews before handoff                            │  │
│   └──────────────────────────┬──────────────────────────────────┘  │
│                              │                                      │
│                              ▼                                      │
│   ┌─────────────────────────────────────────────────────────────┐  │
│   │ 2. Dispatch Spec Reviewer Subagent                          │  │
│   │    - Reads: superspec/changes/[id]/specs/[cap]/spec.md      │  │
│   │    - Reads: Actual implementation code                       │  │
│   │    - Checks:                                                 │  │
│   │      ✓ Every Requirement has implementation                 │  │
│   │      ✓ Every Scenario has test                              │  │
│   │      ✓ No missing features                                   │  │
│   │      ✓ No extra features (not in Spec)                      │  │
│   └──────────────────────────┬──────────────────────────────────┘  │
│                              │                                      │
│                    [Spec Compliant?]                                │
│                    /              \                                 │
│                  No               Yes                               │
│                  │                  │                               │
│                  ▼                  ▼                               │
│   ┌──────────────────────┐   ┌─────────────────────────────────┐  │
│   │ Implementer fixes    │   │ 3. Dispatch Quality Reviewer    │  │
│   │ spec gaps            │   │    - Checks: Error handling,    │  │
│   └──────────┬───────────┘   │      type safety, SOLID, tests  │  │
│              │               │    - Classifies: Critical /      │  │
│              └─→ Re-review   │      Important / Suggestion      │  │
│                              └──────────────┬──────────────────┘  │
│                                             │                      │
│                                   [Quality Approved?]              │
│                                   /              \                 │
│                                 No               Yes               │
│                                 │                  │               │
│                                 ▼                  ▼               │
│                  ┌──────────────────────┐   ┌──────────────────┐  │
│                  │ Implementer fixes    │   │ Mark complete +  │  │
│                  │ quality issues       │   │ UPDATE DOCS!     │  │
│                  └──────────┬───────────┘   └────────┬─────────┘  │
│                             │                        │             │
│                             └─→ Re-review            ▼             │
│                                              Update tasks.md       │
│                                              Update plan.md        │
│                                                                     │
├────────────────────────────────────────────────────────────────────┤
│                                                                     │
│   ┌─────────────────────────────────────────────────────────────┐  │
│   │ 4. External AI Review (if review.enabled in project.yaml)   │  │
│   │    - Check task type: [FRONTEND] or [BACKEND]               │  │
│   │    - Frontend → Use review.frontend.provider (gemini/codex) │  │
│   │    - Backend → Use review.backend.provider (codex/gemini)   │  │
│   │    - Read external-review skill for details                  │  │
│   │    - CRITICAL: Hallucination check before applying fixes!   │  │
│   └─────────────────────────────────────────────────────────────┘  │
│                                                                     │
└────────────────────────────────────────────────────────────────────┘

[All tasks complete]
        ↓
┌────────────────────────────────────────────────────────────────────┐
│ Dispatch Final Code Reviewer                                        │
│ - Reviews entire implementation                                     │
│ - Checks cross-task consistency                                     │
│ - Validates all Specs fully implemented                            │
└────────────────────────────────────────────────────────────────────┘
        ↓
Use superspec:finish-branch

Step 1: Setup (Phase Entry Protocol)

🔴 FOLLOW THE PHASE ENTRY PROTOCOL FROM phase-protocol SKILL:

Copy1. READ phase-protocol skill: skills/phase-protocol/SKILL.md
   → This refreshes your context and provides the full protocol

2. READ tasks.md: superspec/changes/[id]/tasks.md
   → Identify which phase you are starting
   → Get the task list for this phase

3. CREATE TODO IMMEDIATELY ← 🔴 BEFORE READING OTHER DOCS!
   → Use TodoWrite with Entry Gate + Tasks + Exit Gate structure
   → This is CRITICAL: TODO survives context compression
   → See phase-protocol skill for TODO template

4. GATE: Verify TODO completeness
   → Output the completeness checklist
   → Ensure Entry (8) + Tasks (N) + Exit (3) items exist

5. Read plan.md: superspec/changes/[id]/plan.md
   → Extract task details and context
   → Note Spec references for each task

6. Read design.md: superspec/changes/[id]/design.md
   → Understand technical decisions

7. Read specs: superspec/changes/[id]/specs/**/*.md
   → Understand requirements and scenarios

8. **Detect frontend tasks** (see below)

9. GATE: Output key understanding
   → Must output phase goal, tasks, spec refs, technical approach
   → See phase-protocol skill for template

10. BEGIN IMPLEMENTATION
    → Now proceed with Per Task Loop

Frontend Task Detection

For each task, determine if it's a frontend task by checking:

Keywords in task description:

• UI, interface, component, page, view, form, modal, dialog
• Frontend, front-end, client-side
• Button, input, dropdown, navigation, menu, header, footer
• Layout, grid, flex, responsive, mobile
• Style, styling, theme, dark mode, light mode
• Animation, transition, hover, interaction

File types involved:

• .tsx, .jsx, .vue, .css, .scss, .html, .svelte

Capability/Spec names containing:

• ui, frontend, component, page, view, interface

Mark frontend tasks with [FRONTEND] tag in TodoWrite.

When dispatching implementer for a [FRONTEND] task:

• Read frontend-guidelines.md from this directory
• Include the guidelines in the implementer prompt

Step 2: Per Task - Implementer

Dispatch Prompt

For [FRONTEND] tasks, add the Frontend Guidelines section (see below) after the Context section.

CopyYou are implementing a specific task from a plan.

**Task:** [Full task text from plan]

**Spec Reference:**
- Requirement: [Name from plan]
- Scenario: [Name from plan]
- File: superspec/changes/[id]/specs/[cap]/spec.md

**Context:**
[Any additional context needed]

## [FRONTEND ONLY] Frontend Design Guidelines

> **Include this section ONLY for tasks tagged `[FRONTEND]`**
> Read and include content from `frontend-guidelines.md`

This is a frontend task. Follow these additional guidelines:

### Design Thinking (Required Before Coding)

Before writing any frontend code, explicitly address:
1. **Purpose**: What problem does this interface solve? Who uses it?
2. **Tone**: Pick a BOLD aesthetic direction (e.g., minimal, maximalist, retro-futuristic, luxury, playful, editorial, brutalist)
3. **Constraints**: Technical requirements (framework, performance, accessibility)
4. **Differentiation**: What makes this UNFORGETTABLE?

### Frontend Anti-Patterns (NEVER Do)
- Generic AI aesthetics
- Overused fonts (Inter, Roboto, Arial, system fonts)
- Cliched colors (purple gradients on white)
- Predictable layouts

### Additional Report Section

Include this in your report:

Frontend Design Decisions

Aesthetic Direction: [Chosen tone/style] Typography: [Font choices + rationale] Color Palette: [Colors + purpose] Key Visual Elements: [Distinctive design choices]

Copy
---

## CRITICAL: TDD is MANDATORY

You MUST follow TDD strictly. This is NON-NEGOTIABLE.

**The Iron Law:** NO PRODUCTION CODE WITHOUT A FAILING TEST FIRST

**TDD Cycle (follow exactly):**

1. **RED - Write failing test first**
   - Write test based on Scenario
   - Run test: `npm test -- --grep "[Scenario Name]"`
   - **CAPTURE THE OUTPUT** - you must show this later
   - Test MUST fail (not error, FAIL)

2. **GREEN - Write minimal code**
   - Implement ONLY what's needed to pass
   - Run test again
   - **CAPTURE THE OUTPUT** - you must show this later
   - Test MUST pass

3. **REFACTOR - Clean up (if needed)**
   - Keep tests green
   - Don't add features not in Spec

4. **COMMIT with Spec reference**

**Commit format:**

feat([capability]): [description]

Refs: superspec/changes/[id]/specs/[cap]/spec.md Requirement: [Name] Scenario: [Name]

Copy
## MANDATORY Report Format

When done, you MUST report in this exact format:

TDD Evidence

RED Phase (Test First)

Test written: [Show the test code]

Test run output (MUST FAIL):

Copy[Paste actual test output showing FAILURE]

GREEN Phase (Implementation)

Implementation: [Show the implementation code]

Test run output (MUST PASS):

Copy[Paste actual test output showing PASS]

Commit

Commit SHA: [sha] Message: [commit message with Spec reference]

Files Changed

• [list of files]

Copy
**WARNING:** If you cannot show failing test output BEFORE implementation,
you did NOT follow TDD. You must delete code and start over.

Ask questions if anything is unclear BEFORE starting.

If Implementer Asks Questions

• Answer clearly and completely
• Provide additional context if needed
• Don't rush into implementation

Verify TDD Evidence

Before dispatching Spec Reviewer, verify:

• Implementer showed failing test output (RED phase)
• Implementer showed passing test output (GREEN phase)
• Failing test output came BEFORE implementation
• Test failed for correct reason (missing function, not typo)

If TDD evidence is missing:

• Do NOT proceed to Spec Review
• Implementer must restart with proper TDD

Step 3: Per Task - Spec Reviewer

Dispatch Prompt

CopyYou are reviewing code for SPEC COMPLIANCE and TDD DISCIPLINE.
Do NOT review code quality - that's a separate review.

## CRITICAL: Do Not Trust the Implementer's Report

The implementer may have:
- Skipped TDD and written tests after code
- Claimed TDD compliance without evidence
- Missed requirements or added extra features

**YOU MUST verify everything independently by reading actual code.**

**DO NOT:**
- Take their word for what they implemented
- Trust claims about TDD compliance without evidence
- Accept "I followed TDD" without seeing failing test output

**DO:**
- Read the actual test code
- Read the actual implementation code
- Compare to Spec line by line
- Verify TDD evidence exists

## What to Review

**Spec File:** superspec/changes/[id]/specs/[cap]/spec.md

**Task Completed:**
- Requirement: [Name]
- Scenario: [Name]

**Implementer's Report:**
[Paste implementer's report here]

**Code to Review:**
[Git diff or file paths]

## Review Checklist

### 1. TDD Compliance (CRITICAL)

**Did implementer follow TDD?**
- [ ] Report shows failing test output (RED phase)
- [ ] Report shows passing test output (GREEN phase)
- [ ] Failing output appeared BEFORE implementation
- [ ] Test failed for correct reason (missing function, not typo/error)

**If TDD evidence is missing or incomplete:**
- Verdict is ❌ NEEDS WORK
- Implementer MUST restart with proper TDD
- Do NOT accept "I followed TDD" without proof

### 2. Requirement Coverage
- Does implementation match the Requirement description?
- Are all SHALL/MUST statements implemented?

### 3. Scenario Coverage
- Is there a test for this Scenario?
- Does the test cover WHEN condition?
- Does the test verify THEN result?
- Does the test verify AND conditions (if any)?

### 4. No Missing Features
- Are all aspects of the Scenario tested?
- Any edge cases in the Scenario not covered?

### 5. No Extra Features
- Is there any code NOT required by the Spec?
- Any "nice to have" additions not in Spec?

**Output Format:**

Spec Compliance Review

TDD Discipline

[✅ TDD FOLLOWED / ❌ TDD NOT FOLLOWED]

• Failing test evidence: [✅ Present / ❌ Missing]
• Passing test evidence: [✅ Present / ❌ Missing]
• Correct order (test before code): [✅ Yes / ❌ No / ⚠️ Cannot verify]

Requirement: [Name]

[✅ COMPLIANT / ❌ NOT COMPLIANT]

Scenario: [Name]

• Test exists: [✅/❌]
• WHEN covered: [✅/❌]
• THEN verified: [✅/❌]
• AND verified: [✅/❌ or N/A]

Issues Found

1. [Issue description] - [Missing/Extra/Incorrect/TDD violation]

Verdict

[✅ SPEC COMPLIANT / ❌ NEEDS WORK]

If not compliant, list specific fixes needed. If TDD not followed, implementer must DELETE code and restart.

Copy

If Spec Review Fails

1. If TDD not followed:

   • Implementer MUST delete implementation code
   • Implementer restarts with proper TDD (test first, show evidence)
   • This is NON-NEGOTIABLE

2. If Spec gaps found:

   • Implementer fixes issues
   • Spec Reviewer reviews again

3. Repeat until compliant

4. Do NOT proceed to Quality Review until Spec Review passes

Step 4: Per Task - Quality Reviewer

Dispatch Prompt

CopyYou are reviewing code for QUALITY only.
Spec compliance has already been verified.

**Files to Review:**
[Git diff or file paths]

**Review Areas:**

1. **Error Handling**
   - Are errors caught appropriately?
   - Are error messages helpful?
   - No swallowed errors?

2. **Type Safety**
   - Proper TypeScript types?
   - No `any` without justification?
   - Null/undefined handled?

3. **Code Quality**
   - SOLID principles followed?
   - No code duplication?
   - Clear naming?
   - Appropriate abstraction level?

4. **Test Quality**
   - Tests are focused (one thing)?
   - Tests are readable?
   - No testing implementation details?
   - Mocks used appropriately?

**Issue Classification:**
- **Critical**: Must fix before proceeding
- **Important**: Should fix, impacts maintainability
- **Suggestion**: Nice to have, optional

**Output Format:**

Code Quality Review

Strengths

• [Good things about the code]

Issues

Critical

1. [Issue] - [Why it's critical] - [Suggested fix]

Important

1. [Issue] - [Impact] - [Suggested fix]

Suggestions

1. [Suggestion] - [Benefit]

Verdict

[✅ APPROVED / ❌ NEEDS WORK]

If needs work, list required fixes (Critical + Important).

Copy

If Quality Review Fails

1. Implementer (same subagent) fixes Critical and Important issues
2. Quality Reviewer reviews again
3. Repeat until approved
4. Suggestions are optional

Step 4.5: External AI Review (Optional)

Check superspec/project.yaml for review configuration:

Copyreview:
  enabled: true/false    # If false, skip this entire step

  frontend:
    provider: gemini     # gemini | codex | none
    model: gemini-3-pro-preview

  backend:
    provider: codex      # codex | gemini | none
    model: gpt-5.2-codex

When to Execute

Copyreview.enabled == false?  → Skip to Step 5
review.enabled == true?   → Continue below

Determine Task Type and Provider

Task is [FRONTEND]?

• Use review.frontend.provider
• If none → Skip external review

Task is [BACKEND]?

• Use review.backend.provider
• If none → Skip external review

Execute External Review

Read external-review skill for full details: skills/external-review/SKILL.md

Quick reference:

Copy# For Gemini (typically frontend):
uv run ~/.claude/skills/gemini/scripts/gemini.py \
  -m [review.frontend.model] \
  -p "Review this code: $(cat [file])"

# For Codex (typically backend):
uv run ~/.claude/skills/codex/scripts/codex.py \
  "Review @[file] for issues" \
  [review.backend.model]

🔴 CRITICAL: Hallucination Check

Before applying ANY suggestion from external AI:

Copy□ File exists? - Verify mentioned file paths
□ Function exists? - Search codebase for mentioned symbols
□ Makes sense? - Aligns with project architecture?
□ Not duplicate? - Not already implemented?

✅ Validated → Apply
❌ Hallucination → Ignore and document
⚠️ Partial → Modify before applying

DO NOT blindly apply external AI suggestions!

Document Review Results

Copy## External AI Review ([Codex/Gemini])

### Applied
1. [Suggestion] - [File] - [Change made]

### Rejected (Hallucination)
1. [Suggestion] - Reason: [file doesn't exist / etc.]

### Review Complete

Step 5: Mark Complete and Update Documents

⚠️ CRITICAL: Always update documents after completing each task!

REMINDER: TodoWrite is for YOUR internal tracking only. It does NOT update the actual tasks.md file! You MUST edit tasks.md directly using file editing tools.

After both reviews pass, perform these updates:

5.1 Update tasks.md

Open superspec/changes/[id]/tasks.md and mark the completed task:

Copy## Before
- [ ] 1.1 Implement user authentication
  - Spec: `### Requirement: User Auth` → `#### Scenario: Valid login`

## After
- [x] 1.1 Implement user authentication ✓ (completed YYYY-MM-DD)
  - Spec: `### Requirement: User Auth` → `#### Scenario: Valid login`

5.2 Update tasks.md Status Section

Update the status counts at the top of tasks.md:

Copy## Status
- Total Tasks: 10
- Completed: 3      ← Increment this
- In Progress: 1    ← Update as needed
- Pending: 6        ← Decrement this

5.3 Update plan.md Progress (Optional but Recommended)

Add progress markers to superspec/changes/[id]/plan.md:

Copy## Task 1: User Authentication ✅ COMPLETE

**Spec Reference:** `### Requirement: User Auth`
**Completed:** YYYY-MM-DD
**Commits:** abc1234, def5678

[... task details ...]

5.4 Phase Completion

When all tasks in a phase/section are complete:

1. Add phase completion marker to plan.md:

   Copy## Phase 1: Core Foundation ✅ COMPLETE (YYYY-MM-DD)
   

2. Update tasks.md with phase summary:

   Copy## 1. Core Foundation ✅ COMPLETE
   
   - [x] 1.1 Task one ✓
   - [x] 1.2 Task two ✓
   - [x] 1.3 Task three ✓
   
   **Phase completed:** YYYY-MM-DD
   **Total commits:** 5
   

3. Announce phase completion:

   Copy"Phase 1: Core Foundation is complete.
   
   Updated documents:
   - tasks.md: All Phase 1 tasks marked complete
   - plan.md: Phase 1 marked as COMPLETE
   
   Moving to Phase 2..."
   

5.5 Update Completion Tracking Table

CRITICAL: Update the Completion Tracking table at the bottom of tasks.md!

This table provides a summary view of progress across all phases:

Copy## Completion Tracking

| Phase | Tasks | Completed | Status |
|-------|-------|-----------|--------|
| Phase 1 | 3 | 2 | IN_PROGRESS |    ← Update Completed count
| Phase 2 | 6 | 0 | PENDING |
| Phase 3 | 5 | 0 | PENDING |
| **Total** | **14** | **2** | **14%** |  ← Update Total and percentage

After completing each task:

1. Increment the "Completed" count for the current phase
2. Update the phase "Status" (PENDING → IN_PROGRESS → COMPLETE)
3. Update the "Total" row's "Completed" count
4. Recalculate and update the percentage

Phase Status values:

• PENDING - No tasks started
• IN_PROGRESS - At least one task completed, not all done
• COMPLETE - All tasks in phase completed

5.6 Document Update Checklist

Before proceeding to next task, verify:

• tasks.md: Task marked [x] with completion date
• tasks.md: Status counts updated (top section)
• tasks.md: Completion Tracking table updated (bottom section)
• plan.md: Task marked with ✅ COMPLETE (optional)
• If phase complete: Phase marked as complete in both files
• Announced completion status to user

Never proceed to next task without updating documents!

5.7 Phase Exit Protocol (When Phase Complete)

🔴 WHEN ALL TASKS IN CURRENT PHASE ARE COMPLETE, FOLLOW EXIT PROTOCOL:

Copy┌─────────────────────────────────────────────────────────────────┐
│ PHASE EXIT PROTOCOL                                              │
├─────────────────────────────────────────────────────────────────┤
│                                                                   │
│ 1. UPDATE tasks.md (already done in 5.1-5.5)                     │
│                                                                   │
│ 2. GIT COMMIT                                                    │
│    → git add [changed files]                                     │
│    → git commit -m "feat([cap]): complete Phase N"               │
│                                                                   │
│ 3. CHECK - More phases remaining?                                │
│    → Read tasks.md to check                                      │
│    → If NO more phases: Go to Step 6 (Final Review)              │
│    → If YES more phases: Continue to step 4                      │
│                                                                   │
│ 4. RE-READ PHASE-PROTOCOL SKILL ← 🔴 CRITICAL!                   │
│    → Read: skills/phase-protocol/SKILL.md                        │
│    → This refreshes your context for next phase                  │
│                                                                   │
│ 5. CREATE NEXT PHASE TODO                                        │
│    → Follow Entry Protocol for Phase N+1                         │
│    → Use TODO template from phase-protocol skill                 │
│    → Loop back to implementation                                 │
│                                                                   │
└─────────────────────────────────────────────────────────────────┘

Announce phase transition:

Copy---
## Phase N Complete ✅

**Completed tasks:**
- [x] Task 1
- [x] Task 2

**Commit:** [sha] - [message]

**Documents updated:**
- tasks.md: Phase N tasks marked complete
- Completion Tracking: [X]% → [Y]%

---

## Transitioning to Phase N+1

Re-reading phase-protocol skill to refresh context...
[Execute Entry Protocol for Phase N+1]
---

Step 6: Final Review

After ALL tasks complete:

Dispatch Final Reviewer

CopyYou are performing a FINAL CODE REVIEW of the entire implementation.

**Change:** [change-id]

**Specs:**
- superspec/changes/[id]/specs/[cap1]/spec.md
- superspec/changes/[id]/specs/[cap2]/spec.md

**All Commits:**
[Git log since branch start]

**Review Focus:**

1. **Cross-Task Consistency**
   - Do components work together?
   - Any conflicting implementations?
   - Shared code extracted appropriately?

2. **Complete Spec Coverage**
   - Every Requirement implemented?
   - Every Scenario tested?
   - No gaps between tasks?

3. **Integration**
   - Components integrate correctly?
   - No orphaned code?
   - APIs consistent?

4. **Documentation**
   - Code comments where needed?
   - API documentation?
   - README updates?

**Output Format:**

Final Code Review

Overall Assessment

[Summary of implementation quality]

Spec Coverage

• Requirement 1: [✅/❌]
• Requirement 2: [✅/❌] ...

Cross-Cutting Issues

1. [Issue if any]

Verdict

[✅ READY TO MERGE / ❌ NEEDS WORK]

Copy

Red Flags - NEVER Do

Example Flow

CopyYou: I'm using Subagent-Driven Development for add-2fa change.

[Read plan: superspec/changes/add-2fa/plan.md]
[Extract 5 tasks with full text and Spec references]
[Create TodoWrite with all tasks]

--- Task 1: 2FA Setup - Generate QR Code ---

[Dispatch Implementer with Task 1 + Spec reference]

Implementer: Implementing now using TDD...
- Wrote test for Scenario: Generate setup QR code
- Test failed as expected (function not defined)
- Implemented generate2FASetup()
- Test passes
- Committed with Spec reference

[Dispatch Spec Reviewer]

Spec Reviewer:
## Spec Compliance Review
### Requirement: 2FA Setup - ✅ COMPLIANT
### Scenario: Generate setup QR code
- Test exists: ✅
- WHEN (user requests setup): ✅
- THEN (generates secret): ✅
- AND (displays QR): ✅
- AND (displays manual code): ✅
### Verdict: ✅ SPEC COMPLIANT

[Dispatch Quality Reviewer]

Quality Reviewer:
## Code Quality Review
### Strengths
- Clean TDD implementation
- Good error handling
### Issues
#### Important
1. Consider validating userId input
### Verdict: ✅ APPROVED

[Mark Task 1 complete + UPDATE DOCUMENTS]

You: Task 1 complete. Updating documents...

**Updated tasks.md:**
- [x] 1.1 2FA Setup - Generate QR Code ✓ (2024-01-18)

**Updated tasks.md Status:**
- Completed: 1
- In Progress: 0
- Pending: 4

**Updated Completion Tracking table:**
| Phase | Tasks | Completed | Status |
|-------|-------|-----------|--------|
| Phase 1 | 5 | 1 | IN_PROGRESS |
| **Total** | **5** | **1** | **20%** |

**Updated plan.md:**
## Task 1: 2FA Setup - Generate QR Code ✅ COMPLETE

--- Task 2: 2FA Setup - Verify Code ---
[Continue pattern...]

--- After All Tasks ---

[Dispatch Final Reviewer]

Final Reviewer:
## Final Code Review
### Spec Coverage: All requirements implemented ✅
### Cross-Cutting: No issues
### Verdict: ✅ READY TO MERGE

[Use superspec:finish-branch]

Completion

After final review passes:

Final Document Updates

1. Update tasks.md - All tasks should be marked [x]:

   Copy## Status
   - Total Tasks: 5
   - Completed: 5 ✅
   - In Progress: 0
   - Pending: 0
   
   ## All Tasks Complete!
   

2. Update plan.md - Mark all phases complete:

   Copy# [Feature] Implementation Plan ✅ COMPLETE
   
   **Completed:** YYYY-MM-DD
   **Total commits:** N
   

3. Announce completion:

"All tasks complete. Final review approved.

Documents updated:

• tasks.md: All tasks marked complete (5/5)
• plan.md: Plan marked as COMPLETE

Next step: /superspec:verify then /superspec:archive"

Related References

Prompt templates (for customization):

• implementer-prompt.md - Full implementer subagent prompt template
• spec-reviewer-prompt.md - Full spec reviewer subagent prompt template
• code-quality-reviewer-prompt.md - Full quality reviewer prompt template

Supporting skills:

• verification-before-completion - Evidence before claims
• dispatching-parallel-agents - For independent parallel tasks
• executing-plans - Alternative: batch execution with checkpoints

Integration

Required Workflow Skills

• superspec:plan - Creates the plan this skill executes
• superspec:verify - Verifies Spec-Test correspondence after completion
• superspec:finish-branch - Complete development after all tasks

Subagents MUST Use

• tdd - Subagents follow TDD for EVERY task. This is MANDATORY.

TDD Iron Law

CopyNO PRODUCTION CODE WITHOUT A FAILING TEST FIRST

If implementer cannot show failing test output BEFORE implementation:

1. Implementation is REJECTED
2. Code must be DELETED
3. Implementer restarts with proper TDD

This is NON-NEGOTIABLE. TDD discipline is the foundation of quality.