[Plugin] astrbot_plugin_qq_restapi

[Ryan7t]

插件信息

{
  "name": "astrbot_plugin_qq_restapi",
  "display_name": "QQ REST API 适配器",
  "desc": "QQ 官方 REST API 平台适配器，不依赖 botpy，直接 HTTP 调用官方接口。支持 WebSocket/Webhook 双模式、35+ 事件类型、Markdown 模板、按钮面板、自动事件处理等",
  "author": "Wanyi",
  "repo": "https://github.com/Ryan7t/astrbot_plugin_qq_restapi",
  "tags": [
    "平台适配器",
    "QQ"
  ],
  "social_link": "https://github.com/Ryan7t"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_qq_restapi

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

main.py

1. 资源生命周期不完整：_register_private_template_source() 注册了外部模板源，但在 terminate() 中没有对应 unregister_external_template_source()，可能导致插件反复加载/卸载后模板源残留。
2. assert 用于运行时关键检查：assert _PRIVATE_BOT_ROOT is not None 在优化模式（python -O）下会被移除，不适合做生产校验。
3. 潜在隐私泄露日志：helloworld 中直接 logger.info(message_chain)，可能把完整消息链（含用户信息）写入日志。
4. 可维护性问题：大量命令包装函数是重复样板（async for result in xxx_impl），维护成本高、易漏改。

---

init.py

1. 通配符导出风险：from .public_api import * 会污染模块命名空间，降低可读性和静态分析可用性，也增加未来破坏性变更风险。

---

public_api.py

1. 异常吞噬过宽：resolve_qq_platform() 中多处 except Exception: pass，会掩盖真实错误并降低排障效率。
2. 异步环境下暴露同步网络客户端：get_shared_sync_client() 公开同步 HTTP 客户端接口，若在事件循环中被调用，存在阻塞风险（网络 I/O 维度）。

---

adapters/qq_restapi_adapter.py

1. 日志导入不符合框架约束：使用 from astrbot import logger，按要求应为 from astrbot.api import logger。
2. terminate() 未真正停止连接循环：仅打印日志，没有通知 QQGatewayClient 停止，可能导致后台任务持续运行。
3. 参数未使用：platform_settings 形参未使用。
4. 事件处理缺少隔离：_on_dispatch() 内部若抛异常，会向上影响网关循环稳定性（尤其在高频事件场景下）。

---

adapters/qq_restapi_webhook_adapter.py

1. 日志导入不符合框架约束：使用 from astrbot import logger，应改为 from astrbot.api import logger。
2. 参数未使用：platform_settings 未使用。
3. 运行逻辑可疑：run() 仅等待 shutdown_event，未调用 self.server.start()；若非完全依赖统一 webhook 路由，此处会导致 webhook 服务未启动（当前实现与 QQRestAPIWebhookServer.start() 存在职责割裂）。

---

adapters/qq_restapi_webhook_server.py

1. 日志导入不符合框架约束：使用 from astrbot import logger，应为 from astrbot.api import logger。
2. 严重副作用：模块导入时执行
   for handler in logging.root.handlers[:]: logging.root.removeHandler(handler)，会全局移除 root logger handler，影响整个进程日志系统。
3. 潜在敏感信息日志暴露：logger.debug(f"收到 ... 回调: {payload}") 直接记录完整 webhook 负载。
4. 无必要的 async 定义：repeat_seed() 为纯 CPU/字符串操作却定义为 async，增加调用复杂度。
5. 未使用导入：Optional 未使用。

---

adapters/ws_client.py

1. 日志导入不符合框架约束：使用 from astrbot import logger，应为 from astrbot.api import logger。
2. 停止机制不完整：只有 _stop 标志位，没有公开停止方法；结合上层 terminate() 未驱动该标志，连接循环难以优雅退出。
3. 任务取消不完整：heartbeat_task.cancel() 后未 await 回收，可能出现“Task was destroyed but it is pending”类告警。
4. 成功重连后退避未重置：retry_delay 增长后不会在成功连接后复位，后续临时抖动时恢复速度可能不必要地变慢。

---

db/database.py

1. 数据目录获取不符合规范要求：当前使用 get_astrbot_plugin_data_path() + 手工拼接；按约束应通过 StarTools.get_data_dir() 获取插件数据目录。
2. 实例属性命名不规范：self.DATABASE_URL 采用常量命名风格但为实例变量，降低可读性。
3. 事务管理冗余/易混淆：async with self.engine.begin() 块内再显式 await conn.commit()，与上下文自动提交语义重复，增加维护理解成本。

---

db/repository.py

1. API 参数缺少边界保护：list_event_logs(limit=100) 对 limit 无合法性校验（如负数或超大值），存在查询压力和行为不确定性风险。
2. 可维护性问题：upsert_channel(..., type: int | None = None, ...) 使用 type 作为参数名，遮蔽内置标识符。
3. 更新策略副作用：_apply_updates() 忽略 None，会导致无法将字段显式清空（语义上容易与“保持不变”混淆）。

---

db/service.py

1. 日志导入不符合框架约束：使用 from astrbot import logger，应为 from astrbot.api import logger。
2. 布尔解析逻辑错误：user_info.get("bot") or user_info.get("is_bot") 会把 False/0 当成“无值”丢失，导致 bot_flag 误判。
3. 死代码/可读性差：_extract_event_data() 中 if inner_type and not raw.get("t"): pass 没有实际作用。
4. 异常处理过宽且吞错：record_event() 顶层 except Exception 仅记录日志不抛出，可能长期静默丢失数据写入失败。

---

代码评审团

人物一：Linus Torvalds
“你这日志导入风格不统一（astrbot vs astrbot.api）和导入即改全局 logging handler 的做法，属于‘一处图省事，全局跟着遭殃’。建议把副作用清零：模块导入只做声明，不做全局状态破坏。”

人物二：乔布斯（Steve Jobs）
“体验就是细节。terminate() 只打日志不真正停服务，这会让‘看起来结束了’和‘实际上还在跑’出现分裂。把生命周期做完整，用户才会觉得产品是‘it just works’。”

人物三：神奇女侠（Diana Prince）
“代码要真诚：False 就是 False，不该在 or 链里被吞掉。bot_flag 的布尔处理像把真言套索打了结。建议显式判空再解析，保持语义准确和数据可信。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
public_api.py
adapters/__init__.py
adapters/qq_restapi_adapter.py
adapters/qq_restapi_webhook_adapter.py
adapters/qq_restapi_webhook_server.py
adapters/ws_client.py
core/__init__.py
db/__init__.py
db/database.py
db/models.py
db/repository.py
db/service.py
runtime/__init__.py

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

---

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

当前仓库：Ryan7t/astrbot_plugin_qq_restapi
剩余触发次数：4