fix: HEVC 转码条件解耦 + 缩略图函数补全 + extra_meta 字段 #1452

Merged

auto-approve-bot merged 6 commits from fix/ingest-hevc-condition-and-imports into develop 2026-08-20 21:41:01 +08:00

Conversation 5 Commits 6 Files Changed 8

+207 -368

xiaoxia commented 2026-08-20 16:47:13 +08:00

Owner

Copy Link

Copy Source

修复内容

Fix 1 — HEVC 转码条件（P0）

• 文件:
• 改动: HEVC 转码条件去掉 ，不再依赖缩略图提取是否成功

Fix 2 — 缩略图函数补全（P0）

• 文件:
• 改动: 补充 和 两个函数
• 原因: 引用了这两个函数但不存在，导致 ImportError

Fix 3 — extra_meta 字段（P1）

• 文件:
• 改动: GenerationTask dataclass 添加 字段，与 ORM 模型对齐
• mark_confirmed() 方法也增加 extra_meta 合并逻辑

Fix 4 — Code Quality

• ruff check 已全绿，无 unused-import

测试

• 49 个单元测试通过（test_ingest_hevc_transcode + test_generation_task）
• 25 个单元测试通过（test_ingest_validation）

## 修复内容 ### Fix 1 — HEVC 转码条件（P0） - **文件**: - **改动**: HEVC 转码条件去掉 ，不再依赖缩略图提取是否成功 ### Fix 2 — 缩略图函数补全（P0） - **文件**: - **改动**: 补充 和 两个函数 - **原因**: 引用了这两个函数但不存在，导致 ImportError ### Fix 3 — extra_meta 字段（P1） - **文件**: - **改动**: GenerationTask dataclass 添加 字段，与 ORM 模型对齐 - mark_confirmed() 方法也增加 extra_meta 合并逻辑 ### Fix 4 — Code Quality - ruff check 已全绿，无 unused-import ## 测试 - 49 个单元测试通过（test_ingest_hevc_transcode + test_generation_task） - 25 个单元测试通过（test_ingest_validation）

xiaoxia added 1 commit 2026-08-20 16:47:13 +08:00

fix: HEVC 转码条件解耦 + 缩略图函数补全 + extra_meta 字段 ... 72d8e5ae74

1. ingest.py: HEVC 转码不再依赖 extract_success，两个功能解耦
2. thumbnail_generator.py: 补充 generate_and_upload_thumbnail 和
   extract_and_upload_cover_frames 两个函数，修复 render_adapter.py ImportError
3. generation_task.py: 添加 extra_meta 字段，与 ORM 模型对齐

Co-Authored-By: xiaoxia <xiaoxia@agent.com>

gitea-actions bot commented 2026-08-20 16:49:10 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-20 16:52:28 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 52e53d60e3

auto-approve-bot approved these changes 2026-08-20 17:05:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-20 17:05:30 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-08-20 19:20:54 +08:00

fix: OSS 上传/下载超时从 300s 提升到 900s ... 31203613c6

4K 视频文件大（200MB+），5 分钟超时不够用，改为 15 分钟。

Co-Authored-By: xiaoxia <xiaoxia@agent.com>

xiaoxia added 1 commit 2026-08-20 20:49:14 +08:00

test: 修复 create_asset 废弃后的测试用例 ... 6a275324f5

- test_asset_dedup: 补充 storage.get_url mock
- test_create_asset_optional_project_id: 删除（测试已废弃的端点）
- test_assets_api: 集成测试改为直接插入 repository，绕过 410 端点
  TestCreateAsset 改为验证 410 响应

全量测试：14095 passed，4 failed + 14 errors 均为 develop 预存问题

Co-Authored-By: xiaoxia <xiaoxia@agent.com>

auto-approve-bot added 1 commit 2026-08-20 20:54:45 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 6c8827c507

auto-approve-bot commented 2026-08-20 20:57:00 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

