[Plugin] astrbot_plugin_GPT-SoVITS

[w2902171175]

插件信息

{
  "name": "astrbot_plugin_GPT-SoVITS",
  "display_name": "astrbot_plugin_GPT-SoVITS",
  "desc": "本地运行的 GPT-SoVITS-1007-cu124 的 Astrbot 文本转语音插件。",
  "author": "w2902171175",
  "repo": "https://github.com/w2902171175/astrbot_plugin_GPT-SoVITS",
  "tags": [
    "TTS",
    "AI",
    "文本转语音",
    "GPT SoVITS",
    "语音合成"
  ],
  "social_link": "https://github.com/w2902171175"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_GPT-SoVITS

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

main.py

1. _post() 未校验 HTTP 状态码，错误响应可能被当作“正常数据流”处理

   • 位置：_post
   • 问题：resp.raise_for_status() 缺失。当前逻辑会在 4xx/5xx 时继续尝试 resp.json()，失败后返回 resp.content，导致上游只看到“非预期格式”，无法明确区分“服务异常”和“解析异常”。
   • 影响：故障定位困难、重试策略难做、错误被弱化。

2. 存在潜在 SSRF 风险：对 audio_url 缺少来源约束与域名校验

   • 位置：on_decorating_result -> _download_audio(audio_url)
   • 问题：audio_url 来自外部接口返回，当前直接请求，仅做了 0.0.0.0 -> api_host 替换；若返回恶意 URL（如内网地址、云元数据地址），可能触发服务端请求伪造。
   • 影响：可能探测或访问非预期内部网络资源。

3. 会话冷却（cooldown）存在并发竞争窗口

   • 位置：on_decorating_result 中 _session_state 的读写
   • 问题：在异步并发下，同一 session_id 的多个协程可能几乎同时通过冷却检查，然后都写入 last_trigger，导致冷却失效。
   • 影响：高并发场景下触发频率超出预期，增加 TTS 压力。

4. 频繁创建 httpx.AsyncClient，网络层性能与资源利用不佳

   • 位置：_post、_download_audio
   • 问题：每次请求都新建/销毁客户端，无法复用连接池。
   • 影响：吞吐下降、延迟增加，尤其在消息频繁时明显。

5. 类型标注与实际返回不一致

   • 位置：_download_audio(self, url: str) -> bytes
   • 问题：异常时返回 None，与注解 -> bytes 不符。
   • 影响：静态检查失真，维护者容易误判可空性。

6. 配置项缺少有效范围校验，边界输入会产生不可预测行为

   • 位置：__init__ 配置读取（prob/top_p/temperature/text_limit/cooldown 等）
   • 问题：如 prob 可被配置为负数或大于 100，当前逻辑会导致“几乎永不触发/总是触发”等异常行为；其他采样参数也未做约束。
   • 影响：运行行为偏离预期，问题排查困难。

7. @register 在新版本中已废弃，存在后续迁移负担

   • 位置：类装饰器 @register(...)
   • 问题：虽然当前可用，但按你给定规范，v3.5.20+ 推荐自动识别 Star 子类，不再依赖该装饰器。
   • 影响：未来升级时可能产生兼容性维护成本。

8. 临时文件目录固定在代码目录下，部署环境适配风险较高

   • 位置：self.temp_dir = Path(__file__).parent / "temp_audio"
   • 问题：若插件目录只读或受限（容器/托管平台常见），写文件会失败。
   • 影响：TTS 成功但消息发送失败，形成“功能看似随机不可用”。

---

代码评审团

人物一：Linus Torvalds（直球暴躁流）
“你这 _post 不 raise_for_status，然后把错误页当内容吞下去，等于把‘引擎故障灯’拿胶带贴住。先把失败定义清楚，再谈鲁棒性。建议：HTTP 错误立即抛出，业务层按状态分级处理。”

人物二：Steve Jobs（体验偏执流）
“每次请求都新建 AsyncClient，这就像每打一个电话就新建一家运营商。体验会慢、资源会抖。建议：把客户端做成类级复用对象，统一超时和重试策略，性能和稳定性会更优雅。”

人物三：GLaDOS（冷幽默安全流）
“把外部返回的 audio_url 直接抓下来？多么信任世界的做法。世界会奖励你一次 SSRF。建议：限制 scheme、host、port 白名单，只允许你的 TTS 服务域名，其他一律拒绝。‘安全’会让你活得更久。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[w2902171175]

next

[astrbot-plugin-copybara]

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

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