Repository navigation
[SOI] Fix Sage of Ancient Lore's back face dying as a 0/0 on transform - #16587
Merged
xenohedron merged 1 commit intoNov 4, 2026
Merged
Conversation
Both P/T characteristic-defining abilities were wrapped in a
ConditionalContinuousEffect with NotTransformedCondition. On the back face
that condition is false whenever the permanent is transformed, so Werewolf
of Ancient Hunger never got its P/T set and died to state-based actions as
a 0/0 right after transforming.
Use plain SetBasePowerToughnessSourceEffect CDAs on both faces like other
transforming cards (Hanweir Militia Captain, Invasion of Xerex). The front
face text now matches Oracle ("the number of cards in your hand"). Remove
NotTransformedCondition, which had no other usages.
Fixes #16121
Fixes #14156
Co-Authored-By: Claude Opus 5.5 <[email protected]>
This comment was marked as spam.
This comment was marked as spam.
xenohedron
approved these changes
Nov 4, 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
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.
Fixes #16121
Fixes #14156
Werewolf of Ancient Hunger, the back face of Sage of Ancient Lore (SOI), reads "Werewolf of Ancient Hunger's power and toughness are each equal to the total number of cards in all players' hands." In XMage it dies as a 0/0 as soon as Sage of Ancient Lore transforms.
Both faces wrapped their P/T characteristic-defining ability in
ConditionalContinuousEffect(..., NotTransformedCondition.instance, ...)(Mage.Sets/src/mage/cards/s/SageOfAncientLore.java, lines 39-45 and 63-69 on master).NotTransformedConditionreturns!permanent.isTransformed(), which is always false while the back face is up. So the back face's P/T was never set, and state-based actions put it into the graveyard.Changes:
new SimpleStaticAbility(Zone.ALL, new SetBasePowerToughnessSourceEffect(...)). That is the same pattern used by Westvale Cult Leader (Hanweir Militia Captain), Invasion of Xerex and Maro. The front face doesn't need the condition either: when the permanent transforms, it loses the front face's abilities, so they stop applying.CardsInAllHandsCount.getMessage()already starts with "the total number".NotTransformedCondition, as suggested in Sage of Ancient Lore - back face condition incorrectly implemented #16121. Sage of Ancient Lore was its only user.SageOfAncientLoreTest. It checks that the front face is 3/3 with three cards in hand. It also checks that after the card transforms on turn 2, Werewolf of Ancient Hunger is still on the battlefield as a 6/6 (three cards in each player's hand, counting player B's draw).This was not compiled or run locally, so the new test has not been run yet.
🤖 Generated with Claude Code