fix(worker): P1 修复任务状态覆盖 + 封面metadata属性名错误 #1471

Merged

auto-approve-bot merged 2 commits from fix/p1-task-status-cover-metadata into develop 2026-08-23 19:11:01 +08:00

Conversation 5 Commits 2 Files Changed 2

+96 -9

xiaoxia commented 2026-08-23 18:56:30 +08:00

Owner

Copy Link

Copy Source

P1 紧急修复

Bug 1: 任务状态被过期内存对象覆盖回 pending

gen_task 在任务开头加载(status=pending)，中间 _update_task_status 用独立 session 把 DB 改成 running，但内存对象仍为 pending。保存 asset_analyses 时 _repo.update(gen_task) 整体写回，把 status 覆盖回 pending → mark_completed 报非法状态转换 pending→completed。

修复：独立 session 查询最新模型，只更新 extra_meta 字段，删除 _repo.update(gen_task)。

Bug 2: 封面候选帧误用 SQLAlchemy 保留属性 .metadata

GenerationTaskModel 的 ORM 属性名是 extra_meta（DB 列名为 metadata）。代码两处写 _cover_model.metadata 实际赋值给 SQLAlchemy 声明基类保留的 MetaData 对象，持久化完全无效，封面候选帧丢失。

修复：两处 .metadata 全部改为 .extra_meta。

测试

• 新增 tests/unit/test_worker_cover_meta_and_status.py（5 个测试）
• 容器内 8 个相关单测全部通过
• Python 语法 + runtime import + task 签名验证通过

## P1 紧急修复 ### Bug 1: 任务状态被过期内存对象覆盖回 pending gen_task 在任务开头加载(status=pending)，中间 _update_task_status 用独立 session 把 DB 改成 running，但内存对象仍为 pending。保存 asset_analyses 时 _repo.update(gen_task) 整体写回，把 status 覆盖回 pending → mark_completed 报非法状态转换 pending→completed。 修复：独立 session 查询最新模型，只更新 extra_meta 字段，删除 _repo.update(gen_task)。 ### Bug 2: 封面候选帧误用 SQLAlchemy 保留属性 .metadata GenerationTaskModel 的 ORM 属性名是 extra_meta（DB 列名为 metadata）。代码两处写 _cover_model.metadata 实际赋值给 SQLAlchemy 声明基类保留的 MetaData 对象，持久化完全无效，封面候选帧丢失。 修复：两处 .metadata 全部改为 .extra_meta。 ### 测试 - 新增 tests/unit/test_worker_cover_meta_and_status.py（5 个测试） - 容器内 8 个相关单测全部通过 - Python 语法 + runtime import + task 签名验证通过

xiaoxia added 1 commit 2026-08-23 18:56:30 +08:00

fix(worker): P1 修复任务状态被过期对象覆盖回pending + 封面metadata属性名错误 ... 084c0494b7

Bug1: asset_analyses 持久化时使用任务开头加载的 gen_task 内存对象
(status=pending) 整体 _repo.update()，把已被 _update_task_status 改为
running 的 DB status 覆盖回 pending，导致 mark_completed 报非法状态转换。
改为独立 session 查询最新模型，只更新 extra_meta 字段。

Bug2: 封面候选帧持久化使用 _cover_model.metadata（SQLAlchemy 声明基类
保留的 MetaData 对象），赋值无效；ORM 属性名应为 extra_meta（DB列名
为 metadata）。两处全部修正。

新增 5 个防回归单测覆盖两个 Bug。

gitea-actions bot commented 2026-08-23 18:58:48 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot added 1 commit 2026-08-23 18:59:16 +08:00

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

auto-approve-bot commented 2026-08-23 19:01:17 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1471

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

（无）

💡 建议（2个可选）

1. apps/worker/worker_app/tasks/generation.py 第1686行：变量 _m 命名含义不明确，建议重命名为 task_model 或 db_task 以提高代码可读性。
2. apps/worker/worker_app/tasks/generation.py 第1683-1685行：建议确认将 GenerationTaskModel 的 import 放在函数内部是否为了规避循环依赖。如果没有循环依赖问题，建议移至文件顶部统一管理，以减少函数调用时的开销。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无隐患

