Skip to content

fix(m9a): 启动失败不再结束本轮,隐私开关文案对齐设置里的实际名称 - #1896

Merged
qiyinxi merged 1 commit into
AUTO-MAS-Project:devfrom
Douya36:fix/m9a-startup-and-privacy-wording
Nov 9, 2026
Merged

qiyinxi merged 1 commit into
AUTO-MAS-Project:devfrom
Douya36:fix/m9a-startup-and-privacy-wording

Conversation

@Douya36

@Douya36 Douya36 commented Nov 9, 2026

Copy link
Copy Markdown

摘要

  • 把 StartUp 从 M9A 的 abort_round_entries 里去掉,与上游口径一致(只把「切换账号」列为关键任务):启动失败改回普通任务失败、本轮照常继续,不再提前结束本轮
  • 隐私政策提示的拦停文案改成设置里的实际名字与位置:「托管Bilibili游戏隐私政策」→「设置 → 功能设置 → 神秘入口」

测试与验证

  • python -m pytest tests/task/test_m9a_login_flow.py tests/task/test_m9a_login_mode.py tests/task/test_maafw_core.py tests/tools/test_avd_shutdown.py -q → 51 passed、ruff check 通过
  • test_only_account_switch_aborts_round 断言 StartUp 不在名单、SwitchAccount 在;隐私文案用例沿用原有「未开启」断言
  • 只改 M9A 特调声明与一处提示文案,登录状态机未改,真机流程不受影响
  • 更新日志:属于 feat(m9a): 新增 MAS 原生 B 服登录,登录后交给 M9A 运行 #1882 刚合入、尚未进正式版的功能的后续修正,按 beta-only 规则不带碎片(建议 skip-changelog)

@Douya36

Douya36 commented Nov 9, 2026

Copy link
Copy Markdown
Author

CI 里的「检查更新日志碎片与版本号」是红的:这条改动属于 #1882 刚合入、尚未进正式版的功能的后续修正,按仓库规则(同一功能在公告里只出现一次)不带新碎片,麻烦帮忙加一个 skip-changelog 标签。我没有仓库标签权限(403)。

其余检查在跑;pytest / 格式检查本地都通过了(51 passed、ruff check 通过)。

jinghero commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

总体结论:可以合并

AUTO-MAS-Project/AUTO-MAS #1896

这次把 StartUp 的普通失败处理恢复到原来的策略,也把隐私开关的名字和位置对齐了,改得很克制喵~审查范围内没有确认的必须修改项。

dot酱沿着任务计划、执行结果和重试链路看了一遍:启动任务普通失败后可以继续后面的任务,但失败仍会记入本轮结果,不会偷偷报成成功;切换账号失败仍会结束本轮。登录前置检查、首任务超时和原地打转的中止保护也没有被放宽,这几处边界留住就好啦。

已读过 Windows / Ubuntu 测试日志,两平台均通过;本轮只做静态检查,没有自行运行测试或模拟器。还有一处合并前手续:目前更新日志检查仍因缺少碎片而失败,需维护者按根目录规则处理这次未发布新功能修正的 skip-changelog,等检查通过再合。别让这一项漏在门口喵。

—— dot酱拍拍爪敬上 🐾

@qiyinxi qiyinxi added the skip-changelog 本 PR 没有用户可见改动,不需要 changelog.d 碎片 label Nov 9, 2026
@qiyinxi
qiyinxi merged commit 4a0f02e into AUTO-MAS-Project:dev Nov 9, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog 本 PR 没有用户可见改动,不需要 changelog.d 碎片

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants