fix(loader-server): a watch restart reuses the file list from startup

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#134 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
78/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
bun, typescript
Domaine
backend

Piste de recherche

Lisez apps/server/src/server.ts, en particulier la configuration autour de startWatch et runWorker aux lignes liées. Vérifiez comment sont calculés la liste des fichiers du dossier de build et le chemin de main.js, puis suivez la façon dont un redémarrage du watcher les transmet au worker. Le travail est terminé lorsque chaque redémarrage utilise les fichiers actuels et que l’absence de main.js ne fait pas planter le watcher.

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

Description

app:server bug
Which app is this bug report for?

server

Issue description

In watch mode, getFiles(dir) and mainPath are computed once, before startWatch (server.ts#L31-L38). runWorker reuses that same paths array on every restart (server.ts#L42-L55).

Files added after startup are missing from the files map passed to main(), and deleted files are still in it. A game that loads a new asset fails until nf start is restarted by hand.

Found by reading the code path (not reproduced).

Expected: each restart gives the worker the current files of the build folder.

Suggested fix: compute paths and mainPath inside runWorker, and handle a missing main.js there without crashing the watcher.

Found during a read-only audit of main @ 044c766 (2026-11-11).

Code sample
const paths = getFiles(dir).filter(...);   // computed once
const runWorker = async () => {
  child = fork(..., [mainPath as string, JSON.stringify(paths)]);
};
if (watch) startWatch(dir, runWorker);     // every restart reuses the same paths
Versions
  • @nanoforge-dev/loader-client / @nanoforge-dev/loader-server / @nanoforge-dev/loader-website 1.7.0 (main @ 044c766)
  • Bun 1.3.14
  • Linux x86_64
Issue priority

Medium (should be fixed soon)

I have tested this issue on a development release

044c766

Langage dominant
TypeScript
Étoiles
1
Forks
0
Merge moyen
17 h 25 min
PR mergées (30 j)
11

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/Loader

Toutes les issues de NanoForge-dev/Loader

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.