【阻塞级判定】

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

📊 审查概览

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

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

无

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

1. [apps/worker/worker_app/tasks/generation.py:1686] 变量命名可读性

   • 具体内容：代码中使用 _m 作为查询出的模型对象变量名，语义不明。建议改为更具描述性的名称，如 current_task 或 task_model，以便后续维护者快速理解其用途。

2. [apps/worker/worker_app/tasks/generation.py:1683-1685] Import 位置优化

   • 具体内容：GenerationTaskModel 的导入语句位于函数内部。虽然这可以解决某些循环依赖问题，但会增加每次函数调用的开销。如果并非必须放在此处，建议移至模块顶部。

✅ 良好实践

1. 修复数据陈旧问题：通过创建独立的 DB Session (_meta_session) 重新查询并更新 extra_meta，有效避免了使用内存中过期的 gen_task 对象覆盖数据库最新状态（如 status）的严重 Bug。
2. 修复属性误用：将 _cover_model.metadata 修正为 _cover_model.extra_meta，正确区分了 SQLAlchemy 保留属性与业务自定义字段，防止了潜在的数据写入错误。
3. 资源管理规范：新增的 DB Session 操作使用了 try...finally 结构确保 session 正确关闭，避免了连接泄漏风险。
4. 完善的回归测试：新增的单元测试针对性强，覆盖了“属性误用”和“陈旧对象更新”两个核心修复点，有效防止问题回退。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-23 11:01:17 | 模型:}

## 代码审查结果 - PR #1471 ### ⚠️ 问题（0个需要修改） （无） ### 💡 建议（2个可选） 1. **apps/worker/worker_app/tasks/generation.py 第1686行**：变量 `_m` 命名含义不明确，建议重命名为 `task_model` 或 `db_task` 以提高代码可读性。 2. **apps/worker/worker_app/tasks/generation.py 第1683-1685行**：建议确认将 `GenerationTaskModel` 的 import 放在函数内部是否为了规避循环依赖。如果没有循环依赖问题，建议移至文件顶部统一管理，以减少函数调用时的开销。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无隐患 ### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/worker/worker_app/tasks/generation.py:1686] 变量命名可读性** - 具体内容：代码中使用 `_m` 作为查询出的模型对象变量名，语义不明。建议改为更具描述性的名称，如 `current_task` 或 `task_model`，以便后续维护者快速理解其用途。 2. **[apps/worker/worker_app/tasks/generation.py:1683-1685] Import 位置优化** - 具体内容：`GenerationTaskModel` 的导入语句位于函数内部。虽然这可以解决某些循环依赖问题，但会增加每次函数调用的开销。如果并非必须放在此处，建议移至模块顶部。 ### ✅ 良好实践 1. **修复数据陈旧问题**：通过创建独立的 DB Session (`_meta_session`) 重新查询并更新 `extra_meta`，有效避免了使用内存中过期的 `gen_task` 对象覆盖数据库最新状态（如 status）的严重 Bug。 2. **修复属性误用**：将 `_cover_model.metadata` 修正为 `_cover_model.extra_meta`，正确区分了 SQLAlchemy 保留属性与业务自定义字段，防止了潜在的数据写入错误。 3. **资源管理规范**：新增的 DB Session 操作使用了 `try...finally` 结构确保 session 正确关闭，避免了连接泄漏风险。 4. **完善的回归测试**：新增的单元测试针对性强，覆盖了“属性误用”和“陈旧对象更新”两个核心修复点，有效防止问题回退。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-23 11:01:17 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-08-23 19: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-23 19:05:31 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 9fd1246c6b into develop 2026-08-23 19:11:01 +08:00

auto-approve-bot deleted branch fix/p1-task-status-cover-metadata 2026-08-23 19:11:02 +08:00

gitea-actions bot commented 2026-08-23 19:12:05 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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