[Plugin] astrbot_plugin_work_logger

[tilicil]

插件信息

{
  "name": "astrbot_plugin_work_logger",
  "display_name": "工作记录插件",
  "desc": "工作记录插件，支持自然语言记录工作内容，自动生成日报/周报/月报，支持定时推送图片格式工作报告",
  "author": "Tilicil",
  "repo": "https://github.com/tilicil/Astrbot_plugin_work_logger"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_work_logger

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

main.py

1. 日志模块使用不符合框架约束

   • 当前使用了 import logging + logging.getLogger("astrbot")。
   • 按要求必须且只能使用：from astrbot.api import logger。
   • 该问题在框架适配层面属于硬性不合规。

2. 删除逻辑与“会话组聚合展示”存在语义错位

   • /worklog today 使用 resolve_sessions()，会展示“组内所有会话”的合并记录。
   • 但 /worklog delete <编号> 和 delete_work_log 仅对“当前会话”删除。
   • 用户在组模式下看到的是合并视图，却按当前会话删除，容易出现“编号看起来存在但删不掉/删错预期对象”的行为不一致。

3. /worklog today 的编号体系与删除编号不稳定匹配

   • 今日展示文本由 ReportGenerator._format_daily_raw 生成，按人员分组后每组重新从 1 编号。
   • 删除接口期望的是“某一会话某一天 entry_index（1-based）”。
   • 在多人/分组场景下，展示编号和底层删除编号并非同一坐标系，存在可用性和逻辑一致性缺陷。

---

report_generator.py

1. 日志模块使用不符合框架约束

   • 使用了 import logging + logging.getLogger("astrbot")，不符合“只能从 astrbot.api 导入 logger”的要求。

2. 对存储数据结构缺乏防御性校验，存在运行时异常风险

   • 多处直接使用 entry['content']。
   • 当历史 JSON 数据损坏或结构不完整时，会触发 KeyError，影响报表生成稳定性。
   • 在异步机器人场景中，这类异常会直接转化为用户可见失败。

3. 与删除功能的编号语义冲突由该文件放大

   • _format_daily_raw 对每个发送者单独 enumerate(start=1)，导致编号重复。
   • 这与主流程“按编号删除”形成天然冲突，尤其是会话组聚合后更明显。

---

storage.py

1. 日志模块使用不符合框架约束

   • 使用 import logging + logging.getLogger("astrbot")，违反“必须从 astrbot.api 导入 logger”的框架要求。

2. 数据目录获取方式不符合框架规范

   • 当前通过 astrbot.core.utils.astrbot_path.get_astrbot_plugin_data_path + 手动拼接 + fallback 硬编码 data/plugin_data。
   • 要求是通过 from astrbot.api.star import StarTools 并调用 StarTools.get_data_dir() 获取规范目录。
   • 当前实现存在兼容性和一致性风险。

3. 并发写入存在竞态条件，可能导致日志丢失

   • add_log / delete_log 采用“读整月 JSON -> 修改 -> 回写整文件”模式。
   • 在异步并发请求下，两个协程可能基于同一旧快照写回，后写覆盖前写，造成数据丢失。
   • 该问题是实际可触发的数据一致性缺陷。

4. 持久化写入非原子，存在文件损坏窗口

   • 直接覆盖写 JSON，若写入过程中异常中断，可能产生半写入文件。
   • 后续读取会失败并回退为空，导致历史数据不可用。

5. _session_hash 截断 SHA1 到 16 hex，存在哈希碰撞导致串档风险

   • 目录名仅 64 bit 空间，长期运行理论上可碰撞。
   • 一旦碰撞，不同 session 会共享同目录，导致数据隔离失效（高影响低概率问题）。

6. 输入数据校验不足，污染数据会向上游扩散

   • _load_monthly 仅校验顶层 dict，未验证 date -> list[dict] 和关键字段。
   • 非法数据会在报表阶段触发异常或产生畸形输出。

---

代码评审团

人物一：Linus Torvalds（硬核工程派）
“最让我皱眉的是并发写 JSON 的方式：read-modify-write 没有锁，这在并发环境就是经典 data race。你不是‘可能’丢日志，是‘早晚’丢日志。先把一致性兜住，再谈功能花活。”

人物二：Steve Jobs（产品体验派）
“用户看到的编号和删除用的编号不一致，这会直接毁掉信任感。功能再多都没用，交互模型必须直觉一致：用户看到第 N 条，就该删掉第 N 条。”

人物三：Geralt of Rivia（实战主义派，虚构角色）
“怪物不只在外面，也在配置和存储细节里。日志导入方式不合规、数据目录没走标准入口，这些‘小偏差’在维护期会变成真正的诅咒。按框架契约来，能少很多血。”

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

---

🔍 审核摘要

统计信息

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

main.py
report_generator.py
storage.py

---

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

当前仓库：tilicil/Astrbot_plugin_work_logger
剩余触发次数：4

[astrbot-plugin-copybara]

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

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