Files
skills/codex-collab/references/code-review.md
empty 15604b53c5 Fix codex-collab skill structure to comply with Claude Code skill specification
Changes:
- Rename skill.md to SKILL.md (uppercase required)
- Add YAML frontmatter with name and description
- Reorganize templates/ and examples/ into references/
- Remove extra documentation files (README.md, QUICK_REF.md)
- Follow progressive disclosure pattern: keep SKILL.md lean

The skill now complies with the official Claude Code skill format.

Co-Authored-By: Claude <noreply@anthropic.com>
2026-01-11 17:22:33 +08:00

2.2 KiB
Raw Permalink Blame History

Code Review Template

Template Variables

  • {modified_files}: List of modified files
  • {change_description}: Description of changes made
  • {requirement}: Original requirement/user story

Prompt Template

请作为资深代码审查专家,审查以下代码改动:

## 改动文件列表
{modified_files}

## 改动说明
{change_description}

## 原始需求
{requirement}

## 审查检查清单

请按以下维度进行审查:

### 1. 正确性 (Correctness)
- [ ] 代码逻辑是否正确实现了需求
- [ ] 是否有明显的bug或逻辑错误
- [ ] 边界条件和异常情况是否处理
- [ ] 数据验证是否充分

### 2. 代码质量 (Code Quality)
- [ ] 代码可读性和可维护性
- [ ] 命名是否清晰准确
- [ ] 是否有重复代码
- [ ] 函数和类的职责是否单一

### 3. 性能 (Performance)
- [ ] 是否有明显的性能问题
- [ ] 资源使用是否合理
- [ ] 是否有不必要的计算或IO操作
- [ ] 算法复杂度是否合理

### 4. 安全性 (Security)
- [ ] 是否有安全漏洞
- [ ] 输入验证是否充分
- [ ] 敏感信息是否正确处理
- [ ] 权限检查是否完善

### 5. 测试覆盖 (Testing)
- [ ] 关键逻辑是否有测试覆盖
- [ ] 测试用例是否充分
- [ ] 边界条件是否测试

### 6. 文档 (Documentation)
- [ ] 复杂逻辑是否有注释
- [ ] API文档是否完整
- [ ] 变更日志是否更新

## 审查结论

请提供:
1. 总体评价(优秀/良好/需改进/不合格)
2. 主要优点1-3点
3. 必须修复的问题(如有)
4. 建议改进的地方(如有)
5. 其他意见

Usage Example

# After completing code changes
modified_files = """
- src/auth/login.py: 添加用户登录验证
- src/auth/jwt.py: 实现JWT token生成和验证
- tests/test_login.py: 添加登录测试用例
"""

change_description = """
实现了基于JWT的用户认证系统
1. 用户登录验证(用户名+密码)
2. JWT token生成和验证
3. token过期处理
4. 单元测试覆盖
"""

requirement = "实现用户登录功能支持JWT token认证"

# Call Codex for review
codex(
    PROMPT=f"<review template with variables>",
    cd="/project",
    SESSION_ID=previous_session
)