القائمة
Use after writing a plan document in docs/plans/, before requesting user confirmation
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
استنادا إلى تصنيف SOC المهني
Use when receiving an ambiguous feature request, when scope is unclear, or before writing any plan
Use after verification-before-completion passes and code review is clean, to close out the development branch
Use when receiving feedback from code-reviewer subagent or human reviewer
Use when a feature, bugfix, or refactoring step is completed and needs review, or before merging to main, or when user says "review", "审查", "帮我看看代码"
Use when executing an implementation plan with 3+ independent tasks in the current session
Use when facing Xcode build errors, linker failures, XCTest failures, event pipeline bugs, network issues, SDK crashes, or any technical problem where a first attempt didn't work or the cause is unclear
| name | plan-document-review |
| description | Use after writing a plan document in docs/plans/, before requesting user confirmation |
Type: Technique | Discipline: Rigid
在 Planning Gate 输出完规划文档后、请求用户确认前，dispatch 一个 plan reviewer subagent 审查规划本身的质量。
核心原则： plan 是实施的蓝本，蓝本有洞 = 实施有洞。
强制： Planning Gate 保存完 plan 到 docs/plans/ 之后、提示用户确认之前。
可跳过： Planning Gate 未触发的小改动（< 3 文件且不涉及公开 API）。
Agent({
description: "Review plan document",
subagent_type: "general-purpose",
prompt: `
你是 GrowingIO iOS SDK 的规划审查员。验证以下规划是否完备、可实施。
## 待审查规划
文件路径：{PLAN_FILE_PATH}
规划全文（已粘贴，不用再读）：
---
{PLAN_FULL_TEXT}
---
## 原始需求 / 任务描述
{ORIGINAL_TASK_DESCRIPTION}
## 检查维度
### 1. 四节完整性
规划必须包含以下四节，缺一不可：
- [ ] 影响文件列表
- [ ] 公开 API 变更
- [ ] 数据协议变更
- [ ] 需同步修改的文档
### 2. 影响文件列表准确性
- 文件路径是否真实存在？
- 是否有明显遗漏：
- 新增公开 API 但没列对应路径：模块根 \`.h\` / \`GrowingTrackerCore/Public/\` / \`GrowingAutotrackerCore/Public/\` / \`Modules/*/Public/\`
- 修改事件字段但没列对应测试文件
- 修改某模块但没列 podspec / Package.swift
- 有没有列了但实际不需要改的（scope creep）？
### 3. 公开 API 变更一致性
- 是否在影响文件列表中包含了公开头文件 + podspec + Package.swift？
- 签名是否完整（参数类型、返回类型）？
- 删除/重命名是否考虑了兼容性？
### 4. 数据协议变更覆盖
- 涉及字段变更时，"影响产品线"是否完整（SaaS / CDP）？
- 字段类型是否与数据协议规范对齐？
### 5. 文档同步完整性
- 公开 API 变更是否列了对应文档？
- 涉及核心模块是否列了对应模块文档？
### 6. 任务拆分质量
- 任务是否大部分相互独立？
- 任务粒度是否合理？
### 7. 场景覆盖
- Autotrack 改动：iOS / macOS / tvOS 三平台？Hybrid / WebCircle 适配？
- Session 改动：前台/后台/终止三种触发时机？
- 事件改动：Protobuf / JSON 双格式？
## 输出格式
### 问题
#### Critical（阻塞）
#### Important（应修复）
#### Suggestion（建议）
### 检查清单
- [ ] 四节完整
- [ ] 影响文件列表准确无遗漏
- [ ] 公开 API 变更同步头文件 + podspec + Package.swift
- [ ] 数据协议产品线完整
- [ ] 文档同步列表完整
- [ ] 任务拆分合理
- [ ] 场景覆盖完整
### 结论
**通过** / **需要修改** / **需要讨论**
`
})
reviewer 返回结果
├── "通过" → 向用户展示 plan + reviewer 摘要，请求确认
├── "需要修改"
│ ├── Critical/Important → 修改 plan，重新 dispatch reviewer（最多 2 轮）
│ └── 第 2 轮后仍有问题 → 停止，向用户说明
└── "需要讨论" → 与用户讨论后决定
| Excuse | Reality |
|---|---|
| "自己写的 plan 自己审就行" | 自审替代不了独立审查 |
| "跳过 reviewer 直接请求用户确认" | 独立审查是硬性流程 |
| "修完 Critical 不用重新 dispatch" | 必须重新 dispatch |
writing-plans 产出 plan 文件后