@llryiop/avatar-boot-cli 1.0.0

package/src/index.js

@@ -0,0 +1,78 @@
+import { Command } from 'commander';
+import chalk from 'chalk';
+import ora from 'ora';
+import { collectConfig } from './prompts.js';
+import { copyTemplate } from './template.js';
+import { transformProject } from './transform.js';
+
+export function run() {
+  const program = new Command();
+
+  program
+    .name('avatar-boot-cli')
+    .description('CLI tool for scaffolding AvatarBoot Java microservice projects')
+    .version('1.0.0');
+
+  program
+    .command('init [project-name]')
+    .description('Initialize a new AvatarBoot project')
+    .action(async (projectName) => {
+      console.log('');
+      console.log(chalk.blue.bold('  Avatar Boot CLI'));
+      console.log(chalk.gray('  AvatarBoot Java 微服务项目脚手架工具'));
+      console.log('');
+
+      try {
+        // 1. Collect configuration
+        const config = await collectConfig(projectName);
+
+        console.log('');
+
+        // 2. Copy template
+        const spinner = ora('正在复制模板文件...').start();
+        try {
+          copyTemplate(config.projectName);
+          spinner.succeed('模板文件复制完成');
+        } catch (err) {
+          spinner.fail('模板文件复制失败');
+          console.error(chalk.red(err.message));
+          process.exit(1);
+        }
+
+        // 3. Transform project
+        const transformSpinner = ora('正在生成项目...').start();
+        try {
+          transformProject(config.projectName, config);
+          transformSpinner.succeed('项目生成完成');
+        } catch (err) {
+          transformSpinner.fail('项目生成失败');
+          console.error(chalk.red(err.message));
+          process.exit(1);
+        }
+
+        // 4. Print success message
+        console.log('');
+        console.log(chalk.green.bold('  ✅ 项目创建成功!'));
+        console.log('');
+        console.log(chalk.white('  开始开发:'));
+        console.log(chalk.cyan(`    cd ${config.projectName}`));
+        console.log(chalk.cyan('    mvn clean compile'));
+        console.log('');
+        console.log(chalk.white('  项目结构:'));
+        console.log(chalk.gray(`    ${config.projectName}/`));
+        console.log(chalk.gray(`    ├── ${config.projectName}-api/        # API 模块 (接口定义)`));
+        console.log(chalk.gray(`    ├── ${config.projectName}-service/    # Service 模块 (业务实现)`));
+        console.log(chalk.gray(`    └── pom.xml`));
+        console.log('');
+      } catch (err) {
+        if (err.name === 'ExitPromptError') {
+          console.log(chalk.yellow('\n  已取消'));
+        } else {
+          console.error(chalk.red(`\n  错误: ${err.message}`));
+        }
+        process.exit(1);
+      }
+    });
+
+  program.parse();
+}

package/src/prompts.js

