[Plugin] astrbot_plugin_london_gold_alert

[Crystar-1]

插件信息

{
  "name": "astrbot_plugin_london_gold_alert",
  "display_name": "iTick伦敦金价监控",
  "desc": "伦敦金实时价格监控与到价提醒插件。基于iTick专业金融API，支持REST和WebSocket模式获取实时金价，设置价格提醒后自动通知。",
  "author": "Crystar-1",
  "repo": "https://github.com/Crystar-1/astrbot_plugin_london_gold_alert"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_london_gold_alert

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

main.py

1. 权限校验逻辑重复且行为不一致

   • main.py 里 cmd_* 先调用 _check_whitelist/_check_admin，commands.py 里又有 @check_whitelist/@check_admin 二次校验。
   • 结果是：在 main.py 被拦截时直接 return，用户收不到“无权限”提示；而 commands.py 的友好提示永远触发不到（被前置 return 吃掉）。

2. 管理员“默认放行”存在安全风险

   • _check_admin 在未配置 admin_qq_list 时返回 True，等于所有人可执行管理命令（如停止/重启监控、删除他人提醒）。
   • 这属于高风险“安全默认值”问题。

3. 资源关闭重复

   • terminate() 中先 await self.monitor.stop()，而 PriceMonitor.stop() 内已 await self.api.close()；随后 terminate() 又再次 await self.api.close()。
   • 虽然通常不会崩，但属于重复关闭、职责边界不清，易引发后续维护问题。

4. 消息发送参数缺乏输入约束

   • _send_message() 对 session_id/user_id 直接 int()，若来源异常会抛 ValueError（虽被捕获，但表现为泛化失败）。
   • 建议前置校验并记录结构化上下文（session/user/is_group）以便排障。

---

api.py

1. 异步回调在错误事件循环执行的风险（线程 + asyncio 混用）

   • _process_ws_price() 在 WebSocket 线程中处理异步回调：
     • 无运行循环时调用 asyncio.run(callback_result)；
     • 有循环时 asyncio.ensure_future(callback_result)。
   • 这会导致回调不一定运行在主事件循环，若回调依赖主循环上下文（Bot 事件、共享 async 资源）会出现线程安全和跨 loop 问题。

2. fetch_price() 忽略 WebSocket 模式语义

   • 即使 use_websocket=True，fetch_price() 仍固定走 REST。
   • 与“双模式支持”的设计描述不一致，可能导致“以为在用 WS，实际上仍轮询”的逻辑偏差。

3. WebSocket 线程停止存在悬挂风险

   • stop_websocket() 仅 join(timeout=5)，超时后直接将 _ws_thread = None，线程可能仍存活。
   • 后续重启/关闭时可能出现“幽灵线程”与状态不一致问题。

4. 时间新鲜度校验可能受时区/时间基准影响

   • GoldPrice.is_valid() 用 datetime.now() 与服务端时间戳直接比较，未明确时区语义。
   • 在服务端时间与本地时区/时钟偏差场景下，可能误判为过期。

---

commands.py

1. 与主入口重复做权限控制，导致维护分叉

   • 本文件装饰器与 main.py 的权限校验并存，逻辑来源分散，后续变更极易漏改。
   • 已造成“有的路径有提示、有的路径无提示”的行为不一致。

2. 无效/冗余变量

   • cmd_admin_rm() 中 removed, target_user = ...，target_user 从未使用。
   • 降低可读性并提示实现未收敛。

3. 群聊识别规则过于脆弱

   • _is_group() 通过字符串包含判断 ("GroupMessage" / "channel") 推断会话类型。
   • 对平台字段格式变化较敏感，可能误判消息场景（进而影响消息发送 API 分支）。

---

data.py

1. 反序列化边界校验不足

   • PluginData.from_dict() 默认 alerts = data.get("alerts", {}) 后直接双层迭代，未严格校验 alerts[key] 是否为 list、元素是否为 dict。
   • 遇到脏数据可能触发异常或把非法结构带入内存，影响后续逻辑稳定性。

2. 价格键使用浮点直接等值比较

   • add/remove/update/admin_remove 等均用 a.get("price") == price。
   • 浮点比较在边界情况下不稳定，可能出现“看起来同价但删不掉/判重失败”。

3. 未初始化时静默失败

   • 多个方法在 _data is None 时直接返回空或 False，无日志/异常。
   • 一旦上游忘记调用 initialize()，问题会以“功能无响应”表现，定位困难。

---

monitor.py

1. 到价判定存在边界漏触发

   • _check_cross() 条件是
     prev <= target < curr 或 prev >= target > curr
     当 curr == target（刚好到价）不会触发。
   • 对“到价提醒”语义来说这是明显边界遗漏。

2. API 关闭职责重复

   • PriceMonitor.stop() 内部关闭 api，而插件生命周期也会关闭 api。
   • 与 main.py 形成重复关闭链路，增加资源管理复杂度。

---

代码评审团

人物一：Linus Torvalds（毒舌工程流）
“你们把权限检查写了两套，还互相打架——这不是‘防御式编程’，这是‘自我对冲式编程’。把权限逻辑收敛到一个入口，行为统一（尤其是失败提示）。”

人物二：Steve Jobs（产品体验流）
“用户点了命令却没收到任何反馈，是最糟糕的交互之一。权限拒绝、参数错误、系统故障都应该给出清晰可理解的信息，而不是静默返回。”

人物三：Geralt of Rivia（猎魔人式务实风）
“我见过比怪物更麻烦的，是线程和事件循环混用。WebSocket线程里随手 asyncio.run 回调，这种‘看起来能跑’的写法，夜里最容易出事故。把异步回调统一投递回主 loop，再谈稳定。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
api.py
commands.py
data.py
monitor.py

---

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

当前仓库：Crystar-1/astrbot_plugin_london_gold_alert
剩余触发次数：2