[Plugin] astrbot_plugin_local_reminiscence

[olozhika]

插件信息

{
  "name": "astrbot_plugin_local_reminiscence",
  "display_name": "本地回忆[APLR]",
  "desc": "轻量级本地记忆插件，使用本地Embedding模型、本地数据库存储和回忆聊天记录。无需额外API，零Embedding成本，节约Token，完全保护隐私。使用Cron Job自动记录对话，通过关键词和语义匹配帮助AI自动回忆起相关经历和概念。初次安装时请科学上网以便快速下载依赖。",
  "author": "olozhika",
  "repo": "https://github.com/olozhika/astrbot_plugin_local_reminiscence",
  "tags": [
    "记忆",
    "回忆",
    "每日总结",
    "语义识别",
    "关键词",
    "上下文注入",
    "长期记忆"
  ],
  "social_link": "https://github.com/olozhika/"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_local_reminiscence

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

main.py

• 框架适应性检查 (数据持久化)：违反数据持久化规范。代码中在 __init__ 内使用了 Path(__file__).parent 拼接硬编码路径（如 dialog_folder、db_path 等）来存储本地记忆文件和数据库，而没有通过从 astrbot.api.star 导入 StarTools 并调用 StarTools.get_data_dir() 来获取规范的数据存储目录。
• 安全漏洞与最佳实践 (不安全的外部命令执行)：在 _daily_summary_logic 等方法中，通过 os.system(f'python "{extract_script}" "{date_str}" "{self.target_user_id}" "{self.username}"') 启动外部脚本。一方面，这在异步环境中会引起严重的同步阻塞；另一方面，只要拼接进字符串的变量（尤其是来自用户相关的内容）未被严格过滤，便存在命令注入漏洞的巨大风险。强烈建议将脚本逻辑集成进当前进程并使用 asyncio.to_thread 调度，或者使用安全的 asyncio.create_subprocess_exec。
• 潜在缺陷或问题 (异常抛出与逻辑中断)：在 on_llm_request 中，对会话历史判定时的代码 json.loads(conversation.history) == [] 缺乏错误处理。如果 conversation.history 为非法的 JSON 字符串，会直接抛出 JSONDecodeError 并且被最外层的 except Exception 捕获，从而中止剩余所有记忆注入和触发逻辑。
• 可维护性与可读性 (违反单一职责)：on_llm_request 钩子方法体积过于臃肿，它同时包含了会话检测、总结拼接、反思计算、节点检索、基于概率和关键词匹配的事件搜索、相似度排序等多种繁杂的逻辑。建议拆分为多个私有异步子函数。

init.py

未发现问题。

chat_history_extract.py

• 代码质量与编码规范 (裸异常捕获)：在 decode_json_unicode 函数中使用了 except: 捕获异常，这属于典型的 Python 反模式，因为它连 KeyboardInterrupt 和 SystemExit 等系统中断异常都会吞噬。应该修改为 except Exception:。
• 可维护性与可读性 (架构割裂)：此文件被设计为一个供 os.system 调用的独立入口脚本，导致了严重的框架环境割裂。它无法使用主框架的配置体系和路径体系（只能靠冗余的 load_config() 和写死地址来寻找数据库），使整体代码结构变得极其脆弱和难以维护。

database.py

未发现问题。

models.py

未发现问题。

summarizer.py

• 框架适应性检查 (日志记录违规)：严重违反框架日志规范。在文件开头使用了 import logging 和 logger = logging.getLogger(__name__)。必须且只能从 astrbot.api 导入 logger (即 from astrbot.api import logger)，严禁使用 Python 内部的 logging 模块。
• 潜在缺陷或问题 (JSON 剥离鲁棒性低下)：在 extract_nodes_from_events 和 generate_summary 中提取 LLM 的 JSON 输出时，代码只是通过正则剥除了头部和尾部的 ```json 标记。然而，如果大模型返回时附带了前置对话（如“好的，这是为您提取的 JSON 节点：”），这套正则会原样保留这些中文，导致后续 json.loads(content) 发生致命解析崩溃。建议使用更精准的方法来锁定 `{` 和 `}` 或者 `[` 和 `]` 之间的结构体。

vector_db.py

• 潜在缺陷或问题 (CPU 密集型计算阻塞)：利用 SentenceTransformer 进行 self.model.encode(documents, ...) 是一项非常消耗算力的纯 CPU 密集型任务。由于 add_events 和 search_events 都是同步函数，如果它们被直接通过异步事件循环中调用，模型编码的过程会彻底阻塞整个事件循环（Event Loop），导致机器人在这段时间内无法响应任何其他网络 I/O 及其它消息。建议利用 asyncio.to_thread 或者进程池来承载文本转向量的操作。

---

代码评审团

Linus Torvalds：
“你知道什么叫做‘阻塞整个事件循环’吗？！你在 vector_db.py 里肆无忌惮地直接进行本地模型推理，甚至在 main.py 里用该死的 os.system 去唤起一个外部脚本！你把异步框架当成了什么？一个可以随意停下来喝十分钟咖啡的垃圾场吗？别给整个系统踩刹车了！赶紧把你的 CPU 密集型操作扔到线程池里去，别把你的代码写得像一头陷入泥沼的猪。”

Steve Jobs：
“我对这款插件的愿景感到兴奋，让 AI 拥有长期的数字回忆，这听起来像是魔法！但是，我们看看你背后的实现好吗？为什么你的持久化路径要像拼图一样到处东拼西凑？AstrBot 框架已经为你准备好了 StarTools.get_data_dir() 这样极其优雅且统一的工具，但你偏要用硬编码的 Path(__file__).parent 把用户存储目录搞得一团糟，而且还不忘内置个原生的 logging 模块？Stay hungry, stay foolish，但不要在架构一致性上做 foolish 的事情，去把它打磨完美吧。”

Tony Stark：
“嘿，伙计，给你的 AI 装上长线记忆引擎，感觉就像是我在为 J.A.R.V.I.S 编写核心潜意识协议，这很酷。不过你的装甲防御机制就像是用保鲜膜做的。os.system 直接拼接参数变量？只要输入里有人耍点小聪明塞个分号，我就能用一行指令把你的服务器底裤都扒下来！而且你处理 JSON 的正则雷达实在是太脆弱了，只要大模型稍作几句嘘寒问暖，你的数据解析模块就会当场解体崩盘。给它加装一个自适应的内容锁定器吧，别让你的钢铁战衣因为一颗便宜的螺丝钉散架！”

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
chat_history_extract.py
database.py
models.py
summarizer.py
vector_db.py

---

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

[olozhika]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_local_reminiscence

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

main.py

1. 存在命令注入风险（高危）

   • 在 _daily_summary_logic 中使用了：

     os.system(f'python "{extract_script}" "{date_str}" "{self.target_user_id}" "{self.username}"')

   • target_user_id、username 来自配置，若包含引号或 shell 特殊字符，可导致命令拼接被劫持。

2. 异步环境中使用阻塞式系统调用

   • os.system(...) 会阻塞当前线程，放在 async 流程中会卡住事件循环，影响并发响应能力。

3. 会话历史解析缺乏局部容错，可能导致记忆注入整体失败

   • on_llm_request 中直接 json.loads(conversation.history)，若历史字段异常（非 JSON）会抛错，进入总异常分支，导致本次所有记忆注入逻辑中断。

4. N+1 查询与封装泄漏

   • _get_memory_retrieval_text 为每个事件都 with self.db._get_conn() 查询 tags：
     • 性能上是典型 N+1。
     • 直接调用 self.db._get_conn()（私有方法）破坏 MemoryDB 封装边界，可维护性较差。

5. req.system_prompt += ... 缺少空值防护

   • 若 req.system_prompt 不是字符串（如 None），会触发 TypeError，导致钩子执行失败。

---

init.py

• 未发现需要报告的问题。

---

chat_history_extract.py

1. 时间戳“串味”导致日期归档错误

   • timestamp 在每条 row_content 内部复用；当后续 turn 未提取到新时间戳时，会沿用旧值，可能把消息归到错误日期。

2. 一次性 fetchall() 读取全部会话内容，存在内存风险

   • 聊天量大时会显著抬高内存占用，可能导致提取任务不稳定。

3. 代码整洁性问题

   • os、sys 导入未使用，增加噪音并影响可维护性。

---

database.py

1. SQLite 外键约束未显式启用，级联删除可能不生效

   • 虽定义了 ON DELETE CASCADE，但未执行 PRAGMA foreign_keys = ON；SQLite 默认可能关闭外键，导致数据一致性风险。

2. INSERT OR REPLACE + tag 关系存在陈旧关联风险

   • events 使用 REPLACE 时，若外键未启用，旧 event_tags 可能残留，导致事件标签污染。

3. 类型注解不准确

   • get_event_by_id 注解为 dict，但实际会返回 None；应为 dict | None，否则误导调用方。

4. 无效导入

   • json 未使用。

---

models.py

• 未发现需要报告的问题。

---

summarizer.py

1. 敏感信息日志泄漏风险
   • generate_summary 中：

     logger.info(f"[APLR] SYS_PROM: {system_prompt}")

   • 会把完整 system prompt 打进日志，可能包含用户对话、长期记忆节点、行为偏好等敏感内容。

---

vector_db.py

1. 异步环境中的潜在同步网络 I/O 阻塞

   • SentenceTransformer(model_name) 在模型未缓存时可能触发同步下载（网络 I/O），当前为同步初始化，会阻塞执行线程。

2. 鲁棒性不足

   • encode/query/upsert 缺少异常处理，底层模型或向量库异常会直接冒泡到上层 async 逻辑。

3. 无效导入

   • Path 未使用。

---

代码评审团

人物一：Linus Torvalds（硬核、直接）
“os.system(f"...") 这种拼 shell 的写法太脆了，尤其参数还带配置输入。能跑不代表该这么跑。把进程调用和参数传递做成结构化调用，不要让字符串拼接决定安全边界。”

人物二：Steve Jobs（产品与体验导向）
“你做的是‘记忆系统’，却在日志里打印完整系统提示词，这就像把用户日记贴在公告栏。功能上很酷，但隐私体验是产品的一部分，必须做得‘看不见但可靠’。”

人物三：Geralt of Rivia（冷静务实派）
“我见过很多怪物，N+1 查询是其中一种。每条事件单独开连接拿标签，会慢、会累、会出事。一次查全、分组映射，战斗会轻松很多。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
chat_history_extract.py
database.py
models.py
summarizer.py
vector_db.py

---

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

[astrbot-plugin-copybara]

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

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