T
traeai
登录
返回首页
meng shao(@shao__meng)

一起看看 Cursor 团队内部最常用的 Skills

8.2Score
一起看看 Cursor 团队内部最常用的 Skills

TL;DR · AI 摘要

Cursor团队开源了其内部使用的代码质量审计技能thermo-nuclear-code-quality-review,通过四条核心原则和两阶段Agent协作机制,在PR合并前识别并拒绝那些功能正确但降低代码库质量的改动。

核心要点

  • 代码质量审计遵循四原则:删除复杂性而非搬运、阻止超1000行文件、标记薄包装层和泄漏逻辑、拒绝让代码更乱的PR
  • 采用两阶段父子Agent协作:父Agent准备上下文并行收集diff和文件内容,子Agent执行结构化审计
  • Skill具备fallback机制:当cursor-team-kit插件不可用时退化为严厉的可维护性审计

结构提纲

按章节快速跳转。

  1. Cursor团队内部使用的代码质量审计技能专门识别功能正确但降低代码库质量的PR改动并予以拒绝。

  2. 删除复杂性而非搬运复杂性、阻止超过1000行文件、标记薄包装层和泄漏逻辑、拒绝让代码更乱的PR构成了质量审计的核心准则。

  3. 父Agent并行收集git diff和变更文件内容,然后传递给子Agent执行结构化审计,确保输入确定性避免噪声干扰。

  4. cursor-team-kit插件不可用时,Skill会退化为严厉的可维护性审计,保持大胆简化和规范分层的目标对齐。

思维导图

用一张图看清主题之间的关系。

查看大纲文本(无障碍 / 无 JS 友好)
  • Cursor代码质量审计Skill
    • 四条核心原则
      • 删除复杂性而非搬运
      • 阻止超1000行文件
      • 标记薄包装层和泄漏逻辑
      • 拒绝让代码更乱的PR
    • 两阶段Agent机制
      • 父Agent准备上下文
      • 子Agent执行审计
    • 兜底逻辑
      • fallback机制
      • 可维护性审计

金句 / Highlights

值得收藏与分享的关键句。

  • 在 PR 合并之前,识别那些'能跑、但让代码库变得更糟'的改动并拒掉它们。
    ⬇︎ 下载 PNG𝕏 分享到 X
  • 阻止超过 1000 行的文件,硬性上限。超过 1k 行的文件几乎必然意味着职责不清、难以测试、难以并行修改。
    ⬇︎ 下载 PNG𝕏 分享到 X
  • 功能正确 ≠ 可以合并。维护性是一等公民,与功能性同等重要。
    ⬇︎ 下载 PNG𝕏 分享到 X
  • 审计 agent 本身不做信息收集,输入是结构化的、确定的,避免它在探索阶段被噪声干扰。
    ⬇︎ 下载 PNG𝕏 分享到 X
#Cursor#代码质量#AI Agent#PR审核#软件工程
打开原文

thermo-nuclear-code-quality-review,一位苛刻的代码质量审计员,它的设计目标是:在 PR 合并之前,识别那些"能跑、但让代码库变得更糟"的改动并拒掉它们。

在这安装 @cursor_ai Team Kit: https://t.co/5yA5GD848G

四条核心审计原则

  1. https://t.co/sGcvAGl66I" / X

一起看看 Cursor 团队内部最常用的 Skills thermo-nuclear-code-quality-review,一位苛刻的代码质量审计员,它的设计目标是:在 PR 合并之前,识别那些"能跑、但让代码库变得更糟"的改动并拒掉它们。 在这安装

Team Kit: cursor.com/marketplace/cu四条核心审计原则 1. 删除复杂性,而不是搬运复杂性 重构的常见反模式是把一坨复杂逻辑从 A 文件挪到 B 文件,调用栈变深,但总复杂度没降甚至上升。该 skill 要求改动产生净简化,而非位置转移。 2. 阻止超过 1000 行的文件 硬性上限。超过 1k 行的文件几乎必然意味着职责不清、难以测试、难以并行修改。这是一条"宁可错杀"的工程红线。 3. 标记薄包装层和泄漏的逻辑 · Thin wrappers:只是转发参数、没有实质行为的函数/类,徒增间接层。 · Leaked logic:本该封装在某层的业务规则散落到了调用方,破坏边界。 4. 拒绝"能工作但让代码更乱"的 PR 这是最关键的一条立场:功能正确 ≠ 可以合并。维护性是一等公民,与功能性同等重要。 两阶段、父子 Agent 协作的运行机制 阶段 1 — 父 agent 准备上下文(并行) · 一个 shell 子 agent 执行 git diff <base>...HEAD(默认 base 为 main) · 一个 explore 子 agent 抓取所有变更文件的完整内容 阶段 2 — 调用审计 agent 父 agent 把上面两份产物组装成带 ### Git / diff output 和 ### Changed file contents 两个标注段落的 prompt,传给 thermo-nuclear-code-quality-review 子 agent 执行审计。 这个设计有几个值得注意的点: · 审计 agent 本身不做信息收集,输入是结构化的、确定的,避免它在探索阶段被噪声干扰。 · 禁止嵌套子 agent(除非显式要求),强制单次、聚焦的判断。 · 只评判 diff 中看得见的东西,但当改动触及模块边界时要追踪跨文件影响——这避免了"只见树木"的局部审查。 Rubric 的兜底逻辑 Skill 明确写了一个 fallback:如果 cursor-team-kit 插件不在,就退化为一次"严厉的可维护性审计",目标对齐: · 大胆简化(ambitious simplification) · 文件不无故膨胀超过 ~1k 行 · 不容忍随手加 if/else 让分支无序增长 · 显式类型与边界 · 规范的分层(canonical layers)

AI 可能会生成不准确的信息,请核实重要内容

一起看看 Cursor 团队内部最常用的 Skills | meng shao(@shao__meng) | traeai