[Card Bug] Dynamic quantities based on earlier events this turn are swallowed

[PBernaerts]

Card(s): Moraug, Fury of Akoum; Thousand-Year Storm; Fists of Flame

Build/version: Current preview coverage, card_data_hash=f0a6b4b86732b9a2

Game mode: Unknown

Actual behavior: Each card has one Swallow:DynamicQty gap for a quantity derived from prior events in the current turn:

• Moraug: times each creature has attacked this turn
• Thousand-Year Storm: other instant and sorcery spells cast before the triggering spell this turn
• Fists of Flame: cards drawn this turn

No runtime behavior was tested.

Expected behavior: These quantities should remain dynamic and read the relevant per-object or per-player turn history when the effect resolves, rather than being discarded or collapsed to a fixed value.

Steps to reproduce: Inspect the three cards in the current preview coverage export. Each is supported: false, has gap_count: 1, and identifies the affected clause as Swallow:DynamicQty.

[mike-theDude]

Triage analysis

Clarified summary: Three supported-looking ASTs collapse quantities derived from earlier events in the current turn to a fixed single application.

Expected behavior: Each effect resolves its printed turn-history quantity: attacks by each affected creature, prior instant/sorcery casts by the player, or cards drawn by the player.

Observed behavior: The current parse details claim the clauses are supported but encode fixed +1/+0 or a single CopySpell, omitting the dynamic multiplier. The cards remain globally unsupported in the preview export.

Oracle text (verified from client/public/card-data.json):

• Moraug, Fury of Akoum: “Each creature you control gets +1/+0 for each time it has attacked this turn.”
• Thousand-Year Storm: “Whenever you cast an instant or sorcery spell, copy it for each other instant and sorcery spell you've cast before it this turn. You may choose new targets for the copies.”
• Fists of Flame: “Draw a card. Until end of turn, target creature gains trample and gets +1/+0 for each card you've drawn this turn.”

Likely scope: Dynamic quantity parsing and engine turn-history queries, including per-object and per-player event counts.

Acceptance criteria: Parse details preserve each quantity source, and runtime tests demonstrate distinct counts greater than one without conflating object-specific and player-wide history.

[mike-theDude]

Classifier analysis

Verdict: supported_aspect_defect

Matched clause: The reported turn-history quantity clause on Moraug, Thousand-Year Storm, and Fists of Flame (supported: true)

Reasoning: The ASTs are unfaithful despite their supported nodes: Moraug and Fists encode a fixed +1/+0, and Thousand-Year Storm encodes one CopySpell; none preserves the printed per-event count.

---

Posted by bug-coverage-classifier skill.

[matthewevans]

Fixed by merged PR #7279 (4914035). Evidence: QuantityRef::AttackedThisTurn recipient binding for Moraug; SpellsCastBeforeTriggeringSpell parser/runtime resolution for Thousand-Year Storm; and bare cards drawn this turn parsing to CardsDrawnThisTurn for Fists of Flame. The merged change includes parser and runtime regressions for each semantic carrier.