[Plugin] astrbot_plugin_memory

[Bluezeamer]

插件信息

{
  "name": "astrbot_plugin_memory",
  "display_name": "astrbot_plugin_memory",
  "desc": "跨会话持久化记忆插件。支持 Soul 设定、用户画像、历史对话、备忘录、会话 TODO，所有工具结果返回 LLM 推理，不打断推理循环。",
  "author": "Bluezeamer",
  "repo": "https://github.com/Bluezeamer/astrbot_plugin_memory",
  "tags": [
    "memory",
    "持久化记忆",
    "记忆控制"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_memory

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

main.py

以下仅列出发现的问题（按重要性排序）：

1. 数据持久化目录获取方式不符合框架规范

   • 问题：使用 __file__ 反推 _data_root 并拼接 plugin_data/...，未使用 StarTools.get_data_dir()。
   • 影响：在不同部署结构、打包方式或运行目录变化时，路径可能失效或偏移；也不符合 AstrBot 的统一数据目录约定。
   • 建议：改为从 astrbot.api.star 导入 StarTools，通过 StarTools.get_data_dir() 获取插件数据目录（注意返回 Path 对象，后续路径拼接建议统一用 pathlib）。

2. 全局可变状态在异步场景下存在实例污染风险

   • 问题：MEMORY_BASE / TEMPLATE_DIR / HISTORY_TEMPLATE_PATH 为模块级全局变量，在 __init__ 中赋值；大量工具函数依赖这些全局变量。
   • 影响：多实例、热重载、测试并发时可能出现状态串扰或覆盖，导致读写到错误目录。
   • 建议：将路径状态收敛到实例属性（如 self.memory_base），并将辅助函数改为实例方法或显式传参，避免隐式全局依赖。

3. ID 生成存在并发竞争，可能产生重复 ID 或覆盖写入

   • 问题：create_memory/add_memo_block 先扫描现有文件（或文本）再决定后缀，缺少原子性控制。
   • 影响：同一分钟内并发请求可能拿到相同 record_id/block_id，导致索引冲突、内容覆盖或丢失。
   • 建议：引入每用户级别的异步锁（asyncio.Lock）或使用更强唯一 ID（毫秒+随机后缀/UUID），保证写入原子性。

4. req.system_prompt += inject 缺少空值兜底

   • 问题：默认假设 req.system_prompt 一定是字符串。
   • 影响：若框架在某些链路下传入 None，会触发 TypeError。
   • 建议：改为 req.system_prompt = (req.system_prompt or "") + inject。

5. record_id 输入校验不完整

   • 问题：_sanitize_record_id 仅“过滤非法字符”，但不校验格式合法性；空字符串也会继续参与路径拼接（例如 history/.md）。
   • 影响：出现“异常但不报错”的隐蔽行为，增加排障成本。
   • 建议：对 record_id 做严格格式校验（如 ^\d{12}(?:_\d{3})?$），不合法直接返回错误。

6. 默认模板文本存在明显格式/语义破损

   • 问题：_DEFAULT_HISTORY_TEMPLATE 中出现残缺行：- content 只包含正文，必须从 \``，并且与前文“必须从## 用户指令`开始”的约束不一致。
   • 影响：LLM 工具调用参数格式易漂移，造成 create_memory 内容结构不稳定。
   • 建议：统一并修复模板规则，保证“系统提示说明”与“模板示例”一致。

---

代码评审团

人物一：Linus Torvalds（直球工程派）
“你这全局变量路径像是‘先跑起来再说’的典型写法。能跑不等于能维护。把路径状态塞回实例里，再加并发锁，不然线上一并发就开始玄学覆盖文件。”

人物二：Steve Jobs（体验与一致性控）
“产品细节决定成败。模板规则一处写‘从 ## 用户指令 开始’，另一处文本却断裂，这会让模型表现变得不一致。你需要一个‘单一真相源’，别让提示词自己打架。”

人物三：John Carmack（系统稳定性执念）
“ID 生成‘读现状再写入’在并发下不具备确定性，这是经典竞态。要么锁，要么强唯一 ID。还有 req.system_prompt 空值防护，这类边界分支不贵，但能省下大量线上排查时间。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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