/codereview

model: sonnet
description: RTK Code Review — Review locale pre-PR avec auto-fix
argument-hint: "[--fix] [file-pattern]"

RTK Code Review

Review locale de la branche courante avant création de PR. Applique les critères de qualité RTK.

**Principe**: Preview local → corriger → puis créer PR propre.

Usage

/tech:codereview              # 🔴 + 🟡 uniquement (compact)
/tech:codereview --verbose    # + points positifs + 🟢 détaillées
/tech:codereview main         # Review vs main (défaut: master)
/tech:codereview --staged     # Seulement fichiers staged
/tech:codereview --auto       # Review + fix loop
/tech:codereview --auto --max 5

Arguments: $ARGUMENTS

Étape 1: Récupérer le contexte

# Parse arguments
VERBOSE=false
AUTO_MODE=false
MAX_ITERATIONS=3
STAGED=false
BASE_BRANCH="master"

set -- "$ARGUMENTS"
while [[ $# -gt 0 ]]; do
  case "$1" in
    --verbose) VERBOSE=true; shift ;;
    --auto) AUTO_MODE=true; shift ;;
    --max) MAX_ITERATIONS="$2"; shift 2 ;;
    --staged) STAGED=true; shift ;;
    *) BASE_BRANCH="$1"; shift ;;
  esac
done

# Fichiers modifiés
git diff "$BASE_BRANCH"...HEAD --name-only

# Diff complet
git diff "$BASE_BRANCH"...HEAD

# Stats
git diff "$BASE_BRANCH"...HEAD --stat

Étape 2: Charger les guides pertinents (CONDITIONNEL)

| Si le diff contient... | Vérifier | | ------------------------------ | ------------------------------------------ | | `src/**/*.rs` | CLAUDE.md sections Error Handling + Tests | | `src/core/filter.rs` ou `src/cmds/**/*_cmd.rs` | Filter Development Checklist (CLAUDE.md) | | `src/main.rs` | Command routing + Commands enum | | `src/core/tracking.rs` | SQLite patterns + DB path config | | `src/core/config.rs` | Configuration system | | `src/hooks/init.rs` | Init patterns + hook installation | | `.github/workflows/` | CI/CD multi-platform build targets | | `tests/` ou `fixtures/` | Testing Strategy (CLAUDE.md) | | `Cargo.toml` | Dependencies + build optimizations |

Règles clés RTK

**Error Handling**:

• `anyhow::Result` pour tout le CLI (jamais `std::io::Result` nu)
• TOUJOURS `.context("description")` avec `?` — jamais `?` seul
• JAMAIS `unwrap()` en production (tests: `expect("raison")`)
• Fallback gracieux : si filter échoue → exécuter la commande brute

**Performance**:

• Pattern regex fixe et réutilisé → `LazyLock<Regex>`
• JAMAIS dépendance async (tokio, async-std) → single-threaded by design
• Startup time cible: <10ms

**Tests**:

• `#[cfg(test)] mod tests` embarqué dans chaque module
• Fixtures réelles dans `tests/fixtures/<cmd>_raw.txt`
• `count_tokens()` pour vérifier savings ≥60%
• `assert_snapshot!` (insta) pour output format

**Module**:

• `LazyLock<Regex>` pour les patterns fixes et réutilisés
• `exit_code` propagé (0 = success, non-zero = failure)
• `strip_ansi()` depuis `utils.rs` — pas re-implémenté

**Filtres**:

• Token savings ≥60% obligatoire (release blocker)
• Fallback: si filter échoue → raw command exécutée
• Pas d'output ASCII art, pas de verbose metadata inutile

Étape 3: Analyser selon critères

🔴 MUST FIX (bloquant)

• `unwrap()` en dehors des tests
• Pattern regex fixe recompilé dans une fonction
• `?` sans `.context()` — erreur sans description
• Dépendance async ajoutée (tokio, async-std, futures)
• Token savings <60% pour un nouveau filtre
• Pas de fallback vers commande brute sur échec de filtre
• `panic!()` en production (hors tests)
• Exit code non propagé sur commande sous-jacente
• Secret ou credential hardcodé
• **Tests manquants pour NOUVEAU code** :
• Nouveau `*_cmd.rs` sans `#[cfg(test)] mod tests`
• Nouveau filtre sans fixture réelle dans `tests/fixtures/`
• Nouveau filtre sans test de token savings (`count_tokens()`)

🟡 SHOULD FIX (important)

• `?` sans `.context()` dans code existant (tolerable si pattern établi)
• Pattern regex fixe et réutilisé migré vers `LazyLock<Regex>`
• Fonction >50 lignes (split recommandé)
• Nesting >3 niveaux (early returns)
• `clone()` inutile (borrow possible)
• Output format inconsistant avec les autres filtres RTK
• Test avec données synthétiques au lieu de vraie fixture
• ANSI codes non strippés dans le filtre
• `println!` en production (debug artifact)
• **Tests manquants pour code legacy modifié** :
• Fonction existante modifiée sans couverture test
• Nouveau path de code sans test correspondant

🟢 CAN SKIP (suggestions)

• Optimisations non critiques
• Refactoring de style
• Renommage perfectible mais fonctionnel
• Améliorations de documentation mineures

Étape 4: Générer le rapport

Format compact (défaut)

## 🔍 Review RTK

| 🔴  | 🟡  |
| :-: | :-: |
|  2  |  3  |

**[REQUEST CHANGES]** - unwrap() en production + regex non-lazy

---

### 🔴 Bloquant

• `git_cmd.rs:45` - `unwrap()` → `.context("...")?`

\```rust
// ❌ Avant
let hash = extract_hash(line).unwrap();
// ✅ Après
let hash = extract_hash(line).context("Failed to extract commit hash")?;
\```

• `grep_cmd.rs:12` - pattern fixe recompilé dans la fonction → `LazyLock<Regex>`

\```rust
// ❌ Avant (recompile à chaque appel)
let re = Regex::new(r"pattern").unwrap();
// ✅ Après
static RE: LazyLock<Regex> = LazyLock::new(|| Regex::new(r"pattern").unwrap());
\```

### 🟡 Important

• `filter.rs:78` - Fonction 67 lignes → split en 2
• `ls.rs:34` - clone() inutile, borrow suffit
• `new_cmd.rs` - Pas de fixture réelle dans tests/fixtures/

| Prio | Fichier     | L  | Action            |
| ---- | ----------- | -- | ----------------- |
| 🔴   | git_cmd.rs  | 45 | .context() manque |
| 🔴   | grep_cmd.rs | 12 | LazyLock          |
| 🟡   | filter.rs   | 78 | split function    |

**Mode verbose (--verbose)** — ajoute points positifs + 🟢 détaillées.

Règles anti-hallucination (CRITIQUE)

**OBLIGATOIRE avan