[Plugin] astrbot_plugin_touhou_kb

[YunMo-yanyu]

插件信息

{
  "name": "astrbot_plugin_touhou_kb",
  "display_name": "东方知识库",
  "desc": "将 THBWiki 备用站词条清洗后导入 AstrBot 官方知识库，支持会话绑定、人格绑定与东方话题优先路由",
  "author": "YunMo-yanyu",
  "repo": "https://github.com/YunMo-yanyu/astrbot-",
  "tags": [
    "东方Project",
    "THBWiki",
    "知识库",
    "RAG",
    "人格绑定"
  ],
  "social_link": ""
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_touhou_kb

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

main.py

1. 并发任务缺乏互斥控制，可能导致知识库写入竞争

   • sync_touhou_pages 可重复触发 _start_sync_task，未限制“同一知识库仅允许一个同步任务”。
   • 多个任务同时执行 _upsert_page（先删后传）时，可能出现文档抖动、覆盖、统计不一致等竞态问题。

2. 后台任务生命周期管理不完整

   • 通过 asyncio.create_task 创建后台任务后，仅做了 done_callback 清理字典；未看到插件卸载/停机时的统一取消逻辑。
   • 在异步框架下，这会带来“悬挂任务”风险（尤其在热重载或退出时）。

3. 存在潜在跨域重定向抓取风险（安全）

   • _fetch_html 开启 follow_redirects=True，但未校验最终响应 URL 是否仍在 ALLOWED_HOSTS。
   • 即使入口 URL 合法，也可能因重定向访问到非预期域名。

4. 同步队列使用 list.pop(0)，大队列下性能退化

   • _run_sync_task 中 BFS 队列使用 list 头部弹出，时间复杂度 O(n)。
   • 页面规模增大时会造成明显性能损耗，建议 collections.deque。

5. 异常处理过宽且部分吞异常，降低可维护性

   • 多处 except Exception（如 _remove_competing_tools、同步抓取主循环）仅继续执行，可能掩盖编程错误与数据问题。
   • 建议缩小异常范围，并在关键点记录更精确上下文（异常类型、堆栈）。

6. 用户可传入的 top_k 未做边界校验

   • bind_touhou_kb 接收 top_k 后直接写入 session 配置。
   • 若传入 <=0 或异常大值，可能导致检索行为异常或性能问题。

7. inject_touhou_hint 直接拼接检索上下文，存在提示词膨胀风险

   • 每次命中都会尝试把 context_text 注入 system_prompt。
   • 缺少长度上限/裁剪策略，可能导致 token 激增，影响响应质量与成本。

8. 异步事件循环内执行较重 HTML 解析，可能影响响应性

   • BeautifulSoup(..., "lxml") + 大量 DOM 操作都在协程中同步执行。
   • 在高并发消息场景下会占用事件循环，造成延迟抖动（虽非网络阻塞，但属异步运行时性能隐患）。

---

init.py

未发现需报告的问题。

---

代码评审团

人物一：Linus Torvalds
“你这同步流程能跑，但并发上太‘乐观’了。多个任务一起删文档再上传，这种 race condition 在我这儿会被直接打回。先把互斥和任务状态机做扎实，再谈功能扩展。”

人物二：Steve Jobs
“功能很全，但体验细节还不够‘优雅’：system_prompt 一直加上下文，迟早会变成一团噪音。做产品不是堆信息，而是精准呈现。限制上下文长度，用户会明显感受到质量提升。”

人物三：Donald Knuth
“算法层面的小选择会放大成系统级差异：pop(0) 是典型可避免的低效。改用 deque 是非常便宜但收益稳定的优化，建议尽快落实。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。评论 @astrpluginreviewer review 来重新触发 review，请不要重复提交插件发布 issue。

当前仓库：YunMo-yanyu/astrbot-
剩余触发次数：4

[astrbot-plugin-copybara]

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

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