[Plugin] astrbot_plugin_command_router

[PyuraMazo]

插件信息

{
  "name": "astrbot_plugin_command_router",
  "display_name": "智能指令路由器",
  "desc": "基于AI的自然语言指令解析，允许用户使用自然语言调用 AstrBot 中已注册的指令。",
  "author": "PyuraMazo",
  "repo": "https://github.com/PyuraMazo/astrbot_plugin_command_router",
  "tags": [
    "指令解析"
  ],
  "social_link": "https://github.com/PyuraMazo"
}

插件检查清单

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

[astrpluginreviewer]

❌ 插件审核失败

您好！在对您的插件进行审核时遇到了问题：

验证metadata.yaml失败:
- metadata.yaml中的desc字段 "通过AI解析并触发特定指令。" 与JSON中提交的desc "基于AI的自然语言指令解析，允许用户使用自然语言调用 AstrBot 中已注册的指令。" 不一致

---

请根据上述问题进行修改。修改完成后，请在 Issue 正文 中勾选"重新提交审核"复选框以再次触发审核。

此消息由系统自动生成

[PyuraMazo]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_command_router

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

main.py

1. 代码质量与编码规范

• 未处理潜在的 NoneType：在 CommandParser._build_brief 方法中，使用了 args = params.get(sub_command.current_fragment) 和 args = params.get(info.current_fragment)。如果 get 方法没有找到对应的键，它将返回 None。紧接着的列表推导式 for name in args 遇到 None 会直接抛出 TypeError: 'NoneType' object is not iterable 异常。建议修改为 args = params.get(...) or {}。
• 类型安全风险：在 describe_resp 方法中，对 resp.parameters 执行了 " ".join(resp.parameters)。但在 LLMResponse 模型中，parameters 被定义为 list[str] | None = None。一旦大模型未返回该字段导致其值为 None，join 操作也会触发 TypeError。建议修改为 " ".join(resp.parameters or []) or "无"。

2. 功能实现与逻辑正确性

• 大模型响应解析脆弱：LLM.submit 方法中直接使用了 json.loads(llm_resp.completion_text)。在实际运行中，各大模型往往会在 JSON 结果外包裹 Markdown 代码块（如 ```json\n...\n```）或混杂其他提示词。直接 json.loads 极易引发 JSONDecodeError 导致解析崩溃。建议引入正则表达式预先提取大括号 {} 内的 JSON 内容，或捕获解析异常以避免程序意外中断。

3. 安全漏洞与最佳实践

• 代码中未发现明显的安全漏洞。使用了 Pydantic 进行类型验证，符合现代 Python 校验的最佳实践。

4. 可维护性与可读性

• 重复的初始化逻辑：global_parser 和 command_parser 中都包含了完全一样的懒加载初始化判断：

  if not self.parser.init:
      await self.parser.initialize()
      self.parser.init = True

  既然插件本身继承了 Star 并提供有 async def initialize(self): 钩子方法，建议将 self.parser.initialize() 的逻辑放在插件生命周期的 initialize 方法中集中执行。如果因为框架的加载顺序导致无法放入其中，则建议将这段懒加载代码封装为一个单独的 async def get_or_init_parser(self) 异步方法进行调用，提高可读性。

5. 潜在缺陷或问题

• 并发初始化竞争 (Race Condition)：目前的懒加载模式在并发环境下不安全。如果两个消息同时到达触发事件监听器，且此时 self.parser.init 仍为 False，那么 await self.parser.initialize() 可能会被并发调用两次。这将导致 max_id 重复累加，id_dict 紊乱，从而造成逻辑灾难。如果必须使用懒加载，请务必使用 asyncio.Lock() 进行加锁保护。
• Token 暴涨的性能隐患：在 LLM.build_cmd_str 中，通过列表推导式将所有的 CommandBrief 转为 JSON 并全量拼接。随着机器人后续安装的插件越来越多，指令集会非常庞大，这不仅会导致单次 Prompt 长度超过 LLM 的 Token 上限（Context Window Limit），还会大大增加 API 调用的开销。

6. 框架适应性检查

• 日志记录：✅ 已正确使用 from astrbot.api import logger。
• 并发模型：✅ 代码中未使用阻塞性网络 I/O。
• 主类注册：✅ CommandRouterPlugin 继承自 Star，并适应了高版本的无需 @register 的特性。
• filter 装饰器导入：✅ 正确从 astrbot.api.event.filter 导入。
• 事件监听器签名：✅ 所有使用 @filter 装饰的方法签名中均包含了 event: AstrMessageEvent。

---

代码评审团

Linus Torvalds：（皱着眉头，快速翻阅代码）
“这是一场豪赌，而你甚至不知道自己上了牌桌。你居然敢指望大模型乖乖给你返回纯粹、干净的 JSON？json.loads(llm_resp.completion_text)？如果它带了一个换行符或者一个狗屎一样的 Markdown 代码块，你的整个程序就会当场以异常退出。不加校验直接信任不受控的外部输入，这简直是犯罪！还有并发那段，不加 asyncio.Lock 就去初始化全局状态？你是觉得 Python 的 async 事件循环是为你专门开辟单行道的吗？”

埃隆·马斯克 (Elon Musk)：（手托着下巴，若有所思）
“我们要从第一性原理（First Principles）出发考虑效率。你看这个 build_cmd_str 做了什么？把所有的命令详情全量 dump 到大模型的 Prompt 里去。如果用户装了 100 个插件呢？每次对话都要传输好几十 KB 的冗余文本。这会让你的 API 账单直接飞到火星去，而且响应时间极差。我们需要做 RAG（检索增强生成）或者前置的轻量意图过滤来裁剪上下文，现在的做法是不可持续的（Unsustainable）。”

约翰·卡马克 (John Carmack)：（盯着屏幕上的几行逻辑）
“你对数据的结构边界有着非常随意的态度。args = params.get(...) 之后不检查结果，下一行紧接着就拿它当可迭代对象来用（for name in args）。如果键名没对上，返回了 None，这个子系统就直接崩溃了。parameters 类型标注写了可能是 None，后面又直接 " ".join()... 类型提示系统（Type Hinting）存在的意义是为了让你在写代码的时候就防御这类问题的，不是只写着好看的。加上默认的 Fallback，比如 or {}、or []，别让你的代码如此脆弱。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[astrbot-plugin-copybara]

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

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

[Soulter]

把我想做的给做了 😭