关于 merge-ready
merge-ready 是用于准备分支以供维护者最终审查的代码审查技能。它对照 origin/main 检查变更、修复实际问题、运行本地检查,并创建或更新拉取请求,但不会合并。
主要功能
- 01分支落后时获取并合并 origin/main
- 02审查已提交及未提交的分支变更
- 03检查安全、简洁性和账单滥用风险
- 04修复问题并提交变更
- 05运行 pnpm ci:check 和 pnpm test
- 06创建或更新拉取请求
适用场景
- 01准备功能分支以供维护者审查
- 02检查变更中的安全和账单风险
- 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 下。
绝不要合并。最后由维护者审查。