npm run lint
npm run type-check

Constitution: 50%
Specification: 50%
---
TOTAL >= 80% = PASS

conditions_met:
  code_review_passed: true

current_phase: dev
conditions_met:
  implementation_complete: false
  code_review_passed: false

context:
  dev_feedback:
    source: code_review
    score: 65
    issues:
      - type: constitution_violation
        rule: "Naming convention"
        file: src/components/myComponent.tsx
        message: "Should be PascalCase: MyComponent.tsx"
        severity: medium

---
## ✅ Phase REVIEW-CODE terminée

**Score**: 85%
- Constitution: 90%
- Specification: 80%

**Issues mineures** (non bloquantes):
- [Liste si applicable]

**Prochaine phase**: TEST-BROWSER
---

npm run lint
npm run format:check

### Constitution Compliance

| Règle | Status | Détails |
|-------|--------|---------|
| Nommage | ✅ Pass | Conventions respectées |
| Formatage | ✅ Pass | ESLint clean |
| Architecture | ⚠️ Warning | Un composant trop gros |
| Types | ✅ Pass | Pas de `any` |
| Erreurs | ❌ Fail | Catch vide ligne 45 |

**Score**: 4/5 (80%)

### Specification Compliance

#### US-001: [User Story Title]
| Critère | Implémenté | Fichier | Ligne |
|---------|------------|---------|-------|
| Critère 1 | ✅ | src/feature.ts | 23-45 |
| Critère 2 | ✅ | src/feature.ts | 67-89 |
| Critère 3 | ❌ | - | Non trouvé |

#### US-002: [User Story Title]
...

**Score**: 8/9 critères (89%)

# Scan de dépendances
npm audit

# Scan de secrets (si outil installé)
gitleaks detect --source .

### Security Audit

| Catégorie | Status | Détails |
|-----------|--------|---------|
| Input Validation | ✅ Pass | Zod schemas utilisés |
| Auth/Authz | ✅ Pass | Middleware JWT en place |
| Data Protection | ⚠️ Warning | Email loggé (à vérifier) |
| SQL Injection | ✅ Pass | Prisma/ORM utilisé |
| XSS | ✅ Pass | React escape par défaut |
| npm audit | ✅ Pass | 0 vulnérabilités |

**Score**: 5/6 (83%)
**Vulnérabilités critiques**: 0

### Performance Review

| Aspect | Status | Détails |
|--------|--------|---------|
| DB Queries | ✅ Pass | Includes utilisés, pas de N+1 |
| Indexation | ⚠️ Warning | Index manquant sur field_x |
| Bundle Size | ✅ Pass | +12KB (acceptable) |
| Memoization | ✅ Pass | React.memo utilisé |

**Recommandations**:
- Ajouter index sur `feature_x.field_x`

Constitution Compliance: 80% × 0.25 = 20%
Specification Compliance: 89% × 0.30 = 26.7%
Security: 83% × 0.30 = 24.9%
Performance: 75% × 0.15 = 11.25%
---
TOTAL: 82.85%

SI score >= 80% ET pas de vulnérabilité critique :
  -> final_review_passed: true
  -> Continuer vers FINALIZE

SI score >= 80% MAIS warnings importants :
  -> Demander confirmation à l'utilisateur
  -> Si OK: final_review_passed: true
  -> Si refus: déterminer où retourner

SI score < 80% OU vulnérabilité critique :
  -> final_review_passed: false
  -> Analyser les issues:
     - Si problème de code: retour DEV
     - Si problème de tests: retour TEST-AUTO (fix_tests: true)
     - Si les deux: retour DEV

## Workflow Progress
- **current_phase**: review-final
- **current_step**: complete

## Conditions Met
```yaml
auto_tests_passed: true
final_review_passed: true  # <-- Changé à true

review_results:
  constitution_score: 80
  spec_score: 89
  security_score: 83
  performance_score: 75
  total_score: 82.85
  critical_issues: 0
  warnings: 2
  approved_at: {timestamp}

#### Si review échoue (retour DEV) :

```markdown
## Workflow Progress
- **current_phase**: dev  # <-- Retour en dev
- **current_step**: fix_required
- **loop_count**: {N+1}

