fix!(lua.utils): remove dead EDITION branch from card set extractor
@S1M0N38 がすでに取り組んでいます。
2026年7月9日 から。
評価
この issue はまだ評価されていません。
説明
Problem
extract_card() in src/lua/utils/gamestate.lua (lines 316-317) has an unreachable branch:
elseif ability_set == "Edition" then
set = "EDITION"
The matching value is advertised in the Card.Set alias: ---| "EDITION" # Card with an edition (src/lua/utils/enums.lua:81).
Why it's dead
Editions (foil/holo/polychrome/negative) are modeled as card.edition and surfaced in modifier.edition — orthogonal to a card's type. ability.set is only ever Joker/Tarot/Planet/Spectral/Voucher/Booster/Default/Enhanced, never Edition, so the branch never fires.
Empirically verified — added edition cards of every type × every edition and scanned the full gamestate: 0 cards report set == "EDITION".
| Card | set |
modifier.edition |
|---|---|---|
j_joker + polychrome |
JOKER |
e_polychrome |
c_magician + negative |
TAROT |
e_negative |
H_A + foil |
DEFAULT |
e_foil |
The enums.lua comment # Card with an edition is also wrong — editions are modifiers, not a set.
Proposed fix
- Remove the
elseif ability_set == "Edition"branch fromextract_card()(gamestate.lua). - Remove the
---| "EDITION"line from theCard.Setalias (enums.lua). - Regenerate
openrpc.json.
Behavior-neutral (the branch never executes); no test coverage is affected.
- 主要言語
- Python
- スター
- 80
- フォーク
- 23
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
coder/balatrobot のほかの issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
coder/balatrobot#244 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
coder/balatrobot#243 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
coder/balatrobot#235 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
coder/balatrobot#234 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
coder/balatrobot#233 ·
coder/balatrobot の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 1 日以内に返信
-
[Bug] vllm-sr config migrate drops providers.models[].deployment対応中かも @Bevisy が今日担当しました。 オープンneeds-acceptance wg/developer-experience-ecosystem
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
vllm-project/semantic-router#5996 ·
メンテナーはふだん 2 日以内に返信
-
bugfix low-priority plugin:abstract plugin:leyline small-effort source:discussion
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
athola/claude-night-market#1132 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
WeblateOrg/language-data#2773 ·
メンテナーはふだん 1 日以内に返信
-
kind/bug priority/medium qa_automation
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
yugabyte/yugabyte-db#37324 ·
メンテナーはふだん 1 日以内に返信