xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 15 Pull Requests 2 Actions Packages Projects Releases 7 Wiki Activity

refactor(editing-planner): 拆分 ClipDetailSection，按职责拆为 3 个子组件 #1018

Closed
xiaoxia wants to merge 3 commits from refactor/clip-detail-section-split into develop
pull from: refactor/clip-detail-section-split
merge into: xiaoxia:develop
Conversation 5 Commits 3 Files Changed 5
+367 -264
xiaoxia commented 2026-07-27 09:22:35 +08:00
Owner
Copy Link
Copy Source

变更说明

拆分 ClipDetailSection（266 行），按职责拆分为目录结构：

  • ClipTypeAndDuration - 类型选择 + 时长设置
  • AdvancedEntries - 转场/调速/TTS 高级入口按钮
  • VoiceMaterialSection - 配音素材选择 + 起始时间（仅 voice 类型）
  • index - 组合入口

主文件 266 → 70 行（-74%）

导入路径保持不变，向后兼容。

## 变更说明 拆分 ClipDetailSection（266 行），按职责拆分为目录结构： - **ClipTypeAndDuration** - 类型选择 + 时长设置 - **AdvancedEntries** - 转场/调速/TTS 高级入口按钮 - **VoiceMaterialSection** - 配音素材选择 + 起始时间（仅 voice 类型） - **index** - 组合入口 主文件 266 → 70 行（-74%） 导入路径保持不变，向后兼容。
xiaoxia added 1 commit 2026-07-27 09:22:36 +08:00
refactor(editing-planner): 拆分 ClipDetailSection，按职责拆为 3 个子组件
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 40s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m6s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m5s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Failing after 47s
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Failing after 2m13s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 52s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 49s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 1m4s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m15s
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Failing after 2m34s
Details
AI Code Review / AI Code Review (pull_request) Successful in 4m4s
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 8m2s
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Failing after 36s
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
8b441a3615 
主文件 266→70 行（-74%），拆分为：
- ClipTypeAndDuration: 类型选择 + 时长设置
- AdvancedEntries: 转场/调速/TTS 高级入口按钮
- VoiceMaterialSection: 配音素材选择 + 起始时间（仅 voice 类型）
- index: 组合入口

导入路径保持向后兼容
xiaoxia added 1 commit 2026-07-27 12:34:17 +08:00
Merge branch 'develop' into refactor/clip-detail-section-split
CI/CD Pipeline / Check if frontend-only change (pull_request) Has been cancelled
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Has been cancelled
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Has been cancelled
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Has been cancelled
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Frontend Lint (pull_request) Has been cancelled
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / PR Build API Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been cancelled
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been cancelled
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been cancelled
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been cancelled
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled
Details
AI Code Review / AI Code Review (pull_request) Has been cancelled
Details
Preview Deploy / Deploy Preview Environment (pull_request) Has been cancelled
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 1m9s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m1s
Details
8e2e3d69fe
xiaoxia added 1 commit 2026-07-27 12:41:41 +08:00
fix: 恢复 ClipDetailSection.tsx 主入口，re-export 子目录保持路径兼容
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 14s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m4s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Failing after 1m43s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 45s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m18s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 21s
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 1m31s
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m53s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 40s
Details
AI Code Review / AI Code Review (pull_request) Successful in 1m16s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 40s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 12m14s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Has been cancelled
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Failing after 26s
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 21s
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
7ef7163463
auto-approve-bot approved these changes 2026-07-27 15:18:18 +08:00
auto-approve-bot left a comment
Collaborator
Copy Link
Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。
auto-approve-bot approved these changes 2026-07-27 15:18:18 +08:00
auto-approve-bot left a comment
Collaborator
Copy Link
Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。
auto-approve-bot commented 2026-07-27 15:18:19 +08:00
Collaborator
Copy Link
Copy Source

代码审查结果 - PR #1018

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

💡 建议（1个可选）

  1. AdvancedEntries.tsx 第23行及第29行：数据访问存在潜在的运行时错误风险。
    • 说明：代码中直接访问了 clip.transition.duration 和 clip.speed.rate 并调用 .toFixed()。虽然外层有 if (!t) 和 clip.speed ? 的判空，但如果对象存在但属性缺失（例如 speed 为 {} 或 transition 缺少 duration 字段），会导致 Cannot read properties of undefined 错误。
    • 修改建议：增加可选链或默认值保护，例如 t.duration?.toFixed(1) ?? '0.0' 和 clip.speed.rate?.toFixed(2) ?? '1.00'，以提高代码的健壮性。

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

🤖 由 AI 代码审查机器人自动生成 | 2026-07-27 07:18:19 | 模型:

## 代码审查结果 - PR #1018 ### ⚠️ 问题（0个需要修改） ### 💡 建议（1个可选） 1. **AdvancedEntries.tsx 第23行及第29行**：数据访问存在潜在的运行时错误风险。 - 说明：代码中直接访问了 `clip.transition.duration` 和 `clip.speed.rate` 并调用 `.toFixed()`。虽然外层有 `if (!t)` 和 `clip.speed ?` 的判空，但如果对象存在但属性缺失（例如 `speed` 为 `{}` 或 `transition` 缺少 `duration` 字段），会导致 `Cannot read properties of undefined` 错误。 - 修改建议：增加可选链或默认值保护，例如 `t.duration?.toFixed(1) ?? '0.0'` 和 `clip.speed.rate?.toFixed(2) ?? '1.00'`，以提高代码的健壮性。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能检查通过 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-07-27 07:18:19 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
gitea-actions bot commented 2026-07-27 15:18:58 +08:00
Copy Link
Copy Source

🚀 预览环境已部署

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

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

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

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

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1018 | | 预览链接 | [https://pr-1018.preview.xiaoxiajianji.com](https://pr-1018.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。
xiaoxia closed this pull request 2026-07-27 15:28:44 +08:00
gitea-actions bot commented 2026-07-27 17:04:13 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1018 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。
Some checks are pending
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 14s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m4s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Failing after 1m43s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 45s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m18s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 21s
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 1m31s
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m53s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 40s
Details
AI Code Review / AI Code Review (pull_request) Successful in 1m16s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 40s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 12m14s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Has been cancelled
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Failing after 26s
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 21s
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details

Pull request closed

Please reopen this pull request to perform a merge.
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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1018
Delete Branch "refactor/clip-detail-section-split"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?