fix(ci): 修复develop分支CI核心链路失败（Code Quality + Unit Tests + Frontend） #1129

Merged

xiaoxia merged 7 commits from fix/develop-ci-failures-20260729 into develop 2026-07-29 12:12:37 +08:00

Conversation 3 Commits 7 Files Changed 7

+55 -13

xiaoxia commented 2026-07-29 10:51:33 +08:00

Owner

Copy Link

Copy Source

修复内容

develop分支CI恢复后发现3个核心job失败，本次全部修复：

1. Code Quality (63个错误)

• F401: 删除 speed_engine.py 中2个未使用的import（MAX_SPEED, MIN_SPEED）
• B011 (61个): 均在测试文件中，通过 pyproject.toml 配置 per-file-ignores 忽略tests目录的B011规则

2. Unit Tests (14个失败)

• packages/shared/url_security.py: 补全 ALLOWED_VIDEO_MIME_TYPES 的re-export（从domain层导入）
• video_processing/__init__.py: 导出 url_security 子模块，修复 video_processing.url_security 访问失败
• test_classification.py: 更新AssetLibraryKind测试（2→3，新增IMAGE类型）
• test_multi_track_subtitle_concat.py: 修复 POSITION_ALIGNMENT 导入路径（从packages.domain.subtitle_style导入）

3. Frontend Unit Tests (1个失败)

• useGenerateVideo.test.ts: 更新smoke test，使用实际存在的导出（getGenerationPhase, extractBackendError, buildVoiceConfig）

## 修复内容 develop分支CI恢复后发现3个核心job失败，本次全部修复： ### 1. Code Quality (63个错误) - **F401**: 删除 `speed_engine.py` 中2个未使用的import（MAX_SPEED, MIN_SPEED） - **B011 (61个)**: 均在测试文件中，通过 `pyproject.toml` 配置 `per-file-ignores` 忽略tests目录的B011规则 ### 2. Unit Tests (14个失败) - **packages/shared/url_security.py**: 补全 `ALLOWED_VIDEO_MIME_TYPES` 的re-export（从domain层导入） - **video_processing/__init__.py**: 导出 `url_security` 子模块，修复 `video_processing.url_security` 访问失败 - **test_classification.py**: 更新AssetLibraryKind测试（2→3，新增IMAGE类型） - **test_multi_track_subtitle_concat.py**: 修复 `POSITION_ALIGNMENT` 导入路径（从packages.domain.subtitle_style导入） ### 3. Frontend Unit Tests (1个失败) - **useGenerateVideo.test.ts**: 更新smoke test，使用实际存在的导出（getGenerationPhase, extractBackendError, buildVoiceConfig）

xiaoxia added 1 commit 2026-07-29 10:51:34 +08:00

fix(ci): 修复develop分支CI核心链路失败 ... 156390b676

修复项：
1. Code Quality: 修复F401未使用import + 配置tests目录忽略B011(assert-false)
2. Unit Tests(14个失败):
   - shared.url_security补全ALLOWED_VIDEO_MIME_TYPES re-export
   - video_processing.__init__导出url_security子模块
   - test_classification更新AssetLibraryKind(新增IMAGE)
   - test_multi_track_subtitle_concat修复POSITION_ALIGNMENT导入路径
3. Frontend Unit Tests: 更新useGenerateVideo测试为实际存在的导出

gitea-actions bot commented 2026-07-29 10:53:54 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-07-29 10:54:19 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 846e267b11

xiaoxia added 1 commit 2026-07-29 10:57:40 +08:00

fix(ci): 补全 ALLOWED_VIDEO_MIME_TYPES 导出，修复 F401 未使用导入 79f4865a88

xiaoxia added 1 commit 2026-07-29 11:15:38 +08:00

fix(ci): speed_engine添加向后兼容的模块级导出（MAX_SPEED/MIN_SPEED/纯函数） cfc7f2435d

xiaoxia added 1 commit 2026-07-29 11:26:24 +08:00

fix: 删除speed_engine未使用的MAX_SPEED/MIN_SPEED导入 928b6e88ff

xiaoxia added 1 commit 2026-07-29 11:48:50 +08:00

fix(ci): 添加MAX_SPEED/MIN_SPEED向后兼容导出（带noqa防止被auto-format删除） c327ddd2c3

auto-approve-bot added 1 commit 2026-07-29 11:52:05 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 91c5e4eab7

auto-approve-bot commented 2026-07-29 11:55:36 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

无

✅ 良好实践

1. 重构策略稳健：在 speed_engine.py 中，通过添加 noqa: F401 和模块级包装函数来维持向后兼容性，这是一种在不破坏现有调用方的情况下进行底层重构的优秀实践。
2. 领域模型清晰化：将 POSITION_ALIGNMENT、AssetLibraryKind 和 ALLOWED_VIDEO_MIME_TYPES 等常量和类型迁移至 packages.domain，符合领域驱动设计（DDD）原则，有助于解耦业务逻辑与实现细节。
3. 测试同步更新：所有相关的单元测试和烟雾测试都随着重构进行了更新（如 test_classification.py 和 useGenerateVideo.test.ts），确保了代码变更的可靠性。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 03:55:36 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **重构策略稳健**：在 `speed_engine.py` 中，通过添加 `noqa: F401` 和模块级包装函数来维持向后兼容性，这是一种在不破坏现有调用方的情况下进行底层重构的优秀实践。 2. **领域模型清晰化**：将 `POSITION_ALIGNMENT`、`AssetLibraryKind` 和 `ALLOWED_VIDEO_MIME_TYPES` 等常量和类型迁移至 `packages.domain`，符合领域驱动设计（DDD）原则，有助于解耦业务逻辑与实现细节。 3. **测试同步更新**：所有相关的单元测试和烟雾测试都随着重构进行了更新（如 `test_classification.py` 和 `useGenerateVideo.test.ts`），确保了代码变更的可靠性。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 03:55:36 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 76fdab4f63 into develop 2026-07-29 12:12:37 +08:00

xiaoxia deleted branch fix/develop-ci-failures-20260729 2026-07-29 12:12:37 +08:00

gitea-actions bot commented 2026-07-29 12:13:09 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

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#1129