mcpField
C
code-review

code-review

github.com/srbhr测试与质量28.9k54 次安装1 周前更新
代码仓库

关于 code-review

用于按项目的 FastAPI、Next.js 和安全规范评估代码变更的审查技能，提供检查清单、问题严重级别及先验证再实施的反馈流程。

主要功能

  1. 01后端审查清单
  2. 02前端审查清单
  3. 03安全检查项
  4. 04问题严重级别标注
  5. 05先验证再实施的流程

适用场景

  1. 01审查 FastAPI 代码变更
  2. 02检查 Next.js 代码规范
  3. 03评估输入验证与端点安全

安装

复制下面的命令，在终端中运行。

$npx skills add https://github.com/srbhr/resume-matcher --skill code-review

安装后在 Claude Code 中输入/code-review确认已加载。

详细介绍

代码审查

代码审查需要的是技术评估，而不是情绪表演。

核心原则： 先验证再实现，先提问再假设。技术正确性优先于社交舒适感。

审查清单

后端（Python/FastAPI）
  • 所有函数都有类型提示
  • 可变默认值使用 copy.deepcopy()
  • 错误处理：服务端记录详细日志，向客户端返回通用消息
  • API 密钥通过 api_key= 参数传入，而不是 os.environ
  • I/O 操作使用异步函数
  • 请求和响应体使用 Pydantic 模式
  • 异步路由中不使用阻塞调用
  • 共享资源初始化使用 asyncio.Lock()
前端（Next.js/React）
  • 符合瑞士国际主义风格（Swiss International Style）
  • 全局使用 rounded-none
  • 文本域带有 Enter 键处理函数
  • 直接导入图标（不使用 barrel 导入）
  • 大型组件使用 next/dynamic
  • 独立的请求使用 Promise.all()
  • Server Actions 内部检查身份认证
安全
  • 代码中没有密钥
  • 对用户数据进行输入验证
  • 正确配置 CORS
  • 受保护的端点进行身份认证检查

响应模式

WHEN receiving feedback:
1. READ: Complete feedback without reacting
2. UNDERSTAND: Restate requirement in own words
3. VERIFY: Check against codebase reality
4. EVALUATE: Technically sound for THIS codebase?
5. RESPOND: Technical acknowledgment or reasoned pushback
6. IMPLEMENT: One item at a time, test each

禁止的回复

  • "You're absolutely right!"（表演性回应）
  • "Great point!"（表演性回应）
  • "Let me implement that now"（未经验证就动手实现）

何时提出反驳

  • 建议会破坏现有功能
  • 审查者缺乏完整上下文
  • 违反 YAGNI（未使用的功能）
  • 对于当前技术栈在技术上不正确
  • 与架构决策相冲突

严重程度格式

[CRITICAL] file:line - Security, data loss, crash
[ERROR]    file:line - Bug, missing validation
[WARNING]  file:line - Style, convention
[INFO]     file:line - Suggestion, not required

参考资料

  • 后端指南：docs/agent/architecture/backend-guide.md
  • 编码规范：docs/agent/coding-standards.md
  • 风格指南：docs/agent/design/style-guide.md

评论

支持 Markdown