mcpField
M
merge-ready

merge-ready

github.com/every-app测试与质量25.5k2.3k 次安装2 天前更新
代码仓库

关于 merge-ready

merge-ready 是用于准备分支以供维护者最终审查的代码审查技能。它对照 origin/main 检查变更、修复实际问题、运行本地检查,并创建或更新拉取请求,但不会合并。

主要功能

  1. 01分支落后时获取并合并 origin/main
  2. 02审查已提交及未提交的分支变更
  3. 03检查安全、简洁性和账单滥用风险
  4. 04修复问题并提交变更
  5. 05运行 pnpm ci:check 和 pnpm test
  6. 06创建或更新拉取请求

适用场景

  1. 01准备功能分支以供维护者审查
  2. 02检查变更中的安全和账单风险
  3. 03运行项目检查并提交拉取请求

安装

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

$npx skills add https://github.com/every-app/open-seo --skill merge-ready

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

详细介绍

让此分支做好接受维护者最终审查的准备。如果你还被要求构建内容,请先构建。

如果此分支落后于 origin/main,请获取并合并该分支。然后检查此分支相对于它的所有内容,包括未提交和未跟踪的文件。确保你理解每项产品决策和功能变更。

安全吗?

它是否足够简单?根据此分支的最终状态,可以删除或简化什么?是否因为不太可能出现的边缘情况而过度设计?我们倾向于删除代码。

它是否以惯用方式使用 API 和 SDK(TanStack、DataForSEO、Autumn、Cloudflare)?留意那些重新实现已有功能、规避实际上并不存在的限制,或造成不必要开支的代码。只有发现可疑之处时才查阅文档。

是否存在可显著滥用计费机制的方式,例如不付费就获取额度、付费功能或调用服务提供商?忽略没人会费心利用的小额损失。

除此之外,请按你认为最好的方式审查。判断哪些部分出错时可能造成最大损害,并把时间花在这些地方。docs/maintainers/review-guidelines.md 列出了仓库的不变量,以及那些看似有问题但实际上是有意为之的内容。

修复确实存在的问题。随时提交更改。在本地通过 pnpm ci:check 和 pnpm test,然后推送。这些本地运行就是 CI 的替代,不要等 GitHub CI 完成。

针对 main 创建或更新 PR。按 simple-issue-description 格式撰写说明,并将所有留待维护者判断的事项放在 Extra context 下。

绝不要合并。最后由维护者审查。

评论

支持 Markdown