1. [apps/worker/video_processing/thumbnail_generator.py: 173] 异常捕获过于宽泛

   • 具体内容：在 extract_and_upload_cover_frames 函数中使用了 except Exception as e。虽然对于生成“封面候选”列表的场景，部分失败不应中断整体流程是合理的，但捕获所有异常（包括 KeyboardInterrupt 或 SystemExit）通常是不推荐的。建议捕获更具体的异常（如 RuntimeError, IOError 等），或者在日志中增加 exc_info=True 以便排查问题。

2. [tests/integration/test_assets_api.py: 267] 测试辅助函数的健壮性不一致

   • 具体内容：新增的 _direct_insert_asset 函数中使用了 app.dependency_overrides[get_asset_repository]() 直接获取依赖，如果 get_asset_repository 未在 overrides 中注册，会抛出 KeyError。而在同一文件中的 _create_test_assets 函数（第377行）使用了 .get(..., lambda: None) 并进行了判空处理。建议统一处理方式，增加防御性判断以提高测试代码的健壮性。

✅ 良好实践

• 资源管理：thumbnail_generator.py 中使用 tempfile.NamedTemporaryFile 配合 try...finally 确保了临时文件的清理，避免了磁盘空间泄漏。
• 数据安全：generation_task.py 中新增 extra_meta 字段时，使用了 field(default_factory=dict) 避免了可变默认参数的陷阱，并在 create 和 update 方法中正确处理了 None 值和字典拷贝，防止了意外的引用共享。
• 测试重构：针对已废弃的 API（create_asset 返回 410），测试代码及时进行了重构，引入 _direct_insert_asset 辅助函数，保持了测试套件的覆盖率和有效性。

---

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

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-20 12:57:00 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/worker/video_processing/thumbnail_generator.py: 173] 异常捕获过于宽泛** - 具体内容：在 `extract_and_upload_cover_frames` 函数中使用了 `except Exception as e`。虽然对于生成“封面候选”列表的场景，部分失败不应中断整体流程是合理的，但捕获所有异常（包括 KeyboardInterrupt 或 SystemExit）通常是不推荐的。建议捕获更具体的异常（如 `RuntimeError`, `IOError` 等），或者在日志中增加 `exc_info=True` 以便排查问题。 2. **[tests/integration/test_assets_api.py: 267] 测试辅助函数的健壮性不一致** - 具体内容：新增的 `_direct_insert_asset` 函数中使用了 `app.dependency_overrides[get_asset_repository]()` 直接获取依赖，如果 `get_asset_repository` 未在 overrides 中注册，会抛出 `KeyError`。而在同一文件中的 `_create_test_assets` 函数（第377行）使用了 `.get(..., lambda: None)` 并进行了判空处理。建议统一处理方式，增加防御性判断以提高测试代码的健壮性。 ### ✅ 良好实践 - **资源管理**：`thumbnail_generator.py` 中使用 `tempfile.NamedTemporaryFile` 配合 `try...finally` 确保了临时文件的清理，避免了磁盘空间泄漏。 - **数据安全**：`generation_task.py` 中新增 `extra_meta` 字段时，使用了 `field(default_factory=dict)` 避免了可变默认参数的陷阱，并在 `create` 和 `update` 方法中正确处理了 None 值和字典拷贝，防止了意外的引用共享。 - **测试重构**：针对已废弃的 API（create_asset 返回 410），测试代码及时进行了重构，引入 `_direct_insert_asset` 辅助函数，保持了测试套件的覆盖率和有效性。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-20 12:57:00 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia added 1 commit 2026-08-20 21:25:19 +08:00

test: add extra_meta unit tests for diff coverage ... 1744ad7c45

- test_create_with_extra_meta
- test_create_without_extra_meta_defaults_empty
- test_mark_confirmed_with_extra_meta
- test_mark_confirmed_without_extra_meta_preserves_existing

Covers generation_task.py lines 325-326 to meet 40% diff coverage gate.

auto-approve-bot merged commit 161c1a61b6 into develop 2026-08-20 21:41:01 +08:00

auto-approve-bot deleted branch fix/ingest-hevc-condition-and-imports 2026-08-20 21:41:01 +08:00

gitea-actions bot commented 2026-08-20 21:42:19 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

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