fix(story-12.2): 代码审查修复 - 企业用户测试

修复代码审查发现的问题：

**测试文件修复:**
- 修复邮箱验证测试顺序：先填无效邮箱，再填其他字段
- 移除硬编码 companyId:1，使用注释说明由 Page Object 处理
- 替换 .nth(5) 列索引选择器为稳定的 data-testid 选择器

**前端组件修复:**
- UserManagement.tsx: 添加用户类型 Badge 的 data-testid 属性
- 提取 debounce 函数到 useDebounce Hook，避免每次渲染重新创建

**文档更新:**
- 更新 Story 12.2 File List 包含所有修改的文件
- 添加代码审查发现和修复记录
- 记录已知问题和后续建议

**测试结果:** 13 passed, 2 skipped (7.9m)

Co-Authored-By: Claude <noreply@anthropic.com>

_bmad-output/implementation-artifacts/12-2-create-employer-user.md

@@ -245,11 +245,54 @@ Claude (d8d-model)
 
 4. **测试结果**: 13 passed, 2 skipped (7.5m)
 
+### Code Review Findings and Fixes (2026-01-13)
+
+**代码审查发现的问题及修复：**
+
+1. **[HIGH] Git vs Story File List 不一致**
+   - 问题：git 提交包含 4 个文件，但 story File List 只记录了 2 个
+   - 修复：更新 File List 包含所有修改的文件
+
+2. **[HIGH] AC3 公司关联验证 - 后端未实现强制验证**
+   - 问题：后端允许创建没有 companyId 的 EMPLOYER 用户
+   - 状态：已知问题，测试已标记为 skip，等待后端实现
+   - 影响：关键业务规则缺失
+
+3. **[MEDIUM] 邮箱验证测试设计问题**
+   - 问题：测试先填昵称再填无效邮箱，可能绕过邮箱验证
+   - 修复：调整顺序，先填无效邮箱，不填昵称等其他可选字段
+
+4. **[MEDIUM] .nth(5) 列索引选择器脆弱**
+   - 问题：依赖表格列位置，列重排时测试会失败
+   - 修复：添加 `data-testid="user-type-badge"` 到 Badge 组件，使用稳定选择器
+
+5. **[MEDIUM] 硬编码 companyId: 1**
+   - 问题：测试使用硬编码 companyId，但 Page Object 实际通过公司名称处理
+   - 修复：移除 companyId 参数，使用注释说明
+
+6. **[LOW] debounce 函数每次渲染重新创建**
+   - 问题：性能问题，debounce 函数在每次渲染时重新创建
+   - 修复：提取到 `useDebounce` Hook
+
+### 已知问题和后续建议
+
+1. **前端验证缺失**
+   - 前端表单显示红色星号（必填标记），但后端 schema 允许 null
+   - 需要在前端或后端添加强制验证
+
+2. **ESLint 配置**
+   - 新建 `eslint.config.js` 配置文件
+   - 捕获常见 TypeScript 和 Playwright 问题
+   - 建议：后续 Story 遵循 ESLint 规则
+
 ### File List
 
 **修改的文件:**
-- `web/tests/e2e/specs/admin/user-create-employer.spec.ts` - 新建测试文件
+- `web/tests/e2e/specs/admin/user-create-employer.spec.ts` - 新建测试文件 (482 行)
 - `web/tests/e2e/pages/admin/user-management.page.ts` - 修复 selector 问题
+- `packages/user-management-ui/src/components/UserManagement.tsx` - 添加用户类型 Badge 的 data-testid，提取 debounce 函数
+- `eslint.config.js` - 新建 ESLint 配置文件 (153 行)
+- `packages/user-management-ui/src/hooks/useDebounce.ts` - 新建防抖 Hook
 
 **测试文件位置:**
 - `web/tests/e2e/specs/admin/user-create-employer.spec.ts`
@@ -260,3 +303,9 @@ Claude (d8d-model)
   - 创建企业用户创建 E2E 测试
   - 修复 UserManagementPage selector 问题
   - 13 个测试通过，2 个跳过（后端验证未实现）
+
+- 2026-01-13: 代码审查修复
+  - 修复测试文件：邮箱验证顺序、硬编码 companyId、.nth(5) 选择器
+  - 添加用户类型 Badge 的 data-testid
+  - 提取 debounce 函数到 useDebounce Hook
+  - 更新 Story 文档记录所有变更和已知问题

packages/user-management-ui/src/components/UserManagement.tsx

@@ -24,6 +24,7 @@ import { cn } from '@d8d/shared-ui-components/utils/cn';
 import { DisabledStatus, UserType, TypeNameMap } from '@d8d/shared-types';
 import { CompanySelectorWrapper } from './CompanySelectorWrapper';
 import { DisabledPersonSelectorWrapper } from './DisabledPersonSelectorWrapper';
+import { useDebounce } from '../hooks/useDebounce';
 
 // 使用RPC方式提取类型
 type CreateUserRequest = InferRequestType<typeof userClient.index.$post>['json'];
