Skip to content

Migrate abilities from AbilityId to AbilityMechanic#230

Merged
bcollazo merged 4 commits intomainfrom
codex/migrate-ability-id
Mar 30, 2026
Merged

Migrate abilities from AbilityId to AbilityMechanic#230
bcollazo merged 4 commits intomainfrom
codex/migrate-ability-id

Conversation

@bcollazo
Copy link
Copy Markdown
Owner

@bcollazo bcollazo commented Mar 29, 2026

Summary

  • migrate remaining ability execution, hooks, validation, and move generation from AbilityId to AbilityMechanic
  • remove the AbilityId module entirely and replace old card-id checks with effect-mechanic lookups
  • add a migration regression test for all legacy ability-backed cards and rename the example migration tests

Verification

  • cargo check
  • cargo fmt --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test --features test-utils --test ability_mechanic_migration_test --test ability_mechanic_examples_test
  • rg -n "AbilityId|ability_ids|ability_id_migration" /repo

@bcollazo bcollazo merged commit 63ea282 into main Mar 30, 2026
2 checks passed
@bcollazo bcollazo deleted the codex/migrate-ability-id branch March 30, 2026 14:59
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.

1 participant