fix: 修复 voices.py 中 3 处 ruff B904 错误，解除 CI Validate-Style 阻塞 #1667

Merged

xiaoxia merged 1 commits from fix/ruff-b904-voices into develop 2026-09-03 20:35:07 +08:00

Conversation 3 Commits 1 Files Changed 1

+3 -3

xiaoxia commented 2026-09-03 19:53:08 +08:00

Owner

Copy Link

Copy Source

问题

develop 分支上的 Validate - Style CI 检查失败，阻塞了所有新 PR 合并。

根因：PR #1654 引入的 apps/api/app/api/routes/voices.py 中有 3 处 except 块中的 raise 没有使用 from，触发 ruff B904 规则。

修复

except subprocess.TimeoutExpired:
-   raise HTTPException(...)
+   raise HTTPException(...) from None

except Exception as e:
-   raise HTTPException(...)
+   raise HTTPException(...) from e

except IntegrityError:
-   raise HTTPException(...)
+   raise HTTPException(...) from None

影响

• 修复后 Validate - Style 将通过，解除 PR #1665、#1666 等被阻塞的 PR
• 仅改 3 行，无功能变更

## 问题 develop 分支上的 `Validate - Style` CI 检查失败，阻塞了所有新 PR 合并。 根因：PR #1654 引入的 `apps/api/app/api/routes/voices.py` 中有 3 处 `except` 块中的 `raise` 没有使用 `from`，触发 ruff B904 规则。 ## 修复 ``` except subprocess.TimeoutExpired: - raise HTTPException(...) + raise HTTPException(...) from None except Exception as e: - raise HTTPException(...) + raise HTTPException(...) from e except IntegrityError: - raise HTTPException(...) + raise HTTPException(...) from None ``` ## 影响 - 修复后 `Validate - Style` 将通过，解除 PR #1665、#1666 等被阻塞的 PR - 仅改 3 行，无功能变更

gitea-actions bot commented 2026-09-03 19:55:48 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1667
预览链接	https://pr-1667.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1667 | | 预览链接 | [https://pr-1667.preview.xiaoxiajianji.com](https://pr-1667.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

xiaoxia added 1 commit 2026-09-03 20:31:00 +08:00

fix: 修复 voices.py 中 3 处 ruff B904 错误（raise without from） 91dd81073d

xiaoxia force-pushed fix/ruff-b904-voices from f30a678b97 to 91dd81073d 2026-09-03 20:31:00 +08:00 Compare

auto-approve-bot commented 2026-09-03 20:32:27 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

• 是否存在阻塞级问题：否
• 阻塞级问题数量：0 个

📊 审查概览

• 整体评价：通过
• 建议级问题数量：0 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

无

✅ 良好实践

1. 显式异常链处理：在 TimeoutError 和素材库创建失败的异常处理中使用 from None，有效地隐藏了底层实现细节，防止向客户端暴露不必要的堆栈信息，符合安全最佳实践。
2. 保留调试上下文：在通用异常捕获块 except Exception as e 中使用 from e，显式保留了原始异常链，便于在服务器端日志或调试环境中追踪根本原因。
3. 日志记录规范：在捕获异常时正确使用了 logger.exception，确保错误堆栈能被正确记录。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 12:32:26 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **显式异常链处理**：在 `TimeoutError` 和素材库创建失败的异常处理中使用 `from None`，有效地隐藏了底层实现细节，防止向客户端暴露不必要的堆栈信息，符合安全最佳实践。 2. **保留调试上下文**：在通用异常捕获块 `except Exception as e` 中使用 `from e`，显式保留了原始异常链，便于在服务器端日志或调试环境中追踪根本原因。 3. **日志记录规范**：在捕获异常时正确使用了 `logger.exception`，确保错误堆栈能被正确记录。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 12:32:26 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit c7c30936a9 into develop 2026-09-03 20:35:07 +08:00

gitea-actions bot commented 2026-09-03 20:36:58 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1667 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1667 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

xiaoxia referenced this pull request 2026-09-04 23:31:28 +08:00

Release: 同步 develop 到 main（PR #1654~#1698 全量同步） #1699

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1667