@tinkcarlos/skillora 0.2.0

package/.claude/skills/code-review/references/mandatory-checklist.md

@@ -0,0 +1,288 @@
+# 强制执行检查清单 (Mandatory Review Checklist)
+
+> **🔴 NON-NEGOTIABLE**: 每次代码审查必须逐项检查以下清单，不得跳过任何项目。
+
+---
+
+## 🚨 执行规则 (Execution Rules)
+
+### 强制执行原则
+
+| 规则 | 说明 | 违反后果 |
+|------|------|----------|
+| **逐项检查** | 必须检查清单中的每一项 | 审查无效 |
+| **记录证据** | 每项检查必须记录结果 | 审查无效 |
+| **不得跳过** | 即使"看起来没问题"也必须检查 | 审查无效 |
+| **阻塞式门控** | P0 问题必须修复后才能通过 | 代码不得合并 |
+
+### 检查顺序 (MUST FOLLOW)
+
+```
+1. 功能完整性检查 (Phase 0) → 阻塞门控
+2. 逻辑闭环检查 → 阻塞门控
+3. 前端/后端常见问题检查
+4. 安全检查
+5. 性能检查
+6. 编码规范检查
+7. 生成报告
+```
+
+---
+
+## 📋 Phase 0: 功能完整性检查 (BLOCKING)
+
+> ⛔ 此阶段失败 = 审查终止，返回 BLOCKED
+
+### 0.1 应用启动检查
+
+| # | 检查项 | 通过标准 | 结果 | 证据 |
+|---|--------|----------|------|------|
+| 0.1.1 | 后端启动 | 无错误启动 | ⬜ | |
+| 0.1.2 | 前端启动 | 无错误启动 | ⬜ | |
+| 0.1.3 | 数据库连接 | 连接成功 | ⬜ | |
+| 0.1.4 | 健康检查 | 返回 200 | ⬜ | |
+
+### 0.2 核心流程检查
+
+| # | 检查项 | 通过标准 | 结果 | 证据 |
+|---|--------|----------|------|------|
+| 0.2.1 | 登录流程 | 完整可用（非占位符） | ⬜ | |
+| 0.2.2 | 路由守卫 | 未登录被拦截 | ⬜ | |
+| 0.2.3 | Token 存储 | 登录后正确存储 | ⬜ | |
+| 0.2.4 | Token 发送 | 请求携带 Token | ⬜ | |
+| 0.2.5 | 主功能流程 | 端到端可用 | ⬜ | |
+
+### 0.3 API 契约检查
+
+| # | 检查项 | 通过标准 | 结果 | 证据 |
+|---|--------|----------|------|------|
+| 0.3.1 | 请求字段匹配 | 前后端字段名一致 | ⬜ | |
+| 0.3.2 | 响应字段匹配 | 前端正确解析响应 | ⬜ | |
+| 0.3.3 | SSE/WS 事件匹配 | 事件名和字段一致 | ⬜ | |
+
+---
+
+## 📋 Phase 1: 逻辑闭环检查 (BLOCKING)
+
+> ⛔ 逻辑不闭环 = P0 问题，必须修复
+
+### 1.1 条件分支完整性
+
+| # | 检查项 | 通过标准 | 结果 |
+|---|--------|----------|------|
+| 1.1.1 | if 有 else | 所有 if 都有对应处理 | ⬜ |
+| 1.1.2 | switch 有 default | 所有 switch 有 default | ⬜ |
+| 1.1.3 | try 有 catch | 所有 try 有错误处理 | ⬜ |
+| 1.1.4 | Promise 有 catch | 所有 Promise 有错误处理 | ⬜ |
+| 1.1.5 | async 有 try-catch | 所有 async 函数有错误处理 | ⬜ |
+
+### 1.2 循环完整性
+
+| # | 检查项 | 通过标准 | 结果 |
+|---|--------|----------|------|
+| 1.2.1 | 循环有终止条件 | 不会无限循环 | ⬜ |
+| 1.2.2 | break 后有处理 | break 后逻辑完整 | ⬜ |
+| 1.2.3 | continue 后有处理 | continue 后逻辑完整 | ⬜ |
+| 1.2.4 | 空数组处理 | 空数组不会报错 | ⬜ |
+| 1.2.5 | 大数据量处理 | 大数据不会卡死 | ⬜ |
+
+### 1.3 状态流转完整性
+
+| # | 检查项 | 通过标准 | 结果 |
+|---|--------|----------|------|
+| 1.3.1 | 初始状态定义 | 有明确初始状态 | ⬜ |
+| 1.3.2 | 状态转换合法 | 只能按规则转换 | ⬜ |
+| 1.3.3 | 终态可达 | 能到达终止状态 | ⬜ |
+| 1.3.4 | 异常状态处理 | 异常状态有恢复机制 | ⬜ |
+
+### 1.4 数据流完整性
+
+| # | 检查项 | 通过标准 | 结果 |
+|---|--------|----------|------|
+| 1.4.1 | 输入验证 | 所有输入都验证 | ⬜ |
+| 1.4.2 | 输出格式化 | 输出格式正确 | ⬜ |
+| 1.4.3 | 数据转换 | 转换逻辑正确 | ⬜ |
+| 1.4.4 | 空值处理 | null/undefined 处理 | ⬜ |
+| 1.4.5 | 边界值处理 | 边界值不会出错 | ⬜ |
+
+---
+
+## 📋 Phase 2: 前端检查清单
+
+### 2.1 UI 状态完整性
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 2.1.1 | Loading 状态 | 加载时有提示 | P1 | ⬜ |
+| 2.1.2 | Error 状态 | 错误时有提示 | P0 | ⬜ |
+| 2.1.3 | Empty 状态 | 空数据有提示 | P1 | ⬜ |
+| 2.1.4 | Success 状态 | 成功时有反馈 | P2 | ⬜ |
+| 2.1.5 | Disabled 状态 | 禁用时有样式 | P2 | ⬜ |
+
+### 2.2 交互完整性
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 2.2.1 | 按钮防重复点击 | 提交后禁用 | P0 | ⬜ |
+| 2.2.2 | 表单验证 | 提交前验证 | P0 | ⬜ |
+| 2.2.3 | 确认弹窗 | 危险操作有确认 | P1 | ⬜ |
+| 2.2.4 | 取消操作 | 可以取消进行中操作 | P2 | ⬜ |
+| 2.2.5 | 键盘支持 | 支持 Enter/Esc | P2 | ⬜ |
+
+### 2.3 React/Vue 特定检查
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 2.3.1 | useEffect 依赖 | 依赖数组正确 | P0 | ⬜ |
+| 2.3.2 | useEffect 清理 | 有 cleanup 函数 | P0 | ⬜ |
+| 2.3.3 | 闭包陷阱 | 无过期闭包 | P0 | ⬜ |
+| 2.3.4 | key 属性 | 列表有唯一 key | P1 | ⬜ |
+| 2.3.5 | 条件渲染 | 条件完整 | P1 | ⬜ |
+
+---
+
+## 📋 Phase 3: 后端检查清单
+
+### 3.1 API 完整性
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 3.1.1 | 参数验证 | 所有参数都验证 | P0 | ⬜ |
+| 3.1.2 | 权限检查 | 有权限验证 | P0 | ⬜ |
+| 3.1.3 | 错误响应 | 错误有明确响应 | P1 | ⬜ |
+| 3.1.4 | 成功响应 | 成功有正确响应 | P1 | ⬜ |
+| 3.1.5 | 超时处理 | 有超时机制 | P1 | ⬜ |
+
+### 3.2 数据库操作
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 3.2.1 | 事务边界 | 事务范围正确 | P0 | ⬜ |
+| 3.2.2 | N+1 查询 | 无 N+1 问题 | P1 | ⬜ |
+| 3.2.3 | 索引使用 | 查询使用索引 | P1 | ⬜ |
+| 3.2.4 | 连接释放 | 连接正确释放 | P0 | ⬜ |
+| 3.2.5 | SQL 注入 | 使用参数化查询 | P0 | ⬜ |
+
+### 3.3 并发安全
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 3.3.1 | 竞态条件 | 无竞态条件 | P0 | ⬜ |
+| 3.3.2 | 死锁风险 | 无死锁风险 | P0 | ⬜ |
+| 3.3.3 | 共享状态 | 共享状态安全 | P0 | ⬜ |
+| 3.3.4 | 幂等性 | 写操作幂等 | P1 | ⬜ |
+
+---
+
+## 📋 Phase 4: 安全检查清单
+
+### 4.1 认证授权
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 4.1.1 | 密码加密 | 使用 bcrypt/argon2 | P0 | ⬜ |
+| 4.1.2 | Token 验证 | 验证签名和过期 | P0 | ⬜ |
+| 4.1.3 | 权限检查 | 每个端点都检查 | P0 | ⬜ |
+| 4.1.4 | 会话管理 | 登出时失效 | P1 | ⬜ |
+
+### 4.2 输入输出
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 4.2.1 | XSS 防护 | 输出转义 | P0 | ⬜ |
+| 4.2.2 | SQL 注入 | 参数化查询 | P0 | ⬜ |
+| 4.2.3 | 命令注入 | 不用 shell=True | P0 | ⬜ |
+| 4.2.4 | 路径遍历 | 路径验证 | P0 | ⬜ |
+
+### 4.3 敏感数据
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 4.3.1 | 密钥管理 | 不硬编码密钥 | P0 | ⬜ |
+| 4.3.2 | 日志脱敏 | 不记录敏感信息 | P0 | ⬜ |
+| 4.3.3 | 错误信息 | 不暴露内部细节 | P1 | ⬜ |
+
+---
+
+## 📋 Phase 5: 编码规范检查
+
+### 5.1 代码风格
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 5.1.1 | 命名规范 | 符合项目规范 | P2 | ⬜ |
+| 5.1.2 | 代码格式 | 通过 lint 检查 | P2 | ⬜ |
+| 5.1.3 | 注释完整 | 复杂逻辑有注释 | P3 | ⬜ |
+| 5.1.4 | 类型定义 | TypeScript 类型完整 | P2 | ⬜ |
+
+### 5.2 代码质量
+
+| # | 检查项 | 通过标准 | 严重级别 | 结果 |
+|---|--------|----------|----------|------|
+| 5.2.1 | 函数长度 | 单函数 < 50 行 | P3 | ⬜ |
+| 5.2.2 | 圈复杂度 | < 10 | P2 | ⬜ |
+| 5.2.3 | 重复代码 | 无明显重复 | P2 | ⬜ |
+| 5.2.4 | 魔法数字 | 使用常量 | P3 | ⬜ |
+
+---
+
+## 📊 审查结果汇总模板
+
+```markdown
+## Code Review 强制检查结果
+
+### 检查统计
+| 阶段 | 总项数 | 通过 | 失败 | 跳过 |
+|------|--------|------|------|------|
+| Phase 0 功能完整性 | | | | |
+| Phase 1 逻辑闭环 | | | | |
+| Phase 2 前端检查 | | | | |
+| Phase 3 后端检查 | | | | |
+| Phase 4 安全检查 | | | | |
+| Phase 5 编码规范 | | | | |
+
+### 问题汇总
+| 严重级别 | 数量 | 必须修复 |
+|----------|------|----------|
+| P0 | | ✅ 必须 |
+| P1 | | ✅ 必须 |
+| P2 | | ⚠️ 建议 |
+| P3 | | 💡 可选 |
+
+### 最终判定
+- [ ] **APPROVED** - 无 P0/P1 问题
+- [ ] **CHANGES REQUESTED** - 有 P0/P1 问题需修复
+- [ ] **BLOCKED** - Phase 0/1 未通过
+
+### 详细问题列表
+| # | 文件:行号 | 问题描述 | 严重级别 | 修复建议 |
+|---|-----------|----------|----------|----------|
+| 1 | | | | |
+| 2 | | | | |
+```
+
+---
+
+## 🔴 违规处理
+
+### 跳过检查项的后果
+
+1. **审查无效** - 必须重新审查
+2. **问题追溯** - 如果上线后出问题，追溯审查责任
+3. **流程改进** - 记录到 code-review-record.md
+
+### 强制执行机制
+
+```markdown
+## 审查完成确认
+
+我确认已完成以下检查：
+- [ ] Phase 0 功能完整性检查 - 全部通过
+- [ ] Phase 1 逻辑闭环检查 - 全部通过
+- [ ] Phase 2-5 各项检查 - 已记录结果
+- [ ] 所有 P0/P1 问题已标记
+- [ ] 审查报告已生成
+
+审查人: _______________
+日期: _______________
+```