refactor(loader-client): the manifest version is read from public/version in the cwd and nothing uses it

Aperta Adatta ai principianti
  • Qualcosa non va?
#145 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
72/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
typescript
Ambito
backend, documentation

Direzione di ricerca

Leggi apps/client/src/manifest.ts ed esamina i tipi del manifest in apps/client/src/server.ts e apps/website/src/types/manifest.type.ts. Aggiorna la documentazione del client e del sito web in docs/docs/client.mdx e apps/website/README.md affinché corrisponda alla modifica scelta. Il lavoro è completato quando la logica inutilizzata della versione del manifest e i relativi riferimenti sono rimossi in modo coerente, oppure viene implementata l’alternativa documentata.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

app:client app:website enhancement refactor
Which app is this feature request for?

client

Feature

Since the per-file hashes (#8), the website never reads manifest.version. The client still computes it on every /manifest request (manifest.ts#L9-L20):

  • it reads public/version relative to the process cwd, not the game dir, so a stray file there would freeze the value;
  • otherwise it fingerprints every file hash.

This is dead logic, and the docs still describe it (client.mdx#L37, website README.md#L51).

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

Ideal solution or implementation

Remove version from the manifest and its types (apps/client/src/server.ts, apps/website/src/types/manifest.type.ts), and update the docs.

Alternative solutions or implementations

Keep version, but use it in the website as a cheap cache key before checking each file hash, and compute it from the game dir instead of the cwd.

Other context

No response

Lingua principale
TypeScript
Stelle
1
Fork
0
Merge medio
17h 25m
PR unite (30g)
11

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di NanoForge-dev/Loader

Tutte le issue di NanoForge-dev/Loader

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.