Taille du paquet NPM

[mquandalle]

Ce nouveau paquet est bien plus lourd que l'ancien :

Paquet	Taille
https://www.npmjs.com/package/@betagouv/aides-velo	14.3 MB
https://www.npmjs.com/package/aides-velo	597 kB

Est-ce normal ? Alors même que dans le paquet aides-velo le code était dupliqué pour ESM / CJS (et je pense qu'on n'a plus besoin de publier en CJS aujourd'hui)

• https://www.unpkg.com/browse/[email protected]/
• https://unpkg.com/browse/@betagouv/[email protected]/

[EmileRolley]

En effet c'est un sujet à traiter. Cela vient du fait que le paquet expose l'ancien fichier communes.json qui fait plus de 6 MB et qu'il est dupliqué pour ESM/CJS..

[EmileRolley]

Pourquoi considère tu qu'il n'est plus nécessaire de publier en CJS ? Car j'ai eu des problèmes avec l'utilisation du paquet côté J'agis à ce propos et il me semble que c'est bien la version CJS qui est utilisée avec nestjs..

[mquandalle]

Je sais que j'avais eu besoin de la version CJS à l'époque pour aides-jeunes qui était sur une ancienne version de webpack, je ne sais pas si c'est encore utile aujourd'hui. Normalement les bundlers modernes savent importer un module ESM.

Dans le principe c'est moins de complexité de publier juste l'ESM plutôt que 2 versions CJS et ESM. Normalement il ne devrait même plus y avoir besoin d'étape de build pour le js (juste le compile de publicodes). Un réutilisateur pourra même modifier facilement le code à la main directement dans son node-modules pour tester des changements.

[EmileRolley]

Il y a aussi un doublon lié au fait que le modèle compilé avec publicodes compile dans publicodes-build est inliné dans les fichier JS compilé mais il reste nécessaire de l'exposer afin de pouvoir utiliser le système d'import.

En supprimant communes.json du paquet on passe à 1MB, c'est déjà plus acceptable. Se pose également la question de si on souhaite garder aide-collectivities.json 🤔

[mquandalle]

Ah oui un peu dommage d'avoir les aides en double, à la fois dans un fichier dédié et dans une version inlinée. C'est tentant de supprimer complètement l'étape de build et publier les sources ESM telles quelles, avec seulement l'étape publicodes compile.

[EmileRolley]

Ah oui un peu dommage d'avoir les aides en double, à la fois dans un fichier dédié et dans une version inlinée. C'est tentant de supprimer complètement l'étape de build et publier les sources ESM telles quelles, avec seulement l'étape publicodes compile.

Oui après c'est pas le plus lourd non plus. Je vais regarder s'il y a pas moyen d'avoir un import des règles depuis publicodes-build plutôt que de le bundle, au moins pour la version ESM.

[EmileRolley]

En supprimant le fichier communes.json du build et simplifiant les données utilisées avec #7 et #9, on atteint à présent une taille de 1.2 MB. Il faudrait à présent régler le problème du doublon des règles, même si pour l'instant je ne pense pas que ce soit un problème car seules celles utilisées devraient être bundlé dans les apps l'utilisant donc c'est simplement un surplus au moment d'installer les dépendances.

[mquandalle]

Pour moi le plus simple c'est de supprimer tsup et de publier les fichiers sources sur NPM avec le publicodes transformé

[EmileRolley]

En compilant le TS avec tsc ? Il risque pas d'y avoir un soucis de compatibilité avec les projets node en commonjs ?