stripstream/thoughts/reviews/api-get-cleanup-review.md

Validation Report: api-get-cleanup.md

Implementation Status

• ✓ Lot 1 (quick wins) - Majoritairement implémente
• ⚠️ Vérification automatique - Partielle (typecheck OK, lint KO)
• ⚠️ Clôture de lot - Partielle (quelques écarts vs plan)

Automated Verification Results

• ✓ Type checking passe: pnpm typecheck
• ✗ Lint échoue: pnpm lint
  • Erreur observee: Invalid project directory provided, no such directory: /Users/julienfroidefond/Sites/stripstream/lint
  • Impact: impossible de valider la qualite lint via la commande standard du repo

Code Review Findings

Matches Plan

• GET /api/preferences retire du flux client dans src/contexts/PreferencesContext.tsx (plus de fetch XHR direct).
• GET /api/komga/favorites retire du client dans src/components/layout/Sidebar.tsx et src/components/series/SeriesHeader.tsx.
• GET /api/admin/users et GET /api/admin/stats remplaces par server action getAdminDashboardData dans src/app/actions/admin.ts, consommee par src/components/admin/AdminContent.tsx.
• GET /api/komga/libraries retire du client settings via passage de donnees server-side (src/app/settings/page.tsx -> src/components/settings/ClientSettings.tsx -> src/components/settings/BackgroundSettings.tsx).

Deviations from Plan

• Lot 1 / Preferences: le plan mentionnait un fallback temporaire client pour les preferences; l'implementation supprime le fallback et repose uniquement sur les donnees server + reset logout.
  • Assessment: deviation acceptable si PreferencesService.getPreferences() reste fiable pour tous les cas authentifies.
  • Recommendation: confirmer en manuel le comportement apres login, refresh hard et reconnexion.

Potential Issues

• pnpm lint est actuellement non exploitable (script/tooling), donc la verification standard de style/regles n'est pas couverte.
• La logique optimiste favoris en sidebar repose sur l'evenement favoritesChanged enrichi; le fallback router.refresh() couvre le cas sans detail, mais doit etre teste en navigation reelle.

Manual Testing Required

1. Favoris (series + sidebar)

   • Ajouter une serie en favori depuis la page serie, verifier apparition immediate en sidebar.
   • Retirer une serie des favoris, verifier disparition immediate.
   • Recharger la page, verifier persistance exacte des favoris.

2. Preferences

   • Modifier une preference (display/background), verifier persistance apres reload.
   • Se deconnecter/reconnecter, verifier reset puis rechargement correct des preferences.

3. Admin

   • Ouvrir admin, verifier affichage users/stats initiaux.
   • Cliquer "Rafraichir", verifier mise a jour sans appel XHR /api/admin/*.

4. Settings libraries

   • Ouvrir settings/display/background, verifier chargement bibliotheques Komga.
   • Basculer sur fond komga-random et verifier la liste des bibliotheques.

Recommendations

• Corriger la commande lint du projet pour retablir la verification automatique complete.
• Ajouter (ou mettre a jour) un test d'integration pour le flux favoris optimiste (event + rerender sidebar).
• En lot 2, conserver le meme principe server-first pour les routes paginees restantes.