用途
该技能用于代码审查,要求进行技术评估而非情绪化表演。适用于两种场景:收到代码审查反馈时,或提交 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
- 受保护端点有鉴权检查
使用方式(接收反馈的响应模式)
收到反馈时:
- READ:完整阅读反馈,不做出反应
- UNDERSTAND:用自己的话复述需求
- VERIFY:对照代码库实际情况核查
- EVALUATE:对该代码库在技术上是否合理?
- RESPOND:技术性认可或有理有据的反驳
- 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
限制
关于该技能的版本、安装方式、收费情况、平台支持等,资料暂未提供。