Skip to content

fix(data): types et catégories des espèces du référentiel - #34

Closed
Siruv wants to merge 1 commit into
GMS64260:mainfrom
Siruv:fix/types-categories-especes
Closed

fix(data): types et catégories des espèces du référentiel#34
Siruv wants to merge 1 commit into
GMS64260:mainfrom
Siruv:fix/types-categories-especes

Conversation

@Siruv

@Siruv Siruv commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Résumé

Corrige le classement erroné de plusieurs espèces du référentiel : 10 arbres fruitiers (Abricotier, Cerisier, Citronnier, Cognassier, Figuier, Noisetier, Oranger, Pêcher, Châtaignier, Néflier) étaient classés legume, la Vigne et la Sauge étaient mal typées, et quelques ornementales (Cosmos, Tagètes, Pyracantha) aussi.

Changements

  • especes_enriched.csv : correction de la colonne type pour les espèces concernées (source de vérité du référentiel)
  • scripts/fix-especes-types.ts : script de migration idempotent qui corrige type + categorie des bases déjà peuplées (ne touche que les espèces dont le type/categorie diffère de la valeur attendue)
  • docker-entrypoint.sh : appel du script au démarrage (bloc ajouté en fin de fichier, fallback || echo pour ne jamais bloquer le boot)
  • tsconfig.json : exclusion de scripts/ du type-check — répare la CI de main (des scripts importent @/lib/chat qui a été retiré du dépôt public)

Tests effectués (instance Docker locale)

  • Script idempotent : 1re exécution corrige les espèces, 2e exécution → « Aucune mise à jour nécessaire »
  • npx tsc --noEmit : OK
  • npm run lint:ci : dette 506 erreurs < baseline 640, aucune nouvelle erreur sur les lignes ajoutées
  • npx vitest run : uniquement les 2 échecs pré-existants sur main (growth-campaign, irrigation-recording), aucune régression
  • npm run build : OK

@GMS64260 GMS64260 closed this in a2b42b7 Aug 31, 2026
@GMS64260

Copy link
Copy Markdown
Owner

Merci, le fond du diagnostic est repris dans le commit a2b42b7 : 23 lignes du CSV sont réalignées (dont Pommier, Prunier, Olivier, Noyer, Feijoa ou Sureau, qui portaient le même défaut sans être dans ta liste), et une migration reprend les deux seules espèces encore fausses en base, Noisetier et Sauge. Trois divergences avec ta proposition, toutes mesurées sur la base de production : Cosmos et Tagètes restent en type fleur, ce sont des fleurs coupées du référentiel de production florale ajouté le 18/08, pas de l'ornement ; la Vigne reste arbre_fruitier, convention déjà en base et cohérente avec le Kiwi qui est une liane lui aussi ; et le script de correction au démarrage n'est pas retenu, parce qu'il réécrirait le référentiel à chaque boot par-dessus toute correction manuelle d'un administrateur, un motif de régression qu'on a déjà payé ici en mai. Ton nom est sur le commit (co-auteur).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants