tech-review
Skill2 starsChanged 5 months agoMITView on GitHub
技术评审checklist——架构评审、技术选型评审。质量保障SKILL,可独立使用。
5 other projects use a version of this.
Adopted by
Most recently taken up by
- Daeguk-Sun/dcNesstheir own version, first seen Nov 8, 2026
- azukiazusa1/skillstheir own version, first seen Oct 13, 2026
- chenklein26-maker/Harness-Startertheir own version, first seen Sep 24, 2026
- k1ein-chen/Harness-Startertheir own version, first seen Sep 24, 2026
- azukiazusa1/my-skill-repotheir own version, first seen Apr 19, 2026
Other versions5 projects use a version of this.
Word for word · 0
None found.
Edited · 0
None found.
Wrote their own version · 5
- chenklein26-maker/Harness-Starter★ 119same name
- k1ein-chen/Harness-Starter★ 119same name
- azukiazusa1/skills★ 7same name
- Daeguk-Sun/dcNess★ 1same name
- azukiazusa1/my-skill-repo★ 0same name
How matches are found. Word for word: the exact same text. Edited: shares passages with this file (“99% same” means 99% of its five-word phrases also appear here; 10% or more counts). Their own version: similar meaning, or, weakest, the same name. We can’t tell which project had it first. Projects are counted among the 200 most-starred candidates, and mentions are links to the repository we found, up to 50 per site, so both counts are a floor.
In this repository26 other files it gives its agents.
- AGENTS.mdAGENTS.md
- code-reviewSkill
- design-reviewSkill
- main-workflowSkill
- okrSkill
- pr-faqSkill
- release-checklistSkill
- release-reviewSkill
- requirement-clarificationSkill
- requirement-reviewSkill
- retro-meetingSkill
- retro-reviewSkill
- six-pagerSkill
- software-project-governanceSkill
- stage-architectureSkill
- stage-cicdSkill
- stage-developmentSkill
- stage-infraSkill
- stage-initiationSkill
- stage-maintenanceSkill
- stage-operationsSkill
- stage-releaseSkill
- stage-researchSkill
- stage-selectionSkill
- stage-testingSkill
- test-reviewSkill
The file
Files
技术评审 Checklist
本 skill 用于架构设计阶段的技术评审,确保设计方案在提交 Gate 前经过系统化的质量检查。可独立使用。
循环角色
版本背景:本循环语义于 0.65.0 引入;本标题是稳定规范,不随版本号变更。
角色映射: 在目标 Loop 模型中,本审查对应技术深度子迭代 Middle loop 的 loop-entry-gate。0.66.1 的当前运行能力仍是 experimental scaffolding;该映射不表示持久化 Loop runtime 已激活。
当前可执行行为仅为 Coordinator M7.4 的 NEEDS_CHANGE -> 返工 -> 复审,以及 Check 30 对复审链终态、轮次连续性和熔断结果的校验。
<!-- loop-runtime-target:{"claim_id":"LRC-TECH-PLANNED-001","target_version":"0.68.0","status":"planned_not_active"} -->持久化 back-edge、flow-unit loop_count、Middle fuse、PARO transition 与自动升级属于 0.68.0 规划,当前不生效。
Reviewer 只审查并输出结论,不修改产品代码。Reviewer 必须输出 APPROVED、APPROVED_WITH_NOTES、NEEDS_CHANGE 或 BLOCKED;APPROVED_WITH_NOTES 是保留备注的通过终态,只能用于没有未解决 BLOCKING finding 的审查,不得包含未解决的 BLOCKING finding。APPROVED_WITH_NOTES 的审查输出与 REVIEW 证据 MUST 包含独立结构字段 unresolved_blockers=0;字段缺失、非零、非法或重复矛盾时不得通过,自然语言中偶然出现的 blocking 不构成该事实。NEEDS_CHANGE(及兼容输入 NEEDS_CHANGES)不是终态,Coordinator 必须在返工后发起下一轮复审。终态证据由 Check 30 的复审链消费:APPROVED 与 APPROVED_WITH_NOTES 可以通过并结束复审链;BLOCKED 结束链路但不是通过,必须 escalation;NEEDS_CHANGE(S)、未知或格式错误结论必须 fail-closed。超过复审 fuse 的 NEEDS_CHANGE 必须升级为 BLOCKED。
依据 ADR §3.5(loop-engineering-architecture-0.65.0)。完整映射见 共享循环角色映射。
触发条件
- 架构设计文档已完成,需要提交评审
- 概要设计或详细设计完成一个重要模块
- 用户要求对当前设计进行审核
输入
| 输入项 | 说明 | 是否必须 |
| 架构/设计文档 | 待评审的设计产出物 | 必须 |
| 非功能需求列表 | 性能、安全、可用性等要求 | 必须 |
| 技术选型决策记录 | 为什么选这些技术 | 推荐 |
| 已知约束 | 时间、资源、技术限制 | 推荐 |
执行步骤
第一步:结构完整性检查
| # | 检查项 | 通过标准 |
| 1 | 有明确的设计目标吗? | 目标与立项阶段的项目目标一致,且增加了技术层面的具体要求 |
| 2 | 有方案描述吗? | 包含架构风格、模块划分、数据流、接口定义 |
| 3 | 有替代方案吗? | 至少列出 1 个被否决的替代方案及否决理由 |
| 4 | 有风险评估吗? | 列出设计中的技术风险和缓解措施 |
参考 Google Design Doc:设计文档必须包含目标、方案、替代方案、风险四个核心部分。
第二步:架构质量检查
| # | 检查项 | 通过标准 | 严重级别 |
| 1 | 模块职责单一 | 每个模块职责描述 ≤ 3 句话,且不与其他模块重叠 | 阻塞 |
| 2 | 无循环依赖 | 模块间依赖关系是有向无环图 | 阻塞 |
| 3 | 接口定义完整 | 核心接口有输入输出字段、类型和校验规则 | 阻塞 |
| 4 | 数据流清晰 | 数据从输入到存储到输出的路径可端到端追溯 | 关键 |
| 5 | 错误处理覆盖 | 关键路径有明确的错误处理和降级策略 | 关键 |
| 6 | 可扩展性 | 新增功能不需要修改核心架构 | 一般 |
| 7 | 可测试性 | 每个模块可以独立测试(有明确接口) | 一般 |
严重级别说明:
- 阻塞:必须修改才能通过评审
- 关键:强烈建议修改,可在评审中标记为"有条件通过"
- 一般:建议修改,不阻塞通过
第三步:非功能需求映射检查
| # | 检查项 | 通过标准 |
| 1 | 性能需求有对应设计措施 | 每条性能需求在架构中有具体的支撑设计(缓存、异步、索引等) |
| 2 | 安全需求有对应设计措施 | 涉及敏感数据的模块有安全设计(加密、权限、审计) |
| 3 | 可用性需求有对应设计措施 | SLA 要求在架构中有具体支撑(冗余、降级、熔断等) |
| 4 | 可维护性有考量 | 架构支持独立部署、独立测试、独立变更 |
输出:非功能需求映射表
| 非功能需求 | 架构支撑措施 | 满足程度 | 遗留风险 |
| (示例)响应时间 < 200ms | Redis 缓存 + 异步处理 | 满足 | 高并发下需压测验证 |
第四步:蓝军挑战(华为蓝军机制)
参考华为蓝军机制,从反对者视角挑战设计。
单 agent 蓝军协议(结构化视角切换——agent 同时担任红军和蓝军):
蓝军执行前 MUST 执行以下视角切换序列:
Step 1: 框架切换
从 "这个设计为什么是对的" → "这个设计最可能在哪里失败"
列出设计中隐含的 3 个核心假设
Step 2: 角色扮演
依次用以下 3 个角色的视角审查设计:
(a) 攻击者视角:"如果我想让这个系统瘫痪,我会攻击哪里?"
(b) 最愤怒的用户视角:"什么会让我卸载这个产品?"
(c) 维护者视角:"接手这个系统一年后,什么会让我想重写?"
Step 3: 场景推演
对每个角色,追问 "如果 X 发生,后续 3 步是什么?"
不只问"会不会出问题"——推演"出问题之后的第一块多米诺骨牌是什么"蓝军挑战问题(≥3 个必答):
| # | 挑战问题 | 视角 | 目的 |
| 1 | 如果用户量增长 10 倍,架构会怎样? | 攻击者 | 验证可扩展性假设 |
| 2 | 如果核心依赖(数据库/第三方服务)挂了,系统会怎样? | 攻击者 | 验证容错设计 |
| 3 | 如果这个模块需要完全重写,影响范围有多大? | 维护者 | 验证模块独立性 |
| 4 | 有没有过度设计?哪些模块可以更简单? | 维护者 | 防止过度工程化 |
| 5 | 用户最可能因为什么而放弃使用? | 用户 | 验证用户体验假设 |
蓝军输出标准格式(每个挑战问题必须按此格式记录):
| 攻击向量 | 影响评估 | 当前缓解 | 残余风险 | 建议增强 |
| [具体攻击场景] | [影响范围+严重度] | [设计中已有的防护] | [缓解后的剩余风险 高/中/低] | [可以额外做什么] |
strict profile 要求:蓝军 MUST 完成完整视角切换序列 + ≥3 个挑战问题(含标准格式输出)+ 设计方的逐条回应记录。
第五步:独立评审人(Bar Raiser)— 强制
参考 Amazon Bar Raiser 机制,技术评审必须引入至少 1 位项目外部的独立评审人。
独立评审人资格要求:
- 非当前项目组成员(不在 plan-tracker 的 DRI 列表中)
- 具备对应领域知识(能理解架构决策的技术含义)
- 无项目进度压力(不因"赶时间"降低标准)
独立评审人职责:
- 独立审查设计文档,不受项目组内部意见影响
- 有权对评审结论行使否决权——如果标准不达标,可以单方面阻止 Gate 通过
- 否决必须附带具体的否决理由和解除条件(什么情况下可以撤销否决)
单 agent 场景下的最低标准:
- Agent 必须从"对立面分析框架"重新审查设计(切换视角:从"为什么这样设计是对的"→"这个设计最可能在哪里失败")
- 至少挑战 3 个核心假设(如"假设 X 服务永远可用""假设数据量不超过 Y")
- 独立评审结论必须在切换分析框架后得出——不得复用同一分析框架
第六步:评审结论
| 结论 | 条件 | 后续动作 |
| APPROVED(通过) | 无阻塞项,关键项全部解决,独立评审人无否决 | 进入下一阶段 |
| APPROVED_WITH_NOTES(有条件通过) | 无未解决 BLOCKING finding,关键项有非阻塞遗留,独立评审人无否决 | 保留备注并记录到跟踪表,下阶段关闭 |
| NEEDS_CHANGE(需要修改) | 有阻塞项,独立评审人无否决 | 修改后重新评审 |
| BLOCKED(否决) | 独立评审人行使否决权 | Gate 阻塞——必须解除否决(满足解除条件+重新评审)才能通过 Gate。否决不能因"赶时间"被绕过 |
输出
| 产出物 | 格式 |
| 评审结论 | APPROVED / APPROVED_WITH_NOTES / NEEDS_CHANGE / BLOCKED |
| 问题清单 | 每条有严重级别、描述、建议修改方案 |
| 非功能需求映射表 | 需求 → 措施 → 满足程度 |
| 遗留项列表(如有) | 每条有责任人和关闭截止日期 |
| 否决记录(如有) | 否决理由 + 解除条件 + 独立评审人 + 重评日期 |
质量标准
- 结构完整性 4 项检查全部通过
- 无阻塞级别的架构质量问题
- 非功能需求映射表完整(每条需求有对应措施)
- 独立评审人已指定且满足资格要求(非作者、非同一 agent、具备领域知识)
- 评审结论有明确理由;如有否决,否决理由和解除条件已记录
独立使用说明
本 skill 可脱离完整生命周期独立使用。
如果项目已初始化 governance(.governance/ 存在)
- 先执行目标锚定检查(读 plan-tracker → 确认成功标准 → 偏离检查)
- 不需要先通过任何 Gate
- 可用于任何设计文档的评审(不限于架构设计阶段)
- 如果后续接入完整工作流,本 skill 的产出自动成为 G5(设计完成)的证据
如果项目未初始化 governance(.governance/ 不存在)
- 跳过所有治理前置检查
- 直接执行本 skill 的核心流程(设计文档审查 + 蓝军挑战 + 评审结论)
- 执行完毕后提醒用户:"当前项目未初始化 governance。如需自动证据记录、Gate 检查、风险跟踪,运行
/governance-init。"
子工作流映射
| 本 skill 步骤 | 对应子工作流活动 | 映射说明 |
| 结构完整性检查 | 架构设计→概要设计 | 检查概要设计产出物的完整性 |
| 架构质量检查 | 架构设计→系统架构设计 | 逐项验证模块划分和接口定义质量 |
| 非功能需求映射 | 架构设计→需求到设计的映射 | 验证每个非功能需求有架构层面的支撑 |
| 蓝军挑战 | 架构设计→技术评审 | 华为蓝军机制的标准化执行方式 |
| 评审结论 | 架构设计→技术评审 | 直接输出为 Gate G5 要求的评审结论证据 |
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