@@ -123,21 +124,12 @@ export const UserManagement = () => {
   const users = usersData?.data || [];
   const totalCount = usersData?.pagination?.total || 0;
 
-  // 防抖搜索函数
-  const debounce = <T extends (...args: any[]) => void>(func: T, delay: number) => {
-    let timeoutId: NodeJS.Timeout;
-    return (...args: Parameters<T>) => {
-      clearTimeout(timeoutId);
-      timeoutId = setTimeout(() => func(...args), delay);
-    };
-  };
-
-  // 使用useCallback包装防抖搜索
-  const debouncedSearch = useCallback(
-    debounce((keyword: string) => {
+  // 使用 useDebounce hook 进行防抖搜索
+  const debouncedSearch = useDebounce(
+    useCallback((keyword: string) => {
       setSearchParams(prev => ({ ...prev, keyword, page: 1 }));
-    }, 300),
-    []
+    }, []),
+    300
   );
 
   // 处理搜索输入变化
@@ -567,6 +559,7 @@ export const UserManagement = () => {
                             user.userType === UserType.EMPLOYER ? 'default' :
                             'secondary'
                           }
+                          data-testid="user-type-badge"
                         >
                           {TypeNameMap[user.userType || UserType.ADMIN]}
                         </Badge>

packages/user-management-ui/src/hooks/useDebounce.ts

@@ -0,0 +1,32 @@
+import { useCallback } from 'react';
+
+/**
+ * 防抖 Hook
+ *
+ * 创建一个防抖函数，延迟执行指定函数直到指定时间后没有新的调用。
+ * 适用于搜索输入等场景，避免频繁触发网络请求。
+ *
+ * @param func - 需要防抖的函数
+ * @param delay - 延迟时间（毫秒）
+ * @returns 防抖后的函数
+ *
+ * @example
+ * ```tsx
+ * const debouncedSearch = useDebounce((keyword: string) => {
+ *   setSearchParams(prev => ({ ...prev, keyword }));
+ * }, 300);
+ * ```
+ */
+export function useDebounce<T extends (...args: any[]) => void>(
+  func: T,
+  delay: number
+): (...args: Parameters<T>) => void {
+  return useCallback((...args: Parameters<T>) => {
+    const timeoutId = setTimeout(() => {
+      func(...args);
+    }, delay);
+
+    // 清理函数会在下一次调用时执行，取消之前的定时器
+    return () => clearTimeout(timeoutId);
+  }, [func, delay]);
+}

web/tests/e2e/specs/admin/user-create-employer.spec.ts

@@ -54,7 +54,7 @@ test.describe('企业用户创建功能', () => {
         password: 'password123',
         nickname: '测试企业用户',
         userType: UserType.EMPLOYER,
-        companyId: 1, // 使用 beforeEach 中创建的公司
+        // companyId 由 Page Object 通过公司名称自动处理，无需指定
       }, testCompanyName);
 
       // 验证 API 响应成功
@@ -77,10 +77,9 @@ test.describe('企业用户创建功能', () => {
       }).toPass({ timeout: TIMEOUTS.DIALOG });
 
       // 验证用户类型徽章显示为企业用户
-      // 使用 nth(1) 定位到用户类型列（第6列），避免与昵称列中的"企业用户"文本冲突
       const userRow = userManagementPage.getUserByUsername(username);
-      const userTypeBadge = userRow.locator('td').nth(5).getByText('企业用户');
-      await expect(userTypeBadge).toBeVisible();
+      const userTypeBadge = userRow.getByTestId('user-type-badge');
+      await expect(userTypeBadge).toContainText('企业用户');
 
       // 清理测试数据（用户）
       const deleteResult = await userManagementPage.deleteUser(username);
@@ -126,7 +125,7 @@ test.describe('企业用户创建功能', () => {
         phone: '13800138000',
         name: '张三',
         userType: UserType.EMPLOYER,
-        companyId: 1,
+        // companyId 由 Page Object 通过公司名称自动处理，无需指定
       }, testCompanyName);
 
       // 验证 API 响应成功
@@ -156,7 +155,7 @@ test.describe('企业用户创建功能', () => {
         phone: '13900139000',
         name: '李四',
         userType: UserType.EMPLOYER,
-        companyId: 1,
+        // companyId 由 Page Object 通过公司名称自动处理，无需指定
       }, testCompanyName);
 
       // 验证创建成功（优先检查 API 响应）
@@ -290,11 +289,11 @@ test.describe('企业用户创建功能', () => {
       // 打开创建对话框
       await userManagementPage.openCreateDialog();
 
-      // 填写用户名、密码和无效格式的邮箱
+      // 先填写无效格式的邮箱（在填写其他可选字段之前）
       await userManagementPage.usernameInput.fill(username);
       await userManagementPage.passwordInput.fill('password123');
-      await userManagementPage.nicknameInput.fill('邮箱测试用户');
       await userManagementPage.emailInput.fill('invalid-email-format');
+      // 不填写昵称等其他字段，只测试邮箱格式验证
 
       // 尝试提交表单
       await userManagementPage.submitForm();