fix(driver): do not copy loader release descriptions into GameVersion

Aperta Adatta ai principianti
  • Qualcosa non va?
#252 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
82/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
backend

Direzione di ricerca

Leggi crates/driver/src/catalog/maven.rs, in particolare InstallerMavenCatalog::game_versions() intorno alle righe 51–54, e confrontalo con la descrizione di LoaderRelease in crates/driver/src/drivers/neoforge/mod.rs. Controlla come viene valorizzato GameVersion.description per le versioni deduplicate. Il lavoro è terminato quando le versioni di gioco NeoForge non mostrano più la nota sulla provenienza del loader, mentre la nota rimane su LoaderRelease.

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

Descrizione

Problem

For NeoForge, every GameVersion returned by DriverCatalog::game_versions() carries the loader-level provenance note:

Game version is inferred from NeoForge version naming. Installer profile remains the source of truth.

The text is defined on LoaderRelease in crates/driver/src/drivers/neoforge/mod.rs:14 and attached in release() at line 49. InstallerMavenCatalog::game_versions() then copies release.description into the deduplicated GameVersion.description in crates/driver/src/catalog/maven.rs:51-54.

Consumers that render GameVersion.description therefore show this diagnostic sentence next to every NeoForge game version. It is loader-level provenance, not a human-facing game-version description.

Reproduction

let catalog = DriverCatalog::for_driver(NEOFORGE_DRIVER)?;
for version in catalog.game_versions().await? {
    println!("{}: {:?}", version.id, version.description);
}

Every NeoForge entry reports Some("Game version is inferred from NeoForge version naming. Installer profile remains the source of truth.").

Expected

GameVersion.description should only contain a human-facing description of the game version, or be None. Loader-level provenance belongs on LoaderRelease.

Suggested fix

In InstallerMavenCatalog::game_versions, do not forward release.description into GameVersion. Set description: None, or use a dedicated game-version description source. Keep the NeoForge inference note on LoaderRelease only.

Affected version

elemental-driver 0.2.0 (crates.io), Elemental commit 62bb657.

Lingua principale
Rust
Stelle
11
Fork
1
Merge medio
3h 23m
PR unite (30g)
46

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 ElectronicWave/Elemental

Tutte le issue di ElectronicWave/Elemental

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.