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

refactor(assets): Phase 2 - extract UI components #897

Closed
xiaoxia wants to merge 2 commits from refactor/asset-library-phase2 into develop
pull from: refactor/asset-library-phase2
merge into: xiaoxia:develop
Conversation 1 Commits 2 Files Changed 15
+884 -538
xiaoxia commented 2026-07-25 17:59:51 +08:00
Owner
Copy Link
Copy Source

概述

AssetLibrary Phase 2 — UI 组件抽离。将 1122 行的 AssetLibrary.tsx 中的 UI 组件抽离为 12 个独立组件文件。

抽离的组件

组件 行数 说明
AssetCard 128 素材卡片（网格视图）
AssetSkeleton 36 StatusPill + SkeletonCard 小组件
LibrarySidebar 70 左侧素材库列表
AssetFilterBar 65 筛选栏（搜索/类型/时间）
BatchOperationBar 73 批量操作栏
CreateLibraryModal 64 新建视频库弹窗
PlayModal 34 视频播放弹窗
BatchTagModal 84 批量打标签弹窗
BatchClassifyModal 51 批量改分类弹窗
BatchMarkModal 67 批量智能标记弹窗
ResultDrawer 78 操作结果抽屉
UploadProgressModal 56 上传进度弹窗

其他变更

  • kindIcon 工具函数移至 utils/kindIcon.tsx（JSX 需在 tsx 文件中）
  • 所有组件采用受控组件模式（Props 驱动，状态保留在父组件）
  • smoke test 更新，覆盖所有新组件

统计

  • 主文件：1122 行 → 679 行（减少 443 行，约 40%）
  • 新增文件：13 个（12 个组件 + 1 个工具函数）
  • pnpm tsc --noEmit ✅ 全通过
  • pnpm lint ✅ 全通过
  • 0 any、0 ts-ignore、0 eslint-disable
  • 功能完全等价
## 概述 AssetLibrary Phase 2 — UI 组件抽离。将 1122 行的 AssetLibrary.tsx 中的 UI 组件抽离为 12 个独立组件文件。 ## 抽离的组件 | 组件 | 行数 | 说明 | |------|------|------| | AssetCard | 128 | 素材卡片（网格视图） | | AssetSkeleton | 36 | StatusPill + SkeletonCard 小组件 | | LibrarySidebar | 70 | 左侧素材库列表 | | AssetFilterBar | 65 | 筛选栏（搜索/类型/时间） | | BatchOperationBar | 73 | 批量操作栏 | | CreateLibraryModal | 64 | 新建视频库弹窗 | | PlayModal | 34 | 视频播放弹窗 | | BatchTagModal | 84 | 批量打标签弹窗 | | BatchClassifyModal | 51 | 批量改分类弹窗 | | BatchMarkModal | 67 | 批量智能标记弹窗 | | ResultDrawer | 78 | 操作结果抽屉 | | UploadProgressModal | 56 | 上传进度弹窗 | ## 其他变更 - `kindIcon` 工具函数移至 `utils/kindIcon.tsx`（JSX 需在 tsx 文件中） - 所有组件采用受控组件模式（Props 驱动，状态保留在父组件） - smoke test 更新，覆盖所有新组件 ## 统计 - 主文件：1122 行 → 679 行（减少 443 行，约 40%） - 新增文件：13 个（12 个组件 + 1 个工具函数） - `pnpm tsc --noEmit` ✅ 全通过 - `pnpm lint` ✅ 全通过 - 0 any、0 ts-ignore、0 eslint-disable - 功能完全等价
xiaoxia added 1 commit 2026-07-25 17:59:51 +08:00
refactor(assets): Phase 2 - extract UI components
AI Code Review / AI Code Review (pull_request) Failing after 0s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Failing after 50s
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 1m43s
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 - Type Check (mypy) (pull_request) Successful in 1m48s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m51s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 1m4s
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 / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m13s
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 28s
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 / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 21s
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 3m39s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 3m50s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 2m47s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 46m5s
Details
6ca886f6cf 
Extract 12 UI components from AssetLibrary.tsx into components/ directory:
- AssetCard: 素材卡片（网格视图）
- AssetSkeleton: StatusPill + SkeletonCard 小组件
- LibrarySidebar: 左侧素材库列表
- AssetFilterBar: 筛选栏（搜索/类型/时间）
- BatchOperationBar: 批量操作栏
- CreateLibraryModal: 新建视频库弹窗
- PlayModal: 视频播放弹窗
- BatchTagModal: 批量打标签弹窗
- BatchClassifyModal: 批量改分类弹窗
- BatchMarkModal: 批量智能标记弹窗
- ResultDrawer: 操作结果抽屉
- UploadProgressModal: 上传进度弹窗

Also moves kindIcon helper to utils/kindIcon.tsx.

Main file reduced from 1122 lines to 679 lines.
All state remains in parent component (controlled component pattern).
Full tsc + eslint clean.
xiaoxia added 1 commit 2026-07-25 18:11:01 +08:00
chore(assets): fix prettier formatting
AI Code Review / AI Code Review (pull_request) Failing after 0s
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 15s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 53s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m38s
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 - Type Check (mypy) (pull_request) Successful in 1m43s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 56s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 26s
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m7s
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 / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 17s
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 / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 3m44s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m9s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 2m53s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 46m3s
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 14s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Failing after 49s
Details
3002fe0135
xiaoxia closed this pull request 2026-07-25 20:53:41 +08:00
gitea-actions bot commented 2026-07-25 20:56:25 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #897 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。
Some checks are pending
AI Code Review / AI Code Review (pull_request) Failing after 0s
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 15s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 53s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m38s
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 - Type Check (mypy) (pull_request) Successful in 1m43s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 56s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 26s
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m7s
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 / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 17s
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 / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 3m44s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m9s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 2m53s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 46m3s
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 14s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Failing after 49s
Details

Pull request closed

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

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#897
Delete Branch "refactor/asset-library-phase2"

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?