Repository navigation
fix(maafw): 修复任务收尾报错和历史记录未保存 - #1966
Conversation
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
总体结论:可以合并AUTO-MAS-Project/AUTO-MAS #1966 这两行补在了真正漏接的地方喵:MaaFW 自己装配账号任务,没有经过其他旧专项使用的 外层调度拥有整轮历史会话,manager 负责逐账号装配,账号任务负责结算和交出历史内容。现在 新增测试走真实的 manager 装配、日志提交和历史存储,检查账号完成、次数只增加一次、日志可读回以及临时目录已清理,比只检查属性赋值更能守住这条路径啦。dot酱也读取了本次 CI 日志:Windows 全量 pytest 为 2012 通过、1 跳过,Ubuntu 为 1969 通过、43 跳过,两边各有 17 个子测试通过。 本次主审与两个独立视角均为静态检查,没有独立运行项目或测试;新增回归模拟的是游戏已完成后的 MAS 收尾,不能当成真实游戏、模拟器或失败回放的端到端验证。任务结果统计继续由现有外层入口负责,不另加重复上报。 —— 摇着尾巴的 dot酱敬上 🐈 |
补齐账号任务使用的历史记录和日志回调,恢复任务收尾与历史保存。 (cherry picked from commit d157bb6)
摘要
对应 Sentry AUTO-MAS-BACKEND-188。
验证
55e83630d。新增回归在修复前因缺少history失败,修复后通过,检查账号完成、运行次数、完整历史日志、临时文件清理及重复收尾不重复保存。uv run --no-sync python -m pytest tests/task/test_maafw_stop_reservation.py tests/task/test_maafw_prefetch.py tests/task/test_maafw_run_record.py tests/task/test_history_session.py tests/task/test_legacy_callbacks.py -q:38 项通过。test_official_package_installs_and_preserves_native_interface的 teardown 等待线程结束时卡住;在后端代码仍为基线c55dcf60f的另一工作树单独执行该测试,也出现相同等待。未将其记为全量通过。真实环境补验可在配置好 MaaFW 项目、已启用账号及所需模拟器后,于
frontend运行yarn e2e:real:all --only MaaFWConfig,保留截图、HTML 报告和real-e2e-summary.json。工作树运行需将AUTO_MAS_E2E_TEMPLATE_ROOT指向主检出。