mage icon indicating copy to clipboard operation
mage copied to clipboard

AI: Change some incorrect Outcome.Benefit to Outcome.Damage

Open Ebola16 opened this issue 1 year ago • 4 comments

Fixes #10109

For [[Rage Forger]], [[Blazing Sunsteel]] and [[Fiendlash]], make the damage effects not target self.

Ebola16 avatar Jan 16 '24 19:01 Ebola16

[[Rage Forger]], [[Blazing Sunsteel]] and [[Fiendlash]]

JayDi85 avatar Jan 16 '24 23:01 JayDi85

Rage Forger - (Gatherer) (Scryfall) (EDHREC)

{2}{R} Creature — Elemental Shaman 2/2 When Rage Forger enters the battlefield, put a +1/+1 counter on each other Shaman creature you control. Whenever a creature you control with a +1/+1 counter on it attacks, you may have that creature deal 1 damage to target player or planeswalker.

Blazing Sunsteel - (Gatherer) (Scryfall) (EDHREC)

{1}{R} Artifact — Equipment Equipped creature gets +1/+0 for each opponent you have. Whenever equipped creature is dealt damage, it deals that much damage to any target. Equip {4}

Fiendlash - (Gatherer) (Scryfall) (EDHREC)

{1}{R} Artifact — Equipment Equipped creature gets +2/+0 and has reach. Whenever equipped creature is dealt damage, it deals damage equal to its power to target player or planeswalker. Equip {2}{R}

github-actions[bot] avatar Jan 16 '24 23:01 github-actions[bot]

Is AI activates a second optional ability of Rage Forger (additional damage) with new code?

JayDi85 avatar Jan 16 '24 23:01 JayDi85

I suspect the AI will always try to use Rage Forger's 2nd ability but I believe there are many more cases where you want to damage an opponent rather than damaging yourself (previous behavior) or doing nothing.

Ebola16 avatar Jan 16 '24 23:01 Ebola16