Skip to content

Prefer ChainInfo activation snapshot over GetCurrentSolvingChainCard - #240

Open
Wind2009-Louse wants to merge 3 commits into
IceYGO:masterfrom
Wind2009-Louse:patch-chain
Open

Prefer ChainInfo activation snapshot over GetCurrentSolvingChainCard#240
Wind2009-Louse wants to merge 3 commits into
IceYGO:masterfrom
Wind2009-Louse:patch-chain

Conversation

@Wind2009-Louse

Copy link
Copy Markdown
Contributor

migrate(chain): 用 ChainInfo 发动快照替代 GetCurrentSolvingChainCard

Summary

  • 扩展 ChainInfo.IsActivateCode,补充 IList<int> / params int[] 重载,统一用发动快照识别连锁源卡。
  • 将各 Executor 中基于 GetCurrentSolvingChainCard / ChainInfo.IsCode(活卡)的判断,迁移为 GetCurrentSolvingChainInfo + IsActivateCode / ActivateId / ActivatePlayer / ActivateSequence
  • 无限泡影列匹配、调试日志等仍使用 RelatedCard;无连锁时选牌逻辑(Neko / MalissOCG)回退到当前 Card
  • 将误命名为 currentCardChainInfo 变量统一重命名为 currentChain

migrate(chain): prefer ChainInfo activation snapshot over GetCurrentSolvingChainCard

Summary

  • Extend ChainInfo.IsActivateCode with IList<int> / params int[] overloads so chain-source identification consistently uses the activation snapshot.
  • Migrate Executor logic that used GetCurrentSolvingChainCard / ChainInfo.IsCode (live card) to GetCurrentSolvingChainInfo plus IsActivateCode / ActivateId / ActivatePlayer / ActivateSequence.
  • Keep RelatedCard for Infinite Impermanence column matching and debug logging; keep falling back to the current Card when there is no resolving chain (Neko / MalissOCG).
  • Rename ChainInfo locals previously called currentCard to currentChain.

…cks.

* Update AI decks to utilize IsActivateCode for card activation checks, enhancing consistency and functionality across various executors.
* Refactor AI executors to replace `GetCurrentSolvingChainCard` with `GetCurrentSolvingChainInfo`, improving clarity and consistency in card activation checks across multiple decks.
@Wind2009-Louse
Wind2009-Louse marked this pull request as draft July 30, 2026 03:04
@Wind2009-Louse
Wind2009-Louse marked this pull request as ready for review July 30, 2026 03:08
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