feat(#632): 一键生成输出分辨率可配置 #749

Merged

auto-approve-bot merged 1 commits from feat/oneclick-resolution-config into develop 2026-07-23 10:27:37 +08:00

Conversation 5 Commits 1 Files Changed 9

+87 -6

xiaoxia commented 2026-07-23 10:22:44 +08:00

Owner

Copy Link

Copy Source

变更内容

修复 #632 — 一键生成输出分辨率可配置

核心改动

• DB: generation_tasks 表新增 resolution 字段（migration 051，幂等）
• API: CreateGenerationTaskRequest / GenerationTaskResponse 新增 resolution 字段
• Application: CreateGenerationTaskCommand 透传 resolution
• Domain: GenerationTask 实体新增 resolution 属性
• Repository: SQLAlchemy 层 create/update/to_domain 全链路透传
• Worker: _render_video 接收 resolution 参数，优先级：用户指定 > 模板配置 > 默认 1280x720
• 重试: 重试任务保留原分辨率设置

验证

• 125 个 generation 相关单测全部通过
• migration 幂等（PostgreSQL 下先查列是否存在）
• 向后兼容：resolution 为空时完全等价于旧行为

## 变更内容 修复 #632 — 一键生成输出分辨率可配置 ### 核心改动 - **DB**: generation_tasks 表新增 resolution 字段（migration 051，幂等） - **API**: CreateGenerationTaskRequest / GenerationTaskResponse 新增 resolution 字段 - **Application**: CreateGenerationTaskCommand 透传 resolution - **Domain**: GenerationTask 实体新增 resolution 属性 - **Repository**: SQLAlchemy 层 create/update/to_domain 全链路透传 - **Worker**: _render_video 接收 resolution 参数，优先级：用户指定 > 模板配置 > 默认 1280x720 - **重试**: 重试任务保留原分辨率设置 ### 验证 - 125 个 generation 相关单测全部通过 - migration 幂等（PostgreSQL 下先查列是否存在） - 向后兼容：resolution 为空时完全等价于旧行为

xiaoxia added 1 commit 2026-07-23 10:22:45 +08:00

feat(#632): 一键生成输出分辨率可配置 ... 66dd02e03a

- generation_tasks 表新增 resolution 字段（migration 051）
- API 请求/响应新增 resolution 参数，格式 WIDTHxHEIGHT（如 1080x1920）
- 渲染优先级：用户指定 > 模板配置 > 默认 1280x720
- 领域模型 + Command + Repository 全链路透传
- 重试任务保留原分辨率
- 125 个相关单测全绿

Closes #632

auto-approve-bot approved these changes 2026-07-23 10:25:28 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-23 10:25:28 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

gitea-actions bot commented 2026-07-23 10:27:04 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot merged commit 9d3a8852e5 into develop 2026-07-23 10:27:37 +08:00

auto-approve-bot deleted branch feat/oneclick-resolution-config 2026-07-23 10:27:37 +08:00

auto-approve-bot commented 2026-07-23 10:28:39 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #749

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

1. apps/api/app/schemas/generation_task.py 第49行：缺少对 resolution 字段格式的校验。

   • 问题描述：当前仅通过 description 描述了格式要求（"WIDTHxHEIGHT"），但代码层面没有进行正则校验。用户可以传入任意字符串（如 "abc"、"0x0" 或 "-1x-1"），这些无效数据会被传递到底层渲染引擎，可能导致渲染任务失败或资源浪费。
   • 修改建议：添加 @field_validator 或使用 regex 参数限制格式，例如 pattern=r"^\d+x\d+$"，并增加合理的数值范围检查（如 10-7680）。

2. alembic/versions/051_generation_task_resolution.py 第36行：数据库迁移脚本在非 PostgreSQL 数据库下不具备幂等性。

   • 问题描述：代码仅对 PostgreSQL 执行了列存在性检查。如果该迁移在 SQLite（常用于测试环境）或其他数据库上重复执行，由于没有检查列是否存在，直接调用 op.add_column 会抛出 "duplicate column name" 错误导致迁移失败。
   • 修改建议：使用 op.execute 配合 ALTER TABLE ... IF NOT EXISTS（如果DB支持）或使用 batch_alter_table 配合 sa.inspect 来实现跨数据库的幂等检查，或者统一使用 try-except 捕获重复列异常。

💡 建议（1个可选）

1. packages/domain/generation_task.py 第139行：建议增强 strip() 操作的健壮性。
   • 具体内容：虽然 Pydantic 层面通常保证非 None，但在 Domain 层核心逻辑中，直接调用 resolution.strip() 存在理论上的 AttributeError 风险（如果被非 API 入口直接调用并传入 None）。建议修改为 (resolution or "").strip() 以确保绝对安全。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 10:28:39 | 模型:}

## 代码审查结果 - PR #749 ### ⚠️ 问题（2个需要修改） 1. **apps/api/app/schemas/generation_task.py 第49行**：缺少对 `resolution` 字段格式的校验。 - 问题描述：当前仅通过 `description` 描述了格式要求（"WIDTHxHEIGHT"），但代码层面没有进行正则校验。用户可以传入任意字符串（如 "abc"、"0x0" 或 "-1x-1"），这些无效数据会被传递到底层渲染引擎，可能导致渲染任务失败或资源浪费。 - 修改建议：添加 `@field_validator` 或使用 `regex` 参数限制格式，例如 `pattern=r"^\d+x\d+$"`，并增加合理的数值范围检查（如 10-7680）。 2. **alembic/versions/051_generation_task_resolution.py 第36行**：数据库迁移脚本在非 PostgreSQL 数据库下不具备幂等性。 - 问题描述：代码仅对 PostgreSQL 执行了列存在性检查。如果该迁移在 SQLite（常用于测试环境）或其他数据库上重复执行，由于没有检查列是否存在，直接调用 `op.add_column` 会抛出 "duplicate column name" 错误导致迁移失败。 - 修改建议：使用 `op.execute` 配合 `ALTER TABLE ... IF NOT EXISTS`（如果DB支持）或使用 `batch_alter_table` 配合 `sa.inspect` 来实现跨数据库的幂等检查，或者统一使用 try-except 捕获重复列异常。 ### 💡 建议（1个可选） 1. **packages/domain/generation_task.py 第139行**：建议增强 `strip()` 操作的健壮性。 - 具体内容：虽然 Pydantic 层面通常保证非 None，但在 Domain 层核心逻辑中，直接调用 `resolution.strip()` 存在理论上的 `AttributeError` 风险（如果被非 API 入口直接调用并传入 None）。建议修改为 `(resolution or "").strip()` 以确保绝对安全。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 10:28:39 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-07-23 10:31:55 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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