# 故事007.029: 单租户供应商管理界面独立包实现

## 状态

Completed

## 故事

**作为** 系统管理员，
**我想要** 有一个独立的单租户供应商管理界面包，
**以便** 可以在单租户系统中独立管理供应商和联系人信息，而不影响现有的多租户系统。

## 验收标准

1. **AC 1**: 成功创建单租户供应商管理界面包 `@d8d/supplier-management-ui`，包含正确的包配置和依赖管理
2. **AC 2**: 复制前端供应商管理界面 `web/src/client/admin/pages/Suppliers.tsx` 为单租户供应商管理界面包
3. **AC 3**: 实现完整的供应商CRUD操作和联系人管理
4. **AC 4**: 基于React + TypeScript + TanStack Query + React Hook Form技术栈
5. **AC 5**: 依赖共享UI组件包 `@d8d/shared-ui-components`
6. **AC 6**: 依赖供应商模块包 `@d8d/supplier-module`
7. **AC 7**: 提供workspace包依赖复用机制
8. **AC 8**: 支持独立测试和部署
9. **AC 9**: 验证现有功能无回归

## 任务 / 子任务

- [x] 任务 1 (AC: 1, 7): 创建单租户供应商管理界面包结构
  - [x] 创建包目录：`packages/supplier-management-ui/`
  - [x] 创建基础包结构：`src/`、`tests/`、`package.json`
  - [x] 配置包依赖和构建脚本

- [x] 任务 2 (AC: 1): 配置包依赖和构建
  - [x] 创建 `packages/supplier-management-ui/package.json` 包配置 [参考: packages/user-management-ui/package.json]
  - [x] 添加依赖：`@d8d/shared-ui-components`、`@d8d/supplier-module`
  - [x] 配置构建脚本和TypeScript配置
  - [x] 创建 `packages/supplier-management-ui/tsconfig.json` TypeScript配置 [参考: packages/user-management-ui/tsconfig.json]
  - [x] 创建 `packages/supplier-management-ui/vitest.config.ts` 测试配置 [参考: packages/user-management-ui/vitest.config.ts]
  - [x] 创建 `packages/supplier-management-ui/tests/setup.ts` 测试设置文件 [参考: packages/user-management-ui/tests/setup.ts]
  - [x] 创建 `packages/supplier-management-ui/eslint.config.js` ESLint配置文件 [参考: packages/user-management-ui/eslint.config.js]
  - [x] 安装依赖：`cd packages/supplier-management-ui && pnpm install`

- [x] 任务 3 (AC: 3, 6): 创建RPC客户端架构和类型定义
  - [x] 创建单例模式的供应商客户端管理器 [参考: packages/user-management-ui/src/api/userClient.ts]
  - [x] 实现延迟初始化和客户端重置功能 [参考: packages/user-management-ui/src/api/userClient.ts:17-33]
  - [x] 使用Hono的InferRequestType和InferResponseType确保类型安全 [参考: packages/user-management-ui/src/components/UserManagement.tsx:26-29]
  - [x] 提供全局唯一的客户端实例管理 [参考: packages/user-management-ui/src/api/userClient.ts:4-15]
  - [x] 验证RPC客户端在主应用中的正确集成 [参考: web/src/client/api_init.ts]
  - [x] 实现类型安全的API调用模式 [参考: packages/user-management-ui/src/components/UserManagement.tsx:100-112]
  - [x] 调整API客户端，使用供应商模块包
  - [x] 创建 `packages/supplier-management-ui/src/types/supplier.ts` 类型定义
  - [x] 确保所有类型定义与供应商模块包对齐

- [x] 任务 4 (AC: 2, 3): 复制并调整供应商管理界面组件
  - [x] 复制 `web/src/client/admin/pages/Suppliers.tsx` 为 `packages/supplier-management-ui/src/components/SupplierManagement.tsx`
  - [x] 复制 `web/src/client/admin/components/SupplierSelector.tsx` 为 `packages/supplier-management-ui/src/components/SupplierSelector.tsx`
  - [x] 更新组件导入路径，使用共享UI组件包
  - [x] **规范**：共享UI包组件导入必须使用具体组件路径，如 `@d8d/shared-ui-components/components/ui/button`，避免从根导入
  - [x] 使用供应商客户端管理实例.get()来获取供应商RPC客户端
  - [x] **骨架屏优化**：确保骨架屏只在表格数据区域显示，不影响搜索框、筛选器等其他UI元素
  - [x] **RPC管理器规范**：确保所有API调用使用单例模式的供应商客户端管理器，支持延迟初始化和客户端重置功能
  - [x] **类型安全规范**：使用Hono的InferRequestType和InferResponseType确保客户端与后端API的类型一致性
  - [x] **SupplierSelector组件规范**：确保SupplierSelector组件使用单租户供应商模块API，替换原有的多租户API调用
  - [x] **参考UserSelector实现**：参考 `packages/user-management-ui/src/components/UserSelector.tsx` 的实现模式，包括API调用、test ID属性、类型定义等
  - [x] **SupplierSelector测试**：创建完整的SupplierSelector集成测试套件，验证API调用、供应商选择、占位符显示等功能 [参考: packages/user-management-ui/tests/integration/user-selector.integration.test.tsx]
  - [x] **测试稳定性改进**：为SupplierSelector组件添加test ID属性，确保测试中能够准确找到特定组件，避免页面中有多个combobox时的定位问题
  - [x] **包导出规范**：将SupplierSelector组件添加到包的导出接口中，确保可以被其他包使用

