Let the AI use Fiend Artisan's tutor#11368
Open
liamiak wants to merge 1 commit into
Open
Conversation
Fiend Artisan was flagged AI:RemoveDeck:All, but the tutor AI handles it fine:
given a clean main phase it sacrifices spare fodder and fetches the best creature
it can afford. The reason it looked broken was combat.
With the flag off and no NonCombatPriority, the attack step swings the 1/1 body
for one point, tapping the very creature whose {T} tutor is worth far more, so by
main 2 it can no longer pay the cost:
| main 1 board | before | with NonCombatPriority:1 |
|---------------------|----------------------|--------------------------|
| no opposing blocker | attacks, taps, no fetch | holds back, sacs fodder, fetches |
| opposing 3/3 | held back anyway | fetches |
Add SVar:NonCombatPriority:1 so AiAttackController.shouldAttack keeps it home
unless the swing is lethal - the same treatment 53 other cards with tap abilities
already use - and drop the AI:RemoveDeck:All flag.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Agetian
approved these changes
Jul 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fiend Artisan is flagged
AI:RemoveDeck:All, but the tutor AI handles it fine — given a clean main phase it sacrifices spare fodder and fetches the best creature it can afford (Colossal Dreadmaw in my test board). The reason it looks broken is combat.With the flag off and no
NonCombatPriority, the attack step swings the 1/1 body for one damage, tapping the very creature whose{T}tutor is worth far more — so by main 2 it can't pay the cost:NonCombatPriority:1So this adds
SVar:NonCombatPriority:1and drops the flag.AiAttackController.shouldAttackalready honours that SVar, holding a creature back whenpower × priority < oppLifeand it can afford a tap-cost ability; value 1 is the most conservative setting, which 53 other cards with tap abilities already use.Full desktop suite green.
🤖 Implemented with the assistance of Claude Code (Opus).