refactor(assets): 拆分 batchOperations 为4个子Hook（238→8行, -97%） #1168

Merged

auto-approve-bot merged 15 commits from refactor/batch-operations-split into develop 2026-07-30 00:27:41 +08:00

Conversation 5 Commits 15 Files Changed 7

+253 -5

xiaoxia commented 2026-07-29 22:41:13 +08:00

Owner

Copy Link

Copy Source

拆分内容

将 asset-operations/batchOperations.ts（238行）拆分为4个独立的子Hook + 统一入口：

• batch/useBatchDelete.ts (40行) - 批量删除
• batch/useBatchTag.ts (86行) - 批量打标签
• batch/useBatchClassify.ts (58行) - 批量改分类
• batch/useBatchMark.ts (53行) - 批量智能标记
• batch/index.ts - 子模块统一出口
• 原 batchOperations.ts 改为 re-export，保持路径向后兼容

主文件变化

• batchOperations.ts：238行 → 8行（-97%）

影响范围

• useAssetOperations.ts 已有引用无需修改（路径不变）
• 补充 smoke test import 覆盖所有子模块

### 拆分内容 将 `asset-operations/batchOperations.ts`（238行）拆分为4个独立的子Hook + 统一入口： - `batch/useBatchDelete.ts` (40行) - 批量删除 - `batch/useBatchTag.ts` (86行) - 批量打标签 - `batch/useBatchClassify.ts` (58行) - 批量改分类 - `batch/useBatchMark.ts` (53行) - 批量智能标记 - `batch/index.ts` - 子模块统一出口 - 原 `batchOperations.ts` 改为 re-export，保持路径向后兼容 ### 主文件变化 - batchOperations.ts：238行 → 8行（-97%） ### 影响范围 - `useAssetOperations.ts` 已有引用无需修改（路径不变） - 补充 smoke test import 覆盖所有子模块

xiaoxia added 7 commits 2026-07-29 22:41:14 +08:00

refactor(assets): 拆分 batchOperations 子模块 df40f557a8

refactor(assets): 拆分 batchOperations 子模块 ff6ce3c49a

refactor(assets): 拆分 batchOperations 子模块 931daa6e9b

refactor(assets): 拆分 batchOperations 子模块 5e51933d5f

refactor(assets): 拆分 batchOperations 子模块 c1772a9cd3

refactor(assets): batchOperations 改为 re-export 入口 5ffd6dfb23

test: 补充 batchOperations 子模块 smoke test b0d7d81a30

gitea-actions bot commented 2026-07-29 22:47:20 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

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

fix: prettier format index.ts faf7b6958c

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

fix: prettier format useBatchClassify.ts b826332f5b

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

fix: prettier format useBatchDelete.ts 9758eb9e9b

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

fix: prettier format useBatchMark.ts 87357b97cb

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

fix: prettier format useBatchTag.ts deffd97650

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

fix: prettier format batchOperations.ts 68c29c64b6

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

fix: prettier format smoke.test.tsx 329aa797f4

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

fix: prettier format with project config (semi:false) 4e8eb6e9df

auto-approve-bot commented 2026-07-30 00:20:30 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：有建议
• 建议级问题数量：2 个

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

无

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

1. [useBatchDelete.ts, useBatchClassify.ts, useBatchMark.ts, useBatchTag.ts] 缺少选中项数量校验

   • 具体内容：建议在各个 handleBatch 函数的最开始增加对 selectedIds.size === 0 的判断。虽然后端 API 可能支持处理空数组，但前端提前拦截可以避免发起无效的网络请求，防止出现“成功 0 个素材”的非预期提示，提升用户体验。

2. [useBatchDelete.ts, useBatchClassify.ts, useBatchMark.ts, useBatchTag.ts] 建议增加防重复提交逻辑

   • 具体内容：建议在各个 handleBatch 函数中增加 if (batchLoading) return; 的前置判断。尽管通常 UI 层会通过禁用按钮来防止重复点击，但在 Hook 层增加双重校验可以防止极端情况下的并发请求（如网络延迟时的快速点击），增强代码的健壮性。

✅ 良好实践

• 异常处理结构清晰，所有异步操作均使用了 try-catch-finally 结构，确保了 loading 状态在任何情况下都能正确重置，避免 UI 卡死。
• useBatchTag 中对空标签进行了前置校验，逻辑严谨。
• 使用 Set 管理选中 ID，保证了 ID 的唯一性，Array.from 转换操作使用正确。
• 代码结构拆分合理，将不同的批量操作拆分为独立的 Hook，符合单一职责原则，便于维护。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 16:20:30 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：有建议 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[useBatchDelete.ts, useBatchClassify.ts, useBatchMark.ts, useBatchTag.ts] 缺少选中项数量校验** - 具体内容：建议在各个 `handleBatch` 函数的最开始增加对 `selectedIds.size === 0` 的判断。虽然后端 API 可能支持处理空数组，但前端提前拦截可以避免发起无效的网络请求，防止出现“成功 0 个素材”的非预期提示，提升用户体验。 2. **[useBatchDelete.ts, useBatchClassify.ts, useBatchMark.ts, useBatchTag.ts] 建议增加防重复提交逻辑** - 具体内容：建议在各个 `handleBatch` 函数中增加 `if (batchLoading) return;` 的前置判断。尽管通常 UI 层会通过禁用按钮来防止重复点击，但在 Hook 层增加双重校验可以防止极端情况下的并发请求（如网络延迟时的快速点击），增强代码的健壮性。 ### ✅ 良好实践 - 异常处理结构清晰，所有异步操作均使用了 `try-catch-finally` 结构，确保了 `loading` 状态在任何情况下都能正确重置，避免 UI 卡死。 - `useBatchTag` 中对空标签进行了前置校验，逻辑严谨。 - 使用 `Set` 管理选中 ID，保证了 ID 的唯一性，`Array.from` 转换操作使用正确。 - 代码结构拆分合理，将不同的批量操作拆分为独立的 Hook，符合单一职责原则，便于维护。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 16:20:30 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-07-30 00:21:06 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-30 00:21:06 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 9d97e8aacc into develop 2026-07-30 00:27:41 +08:00

auto-approve-bot deleted branch refactor/batch-operations-split 2026-07-30 00:27:42 +08:00

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

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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