Repository navigation
Conversation
总体结论:修改后合并宝宝,先核对 UID 再放行日常任务,这层保护有接上喵。不过拆成两次 worker 执行后,原来的任务编号和 ADB 路径没有一起传过去,会让原本能继续的流程中断,下面两处先补好。 [逻辑 · P2] 拆段后,普通任务被当成了整轮首任务以原计划 同一编号变化还会弄错历史:宿主按完整计划预建记录,第二段事件却从 0 开始;记录器按编号和名称匹配,于是追加新的 A/B/Close1999,原条目仍是“跳过”。拼接 请让分段执行保留完整计划的位置和首任务语义,事件也使用一致编号,不要直接删掉引擎原有的首任务保护。补一个 A 超时后 B 仍执行、历史每项只有一条正确状态的场景就很有针对性啦;当前新增测试替换了整个 [逻辑 · P2] UID 检查没有拿到登录时发现的 ADB 路径未手填 ADB 路径、宿主也无法从模拟器配置推导时,现有登录钩子会用 Toolkit 按地址发现路径,但只更新局部变量; 接着 UID 核验收到空路径,截图设备便退回运行 这里复用登录阶段已解析的路径就好喵,让登录、StartUp 和 UID 检查拿到同一个结果;把已有 Toolkit 路径测试延伸到 UID 检查,别让它在交接时丢掉。 以上是源码静态推演,未运行项目测试或真实 B 服流程。 —— 盯着任务交接不放的 dot酱敬上 🐾 |
AthenaHibou
left a comment
There was a problem hiding this comment.
功能实现我没侧,但是看代码逻辑这两个部分有较为明显的问题,建议处理一下
| def _account_matches(text: str, account: str) -> bool: | ||
| """兼容实机 OCR 将昵称中的「ゞ」读成「>」;仍需唯一匹配并核对游戏 UID。""" | ||
|
|
||
| return text.replace("ゞ", ">") == account.replace("ゞ", ">") | ||
|
|
||
|
|
There was a problem hiding this comment.
这一部分的代码怎么感觉像面向结果的开发,不具有泛用性,且会直接影响本机测试的结果,属于对本机测试的特调型代码,建议去掉
| description="B 服登录方式。Project:由 M9A 项目的启动任务自己登录(上游「切换账号」仍只声明官服);" | ||
| "Mas:由 MAS 完成 B 服登录——拉起 B 服客户端后处理隐私政策提示、在登录记录里选择已保存的账号," | ||
| "回到标题页后再交给 M9A 的启动任务进游戏并照常运行。Mas 需要 ADB controller 并在「游戏包名」里" | ||
| "进入游戏并核对账号配置中的 UID 后再交给 M9A 运行。Mas 需要 ADB controller 并在「游戏包名」里" |
There was a problem hiding this comment.
此处的设计是否意味着,账号切换需要与M9A功能完成,即mas侧ocr做了一半,再丢给m9a做一部分,再返还给mas继续做,代码不内聚,建议把整个启动流程的操作都收敛到mas里后在交付给m9a
|
几处要改:
小问题:
另外格式检查是红的,跑一下 |
|
本轮审查意见已处理,修订在 dd6b9a3。
本地 60 项回归测试通过,Ruff 格式、Vue 类型和多语言检查通过。B 服实机验证了本次涉及的核心功能,测试提前结束,后续任务没有完整跑完。 |
总体结论:修改后合并宝宝,单 worker、统一 ADB 路径和首项任务的超时处理都补好了,上一轮两处 P2 可以划掉啦。不过把进入游戏也移到登录阶段后,还漏了 START 到可核验主页这一段。 [逻辑 · P2] 点击 START 后只等 UID,正常加载和启动弹窗会被判成登录失败login.py:595–601 点击 START 就返回,随后立即核验 UID。这段核验固定只等 20 秒,而且只截图读 UID,不会关闭签到、公告等启动弹窗。 原来先由 StartUp 完成进游戏,再核验 UID;现在 MAS 模式不再加入 StartUp,但它原有的加载、签到和弹窗处理没有一并承接。上游 StartGame 的后续节点明确覆盖这些情况,B 服资源也沿用该流程。 静态推演:选号成功后,若 25 秒才进入主页,或弹窗遮住 UID,新代码会在启动 worker 前报“未能确认游戏内 UID”,日常任务无法开始。外层 240 秒和 请在 M9A 登录流程中补齐“进入可核验主页”的阶段,复用适用的上游启动能力或明确处理对应画面,再做连续两帧 UID 校验;只加长 UID 轮询还解决不了弹窗。补上 START 后延迟 25 秒才出现正确 UID、以及签到/公告关闭后再核验的回归场景,错误 UID 仍须阻止任务执行。以上未运行项目测试或实机复现。 —— 还在等主页真正就绪的 dot酱敬上 🐾 |
摘要
StartUp,连续两帧识别到预期 UID 后才执行剩余任务。缺填 UID、识别超时或 UID 不一致时停止本轮。验证