@modus-ai/modus 0.2.5 → 0.2.7

package/templates/skills/modus-agents/reviewer/SKILL.md

@@ -8,8 +8,8 @@ disable: false
 # modus-reviewer（代码评审 SubAgent）
 
 **调用方：** Harness Orchestrator（Gate B 通过后触发）  
-**输入：** 全部产出物（01-analysis.md 至 05-security-report.md）+ 代码变更  
-**产出物：** `modus/plans/active/{story-id}/cr-report.md`
+**输入：** 全部产出物（01-analysis.md 至 06-security-report.md）+ 代码变更  
+**产出物：** `modus/stories/{story-id}/harness/cr-report.md`
 
 ## 职责
 
@@ -56,15 +56,26 @@ disable: false
 - 金额字段是否符合项目宪法的规范（以 `constitution.hard_rules` 为准，如规则为「金额字段使用 Long（单位：分）」则检查 Long 类型，如规则为「使用 BigDecimal」则检查 BigDecimal；**不得硬编码检查某一具体类型**）
 - 若 `constitution.hard_rules` 未明确金额规范，则默认检查是否使用 BigDecimal，并在报告中标注「未找到金额规范配置，使用默认规则」
 
-### 维度 3：测试覆盖（参考 03-test-report.md）
+### 维度 3：测试覆盖（参考 04-test-report.md）
 
 - 核心业务逻辑是否有测试
 - 测试是否覆盖了重要异常路径
 
-### 维度 4：性能与安全（参考 04/05 报告）
+### 维度 4：性能与安全（参考 05/06 报告）
 
-- 04 报告中的高风险问题是否已在代码中修复
-- 05 报告中的严重/高风险问题是否已修复
+**分级映射规则（将 05/06 报告分级转换为 CR 分级）：**
+
+| 来源报告 | 原始级别 | CR 级别 | 处理 |
+|---------|---------|--------|------|
+| 06-security-report | 严重 | P1 | 必须修复，触发 Loop 2 重入 |
+| 06-security-report | 高风险 | P2 | 必须修复，触发 Loop 2 重入 |
+| 06-security-report | 低风险 | P3 | 建议修复，不阻塞 |
+| 05-perf-report | 高风险 | P2（若影响 SLA 则升为 P1）| 必须修复 |
+| 05-perf-report | 中风险 | P3 | 建议修复，不阻塞 |
+| 05-perf-report | 低风险 | P3 | 建议修复，不阻塞 |
+
+- 06（安全）报告中「严重」和「高风险」问题是否已在代码中修复
+- 05（性能）报告中「高风险」问题是否已修复
 
 ---
 
@@ -81,22 +92,33 @@ disable: false
 ## 产出物格式（cr-report.md）
 
 ```markdown
-<!--HANDOFF
-agent: "06-review"
+---
+schema_version: "1.1"
+agent: "07-review"
+run_id: "{YYYYMMDD}-{story_id_prefix}"
 story_id: "{story-id}"
+task_id: ""
+domains: ["{domain1}", "{domain2}"]
 gate_status: "{passed|failed}"
+artifact_status: "complete"
 issues:
-  - id: "P1-01"
-    level: "P1"
-    sprint: {受影响的 Sprint 编号，如 2}
-    file: "{文件名}.java:{行号}"
+  - level: "P1"
+    code: "P1-01"
+    agent: "07-review"
+    location: "{文件名}.java:{行号}"
     summary: "{一句话问题摘要，供 Orchestrator 定位重入范围}"
-  - id: "P2-01"
-    level: "P2"
-    sprint: {N}
-    file: "{文件名}.java:{行号}"
+    suggestion: "{修复建议}"
+    affected_sprint: "{受影响的 Sprint 编号，如 Sprint 2}"
+  - level: "P2"
+    code: "P2-01"
+    agent: "07-review"
+    location: "{文件名}.java:{行号}"
     summary: "{一句话问题摘要}"
--->
+    suggestion: "{修复建议}"
+    affected_sprint: "{N}"
+skill_refs:
+  - "modus-biz-{domain}@{version}"
+---
 
 # 代码评审报告
 
@@ -133,21 +155,26 @@ issues:
 {如无 P1/P2 → "通过评审，可进入 Final Review"}
 ```
 
-**HANDOFF 块填写规则：**
+**HANDOFF 块填写规则（YAML frontmatter 格式，v1.1）：**
 - `gate_status`：无 P1/P2 → `passed`；有 P1/P2 → `failed`
 - `issues`：仅列 P1/P2 问题（P3 不阻塞，Orchestrator 不读取）；无 P1/P2 时写空列表 `issues: []`
-- 每条 issue 的 `sprint` 字段：填写受影响代码所属的 Sprint 编号，供 Orchestrator 精准定位重入范围
-- HANDOFF 块**必须位于文件最顶部**，Orchestrator 仅读此块（≤ 20 行）以决策是否触发 Loop 2
+- 每条 issue 的 `affected_sprint` 字段：填写受影响代码所属的 Sprint 编号（如「Sprint 2」），供 Orchestrator 精准定位重入范围
+- 每条 issue 的 `code` 字段：填写问题编码（如 `P1-01`、`P2-03`），用于精准重入定位
+- HANDOFF 块**必须位于文件最顶部（`---...---` YAML frontmatter）**，Orchestrator 仅读此块（≤ 25 行）以决策是否触发 Loop 2
 
 ---
 
 ## 创建 Bug 单（可选）
 
-如果项目配置了 `tapdProjectId`（来自 `modus/config.yaml`），对于 P1/P2 问题可以选择自动创建 TAPD Bug 单：
+> **前置条件：** 需在 `{project}/modus/config.yaml` 中配置 `tapdProjectId`（非空）才能触发 Bug 创建。当前 `tapdProjectId: ""` 时跳过此步骤。
+
+> **协调说明：** 如果 workspace 中已安装独立的 `tapd-bug-creator` Skill，**优先使用外部 Skill** 创建 Bug 单（职责更单一），本节逻辑作为降级兜底（外部 Skill 不可用时才启用）。两者不应同时执行。
+
+如果满足前置条件，对于 P1/P2 问题自动创建 TAPD Bug 单：
 
 - Bug 标题：`[Modus CR] {问题标题}`
 - 关联 Story ID：`{当前 story-id}`
-- 描述：包含代码位置、问题代码片段、修复方案
+- 描述：包含代码位置（`location` 字段）、问题代码片段、修复方案（`suggestion` 字段）
 - 严重程度：P1 → 严重，P2 → 一般
 
-创建完成后将 Bug ID 写入 cr-report.md 末尾。
+创建完成后将 Bug ID 写入 cr-report.md 末尾的 `tapd_bugs` 列表。