Skip to content

Read "Draw a card and lose N life" - #433

Merged
valzargaming merged 2 commits into
mainfrom
claude/card-coverage-gaps-sc84tj
Oct 26, 2026
Merged

valzargaming merged 2 commits into
mainfrom
claude/card-coverage-gaps-sc84tj

Conversation

@valzargaming

Copy link
Copy Markdown
Member

Requested by Val · project thread

Before: "Draw a card and lose 1 life." (with no "you") was listed as unsupported text. "You draw a card and you lose 1 life." already read.

After: the shorter wording reads too, for any number of cards and any amount of life.

How: TextParser::sentenceEffects() rewrites it to the "you" form, which the existing rules then read. A new fixture and test kill a creature that has the wording on its death trigger, then check the draw and the life loss. 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
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FFHHk3MFEhWxdBccQvEG7C
@valzargaming
valzargaming merged commit b9a3040 into main Oct 26, 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