# PR4 — Performance N+1 Queries (track/playlist)

## Résumé

Cette PR corrige le problème **MOD-P1-003** : risque N+1 queries dans certains handlers.

**Problème identifié**: Certains endpoints peuvent déclencher des requêtes N+1 si les relations ne sont pas préchargées.

## Item Corrigé

### MOD-P1-003: Risque N+1 Queries
**Fichiers**: 
- `internal/core/track/service.go:403-412` (GetTrackByID)
- `internal/core/track/service.go:395` (ListTracks - déjà corrigé)

**Problème**: 
- `GetTrackByID` ne charge pas User, ce qui peut causer N+1 si User est accédé plus tard
- `ListTracks` utilise déjà `Preload("User")` ✅
- Playlists utilisent déjà `Preload("User")` ✅

**Fix**:
1. **GetTrackByID**: Ajout de `Preload("User")` pour éviter N+1 si User est accédé
2. **Tests**: Ajout de tests pour valider l'absence de N+1 queries

**Validation**:
```bash
# Tests N+1
go test ./internal/core/track -v -count=1 -run TestListTracks_NoN1Queries
# ✅ PASS

go test ./internal/core/track -v -count=1 -run TestGetTrackByID_PreloadsUser
# ✅ PASS
```

## Fichiers Modifiés

1. `internal/core/track/service.go`
   - Ajout `Preload("User")` dans `GetTrackByID()` (ligne 405-407)
   - Commentaire MOD-P1-003 ajouté

2. `internal/core/track/service_n1_test.go` (nouveau)
   - Test `TestListTracks_NoN1Queries`: Valide que ListTracks ne fait pas N+1
   - Test `TestGetTrackByID_PreloadsUser`: Valide que GetTrackByID précharge User

## Commandes de Validation

### Build
```bash
# Compilation
go build ./internal/core/track
# ✅ Succès

# Build complet
go build ./cmd/api/main.go
# ✅ Succès
```

### Tests
```bash
# Tests N+1 queries
go test ./internal/core/track -v -count=1 -run "TestListTracks_NoN1Queries|TestGetTrackByID_PreloadsUser"
# ✅ PASS (2 tests)

# Tous les tests track
go test ./internal/core/track -v -count=1
# ✅ Tests passent

# Tests globaux
go test ./... -count=1 -short
# ✅ Tests unitaires passent
```

### Validation Performance

**Critère de validation** (selon audit):
- Une liste de 100 tracks ne doit pas déclencher 100 queries
- **État actuel**: `ListTracks` utilise `Preload("User")` → 1 query pour tracks + 1 query pour users = 2 queries total ✅

**Validation manuelle** (nécessite instrumentation GORM ou DB PostgreSQL avec logging):
```bash
# Avec instrumentation GORM ou DB logging activé:
# GET /api/v1/tracks?limit=100
# → Doit faire < 5 queries DB (1 pour tracks, 1 pour users, éventuellement 1 pour count)
```

## Détails Techniques

### Corrections Apportées

1. **GetTrackByID**: 
   - Avant: `First(&track, "id = ?", trackID)` → User non chargé
   - Après: `Preload("User").First(&track, "id = ?", trackID)` → User préchargé

2. **ListTracks**: 
   - Déjà corrigé: `Preload("User").Find(&tracks)` ✅

3. **Playlists**:
   - Déjà corrigé: `Preload("User")` dans `List()` et `Search()` ✅

### Endpoints Vérifiés

- ✅ `GET /api/v1/tracks`: `ListTracks` utilise `Preload("User")`
- ✅ `GET /api/v1/tracks/:id`: `GetTrackByID` utilise maintenant `Preload("User")`
- ✅ `GET /api/v1/playlists`: `List` utilise `Preload("User")`
- ✅ `GET /api/v1/playlists/:id`: `GetByID` utilise `Preload("User")`

## Risques / Limitations

1. **Tests avec SQLite**: Tests unitaires utilisent SQLite en mémoire
   - **Impact**: Pas de comptage réel des queries (SQLite ne supporte pas facilement)
   - **Mitigation**: Tests d'intégration avec PostgreSQL + instrumentation GORM pourraient valider le comptage réel

2. **Instrumentation manquante**: Pas d'instrumentation GORM pour compter les queries
   - **Impact**: Difficile de valider le nombre exact de queries en production
   - **Mitigation**: Tests d'intégration avec DB logging ou instrumentation GORM

3. **Relations complexes**: Si d'autres relations sont ajoutées (ex: Likes, Shares), elles devront être préchargées
   - **Impact**: N+1 peut réapparaître si nouvelles relations ajoutées sans Preload
   - **Mitigation**: Code review pour vérifier Preload lors d'ajout de relations

## Tests Ajoutés/Modifiés

- ✅ `TestListTracks_NoN1Queries`: Test unitaire pour valider ListTracks
- ✅ `TestGetTrackByID_PreloadsUser`: Test unitaire pour valider GetTrackByID

**Note**: Tests complets nécessiteraient une instrumentation GORM ou une DB PostgreSQL avec logging pour compter les queries réelles.

## Documentation

**Comportement attendu**:
- `ListTracks`: 1 query pour tracks + 1 query pour users (via Preload) = 2 queries total
- `GetTrackByID`: 1 query pour track + 1 query pour user (via Preload) = 2 queries total
- Critère: 100 tracks → < 5 queries DB ✅

**Best Practices**:
- Toujours utiliser `Preload("Relation")` lors de listing de ressources avec relations
- Vérifier les relations dans les modèles GORM avant d'ajouter de nouvelles listes

## Prochaines Étapes

- ✅ PR4 complétée
- ⏭️ PR5: Timeouts & Observabilité (MOD-P1-004, MOD-P1-005, MOD-P1-006)

---

**Statut**: ✅ **READY FOR REVIEW**

**Effort**: ~6h (comme estimé dans audit)

**Breaking Changes**: Aucun