SuperAPI AI 工具集

code-review

编程与开发

查看项目

更新于 10-16 01:58

技能介绍

用途

该技能用于代码审查,要求进行技术评估而非情绪化表演。适用于两种场景:收到代码审查反馈时,或提交 PR 之前。

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

主要能力(审查清单)

后端(Python/FastAPI):

  • 所有函数都有类型注解
  • 可变默认值使用 copy.deepcopy()
  • 错误处理:服务端记录详细日志,向客户端返回通用消息
  • API 密钥通过 api_key= 参数传入,而非 os.environ
  • I/O 操作用异步函数
  • 请求/响应体使用 Pydantic schemas
  • 异步路由中无阻塞调用
  • 共享资源初始化使用 asyncio.Lock()

前端(Next.js/React):

  • 符合 Swiss International Style
  • 处处使用 rounded-none
  • Textarea 具有 Enter 键处理程序
  • 直接导入图标(不使用 barrel)
  • 重型组件使用 next/dynamic
  • 独立请求使用 Promise.all()
  • Server Actions 内部检查鉴权

安全:

  • 代码中无密钥
  • 对用户数据进行输入校验
  • 正确配置 CORS
  • 受保护端点有鉴权检查

使用方式(接收反馈的响应模式)

收到反馈时:

  1. READ:完整阅读反馈,不做出反应
  2. UNDERSTAND:用自己的话复述需求
  3. VERIFY:对照代码库实际情况核查
  4. EVALUATE:对该代码库在技术上是否合理?
  5. RESPOND:技术性认可或有理有据的反驳
  6. IMPLEMENT:一次实现一项,逐项测试

禁止的回应(表演性):

  • "You're absolutely right!"
  • "Great point!"
  • "Let me implement that now"(在验证之前)

何时应当反驳:

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

严重程度格式

[CRITICAL] file:line - 安全、数据丢失、崩溃 [ERROR] file:line - Bug、缺少校验 [WARNING] file:line - 风格、约定 [INFO] file:line - 建议,非必需

参考文档

  • 后端指南:docs/agent/architecture/backend-guide.md
  • 编码标准:docs/agent/coding-standards.md
  • 风格指南:docs/agent/design/style-guide.md

限制

关于该技能的版本、安装方式、收费情况、平台支持等,资料暂未提供。