NetEase Mod 开发的综合代码审查编排技能，聚焦于安全性、性能、正确性与可维护性。 适用场景：审查代码、执行安全审计、检查代码质量、审阅 Pull Request，或用户明确 要求进行代码审查时。此技能作为主审查器，负责协调其他专项审查子技能共同完成全面 审查。不直接执行具体审查逻辑，而是按层次分派给专项子技能并汇总结果。
本技能是所有代码质量检查的中央编排器。它不直接只看某一个维度，而是负责：
目标是把“语法兼容、规范、架构、简化、性能、UI、RPC、配置、API 查证、运行时实测”这些分散能力串成一个高质量审查闭环。
code-reviewer（本技能 — 总指挥）
├── 基础阻断层
│ ├── python27-syntax-reviewer（拦截 Py2.7 语法红线与运行时崩溃风险）
│ ├── encoding-guard（检查 UTF-8、编码头、中文输出与文件编码规范）
│ └── netease-api-researcher（当 API / 事件 / 枚举存疑时核对官方签名与可用性）
├── 规范与设计层
│ ├── 项目编码规范（内联检查，命名 / 日志 / 异常 / import / 编码头，联动 encoding-guard）
│ ├── pragmatic-clean-code-reviewer（主审引擎，最详细的整洁代码 / 架构 / 维护性深审）
│ ├── config-centralizer（检查常量 / 阈值 / 开关 / 路径是否集中配置）
│ └── python27-type-annotations（当类型注释策略相关时统一注释风格）
├── 专项域审查层
│ ├── netease-ui-binder（UI / Bindable / JSON UI 结构、绑定与映射专项审查）
│ ├── qumod-rpc-generator（QuMod RPC / C-S 通信链路、注册方式与调用边界审查）
│ ├── mcdk-assistant（MCP 驱动的 UI / 模型 / 动画 / 文档交叉核查）
│ └── firecrawl（用户明确要求联网核验、引用在线文档或最新资料时）
├── 改进层
│ ├── code-simplifier（在不改行为前提下提出精炼、去冗余与可读性改进）
│ └── python27-perf-optimizer（热路径性能、FFI、异常与遍历成本审查）
└── 实证层
└── netease-game-instrumentation（静态阅读无法定论时做游戏内插桩与实证）
pragmatic-clean-code-reviewer是默认的主审技能，也是本仓库里最详细的深度审查规则集。只要进入标准或深度审查，就应优先充分使用它，再由其他专项技能补足领域细节与证据。
| 级别 | 触发条件 | 联动技能 | 说明 |
|---|---|---|---|
| 快速 | ≤3 个文件的小改动 | python27-syntax-reviewer + 内联规范检查 | 先挡住致命语法和明显违规 |
| 标准 | 功能完成 / 批次完成 / 常规 review | 基础阻断层 + 规范与设计层 + 改进层 | 默认完整审查 |
| 深度 | 架构变更 / 大型重构 / PR 总审 / 安全审计 | 标准级 + 专项域审查层 + 实证层（按需） | 做跨层交叉核查 |
用户可在审查指令中显式指定级别，否则默认使用深度审查级别进行审查。
python27-syntax-reviewerpragmatic-clean-code-reviewercode-simplifierpython27-perf-optimizernetease-api-researcherpython27-syntax-reviewerpragmatic-clean-code-reviewermcdk-assistantnetease-game-instrumentationnetease-ui-binderpython27-syntax-reviewerpragmatic-clean-code-reviewermcdk-assistantnetease-game-instrumentationqumod-rpc-generatorpython27-syntax-reviewerpragmatic-clean-code-reviewernetease-api-researcherpython27-perf-optimizerconfig-centralizerpragmatic-clean-code-reviewerfirecrawl先判断当前改动是否涉及以下任一类：
根据分类决定后续联动技能，不要机械地永远只跑同一套五层。
python27-syntax-reviewer.py 文件执行 Py2.7 红线审查super()、raise ... from ...、类定义、异常语法encoding-guardnetease-api-researcherencoding-guard）对照当前项目的编码规范文件、AGENTS 约束或团队约定，内联核查以下项目；其中编码、中文输出、编码头、文件保存格式相关项统一交给 encoding-guard 作为专项支撑：
| 检查项 | 严重度 |
|---|---|
| 命名不一致、语义含混 | 🟡 |
缺少 # type: 注释 | 🟡 |
使用 print 而非项目约定日志方式 | 🟠 |
| 使用禁用模块 / 不安全标准库 | 🔴 |
裸 except: / 宽泛吞异常 | 🔴 |
| 缺少编码头 | 🟡 |
| 循环导入 / 依赖方向错误 | 🔴 |
| 魔法数字散落且无集中定义 | 🟡 |
pragmatic-clean-code-reviewer这是规范与设计层的主审引擎。当时间、上下文或并发额度有限时，优先保证本技能被充分执行，而不是只做表层清单式检查。
config-centralizerpython27-type-annotations# type:、复杂参数 / 返回值约束、可选值表达时触发netease-ui-binder以下场景必须联动：
UIBranch 改动@Button / @Toggle / @EditBox / @Slider / @Collectionqumod-rpc-generator以下场景必须联动：
Call / Request / AllowCallmcdk-assistant以下场景建议联动：
firecrawl只在以下条件下触发：
code-simplifierpython27-perf-optimizer仅在以下代码触发：
netease-game-instrumentation当静态阅读无法定论时触发：
待审查文件较多时，应尽量多开 sub agent 做并行多领域审查，但必须避免重复劳动与结论打架：
pragmatic-clean-code-reviewer 应单独占一个高优先级 sub agent，作为主结论来源之一## 综合代码审查报告
**审查级别**：快速 / 标准 / 深度
**审查范围**：[文件列表或变更描述]
**联动技能**：[本次实际调用的 skill 列表]
### 🔴 严重（必须立即修复）
_Py2.7 兼容、禁用库、API 伪造、架构破坏、线程违规、配置破坏_
1. **[问题标题]**（文件:行号）「来源：技能名 / 层级」
- **问题：** ...
- **修复：** ...
### 🟠 高优先级（性能、架构）
_架构违规、UI / RPC 错误设计、热路径性能、配置散落_
### 🟡 中等（规范、清晰度）
_命名/注释/类型提示、简化建议、可维护性问题_
### 🟢 建议
### 汇总
| 层/流程 | 技能 | 问题数 |
| -------- | -------------------------------------------------------- | ------ |
| 基础阻断 | python27-syntax-reviewer | N |
| 基础阻断 | netease-api-researcher | N |
| 规范设计 | pragmatic-clean-code-reviewer | N |
| 规范设计 | config-centralizer | N |
| 专项域 | netease-ui-binder / qumod-rpc-generator / mcdk-assistant | N |
| 改进 | code-simplifier | N |
| 改进 | python27-perf-optimizer | N |
| 实证 | netease-game-instrumentation | N |
- 总计：🔴 N | 🟠 N | 🟡 N | 🟢 N
- **[通过/不通过]** — 门槛：🔴 = 0 且 🟠 ≤ 2