Autotap - Add support for filter abilities

[Eradev]

Changes to auto-tap detection:

• Added support for filter abilities with generic cost (e.g. {1}: Add {R}.), and filter lands (e.g. Cascade Bluffs):
  • Doesn't chain filter;
  • Skip useless filters (no gains);
• Detection of remaining cards in hand/command zones to select which asset to use first to try and maximize plays;
• Added tests for use cases;

You can add the VM option -Dforge.debugManaPayment=true to help visualize what it attempts to use.

Let me know if there are edge cases it misses.

[tool4ever]

Hopefully this'll work better than the last AI attempt 🤔

Anyway this is huge and the highlighting part is already a separate feature: #5112

Please extract that into its own PR first, it'll certainly be useful for testing the rest of this 👍

[Eradev]

Hopefully this'll work better than the last AI attempt 🤔

Anyway this is huge and the highlighting part is already a separate feature: #5112

Please extract that into its own PR first, it'll certainly be useful for testing the rest of this 👍

Moved to #11176

[tool4ever]

after we're done with most code concerns need to remove AI:RemoveDeck:All from all applicable cards so I can batch fuzz random generated decks for crashes

[tool4ever]

this will need a cleaner solution than this massive wrap for which you're trying to ignore the code style

at the same time a ThreadLocal seems overkill and only risks wrong usage of such non obviously "constrained in concurrency" class

why not simply some overload for a nested entrypoint instead?