mcpField
R
review-debt

review-debt

github.com/gtrabanco测试与质量2286 次安装今天更新
代码仓库

关于 review-debt

review-debt 是用于审查流程的内部 Skill,将综合审查表中的技术债发现转化为明确的待偿还事项。它保留既有归属判断,并为事项记录证据和触发条件,不修改代码。

主要功能

  1. 01将技术债发现转为待偿事项
  2. 02保留既有归属轴
  3. 03要求提供证据和偿还触发条件
  4. 04标记被误判为非阻塞的当前单元债务
  5. 05输出固定格式的审查表

适用场景

  1. 01将 TODO 或 FIXME 发现转为债务事项
  2. 02记录延期工作的触发条件
  3. 03检查当前单元债务是否被误分类

安装

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

$npx skills add https://github.com/gtrabanco/agentic-workflow --skill review-debt

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

详细介绍

审查技术债(内部)

由 review-change / product-audit 在对话中编写——对于任何 Agent,均应将此文件作为路由步骤内联遵循。仅报告发现;绝不编辑, 绝不重构。

范围

调用方的综合发现表(合并并分类后的决策表)。此轮检查不 重新扫描差异:具有技术债特征的发现已在表中,且所有归属决策都已在发现轴 (review-code、review-verify 等)中完成。请在返回表格的顶部说明范围—— 即该表综合所依据的变更。

转换(评估每一行——无一例外;必须说明 n/a)

将表中具有技术债特征的行(TODO/FIXME/HACK、重复、过时或孤立的抽象、 死代码、复杂度热点、缺少测试、依赖上游修复的变通方案)转化为明确、可偿付的债务项:

✓ 在 file:line 处重述每项具有技术债特征的发现,并说明其推迟了什么 ✓ 将其归属到表中已记录的轴——绝不重新争论归属 ✓ 确认每项债务都带有 TRIGGER:必须偿付该债务的条件(例如“出现第 3 个使用方”、 “>100k 行”)——没有触发条件的债务项本身就是一项发现 ✓ 核实没有将当前单元的债务误标为非阻塞:当前单元的工作不能标为 postpone / tradeoff / wontfix——若表中出现此类标记,应退回分类器处理,绝不在此处重新分类 ✓ 遵守死代码例外:经路线图 / SPEC / TASKS 交叉核对后确认的暂存 / 计划代码不属于死代码——不确定时标记为 verify,绝不妄下结论

返回内容必须完全如下

REVIEW TECH DEBT — scope: <scope>

| # | Finding | Sev | Evidence | Suggested fix |
|---|---------|-----|----------|---------------|
| 1 | <what>  | critical|major|minor | <file:line> | <trigger + smallest action> |

Rows: <n> transformed, <n> with trigger, <n> findings, <n> n/a (<which + why>)
Summary: <1-2 sentences>
Decision: PASS | FAIL

本轮检查中,“建议修复”列须包含 TRIGGER——必须偿付该债务的条件——以及最小化操作。

如果有任何严重或主要发现尚未解决,则 FAIL;否则 PASS。次要发现绝不阻塞——它们会作为债务说明出现在调用方的报告中。

完成条件

  • 综合表中每一行具有技术债特征的内容都已根据证据(file:line)完成转换,或明确标记为 n/a 并说明原因。
  • 每项债务都有触发条件;没有债务被误标为当前单元。
  • 返回上述固定格式的内容——不多也不少——且未更改任何代码。

评论

支持 Markdown