refactor(titles): 拆分 useTitleLibrary 为4个子Hook（174→58行, -67%） #1156

Merged

auto-approve-bot merged 9 commits from refactor/title-library-hook into develop 2026-07-30 01:19:17 +08:00

Conversation 5 Commits 9 Files Changed 7

+276 -174

xiaoxia commented 2026-07-29 19:13:59 +08:00

Owner

Copy Link

Copy Source

变更内容

将 useTitleLibrary.ts（174行）拆分为目录结构，按职责分离：

• useTitleData - 数据获取（useQuery）+ 分类派生
• useTitleFilters - 筛选逻辑（搜索/类型/行业/使用频率/分类）
• useTitleMutations - CRUD mutations（create/update/delete）
• useTitleActions - 操作 handlers（收藏/复制/删除）
• index.ts - 主入口，组合子Hook

文件变化

• 删除：hooks/useTitleLibrary.ts (174行)
• 新增：hooks/useTitleLibrary/index.ts (58行)
• 新增：hooks/useTitleLibrary/useTitleData.ts (47行)
• 新增：hooks/useTitleLibrary/useTitleFilters.ts (84行)
• 新增：hooks/useTitleLibrary/useTitleMutations.ts (34行)
• 新增：hooks/useTitleLibrary/useTitleActions.ts (37行)
• 新增：test/pages/titles/smoke.test.tsx

兼容性

• 原导入路径 @/pages/titles/hooks/useTitleLibrary 保持不变（目录解析到 index.ts）

### 变更内容 将 `useTitleLibrary.ts`（174行）拆分为目录结构，按职责分离： - **useTitleData** - 数据获取（useQuery）+ 分类派生 - **useTitleFilters** - 筛选逻辑（搜索/类型/行业/使用频率/分类） - **useTitleMutations** - CRUD mutations（create/update/delete） - **useTitleActions** - 操作 handlers（收藏/复制/删除） - **index.ts** - 主入口，组合子Hook ### 文件变化 - 删除：`hooks/useTitleLibrary.ts` (174行) - 新增：`hooks/useTitleLibrary/index.ts` (58行) - 新增：`hooks/useTitleLibrary/useTitleData.ts` (47行) - 新增：`hooks/useTitleLibrary/useTitleFilters.ts` (84行) - 新增：`hooks/useTitleLibrary/useTitleMutations.ts` (34行) - 新增：`hooks/useTitleLibrary/useTitleActions.ts` (37行) - 新增：`test/pages/titles/smoke.test.tsx` ### 兼容性 - 原导入路径 `@/pages/titles/hooks/useTitleLibrary` 保持不变（目录解析到 index.ts）

xiaoxia added 1 commit 2026-07-29 19:13:59 +08:00

refactor(titles): 拆分 useTitleLibrary 为4个子Hook（174→58行, -67%） ... ab8d59ad9d

- useTitleData: 数据获取 + 分类派生
- useTitleFilters: 筛选逻辑（搜索/类型/行业/频率/分类）
- useTitleMutations: CRUD mutations（create/update/delete）
- useTitleActions: 操作 handlers（收藏/复制/删除）
- index.ts: 主入口，组合子Hook
- 新增 titles 模块 smoke test

xiaoxia added 1 commit 2026-07-29 21:51:46 +08:00

fix(ts): 移除未使用的 queryClient import 520c0ac40f

xiaoxia added 1 commit 2026-07-29 21:52:10 +08:00

fix(ts): 移除未使用的 queryClient f55a790f84

xiaoxia added 1 commit 2026-07-29 21:52:11 +08:00

fix(ts): 移除未使用的 UseTitleActionsOptions interface 728579c777

xiaoxia added 1 commit 2026-07-29 21:52:14 +08:00

fix(ts): 移除未使用的 interface 和 categories 参数 ea5816379b

gitea-actions bot commented 2026-07-29 21:55:28 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-07-29 22:01:44 +08:00

fix(prettier): 修复格式 cdeac6c849

xiaoxia added 1 commit 2026-07-29 22:33:07 +08:00

fix(lint): Prettier format useTitleMutations ad2cf754a7

auto-approve-bot approved these changes 2026-07-29 22:43:01 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-29 22:43:01 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

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

fix: 删除成功提示移到deleteMutation的onSuccess中 209c1f62b4

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

fix: 移除handleDelete中同步的删除成功提示（已移至onSuccess） f1691e08ca

auto-approve-bot commented 2026-07-30 01:06:16 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

1. [apps/web/src/pages/titles/hooks/useTitleLibrary/useTitleFilters.ts: 第10行] 移除未使用的参数

   • 具体内容：函数参数 _categories 被传入但在函数体内未被使用。虽然加下划线表示有意忽略，但既然不需要，建议直接从参数列表中移除，保持接口简洁。

2. [apps/web/src/pages/titles/hooks/useTitleLibrary/useTitleData.ts: 第44行] 优化 activeCategory 的计算性能

   • 具体内容：activeCategory 是通过 find 计算得出的，每次组件渲染都会生成一个新的引用（即使值未变）。由于它是 useTitleFilters 中 filteredTitles 的依赖项，这会导致筛选逻辑在每次渲染时都重新执行。建议使用 useMemo 包裹 activeCategory 的计算，仅在 categories 或 activeCatId 变化时重新计算。

✅ 良好实践

• 代码重构：将庞大的 Hook 拆分为 Data、Filters、Mutations、Actions 四个职责单一的 Hook，显著提升了代码的可维护性和可测试性。
• 逻辑修复：在 useTitleMutations.ts 中，将删除成功的提示（message.success）从调用处移到了 onSuccess 回调中，修复了原代码中无论请求成功与否都会提示“已删除”的逻辑缺陷。
• 冒烟测试：新增 smoke.test.tsx 确保模块加载无语法错误，符合工程化最佳实践。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/web/src/pages/titles/hooks/useTitleLibrary/useTitleFilters.ts: 第10行] 移除未使用的参数** - 具体内容：函数参数 `_categories` 被传入但在函数体内未被使用。虽然加下划线表示有意忽略，但既然不需要，建议直接从参数列表中移除，保持接口简洁。 2. **[apps/web/src/pages/titles/hooks/useTitleLibrary/useTitleData.ts: 第44行] 优化 `activeCategory` 的计算性能** - 具体内容：`activeCategory` 是通过 `find` 计算得出的，每次组件渲染都会生成一个新的引用（即使值未变）。由于它是 `useTitleFilters` 中 `filteredTitles` 的依赖项，这会导致筛选逻辑在每次渲染时都重新执行。建议使用 `useMemo` 包裹 `activeCategory` 的计算，仅在 `categories` 或 `activeCatId` 变化时重新计算。 ### ✅ 良好实践 - **代码重构**：将庞大的 Hook 拆分为 `Data`、`Filters`、`Mutations`、`Actions` 四个职责单一的 Hook，显著提升了代码的可维护性和可测试性。 - **逻辑修复**：在 `useTitleMutations.ts` 中，将删除成功的提示（`message.success`）从调用处移到了 `onSuccess` 回调中，修复了原代码中无论请求成功与否都会提示“已删除”的逻辑缺陷。 - **冒烟测试**：新增 `smoke.test.tsx` 确保模块加载无语法错误，符合工程化最佳实践。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 17:06:15 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit 0f27bfa999 into develop 2026-07-30 01:19:17 +08:00

auto-approve-bot deleted branch refactor/title-library-hook 2026-07-30 01:19:18 +08:00

gitea-actions bot commented 2026-07-30 01:23:09 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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