TTRPG-Dev / ex_ttrpg_dev / e30f823c6043e103de397fe8fe7db1d4f8ea5385-PR-78

95%

main: 87%

Pull #78

github

fix: exclude inventory-granted equipment from active_concepts

collect_active_concepts was recursing into choices that have
grants_to = "inventory", adding the selected equipment concept to
active_concepts. This caused their system effects (e.g. armor
contributes with `when = "item.equipped"`) to be included in
active_effects without item_fields populated, crashing the evaluator
with "undefined variable item".

Equipment selected via inventory-granting choices is already handled by
inventory_effects, which attaches the correct item_fields. Skip those
choices in collect_active_concepts to avoid the double-inclusion.

26 of 26 new or added lines in 3 files covered. (100.0%)

465 of 487 relevant lines covered (95.48%)

6290.49 hits per line