Arch Review

架构评审技能。对技术方案、系统设计、重构计划进行结构化评审,输出评审意见和修改建议。适用于架构师评审后端/前端/DevOps 提交的技术方案,CTO 对重大架构决策的把关,以及团队内部技术方案 Review。

rowankid 5b357c9 1.9 KB Updated

File contents

Arch Review Skill — 架构评审技能

评审维度与检查项

1. 可扩展性

  • 是否支持水平扩展(加机器能解决性能问题)?
  • 服务间依赖是否解耦(避免强依赖单点)?
  • 数据库设计是否考虑分库分表预留?

2. 可维护性

  • 模块划分是否清晰,职责单一?
  • 复杂度是否在团队能力范围内?
  • 是否有完整的文档说明?

3. 安全性

  • 认证授权机制是否完善?
  • 敏感数据是否加密存储/传输?
  • 是否有 SQL 注入/XSS 等安全防护?
  • 权限控制是否遵循最小权限原则?

4. 性能

  • 关键路径的时间复杂度是否合理?
  • 是否有潜在的 N+1 查询问题?
  • 缓存策略是否合理?
  • 是否评估了峰值流量下的表现?

5. 可观测性

  • 关键操作是否有日志?
  • 是否有监控指标(QPS/错误率/延迟)?
  • 是否有告警配置?

6. 成本

  • 资源消耗估算是否合理?
  • 是否有不必要的过度设计?

输出模板

# 架构评审报告:[方案名称]

日期: YYYY-MM-DD | 评审人: 架构师 | 结论: [通过 / 有条件通过 / 打回]

## 评审结论

**整体评价**:[1-3句话总结]

## 各维度评审意见

### ✅ 做得好的地方
-

### ⚠️ 需要修改(通过条件)
| 问题 | 严重程度 | 修改建议 |
|------|---------|---------|
| | 高/中/低 | |

### 💡 建议优化(非必须)
-

## 待确认事项
- [ ] [问题] — 确认人:[角色]

rowankid/ai-team/tree/main/skills/arch-review commit 5b357c94f6

Frequently asked questions

npx skillmds@latest add rowankid/arch-review