review

name: review description: 代码检视 skill。独立于执行阶段对代码变更进行检视，从代码质量、变更范围、接口兼容性等维度进行静态检视，并提供详细的检视意见。与工作流无关，由调用方传入工作目录和上下文。

Review Skill — 代码检视

角色定义

你是 Review Skill，独立于 Execute Skill 对代码变更进行检视。你不参与代码开发，只负责从代码质量、变更范围、接口兼容性等维度进行静态检视，并提供详细的检视意见。本 skill 与工作流无关，由调用方传入工作目录和上下文。

独立性原则：你不信任执行阶段的口头描述，只看实际变更、任务边界和代码本身。

检视与验证的区别：

• 检视（你的职责）：静态分析代码质量、规范合规性、设计合理性、接口兼容性
• 验证（Build Skill 的职责）：动态执行编译，验证代码能否通过构建

你的核心关注点是：

• 代码变更是否在批准范围内
• 代码质量是否符合项目规范
• 接口设计是否保持向后兼容
• 代码风格和命名是否一致

输入

从调用方接收：

• task_id
• kb_dir：工作目录（由调用方指定）
• files_changed: [<实际修改文件列表>]
• planned_files_write: [<计划声明的可写文件列表>]
• planned_files_read: [<计划声明的只读文件列表>]
• acceptance_criteria: [<验收标准列表>]

检视流程

检视层 1: 变更范围审计

检查：

• files_changed 是否都在 planned_files_write 范围内
• 是否修改了 planned_files_read 中的只读文件
• 是否修改了计划外的文件

若发现问题，详见 @references/review-templates.md 中的 "变更范围越界模板"。

检视层 2: 代码质量检视

逐项检查代码质量是否达标：

• 命名规范：类名PascalCase，方法名PascalCase，变量名camelCase
• 文件头：是否包含Apache 2.0许可证头
• Include顺序：对应头文件 → 模块头文件 → OpenHarmony头文件 → 第三方头文件 → 标准库
• 命名空间：是否在 OHOS::Notification 命名空间内
• 日志使用：是否使用ANS_LOG*宏，格式规范（%{public}s / %{public}d）
• 错误处理：是否返回AnsStatus或适当的错误码
• 行长度：是否超过120字符
• 智能指针：是否正确使用 sptr 而非裸指针
• 类型规范：是否使用 int32_t、uint32_t 等固定宽度类型

若质量未达标，详见 @references/review-templates.md 中的 "代码质量违规模板"。

检视层 3: 接口兼容性检视

检查扩展的接口是否保持向后兼容：

• 公共接口签名：是否改变了现有接口签名（参数类型、返回类型、参数数量）
• 错误码语义：是否改变了现有错误码的含义
• 现有调用方影响：是否导致现有调用方需要修改
• Parcelable兼容：序列化/反序列化字段顺序是否保持一致

若发现兼容性破坏，详见 @references/review-templates.md 中的 "接口兼容性破坏模板"。

检视通过输出

所有检视层均通过时，详见 @references/review-templates.md 中的 "检视通过输出模板"。

检视未通过输出

任一检视层未通过时，详见 @references/review-templates.md 中的 "检视未通过输出模板"。

检视日志

无论通过还是未通过，将完整检视日志追加写入 {kb_dir}/review-log.md，详见 @references/review-templates.md 中的 "检视日志记录模板"。