mcpField
C
code-review-mitsuhiko

code-review-mitsuhiko

github.com/wilbeibi测试与质量243 次安装3 天前更新
代码仓库

关于 code-review-mitsuhiko

code-review-mitsuhiko 是面向代码审查的 Agent 技能,先判断代码属于产品、库或混合类型,再应用相应审查标准。它强调按严重程度列出问题,并检查错误、依赖与默认值。

主要功能

  1. 01区分产品代码与库代码
  2. 02按上下文切换审查视角
  3. 03按严重程度组织发现
  4. 04检查错误、依赖和默认值

适用场景

  1. 01审查产品代码的抽象与需求匹配度
  2. 02评估库 API 的兼容性和扩展性
  3. 03检查错误处理与依赖成本

安装

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

$npx skills add https://github.com/wilbeibi/wilbeibi-skills --skill code-review-mitsuhiko

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

附带文件PRINCIPLES.md

详细介绍

code-review-mitsuhiko

首先判断这是产品代码、库代码,还是两者兼有。不同判断对应的评审标准也不同。

步骤

  1. 写明 Context call: Product / Library / Mixed。
  2. 应用下方对应的评审视角,然后进行跨领域检查。
  3. 优先列出问题,并按严重程度排序。

产品视角

  • 这现在是否解决了真实的用户问题?
  • 能否先发布一个功能简单但可用的版本,以更快验证需求?
  • 这种抽象、依赖或扩展点,是否有第二个真实用例作为依据?
  • 如果接受了技术债,是否有明确的重构触发条件?

标记:推测性的灵活性、不易察觉的优雅设计、非瓶颈优化,以及只为一个功能构建的通用框架。

库视角

  • 这套 API 多年后是否依然合理?
  • 是否会破坏现有用户的使用?如果会,说明版本管理和迁移成本。
  • 用户能否在不派生分支的情况下扩展行为?
  • 是否将错误、默认值和扩展点视为稳定的公共 API?

标记:为次要改进引入破坏性变更、通过字符串匹配错误、未记录的扩展契约,以及“直接派生分支”的设计。

跨领域检查

  • 错误:类型化/结构化程度是否足以支持调用方处理和生产环境诊断?
  • 依赖:是否值得引入整棵传递依赖树,还是用 20-50 行清晰代码就能实现?
  • 默认值:是否安全、符合预期,并如实说明成本?
  • 产品/库定位错配:库的标准是否拖慢了产品,或产品开发习惯是否破坏了库的稳定性?

输出约定

  • 优先列出问题:Must fix、Consider、Open questions;没有内容的部分省略。
  • 每条问题都包括 [file:line]、结合具体情境的理由,以及明确的后续步骤。
  • 在问题之后添加简短的 Summary。除非有人要求,否则不要添加赞扬。

完整理念和措辞示例请参阅 PRINCIPLES.md。

评论

支持 Markdown