[Plugin] astrbot_plugin_memos_manager

[cyilin36]

插件信息

{
  "name": "astrbot_plugin_memos_manager",
  "display_name": "Memos Manager",
  "desc": "一个能对usememos/memos进行管理的插件",
  "author": "cyilin36",
  "repo": "https://github.com/cyilin36/astrbot_plugin_memos_manager",
  "tags": [
    "笔记",
    "note"
  ],
  "social_link": "https://cyilin36.github.io/"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_memos_manager

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

main.py

1. start_date / end_date 类型边界处理不完整，可能触发非预期异常

   • run_search() 里直接把 kwargs.get("start_date") / kwargs.get("end_date") 传给 _parse_date_bound()。
   • _parse_date_bound() 会调用 raw.strip()，若传入非字符串（如整数、对象）会抛 AttributeError，最终走到“unexpected error”，而不是清晰的输入错误。
   • 影响：错误语义不稳定，调用方难以定位问题。

2. pinned 参数未做类型归一化，可能把非法类型传给后端

   • MemosUpdateTool.call() 中 pinned=kwargs.get("pinned") 直接下传。
   • 若 LLM 或上游传入 "true"/"false" 字符串，将以字符串进入 API，可能导致后端 4xx。
   • 影响：工具层“看似成功调用”，实际由后端报错，增加排障成本。

3. 错误信息回传过于直接，存在内部细节泄露风险

   • 多处 except Exception as exc 后返回 errors: [str(exc)] 给上层。
   • 影响：可能将内部实现细节/异常文本暴露给最终用户或模型上下文，属于信息暴露风险（尤其在生产环境）。

4. 鉴权失败与输入错误会生成新的 trace_id，与上层链路可能不一致

   • BaseMemosTool._check_auth_or_return()、_handle_archive_set()在工具入口处单独新建 trace。
   • 影响：若平台已有请求级 trace，链路关联性下降，排查跨组件问题时不够友好。

---

init.py

• 未发现需报告的问题。

---

memos_client.py

1. 每次请求都新建 httpx.AsyncClient，连接无法复用，存在性能瓶颈

   • _request() 内部 async with httpx.AsyncClient(...) 每次创建/销毁客户端。
   • 影响：高并发下 TLS 握手和连接建立成本高，吞吐与延迟都会变差。
   • 建议方向：在 MemosClient 生命周期内复用一个 AsyncClient（或由上层注入共享 client）。

2. list_recent_memos(limit) 对负数参数处理有逻辑漏洞

   • page_size 用 max(1, int(limit))，但返回时用 memos[:limit]。
   • 当 limit < 0 时，Python 切片会返回“去掉末尾若干项”的结果，而不是空列表或报错。
   • 影响：返回条数语义错误，可能引发调用侧误判。

3. name 校验仅 startswith("memos/")，约束过宽

   • update_memo/delete_memo 仅检查前缀，像 memos/../x 这类异常路径片段仍可能通过。
   • 影响：虽然不一定被服务端接受，但客户端侧缺少严格输入约束，鲁棒性不足。
   • 建议方向：使用更严格的资源名格式校验（如正则白名单）。

---

tool_models.py

• 未发现需报告的问题。

---

代码评审团

人物一：Linus Torvalds（直球派）
“memos_client.py 每个请求都 new 一个 AsyncClient，这就像每次开车先造一辆车再开。能跑，不代表对。把连接复用起来，这是异步网络代码的基本卫生。”

人物二：Steve Jobs（产品体验派）
“错误信息要对用户友好，但不该把内部异常原样端出去。现在这套 str(exc) 很‘工程师’，不够‘产品’。把用户可见错误和调试日志分层，你的体验会专业很多。”

人物三：Geralt of Rivia（猎魔人式务实）
“怪物通常藏在边界里。limit < 0、start_date 不是字符串、pinned 是字符串——这些都不是大问题，直到某天它们一起出现。先在工具层把输入驯服，再去打后端的仗。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
memos_client.py
tool_models.py

---

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

[astrbot-plugin-copybara]

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

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