[Plugin] astrbot_plugin_GPT-SoVITS

[w2902171175]

插件信息

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

插件检查清单

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

[astrpluginreviewer]

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

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

main.py

1. 事件装饰器来源不符合规范（框架适配性）
   你当前使用的是：

   from astrbot.core.star.register import register_on_decorating_result as on_decorating_result
   @on_decorating_result()

   按你给定的审查规范，事件监听装饰器应来自 filter 对象（from astrbot.api.event import filter）。
   这会带来两类风险：

   • 使用 astrbot.core.* 内部路径，稳定性和可维护性较差；
   • 与统一事件注册规范不一致，后续迁移/维护成本上升。

2. @register 在新版本中已不推荐继续使用（main.py专项）
   你现在仍使用 @register(...)。按规范说明，v3.5.20 之后建议由框架自动发现 Star 子类。
   这不是“立刻报错”的问题，但属于兼容性/维护性隐患（未来升级时容易成为摩擦点）。

3. speed_facter 命名疑似拼写错误，可能导致参数失效（逻辑正确性）
   配置项和 payload 都使用 speed_facter：

   self.speed_facter = ...
   "speed_facter": self.speed_facter

   常见字段是 speed_factor。若后端接口实际字段名是 speed_factor，这里会导致语速参数完全不生效，且不易排查。

4. HTTP 响应未做状态码校验（功能与健壮性）
   _post() 里没有 resp.raise_for_status()。当服务端返回 4xx/5xx 时，代码仍可能继续尝试 resp.json()，最终把错误当成“格式异常”处理，定位问题困难。
   建议在解析前先校验状态码。

5. _download_audio 返回类型与注解不一致（可维护性）
   函数注解为 -> bytes，但异常时返回 None：

   async def _download_audio(self, url: str) -> bytes:
       ...
       return None

   这会误导调用方和类型检查工具。应改为 Optional[bytes]。

6. SSRF 风险：直接信任 audio_url 并发起下载（安全）
   代码直接使用接口返回的 audio_url 请求资源，只替换了 0.0.0.0：

   audio_url = resp.get("audio_url")
   audio_data = await self._download_audio(audio_url)

   如果 TTS 服务返回恶意 URL，插件会去访问任意地址。建议校验 scheme/host/port，至少限制在预期服务地址范围内。

7. 网络客户端重复创建，连接池未复用（性能）
   每次 _post / _download_audio 都新建 httpx.AsyncClient。高并发下会增加连接建立开销。
   更佳方式是插件生命周期内复用一个 AsyncClient（并在插件结束时关闭）。

8. 会话状态 _session_state 无清理策略，存在内存增长风险（潜在缺陷）
   session_id 持续累积，仅记录 last_trigger，无过期清除逻辑。长时间运行、会话多时可能导致内存持续增长。

9. 临时文件目录放在代码目录下，部署场景可能不可写（可运维性）
   使用：

   self.temp_dir = Path(__file__).parent / "temp_audio"

   在只读挂载、容器镜像层、受限权限环境中可能失败。更稳妥的是使用框架提供的数据目录能力（如规范中的 StarTools.get_data_dir()）管理可写文件路径。

10. 概率参数未做边界约束（逻辑边界）

    self.prob = float(config.get('prob', 100.0)) / 100.0

    若配置超出范围（如 -10 或 200），会导致触发行为异常（永不触发或总触发之外的隐蔽逻辑偏差）。应 clamp 到 [0, 1]。

---

代码评审团

人物一：Linus Torvalds（直球毒舌流）
“你这 HTTP 错误处理像把汽车仪表盘贴黑胶带：看不见故障就当没故障。raise_for_status() 不加，线上出了 500 你还在那儿猜 JSON。先把失败路径做扎实，别把日志当占卜工具。”

人物二：Steve Jobs（产品体验导向）
“细节决定体验。speed_facter 这种拼写级问题会直接让‘语速调节’这个功能看起来像坏的。用户不会关心后端字段名，他们只会觉得产品不可靠。把命名和参数契约做到‘看不见的完美’。”

人物三：Tony Stark / 钢铁侠（工程效率+未来扩展）
“每次请求都 new 一个 AsyncClient？这不是智能，是重复造反应堆。复用连接池、限制 audio_url 出站目标、给 _session_state 加 TTL 清理——这三件做完，你的插件就从‘能跑’进化到‘能打’。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[astrbot-plugin-copybara]

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

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