Skip to content

Fix auto-payment for reusable filter-land mana abilities#11131

Open
Madwand99 wants to merge 3 commits into
Card-Forge:masterfrom
Madwand99:RecognizeAvailableMana
Open

Fix auto-payment for reusable filter-land mana abilities#11131
Madwand99 wants to merge 3 commits into
Card-Forge:masterfrom
Madwand99:RecognizeAvailableMana

Conversation

@Madwand99

Copy link
Copy Markdown
Contributor

This fixes a mana-planning blind spot where Forge ignored mana abilities that had mana in their activation cost. That meant reusable filter lands like Flooded Grove were not considered by available-action detection or auto-payment, so a hand containing Rampant Growth could be treated as unplayable even with a valid line through Plains -> Flooded Grove. This also prevents auto-yield from fast-forwarding when you have viable spells to play!

Changes:

  • Allow AI/auto-payment mana discovery to consider mana-costed mana abilities when their activation costs use reusable resources, such as filter lands.
  • Prefer multi-mana producers for colored costs when generic mana remains, so a filter land can produce mana that pays both the colored requirement and leftover generic cost.

Comment thread forge-ai/src/main/java/forge/ai/ComputerUtilMana.java Outdated
Comment thread forge-ai/src/main/java/forge/ai/ComputerUtilMana.java Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants