refactor: HUD描画の堅牢化とコード/ドキュメント整理#2
Open
fubira wants to merge 5 commits into
Open
Conversation
- HUD行描画のindexOf(s)起因で重複文字列が重なるバグをインデックスループで修正 - areaWidth算出をmapToInt().max().orElse(0)にし、空リスト時の例外を回避 - エリトラ色判定の変数leftを実体(消耗率)に合わせwearへ改名し早期リターン化 - getElytraLifeの-1の意図(残り耐久1で飛行不能)をコメントで明示 - 行高・HUD座標・色・速度換算・カメラ切替tick等のマジックナンバーを定数化 - fabric/neoforge両ローダーに同一の修正を適用 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- neoforge/.eclipse の Eclipse 実行設定(.launch 12ファイル)を追跡から除外 - .gitignore の */.eclipse/ で無視対象だが追跡済みだったため git rm --cached で除外 - .gitignore の実ファイルと矛盾する無意味な記述(CHANGELOG, LICENSE.txt)を削除 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- getFlightInfoString を返り値(List)に合わせ getFlightInfoLines へ改名 - ローカル変数 stringArray/flightInfoString を lines/flightInfoLines へ改名 - neoforge の new ArrayList<String>() を diamond 演算子に統一 - 未使用フィールド buildId を削除 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- WorldRenderLastEventListener を実態(エリトラ飛行監視とカメラ切替)に合わせ ElytraFlightTracker へ改名 - fabric/neoforge 両方のクラス名・ファイル名・参照箇所を更新 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- README に機能・操作(Vキー3モード)・技術スタック・ビルド手順・構成を記載 - 対応バージョンは変動するため gradle.properties 参照とし、ビルド状態はCIバッジで表示 - 古い CHANGELOG.md を削除し、変更履歴は GitHub Release(CI生成)へ一本化 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
挙動を変えない範囲でのコード品質改善・Git 管理整理・ドキュメント整備をまとめたブランチである。fabric/neoforge 両ローダーに同じ修正を適用している。
変更内容
バグ修正・堅牢化
indexOfを使っていたため、重複文字列があると同じ行に重なるバグをインデックスループで修正Optional.get()をorElse(0)にし、空リスト時の例外を回避コード品質（挙動不変）
leftを実体（消耗率）に合わせwearへ改名し早期リターン化getFlightInfoStringを返り値に合わせgetFlightInfoLinesへ改名buildIdを削除、diamond 演算子に統一WorldRenderLastEventListenerを実態に合わせElytraFlightTrackerへリネームGit 管理・ドキュメント
neoforge/.eclipseの IDE 設定を追跡から除外し、.gitignoreの無意味な記述を削除検証
各変更ごとに
./gradlew buildの成功（警告なし）を確認済み。未対応
fabric=tick 駆動 / neoforge=フレーム駆動 のイベント統一は挙動変更を伴うため本 PR には含めない。
🤖 Generated with Claude Code