fix: 删除封面生成多余logger调用+增加source_edit_plan_id查找兜底 #1372

Merged

xiaoxia merged 3 commits from fix/cover-500-logger-and-source-plan-fallback into develop 2026-08-15 10:09:16 +08:00

Conversation 5 Commits 3 Files Changed 2

+261 -4

xiaoxia commented 2026-08-15 09:41:02 +08:00

Owner

Copy Link

Copy Source

修复内容

修复1: 删除多余 logger 调用

generation_cover.py 步骤2中存在 logger.info(plan_id, generation_task_id) — 没有格式化字符串，会导致 TypeError → 500 错误。

修复2: 新增 source_edit_plan_id 查找兜底

在步骤2（generation_task_id 查找）和步骤3（user+template 查找）之间，新增按 plan_id 作为 source_edit_plan_id 精确匹配关联的已完成预览任务。

修复3: 确认排序

list_by_source_edit_plan 已有 order_by(created_at.desc())，无需修改。

测试

• 新增 4 个单元测试覆盖 source_edit_plan_id 兜底逻辑
• 验证多余 logger 调用已删除

## 修复内容 ### 修复1: 删除多余 logger 调用 `generation_cover.py` 步骤2中存在 `logger.info(plan_id, generation_task_id)` — 没有格式化字符串，会导致 TypeError → 500 错误。 ### 修复2: 新增 source_edit_plan_id 查找兜底 在步骤2（generation_task_id 查找）和步骤3（user+template 查找）之间，新增按 `plan_id` 作为 `source_edit_plan_id` 精确匹配关联的已完成预览任务。 ### 修复3: 确认排序 `list_by_source_edit_plan` 已有 `order_by(created_at.desc())`，无需修改。 ## 测试 - 新增 4 个单元测试覆盖 source_edit_plan_id 兜底逻辑 - 验证多余 logger 调用已删除

xiaoxia added 1 commit 2026-08-15 09:41:02 +08:00

fix: 删除封面生成多余logger调用+增加source_edit_plan_id查找兜底 ... 74d536db15

修复1: 删除步骤2中多余的 logger.info(plan_id, generation_task_id)
        无格式化字符串的调用会导致 TypeError 500
修复2: 步骤2和步骤3之间新增 source_edit_plan_id 查找逻辑
        通过 plan_id 精确匹配关联的已完成预览任务
修复3: list_by_source_edit_plan 已有 order_by(created_at.desc()) 无需修改

新增4个单元测试覆盖新逻辑

gitea-actions bot commented 2026-08-15 09:42:31 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-15 09:45:19 +08:00

style: auto-format with black + isort + prettier [skip ci-format-check] 8840672c16

auto-approve-bot approved these changes 2026-08-15 09:50:31 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-15 09:50:31 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-08-15 10:01:53 +08:00

test: 修复 generation_cover 测试 mock 路径 ... a5f34b9a47

- run_generate_cover 需要 patch packages.shared.ai_service 而非 route 模块
  (因为代码使用函数内 inline import)
- 补充 test_step25_skips 和 test_step25_exception 缺失的 run_generate_cover mock

auto-approve-bot commented 2026-08-15 10:04:12 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1372

⚠️ 问题（0个需要修改）

💡 建议（2个可选）

1. apps/api/app/api/routes/generation_cover.py 第125-126行：循环内重复创建对象。建议将 video_repo 和 use_case 的实例化移至 for 循环外部。虽然逻辑上 break 会提前终止，但在循环中重复创建对象属于不必要的开销，且不符合最佳实践。
2. tests/unit/test_generation_cover.py 第239行：测试用例依赖源码字符串匹配。使用 inspect.getsource 并硬编码缩进和换行符来检查代码，极易因代码格式化工具（如 Black、Ruff）的运行或微小的格式调整导致测试失败，建议改为验证行为或使用更鲁棒的检查方式。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能

---

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

## 代码审查结果 - PR #1372 ### ⚠️ 问题（0个需要修改） ### 💡 建议（2个可选） 1. **apps/api/app/api/routes/generation_cover.py 第125-126行**：循环内重复创建对象。建议将 `video_repo` 和 `use_case` 的实例化移至 `for` 循环外部。虽然逻辑上 `break` 会提前终止，但在循环中重复创建对象属于不必要的开销，且不符合最佳实践。 2. **tests/unit/test_generation_cover.py 第239行**：测试用例依赖源码字符串匹配。使用 `inspect.getsource` 并硬编码缩进和换行符来检查代码，极易因代码格式化工具（如 Black、Ruff）的运行或微小的格式调整导致测试失败，建议改为验证行为或使用更鲁棒的检查方式。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-15 02:04:12 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 206d517a91 into develop 2026-08-15 10:09:16 +08:00

gitea-actions bot commented 2026-08-15 10:10:09 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

xiaoxia referenced this pull request 2026-08-15 11:31:11 +08:00

fix: 统一封面管道增加cover_url多步fallback查找，修复500错误 #1373

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