fix!(lua.utils): remove dead EDITION branch from card set extractor
@S1M0N38 ci sta già lavorando.
Dal 9/7/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
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.
- Lingua principale
- Python
- Stelle
- 80
- Fork
- 23
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di coder/balatrobot
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
coder/balatrobot#244 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
coder/balatrobot#243 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
coder/balatrobot#235 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
coder/balatrobot#234 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
coder/balatrobot#233 ·
Tutte le issue di coder/balatrobot
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno
-
[Bug] vllm-sr config migrate drops providers.models[].deploymentForse già presa @Bevisy l’ha presa oggi. Apertaneeds-acceptance wg/developer-experience-ecosystem
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
vllm-project/semantic-router#5996 ·
I maintainer di solito rispondono entro 2 giorni
-
bugfix low-priority plugin:abstract plugin:leyline small-effort source:discussion
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
athola/claude-night-market#1132 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
WeblateOrg/language-data#2773 ·
I maintainer di solito rispondono entro 1 giorno
-
[yugabyted] xcluster set_up and add_to_replication fail for database names containing - . or (Apertakind/bug priority/medium qa_automation
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
yugabyte/yugabyte-db#37324 ·
I maintainer di solito rispondono entro 1 giorno