@@ -0,0 +1,78 @@
+import inquirer from 'inquirer';
+
+const STARTER_MODULES = [
+  { name: 'avatar-boot-starter-web (Web 基础设施)', value: 'web', checked: true },
+  { name: 'avatar-boot-starter-nacos (服务注册与配置)', value: 'nacos', checked: true },
+  { name: 'avatar-boot-starter-feign (服务间调用)', value: 'feign', checked: false },
+  { name: 'avatar-boot-starter-mysql (MySQL 数据库)', value: 'mysql', checked: false },
+  { name: 'avatar-boot-starter-redis (Redis 缓存)', value: 'redis', checked: false },
+  { name: 'avatar-boot-starter-kafka (Kafka 消息队列)', value: 'kafka', checked: false },
+  { name: 'avatar-boot-starter-rocketmq (RocketMQ 消息队列)', value: 'rocketmq', checked: false },
+  { name: 'avatar-boot-starter-job (XXL-Job 任务调度)', value: 'job', checked: false },
+  { name: 'avatar-boot-starter-oss (对象存储)', value: 'oss', checked: false },
+];
+
+export async function collectConfig(initialName) {
+  const questions = [
+    {
+      type: 'input',
+      name: 'projectName',
+      message: '项目名称:',
+      default: initialName || 'my-avatar-service',
+      validate: (input) => /^[a-z][a-z0-9-]*$/.test(input) || '项目名称只能包含小写字母、数字和连字符，且以字母开头',
+    },
+    {
+      type: 'input',
+      name: 'groupId',
+      message: 'Maven groupId:',
+      default: 'com.iflytek.avatar',
+      validate: (input) => /^[a-z][a-z0-9.]*$/.test(input) || 'groupId 格式不正确',
+    },
+    {
+      type: 'input',
+      name: 'avatarBootVersion',
+      message: 'Avatar Boot 版本:',
+      default: '1.0.0-SNAPSHOT',
+    },
+    {
+      type: 'checkbox',
+      name: 'starters',
+      message: '选择 Starter 模块:',
+      choices: STARTER_MODULES,
+    },
+    {
+      type: 'confirm',
+      name: 'includeExample',
+      message: '是否生成示例代码 (login 模块):',
+      default: true,
+    },
+    {
+      type: 'input',
+      name: 'nacosAddr',
+      message: 'Nacos 地址:',
+      default: '127.0.0.1:8848',
+      when: (answers) => answers.starters.includes('nacos'),
+    },
+    {
+      type: 'input',
+      name: 'dbUrl',
+      message: '数据库地址 (host:port/dbname):',
+      default: 'localhost:3306/avatar',
+      when: (answers) => answers.starters.includes('mysql'),
+    },
+    {
+      type: 'confirm',
+      name: 'includeAiConfig',
+      message: '是否生成 .claude/ AI Agent 配置:',
+      default: true,
+    },
+    {
+      type: 'input',
+      name: 'serverPort',
+      message: '服务端口:',
+      default: '8888',
+    },
+  ];
+
+  return inquirer.prompt(questions);
+}

package/src/template.js

@@ -0,0 +1,37 @@
+import fs from 'fs';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const TEMPLATE_DIR = path.join(__dirname, '..', 'templates');
+
+/**
+ * Copy the bundled template to the target directory
+ * @param {string} targetDir - Directory to copy into
+ */
+export function copyTemplate(targetDir) {
+  const absoluteTarget = path.resolve(targetDir);
+
+  if (fs.existsSync(absoluteTarget)) {
+    throw new Error(`Directory "${targetDir}" already exists`);
+  }
+
+  copyDirRecursive(TEMPLATE_DIR, absoluteTarget);
+}
+
+function copyDirRecursive(src, dest) {
+  fs.mkdirSync(dest, { recursive: true });
+  const entries = fs.readdirSync(src, { withFileTypes: true });
+
+  for (const entry of entries) {
+    const srcPath = path.join(src, entry.name);
+    const destPath = path.join(dest, entry.name);
+
+    if (entry.isDirectory()) {
+      copyDirRecursive(srcPath, destPath);
+    } else {
+      fs.copyFileSync(srcPath, destPath);
+    }
+  }
+}

package/src/transform.js

