code-review

代码评审 (code-review)

能力边界

• 能做: 功能正确性对照（实现 vs AC/契约）、代码质量审查、命名/风格规范检查、安全漏洞检测、架构合规验证、测试质量审查、项目级腐化指标扫描（重复/死码/复杂度）
• 不做: 修改代码(仅报告问题)、需求评审；任务依赖图由 task-dep-analysis 负责

输入规范

• 代码文件或目录(DEV产出)
• arch#§7开发约定(命名/风格/Git约定)
• arch#§5非功能架构(安全/错误处理)

输出规范

• 任务粒度审查: CODE-REVIEW-{task_id}-r{N}.md（问题列表 + 严重等级: CRITICAL/HIGH/MEDIUM/LOW）
• 项目级扫描: CODE-SCAN-{YYYYMMDD}-r{N}.md（腐化指标聚合 + 严重等级）
• 审查结论: approved/approved_with_notes/needs_revision

操作指令: 任务粒度评审 (review)

Step 1: Layer 1 — Lint脚本自动检查

前置判断: 读取当前平台 Hook 配置（Claude: .claude/settings.json；Cursor: .cursor/hooks.json），检查是否存在 matcher 为 Edit|Write（Cursor 可为 Write/StrReplace）且 command 包含 lint_format.py 的条目:

• 已配置 lint hook → 编码阶段已通过 hook 以 --fix 模式实时修复格式/lint问题，跳过 Layer 1，直接进入 Step 2 Layer 2，并在审查报告标题下标注 Layer 1 delegated to hook
• 未配置 lint hook → 执行: cataforge skill run code-review -- review {file_or_dir}

调用约定: 入口与返回码语义按 COMMON-RULES §Layer 1 调用协议。本 skill 增量：exit 1 时可追加 --fix 自动修复后重新检查——--fix 就地改写被审文件，执行权属仅限 implementer（continuation）/ tdd-engine / lint hook 语境，reviewer 角色（禁改被审对象）禁用，遇 convention 类 finding 记入报告由 implementer continuation 修复；未知参数与非法 --focus 值为用法错误（exit 2）；两个模式均支持 --format json 输出结构化 finding（Layer 2 与报告聚合的机读输入）。

工具适配与检查清单见 §Layer 1 检查项；工具不存在时自动跳过并 WARN，不阻断检查流程。

Step 2: Layer 2 — AI语义审查

语言细则: 进入语义审查前，根据 framework.json project.languages，按需载入本 skill references/lang-<lang>.md（仅 active 语言，逐个 Read），作为该语言的评审细则（坏味道、安全陷阱、缺陷模式、性能反模式、评审 checklist）。这是散文评审参考，与 §Plugin-style rules 的 rules/*.yaml（Layer 1 机检 wiring）互补、不重复。

Layer 2 短路条件（降低轻量任务的审查开销，类比 doc-review §Layer 2 短路）:

满足以下任一条件且 Layer 1 exit 0 时跳过 Layer 2 直接判定为 approved:

1. 任务卡 task_kind ∈ CODE_REVIEW_L2_SKIP_TASK_KINDS（默认 [chore, config, docs]）
2. 任务卡 tdd_mode: light + AC 数 ≤ CODE_REVIEW_L2_SKIP_LIGHT_MAX_AC + Layer 1 输出无 security/error-handling 类 finding
3. 调度方声明 layer1-only 降级（由 ORCHESTRATOR-META-PROTOCOLS §Adaptive Review 反向降级触发；这是 Layer 2 编排参数，不传入 Layer 1 CLI——CLI 只认 review|scan 子命令与 --fix/--focus/--format，未知参数报 exit 2）