[EllaCoat]

概要

CI の linter を datapack-linter（DHP 3.4.19）から mcf-linter-spark v0.1.1 に替え、spark が master で出していた error 2 件と warning 1 件を直した。どの修正もゲームの動作は変えない。

主な変更

CI の付け替えとナレッジの更新、spark の診断を消す修正 2 つを入れた。

• 必須チェックの lint で検査を続けるよう、workflow の lint の step を mcf-linter-spark check に替えた
• 毎回のビルドとデータの取得を省くよう、spark の実行ファイルとバニラのデータを actions/cache で残す
• ナレッジの CI の説明が実際と合うよう、docs/knowledge/runtime-and-tools.md の linter の記述を直した
• tag から呼ぶ関数が公開範囲の外と報告されないよう、1615・0554 の 1.trigger の @within を tag の ID に合わせた
• 範囲外の数値の warning を消すよう、Object 2140 の teleport_duration を 60 から 59 にした（ゲームは 59 に丸めるので動きは同じ）

設計判断

spark の入れ方と、lint の設定の扱いを決めた。

• spark は cargo install --git で EllaCoat/mcf-linter-spark の tag v0.1.1 を入れる
• spark は .vscode/settings.json の datapack.lint.* を DHP と同じ意味で読むので、設定は変えていない

動作確認

手元の検査と、この branch の push の CI が通った。

• この branch で spark は error・warning 0 件
• 今 open の PR のうち 6 本（🐛 [Mob 301] 業深き想いの死亡時に再生中のアニメーションが止まらない問題を修正 #2208〜✨ [Artifact 1694] アクアバレットを作成 #2216）を master と merge した状態で検査し、PR 由来の診断は 0 件
• tag v0.1.1 で入れた spark で、この branch の push の lint 通過（spark のビルド 33 秒、検査 1 秒）

未対応・注意

merge の前に要る作業は無い。merge の後は検査の範囲が pack 全体に変わる。

• datapack-linter と違い毎回すべてのファイルを検査するので、master に残った診断はどの PR の lint にも出る
• TheSkyBlessing の CI の付け替え（👷 CI の linter を datapack-linter から mcf-linter-spark に替える TheSkyBlessing#2326）とは、どちらを先に merge してもよい