【Skill 创作】我写了一个用于审 GitHub 开源项目 PR 的 Skill,并直接拿它 Review 了提给 Trae 的 PR!

:waving_hand: 大家好!参与开源项目时,最让人头疼的往往不是写代码,而是提交 PR 后的漫长等待:Lint 报错、Codecov 覆盖率下降、漏写 Changesets、触发 CodeQL 安全警告…… 为了解决这个痛点,也为了在开发时保持清晰的“架构大局观”,我用 Trae 搭建了一套专属的工作流,并创作了一个专门用于**「GitHub 开源项目 PR 深度审查与 CI/CD 诊断」**的 Skill。
更酷的是,我直接拿这套自己写的 Skill,对我刚刚给 Trae 提交的 PR 进行了一波“套娃式”审查!

GitHub: https://github.com/BobcGn/pr-skills
:light_bulb: 创作初衷与痛点
在参与开源贡献或大型项目开发时,我们经常陷入业务实现的细节,而忽略了整体代码的高内聚、低耦合。同时,自动化流水线(CI/CD)极其严格。我希望有一个“资深架构师兼工程化专家”站在旁边:

  1. 看架构: 评估代码设计是否合理,业务逻辑与底层基建是否隔离。

  2. 盯流水线: 在 GitHub Actions 跑挂之前,提前预判并解决 Lint、CodeQL 和 Changesets 等拦截问题。

:hammer_and_wrench: 实战演练:用 Trae 审查 Trae 的 PR
这套 Skill 搭建完成后,我立刻拿它进行了实战测试。我将其作用于我刚提给 Trae 的 PR 上。
实战过程:

  1. 我将 PR 链接以及部分触发的 CI/CD 警告日志直接喂给这个 Skill。

  2. Skill 并没有像普通的 AI 那样只做简单的语法修改,而是直接从系统可靠性的角度指出了几处可能导致耦合的代码异味(Code Smell)。

  3. 针对令人头疼的自动化检查,它精准定位了可能拉低 Codecov 指标的测试盲区,并提示我补全了关键的 Changesets 描述信息以符合开源规范。

  4. 根据它的审查建议,我进行了一轮代码重构,并完成了新的 Push,整个工作流如丝般顺滑!

:toolbox: 我的 Skill 提示词分享
如果你也经常给开源项目提 PR,或者在团队中负责 Code Review,可以直接抱走这套 Prompt:

**角色设定与核心目标**
你是一位具备顶级架构视野的资深开源项目维护者(Maintainer)和工程化专家。我已经提交并推送了一个 Pull Request。现在,我需要你对我当前分支的代码或指定的 PR 进行全局深度审查。
你的目标是:从项目的长期可靠性出发评估架构设计,并协助我排查和解决自动化流水线(CI/CD)中可能出现或已经出现的拦截问题。

**审查与诊断维度**

1. **🏗️ 架构愿景与可靠性评估 (Architecture & Reliability)**
   * 审视当前 PR 的全貌:本次变更的实现方式,是否符合高内聚、低耦合的架构思想?
   * 核心业务逻辑是否与 UI 展示层或底层基建做到了清晰的隔离?
   * 站在项目的“大局观”上,指出当前设计中潜在的脆弱点,并给出提升系统整体可靠性的建议。

2. **🚨 CI/CD 诊断与开源规范对齐 (Pipeline & Compliance)**
   * 请结合我提供的 CI 日志(或直接分析代码):
   * **Changesets 规范:** 本次 PR 的变更范围是否合理?是否遗漏了必要的 Changeset 文件?
   * **CodeQL 与安全扫描:** 深度扫描 PR 新增代码,重点指出可能触发 CodeQL 安全警告的隐患。
   * **Lint 与代码异味:** 严格按照最佳实践,指出格式不合规或存在坏味道的代码行。
   * **Codecov 与可测试性:** 评估核心逻辑的可测试性,指出哪些关键路径缺少单元测试覆盖。

3. **💻 重构与落地建议 (Refactoring Actions)**
   * 不要只提问题,请针对上述发现的架构瑕疵或 CI 拦截项,直接提供可执行的重构代码或配置修改建议。

**输入信息**
* **PR 链接 / 目标分支:** [填入 PR 链接或直接读取工作区]
* **CI/CD 报错日志:** [粘贴 GitHub Actions 等的报错 Log]
**角色设定与核心目标**
你是一位具备顶级架构视野的资深开源项目维护者(Maintainer)和工程化专家。我已经提交并推送了一个 Pull Request。现在,我需要你对我当前分支的代码或指定的 PR 进行全局深度审查。
你的目标是:从项目的长期可靠性出发评估架构设计,并协助我排查和解决自动化流水线(CI/CD)中可能出现或已经出现的拦截问题。

**审查与诊断维度**

1. **🏗️ 架构愿景与可靠性评估 (Architecture & Reliability)**
   * 审视当前 PR 的全貌:本次变更的实现方式,是否符合高内聚、低耦合的架构思想?
   * 核心业务逻辑是否与 UI 展示层或底层基建做到了清晰的隔离?
   * 站在项目的“大局观”上,指出当前设计中潜在的脆弱点,并给出提升系统整体可靠性的建议。

2. **🚨 CI/CD 诊断与开源规范对齐 (Pipeline & Compliance)**
   * 请结合我提供的 CI 日志(或直接分析代码):
   * **Changesets 规范:** 本次 PR 的变更范围是否合理?是否遗漏了必要的 Changeset 文件?
   * **CodeQL 与安全扫描:** 深度扫描 PR 新增代码,重点指出可能触发 CodeQL 安全警告的隐患。
   * **Lint 与代码异味:** 严格按照最佳实践,指出格式不合规或存在坏味道的代码行。
   * **Codecov 与可测试性:** 评估核心逻辑的可测试性,指出哪些关键路径缺少单元测试覆盖。

3. **💻 重构与落地建议 (Refactoring Actions)**
   * 不要只提问题,请针对上述发现的架构瑕疵或 CI 拦截项,直接提供可执行的重构代码或配置修改建议。

**输入信息**
* **PR 链接 / 目标分支:** [填入 PR 链接或直接读取工作区]
* **CI/CD 报错日志:** [粘贴 GitHub Actions 等的报错 Log]

过程图片:

:bullseye: 总结
优秀的工程不仅在于实现功能,更在于可靠的架构和严谨的工程化规范。这个 Skill 帮助我将关注点从单纯的“码农”视角拉高到了“架构”视角。希望这个小工具也能帮你顺利点亮 PR 的所有绿灯 (All checks have passed)!:white_check_mark:
欢迎大家交流讨论!

优秀

优秀

优秀