refactor(#781): 统一应用层异常定义,消除重复异常类 #802

Merged
xiaoxia merged 1 commits from refactor/unified-exceptions into develop 2026-07-24 08:03:11 +08:00
Owner

改动内容

统一应用层重复的异常类定义,消除各模块分散定义的相同异常。

通用异常(domain层)

新增 packages/domain/exceptions.py

  • DomainError - 领域层异常基类
  • NotFoundError - 原5个模块(recipe/template/title_library/video_share/voice_library)各定义一份,完全相同
  • ValidationError - 原template模块定义
  • QuotaExceededError - 原title_library/voice_library各定义一份,完全相同

模块内重复

新增 packages/application/tts_job/exceptions.py

  • TTSJobNotFoundError - 原 use_cases.py 和 workflow.py 各定义一份,完全相同

向后兼容

  • 所有异常类名、继承关系、行为完全不变
  • 只是 import 来源从本地定义改为统一模块
  • 所有 except 捕获不受影响

测试结果

  • 4383 passed, 8 skipped

影响范围

  • 9个文件,+49/-54,净减5行
## 改动内容 统一应用层重复的异常类定义,消除各模块分散定义的相同异常。 ### 通用异常(domain层) 新增 `packages/domain/exceptions.py`: - `DomainError` - 领域层异常基类 - `NotFoundError` - 原5个模块(recipe/template/title_library/video_share/voice_library)各定义一份,完全相同 - `ValidationError` - 原template模块定义 - `QuotaExceededError` - 原title_library/voice_library各定义一份,完全相同 ### 模块内重复 新增 `packages/application/tts_job/exceptions.py`: - `TTSJobNotFoundError` - 原 use_cases.py 和 workflow.py 各定义一份,完全相同 ### 向后兼容 - 所有异常类名、继承关系、行为完全不变 - 只是 import 来源从本地定义改为统一模块 - 所有 except 捕获不受影响 ## 测试结果 - 4383 passed, 8 skipped ## 影响范围 - 9个文件,+49/-54,净减5行
xiaoxia added 1 commit 2026-07-24 00:40:04 +08:00
refactor(#780): 统一应用层异常定义,消除重复异常类
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 5s
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m1s
CI/CD Pipeline / Validate - Code Quality (pull_request) Failing after 1m28s
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 39s
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 56s
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 17s
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 36s
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 12s
AI Code Review / AI Code Review (pull_request) Successful in 2m37s
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m13s
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 4m41s
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 10m9s
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
CI/CD Pipeline / Integration Tests (pull_request) Successful in 2m29s
CI/CD Pipeline / Unit Tests (pull_request) Successful in 3m9s
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 45s
08d4f4bd07
- 新增 packages/domain/exceptions.py:定义领域层通用异常基类
  - DomainError 基类
  - NotFoundError(原5个模块各定义一份,完全相同)
  - ValidationError(原template模块定义)
  - QuotaExceededError(原title_library/voice_library各定义一份)
- 新增 packages/application/tts_job/exceptions.py:TTSJobNotFoundError 统一到模块公共文件
  - 消除 use_cases.py 和 workflow.py 中的重复定义
- 各模块改为从统一位置 import,向后兼容完全保持
- 9个文件,+77/-54,净减23行
Collaborator

代码审查结果 - PR #802

⚠️ 问题(0个需要修改)

💡 建议(0个可选)


格式检查通过 | 逻辑审查通过 | 性能无问题


🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 17:06:28 | 模型:

## 代码审查结果 - PR #802 ### ⚠️ 问题(0个需要修改) 无 ### 💡 建议(0个可选) 无 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无问题 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 17:06:28 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
xiaoxia merged commit c74f1a82d9 into develop 2026-07-24 08:03:11 +08:00
xiaoxia deleted branch refactor/unified-exceptions 2026-07-24 08:03:11 +08:00

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #802 已关闭或合并,对应的预览环境已被清理。 > 如有需要,可以重新打开 PR 来重新生成预览环境。
Sign in to join this conversation.