Skip to content

[J21] Implement Skyshroud Ambush#14986

Open
muz wants to merge 1 commit into
magefree:masterfrom
muz:j21_skyshroud_ambush
Open

[J21] Implement Skyshroud Ambush#14986
muz wants to merge 1 commit into
magefree:masterfrom
muz:j21_skyshroud_ambush

Conversation

@muz
Copy link
Copy Markdown
Contributor

@muz muz commented May 31, 2026

Linked to #8105

@github-actions github-actions Bot added the cards label May 31, 2026
}
MageObjectReference controlledRef = new MageObjectReference(controlled, game);
MageObjectReference uncontrolledRef = new MageObjectReference(uncontrolled, game);
controlled.fight(uncontrolled, source, game);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't you use the standard FightTargetsEffect for the first effect, with the custom effect only for the second effect?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could be mistaken, but the problem is that I need to call processAction() and have it all run in one apply() block to correctly capture SBAs firing and being able to assess which creature(s) died, if any.

The alternative would be to force SBAs to fire before the second effect, but this feels hacky and incorrect

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

processAction is called by the engine between effects. It's not a state based action check at all though

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you need a delayed trigger rather than a reflexive trigger.
Triggers when that creature dies, while the other creature is still on battlefield, but with very short duration, needs to be discarded after first try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants