fix: 确认生成兜底增强 — project_id 为空时通过 user_id 查找素材 #1338

Merged

xiaoxia merged 4 commits from fix/generate-fallback-user-assets into develop 2026-08-10 21:32:15 +08:00

Conversation 5 Commits 4 Files Changed 4

+435 -70

xiaoxia commented 2026-08-10 20:19:28 +08:00

Owner

Copy Link

Copy Source

问题

模板编辑器直接创建的草稿 plan 没有 project_id 和 config.asset_ids，导致：

• _auto_fallback_auto_material_mode 因 project_id 为空直接 return
• can_generate 发现 clips 都无 asset_id，报错"没有可渲染的就绪片段"

修复

• asset_repository 新增 find_ready_videos_by_user(user_id) 方法
• _auto_fallback_auto_material_mode 增加策略2：project_id 为空时通过 uploaded_by_user_id 查找用户上传的就绪视频
• generation.py 传递 current_user.user.id 给兜底函数

影响文件

• packages/adapters/sqlalchemy_impl/asset_repository.py
• apps/api/app/api/routes/templates_editor/_fallback.py
• apps/api/app/api/routes/templates_editor/generation.py

## 问题 模板编辑器直接创建的草稿 plan 没有 `project_id` 和 `config.asset_ids`，导致： - `_auto_fallback_auto_material_mode` 因 project_id 为空直接 return - `can_generate` 发现 clips 都无 asset_id，报错"没有可渲染的就绪片段" ## 修复 - `asset_repository` 新增 `find_ready_videos_by_user(user_id)` 方法 - `_auto_fallback_auto_material_mode` 增加策略2：project_id 为空时通过 `uploaded_by_user_id` 查找用户上传的就绪视频 - `generation.py` 传递 `current_user.user.id` 给兜底函数 ## 影响文件 - `packages/adapters/sqlalchemy_impl/asset_repository.py` - `apps/api/app/api/routes/templates_editor/_fallback.py` - `apps/api/app/api/routes/templates_editor/generation.py`

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

fix: 确认生成兜底增强 — project_id 为空时通过 user_id 查找素材 ... e09fdda74c

根因：模板编辑器直接创建的草稿 plan 没有 project_id 和 config.asset_ids，
导致 _auto_fallback_auto_material_mode 跳过素材分配，
can_generate 报 "没有可渲染的就绪片段"。

修复：
- asset_repository 新增 find_ready_videos_by_user 方法
- _auto_fallback_auto_material_mode 增加策略2：project_id 为空时
  通过 uploaded_by_user_id 查找用户上传的就绪视频
- generation.py 传递 current_user.user.id 给兜底函数

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

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia referenced this issue from a commit 2026-08-10 21:17:40 +08:00

test: 补充 PR #1338 兜底增强单测 (find_ready_videos_by_user + user_id 策略2)

xiaoxia added 1 commit 2026-08-10 21:17:40 +08:00

test: 补充 PR #1338 兜底增强单测 (find_ready_videos_by_user + user_id 策略2) c528bc50d3

auto-approve-bot added 1 commit 2026-08-10 21:22:25 +08:00

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

xiaoxia added 1 commit 2026-08-10 21:24:20 +08:00

style: black + isort 格式化修复 8d260f16f2

auto-approve-bot commented 2026-08-10 21:29:04 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

1. [apps/api/app/api/routes/templates_editor/generation.py:79] current_user 属性访问确认
   • 具体内容：代码中使用了 current_user.user.id。请确认 current_user 依赖注入返回的对象结构。在标准的 FastAPI get_current_user 依赖中通常直接返回 User 对象，即 current_user.id。如果 current_user 是一个包含 user 属性的包装类（如 TokenData 或自定义 AuthContext），则当前代码是正确的。建议确认该属性路径是否与项目其他路由保持一致。

✅ 良好实践

1. 防御性编程：在调用 asset_repo.find_ready_videos_by_user 前使用了 hasattr 检查，增强了代码对不同 Repository 实现的兼容性。
2. 日志脱敏：在日志中输出 user_id 时使用了切片 [:8] + "..."，有效避免了敏感信息（完整用户ID）在日志中泄露。
3. 测试覆盖充分：新增的单元测试覆盖了策略优先级、空列表、边界条件等场景，确保了兜底逻辑的可靠性。
4. 代码重构：将素材分配的循环逻辑提取到策略判断之外，消除了代码重复，逻辑更加清晰。

---

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

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/api/app/api/routes/templates_editor/generation.py:79] current_user 属性访问确认** - 具体内容：代码中使用了 `current_user.user.id`。请确认 `current_user` 依赖注入返回的对象结构。在标准的 FastAPI `get_current_user` 依赖中通常直接返回 User 对象，即 `current_user.id`。如果 `current_user` 是一个包含 `user` 属性的包装类（如 `TokenData` 或自定义 `AuthContext`），则当前代码是正确的。建议确认该属性路径是否与项目其他路由保持一致。 ### ✅ 良好实践 1. **防御性编程**：在调用 `asset_repo.find_ready_videos_by_user` 前使用了 `hasattr` 检查，增强了代码对不同 Repository 实现的兼容性。 2. **日志脱敏**：在日志中输出 `user_id` 时使用了切片 `[:8] + "..."`，有效避免了敏感信息（完整用户ID）在日志中泄露。 3. **测试覆盖充分**：新增的单元测试覆盖了策略优先级、空列表、边界条件等场景，确保了兜底逻辑的可靠性。 4. **代码重构**：将素材分配的循环逻辑提取到策略判断之外，消除了代码重复，逻辑更加清晰。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-10 13:29:04 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit aff62b29fc into develop 2026-08-10 21:32:15 +08:00

gitea-actions bot commented 2026-08-10 21:33:14 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1338 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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#1338