Skill: sdd-code-review

描述

L2 特性级多维度代码审查。Feature 全部任务完成后执行（SDD 第 8 步）。

本 skill 是仓库内的唯一原件：.cursor/skills 与 .agents/skills 都是指向 .claude/skills 的软链接，改这里即可。

触发

/sdd-code-review --base feat/3.0.0

--base 取 Feature 要合入的目标分支，下文用 <base> 指代。

审查流程

1. 执行 git diff <base>...HEAD --stat 获取变更文件列表
2. 执行 git diff <base>...HEAD 获取完整 diff
3. 对照 Feature 的 spec.md、design.md 和 tasks.md
4. 按 7 维度逐一审查
5. 输出审查报告

7 维度审查框架

维度 1：边界条件

维度 2：权限与认证

以 docs/constitution.md C4 为准；本表只列审查时要看的点。

维度 3：并发安全

维度 4：信息泄漏

维度 5：测试覆盖

务实适配：核心 Service 方法必须有测试。 前端：可抽成纯函数的逻辑（数据转换、校验、兼容升级）要有 vitest 单测；交互与视觉走 /e2e-test 的手动验证清单，tasks.md 只写「手动验证」不算覆盖。

维度 6：代码风格

维度 7：文档同步（design.md 现状快照）

目的：确保 feature 合入后，新 agent 接手时读 design.md 能 5 分钟建立准确认知 —— 没有过期描述、没有缺失的反直觉坑、没有未声明的对外契约。

判定要点：

• 若 design.md 与代码现状偏离 → 必须修复（视为 HIGH），不能合入
• 仅本地小修小补（不改变对外行为、不引入新坑、不动决策）→ 本维度全 PASS 即可
• design.md 不存在但已在 SDD 流程要求之内 → HIGH，要求按模板补全后再审

判定规则

输出格式

# Code Review Report

**Feature**: <feature_name>
**Review scope**: <描述>
**Base branch**: <base>
**Changed files**: <数量>

## Summary

| Dimension | High | Medium | Low | Status |
|-----------|------|--------|-----|--------|
| Boundary Conditions | 0 | 0 | 0 | PASS |
| Permission & Auth | 0 | 0 | 0 | PASS |
| Concurrency Safety | 0 | 0 | 0 | PASS |
| Information Leakage | 0 | 0 | 0 | PASS |
| Test Coverage | 0 | 0 | 0 | PASS |
| Code Style | 0 | 0 | 0 | PASS |
| Docs Sync (design.md) | 0 | 0 | 0 | PASS |

## Findings（如有）

### HIGH
- [Permission] `xxx_endpoint.py:42` — 资源删除前未调用 require_business_action(action="delete")

### MEDIUM
- [Style] `xxx_service.py:18` — DAO 方法未使用 @classmethod

## Overall: PASS / PASS_WITH_WARNINGS / NEEDS_FIX