## Conditions Met
```yaml
implementation_complete: false  # <-- Reset
code_review_passed: false
browser_tests_passed: false
auto_tests_passed: false
final_review_passed: false

dev_feedback:
  source: final_review
  fix_tests: false  # true si c'est un problème de tests
  score: 65
  issues:
    - type: constitution_violation
      rule: "Error handling"
      file: src/api/feature.ts
      line: 45
      message: "Empty catch block"
      severity: high
      fix: "Add proper error logging and handling"
    - type: security
      category: "Data Protection"
      file: src/utils/logger.ts
      message: "Email logged in plain text"
      severity: medium
      fix: "Mask email in logs"

#### Si review échoue (retour TEST-AUTO pour fix tests) :

```markdown
## Workflow Progress
- **current_phase**: test-auto  # <-- Retour aux tests
- **current_step**: fix_tests
- **loop_count**: {N+1}

## Conditions Met
```yaml
auto_tests_passed: false  # <-- Reset
final_review_passed: false

dev_feedback:
  source: final_review
  fix_tests: true  # Indique que c'est un problème de tests
  issues:
    - type: test_coverage
      message: "Coverage insuffisant sur module X"
      fix: "Ajouter tests pour les cas limites"
    - type: test_quality
      file: tests/unit/feature.test.ts
      message: "Tests trop superficiels"
      fix: "Ajouter assertions plus précises"

---

## Rapport de Review Complet

Générer un rapport markdown complet :

```markdown
# Code Review Report

> Feature: {feature_id} - {feature_description}
> Date: {timestamp}
> Reviewer: AI Review Agent

## Summary

| Metric | Score | Status |
|--------|-------|--------|
| Constitution | 80% | ✅ Pass |
| Specification | 89% | ✅ Pass |
| Security | 83% | ✅ Pass |
| Performance | 75% | ⚠️ Acceptable |
| **TOTAL** | **82.85%** | **✅ APPROVED** |

## Critical Issues
None

## Warnings
1. Index manquant sur `feature_x.field_x`
2. Email visible dans logs (non critique)

## Detailed Reports

### Constitution Compliance
[Détails...]

### Specification Compliance
[Détails...]

### Security Audit
[Détails...]

### Performance Review
[Détails...]

## Recommendations
1. Ajouter index DB avant mise en production
2. Masquer emails dans logs (prochaine itération)

## Conclusion
✅ Code approuvé pour merge avec les warnings notés.

---
## ✅ Phase REVIEW-FINAL terminée

**Score global**: 82.85%

**Résumé**:
| Catégorie | Score |
|-----------|-------|
| Constitution | 80% |
| Specification | 89% |
| Security | 83% |
| Performance | 75% |

**Issues critiques**: 0
**Warnings**: 2 (documentés)

**Rapport complet**: `.heracles/sessions/{session_id}/review-reports/final-review.md`

**Prochaine phase**: FINALIZE

L'orchestrateur peut maintenant appeler @he-workflow action=finalize.
---

---
## ❌ Phase REVIEW-FINAL - Retour en DEV requis

**Score global**: 65% (minimum: 80%)

**Issues à corriger**:
| Sévérité | Catégorie | Problème |
|----------|-----------|----------|
| HIGH | Constitution | Empty catch block |
| MEDIUM | Security | Email en clair dans logs |

**Détails envoyés** à `WORKFLOW_STATE.md > context.dev_feedback`

**Retour à**: Phase DEV (loop_count: {N+1})

L'orchestrateur doit appeler @he-dev pour corrections.
---

---
## ❌ Phase REVIEW-FINAL - Amélioration des tests requise

**Score global**: 78% (minimum: 80%)

**Problème principal**: Couverture de tests insuffisante

**Issues à corriger**:
| Type | Détail |
|------|--------|
| Coverage | Module X non couvert (45%) |
| Quality | Tests trop superficiels sur Y |

**Détails envoyés** à `WORKFLOW_STATE.md > context.dev_feedback`

**Retour à**: Phase TEST-AUTO (pour améliorer les tests)

L'orchestrateur doit appeler @he-test action=run_tests.
---