fix!(lua.utils): remove dead EDITION branch from card set extractor
@S1M0N38 đang làm issue này rồi.
Từ ngày 9/7/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
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.
- Ngôn ngữ chính
- Python
- Star
- 80
- Fork
- 23
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của coder/balatrobot
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
coder/balatrobot#244 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
coder/balatrobot#243 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
coder/balatrobot#235 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
coder/balatrobot#234 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
coder/balatrobot#233 ·
Tất cả issue của coder/balatrobot
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug] vllm-sr config migrate drops providers.models[].deploymentCó thể đã có người làm @Bevisy đã nhận hôm nay. Đang mởneeds-acceptance wg/developer-experience-ecosystem
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
vllm-project/semantic-router#5996 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
bugfix low-priority plugin:abstract plugin:leyline small-effort source:discussion
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
athola/claude-night-market#1132 ·
-
Add support for Sinhala languageĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
WeblateOrg/language-data#2773 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[yugabyted] xcluster set_up and add_to_replication fail for database names containing - . or (Đang mởkind/bug priority/medium qa_automation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
yugabyte/yugabyte-db#37324 ·
Maintainer thường phản hồi trong vòng 1 ngày