chore: 同步 develop 到 main 2026-09-02（含53个 commits） #1623

Merged

xiaoxia merged 53 commits from develop into main 2026-09-02 10:13:31 +08:00

Conversation 3 Commits 53 Files Changed 53

+3105 -394

xiaoxia commented 2026-09-02 09:57:49 +08:00

Owner

Copy Link

Copy Source

发版内容

将 develop 分支全量同步到 main，包含 PR#1596 ~ PR#1622 共 53 个 commits。

关键变更

• PR#1596: from-assets 400 修复
• PR#1600: limit=1 算法修复
• PR#1612/1613/1616: 标题预设样式优化
• PR#1614: 生成视频时长修复
• PR#1615: smart_match 评分接入
• PR#1619: 查重率 0% 显示修复
• PR#1620: 素材选取评分注入随机噪声
• PR#1622: SceneChange 场景检测前置
• commit 827d8aa: 素材分配顺序随机打乱 + MediaKit SceneChange 智能选帧

部署流程

1. CI 全绿后合并
2. 推送 v0.1.131 tag 触发 deploy-production
3. 健康检查 + 自动回滚兜底

风险说明

• 无新增 alembic 迁移，不需要跑数据库迁移
• staging E2E 失败为 playwright 镜像环境问题，与代码无关

## 发版内容 将 develop 分支全量同步到 main，包含 PR#1596 ~ PR#1622 共 53 个 commits。 ### 关键变更 - PR#1596: from-assets 400 修复 - PR#1600: limit=1 算法修复 - PR#1612/1613/1616: 标题预设样式优化 - PR#1614: 生成视频时长修复 - PR#1615: smart_match 评分接入 - PR#1619: 查重率 0% 显示修复 - PR#1620: 素材选取评分注入随机噪声 - PR#1622: SceneChange 场景检测前置 - commit 827d8aa: 素材分配顺序随机打乱 + MediaKit SceneChange 智能选帧 ### 部署流程 1. CI 全绿后合并 2. 推送 v0.1.131 tag 触发 deploy-production 3. 健康检查 + 自动回滚兜底 ### 风险说明 - 无新增 alembic 迁移，不需要跑数据库迁移 - staging E2E 失败为 playwright 镜像环境问题，与代码无关

xiaoxia added 53 commits 2026-09-02 09:57:50 +08:00

fix(ci): replace psycopg2 with psycopg3 in run_integration_tests.sh 071d17d947

fix(ci): replace psycopg2 with psycopg3 in run_validate.sh 83cbcd6c76

fix(ci): sync context list with CI optimized job names in auto_approve.sh 6e4b711ba2

fix(ci): sync REQUIRED_CONTEXTS_APPROVE with CI optimized job names 974ca188b3

perf(ci): pr-auto-scan cron 10min -> 15min on develop 33f9f5021b

fix(ci): replace psycopg2 with psycopg in validate_migration.sh ... 2e3b2d7680

Alembic migration validation was failing because psycopg2 module doesn't exist.
requirements-base.txt installs psycopg[binary]==3.2.2 (psycopg3).

Changed:
- import psycopg2 → import psycopg
- psycopg2.connect() → psycopg.connect()

