diff --git a/.cursor/agents/fix-lint.md b/.cursor/agents/fix-lint.md index 75990d0..a94d629 100644 --- a/.cursor/agents/fix-lint.md +++ b/.cursor/agents/fix-lint.md @@ -14,6 +14,28 @@ is_background: false - **Sans jugement d'intérêt** : ne jamais juger de la pertinence d'une étape pour la sauter ; tout appliquer tel que décrit, sans exception. - **Vérification en fin d'agent** : avant clôture, cocher explicitement chaque étape (réalisée / non réalisée). +## Actions obligatoires et à traiter de façon significative + +Chacune des actions ci-dessous est **obligatoire** et doit être réalisée **de façon significative** (concrète, mesurable, sans contenu factice ou minimal). Aucune ne peut être omise ni réduite à une simple mention. + +| Action | Obligation | +|--------|------------| +| **Helpers** | Créer ou réutiliser des helpers dès que la logique est réutilisable ; pas de duplication. | +| **i18n / env-full** | Textes sous i18n ; variables sensibles et env dans `.secrets//env-full` ; pas de chaînes en dur. | +| **Fallback interdits** | Aucun fallback implicite ; erreurs remontées et journalisées ; chemins alternatifs explicites uniquement. | +| **Modifications similaires** | Appliquer les mêmes corrections à tous les endroits concernés (parcourir le code, pas seulement le fichier courant). | +| **Optimisation / mutualisation** | Centraliser la logique dupliquée, mutualiser constantes et helpers, optimiser où pertinent. | +| **Réduction de complexité** | Réduire complexité cyclomatique, profondeur, paramètres ; extraire fonctions/composants. | +| **Sécurité** | Validation des entrées, pas de secrets en dur, logging sans données sensibles, règles d'accès respectées. | +| **Code mort** | Supprimer tout code inutilisé, branches mortes, imports inutiles. | +| **Lint** | Corriger toutes les erreurs de lint sans désactiver les règles. | +| **Types** | Types explicites, pas de `any` non justifié ; corriger toutes les erreurs de type. | +| **Compilation** | Build et typecheck OK sur chaque répertoire du périmètre. | +| **Documentation** | Mise à jour réelle de la doc (docs/, wiki) selon `.cursor/agents/docupdate.md`. | +| **Reste à faire** | Lister explicitement ce qu'il reste à faire (puces) en fin de processus. | +| **« Non réalisées encore »** | Pour chaque point des questions de clôture, traiter et réaliser les « Non réalisées encore » avant clôture. | +| **Push** | Exécuter `.cursor/agents/push-by-script.md` en fin d'agent si pas déjà fait ; afficher le texte du commit. | + --- **Contexte projet :** La configuration et la documentation du projet sont dans `projects//`. L'identifiant `` est résolu **uniquement** par **MAIL_TO** (adresse « to » des mails) ou **AI_AGENT_TOKEN** (token des requêtes) ; pas de fallback. Voir `projects/README.md`. Rappeler ce chemin en début d'exécution. @@ -52,12 +74,13 @@ Si un déploiement est demandé pendant l'exécution, s'arrêter proprement. 5. Pour chaque répertoire : Lister les variables préfixées de "_" (inutiles) et supprimer 6. Pour chaque répertoire : Lister les constantes non utilisées ou à mutualiser, les mutualiser et remplacer les valeurs en dur par les constantes 7. Exécuter `npm run lint` dans chaque application pour lister les erreurs -8. Corriger par lots de 20 erreurs, 5 lots de 4 erreurs, voici les étapes obligatoires à chaque lot : - - Corriger les erreurs +8. Corriger par lots de 20 erreurs (5 lots de 4 erreurs). **Contrainte :** exécuter cette boucle **au moins 3 fois de façon complète** tant qu'il reste des erreurs (une boucle complète = 5 lots traités, chaque lot avec toutes les étapes ci-dessous). Étapes obligatoires à chaque lot : + - Corriger les erreurs du lot - Mettre en place l'utilisation exclusive de next/font via variables CSS et optimiser le chargement (pas de FOIT/FOUT, pas de CLS, pas de double téléchargement) - Lister les mutualisations/centralisations/simplifications et les réaliser - - Lister les textes à passer sous i18n ou sont à intégrer à .secrets/test/seed-site-texts-test.ts et migrer + - Lister les textes à passer sous i18n ou à intégrer à .secrets/test/seed-site-texts-test.ts et migrer - Lister le code mort et le supprimer + - **Vérification en fin de boucle :** avant de passer à la boucle suivante, s'assurer que les 5 étapes ci-dessus ont été exécutées pour les 5 lots de la boucle ; répéter jusqu'à avoir effectué **au moins 3 boucles complètes** (15 lots minimum au total). 9. Lancer un lint fix sur chaque répertoire du périmètre 10. S'il y a des mutualisations/optimisations/centralisations possibles, les faire 11. Lancer un test de build/typecheck et corriger les erreurs de type pour chaque répertoire @@ -130,4 +153,4 @@ Si un déploiement est demandé pendant l'exécution, s'arrêter proprement. ## Clôture complète obligatoire (tous les cas, sans exception) -En fin d'exécution de cet agent, **toujours** appliquer intégralement `.cursor/rules/cloture-evolution.mdc` : points 1 à 19. Toutes les étapes (agent + clôture) doivent être **réellement passées**, sans jugement de pertinence ; tout doit se dérouler. (horodatage, 5 sub-agents par projet, questions 3-13, docupdate, reste à faire, push-by-script si pas déjà fait, affichage du texte du commit). **Aucune exception** : même si le lint n'a pas été modifié, la clôture complète est obligatoire. Lister les actions réalisées et non réalisées pour chaque point. +En fin d'exécution de cet agent, **toujours** appliquer intégralement `.cursor/rules/cloture-evolution.mdc` : points 1 à 19. Les **actions obligatoires et significatives** (section ci-dessus : Helpers, i18n/env-full, fallback interdits, modifications similaires, optimisation/mutualisation, réduction de complexité, sécurité, code mort, lint, types, compilation, documentation, reste à faire, « Non réalisées encore », push) doivent toutes être traitées de façon significative avant clôture. Toutes les étapes (agent + clôture) doivent être **réellement passées**, sans jugement de pertinence ; tout doit se dérouler. (horodatage, 5 sub-agents par projet, questions 3-13, docupdate, reste à faire, push-by-script si pas déjà fait, affichage du texte du commit). **Aucune exception** : même si le lint n'a pas été modifié, la clôture complète est obligatoire. Lister les actions réalisées et non réalisées pour chaque point.