Skip to content

feat(parser): legend rule exemption for commanders you control#3430

Open
kiannidev wants to merge 2 commits into
phase-rs:mainfrom
kiannidev:feat/issue-3423-legend-commanders
Open

feat(parser): legend rule exemption for commanders you control#3430
kiannidev wants to merge 2 commits into
phase-rs:mainfrom
kiannidev:feat/issue-3423-legend-commanders

Conversation

@kiannidev

Copy link
Copy Markdown
Contributor

Summary

Closes #3423. Parses "The \"legend rule\" doesn't apply to commanders you control." into LegendRuleDoesntApply with an IsCommander controller-scoped filter.

Changes

  • parse_legend_rule_scope "commanders" arm
  • Parser unit test; defers Brothers Yamazaki conditional form

Implementation method (required)

  • Not /engine-implementer — parser-only scope extension

CR references

CR 704.5j, CR 903.3

Verification

  • cargo test -p engine static_legend_rule_commanders --lib

Made with Cursor

Closes phase-rs#3423. Parser + unit tests for commander-scoped legend-rule exemption.

Co-authored-by: Cursor <cursoragent@cursor.com>
@kiannidev kiannidev requested a review from matthewevans as a code owner June 16, 2026 00:01
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Drop cross-PR opponent-search and kicked-spell tests that caused compile
and runtime failures on this branch.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

feat(parser): legend rule exemption for commanders you control

1 participant