- [x] 任务 5 (AC: 3, 4): 实现完整的供应商管理功能
  - [x] 实现供应商列表查询和分页功能
  - [x] 实现供应商创建、编辑、删除功能
  - [x] 实现联系人管理和状态管理
  - [x] 实现搜索和过滤功能

- [x] 任务 6 (AC: 8): 创建测试套件
  - [x] 创建集成测试：`packages/supplier-management-ui/tests/integration/supplier-management.integration.test.tsx`
  - [x] 创建测试设置文件：`packages/supplier-management-ui/tests/setup.ts` [参考: packages/user-management-ui/tests/setup.ts]

- [x] 任务 7 (AC: 1, 7): 配置包导出接口
  - [x] 创建 `packages/supplier-management-ui/src/index.ts` 包导出主入口
  - [x] 确保所有导出组件、hook和类型定义正确
  - [x] 验证导出脚本正常工作

- [x] 任务 8 (AC: 9): 验证功能无回归
  - [x] 运行包构建：`pnpm build`
  - [x] 运行所有测试：`pnpm test`
  - [x] 验证供应商管理功能正常
  - [x] 验证与现有系统兼容性


- [x] 任务 10 (新增任务): 安装包依赖
  - [x] 在包目录中运行 `pnpm install` 安装所有依赖
  - [x] 验证依赖安装成功，无版本冲突
  - [x] 确保所有开发依赖正确安装

## Dev Notes

