Fix #4835: Intimidation Tactics exiles chosen hand card

[kiannidev]

Summary

• Fix ChangeZone hand exiles after RevealHand choice so ParentTarget cards in hand are exiled directly instead of being misrouted through dig tracked-set logic.
• Preserve Expressive Iteration dig-tail behavior by only using tracked-set library exile when the looked-at set still has library members.
• Add integration tests for Intimidation Tactics reveal, filter, and exile flow.

Test plan

• cargo test -p engine --test integration intimidation_tactics
• cargo test -p engine --lib expressive_iteration_dig_chain_reaches_library_bottom_and_exile

Fixes #4835

Made with Cursor

[gemini-code-assist]

Warning

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

[matthewevans]

Thanks for the targeted bug fix. I reviewed the backend path and the current blocker is mechanical: the Rust lint workflow is failing at cargo fmt --all -- --check on crates/engine/src/game/effects/change_zone.rs and crates/engine/tests/integration/issue_4835_intimidation_tactics.rs.

Please run cargo fmt --all and push the formatted result. I’m holding approval/enqueue until the head is formatted.

[matthewevans]

Backend review note on current head 70692210f5da0e11127756336d2e23e6f51914b1: the previous formatting blocker is addressed and I do not see a remaining backend implementation blocker. This PR is still textually conflicted with main only in the recurring integration test module list (crates/engine/tests/integration/main.rs). I resolved that mechanically in a maintainer worktree by keeping current main and adding mod issue_4835_intimidation_tactics;, but GitHub rejected my maintainer push to the fork branch, so I can’t update/approve/enqueue it from the sweep. Please merge current main and keep both module lines.

[matthewevans]

Reviewed current head a501b68. Backend fix is at the ChangeZone continuation seam, preserves the tracked-set dig-tail path, and the new integration test drives the real RevealChoice -> ChangeZone pipeline for Intimidation Tactics.

[matthewevans]

Re-approving current merge-forward head 231e343. The only post-review change is the deterministic origin/main merge resolving the integration main.rs module list after #4852 landed.