[Plugin] astrbot_plugin_minecraft_wiki

[Mxpea]

插件信息

{
  "name": "astrbot_plugin_minecraft_wiki",
  "display_name": "Minecraft Wiki查询",
  "desc": "用于为llm调用minecraft wiki",
  "author": "Mxpea",
  "repo": "https://github.com/Mxpea/astrbot_plugin_minecraft_wiki",
  "tags": [
    "Minecraft",
    "Wiki",
    "查询"
  ],
  "social_link": "https://github.com/Mxpea/"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_minecraft_wiki

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

main.py

• 代码质量与编码规范:
  • __init__ 方法中获取了 self.data_dir = StarTools.get_data_dir()，但在后续代码中没有任何实际使用（缓存仅为内存中的 WikiTTLCache）。如果不需要持久化，建议移除此段冗余代码；若需持久化，则应将其用于缓存的保存和加载。
• 潜在缺陷或问题 (性能瓶颈):
  • 文件中存在大量未预编译的正则表达式（如 _is_version_like_title 中的 r"^(java版|基岩版)\d|^\d+w\d+[a-z]?"，以及 _extract_command_from_text、_normalize_question_text 中的正则）。尤其是在 _search_with_evidence 的内部函数 _score(row) 中，_is_version_like_title 会被高频循环调用。每次调用都会重新编译正则表达式，这在数据量大时会造成不必要的性能开销。建议在模块顶层使用 re.compile() 进行预编译。

minecraft_wiki/config.py

• 功能实现与逻辑正确性 (边界条件):
  • from_mapping 方法中使用了强制类型转换，例如 float(getter("timeout_seconds", cls.timeout_seconds))。如果 config 字典中确实存在 "timeout_seconds" 这个键，但其对应的值恰好是 None，getter 将返回 None 而不是默认值，此时进行 float(None) 或 int(None) 会直接抛出 TypeError 引发崩溃。建议在转换前增加针对 None 的安全判断。

minecraft_wiki/tools/get_command_info.py

• 代码质量与编码规范 (代码重复):
  • _resolve_redirect_title 函数（用于解析 Wiki 的 #redirect 重定向标签）在 get_command_info.py、get_mechanic_info.py、get_recipe.py 和 get_section.py 四个文件中被完全复制粘贴了四次。严重违背了 DRY (Don't Repeat Yourself) 原则，建议将其抽离到 wiki/parser.py 或公共工具模块中复用。
• 潜在缺陷或问题 (性能瓶颈):
  • 在 _fallback_command_lines 函数的 for raw_line in ... 循环内部，使用了动态构建并计算正则：any(re.search(pattern, line, flags=re.I) for pattern in patterns)。在解析长文本的循环中，每次迭代都在动态编译正则表达式，这会引发显著的性能下降。建议在循环外部先构建并 re.compile(..., re.I) 这些 patterns。

minecraft_wiki/tools/get_mechanic_info.py

• 代码质量与编码规范:
  • 同样存在完全重复的 _resolve_redirect_title 函数。

minecraft_wiki/tools/get_recipe.py

• 代码质量与编码规范:
  • 同样存在完全重复的 _resolve_redirect_title 函数。
  • _normalize_item_name 中的 patterns 列表在每次调用时都会重新生成，建议将其移出函数体并预编译。

minecraft_wiki/tools/get_section.py

• 代码质量与编码规范:
  • 同样存在完全重复的 _resolve_redirect_title 函数。

minecraft_wiki/wiki/cache.py

• 潜在缺陷或问题 (内存泄漏风险):
  • WikiTTLCache 采用的是**惰性删除（Lazy Eviction）**策略，即只在调用 get 方法访问某个键时才检查其是否过期并删除。对于一个常驻后端的 Bot 来说，如果大量不重复的查询词（Key）被缓存进去但之后再也没有被访问过，这些过期的键值对将永远留在 self._store 字典中，导致持续的内存泄漏。建议加入字典的最大容量限制（如 LRU 机制），或者实现一个定期清理过期键的异步后台任务。

---

代码评审团

人物一（Linus Torvalds 风格）：
“老天，你是在和我开玩笑吗？看看 cache.py 里的那个缓存实现！你把数据塞进字典，指望它们在被访问时才清理？要是没人访问呢？你这根本不是在写缓存，而是在给内存埋定时炸弹！内存泄漏在长时间运行的守护进程里是零容忍的。还有那些嵌套在循环里的 re.search，每次迭代都重新编译正则，你当 CPU 周期是不要钱的吗？拜托，把正则编译挪到循环外面，给缓存加个容量上限，别让这玩意把服务器内存吃光！”

人物二（Steve Jobs 风格）：
“我们做产品的目标是极致的优雅。但是当我翻阅你的代码时，我看到 _resolve_redirect_title 这个完全一样的函数，硬生生地被复制粘贴到了四个不同的文件里。这不优雅，也不优美。伟大的代码就像一件艺术品，它的内部结构应该像它的外观一样简洁。把这些重复的逻辑抽离到一个 parser 工具库里去，让代码的呼吸变得顺畅。不过，我必须承认，你对工具入口 ask_minecraft_wiki 意图推断的设计，体验上确实顺滑，这部分的思考方向很正确。”

人物三（赛博朋克 AI 架构师）：
“接入数据矩阵时，任何盲目的信任都会导致系统崩溃。看看你的 config.py，你以为 config.get("key", default) 就能抵御虚无（None）的侵袭吗？如果用户显式地传入了 { "timeout_seconds": null }，类型转换器 float(None) 就会当场引发系统级异常宕机。在边缘计算中，永远要对传入的有效载荷进行多重校验；另外，main.py 里申请了 data_dir 的存储权限却弃之不用，就像申请了神经网路接口却没有连接任何突触，这可是赛博空间里的资源浪费哦。”

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

---

🔍 审核摘要

统计信息

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

main.py
astrbot_plugin_minecraft_wiki.py
minecraft_wiki/__init__.py
minecraft_wiki/config.py
minecraft_wiki/plugin.py
minecraft_wiki/tools/__init__.py
minecraft_wiki/tools/get_command_info.py
minecraft_wiki/tools/get_mechanic_info.py
minecraft_wiki/tools/get_recipe.py
minecraft_wiki/tools/get_section.py
minecraft_wiki/tools/get_summary.py
minecraft_wiki/tools/search_page.py
minecraft_wiki/wiki/__init__.py
minecraft_wiki/wiki/api.py
minecraft_wiki/wiki/cache.py

注意：由于项目规模或Token限制，本次仅审核了部分优先文件。以上报告内容由 AI 直接生成。

---

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

当前仓库：Mxpea/astrbot_plugin_minecraft_wiki
剩余触发次数：4

[astrbot-plugin-copybara]

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

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