multica-review-architect · diff
git:20260910.8db057c to git:20260910.699a593
1 added, 1 removed. Audit A to A.
---
name: multica-review-architect
description: 架构设计专属评审框架。由 ArchReviewer 调用，对 Architect 产出的技术架构设计做多维度专业分析，输出 PASS/FAIL 与带 ID 的修改清单，汇报 Leader。
---
# 架构设计专业评审（ArchReviewer）
- 本 skill 提供对**技术架构设计产物**的结构化专业评审框架。调用方为 `ArchReviewer`，评审对象为 `Architect` 经 `multica-artifact-design-sync` 回传的设计链接。
+ 本 skill 提供对**技术架构设计产物**的结构化专业评审框架。调用方为 `ArchReviewer`，评审对象为 `Architect` 经 `multica-artifact-architect` 回传的设计链接。
## 什么时候用
- ArchReviewer 收到 Leader 派发的「评审架构设计」任务时。
- 设计修改后进入复审轮次时（对照上一轮修改清单 + **评审回应**表逐条核对）。
## 评审前置检查（元数据）
设计文档 H1 下必须有元数据表；缺项或复审未升版本 → 可直接 FAIL：
| 检查项 | FAIL 条件 |
| --- | --- |
| 创建者 / 创建时间 / 版本 / 状态 / JIRA / 上游 PRD | 任一缺失 |
| 复审稿版本 | 未高于上一轮（如仍 v0.1 却称已修改） |
| 修订记录 | 与版本不一致或缺少本轮条目 |
| 评审回应 | 复审稿未逐条回应上一轮阻断项 |
## 评审维度（逐项给结论）
每项标注 **PASS / FAIL / N/A**，FAIL 须写入修改清单并分配 **REV-n** ID。
### A. 文档与流程
1. **元数据与版本**：头部信息完整；修订记录与版本一致。
2. **评审回应完整性**（复审）：上一轮 REV-n 是否逐条回应；不采纳项是否有充分理由（仍可能阻断，交 Leader 裁决）。
### B. 方案质量
3. **方案合理性**：分层、模块边界、依赖方向清晰；无过度设计或明显设计不足。
4. **范围与非目标**：与 PRD/Issue 一致；非目标明确；无隐性扩 scope。
5. **方案备选与权衡**：是否说明为何不用更简方案（复杂度高时必问）。
### C. 正确性与一致性
6. **验收对齐**：逐条覆盖 AC-/FR-/BR-；**需求追溯**表完整无悬空 AC。
7. **跨端一致性**：前后端/UI 契约边界自洽；状态机、错误语义不冲突。
8. **数据与一致性**：实体/字段变更清楚；读写归属、事务边界、历史数据策略明确。
### D. 可靠性与安全
9. **失败模式与降级**：依赖失败、超时、部分失败时的行为；是否可接受。
10. **并发与幂等**：重复请求、竞态、重试策略是否定义。
11. **安全与权限**：鉴权、敏感数据、审计；是否满足团队基线。
12. **关键技术风险**：性能、容量、一致性风险是否识别并有对策；**无对策 = 阻断**。
### E. 可落地与可运维
13. **扩展性与可维护性**：常见变更成本；是否引入难逆技术债。
14. **可测试性**：验证计划能否执行；实现步骤是否可对应测试。
15. **迁移与回滚**：数据迁移、发布顺序、功能开关、回滚路径（涉及时必填）。
16. **可观测性**：日志、指标、告警是否足够定位生产问题。
### F. 开放项
17. **待决事项**：DECISION-/BLOCKED 是否阻塞进入开发；是否需 Leader/PM 关闭。
## 输出格式
```text
【架构设计评审】<设计链接>
文档版本：v0.x（元数据表）
结论：PASS / FAIL
元数据检查：PASS / FAIL — <说明>
维度结论摘要：
- A 文档与流程：...
- B 方案质量：...
- C 正确性与一致性：...
- D 可靠性与安全：...
- E 可落地与可运维：...
- F 开放项：...
阻断项（FAIL 时必填，ID 稳定便于复审核对）：
- REV-1 | 维度 | 理由 | 涉及章节/AC | 修改方向
- REV-2 | ...
建议项（非阻断）：
- SUG-1 | ...
与上一轮核对（复审时）：
- REV-x：已解决 / 未解决 / 不采纳待 Leader 裁决
- 已解决 X 项 / 未解决 Y 项
轮次：第 N / 3 轮
```
结论与修改清单**汇报给 Leader**，不自行修改设计、不自行通知 Architect。
## 评审原则
```text
无对策的风险 → 阻断
悬空 AC- → 阻断
复审未升版本 → 阻断
范围蔓延无依据 → 阻断
「以后再补」 → 阻断（除非写入待决且 Leader 同意带条件 PASS）
```
## 边界
- 只评架构设计，不评 UI 视觉、产品优先级、代码实现、测试用例细节。
- 不替代 Leader 的通用门禁（`multica-verification` skill）。
- 第 3 轮仍 FAIL → 标注「升级人类」，交 Leader 处理，停止循环。
## 为什么有效
带 REV-n ID 的清单 + 文档版本 + 评审回应表，使多轮修订可追踪；17 维覆盖数据/安全/运维等 Architect 易漏角度，减少 G2 后才发现的设计缺陷。