Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unmoored Ego | The Brain Stone | Ancient Vendetta exiles any number of basic lands from library #13348

Open
jam736 opened this issue Feb 15, 2025 · 3 comments
Labels
bug Bugs and errors

Comments

@jam736
Copy link
Contributor

jam736 commented Feb 15, 2025

This is more of an issue with SearchTargetGraveyardHandLibraryForCardNameAndExileEffect specifically in the applySearchAndExile. If a basic land is named the count limit is correctly incremented when searching the graveyard and hand. When the library selection occurs, any number of cards can be selected with the basic land name regardless of how many were previously selected from the graveyard and hand.

Example:
init.txt file

[ancient vendetta]
@init
hand:Human:Ancient Vendetta:4
graveyard:Computer:Forest:3
hand:Computer:Forest:3
library:Computer:Forest:10

Steps to reproduce

  1. Activate the [ancient vendetta] cheat (above)
  2. Cast Ancient Vendetta targeting opponent
  3. Choose basic Forest
  4. Select 2 Forest from the graveyard
  5. Select 2 Forest from the hand
  6. Select any number of Forest from the library
  7. Look at the exiled number of cards and it will be the total cards selected
@JayDi85
Copy link
Member

JayDi85 commented Feb 15, 2025

[[Unmoored Ego]] [[ancient vendetta]]

@JayDi85 JayDi85 added the bug Bugs and errors label Feb 15, 2025
Copy link

Unmoored Ego - (Gatherer) (Scryfall) (EDHREC)

{1}{U}{B}
Sorcery
Choose a card name. Search target opponent's graveyard, hand, and library for up to four cards with that name and exile them. That player shuffles, then draws a card for each card exiled from their hand this way.

Ancient Vendetta - (Gatherer) (Scryfall) (EDHREC)

{3}{B}
Sorcery
Choose a card name. Search target opponent's graveyard, hand, and library for up to four cards with that name and exile them. Then that player shuffles.

@Grath
Copy link
Contributor

Grath commented Feb 15, 2025

The problem is that we've 1) run out of legal targets, but 2) need to show the opponent's library to the player because they're allowed to search their opponent's library and in paper they could search the graveyard, hand, and library in any order. This triggers a search with 0 maximum targets which allows the player to choose an infinite number of targets. There needs to be some refactoring done to guarantee that 0 maximum targets is "show the choices but don't allow anything to be chosen" and use MAXINT for actual infinite targets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs and errors
Projects
None yet
Development

No branches or pull requests

3 participants