chore: nightly.sh points at removed packages and skips modules

Ouverte Adaptée aux débutants
#511 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
77/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
bash, node.js, typescript

Piste de recherche

Commence par les deux sections citées dans scripts/nightly.sh et lis comment le rapport de couverture nocturne découvre les paquets et gère la commande de build. Vérifie la structure actuelle de packages/ et modules/, puis exécute le script nocturne pour confirmer que le rapport inclut les modules et rend visibles les échecs du build. Le travail est terminé lorsque les références obsolètes aux paquets sont supprimées et que le rapport couvre les deux répertoires en signalant clairement les échecs du build.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug chore tests
Which package is this bug report for?

other

Issue description

The setup copies prebuilt WASM between packages/ecs-lib, ecs-client and ecs-server, which no longer exist. discover_packages scans only packages/*, so the nightly coverage report misses modules/*, the riskiest code (network, ecs). pnpm run build || true hides build failures.

Where:

Evidence:

if [[ -d "packages/ecs-lib/lib" ]]; then
...
for p in packages/*; do

Suggested fix: Delete the ecs-lib block, scan packages/* modules/*, and fail (or report) when the build fails.

Found during a code audit of main at be72076.

Code sample

No response

Versions
  • Engine main at be72076
  • pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority

Low (slightly annoying)

I have tested this issue on a development release

be72076

Langage dominant
TypeScript
Étoiles
7
Forks
2
Merge moyen
1 j 17 h
PR mergées (30 j)
17

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de NanoForge-dev/Engine

Toutes les issues de NanoForge-dev/Engine

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.