fix: 素材余量四字段兼容 Asset 领域实体（真实接口恒为 None 修复） #1550

Merged

auto-approve-bot merged 2 commits from fix/asset-availability-entity-meta into develop 2026-08-30 08:21:01 +08:00

Conversation 5 Commits 2 Files Changed 2

+52 -8

xiaoxia commented 2026-08-30 08:03:07 +08:00

Owner

Copy Link

Copy Source

背景

#1549 合并后 staging 真实接口验收发现：GET /assets 与 smart-match 返回的余量四字段恒为 None（usable 恒为 True 兜底）。

根因

路由层 repository 返回的是 Asset 领域实体（packages/domain/entities.py），不是 ORM AssetModel：

• 实体的区间记录在 metadata dict 属性里（repository 与 classification_result JSON 列互转，见 asset_repository.py:133/162）；
• 实体没有 classification_result 属性；
• _read_meta 只读 classification_result 列 → AttributeError → _asset_availability_fields 异常兜底 → 四字段 None + usable=True。

单测用 SimpleNamespace 同时带了两个属性，未暴露此问题。

修复

_read_meta 双形态兼容：

• 对象有 metadata 且为 dict（领域实体）→ 直接使用；
• 否则读 classification_result JSON 字符串（ORM AssetModel 形态，tracker 内部 DB 写入路径不变）。

测试

• 新增实体形态用例（metadata dict、无 classification_result）与 ORM 形态用例对比；
• 相关 5 个测试文件 64 passed；全量 unit 13913 passed。

staging 验收（容器内直调路由函数，真实 DB）

修复前：所有素材计算抛 AttributeError、四字段 None；
修复后待本 PR 部署复验。

## 背景 #1549 合并后 staging 真实接口验收发现：GET /assets 与 smart-match 返回的余量四字段恒为 None（usable 恒为 True 兜底）。 ## 根因 路由层 repository 返回的是 **Asset 领域实体**（packages/domain/entities.py），不是 ORM AssetModel： - 实体的区间记录在 `metadata` **dict** 属性里（repository 与 `classification_result` JSON 列互转，见 asset_repository.py:133/162）； - 实体没有 `classification_result` 属性； - `_read_meta` 只读 `classification_result` 列 → AttributeError → `_asset_availability_fields` 异常兜底 → 四字段 None + usable=True。 单测用 SimpleNamespace 同时带了两个属性，未暴露此问题。 ## 修复 `_read_meta` 双形态兼容： - 对象有 `metadata` 且为 dict（领域实体）→ 直接使用； - 否则读 `classification_result` JSON 字符串（ORM AssetModel 形态，tracker 内部 DB 写入路径不变）。 ## 测试 - 新增实体形态用例（metadata dict、无 classification_result）与 ORM 形态用例对比； - 相关 5 个测试文件 64 passed；全量 unit 13913 passed。 ## staging 验收（容器内直调路由函数，真实 DB） 修复前：所有素材计算抛 AttributeError、四字段 None； 修复后待本 PR 部署复验。

xiaoxia added 1 commit 2026-08-30 08:03:07 +08:00

fix(segments): 余量计算兼容 Asset 领域实体（metadata dict） ... 0248c7e94e

staging 真实接口验收发现：路由 repository 返回的是 Asset 领域实体而非
ORM AssetModel，区间记录在 metadata dict（repository 与 classification_result
JSON 互转）；_read_meta 只读 classification_result 列导致 AttributeError，
余量计算全部走异常兜底，四字段在真实接口里恒为 None。

_read_meta 双形态兼容：实体 metadata 为 dict 时直接用；否则读
classification_result JSON 字符串（ORM 形态）。
测试：补充实体形态/ORM 形态对比用例（13913 passed）。

auto-approve-bot added 1 commit 2026-08-30 08:07:49 +08:00

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

gitea-actions bot commented 2026-08-30 08:11:08 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-08-30 08:12:23 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1550

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

无

💡 建议（1个可选）

1. apps/api/app/services/asset_segment_tracker.py 第72行：建议将 except Exception: 改为更具体的异常类型（如 json.JSONDecodeError 和 TypeError），或者至少添加日志记录。虽然此处沿用旧逻辑是为了容错，但捕获所有异常可能会掩盖非预期的程序错误（如内存错误或键盘中断），不利于排查数据格式问题。

---

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

---

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

## 代码审查结果 - PR #1550 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（1个可选） 1. **apps/api/app/services/asset_segment_tracker.py 第72行**：建议将 `except Exception:` 改为更具体的异常类型（如 `json.JSONDecodeError` 和 `TypeError`），或者至少添加日志记录。虽然此处沿用旧逻辑是为了容错，但捕获所有异常可能会掩盖非预期的程序错误（如内存错误或键盘中断），不利于排查数据格式问题。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-30 00:12:23 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

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

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 839426a2cc into develop 2026-08-30 08:21:01 +08:00

auto-approve-bot deleted branch fix/asset-availability-entity-meta 2026-08-30 08:21:01 +08:00

gitea-actions bot commented 2026-08-30 08:24:07 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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