### 技术栈和架构上下文
- **技术栈**: React 19 + TypeScript + TanStack Query + React Hook Form [Source: architecture/tech-stack.md#现有技术栈维护]
- **前端框架**: React 19.1.0 用于用户界面构建 [Source: architecture/tech-stack.md#现有技术栈维护]
- **状态管理**: React Query 5.83.0 用于服务端状态管理 [Source: architecture/tech-stack.md#现有技术栈维护]
- **构建工具**: Vite 7.0.0 用于开发服务器和构建 [Source: architecture/tech-stack.md#现有技术栈维护]

### 项目结构
- **包位置**: `packages/supplier-management-ui/` [Source: architecture/source-tree.md#实际项目结构]
- **源码结构**:
  - `src/components/` - React组件
  - `src/hooks/` - 自定义React hooks
  - `src/api/` - API客户端
  - `src/types/` - TypeScript类型定义
  - `tests/unit/` - 单元测试
  - `tests/integration/` - 集成测试
- **依赖管理**: 使用pnpm workspace依赖管理 [Source: architecture/source-tree.md#集成指南]

### 依赖关系
- **共享UI组件包**: `@d8d/shared-ui-components` - 提供基础UI组件 [Source: architecture/source-tree.md#实际项目结构]
- **单租户供应商模块**: `@d8d/supplier-module` - 提供供应商管理API [Source: docs/prd/epic-007-multi-tenant-package-replication.md#供应商管理界面包]

### 从前一个故事吸取的经验教训
- **useQuery测试策略**: 使用真实的QueryClientProvider而不是mock react-query，在TestWrapper中提供完整的react-query上下文 [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试策略关键发现]
- **UI组件测试策略**: 使用data-testid进行元素定位比placeholder/role更准确稳定，避免因UI变化导致测试失败 [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试策略关键发现]
- **React Hook Form处理**: 需要过滤React Hook Form的props避免React警告，改进mock策略 [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试策略关键发现]
- **Router上下文**: 需要提供BrowserRouter上下文或mock useNavigate [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试策略关键发现]

### 测试标准
- **测试框架**: Vitest + Testing Library [Source: architecture/testing-strategy.md#单元测试]
- **测试位置**: `packages/supplier-management-ui/tests/unit/` 和 `packages/supplier-management-ui/tests/integration/` [Source: architecture/testing-strategy.md#单元测试]
- **测试覆盖率目标**: ≥ 80% 单元测试覆盖率 [Source: architecture/testing-strategy.md#各层覆盖率要求]
- **测试执行**: 使用 `pnpm test` 运行所有测试 [Source: architecture/testing-strategy.md#本地开发测试]
- **测试模式**: 遵循测试金字塔模型，包含单元测试、组件测试和集成测试 [Source: architecture/testing-strategy.md#测试金字塔策略]

### 关键实施要点
- **包命名**: 使用标准命名约定，不添加特殊后缀 [Source: docs/prd/epic-007-multi-tenant-package-replication.md#包命名约定]
- **API客户端**: 使用Hono客户端调用单租户供应商API [Source: docs/stories/007.015.auth-management-ui-package.story.md#任务-5]
- **导出接口**: 提供完整的组件、hook和类型定义导出 [Source: docs/stories/007.015.auth-management-ui-package.story.md#任务-7]
- **组件复用**: 基于现有供应商管理界面实现，确保功能完整性和一致性

### 供应商管理功能特性
- **供应商列表**: 支持分页、搜索、过滤功能
- **供应商CRUD**: 完整的创建、读取、更新、删除操作
- **联系人管理**: 供应商联系人信息管理
- **状态管理**: 供应商启用/禁用状态控制
- **表单验证**: 完整的表单验证和错误处理

## 测试

#### 测试标准和框架
- **测试框架**: Vitest 3.2.4 + Testing Library 16.3.0 [Source: architecture/testing-strategy.md#工具版本]
- **测试位置**:
  - 集成测试: `packages/supplier-management-ui/tests/integration/**/*.test.tsx`
  [Source: architecture/testing-strategy.md#单元测试]

#### 测试模式和策略
- **useQuery测试**: 使用真实的QueryClientProvider而不是mock react-query [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试策略关键发现]
- **元素定位**: 使用data-testid进行元素定位，比placeholder/role更准确稳定 [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试策略关键发现]
- **Mock策略**: 使用智能mock过滤React Hook Form props [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试架构改进]
- **测试工具**: 提供QueryClientProvider和必要的上下文 [Source: docs/stories/007.015.auth-management-ui-package.story.md#测试架构改进]

#### 特定测试要求
- **供应商CRUD测试**: 验证供应商创建、读取、更新、删除功能
- **联系人管理测试**: 验证联系人信息管理功能
- **搜索过滤测试**: 验证搜索和过滤功能正常工作
- **表单验证测试**: 验证表单验证和错误处理
- **API集成测试**: 验证与供应商模块的API集成

#### 测试执行命令
- 运行所有测试: `cd packages/supplier-management-ui && pnpm test`
- 运行单元测试: `cd packages/supplier-management-ui && pnpm test:unit`
- 运行集成测试: `cd packages/supplier-management-ui && pnpm test:integration`
- 生成覆盖率报告: `cd packages/supplier-management-ui && pnpm test:coverage`

## 变更日志

| 日期 | 版本 | 描述 | 作者 |
|------|------|------|------|
| 2025-11-16 | 1.0 | 初始故事创建 | Bob (Scrum Master) |

## Dev Agent Record

### 开发过程记录

**实施时间**: 2025-11-17
**开发代理**: Claude Code

#### 关键实施步骤
1. **包结构创建** - 成功创建了完整的供应商管理UI包结构，包括所有必要的配置文件和目录
2. **组件迁移** - 从现有供应商管理界面复制并调整了组件，确保与单租户架构兼容
3. **API客户端集成** - 实现了基于Hono RPC的单例模式供应商客户端管理器
4. **测试套件创建** - 创建了完整的集成测试套件，覆盖所有CRUD操作和错误处理
5. **表单验证修复** - 解决了表单提交测试问题，确保手机号字段验证正确
6. **骨架屏优化** - 优化了骨架屏显示，确保只在表格数据区域显示，不影响其他UI元素

#### 技术挑战和解决方案
- **表单提交问题**: 发现表单验证失败导致API调用未触发，通过添加手机号字段填写和修复表单处理函数参数解决
- **测试交互问题**: 使用`userEvent.click`替代`fireEvent.click`，更接近真实用户交互
- **骨架屏覆盖问题**: 重构骨架屏实现，确保搜索和按钮功能在加载状态下可用

#### 验证结果
- ✅ 所有4个集成测试通过
- ✅ 完整的CRUD流程正常工作
- ✅ API错误处理正常工作
- ✅ 搜索功能正常工作
- ✅ 供应商状态显示正常工作

### 技术实现细节
- **包名称**: `@d8d/supplier-management-ui`
- **主要组件**: `SupplierManagement`
- **API客户端**: 基于Hono RPC的单例模式
- **测试框架**: Vitest + Testing Library
- **表单处理**: React Hook Form + Zod schema验证

## QA Results

*此部分将在质量保证审查过程中由QA代理填充*