Skip to content

Read "Whenever a player casts a … spell, … that player" - #431

Merged
valzargaming merged 1 commit into
mainfrom
claude/card-coverage-gaps-sc84tj
Oct 24, 2026
Merged

valzargaming merged 1 commit into
mainfrom
claude/card-coverage-gaps-sc84tj

Conversation

@valzargaming

Copy link
Copy Markdown
Member

Requested by Val · project thread

Before: Ruric Thar's "Whenever a player casts a noncreature spell, CARDNAME deals 6 damage to that player." (and "… that player loses N life") was listed as unsupported text, because "that player" can be you.

After: it reads for creature, noncreature, artifact, enchantment, instant-or-sorcery and multicolored spells. "That player" is the spell's caster, whether that is an opponent or the permanent's own controller.

How: TextParser reads the effect as aimed at a target player, then marks it thatPlayer. For any_cast_kind, Game::trigger() resolves that player to the controller of the spell it was given. A new fixture and test cover a creature spell (nothing happens), a noncreature spell from the controller, and one from the opponent. Recognition only; the ledger's source pins are refreshed.


Generated by Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FFHHk3MFEhWxdBccQvEG7C
@valzargaming
valzargaming merged commit 79e1c88 into main Oct 24, 2026
4 checks passed
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