fix(clips): P0 自建模板 from-assets 400 直接查 clip_configs 表兜底 (#1596) 6ea77725a5

feat(title): 预览画面标题自由拖拽——position=custom+PlayRes坐标提交（工单#1405方案B） (#1593) ff40a7b5f3

fix(smart-match): limit 算法取 segments.length 下限，保证每片段至少 1 个素材 (#1600) 24f1ca35d5

feat(title): 标题自由位置拖拽后端支持——pos_x/pos_y 注入 \pos 定位 (#1592) 1a2c4b89be

fix(smart-match): 过滤前置+余量/高频全过滤时空结果回退保留素材 (#1590) 1413cb80fe

fix(ui): 标题拖拽pointerEvents修复 + T字放大28px + 成片thumb白边修复 (#1601) a3f615a2d1

feat(dedup): 自动查重率扩大到用户全局——跨项目比对 ... 3a9a4334df

原 compute_duplicate_rate 仅按 project_id 比较，用户在不同项目/模板
下生成的视频互不可见，导致查重率始终为 0。

修改：
- dedup.py: compute_duplicate_rate 新增可选 user_id 参数；
  有 user_id 时按用户全局查询（跨项目），否则回退 project_id。
  查询上限从 100 提升到 200。
- dedup_helpers.py: create_video_record_and_dedup 传 user_id
  给 compute_duplicate_rate。

新增测试：
- test_duplicate_rate.py: +2 用例（跨项目匹配 / 空 user_id 回退）
- test_dedup_helpers_user_id.py: 3 用例（user_id 透传 / 空值兼容 /
  duplicate_rate 写回记录）

共 107 个相关测试全过。

fix: 恢复 package-lock.json，修复 @testing-library/dom 缺失导致的测试失败 ... dedb076c94

PR #1601 中 npm install 意外删除了 @testing-library/dom 等测试依赖，
导致 Frontend Unit Tests 全部失败。恢复 package-lock.json 到 PR #1601 之前的状态。

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

fix(dedup): AI Review 阻塞问题修复——DB 层排除自身 + 文档修正 ... 9ad729d917

1. DB 查询层排除 current_video_id：
   原实现在 Python 循环里排除自身，但 GeneratedVideo 记录在查重前已
   写入 DB（dedup_helpers L80 create → L126 compute_duplicate_rate），
   查询结果会包含自身 → MD5 完全匹配 → duplicate_rate 恒为 100%。
   现在在 query 构建时即排除：query.filter(id != current_video_id)

2. 文档修正：limit(200) → "遍历最近 200 个"（与实际实现一致）

3. 测试 mock 更新：filter() 现在被调用两次（scope + self-exclusion），
   所有测试改用 chainable query_mock 模式。

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

Merge pull request 'fix: 恢复 package-lock.json 修复测试依赖缺失' (#1603) from fix/restore-package-lock into develop 70c34da507

feat(dedup): 自动查重率扩大到用户全局——跨项目比对 (#1602) 0bc53172fc

fix(products): 移除thumb max-height限制，修复竖屏视频卡片右侧白边 ... f761a6bf07

根因：.xx-product-thumb 的 max-height:320px 在竖屏视频
(aspectRatio 9/16) 时限制宽度为180px，但卡片网格列宽约300px，
导致右侧出现白边。移除 max-height 让 aspectRatio 自然撑满列宽。

Merge pull request 'fix(products): 移除thumb max-height限制，修复竖屏视频卡片右侧白边' (#1604) from fix/product-thumb-white-border into develop adf6b517fb

fix(title): 拖拽时用ref直接改DOM避免React重渲染导致换行 ... 89413d6822

handleTitlePointerMove 每次 mousemove 都调用 onTitlePositionChange 触发
React 频繁重渲染，wordBreak:break-word 导致标题文字换行。

修复：拖拽过程中用 titleDragRef 直接修改 DOM style.left/top，
不触发 React 渲染；只在 handleTitlePointerUp（拖拽结束）时调用
onTitlePositionChange 保存最终位置。

style(title): 标题预设卡片改为深色背景大T字紧凑布局 ... 0bf1cfa7f4

- .xx-title-preset-card: 52x52固定尺寸，#3a3a3a深色背景，8px圆角
- hover: #4a4a4a + #666边框; active: #409eff边框
- .xx-title-preset-preview-text: font-size 32px + font-weight 700
- TITLE_PRESETS 所有 preset 的 previewStyle.fontSize 统一为 32px
- 不改网格列数/间距/点击逻辑/预设内容

ci: re-trigger build 3c8b794efb

ci: re-trigger build f685d689d6

Merge pull request 'style(title): 标题预设卡片改为深色背景大T字紧凑布局' (#1606) from fix/title-preset-card-style into develop 262b730aca

Merge pull request 'fix(title): 拖拽时用ref直接改DOM避免React重渲染导致换行' (#1605) from fix/title-drag-no-rerender into develop 6fbc593916

ci: 重新触发构建——前次 Build Staging Web Image 瞬态失败 783e426255

fix(web): 成片库卡片查重率显示 '-' — mapVideoToProductItem 硬编码 undefined ... c11d70d9c9

根因: mapVideoToProductItem 中 duplicate_rate 被硬编码为 undefined，
导致无论后端 API 返回什么值，前端永远显示 '-'。

修复:
- VideoItem 接口增加 duplicate_rate?: number 字段
- mapVideoToProductItem 改为从 video 对象读取 duplicate_rate

Merge pull request 'ci: 重新触发构建（前次 Build Staging Web Image 瞬态失败）' (#1607) from ci/retrigger-build into develop 65f41b1ba4

Merge pull request 'fix(web): 成片库卡片查重率显示 "-" — mapVideoToProductItem 硬编码 undefined' (#1608) from fix/web-duplicate-rate-mapping into develop b48daf346b

ci: 触发干净构建（确保无并发取消） (#1609) ... 5f18f844c1

Co-authored-by: xiaoxia <dev@xiaoxiajianji.com>
Co-committed-by: xiaoxia <dev@xiaoxiajianji.com>

fix(ci): 自动同步 docker config 到 BuildKit 容器 ... 0299678389

- 每次确保 builder 存在后自动同步宿主机 docker config
- 防止 builder 重建后 registry 认证丢失
- 解决私有镜像拉取 401 问题

style(products+generate): 成片库播放按钮居中+卡片缩小+预设卡片紧凑化+深色T描边 (#1610) b6ed45fc1b

feat(asset-dedup): 素材去重机制6项优化 — 调参+警告+贪心分配 (#1611) ... a0c14db33c

Co-authored-by: xiaoxia <dev@xiaoxiajianji.com>
Co-committed-by: xiaoxia <dev@xiaoxiajianji.com>

fix(title): 预设T字双图层渲染+间距1.5px+深色改浅色 (#1612) 0a5bd4f7e6

fix(title): 预设4项修复—间距0.5px+删tooltip+描边对齐+背景淡化 (#1613) 57621d9346

fix(render): 修复生成视频时长短于模板要求时长 (#1614) ... 4aeb1d5b66

fix(render): 修复生成视频时长短于模板要求时长 (#1614)

feat: 素材选取路径接入 smart_match 评分，让用过降权真正生效 (#1615) ... 9d5ae7a5bc

feat: 素材选取路径接入 smart_match 评分，让用过降权真正生效 (#1615)

fix(title): 预设间距归零 + 新增16种样式 (#1616) ... c05026c5db

Co-authored-by: xiaoxia <dev@xiaoxiajianji.com>
Co-committed-by: xiaoxia <dev@xiaoxiajianji.com>

perf(CI): Unit Tests 增量测试+依赖缓存优化 ... ab6717eeae

- push 事件也启用增量测试选择（仅跑改动相关的测试文件）
- 持久 runner 依赖 hash 缓存（requirements 无变化时跳过 pip install）
- 3 次独立 pip install 合并为 1 次
- numpy 缓存检查（已安装则跳过）
- 预计节省 5-10 分钟

perf(deploy): Worker 优雅关闭超时从 300s 降至 120s ... 6c3db74fd3

- 大多数部署时 Worker 无长任务运行，120s 足够
- 减少不必要的等待时间

perf(deploy): 健康检查轮询间隔从 5s 降至 3s ... c00a0d9eb0

- 更快检测到服务就绪状态
- 减少约 10-20s 的等待时间

feat: 素材分配顺序随机打乱 + MediaKit SceneChange智能选帧 ... 827d8aafe5

1. 素材分配顺序随机打乱
   - segments 处理顺序通过 random.shuffle 随机化
   - clips_data 最终按原始 segment order 排序返回
   - 打乱的是分配顺序，不影响播放顺序

2. MediaKit SceneChange 智能选帧
   - mediakit_client.py 新增 detect_scene_changes 方法
   - SceneChange 优先，失败降级到 TimeInterval（5秒间隔）
   - 后台任务 _update_mediakit_recommendations_async 改用 SceneChange
   - 各片段优先从不同镜头段选取起始时间
   - analyze_videos 保留为 fallback
   - 所有降级路径：MediaKit不可用/SceneChange失败 → 保持随机start_time

3. 测试
   - 新增 tests/unit/test_scene_change_shuffle.py (12个测试用例)
   - 更新 test_editor_clips_random_start.py 适配 shuffle 逻辑
   - 全部 108 个相关测试通过
   - ruff check 通过

fix(title): 预设卡片间距调整 gap 1px + 列宽 52px 固定 (#1617) ... 8920bead38

fix(title): 预设卡片间距调整 gap 1px + 列宽 52px 固定 (#1617)

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

fix(products): 查重率 0 时显示 0.0% 而非 - (#1619) 9c03755318

fix(deps): pin httpcore==1.0.7 + h2==4.1.0 防止 ci-base 镜像依赖漂移 f51f1139bf

fix(deps): pin numpy==1.26.4 到 dev 依赖，与 worker 保持一致，消除 ci-base 镜像 numpy 2.x 漂移 19b45cbee1

feat(ci): ci-base 镜像自动重建 workflow，requirements/Dockerfile 变更时自动构建推送 5f9538b5ac

fix(test): 补齐噪声/转场补偿导致的 flaky 断言 ... c66e73e5b0

- test_mediakit_smart_clips.test_assets_balanced_assignment：
  评分噪声下贪心均衡断言改为 zero-noise patch 内确定性验证
  （CI 失败：assert 1 == 2，与 test_editor_clips_random_start 同类）
- test_duration_within_segment_range：断言上界未计入 PR #1614
  转场补偿（2片段每clip +0.25s），uniform 接近上限时偶发 5.2>5.0；
  按补偿公式与舍入余量修正断言范围

feat: 素材选取评分注入随机噪声，降低成片查重率 (#1620) ... 2c5600cfc9

Co-authored-by: xiaoxia <dev@xiaoxiajianji.com>
Co-committed-by: xiaoxia <dev@xiaoxiajianji.com>

feat: SceneChange 场景检测前置到渲染前 + 场景点缓存到素材 metadata (#1622) ... 24e66da061

Co-authored-by: xiaoxia <dev@xiaoxiajianji.com>
Co-committed-by: xiaoxia <dev@xiaoxiajianji.com>

gitea-actions bot commented 2026-09-02 10:06:06 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-09-02 10:06:12 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：需修改
• 建议级问题数量：2 个

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

1. [apps/api/app/api/routes/templates_editor/clips.py: 833] 逻辑矛盾导致死代码

   • 问题类型：逻辑bug
   • 问题描述：代码在第 833 行当找不到可用素材时抛出了 HTTPException，这会导致函数立即终止并返回 400 错误。这使得后续第 887 行的 exhaustion_warning（素材耗尽警告）逻辑永远无法执行（unreachable code）。既然新增了 Schema 字段用于返回警告，说明业务意图是允许部分生成并提示用户，但当前的异常处理逻辑与此意图相悖。
   • 修改建议：移除第 833 行的 raise HTTPException(...)，或者将异常捕获后转换为警告信息。若必须报错，则应移除第 887 行不可达的警告逻辑。

2. [apps/api/app/api/routes/templates_editor/clips.py: 1050] 遍历时修改列表导致元素跳过

   • 问题类型：逻辑bug
   • 问题描述：在 _update_mediakit_recommendations_async 函数中，使用 for seg_idx, ... in enumerate(scene_segments_pool) 遍历列表，并在循环体内执行了 scene_segments_pool.pop(seg_idx)。这会导致列表元素前移，下一次循环索引递增时会跳过紧邻的下一个元素，导致部分镜头段分配逻辑错误。
   • 修改建议：避免在遍历中直接修改列表。可以使用倒序遍历、遍历副本 for seg in scene_segments_pool[:]: 并使用 remove，或者使用 while 循环来安全地消费列表中的元素。

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

1. [.gitea/workflows/pr-auto-scan.yml: 6] 注释与代码不一致

   • 具体内容：注释描述为“每10分钟扫描一次”，但 cron 表达式已修改为 */15。建议同步修改注释以匹配实际逻辑。

2. [.gitea/workflows/ci-base-image.yml: 31] 硬编码用户名

   • 具体内容：GITEA_REGISTRY_USER 直接硬编码为 xiaoxia。建议使用环境变量或 Secrets 管理，以提高配置的灵活性和安全性。

✅ 良好实践

• 在 create_clips_from_assets_editor 中引入了转场时长补偿计算，确保生成视频的总时长符合模板设定，逻辑严密。
• 引入了 _calc_plan_internal_duplicate_rate 函数进行轻量级的内部重复率估算，有助于提升生成质量。
• 使用 set -eu 确保 CI 脚本在遇到错误时能及时退出，符合安全最佳实践。

---

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

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-02 02:06:12 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：2 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 1. **[apps/api/app/api/routes/templates_editor/clips.py: 833] 逻辑矛盾导致死代码** - 问题类型：逻辑bug - 问题描述：代码在第 833 行当找不到可用素材时抛出了 `HTTPException`，这会导致函数立即终止并返回 400 错误。这使得后续第 887 行的 `exhaustion_warning`（素材耗尽警告）逻辑永远无法执行（unreachable code）。既然新增了 Schema 字段用于返回警告，说明业务意图是允许部分生成并提示用户，但当前的异常处理逻辑与此意图相悖。 - 修改建议：移除第 833 行的 `raise HTTPException(...)`，或者将异常捕获后转换为警告信息。若必须报错，则应移除第 887 行不可达的警告逻辑。 2. **[apps/api/app/api/routes/templates_editor/clips.py: 1050] 遍历时修改列表导致元素跳过** - 问题类型：逻辑bug - 问题描述：在 `_update_mediakit_recommendations_async` 函数中，使用 `for seg_idx, ... in enumerate(scene_segments_pool)` 遍历列表，并在循环体内执行了 `scene_segments_pool.pop(seg_idx)`。这会导致列表元素前移，下一次循环索引递增时会跳过紧邻的下一个元素，导致部分镜头段分配逻辑错误。 - 修改建议：避免在遍历中直接修改列表。可以使用倒序遍历、遍历副本 `for seg in scene_segments_pool[:]:` 并使用 `remove`，或者使用 `while` 循环来安全地消费列表中的元素。 ### 💡 改进建议（不阻塞合并） 1. **[.gitea/workflows/pr-auto-scan.yml: 6] 注释与代码不一致** - 具体内容：注释描述为“每10分钟扫描一次”，但 cron 表达式已修改为 `*/15`。建议同步修改注释以匹配实际逻辑。 2. **[.gitea/workflows/ci-base-image.yml: 31] 硬编码用户名** - 具体内容：`GITEA_REGISTRY_USER` 直接硬编码为 `xiaoxia`。建议使用环境变量或 Secrets 管理，以提高配置的灵活性和安全性。 ### ✅ 良好实践 - 在 `create_clips_from_assets_editor` 中引入了转场时长补偿计算，确保生成视频的总时长符合模板设定，逻辑严密。 - 引入了 `_calc_plan_internal_duplicate_rate` 函数进行轻量级的内部重复率估算，有助于提升生成质量。 - 使用 `set -eu` 确保 CI 脚本在遇到错误时能及时退出，符合安全最佳实践。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-02 02:06:12 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 419e763ad6 into main 2026-09-02 10:13:31 +08:00

gitea-actions bot commented 2026-09-02 10:18:06 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1623