Skip to content

fix: 修复 AI 暂停、联机邀请和引擎进程清理 - #9

Merged
Barbatosishere merged 1 commit into
mainfrom
fix/audit-ai-invites-process-cleanup
Oct 20, 2026
Merged

Barbatosishere merged 1 commit into
mainfrom
fix/audit-ai-invites-process-cleanup

Conversation

@Barbatosishere

Copy link
Copy Markdown
Owner

修复退出确认框期间 AI 继续落子、迟到邀请消息关闭无关界面,以及 KataGo 启动器退出后引擎子进程残留等问题,并修正两处国际象棋终局判定。

  • 中国象棋暂停时停止 AI 调度;国际象棋先保存搜索结果,恢复后再落子,重开和退出时清理旧结果,同时保留 AI 自动升后的行为。
  • 联机邀请在收包时清理过期状态;已接受邀请的取消消息仅能关闭游戏类型和对端均匹配的当前联机界面。
  • KataGo 正常关闭、中断和训练超时统一清理启动器及后代进程,再关闭通信管道;新增 3 项真实进程树回归测试。
  • 国际象棋优先判定将死/僵局,再判断 50 回合规则;三次重复的局面键仅在有合法吃过路兵走法时计入目标格。

验证结果:

  • test build 成功:40 个套件、268 项测试,失败 0、错误 0、跳过 0,包括 5 项 GPU 压力和 8 项 OpenCL 生命周期测试。
  • 临时开发验证模组在实际 Minecraft 1.21.1 / NeoForge 21.1.241 客户端执行:76 项综合检查、18 项生命周期检查、11 项持续运行检查全部通过;持续运行共 1800 个实际 tick。
  • 专用服务器和两个实际客户端完成井字棋 20 局、19 次重开、邀请/接受及断线收尾,双端各 24 项断言全部通过;音频压力累计 120000 次请求通过。
  • git diff --check 和打包检查通过,JAR 不包含临时验证模组、测试辅助类或内嵌 JNA。

构建使用已有同版本 Minecraft 依赖,以 --offline --no-configuration-cache -x createMinecraftArtifacts 执行。首次全量运行有 1 项 OpenCL 测试超过原有 15 秒上限;未修改其限时或相关代码,独立复跑及最终全量复跑均通过。双客户端完整对局覆盖井字棋和共享联机流程;其他联机游戏特有玩法仍以现有规则/路由检查为主。

@kilo-code-bot

kilo-code-bot Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (8 files)
  • src/main/java/com/wzz/game_console/client/screens/MultiplayerLobbyScreen.java
  • src/main/java/com/wzz/game_console/client/screens/games/ChessGameScreen.java
  • src/main/java/com/wzz/game_console/client/screens/games/WesternChessScreen.java
  • src/main/java/com/wzz/game_console/client/screens/games/gogame/GoAdversarialTrainer.java
  • src/main/java/com/wzz/game_console/client/screens/games/gogame/KataGoGoAI.java
  • src/test/java/com/wzz/game_console/client/screens/games/gogome/EngineProcessFixture.java
  • src/test/java/com/wzz/game_console/client/screens/games/gogome/KataGoGoAITest.java
  • src/test/java/com/wzz/game_console/client/screens/games/gogome/KataGoProcessCleanupTest.java

Reviewed by free · Input: 75.7K · Output: 7.3K · Cached: 591.7K

@Barbatosishere
Barbatosishere merged commit f2556ef into main Oct 20, 2026
2 checks passed
@Barbatosishere
Barbatosishere deleted the fix/audit-ai-invites-process-cleanup branch October 20, 2026 05:46
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