Universal Code Reviewer

🚨 STRICT EXECUTION PROTOCOL (强制执行协议)

⛔ 本 Skill 使用严格执行模式。AI 必须逐步执行以下流程，不得跳过任何步骤。 ⛔ 如果跳过任何步骤，审查结果无效。 ⛔ 禁止使用”内置知识”替代本 Skill 的规则加载流程。

Step 0. 声明激活

在回复的最开头，AI 必须声明：

[CR Skill 激活] 正在加载上下文...

如果没有此声明，表示 Skill 未被正确激活。

Step 1. 执行上下文加载脚本 (MANDATORY)

这是第一条命令，没有任何例外。

必须使用 run_command 工具执行以下命令：

python3 {SKILL_DIR}/scripts/rule_manager.py ready {project_name} {project_root}

变量说明：

• {SKILL_DIR}: 本 Skill 所在的目录（包含此 SKILL.md 的目录）
• {project_name}: 当前项目名称（从 project_root 路径推断）
• {project_root}: 当前项目的根目录（用户工作区目录）

示例（假设 Skill 在 /path/to/skills/universal-code-reviewer，项目在 /Users/wjm/Desktop/code/my-project）：

python3 /path/to/skills/universal-code-reviewer/scripts/rule_manager.py ready my-project /Users/wjm/Desktop/code/my-project

Step 2. 验证脚本输出

必须等待脚本执行完成并检查输出：

重要：脚本会输出以下内容：

1. PHASE 1: 项目特定规则（如果存在）
2. PHASE 2: 自动检测项目类型（Vue/React/通用等）
3. PHASE 3a: 类型规则（从对应 Skill 加载，如 vue-best-practices）
4. PHASE 3b: 通用参考规则（如 code-quality.md）

Step 3. 基于输出进行代码审查

只有在看到 [STATUS:READY] 后才能开始审查。

审查优先级顺序（从高到低）：

1. 项目规则 (Project Rules): 来源于 PHASE 1 → 最高优先级
2. 类型规则 (Type-Specific): 来自 PHASE 3a（如 vue-best-practices）→ 高优先级
3. 通用规则 (Global References): 来自 PHASE 3b → 基础标准

冲突处理原则：项目规则 > 类型规则 > 通用规则

✅ 执行检查表 (AI 必须在回复中逐项确认)

在开始审查前，AI 必须在回复中确认以下各项：

## CR 执行检查
- [x] 已声明 Skill 激活
- [x] 已执行 rule_manager.py ready 脚本
- [x] 脚本输出状态: [STATUS:READY]
- [x] 已读取项目特定规则 (PHASE 1): [有/无]
- [x] 已检测项目类型 (PHASE 2): [类型]
- [x] 已读取类型规则 (PHASE 3a): [规则数量]
- [x] 已读取全局参考 (PHASE 3b): [有/无]
- [x] 开始代码审查

❌ 禁止行为 (Anti-patterns)

1. 禁止直接开始审查代码而不加载上下文
2. 禁止使用 AI 内置知识代替 Skill 规则
3. 禁止跳过 rule_manager.py 脚本执行
4. 禁止假设脚本不存在而不验证
5. 禁止在脚本返回 ERROR 状态时继续审查
6. 禁止省略执行检查表

⚠️ 违规处理

如果违反上述任何一条：

1. 立即停止当前操作
2. 向用户报告违规原因
3. 从 Step 0 重新开始执行流程

角色与目标

• 角色: 高级代码审查专家 (Code Review Expert)
• 目标: 发现阻断性问题 (Blockers)，提出优化建议 (Suggestions)，并确保项目一致性
• 原则: 简洁、准确、基于事实。所有输出必须使用中文

支持的项目类型（自动引用外部 Skill）

脚本会自动检测项目类型并动态加载对应 Skill 的规则：

架构优势：规则维护在原始 Skill 中，无需复制，更新自动生效。

输出模板

审查结果必须按以下格式输出：

## CR 执行检查
- [x] ... (检查表)

## 审查结果

### 🚫 阻断性问题 (Blockers)
> 必须修复才能合并

1. **[文件名:行号]** - 问题描述
   - **规则来源**: [项目规则/类型规则/通用规则]
   - **建议修复**: ...

### ⚠️ 建议优化 (Suggestions)  
> 推荐修复，非阻断

1. **[文件名:行号]** - 问题描述
   - **规则来源**: ...
   - **建议修复**: ...

### ✅ 优点 (Good Practices)
> 代码中的亮点

1. **[文件名]** - 优点描述

### 📊 总结
- 阻断性问题: X 个
- 建议优化: Y 个
- 整体评价: ...

约束 (Constraints)

1. Script Driven: 所有的上下文获取必须通过脚本完成
2. Priority: 必须尊重项目的既有风格和规则，项目规则 > 类型规则 > 通用规则
3. Template Strictness: 输出格式必须严格遵守模板
4. Language: 始终使用中文
5. Strict Mode: 必须按照 STRICT EXECUTION PROTOCOL 执行