[Plugin] astrbot_plugin_Scheduled-Task

[Marick111]

插件信息

{
  "name": "astrbot_plugin_Scheduled-Task",
  "display_name": "定时任务",
  "desc": "定时任务",
  "author": "Marick",
  "repo": "https://github.com/Marick111/astrbot_plugin_Scheduled-Task"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_Scheduled-Task

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

init.py

代码质量与编码规范

• 未使用的导入：from pathlib import Path 未使用；Event 已使用但 StarTools、logger 使用正确。
• 正则写法可读性与性能问题：多条 pattern 以 ^.* 开头，既冗余又容易造成“贪婪匹配 + 回溯”，可读性也较差；并且会让“非提醒意图”的普通对话更容易误匹配。

功能实现与逻辑正确性

• 明显的属性错误：start_check_task() 内异常分支使用了 self.logger.error(...)，但类中并没有 self.logger，会在异常发生时再次抛异常，导致检查循环不稳定（应使用已导入的 logger）。
• 任务取消处理不当：
  • check_reminders() 的 except Exception 可能会吞掉取消信号（在 Python 3.10 下 asyncio.CancelledError 属于 Exception 体系），导致插件卸载时 cancel() 后仍继续 sleep，出现卸载卡住或延迟退出的问题。
• 日期合法性未校验：calculate_reminder_time() 的 date 分支直接 now.replace(year=..., month=..., day=...)，若用户输入 2月30日 等非法日期会抛 ValueError，当前仅在 extract_reminder_info() 外层笼统捕获并记录“解析提醒时间失败”，但用户侧没有明确提示原因。
• 提醒发送条件不足可能导致“静默丢提醒”：send_reminder() 中若 platform/user_id 缺失则不会发送消息，但仍返回 True，该提醒会在 check_pending_reminders() 中被当作成功并移除，造成数据丢失。（虽然 add_reminder 会写入这两项，但历史数据/损坏数据/手工编辑 JSON 都可能触发。）

安全漏洞与最佳实践

• 在持锁状态下进行网络 I/O（并发最佳实践问题）：check_pending_reminders() 在 async with self.lock: 内部调用 await self.send_reminder(reminder)，而 send_reminder() 里 await self.bot.send_message(...) 属于网络 I/O。这样会导致：
  • 锁被长时间占用，阻塞 add_reminder() 等其它需要锁的协程；
  • 一旦网络慢/平台异常，整个提醒检查与写入都会被拖住，放大故障影响面。
• 提醒 ID 生成不稳定：id 使用 hash(task) % 10000。Python 的 hash() 默认带随机化（跨进程/重启不稳定），会导致同一任务在不同进程生命周期下生成不同 ID；且 % 10000 碰撞概率较高，ID 不能可靠用于去重/管理。

可维护性与可读性

• 职责耦合偏重：extract_reminder_info() 同时负责匹配、解析、计算时间、提取任务字段、落库与回复，分支较多（result 不同类型用不同下标取 task），后续扩展/修复容易引入回归。
• 配置与时区对象更新不同步：self.tz 在 __init__ 中根据 self.config["timezone"] 初始化一次；若插件支持热更新配置（或未来支持），self.tz 不会随配置改变而更新，行为可能不符合预期。

潜在缺陷或性能问题

• 提醒检查存在“时间漂移”：循环是“执行检查 + sleep 固定间隔”，若 check_pending_reminders() 执行耗时较长（尤其网络 I/O），下一轮检查会整体后移，导致提醒延迟累积。
• 一次性顺序发送可能拖慢：到点提醒较多时，在单协程内逐条 await 发送，会造成后面的提醒被延迟发送；并且结合“持锁网络 I/O”会进一步放大延迟与阻塞。

---

🔍 审核摘要

统计信息

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

__init__.py

---

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