audit-review

审计：请求代码审查

概述

派遣 the-inspector，在问题累积前及时发现它们。为 inspector 精心准备评估所需的上下文——绝不提供会话历史。这样可以让 inspector 专注于工作成果，而非思考过程，并为你保留上下文，以便继续工作。

核心原则： 尽早审计，经常审计。

何时请求

必须：

• 并行执行（The Hit）中的每项任务完成后
• 完成重大功能后
• 合并到 main 之前

可选但值得：

• 陷入困境时（获取新的视角）
• 重构之前（检查基线）
• 修复复杂错误后

如何请求

1. 获取 Git SHA

BASE_SHA=$(git rev-parse HEAD~1)  # or origin/main
HEAD_SHA=$(git rev-parse HEAD)

2. 派遣 the-inspector

使用 Task 工具派遣 the-inspector Agent，并设置 subagent_type: "the-inspector"。切勿使用 "general" 或 "general-purpose"——它们在 Gangsta Agents 安装中无效。

在此技能目录下的 the-inspector-prompt.md 中填写模板，并替换以下占位符：

• {WHAT_WAS_IMPLEMENTED} — 你刚刚构建的内容
• {PLAN_OR_REQUIREMENTS} — 应实现的内容（Contract 条款、规范章节）
• {BASE_SHA} — 起始提交
• {HEAD_SHA} — 结束提交
• {DESCRIPTION} — 更改的简要摘要

3. 处理审查结果

严重程度	操作
Critical	立即修复。解决前不得继续。
Important	进入下一项任务前修复。
Minor	记下来，留待以后处理。不要因此阻碍进度。
Wrong	提供技术依据进行反驳。

示例

[Just completed implementing auth middleware]

1. Get SHAs:
   BASE_SHA=$(git log --oneline | grep "previous task" | head -1 | awk '{print $1}')
   HEAD_SHA=$(git rev-parse HEAD)

2. Dispatch the-inspector:
   WHAT_WAS_IMPLEMENTED: JWT auth middleware with role-based access
   PLAN_OR_REQUIREMENTS: Contract section 3.2 — Authentication
   BASE_SHA: a7981ec
   HEAD_SHA: 3df7661
   DESCRIPTION: Added auth middleware, role guards, token validation

3. Inspector returns:
   Strengths: Clean separation, real tests
   Issues:
     Important: Missing token expiry check
     Minor: Magic number for token TTL
   Assessment: Ready with fixes

4. Fix important issue, proceed to next task.

与 Heist Pipeline 集成

• The Hit： 每位 Crew Lead 完成其负责区域后进行审计
• Laundering： Don 批准前进行最终审计
• 临时工作： 合并前进行审计

危险信号

切勿：

• 因为“很简单”而跳过审计
• 忽略 Critical 问题
• 未修复 Important 问题就继续
• 在没有证据的情况下反驳合理的技术反馈

如果 inspector 有误：

• 提供技术依据进行反驳
• 展示能证明其正常工作的代码或测试
• 使用 gangsta:receiving-orders 严谨地处理反馈

Omerta 合规性

• 真实原则：Inspector 审查实际差异，而非口头声明
• 介绍原则：Inspector 通过审计技能进行沟通，而非直接联系 Workers