dsh-requesting-code-review

Pre-Commit Code Verification（DSH 版）

Automated verification pipeline before code lands. Static scans, baseline-aware quality gates, an independent reviewer subagent, and an auto-fix loop.

Core principle: No agent should verify its own work. Fresh context finds what you miss.

与 Hermes 版的关系：流程 / 契约 / 铁律同源。差异只在平台机制——DSH 用批量并行 subagent（同一条消息多个调用并行执行，默认并发上限 10）替代 delegate_task(tasks=[...])；审查员没有 toolsets 参数可传，改用 prompt 指令约束"不要调用工具"；大规模审查可打包成 references/dual-axis-review.md 的 workflow（agent() 的 opts.schema 在工具层强校验 JSON 契约）。

When to Use

• After implementing a feature or bug fix, before git commit or git push
• When user says "commit", "push", "ship", "done", "verify", or "review before merge"
• After completing a task with 2+ file edits in a git repo
• 多任务流水线的每个任务后（作为质量闸门）

Skip for: documentation-only changes, pure config tweaks, or when user says "skip verification".

Step 1 — Get the diff

git diff --cached

Empty → git diff，再空 → git diff HEAD~1 HEAD。若 --cached 空但 git diff 有内容，提示用户 git add <files>（按 dsh-dev-pm-flow 流程改动通常未暂存，直接审查 git diff 即可）。仍空 → git status，没什么可验。

diff 超 15,000 字符 → 按文件拆分：

git diff --name-only
git diff HEAD -- specific_file.cs

Step 2 — Static security scan

只扫新增行。任何命中 = 安全信号，喂给 Step 5。

按项目技术栈选择扫描 pattern 库（单一事实源，复用仓库级 requesting-code-review/references/dotnet.md，不复制；作为 DSH 技能安装后随包部署于 references/dotnet.md）：

• .NET / C# / SqlSugar / Oracle → 读 requesting-code-review/references/dotnet.md，执行其中的扫描命令与 checklist；pwsh 等价：git diff --cached | Select-String -Pattern '^\+[^\+]' | Select-String -Pattern '<同一条 -iE 正则>'
• 其他技术栈（Java/Go/Python…）→ 手动按对应语言补充 pattern；没有对应参考文件时明说"该栈无内置 pattern"
• 硬约束：未读对应 references 文件（.NET 即 dotnet.md）不得进入扫描——SKILL.md 不内置任何可执行 pattern，漏读 = 空转还自以为扫过

通用原则：Select-String -Pattern '^\+[^\+]' 排除 +++ b/file diff 头行；扫描只针对新增行，命中 = 需人工复核的信号。

Step 3 — Baseline tests and linting

检测项目语言，跑对应工具。先测改动前失败数 baseline_failures（stash 改动 → 跑 → pop），只统计你改动新增的失败。