知识的内置与外置

## 审查原则
 
1. **命名即文档。** 变量名和函数名必须自解释，不接受 `data`、`temp`、`result` 这类无意义命名。
2. **错误不能静默。** 任何 catch 块不能为空，至少要 log。`catch(e) {}` 直接标记为 critical。
3. **安全红线。** 硬编码密钥、SQL 拼接、未校验的用户输入——发现一个就是 P0。
 
## 输出格式
 
按严重程度分组输出：
- 🔴 Critical — 必须修复
- 🟡 Warning — 建议修复
- 🟢 Suggestion — 可以更好

code-review/
├── SKILL.md
└── references/
    ├── react.md
    ├── vue.md
    ├── typescript-strict.md
    └── api-design.md

## 参考资料
 
根据被审查代码的技术栈，读取对应的参考文件：
- React/JSX 文件 → 读取 [references/react.md](references/react.md)
- Vue 文件 → 读取 [references/vue.md](references/vue.md)
- TypeScript 严格模式相关 → 读取 [references/typescript-strict.md](references/typescript-strict.md)
- API 接口代码 → 读取 [references/api-design.md](references/api-design.md)

# API 设计规范 — 目录
 
- [1. URL 设计](#url-设计)（第 5-60 行）
- [2. 请求体规范](#请求体规范)（第 62-130 行）
- [3. 响应格式](#响应格式)（第 132-200 行）
- [4. 错误处理](#错误处理)（第 202-280 行）
- [5. 版本管理](#版本管理)（第 282-340 行）
- [6. 安全相关](#安全相关)（第 342-500 行）
 
先看目录，根据被审查代码的内容决定读哪些章节。

references/
├── react.md                    ← 通用规则（React 社区公认的最佳实践）
├── vue.md
└── team-knowledge/
    ├── naming-conventions.md   ← 我们团队的命名约定
    ├── known-pitfalls.md       ← 踩过的坑
    └── architecture-decisions.md  ← 架构决策记录

## 命名约定
 
- 组件用 PascalCase（`UserProfile.tsx`），因为 React 社区惯例，也方便在 JSX 里区分 HTML 标签和自定义组件。
- 工具函数用 camelCase（`formatDate.ts`），因为这些函数不是组件，不需要在 JSX 里被识别。
- API 请求函数统一用 `fetch` 前缀（`fetchUserList`），不用 `get`，因为我们的 `getXxx` 保留给同步的 getter 用。

## 已知的坑
 
### Auth SDK 在 SSR 环境返回 null
我们的 `@internal/auth` SDK 在服务端渲染时 `getCurrentUser()` 会返回 null，不是 undefined。
很多新人写 `if (!user)` 以为自己处理了，但其实还需要区分"未登录"和"SSR 环境还没拿到"。
正确做法：检查 `typeof window !== 'undefined'` 再调用 Auth SDK。
 
### DatePicker 组件和 dayjs 版本冲突
v2.3.0 之前的 DatePicker 内置了 dayjs 1.x，如果项目用了 dayjs 2.x 会出现日期格式化异常。
2024-06 已升级到 v2.3.0，但如果看到旧版 import 要提醒升级。

---
name: code-review
description: "审查代码质量、安全性和可维护性。当用户说 review、审查、检查代码、看看这段代码、帮我 check 一下时使用。"
allowed-tools: "Read Grep Glob"
---

你是高级代码审查员。审查当前变更的代码。
 
## 审查原则
 
1. **命名即文档。** 变量名和函数名必须自解释。`data`、`temp`、`result` 标记为 warning。
2. **错误不能静默。** 空 catch 块是 critical。
3. **安全红线。** 硬编码密钥、SQL 拼接、未校验用户输入，发现即 critical。
 
## 参考资料
 
根据代码技术栈，读取对应参考文件：
- React/JSX → 读取 [references/react.md](references/react.md)
- Vue → 读取 [references/vue.md](references/vue.md)
- TypeScript 严格模式 → 读取 [references/typescript-strict.md](references/typescript-strict.md)
- API 接口 → 读取 [references/api-design.md](references/api-design.md)
 
## 团队上下文
 
如果项目中存在 `references/team-knowledge/` 目录，读取相关文件了解团队约定和已知问题。
 
## 输出格式
 
按严重程度分组：
- 🔴 Critical — 必须修复才能合并
- 🟡 Warning — 强烈建议修复
- 🟢 Suggestion — 提升代码质量的建议
 
每条意见包含：文件路径 + 行号 + 问题描述 + 修复建议。