[Plugin] astrbot_plugin_plugin_help

[vintro3]

插件信息

{
  "name": "astrbot_plugin_plugin_help",
  "display_name": "插件指令帮助",
  "desc": "对于大多数 bot 而言，所使用的插件都不会是仅一两个。插件多了便不好记忆具体插件及对应指令，本插件即用于查询 bot 所安装的插件及所包含的指令。",
  "author": "vintro",
  "repo": "https://github.com/vintro3/astrbot_plugin_plugin_help"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_plugin_help

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

main.py

1. from astrbot.api.all import * 存在可维护性与命名污染风险（代码规范）

   • 通配符导入会把大量符号注入当前命名空间，降低可读性，也更容易引发隐式覆盖问题（例如与后续导入名冲突）。
   • 在 Python 3.10 项目中，这类写法也会让静态分析、类型检查与重构工具效果变差。
   • 建议改为显式导入实际使用的对象。

2. star_handlers_registry 的导入失败被静默降级为空列表，可能掩盖真实故障（逻辑正确性/可维护性）

   • 目前 ImportError 后直接 star_handlers_registry = []，插件功能会“看起来正常运行但始终无结果”，这属于“静默失败”。
   • 这会增加排障难度：用户只会看到“未检测到已加载的外部插件指令”，而不是明确知道注册表加载失败。
   • 建议至少记录告警日志（使用 from astrbot.api import logger），避免隐性故障。

3. 通过 type(f).__name__ 判断过滤器类型，属于脆弱实现（功能鲁棒性）

   • 使用字符串 "CommandFilter" / "CommandGroupFilter" 匹配类型名，对重命名、代理类、继承层变化非常敏感。
   • 一旦框架内部类名调整，功能会直接失效但不易察觉。
   • 建议使用更稳定的类型判断方式（例如 isinstance，若框架公开了类型）。

4. 插件名来源与黑名单过滤逻辑较“硬编码”，可能误伤合法插件（逻辑边界）

   • if plugin_name in ["未知插件", "star_handler", "builtin", "core", "command_menu", "CommandMenu"] 这类固定排除规则可能误伤真实插件（例如插件真叫 core）。
   • 同时，插件名推断依赖 handler_module_path 字符串切分，也比较脆弱。
   • 建议把“是否外部插件”的判定改为更明确的结构化元数据判断，减少魔法字符串。

5. 字符串拼接构造消息在循环中反复 +=，在插件/命令多时有性能劣化（潜在性能）

   • 虽然通常量级不大，但在大量插件场景会产生不必要的中间字符串对象。
   • 建议用列表收集后 "\n".join(...) 组装消息。

6. 命令参数 plugin_name 未做空白归一化，导致边界输入体验不一致（功能细节）

   • 用户输入仅空格（如 /菜单 ）时，if plugin_name: 的行为取决于框架传参细节，可能出现“按插件检索分支被触发但匹配为空”的体验问题。
   • 建议在进入逻辑前 plugin_name = plugin_name.strip()。

---

代码评审团

人物一：Linus Torvalds（直言不讳型）
“import * 这种写法就是在给未来的你挖坑。现在省两行，半年后调试时会骂自己。把依赖写清楚，代码才配叫工程代码。”

人物二：Steve Jobs（产品体验型）
“你这个菜单功能方向是对的，但‘静默失败’非常不产品化。用户看到空列表，只会怀疑产品不可靠。失败应该可感知、可解释。”

人物三：Tony Stark / 钢铁侠（工程效率型）
“我喜欢自动扫描插件的想法，但你用字符串猜类型、猜插件来源，这像是‘战甲胶带修复模式’。把判定逻辑升级成结构化协议，后续扩展会轻松很多。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

当前仓库：vintro3/astrbot_plugin_plugin_help
剩余触发次数：4