Don't search library while opponent controls Ob Nixilis, Unshackled (#8346)

[CTimmerman]

Fixes #8346.

Test result from game log (why is that most recent on top?):

Resolve stack: Imprint — Whenever a nontoken creature dies, you may exile that card. If you do, return each other card exiled with Mimic Vat to its owner's graveyard. [Zone Changer: Ob Nixilis, Unshackled (404)]
Resolve stack: Evolving Wilds (14) - Sedar searches their library for a Land.Basic card, puts it onto the battlefield tapped, then shuffles.
Resolve stack: Evolving Wilds (221) - Veldahar searches their library for a Land.Basic card, puts it onto the battlefield tapped, then shuffles.
Add to stack: Veldahar activated Evolving Wilds (221)
Add to stack: Sedar activated Evolving Wilds (14)
Add to stack: Veldahar triggered Mimic Vat (227)
Resolve stack: Murder (43) - Destroy Ob Nixilis, Unshackled (404).
Add to stack: Sedar cast Murder (43) targeting [Ob Nixilis, Unshackled (404)]

[github-actions]

This PR has not been updated in a while nad has been marked on stale. Stale PRs will be auto closed

[tool4ever]

checks are once again just too vague, e.g. this would also stop other kind of hidden mode zone changing when the cards are in play

[CTimmerman]

checks are once again just too vague, e.g. this would also stop other kind of hidden mode zone changing when the cards are in play

I did the explicit name first, then rewrote it to be generic to address #8347 (comment) and that matches only the one card last i checked, so I don't see how it's too vague now. Is it my addition of Opposition Agent effects? "ControlOpponentsSearchingLibrary" sounds like something you also don't want to trigger with a library search.

[tool4ever]

you're not considering the bigger picture which would cause negative impact: the method is in fact already named "hiddenOriginCanPlayAI" to express it handles way more variants

[CTimmerman]

So this patch isn't broad enough? What else should it cover?

[tool4ever]

It's possible to search other libraries than your own...