mcpField
D
ds-review-loop

ds-review-loop

github.com/aicodeking测试与质量263440 次安装2 周前更新
代码仓库

关于 ds-review-loop

ds-review-loop 是一种代码审查工作流技能，通过一位未参与过的审查者反复检查限定范围的改动来提高正确性。它规定检查、评估问题、记录处理结果及停止条件。

主要功能

  1. 01限定改动范围并运行检查
  2. 02请求独立且聚焦的审查
  3. 03分类并评估审查问题
  4. 04依据证据记录处理决定
  5. 05每轮审查后重新检查改动

适用场景

  1. 01提交前审查代码改动
  2. 02以最小修复解决报告的问题
  3. 03记录有争议或未采纳的审查意见

安装

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

$npx skills add https://github.com/aicodeking/deepseek-stack --skill ds-review-loop

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

详细介绍

通过让一位新的评审者提出异议，直到双方达成一致，使变更更加可靠。

步骤

  1. 明确变更范围。运行 git status --short。不要暂存、丢弃或纳入无关文件。查找基准：运行 git fetch origin，然后运行 git merge-base HEAD origin/<base branch>。每一轮只使用一条 diff 命令。

  2. 用两句话写明目标和范围外事项。运行覆盖性检查。不要提交未通过检查的代码。

  3. 选择评审者：如果评审环境能访问更强的模型，就选择更强的模型（设置了模型的子 Agent、团队成员或第二个 CLI）；否则选择一个没有历史记录的全新 DeepSeek 上下文。记录所选的评审者。

  4. 发送以下简要说明：

    Review only. Do not edit, commit, push, comment or start subagents.
Repo: <path>  Base: <sha>  Diff: git diff <sha> [-- <paths>]
Goal: <goal>  Out of scope: <list>  Checks: <results>
Read the code around the diff. Find bugs, broken callers and gaps against the goal.
Flag code the goal does not need. Ignore style unless it hides a bug.
blocker = goal case is wrong. should = real bug users will hit. nit = anything else.
Reply: VERDICT: APPROVE or CHANGES, then one line per finding:
R1 [blocker|should|nit] file:line claim. Fix: suggestion

  5. 等待回复。评审期间不要编辑已限定范围内的文件。超时或出错不代表批准。

  6. 自行判断每条发现。接受：进行最小幅度的修复，优先删除或缩小代码范围。不接受：提供证据（文件:行号、测试输出）进行回复。小问题或范围外事项：用一句话说明不予处理。

  7. 保留记录：编号、严重程度、处理结果、证据。重新运行检查，并运行 git diff --shortstat。如果 diff 扩大了，就精简回去。

  8. 下一轮发送简要说明、记录，以及自上一轮以来的变更。请评审者只检查仍未解决的编号和变更的代码。

停止条件

  • 评审者针对当前 diff 回复 APPROVE，且没有未解决的 blocker 或 should，也没有你仍在意的问题。
  • 某一轮只返回小问题。
  • 已完成四轮，或某项在两次交流后仍有争议。分别用两到三行报告双方的观点，让用户决定。这不代表批准。

规则

  • 一次只允许一位评审者。绝不要为了结束循环而勉强同意。
  • 循环期间不要推送。只有用户要求推送或发布时，才在之后推送。
  • 如果更新 PR，请添加：Reviewed with ds-review-loop: <N> rounds, <reviewer>, verdict <verdict>.
  • 报告轮数、评审者、已修复和未采纳的发现及其原因，以及变更前后的 diff 大小。

评论

支持 Markdown