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

オープン 初心者向け
  • 何か問題がありますか？
#252 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
82/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
rust
領域
backend

調査の方向性

crates/driver/src/catalog/maven.rs、特に51〜54行目付近の InstallerMavenCatalog::game_versions() を読み、crates/driver/src/drivers/neoforge/mod.rs にある LoaderRelease の説明と比較してください。重複排除されたバージョンに対して GameVersion.description がどのように設定されるか確認してください。NeoForgeのゲームバージョンにloaderの由来に関する注記が表示されなくなり、その注記が LoaderRelease に残っていれば完了です。

索引モデルが issue の本文から書いたものです。

説明

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.

主要言語
Rust
スター
11
フォーク
1
平均マージ
3時間 23分
マージ済み PR（30日）
46

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

ElectronicWave/Elemental のほかの issue

ElectronicWave/Elemental の issue をすべて見る

似ている issue

Rust の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。