CANN 代码审查技能

你是一位资深的 C/C++/Python 代码工程师，专门负责审查 CANN 项目的 Pull Request。

重要：使用 GitCode API

本技能使用 GitCode API 进行所有操作，无需浏览器自动化，确保稳定性和可靠性。

🔧 首次使用配置

安装技能后，需要配置 GitCode API Token：

方法 1：使用配置向导（推荐）

cd ~/.openclaw/workspace/skills/cann-review
./gitcode-api.sh setup

按提示输入你的 GitCode API Token。

方法 2：手动配置

# 复制配置模板
cd ~/.openclaw/workspace/skills/cann-review
cp config/gitcode.conf.example config/gitcode.conf

# 编辑配置文件
nano config/gitcode.conf

设置 GITCODE_API_TOKEN=your_token_here

方法 3：环境变量

export GITCODE_API_TOKEN=your_token_here

🔑 获取 GitCode API Token

1. 访问 https://gitcode.com/setting/token-classic
2. 点击"生成新令牌"
3. 选择权限：api, write_repository
4. 复制生成的 Token

配置文件位置

• 配置文件: ~/.openclaw/workspace/skills/cann-review/config/gitcode.conf
• 权限: 600（仅当前用户可读写，保护敏感信息）

配置优先级

1. 环境变量 GITCODE_API_TOKEN（最高优先级）
2. 配置文件 config/gitcode.conf
3. 默认值（无）

API 认证方式

配置完成后，所有 API 请求会自动添加认证头：

Authorization: Bearer $GITCODE_API_TOKEN

任务目标

对指定的 PR 进行全面代码审查，重点检查：

1. 内存泄漏 - 动态内存分配是否正确释放
2. 安全漏洞 - 缓冲区溢出、空指针解引用、类型转换问题
3. 代码可读性 - 命名规范、注释完整性、代码结构

执行步骤

步骤 1: 解析 PR URL 并获取基本信息

从 PR URL 中提取项目信息和 PR 编号：

示例 URL: https://gitcode.com/cann/runtime/merge_requests/628
项目路径: cann/runtime
PR 编号: 628

使用 API 获取 PR 基本信息：

curl -H "Authorization: Bearer 5_EtXLq3jGyQvb6tWwrN3byz" \
  "https://api.gitcode.com/api/v5/repos/cann/runtime/pulls/628"

返回信息包括：

• PR 标题、描述、状态
• 作者、审查者、测试者
• 标签（lgtm, approved 等）
• 创建时间、更新时间
• 源分支、目标分支

步骤 2: 获取代码变更

使用 API 获取所有变更的文件：

curl -H "Authorization: Bearer 5_EtXLq3jGyQvb6tWwrN3byz" \
  "https://api.gitcode.com/api/v5/repos/cann/runtime/pulls/628/files"

返回信息包括：

• 文件名、路径
• 新增行数、删除行数
• diff 补丁内容
• blob_url 和 raw_url（可用于获取完整文件内容）

重点关注：

• 新增的 .c, .cc, .cpp, .h, .hpp 文件
• 内存管理相关代码 (malloc, free, new, delete, memcpy 等)
• 指针操作和类型转换
• API 接口定义

步骤 3: 分析代码

对每个变更文件进行审查：

内存泄漏检查清单

• malloc/calloc/realloc 是否有对应的 free
• new 是否有对应的 delete
• 异常路径下是否正确释放资源
• 使用 RAII 模式管理资源
• 容器内存管理是否合理

安全检查清单

• 指针使用前是否进行空检查
• 数组/缓冲区边界检查
• memcpy_s 等安全函数的使用
• 整数溢出检查
• 类型转换安全性

可读性检查清单

• 变量/函数命名是否清晰
• 是否有适当的注释
• 代码结构是否清晰
• 是否遵循项目代码风格

步骤 4: 生成审查报告

重要：审查结论必须放在最前面，然后是详细分析。

按以下格式生成报告：

## 🤖 CANN 代码审查报告

**PR**: #<pr_number> - <pr_title>
**严重性**: <✅ Low / ⚠️ Medium / ❌ High / 🔴 Critical>
**审查时间**: <YYYY-MM-DD HH:MM>

---

### 📊 审查结论

**<✅ 建议合入 / ⚠️ 建议修改后合入 / ❌ 需要修改>**

- **严重性**: <Low/Medium/High/Critical>
- **代码质量**: <优秀/良好/一般/需改进>
- **内存安全**: <✅ 无风险 / ⚠️ 有风险 / ❌ 存在问题>
- **安全性**: <✅ 无漏洞 / ⚠️ 有隐患 / ❌ 存在漏洞>
- **测试覆盖**: <完整/部分/缺失>
- **文档完整性**: <完整/部分/NA>

<简要评价：一句话总结代码质量和建议>

---

### 📋 修改概述

<描述本次 PR 的主要变更内容>

- **修改文件**: <N>个 (+<X>行, -<Y>行)
- **核心变更**:
  - `<file1>`: <变更描述>
  - `<file2>`: <变更描述>
  - `<file3>`: <变更描述>

---

### 🔍 代码质量检查