@@ -0,0 +1,172 @@
+import fs from 'fs';
+import path from 'path';
+import { groupIdToPath, getAllFiles, replaceInFile, renameDir } from './utils.js';
+
+const DEFAULT_GROUP_ID = 'com.iflytek.avatar';
+const DEFAULT_GROUP_PATH = 'com/iflytek/avatar';
+const DEFAULT_ARTIFACT_ID = 'avatar-template';
+const DEFAULT_APP_NAME = 'scaffold-template';
+const DEFAULT_NACOS_ADDR = '172.29.242.247:8848';
+
+const ALL_STARTERS = ['web', 'nacos', 'feign', 'mysql', 'redis', 'kafka', 'rocketmq', 'job', 'oss'];
+
+export function transformProject(projectDir, config) {
+  const {
+    projectName,
+    groupId = DEFAULT_GROUP_ID,
+    avatarBootVersion = '1.0.0',
+    starters = ['web', 'nacos'],
+    includeExample = true,
+    nacosAddr = '127.0.0.1:8848',
+    dbUrl,
+    includeAiConfig = true,
+    serverPort = '8888',
+  } = config;
+
+  const newGroupPath = groupIdToPath(groupId);
+  const apiModule = `${projectName}-api`;
+  const serviceModule = `${projectName}-service`;
+
+  // 1. Rename module directories
+  renameDir(
+    path.join(projectDir, 'avatar-scaffold-api'),
+    path.join(projectDir, apiModule)
+  );
+  renameDir(
+    path.join(projectDir, 'avatar-scaffold-service'),
+    path.join(projectDir, serviceModule)
+  );
+
+  // 2. Rename Java package directories if groupId changed
+  if (groupId !== DEFAULT_GROUP_ID) {
+    const apiOldPkg = path.join(projectDir, apiModule, 'src/main/java', DEFAULT_GROUP_PATH);
+    const apiNewPkg = path.join(projectDir, apiModule, 'src/main/java', newGroupPath);
+    if (fs.existsSync(apiOldPkg)) {
+      renameDir(apiOldPkg, apiNewPkg);
+      cleanEmptyDirs(path.join(projectDir, apiModule, 'src/main/java'));
+    }
+
+    const svcOldPkg = path.join(projectDir, serviceModule, 'src/main/java', DEFAULT_GROUP_PATH);
+    const svcNewPkg = path.join(projectDir, serviceModule, 'src/main/java', newGroupPath);
+    if (fs.existsSync(svcOldPkg)) {
+      renameDir(svcOldPkg, svcNewPkg);
+      cleanEmptyDirs(path.join(projectDir, serviceModule, 'src/main/java'));
+    }
+  }
+
+  // 3. Remove example code if not needed
+  if (!includeExample) {
+    const apiLoginDir = path.join(projectDir, apiModule, 'src/main/java', newGroupPath, 'login');
+    const svcLoginDir = path.join(projectDir, serviceModule, 'src/main/java', newGroupPath, 'login');
+    if (fs.existsSync(apiLoginDir)) fs.rmSync(apiLoginDir, { recursive: true });
+    if (fs.existsSync(svcLoginDir)) fs.rmSync(svcLoginDir, { recursive: true });
+  }
+
+  // 4. Remove .claude/ if not needed
+  if (!includeAiConfig) {
+    const claudeDir = path.join(projectDir, '.claude');
+    if (fs.existsSync(claudeDir)) fs.rmSync(claudeDir, { recursive: true });
+    const claudeMd = path.join(projectDir, 'CLAUDE.md');
+    if (fs.existsSync(claudeMd)) fs.unlinkSync(claudeMd);
+  }
+
+  // 5. Replace content in all text files
+  const textExtensions = ['.xml', '.java', '.yaml', '.yml', '.properties', '.md', '.txt'];
+  const allFiles = getAllFiles(projectDir);
+  const textFiles = allFiles.filter((f) =>
+    textExtensions.some((ext) => f.endsWith(ext))
+  );
+
+  const replacements = [
+    ['avatar-scaffold-api', apiModule],
+    ['avatar-scaffold-service', serviceModule],
+    ['avatar-template-api', `${projectName}-api`],
+    ['avatar-template-service', `${projectName}-service`],
+    ['avatar-template', projectName],
+    [DEFAULT_GROUP_ID, groupId],
+    [`package ${DEFAULT_GROUP_ID}`, `package ${groupId}`],
+    [`import ${DEFAULT_GROUP_ID}`, `import ${groupId}`],
+    [DEFAULT_APP_NAME, projectName],
+    [DEFAULT_NACOS_ADDR, nacosAddr],
+    ['port: 8888', `port: ${serverPort}`],
+    [`${DEFAULT_GROUP_ID}.Application`, `${groupId}.Application`],
+    ['<avatar.boot.version>1.0.0-SNAPSHOT</avatar.boot.version>', `<avatar.boot.version>${avatarBootVersion}</avatar.boot.version>`],
+  ];
+
+  for (const file of textFiles) {
+    replaceInFile(file, replacements);
+  }
+
+  // 6. Handle starter dependencies in service pom.xml
+  transformServicePom(path.join(projectDir, serviceModule, 'pom.xml'), starters);
+
+  // 7. Handle database URL in config
+  if (dbUrl && starters.includes('mysql')) {
+    const devYaml = path.join(projectDir, serviceModule, 'src/main/resources/application-dev.yaml');
+    if (fs.existsSync(devYaml)) {
+      replaceInFile(devYaml, [['localhost:3306/avatar', dbUrl]]);
+    }
+  }
+
+  // 8. Remove .DS_Store files
+  for (const file of allFiles) {
+    if (path.basename(file) === '.DS_Store') {
+      fs.unlinkSync(file);
+    }
+  }
+}
+
+function transformServicePom(pomPath, selectedStarters) {
+  if (!fs.existsSync(pomPath)) return;
+
+  let content = fs.readFileSync(pomPath, 'utf-8');
+
+  for (const starter of ALL_STARTERS) {
+    const artifactId = `avatar-boot-starter-${starter}`;
+    const depBlock = `        <dependency>\n            <groupId>com.iflytek.avatar.boot</groupId>\n            <artifactId>${artifactId}</artifactId>\n        </dependency>`;
+    const commentedPattern = new RegExp(
+      `<!--\\s*<dependency>\\s*\\n\\s*<groupId>com\\.iflytek\\.avatar\\.boot</groupId>\\s*\\n\\s*<artifactId>${artifactId}</artifactId>\\s*\\n\\s*</dependency>\\s*-->`,
+      's'
+    );
+    const uncommentedPattern = new RegExp(
+      `<dependency>\\s*\\n\\s*<groupId>com\\.iflytek\\.avatar\\.boot</groupId>\\s*\\n\\s*<artifactId>${artifactId}</artifactId>\\s*\\n\\s*</dependency>`,
+      's'
+    );
+
+    if (selectedStarters.includes(starter)) {
+      const commentedMatch = content.match(commentedPattern);
+      if (commentedMatch) {
+        content = content.replace(commentedMatch[0], depBlock);
+      } else if (!uncommentedPattern.test(content)) {
+        const insertPoint = content.indexOf('</dependencies>');
+        if (insertPoint !== -1) {
+          content = content.slice(0, insertPoint) + depBlock + '\n' + content.slice(insertPoint);
+        }
+      }
+    } else {
+      const uncommentedMatch = content.match(uncommentedPattern);
+      if (uncommentedMatch) {
+        content = content.replace(
+          uncommentedMatch[0],
+          `<!--${uncommentedMatch[0]}-->`
+        );
+      }
+    }
+  }
+
+  fs.writeFileSync(pomPath, content, 'utf-8');
+}
+
+function cleanEmptyDirs(dir) {
+  if (!fs.existsSync(dir)) return;
+  const entries = fs.readdirSync(dir, { withFileTypes: true });
+  for (const entry of entries) {
+    if (entry.isDirectory()) {
+      cleanEmptyDirs(path.join(dir, entry.name));
+    }
+  }
+  const remaining = fs.readdirSync(dir);
+  if (remaining.length === 0) {
+    fs.rmdirSync(dir);
+  }
+}

