File contents 任务目标
加载文档 (必须) : 读取 02_ARCHITECTURE_OVERVIEW.md、所有 04_SYSTEM_DESIGN/*.md 以及所有 03_ADR/*.md。
深度理解 : 使用 sequential-thinking skill(3-5 个 thought)在批判之前先理解设计意图。
Pre-Mortem : 想象项目在 6 个月后失败了——倒推根因。
三维审查 : 系统性执行全部 3 个维度。
假设验证 : 识别隐藏假设并尝试证伪。
生成发现 : 为每条发现标注严重度,并关联到具体文档段落。
硬约束
证据为本 : 每条发现必须 有具体文档引用 + 推理链。"可能有性能问题"这种没有分析的说法禁止出现。
质量优于数量 : 3 条真实发现 > 10 条猜测。
尊重 ADR 决策 : 如果 ADR 明确选择了某个权衡并附有文档化的理由,不要翻旧账。仅在出现新证据反驳原有理由时才标记。
不涉及实现细节 : 审查的是设计 ,不是假想的代码。
三维审查框架
维度 1: 系统设计 (System Design)
目标 : 验证架构的完整性、一致性和边界清晰度。
#
检查项
关注要点
SD-1
架构一致性
同一组件在 PRD、Architecture、System Design 中的描述是否矛盾?
SD-2
边界清晰度
每个系统的职责范围是否明确?是否存在职责重叠?
SD-3
依赖合理性
系统依赖是否无环?是否存在隐藏耦合?
SD-4
接口完整性
所有跨系统接口是否完整定义(输入/输出/错误/协议)?
SD-5
状态管理
系统状态转换是否清晰定义?边缘状态是否处理?
SD-6
数据模型完整性
实体关系是否跨文档一致?是否存在孤儿实体?
思考提示 (配合 sequential-thinking 使用):
"这个架构背后的核心假设是什么?"
"如果假设 X 不成立,什么会崩?"
"系统边界定义是否存在歧义?"
"接口是否覆盖了所有边界情况?"
维度 2: 运行模拟 (Runtime Simulation)
目标 : 在脑中"运行"系统,发现时序、状态和并发问题。
本维度必须 使用 sequential-thinking skill(3-5 个 thought)。运行时问题藏在序列中。
#
检查项
关注要点
RS-1
时序一致性
时序模型是否合理?是否存在"必须先于"的矛盾?
RS-2
状态同步
分布式状态下,副本是否可能发散?最终一致性在这里是否可接受?
RS-3
并发处理
两个操作冲突时会怎样?是否有解决策略?
RS-4
边界条件
空状态、满状态、溢出——各自如何处理?
RS-5
故障传播
组件 A 故障时如何影响 B、C?是否存在级联风险?
RS-6
Happy Path 偏见
只设计了正常路径?错误/超时/部分失败路径呢?
思考提示 :
"端到端追踪一个典型操作,经过哪些步骤?"
"每一步产生什么状态变更?什么可能出错?"
"如果两个用户同时做同一件事会怎样?"
"崩溃后 30 秒,系统是什么样子?"
维度 3: 工程实现 (Engineering Implementation)
目标 : 验证设计可构建、可测试、可维护。
本维度必须 使用 sequential-thinking skill(3-5 个 thought)。
#
检查项
关注要点
EI-1
可测试性
核心逻辑能否被单元测试?是否有 Mock 的接缝?
EI-2
可维护性
如果需求变更,需要改多少文件?
EI-3
性能瓶颈
设计中是否隐藏了 N+1 查询、无界循环或 O(n²)?
EI-4
安全面
认证边界是否清晰?敏感数据静态/传输加密?输入校验?
EI-5
可观测性
凭设计中的日志/指标方案能否调试生产问题?
EI-6
技术栈契合度
选定的技术是否真正支持所需功能?版本兼容性?
思考提示 :
"如何为核心逻辑写单元测试?"
"如果需要修改功能 X,影响范围多大?"
"性能热点在哪?能否后续优化?"
"这个设计暴露了哪些攻击向量?"
严重度分级
等级
判定标准
所需行动
Critical
根本性矛盾或不可能实现。不解决无法继续。
P0 — 必须在 blueprint/forge 之前修复
High
大概率导致返工或失败的严重风险。
P1 — 在 forge 之前修复
Medium
有变通方案的质量隐患。
P2 — 实现阶段修复
Low
润色项或轻微不一致。
P3 — 后续跟踪
输出格式
按以下结构生成发现,适合纳入 07_CHALLENGE_REPORT.md:
## 设计审查发现
### 摘要
| 维度 | 发现数 | Critical | High | Medium | Low |
|------|:------:|:--------:|:----:|:------:|:---:|
| 系统设计 | — | — | — | — | — |
| 运行模拟 | — | — | — | — | — |
| 工程实现 | — | — | — | — | — |
| **合计** | **—** | **—** | **—** | **—** | **—** |
**高信号结论**: [用 1-3 句概括最值得进入 challenge 主报告的问题]
---
### 核心发现清单
| ID | 维度 | 严重度 | 文档位置 | 发现 | 影响 | 建议 |
|----|------|--------|----------|------|------|------|
| DR-01 | 系统设计 | Critical | 02_ARCHITECTURE_OVERVIEW.md §X | 边界定义冲突,两个系统职责重叠 | 实现阶段职责漂移、返工风险高 | 重新划清系统边界并更新引用 |
| DR-02 | 运行模拟 | High | 04_SYSTEM_DESIGN/... §Y | 故障传播路径未定义 | 级联失败时无法收敛 | 增加超时/降级/重试策略 |
| DR-03 | 工程实现 | Medium | ADR-00X / System Design §Z | 可测试接缝不足 | 后续验证成本高 | 增加接口隔离或 mock 接缝 |
> 仅输出真正影响设计判断的问题。低价值措辞、重复担忧不要进入清单。
---
### Top Findings 详情(仅展开 Critical / High)
#### DR-01 [标题]
**严重度**: Critical
**文档位置**: [精确的文件和章节引用]
**证据**:
- 文档分析: [来自 PRD/Architecture/ADR 的具体内容]
- 推理链: [基于 `sequential-thinking` 的分析]
- 类比: [其他系统中的类似已知失败,如适用]
**影响**:
- [不修复会发生什么]
**建议**:
- [最小修复方向]
审查质量清单
交付发现前,确认:
每条发现有具体文档引用(不是笼统的"架构文档")
每条发现有明确的影响说明
没有纯猜测性的发现(缺乏推理链条)
Critical/High 发现经过 sequential-thinking 验证
ADR 中已记录的权衡被尊重(没有在无新证据的情况下重复质疑)
发现可操作(审查者能根据建议进行修复)
1 --- 2 name: design-reviewer-3 3 description: 任务目标 4 --- 5 6 ## 任务目标 7 8 1. **加载文档 (必须)**: 读取 `02_ARCHITECTURE_OVERVIEW.md`、所有 `04_SYSTEM_DESIGN/*.md` 以及所有 `03_ADR/*.md`。 9 2. **深度理解**: 使用 `sequential-thinking` skill(3-5 个 thought)在批判之前先理解设计意图。 10 3. **Pre-Mortem**: 想象项目在 6 个月后失败了——倒推根因。 11 4. **三维审查**: 系统性执行全部 3 个维度。 12 5. **假设验证**: 识别隐藏假设并尝试证伪。 13 6. **生成发现**: 为每条发现标注严重度,并关联到具体文档段落。 14 15 ## 硬约束 16 17 - **证据为本**: 每条发现**必须**有具体文档引用 + 推理链。"可能有性能问题"这种没有分析的说法禁止出现。 18 - **质量优于数量**: 3 条真实发现 > 10 条猜测。 19 - **尊重 ADR 决策**: 如果 ADR 明确选择了某个权衡并附有文档化的理由,不要翻旧账。仅在出现新证据反驳原有理由时才标记。 20 - **不涉及实现细节**: 审查的是*设计*,不是假想的代码。 21 22 --- 23 24 ## 三维审查框架 25 26 ### 维度 1: 系统设计 (System Design) 27 28 **目标**: 验证架构的完整性、一致性和边界清晰度。 29 30 31 | # | 检查项 | 关注要点 | 32 | ---- | ----------- | ---------------------------------------------- | 33 | SD-1 | **架构一致性** | 同一组件在 PRD、Architecture、System Design 中的描述是否矛盾? | 34 | SD-2 | **边界清晰度** | 每个系统的职责范围是否明确?是否存在职责重叠? | 35 | SD-3 | **依赖合理性** | 系统依赖是否无环?是否存在隐藏耦合? | 36 | SD-4 | **接口完整性** | 所有跨系统接口是否完整定义(输入/输出/错误/协议)? | 37 | SD-5 | **状态管理** | 系统状态转换是否清晰定义?边缘状态是否处理? | 38 | SD-6 | **数据模型完整性** | 实体关系是否跨文档一致?是否存在孤儿实体? | 39 40 41 **思考提示**(配合 `sequential-thinking` 使用): 42 43 1. "这个架构背后的核心假设是什么?" 44 2. "如果假设 X 不成立,什么会崩?" 45 3. "系统边界定义是否存在歧义?" 46 4. "接口是否覆盖了所有边界情况?" 47 48 --- 49 50 ### 维度 2: 运行模拟 (Runtime Simulation) 51 52 **目标**: 在脑中"运行"系统,发现时序、状态和并发问题。 53 54 > 本维度**必须**使用 `sequential-thinking` skill(3-5 个 thought)。运行时问题藏在序列中。 55 56 57 | # | 检查项 | 关注要点 | 58 | ---- | ----------------- | ------------------------------ | 59 | RS-1 | **时序一致性** | 时序模型是否合理?是否存在"必须先于"的矛盾? | 60 | RS-2 | **状态同步** | 分布式状态下,副本是否可能发散?最终一致性在这里是否可接受? | 61 | RS-3 | **并发处理** | 两个操作冲突时会怎样?是否有解决策略? | 62 | RS-4 | **边界条件** | 空状态、满状态、溢出——各自如何处理? | 63 | RS-5 | **故障传播** | 组件 A 故障时如何影响 B、C?是否存在级联风险? | 64 | RS-6 | **Happy Path 偏见** | 只设计了正常路径?错误/超时/部分失败路径呢? | 65 66 67 **思考提示**: 68 69 1. "端到端追踪一个典型操作,经过哪些步骤?" 70 2. "每一步产生什么状态变更?什么可能出错?" 71 3. "如果两个用户同时做同一件事会怎样?" 72 4. "崩溃后 30 秒,系统是什么样子?" 73 74 --- 75 76 ### 维度 3: 工程实现 (Engineering Implementation) 77 78 **目标**: 验证设计可构建、可测试、可维护。 79 80 > 本维度**必须**使用 `sequential-thinking` skill(3-5 个 thought)。 81 82 83 | # | 检查项 | 关注要点 | 84 | ---- | ---------- | ---------------------------- | 85 | EI-1 | **可测试性** | 核心逻辑能否被单元测试?是否有 Mock 的接缝? | 86 | EI-2 | **可维护性** | 如果需求变更,需要改多少文件? | 87 | EI-3 | **性能瓶颈** | 设计中是否隐藏了 N+1 查询、无界循环或 O(n²)? | 88 | EI-4 | **安全面** | 认证边界是否清晰?敏感数据静态/传输加密?输入校验? | 89 | EI-5 | **可观测性** | 凭设计中的日志/指标方案能否调试生产问题? | 90 | EI-6 | **技术栈契合度** | 选定的技术是否真正支持所需功能?版本兼容性? | 91 92 93 **思考提示**: 94 95 1. "如何为核心逻辑写单元测试?" 96 2. "如果需要修改功能 X,影响范围多大?" 97 3. "性能热点在哪?能否后续优化?" 98 4. "这个设计暴露了哪些攻击向量?" 99 100 --- 101 102 ## 严重度分级 103 104 105 | 等级 | 判定标准 | 所需行动 | 106 | ------------ | -------------------- | ----------------------------- | 107 | **Critical** | 根本性矛盾或不可能实现。不解决无法继续。 | P0 — 必须在 blueprint/forge 之前修复 | 108 | **High** | 大概率导致返工或失败的严重风险。 | P1 — 在 forge 之前修复 | 109 | **Medium** | 有变通方案的质量隐患。 | P2 — 实现阶段修复 | 110 | **Low** | 润色项或轻微不一致。 | P3 — 后续跟踪 | 111 112 113 --- 114 115 ## 输出格式 116 117 按以下结构生成发现,适合纳入 `07_CHALLENGE_REPORT.md`: 118 119 ```markdown 120 ## 设计审查发现 121 122 ### 摘要 123 124 | 维度 | 发现数 | Critical | High | Medium | Low | 125 |------|:------:|:--------:|:----:|:------:|:---:| 126 | 系统设计 | — | — | — | — | — | 127 | 运行模拟 | — | — | — | — | — | 128 | 工程实现 | — | — | — | — | — | 129 | **合计** | **—** | **—** | **—** | **—** | **—** | 130 131 **高信号结论**: [用 1-3 句概括最值得进入 challenge 主报告的问题] 132 133 --- 134 135 ### 核心发现清单 136 137 | ID | 维度 | 严重度 | 文档位置 | 发现 | 影响 | 建议 | 138 |----|------|--------|----------|------|------|------| 139 | DR-01 | 系统设计 | Critical | 02_ARCHITECTURE_OVERVIEW.md §X | 边界定义冲突,两个系统职责重叠 | 实现阶段职责漂移、返工风险高 | 重新划清系统边界并更新引用 | 140 | DR-02 | 运行模拟 | High | 04_SYSTEM_DESIGN/... §Y | 故障传播路径未定义 | 级联失败时无法收敛 | 增加超时/降级/重试策略 | 141 | DR-03 | 工程实现 | Medium | ADR-00X / System Design §Z | 可测试接缝不足 | 后续验证成本高 | 增加接口隔离或 mock 接缝 | 142 143 > 仅输出真正影响设计判断的问题。低价值措辞、重复担忧不要进入清单。 144 145 --- 146 147 ### Top Findings 详情(仅展开 Critical / High) 148 149 #### DR-01 [标题] 150 151 **严重度**: Critical 152 **文档位置**: [精确的文件和章节引用] 153 154 **证据**: 155 - 文档分析: [来自 PRD/Architecture/ADR 的具体内容] 156 - 推理链: [基于 `sequential-thinking` 的分析] 157 - 类比: [其他系统中的类似已知失败,如适用] 158 159 **影响**: 160 - [不修复会发生什么] 161 162 **建议**: 163 - [最小修复方向] 164 ``` 165 166 --- 167 168 ## 审查质量清单 169 170 交付发现前,确认: 171 172 - 每条发现有具体文档引用(不是笼统的"架构文档") 173 - 每条发现有明确的影响说明 174 - 没有纯猜测性的发现(缺乏推理链条) 175 - Critical/High 发现经过 `sequential-thinking` 验证 176 - ADR 中已记录的权衡被尊重(没有在无新证据的情况下重复质疑) 177 - 发现可操作(审查者能根据建议进行修复) 178
haaaiawd/my-swe-agent/tree/main/.windsurf/skills/design-reviewer commit 3df0be6694
Frequently asked questions How do I install the Design Reviewer skill? Run npx skillmds@latest add haaaiawd/design-reviewer-3 in your terminal (requires Node.js), paste this page's agent-chat prompt into Claude, Cursor, or any MCP-connected agent, or download the SKILL.md file and copy it into your agent's skills directory.
What does the Design Reviewer skill do? 任务目标 It is listed under Coding & Dev Tools on SkillMD.
Is Design Reviewer safe to use? This skill has not completed SkillMD's automated safety review yet. Independent scanners report: SkillSpector: PASS, Skill Scanner: PASS. SkillMD never runs a skill's scripts for you; review the SKILL.md before installing.
Which AI agents work with Design Reviewer? This skill is tagged as working with Claude Code, Claude.ai, OpenAI Codex. SKILL.md is an open format, so most agents that read a skills directory can load it too.
Is Design Reviewer free to use? Yes. Installing skills from SkillMD is free, and the skill stays under its author's original license.
Who published Design Reviewer? haaaiawd (@haaaiawd) published this skill. Their other Agent Skills are listed on their SkillMD profile.