[Plugin] astrbot_plugin_yunhuAdapter

[Soffd]

插件信息

{
  "name": "astrbot_plugin_yunhuAdapter",
  "display_name": "云湖机器人平台适配器",
  "desc": "将 AstrBot 接入云湖机器人平台的插件，通过 WebSocket 与云湖 SDK 桥接通信。",
  "author": "Yuki Soffd",
  "repo": "https://github.com/Soffd/astrbot_plugin_yunhuAdapter",
  "tags": [
    "云湖",
    "平台适配器"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_yunhuAdapter

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

main.py

• 功能实现与逻辑正确性：
  • 缺失基类初始化：在 YunhuPlugin 的 __init__ 方法中，缺少了对父类 Star 的初始化调用 super().__init__(context)。这是非常严重的逻辑错误，会导致插件上下文（context）未能正确挂载，插件功能将无法正常运行。
• 框架适应性检查：
  • 插件注册废弃提示：虽然使用了正确的 @register 格式，但根据框架规范（v3.5.20 之后），@register 装饰器已废弃。建议直接移除装饰器，框架会自动识别继承自 Star 的类。
• 代码质量与编码规范：
  • 无用/隐式导入：__init__ 中 from .yunhu_adapter import YunhuAdapter 导入后未使用。如果是为了触发模块内部的 register_platform_adapter 注册机制，建议在模块顶部正常导入或添加注释说明，否则会被 linter 工具（如 flake8）标记为未使用，且可读性较差。

yunhu_adapter.py

• 框架适应性检查：
  • 日志组件导入违规：代码中出现了 import logging（内置日志库），且使用了 from astrbot import logger。根据严格规范，必须且只能使用 from astrbot.api import logger。严禁导入和使用 Python 内置的 logging 模块。
• 可维护性与可读性：
  • 冗余的方法重写：send_by_session 方法除了调用 await super().send_by_session(session, message_chain) 之外没有任何其他逻辑。这种无意义的重写增加了代码行数且无任何实际作用，建议直接删除该方法。

yunhu_event.py

• 框架适应性检查：
  • 日志组件严重违规：在文件顶部使用了 import logging，并定义了 logger = logging.getLogger("yunhu.event")。这严重违反了框架规范，必须改为 from astrbot.api import logger，并移除内置 logging 的所有相关调用。
• 代码质量与编码规范：
  • 严重违反 DRY（不要重复自己）原则：在 send() 方法中，针对 Image、File/Record、Video 三个组件的处理逻辑高度重复。它们都在做相同的事情：检查前缀、读取本地文件转 Base64、判断 http 并组装 payload。这部分逻辑应该被提取成一个独立的异步或同步私有辅助函数（例如 _build_media_payload），以显著提高代码可读性和可维护性。
• 功能实现与逻辑正确性：
  • 跨平台路径判断缺陷：在判断本地文件路径时使用了 url.startswith("/")（例如 if url.startswith("file:///") or (url.startswith("/") and os.path.exists(url)):）。这种判断方式在 Windows 系统下会失效（如 C:\path\to\file 既不以 file:/// 开头，也不以 / 开头）。建议使用 os.path.isabs(url) 配合 os.path.exists(url) 来做跨平台的本地绝对路径检测。

---

代码评审团

Linus Torvalds：（满脸不耐烦，指着屏幕）

“我简直不敢相信我的眼睛。你在 main.py 里继承了一个类，然后把 super().__init__() 给吃了吗？这就像你造了一辆汽车，却忘了装引擎，然后指望它能跑起来！还有 yunhu_event.py 里那一坨又长又臭的 if-elif，你把完全相同的判断逻辑复制粘贴了三次？三次！写一个辅助函数对你来说是会触发什么物理伤害吗？把你那违背 DRY 原则的垃圾代码清理掉，不然我看到一次就想把你的键盘砸了！”

Sherlock Holmes：（拿着放大镜，冷静且带着一丝优越感）

“显而易见，我亲爱的朋友。你看 yunhu_event.py 里的这句 url.startswith("/")，在 Unix 系统的温室里它确实能如你所愿。但如果你把这串代码带到广袤的 Windows 荒野中，例如面对一个名为 C:\Users\File.png 的嫌疑人，你的防线将瞬间崩溃。细节决定成败，使用 os.path.isabs() 才是无懈可击的推理。另外，你在 main.py 的构造函数里悄悄塞了一个未使用的 import，这是某种掩人耳目的魔法，还是单纯的疏忽？我更倾向于后者。”

GLaDOS：（机械音，冷酷且充满讽刺）

“测试结果表明，您对规则的漠视达到了令人惊叹的水平。系统明确要求日志必须只能从 astrbot.api 获取，但您却在 yunhu_event.py 里私自使用了 logging.getLogger。这就好比我为您准备了无毒的蛋糕，您非要去吃地上的放射性废料。顺便一提，您的 send_by_session 函数什么都没做，只是把工作推给了父类。如果您也是这样对待您的生活的，那真是一场有趣的实验。请修复这些错误，除非您想面临……更深入的‘测试’。”

---

🔍 审核摘要

统计信息

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

main.py
yunhu_adapter.py
yunhu_event.py

---

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

[astrbot-plugin-copybara]

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

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