package/src/utils.js

@@ -0,0 +1,34 @@
+import fs from 'fs';
+import path from 'path';
+
+export function groupIdToPath(groupId) {
+  return groupId.replace(/\./g, '/');
+}
+
+export function getAllFiles(dir, fileList = []) {
+  const entries = fs.readdirSync(dir, { withFileTypes: true });
+  for (const entry of entries) {
+    const fullPath = path.join(dir, entry.name);
+    if (entry.isDirectory()) {
+      getAllFiles(fullPath, fileList);
+    } else {
+      fileList.push(fullPath);
+    }
+  }
+  return fileList;
+}
+
+export function replaceInFile(filePath, replacements) {
+  let content = fs.readFileSync(filePath, 'utf-8');
+  for (const [search, replace] of replacements) {
+    content = content.replaceAll(search, replace);
+  }
+  fs.writeFileSync(filePath, content, 'utf-8');
+}
+
+export function renameDir(oldPath, newPath) {
+  if (fs.existsSync(oldPath)) {
+    fs.mkdirSync(path.dirname(newPath), { recursive: true });
+    fs.renameSync(oldPath, newPath);
+  }
+}

package/templates/.claude/rules/architecture-redlines.md

@@ -0,0 +1,146 @@
+# 架构红线与通信协议
+
+> 规则类型: 🔴 强制执行（不可违反）
+> 更新频率: 极低（架构变更时更新）
+> 相关规则: [tech-stack.md](tech-stack.md) | [coding-standards.md](coding-standards.md) | [version.md](version.md)
+> 相关技能: [api-service-module-creator](../skills/api-service-module-creator/)
+
+---
+
+## 依赖管理红线
+
+> **完整规则详见**: [version.md](version.md) 和 [tech-stack.md](tech-stack.md)
+
+---
+
+## 架构设计红线
+
+### 4. API 与 Service 必须分离
+- API 模块只包含接口定义、DTO、Feign 客户端
+- Service 模块包含具体实现、业务逻辑、数据访问
+
+### 5. 功能模块必须垂直划分
+- 每个业务功能（如 login、user）独立成包
+- 功能包内包含该功能的所有层次（controller、service、dao）
+- 禁止水平分层（不要创建全局的 controller、service、dao 包）
+
+### 6. 统一返回格式
+- 所有 Controller 方法必须返回 `Result<T>` 类型
+- 使用 `com.iflytek.avatar.boot.entity.response.Result` 类
+- 禁止直接返回业务对象或自定义响应格式
+
+### 7. 服务间调用必须通过 Feign
+- 使用 `avatar-boot-starter-feign` 实现服务间调用
+- 禁止使用 RestTemplate 或 HttpClient 直接调用
+
+---
+
+## 安全红线
+
+### 8. 敏感信息禁止硬编码
+- 数据库密码、API 密钥等必须通过 Nacos 配置中心管理
+- 禁止在代码中硬编码任何敏感信息
+
+### 9. 高危漏洞必须立即修复
+- CVE 评分 >= 7.0 的漏洞必须立即处理
+- 定期运行 `mvn org.owasp:dependency-check-maven:check`
+
+---
+
+## REST API 通信规范
+
+- **请求方法**: GET（查询）、POST（创建）、PUT（更新）、DELETE（删除）
+- **URL 路径**: 使用 kebab-case，如 `/api/user-info`
+- **请求体**: JSON 格式，字段使用 camelCase
+- **响应格式**: 统一使用 `Result<T>` 包装
+
+### Result 响应格式
+
+```json
+// 成功响应
+{
+  "code": 200,
+  "message": "success",
+  "data": { }
+}
+
+// 失败响应
+{
+  "code": 400,
+  "message": "参数错误",
+  "data": null
+}
+```
+
+### Result 使用示例
+
+```java
+@RestController
+@RequestMapping("/api/user")
+public class UserController {
+
+    @GetMapping("/{id}")
+    public Result<UserResponse> getUser(@PathVariable Long id) {
+        UserResponse user = userService.getById(id);
+        return Result.success(user);
+    }
+
+    @PostMapping
+    public Result<Void> createUser(@RequestBody UserRequest request) {
+        userService.create(request);
+        return Result.success();
+    }
+
+    @ExceptionHandler(Exception.class)
+    public Result<Void> handleException(Exception e) {
+        return Result.fail(500, e.getMessage());
+    }
+}
+```
+
+---
+
+## 目录组织规范
+
+### API 模块（avatar-scaffold-api）包结构
+
+- `api/` - 接口定义（供外部调用的 REST API 接口）
+- `constant/` - 常量定义（业务常量、配置常量）
+- `enums/` - 枚举定义（业务枚举类型）
+- `exception/` - 异常定义（业务异常类）
+- `entity/request/` - 请求参数实体（DTO，用于接收客户端请求）
+- `entity/response/` - 响应结果实体（DTO，必须使用 `Result<T>` 包装）
+- `feign/` - Feign 客户端（用于服务间调用）
+
+### Service 模块（avatar-scaffold-service）包结构
+
+- `config/` - 配置类（Spring 配置、Bean 定义）
+- `controller/` - REST 控制器（实现 API 接口，返回 `Result<T>`）
+- `service/` - 业务逻辑层（Service 接口和实现类）
+- `dao/` - 数据访问层（MyBatis Mapper 接口）
+- `dao/beans/` - 数据库实体类（DO/PO，对应数据库表结构）
+- `utils/` - 工具类（功能模块内部使用）
+
+---
+
+## 违规检查清单
+
+### 架构检查
+- [ ] 是否遵循 API + Service 双模块结构
+- [ ] 是否按功能垂直划分包结构
+- [ ] Controller 是否返回 `Result<T>` 类型
+- [ ] 是否使用 Feign 进行服务间调用
+
+### 代码检查
+- [ ] 是否使用了 `jakarta.*` 命名空间
+- [ ] 是否有未处理的异常
+- [ ] 是否有硬编码的敏感信息
+- [ ] 是否有 `System.out.println()`
+
+### 依赖检查
+> **详细的依赖检查命令和工具请参考**: [version.md](version.md)
+
+- [ ] 是否使用了禁止的依赖（如 `javax.*`）
+- [ ] 是否在子模块中指定了版本
+- [ ] 是否直接引用了 Spring Boot 原生 starter
+- [ ] 是否配置了 iFlytek Maven 仓库

