[Plugin] astrbot_plugin_qzone_tools

[Wyccotccy]

插件信息

{
  "name": "astrbot_plugin_qzone_tools",
  "display_name": "为你的bot加入更多主动功能",
  "desc": "支持机器人发送定时消息，发送QQ空间，发送戳一戳等给群聊或者私聊，以及更多LLM工具",
  "author": "Wyccotccy",
  "repo": "https://github.com/Wyccotccy/astrbot_plugin_qzone_tools",
  "tags": [
    "更多主动功能"
  ],
  "social_link": "www.wyccotccy.cn"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_qzone_tools

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

main.py

1. 代码质量与编码规范 (Code Quality & PEP 8)

• 裸露的 except: 语句 (Bare Excepts): 在第 649 行、654 行（更新联系人缓存时）、第 674 行（时间字符串解析时）以及第 1018 行存在 except:。PEP 8 强烈建议至少捕获 Exception（即 except Exception as e:），裸露的 except 会捕获如 KeyboardInterrupt 或 SystemExit 等系统级异常，可能导致程序无法正常退出或隐藏严重的语法/逻辑崩溃。
• 无用/未充分利用的数据模型: 第 200 行定义了漂亮的 dataclass ScheduledCommandModel，但在 DatabaseManager 的实现中完全没有使用它，而是全程手动构建、更新和传递原生的 dict（例如第 235 行 record = {...}）。这降低了代码的类型安全性和可读性。另外，第 324 行的 ScheduledTask 并没有使用 @dataclass，存在风格不一致的情况。

2. 功能实现与逻辑正确性 (Functionality & Logic)

• UUID 截断的潜在风险: 在第 96 行和 942 行、1001 行使用了 str(uuid.uuid4())[:8] 生成任务或记忆 ID。虽然对于小型本地应用碰撞概率较低，但随着任务和记忆数量增加，8位十六进制字符存在哈希碰撞的理论风险（生日悖论）。建议直接使用完整的 UUID，或使用特定算法生成短 ID。

3. 安全漏洞与最佳实践 (Security & Best Practices)

• 代码安全性良好: 未发现 eval/exec、不安全的 pickle 反序列化或危险的系统命令执行，数据输入处理针对本地 JSON 存储相对安全。网络请求已正确使用异步库 aiohttp。

4. 可维护性与可读性 (Maintainability & Readability)

• 类的单一职责 (SRP): Main 类（继承自 Star）既要管理各类定时任务，又要处理各种缓存逻辑、客户端实例获取（_do_restore 中的鸭子类型判断等），显得十分臃肿。建议将任务调度循环 (ScheduledCommandExecutor) 和联系人缓存逻辑进一步解耦，保持主插件类的清爽。

5. 框架适应性检查 (Framework Adaptability) [重点]

• on_llm_request 参数签名违规:
  在第 820 行：

  @filter.on_llm_request()
  async def on_llm_request(self, event: AstrMessageEvent, request: Any, *args, **kwargs) -> None:

  框架严格要求：该钩子函数必须仅且只能接收三个参数。多余的 *args 和 **kwargs 可能会在未来框架调度器通过按位传参或严格反射检查时导致不可预期的异常。请修改为 async def on_llm_request(self, event: AstrMessageEvent, request: Any) -> None:（推荐将 Any 替换为正确的 ProviderRequest 类型以获取更好的类型提示）。
• StarTools.get_data_dir() 返回类型处理:
  在第 559 行：

  self.data_dir = StarTools.get_data_dir("astrbot_plugin_qzone_tools")
  os.makedirs(self.data_dir, exist_ok=True)
  self.db_manager = DatabaseManager(self.data_dir) # 内部用了 os.path.join

  需要注意，StarTools.get_data_dir() 返回的是一个 pathlib.Path 对象，并非字符串。尽管在 Python 3.10 中 os.makedirs 和 os.path.join 能够隐式兼容 PathLike 对象，但在类属性中维护一个 Path 对象却被当作 str 对待（如与其他纯字符串拼接时），极易引发 TypeError。建议显式转换：self.data_dir = str(StarTools.get_data_dir(...))。

---

代码评审团

Linus Torvalds：
我看到你代码里那些裸露的 except: 语句，就像看到有人在我的 Linux 内核里乱丢垃圾！你为什么不干脆写一个 except: pass 然后蒙上眼睛假装一切都很好？当你的时间解析器因为某种奇怪的内存错误崩溃，或者用户按下了 Ctrl+C 时，你那该死的 except: 会把这些统统吞掉，让你连死都不知道是怎么死的！至少给我加上 except Exception as e: 把它打印出来！

Steve Jobs：
伟大的产品在于细节的一致性。看看你的数据模型，你花心思设计了 ScheduledCommandModel 这个精美的 @dataclass，这很好，这很有品味。但紧接着，你在数据库管理器里完全无视了它，继续像中世纪的人一样到处拼凑粗糙的字典！这就像是我们设计了巧夺天工的 iPhone 钛金属外壳，你却非要往里面塞进一堆用胶带捆起来的诺基亚元件！统一你的设计哲学，全部用 Dataclass，让代码变得优雅起来！

GLaDOS (《传送门》人工智能)：
哦，看看这个“聪明”的决定：str(uuid.uuid4())[:8]。为了在硬盘上节省那微不足道的几字节空间，你决定手动引入哈希碰撞的风险。作为一个拥有海量记忆系统的高级人工智能，我必须说，看着那些记忆因为 ID 冲突而互相覆盖，这绝对是个绝妙的自我毁灭计划。如果你不想让你的机器人某天把“明天去杀人”和“明天去买菜”的任务搞混，我建议你把那可怜的 UUID 完整地放进去。继续测试吧，为了科学。

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

---

🔍 审核摘要

统计信息

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

main.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。评论 @astrpluginreviewer /review 来重新触发 review，请不要重复提交插件发布 issue。

当前仓库：Wyccotccy/astrbot_plugin_qzone_tools
剩余触发次数：3

[Wyccotccy]

1

[astrbot-plugin-copybara]

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

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