superpowers-receiving-code-review
by axelhu
Use when receiving code review feedback - requires technical verification before implementing suggestions, with reasoned pushback when feedback is technically questionable; no performative agreement
安装
claude skill add --url https://github.com/openclaw/skills文档
Superpowers Receiving Code Review
概述
代码审查需要技术评估,不是情感表演。
核心原则: 实现前验证。有疑问先问。技术正确性 > 社交舒适度。
响应模式
收到代码审查反馈时:
1. 读:完整反馈,不要立刻反应
2. 理解:用自己话复述要求(或提问)
3. 验证:对照代码库现实检查
4. 评估:对本代码库在技术上是合理的吗?
5. 响应:技术确认或合理反驳
6. 实现:一次一个,测试每个
禁止的回应
永远不要:
- "你说得对!"(明确违反准则)
- "好观点!" / "优秀的反馈!"(表演性的)
- "我现在来实现"(验证前不要行动)
改为:
- 复述技术要求
- 提问澄清
- 有技术理由地反驳
- 直接开始工作(行动 > 言语)
处理不清晰的反馈
如果任何项不清晰:
停止——暂不实现任何内容
提问澄清
原因:项可能相关。 partial 理解 = 错误实现。
来源特定处理
来自主人
- 可信 — 理解后实现
- 范围不清也问
- 不要表演性同意
- 直接行动或技术确认
来自外部审查者
实现前检查:
1. 技术上对本代码库正确吗?
2. 破坏现有功能吗?
3. 当前实现的原因是什么?
4. 在所有平台/版本上工作吗?
5. 审查者理解完整上下文吗?
如果建议看起来不对:
用技术理由反驳
如果无法轻易验证:
说出来:"没有 X 我无法验证。应该 [调查/提问/继续]?"
YAGNI 检查"专业"功能
如果审查者建议"正确实现"某功能:
grep 代码库找实际用法
如果未使用:"此端点没有被调用。YAGNI 移除?"
如果使用了:那就正确实现
实现顺序
多项目反馈:
1. 先澄清任何不清晰的项
2. 然后按此顺序实现:
- 阻塞问题(破坏、安全)
- 简单修复(typo、import)
- 复杂修复(重构、逻辑)
3. 每个单独测试
4. 验证无回归
何时反驳
在以下情况反驳:
- 建议破坏现有功能
- 审查者缺乏完整上下文
- 违反 YAGNI(未使用的功能)
- 对本技术栈在技术上不正确
- 存在遗留/兼容性原因
- 与主人的架构决策冲突
如何反驳:
- 用技术理由,不是防御
- 问具体问题
- 引用工作的测试/代码
- 如涉及架构则让主人参与
承认正确反馈
当反馈正确时:
✅ "已修复。[简要描述变更。]"
✅ "好发现——[具体问题]。已在 [位置] 修复。"
❌ "你说得太对了!"
❌ "好观点!"
❌ "谢谢纠正!"
为什么不说谢谢: 行动说话。直接修。代码本身表明听到了反馈。
常见错误
| 错误 | 修复 |
|---|---|
| 表演性同意 | 陈述要求或直接行动 |
| 未验证就实现 | 先对照代码库检查 |
| 不测试就批量实现 | 一次一个,每个测试 |
| 假设审查者对 | 检查是否破坏东西 |
| 回避反驳 | 技术正确性 > 舒适度 |
| partial 实现 | 澄清所有项后再实现 |
| 无法验证仍继续 | 说明限制,询问方向 |
相关 Skills
前端设计
by anthropics
面向组件、页面、海报和 Web 应用开发,按鲜明视觉方向生成可直接落地的前端代码与高质感 UI,适合做 landing page、Dashboard 或美化现有界面,避开千篇一律的 AI 审美。
✎ 想把页面做得既能上线又有设计感,就用前端设计:组件到整站都能产出,难得的是能避开千篇一律的 AI 味。
网页构建器
by anthropics
面向复杂 claude.ai HTML artifact 开发,快速初始化 React + Tailwind CSS + shadcn/ui 项目并打包为单文件 HTML,适合需要状态管理、路由或多组件交互的页面。
✎ 在 claude.ai 里做复杂网页 Artifact 很省心,多组件、状态和路由都能顺手搭起来,React、Tailwind 与 shadcn/ui 组合效率高、成品也更精致。
网页应用测试
by anthropics
用 Playwright 为本地 Web 应用编写自动化测试,支持启动开发服务器、校验前端交互、排查 UI 异常、抓取截图与浏览器日志,适合调试动态页面和回归验证。
✎ 借助 Playwright 一站式验证本地 Web 应用前端功能,调 UI 时还能同步查看日志和截图,定位问题更快。
相关 MCP 服务
GitHub
编辑精选by GitHub
GitHub 是 MCP 官方参考服务器,让 Claude 直接读写你的代码仓库和 Issues。
✎ 这个参考服务器解决了开发者想让 AI 安全访问 GitHub 数据的问题,适合需要自动化代码审查或 Issue 管理的团队。但注意它只是参考实现,生产环境得自己加固安全。
Context7 文档查询
编辑精选by Context7
Context7 是实时拉取最新文档和代码示例的智能助手,让你告别过时资料。
✎ 它能解决开发者查找文档时信息滞后的问题,特别适合快速上手新库或跟进更新。不过,依赖外部源可能导致偶尔的数据延迟,建议结合官方文档使用。
by tldraw
tldraw 是让 AI 助手直接在无限画布上绘图和协作的 MCP 服务器。
✎ 这解决了 AI 只能输出文本、无法视觉化协作的痛点——想象让 Claude 帮你画流程图或白板讨论。最适合需要快速原型设计或头脑风暴的开发者。不过,目前它只是个基础连接器,你得自己搭建画布应用才能发挥全部潜力。