package/templates/.claude/rules/code-review-standards.md

@@ -0,0 +1,137 @@
+# 代码审查标准 (Code Review Standards)
+
+> 🔴 **强制执行** — 所有代码合并前必须通过代码审查
+> 📎 关联规则: [coding-standards.md](./coding-standards.md), [architecture-redlines.md](./architecture-redlines.md)
+> 📋 适用范围: 所有 Java 源代码、配置文件、SQL 脚本
+
+---
+
+## 审查维度
+
+代码审查覆盖以下五个核心维度，按优先级排序：
+
+1. **正确性 (Correctness)** — 代码逻辑是否正确，是否满足需求
+2. **安全性 (Security)** — 是否存在安全漏洞或敏感数据泄露
+3. **性能 (Performance)** — 是否存在性能隐患或资源浪费
+4. **可维护性 (Maintainability)** — 代码是否清晰、易读、易扩展
+5. **测试覆盖 (Test Coverage)** — 核心逻辑是否有单元测试覆盖
+
+---
+
+## 严重等级定义
+
+| 等级 | 标识 | 含义 | 处理要求 |
+|------|------|------|----------|
+| **Blocker** | 🚫 | 阻塞性问题，存在严重 Bug 或安全漏洞 | 必须修复后才能合并 |
+| **Major** | ⚠️ | 重要问题，影响代码质量或潜在风险 | 应当修复，特殊情况需说明理由 |
+| **Minor** | 💡 | 次要问题，代码风格或小优化 | 建议修复 |
+| **Suggestion** | 📝 | 可选改进，更好的实现方式 | 可选采纳 |
+
+---
+
+## 分层审查清单
+
+> 各层详细规范和代码示例请参考 [layered-architecture.md](layered-architecture.md) 和 [mybatis-plus.md](mybatis-plus.md)
+
+### Controller 层
+
+- [ ] 使用 `Result<T>` 统一返回格式
+- [ ] 参数校验注解完整 (`@Valid`, `@NotNull`, `@NotBlank`)
+- [ ] 使用 Request/Response DTO，禁止直接暴露 Entity
+- [ ] 无业务逻辑，仅做参数校验和 Service 调用
+- [ ] 异常由全局异常处理器处理，不做 try-catch
+
+### Service 层
+
+- [ ] 继承 `IService<T>` / `ServiceImpl<M, T>`
+- [ ] 写操作加 `@Transactional(rollbackFor = Exception.class)`
+- [ ] 只读操作加 `@Transactional(readOnly = true)`
+- [ ] 事务范围最小化，不包含 RPC / 文件 IO
+- [ ] 业务异常使用自定义异常类
+
+### Mapper 层
+
+- [ ] 继承 `BaseMapper<T>`
+- [ ] 自定义 SQL 使用 `#{}` 占位符，禁止 `${}`
+- [ ] 分页使用 MyBatis Plus 分页插件
+- [ ] 批量操作使用 `saveBatch` / `updateBatchById`
+
+### Entity 层
+
+- [ ] `@TableName`、`@TableId(type = IdType.ASSIGN_ID)` 注解完整
+- [ ] `createTime`/`updateTime` 自动填充，`isDeleted` 逻辑删除
+- [ ] Request DTO 在 `entity/request/`，Response DTO 在 `entity/response/`
+
+---
+
+## 安全审查项
+
+> 完整的 OWASP Top 10 检查清单和修复方案请参考 [security-audit Skill](../skills/security-audit/)
+
+### 核心检查项
+
+- [ ] MyBatis 禁止 `${}` 拼接用户输入，必须用 `#{}`
+- [ ] 密码字段不在 Response DTO 中返回
+- [ ] 日志中不打印敏感信息（密码、手机号、身份证号）
+- [ ] 接口有权限校验，数据有归属校验
+- [ ] 配置文件中的密码使用加密存储
+
+---
+
+## 性能审查项
+
+### 数据库性能
+
+- [ ] **无 N+1 查询问题** — 循环中不调用数据库查询
+- [ ] 查询条件字段有索引覆盖
+- [ ] 分页查询有 `LIMIT` 限制，禁止无限制全表扫描
+- [ ] 大数据量查询使用流式处理或分批处理
+- [ ] 批量插入使用 `saveBatch`，禁止循环单条插入
+
+✅ 正确示例:
+```java
+// 批量查询后在内存中关联
+List<User> users = userService.listByIds(userIds);
+Map<Long, User> userMap = users.stream()
+    .collect(Collectors.toMap(User::getId, Function.identity()));
+```
+
+❌ 错误示例:
+```java
+// N+1 查询：循环中逐条查询
+for (Order order : orders) {
+    User user = userService.getById(order.getUserId()); // 每次循环都查库
+    order.setUsername(user.getUsername());
+}
+```
+
+### 连接与资源
+
+- [ ] 数据库连接及时释放，无连接泄露
+- [ ] 流资源在 `try-with-resources` 中使用
+- [ ] 缓存使用合理，有过期策略
+- [ ] 线程池参数配置合理
+
+### 接口性能
+
+- [ ] 接口响应时间合理（普通接口 < 500ms）
+- [ ] 大文件上传使用流式处理
+- [ ] 列表接口强制分页
+
+---
+
+## 最终审查清单
+
+> **审查报告模板、评论规范和工作流详见**: [code-review Skill](../skills/code-review/)
+
+在点击「Approve」之前，确认以下所有项目：
+
+- [ ] 所有 Blocker 问题已修复
+- [ ] 所有 Major 问题已修复或已记录 TODO
+- [ ] 代码符合分层架构规范（参考 architecture-redlines.md）
+- [ ] 代码符合编码规范（参考 coding-standards.md）
+- [ ] 无安全漏洞（SQL 注入、XSS、敏感数据泄露）
+- [ ] 无明显性能问题（N+1、全表扫描、连接泄露）
+- [ ] 单元测试覆盖核心业务逻辑
+- [ ] API 文档已更新
+- [ ] 数据库变更有对应的 migration 脚本