refactor: 统一智能选素材逻辑，迁移 generation_tasks smart 模式到 smart_match #1243

Merged

xiaoxia merged 1 commits from feat/unify-asset-selection into develop 2026-08-05 11:27:57 +08:00

Conversation 2 Commits 1 Files Changed 8

+29 -2779

xiaoxia commented 2026-08-05 11:13:44 +08:00

Owner

Copy Link

Copy Source

背景

PR #1242 新增了 packages/domain/smart_match.py 统一智能选素材算法（四维评分 + 时长分桶多样性），但后端仍存在多套重叠的选素材/评分逻辑，需要清理统一。

变更内容

1. generation_tasks.py smart 模式迁移

之前： 内联的简单排序 quality_score desc, duration desc，注释中提到 SmartAssetSelector 但并未实际调用。

之后： 改为调用 packages/domain/smart_match.py 的 smart_select_assets()，复用统一的多维评分（质量40% + 时长适配30% + 新鲜度20% + 未使用加分10%）和时长分桶多样性算法。

# 旧逻辑
scored_assets = sorted(ready_video_assets, key=lambda a: (-(a.quality_score or 0), -(a.duration or 0)))
if count > 0: scored_assets = scored_assets[:count]
return [a.id for a in scored_assets]

# 新逻辑
limit = count if count > 0 else None
results = smart_select_assets(ready_video_assets, limit=limit, kind="video")
return [r.asset.id for r in results]

2. 删除冗余代码（净减 2750 行）

文件	说明
packages/domain/asset_scoring.py	旧4维评分（质量0.5+分辨率0.2+时长0.2+码率0.1），已被 smart_match.py 替代
apps/api/app/services/smart_asset_selector.py	SmartAssetSelector 类，无任何生产代码调用，仅测试引用
tests/unit/test_asset_scoring.py	asset_scoring.py 的测试
tests/unit/test_asset_scoring_domain.py	asset_scoring.py 的测试
tests/unit/domain/test_asset_scoring.py	asset_scoring.py 的测试
tests/unit/test_smart_asset_selector.py	SmartAssetSelector 的测试

3. 更新 test_asset_select_mode.py

smart 模式测试用例适配新评分体系（quality_score 从 0-1 标度改为 0-100 标度，匹配 smart_match.score_asset 的输入约定）。

4. 保留 auto_clip_service.py 的 _score_candidate

经评估，AutoClipService._score_candidate 是 per-clip 模板匹配 场景（评分维度：质量50% + 时长匹配30% + 分类匹配20%），与 smart_match 的 全局通用选素材 场景（质量40% + 时长30% + 新鲜度20% + 未使用10%）用途不同，不适合统一。

## 背景 PR #1242 新增了 `packages/domain/smart_match.py` 统一智能选素材算法（四维评分 + 时长分桶多样性），但后端仍存在多套重叠的选素材/评分逻辑，需要清理统一。 ## 变更内容 ### 1. `generation_tasks.py` smart 模式迁移 **之前：** 内联的简单排序 `quality_score desc, duration desc`，注释中提到 SmartAssetSelector 但并未实际调用。 **之后：** 改为调用 `packages/domain/smart_match.py` 的 `smart_select_assets()`，复用统一的多维评分（质量40% + 时长适配30% + 新鲜度20% + 未使用加分10%）和时长分桶多样性算法。 ```python # 旧逻辑 scored_assets = sorted(ready_video_assets, key=lambda a: (-(a.quality_score or 0), -(a.duration or 0))) if count > 0: scored_assets = scored_assets[:count] return [a.id for a in scored_assets] # 新逻辑 limit = count if count > 0 else None results = smart_select_assets(ready_video_assets, limit=limit, kind="video") return [r.asset.id for r in results] ``` ### 2. 删除冗余代码（净减 2750 行） | 文件 | 说明 | |------|------| | `packages/domain/asset_scoring.py` | 旧4维评分（质量0.5+分辨率0.2+时长0.2+码率0.1），已被 `smart_match.py` 替代 | | `apps/api/app/services/smart_asset_selector.py` | `SmartAssetSelector` 类，**无任何生产代码调用**，仅测试引用 | | `tests/unit/test_asset_scoring.py` | `asset_scoring.py` 的测试 | | `tests/unit/test_asset_scoring_domain.py` | `asset_scoring.py` 的测试 | | `tests/unit/domain/test_asset_scoring.py` | `asset_scoring.py` 的测试 | | `tests/unit/test_smart_asset_selector.py` | `SmartAssetSelector` 的测试 | ### 3. 更新 `test_asset_select_mode.py` smart 模式测试用例适配新评分体系（quality_score 从 0-1 标度改为 0-100 标度，匹配 `smart_match.score_asset` 的输入约定）。 ### 4. 保留 `auto_clip_service.py` 的 `_score_candidate` 经评估，`AutoClipService._score_candidate` 是 **per-clip 模板匹配** 场景（评分维度：质量50% + 时长匹配30% + 分类匹配20%），与 `smart_match` 的 **全局通用选素材** 场景（质量40% + 时长30% + 新鲜度20% + 未使用10%）用途不同，不适合统一。

xiaoxia added 1 commit 2026-08-05 11:13:44 +08:00

refactor: 统一智能选素材逻辑，迁移 generation_tasks smart 模式到 smart_match ... 012d97ae4e

- generation_tasks.py 的 smart 模式从内联 quality+duration 排序改为调用
  packages/domain/smart_match.py 的 smart_select_assets()，复用统一的多维
  评分（质量40%+时长30%+新鲜度20%+未使用10%）和时长分桶多样性算法
- 删除已无引用的旧代码：
  - packages/domain/asset_scoring.py（旧4维评分：质量0.5+分辨率0.2+时长0.2+码率0.1）
  - apps/api/app/services/smart_asset_selector.py（SmartAssetSelector 类，无生产调用方）
  - 对应测试文件：test_asset_scoring.py、test_asset_scoring_domain.py、
    tests/unit/domain/test_asset_scoring.py、test_smart_asset_selector.py
- 更新 test_asset_select_mode.py 的 smart 模式测试用例，适配新评分体系
- 全量单测 13587 passed 通过

gitea-actions bot commented 2026-08-05 11:15:19 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia merged commit bb5bed0f4c into develop 2026-08-05 11:27:57 +08:00

gitea-actions bot commented 2026-08-05 11:29:20 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

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

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1243