Skip to content

fix(transcriber): kuaishou ASR API 永久下线,改为清晰错误提示 - #448

Open
EricWang1358 wants to merge 1 commit into
JefferyHcool:masterfrom
EricWang1358:fix/transcriber-kuaishou-disabled-clear-error
Open

fix(transcriber): kuaishou ASR API 永久下线,改为清晰错误提示#448
EricWang1358 wants to merge 1 commit into
JefferyHcool:masterfrom
EricWang1358:fix/transcriber-kuaishou-disabled-clear-error

Conversation

@EricWang1358

Copy link
Copy Markdown

改动概述

ix(transcriber): 快手 ASR 接口已永久下线(返回 code=501, msg=效果subtitle_generate禁用),本次改动让 KuaishouTranscriber._submit 在入口直接抛带操作指引的 RuntimeError,提示用户切换到 bcut 或 Faster Whisper。

为什么

旧实现会把一条 501 错误冒泡出去,前端只能展示「快手 API 返回错误: 未知错误」之类的笼统提示,用户无从得知是该接口本身已下线;且仍会浪费一次完整上传(若是大音频文件,网络与磁盘 IO 都白做)。
用户场景:用户在「设置 → 音频转写配置」中误选「快手 ASR」后,提交笔记生成会跑完整个下载 / 上传链路,最后才在结果页看到「未知错误」。

做了什么

  • �ackend/app/transcriber/kuaishou.py: _submit 入口直接
    aise RuntimeError(...),并清理掉随之变得不可达的旧 try 块。
  • �ackend/tests/test_kuaishou_disabled.py (新增, 2 个测试): 验证抛出错误信息包含 bcut 与 whisper 提示;并通过 monkeypatch 监测
    equests.post 不再被调用。
  • CHANGELOG.md: 新增 [Unreleased] / Fixed 段。

测试方式

  • python -m pytest backend/tests/test_kuaishou_disabled.py 2 passed

回归风险

  • 影响面:仅 �pp.transcriber.kuaishou,与该模块测试。_submit 一直失败,本次只把错误从"远端 501"前移为"本地短路"——所有看到快手 ASR 报错的用户,本次都会从「未知错误」升级为明确的操作提示,不算回归
  • 行为变化:无新增外部依赖、无 API 变更。
  • 可被撤销:若快手未来恢复该接口,只需恢复旧 _submit 实现即可,测试是新增的不会因回滚而失配。

Checklist

  • 分支命名遵循 CONTRIBUTING.md §3
  • base 指向 master(远程无 develop 分支)
  • Commit message 遵循 ype(scope): subject
  • 已自测
  • 已更新 CHANGELOG.md
  • 未夹带 secrets / 大型二进制
  • 单 PR 不跨多个工作区做无关改动

快手官方已永久关闭 ai.kuaishou.com/api/effects/subtitle_generate
接口,所有调用都返回 code=501 msg='效果subtitle_generate禁用'。
旧实现默默把 501 错误冒泡出去,前端只能展示'快手 API 返回错误: 未知错误',
用户无法定位到是该接口本身已下线,还会浪费上传带宽。

本次改动:
- KuaishouTranscriber._submit 直接在入口处 raise RuntimeError,
  提示用户去设置里把转写引擎切到 bcut 或 Faster Whisper
- 移除 _submit 里现已无用的 try 块
- 新增 test_kuaishou_disabled.py 覆盖短路行为 + 不发网络请求
- CHANGELOG 加 [Unreleased] 段
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant