@archsight/aios 1.0.0 → 1.0.1

package/workflows/review.md

@@ -1,70 +1,74 @@
-# Review 流程
-
-状态：治理基线  
-适用场景：代码审查、架构审查、安全审查、AI 生成内容审查
-
----
-
-## 目标
-
-在合并或交付前发现真实风险，优先阻止缺陷、越界修改、过度设计和安全问题。
-
-## 与 `code-review` 的关系
-
-`review.md` 是综合评审入口，适用于代码、架构、安全、AI 生成内容和交付完整性的总览审查。
-
-如果任务对象是 PR、diff、commit 或 AI 生成代码，应进入 [Code Review Workflow](code-review.md)，并使用 `aios-review`。
-
-## 角色与 Skill 路由
-
-| 审查对象 | 主 Agent | Skill | 后续 Workflow |
-| --- | --- | --- | --- |
-| PR / diff / AI 生成代码 | Argus | `aios-review` | `code-review` |
-| 架构边界 / 技术选型 | Atlas | `aios-arch` | `architecture-review` |
-| 交付计划 / 测试完整性 | Mason | `aios-plan` | `feature-development` |
-| RAG / MCP / Memory / Tool | Daedalus | `aios-runtime` | `rag-pipeline` |
-| BIM / IFC / 规范语义 | Vitruvius | `aios-knowledge` | `rag-pipeline` |
-
-## 输入
-
-- diff 或 PR。
-- 需求背景。
-- 测试结果。
-- 影响范围说明。
-
-## 输出
-
-- 按严重程度排序的 review finding。
-- 必须修复项。
-- 可选改进项。
-- 测试缺口。
-- 推荐进入的后续 workflow。
-
-## 审查重点
-
-- 是否满足明确需求。
-- 是否存在无关改动。
-- 是否过度设计。
-- 是否引入安全、权限、数据泄露风险。
-- 是否缺少关键测试。
-- 是否破坏既有行为。
-
-## 验收标准
-
-- P0 / P1 问题必须修复后才能继续。
-- P2 问题必须有修复或明确接受风险。
-- P3 建议不得阻塞交付。
-- 每个发现都应能路由到明确 Agent、Skill 或 Workflow。
-
-## 回滚与恢复
-
-- 如果评审发现阻断问题，停止当前交付，回到对应 workflow 的修复阶段。
-- 如果评审对象不清，先回到 Mason 做任务拆解。
-- 如果问题属于架构边界，回到 Atlas 做架构评审。
-- 如果已进入代码修改，回滚策略以具体项目的 Git 分支、PR 和发布策略为准。
-
-## 禁止事项
-
-- 不用风格偏好阻塞交付。
-- 不把可选重构伪装成必须修复。
-- 不忽略 AI 生成代码的越界风险。
+# Review 流程
+
+状态：治理基线  
+适用场景：代码审查、架构审查、安全审查、AI 生成内容审查
+
+---
+
+## 目标
+
+在合并或交付前发现真实风险，优先阻止缺陷、越界修改、过度设计和安全问题。
+
+## AIOS 适用性
+
+本 Workflow 面向建筑行业项目的综合评审增强。普通非建筑 PR、diff、架构或安全审查优先使用宿主工具的通用 review 能力；只有项目 profile、上下文或任务事实涉及建筑行业语义、工程知识、RAG / GraphRAG、审图、证据链、人工复核或审计留痕时，才启用 `aios-*` 行业增强。
+
+## 与 `code-review` 的关系
+
+`review.md` 是综合评审入口，适用于代码、架构、安全、AI 生成内容和交付完整性的总览审查。
+
+如果任务对象是 PR、diff、commit 或 AI 生成代码，应进入 [Code Review Workflow](code-review.md)，并使用 `aios-review`。
+
+## 角色与 Skill 路由
+
+| 审查对象 | 主 Agent | Skill | 后续 Workflow |
+| --- | --- | --- | --- |
+| 建筑行业项目中的 PR / diff / AI 生成代码 | Argus | `aios-review` | `code-review` |
+| 建筑行业项目中的架构边界 / 技术选型 | Atlas | `aios-arch` | `architecture-review` |
+| 建筑行业项目中的交付计划 / 测试完整性 | Mason | `aios-plan` | `feature-development` |
+| 建筑行业项目中的 RAG / MCP / Memory / Tool | Daedalus | `aios-runtime` | `rag-pipeline` |
+| BIM / IFC / 规范语义 | Vitruvius | `aios-knowledge` | `rag-pipeline` |
+
+## 输入
+
+- diff 或 PR。
+- 需求背景。
+- 测试结果。
+- 影响范围说明。
+
+## 输出
+
+- 按严重程度排序的 review finding。
+- 必须修复项。
+- 可选改进项。
+- 测试缺口。
+- 推荐进入的后续 workflow。
+
+## 审查重点
+
+- 是否满足明确需求。
+- 是否存在无关改动。
+- 是否过度设计。
+- 是否引入安全、权限、数据泄露风险。
+- 是否缺少关键测试。
+- 是否破坏既有行为。
+
+## 验收标准
+
+- P0 / P1 问题必须修复后才能继续。
+- P2 问题必须有修复或明确接受风险。
+- P3 建议不得阻塞交付。
+- 每个发现都应能路由到明确 Agent、Skill 或 Workflow。
+
+## 回滚与恢复
+
+- 如果评审发现阻断问题，停止当前交付，回到对应 workflow 的修复阶段。
+- 如果评审对象不清，先回到 Mason 做任务拆解。
+- 如果问题属于架构边界，回到 Atlas 做架构评审。
+- 如果已进入代码修改，回滚策略以具体项目的 Git 分支、PR 和发布策略为准。
+
+## 禁止事项
+
+- 不用风格偏好阻塞交付。
+- 不把可选重构伪装成必须修复。
+- 不忽略 AI 生成代码的越界风险。