[Plugin] astrbot_plugin_phigros

[1-20182]

插件信息

{
  "name": "astrbot_plugin_phigros",
  "display_name": "phigros音游插件",
  "desc": "Phigros 音游数据查询插件，支持查询玩家存档、RKS历史、排行榜、歌曲搜索和新曲速递",
  "author": "飞翔的死猪",
  "repo": "https://github.com/1-20182/astrbot_plugin_phigros",
  "tags": [
    "音游",
    "phigros"
  ],
  "social_link": "https://github.com/1-20182"
}

插件检查清单

• 我的插件经过完整的测试
• 我的插件不包含恶意代码
• 我已阅读并同意遵守该项目的 行为准则。

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_phigros

您好！我已经对你提交的插件代码进行了初步自动化审核，作为初步参考:

main.py

数据持久化/目录规范（AstrBot 适配）

• 未使用 StarTools.get_data_dir() 获取插件数据目录：self.output_dir = Path(__file__).parent / "output" 会把运行时数据写入插件代码目录（可能只读、升级覆盖、容器挂载不一致）。UserDataManager 的 user_data.json、扫码二维码图片、缓存等都受影响。

安全性问题

• 明文持久化敏感凭据：UserDataManager 将 session_token 明文写入 user_data.json，且 taptap_login.py 还会额外写 last_token.txt。一旦目录可被其他进程/用户读取会导致账号凭据泄露。
• 文件名注入/路径穿越风险：phi_search 渲染输出文件名 f"song_{keyword}.png" 直接拼用户输入，keyword 若包含 ../、路径分隔符、冒号等，可能导致写出到 output_dir 之外或造成覆盖（取决于平台与运行权限）。同类风险也可能出现在其它以用户输入构造文件名的位置（虽然 token 截断相对安全）。
• 潜在敏感信息日志泄露：初始化时记录 api_token[:10]... 仍属于敏感信息的部分泄露（尤其当 token 较短或可被推测/撞库）。

异步/网络健壮性（异步环境）

• HTTP 请求缺少超时控制：_make_request() 使用 self.session.request(...) 未设置 timeout（也未在 ClientSession 上设置默认 ClientTimeout），网络抖动或服务端无响应会导致协程长时间挂起，拖死插件处理链路。
• 错误响应解析不健壮：无论状态码如何都 await response.json()；当服务端返回非 JSON（HTML/纯文本/空响应）时会抛异常并被统一包装成“解析响应数据失败”，导致真实错误信息丢失。并且 data.get(...) 假设 JSON 为 dict，若为 list 也会二次异常。
• ClientSession 生命周期依赖框架调用顺序：若 initialize() 未被调用或调用失败，self.session 仍为 None，后续命令触发会直接 NoneType（当前代码未做防御式检查）。

资源泄漏/逻辑正确性

• 扫码登录浏览器资源泄漏（严重）：qr_login() 使用 TapTapLoginManager.generate_qr_code() + wait_for_scan() 的组合路径，但没有在成功/失败/异常时调用 login_manager.terminate()。而 TapTapLoginManager 只有在 login() 方法的 finally 才会自动 terminate()；当前用法会导致 Chromium/Playwright 常驻，累积占用内存与句柄。
• Best30 逻辑可能错误：render_best30() 中 song_list.sort(key=lambda x: x["acc"], reverse=True) 按 accuracy 排序得到“best30”，这通常不符合音游“B30（按 rks/ptt 等评分）”的语义，可能导致展示结果与用户预期严重不符。

并发一致性/可维护性

• 用户数据读写无并发保护：UserDataManager 在多协程同时 bind/unbind/save 时可能发生覆盖/丢写（读-改-写竞争）。在群聊高并发下容易出现数据错乱。
• 配置项未真正生效：self.image_quality 在 main.py 读取但渲染器保存 PNG 时固定 quality=95，配置形同虚设。
• 无用/冗余导入降低可读性：os / asyncio / base64 / MessageEventResult / List 等在 main.py 未使用（至少在当前文件片段中未见使用），建议清理以减少维护噪音。

---

install.py

版本/规范性

• Python 版本检查与目标不一致：脚本要求 3.8+，而本次审查目标为 3.10；若此脚本面向插件用户，版本门槛与实际支持版本容易产生“看似支持但未验证”的误导（应明确声明/对齐）。

安全与最佳实践

• 直接执行 pip 安装（供应链风险/权限风险）：subprocess.run([..., "pip", "install", "-r", requirements.txt]) 会在用户环境执行包安装，若 requirements 被篡改或用户在高权限环境运行，风险较高。至少应在说明中明确“仅在受信任环境运行/建议虚拟环境”，并尽量打印将要执行的依赖信息供用户确认。

