跳转到内容
关系图谱

no-comments(pstack)

476 字 2 分钟

属性 8
repo
pstack
skill
no-comments
source
pstack/skills/no-comments/SKILL.md
invoke
user
phase
review
cost
mid
status
stable
local
missing
一句话

评审前把注释交给没写过它们的 Comment Sicko 子代理屠杀一遍,接受的发现按根因修掉,声称有约束的注释提议编码成类型、测试或 lint。

何时用 ​

  • 适合:合并前清理 diff 里的叙述性注释、workaround 辩解、do not remove 类警告。
  • 不适合:注释作者本人自己删(作者会为自己的注释辩护,这正是要换新鲜眼睛的原因)。

怎么用 ​

text
/no-comments 清理这个 diff
  1. 以 subagent_type: "comment-sicko" 起只读子代理,范围用调用方给的文件或默认对 main 的 diff。
  2. 审报告和 diff:拒绝越界删改、应用代码编辑、误伤例外清单内的保留项;含糊的 kill 不恢复,被驳倒或仍含糊的 keep 删除。
  3. 琐碎修复直接做;需要动形状时跑一次 pstack-architect 拿草图就停。
  4. 实现范围内最小的根因修复,不加症状护栏;根因在范围外就报 open。
  5. do not remove 类约束注释:提议最便宜的编码手段(类型、运行时检查、测试、CI lint),获批后编码再删注释,否则删除并报告约束未落实。
  6. 汇报删除数、恢复数、重跑数、修复、编码提议与结果、未决项。

产出 ​

  • 清理后的 diff 加一份处理报告;Comment Sicko 只删注释和标 MUST KILL,不碰代码。

要点与坑 ​

  • Comment Sicko 的保留清单很短:license 头、公共 API 文档注释、解释代码无法表达约束的链接、外部依赖强制的非显然行为。
  • IMPORTANT/do not remove 只是嫌疑不是豁免;判之前先对相关符号跑 pstack-how 或 pstack-why。
  • 依赖 Cursor comment-sicko agent 类型;Devin CLI 里可用普通只读子代理加同一份保留清单模仿。

关联 ​