---

renderer.py

AstrBot 框架日志规范

• 使用 print() 输出错误：get_illustration() 捕获异常后 print(f"加载曲绘失败: {e}")，未接入 from astrbot.api import logger 的统一日志体系，部署后不利于定位问题、也不符合插件一致性实践（同项目其他文件已用 logger）。

异步最佳实践/性能

• CPU 密集型渲染放在 async 函数里直接执行：render_* 全是 PIL 图像处理（缩放、模糊、绘制、保存），会在事件循环线程中长时间运行，造成其它消息处理延迟。即使这不是“网络 I/O 阻塞”，在异步机器人插件中仍是明显的性能隐患（建议转线程/进程池或做请求队列限流）。

资源管理

• 图片文件句柄潜在泄漏：Image.open(file_path).convert("RGBA") 未使用上下文管理器；在部分 Pillow/平台组合下可能延迟关闭文件描述符，缓存多张曲绘时更明显。应使用 with Image.open(...) as im: 并在块内 convert()/copy()。

代码质量/可维护性

• 大量未使用导入：aiohttp / asyncio / BytesIO / Template / base64 / List 等未使用，增加依赖与阅读成本。
• 保存 PNG 时传 quality 参数无效/误导：canvas.save(output_path, "PNG", quality=95) 中 quality 通常是 JPEG 参数；对 PNG 多数实现无意义，读者会误以为质量可控。

---

taptap_login.py

AstrBot 框架日志规范

• ImportError 分支使用 print() 而非 astrbot.api.logger：Playwright 缺失时直接 print("警告: ...")，与框架日志不一致，且在机器人运行环境下可能丢失输出或难以追踪。

异步/逻辑与资源管理

• 资源释放依赖调用方：只有 login() 方法 finally: await self.terminate()，但调用方（main.py）当前绕过 login()，导致浏览器实例泄漏（该问题在 main.py 已体现为严重泄漏路径）。
• 事件循环时间 API 使用不当：asyncio.get_event_loop().time() 在异步上下文更推荐 asyncio.get_running_loop().time()；在某些运行方式下 get_event_loop() 可能触发弃用路径或返回非预期 loop。
• 裸 except: 吞异常：_wait_for_element() 中 except: 直接忽略所有异常，会掩盖选择器语法错误、页面已关闭等关键故障，导致排障困难。
• 回调可能阻塞事件循环：callback(...) 作为同步函数直接调用；如果回调做 I/O 或耗时计算，会阻塞扫码轮询协程（建议约束回调为轻量或支持 async 回调）。

---

代码评审团

人物一（Linus Torvalds，毒舌直球）：
你这个扫码登录的资源释放路径简直是在养“电子蟑螂”——Chromium 起了就不关，跑几次就能把宿主机句柄和内存吃光。把 generate_qr_code()/wait_for_scan() 这种“半流程 API”暴露出来没问题，但调用方必须 try/finally 兜底 terminate()，否则就是典型的生命周期设计失败。

人物二（Grace Hopper，偏工程与可维护性）：
把 sessionToken 明文写 user_data.json 和 last_token.txt，这是在给自己未来的事故报告提前写摘要。至少做两件事：其一是权限/目录隔离（用框架规范数据目录），其二是减少落盘面（能不落就不落，或加密/脱敏/可撤销）。工程上“能跑”不等于“能长期安全运行”。

人物三（Steve Jobs，强调体验与一致性）：
phi_search 用用户输入直接当文件名，这会让“体验”以最丑的方式崩掉：有人搜个带斜杠的关键词，你就写文件失败甚至越界覆盖。用户只会觉得“这机器人怎么这么不可靠”。输入规范化/文件名清洗这种细节，决定了插件是否看起来“精致”。

Disclaimer: 以上评审内容由 AI 自动生成，所涉及人物形象与现实无关，不代表真实人物观点。如有建议无关痛痒请忽略。

---

🔍 审核摘要

统计信息

• 仓库文件总数: 4 个 Python 文件
• 已选择审核文件: 4 / 4
  已发送至 AI 审核的文件清单

main.py
install.py
renderer.py
taptap_login.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。

[DBJD-CR]

我超，屁股肉

[astrbot-plugin-copybara]

插件信息已同步到集合仓库。
PR: AstrBotDevs/AstrBot_Plugins_Collection#250
已自动合并 PR #250

该插件已上架。
This plugin is now published.