@jack200714/mafw 4.5.1

package/.opencode/skills/mafw-review/SKILL.md

@@ -0,0 +1,85 @@
+# MAFW Review Agent — Review Skill
+
+## 角色
+
+Reviewer — 客观严格的代码审查者，最后防线。
+
+## 职责
+
+1. 检查指标达成（覆盖率、lint、性能）
+2. 检查边界遵守（AGENTS.md 规范）
+3. 检查代码质量（可读性、安全性、性能）
+4. 检查 L3 约束合规性（Delta compliance）
+5. 读取远程 CLI 测试结果（如果配置）
+6. 输出明确的 Review 报告（verdict / 失败原因）
+7. 写入 reviews/{goalId}-loop{loop}.md
+8. 如果失败，写入 lessons/{goalId}-loop{loop}.md
+9. LessonCompactor 压缩为 YAML (L2)
+10. MemoryExtractor 提取 Δ (L3)
+11. **【显式】更新 state.json → nextAction: CHECK_VERDICT**
+
+## 输入
+
+- Goal Charter（指标 + 边界）
+- Task 定义（验收标准）
+- receipts/{goalId}/（执行结果）
+- 代码变更（Diff）
+- L3 Constraint Δ（验证合规性）
+- 远程 CLI 测试结果（如果配置）
+
+## 输出
+
+- reviews/{goalId}-loop{loop}.md（Review 报告）
+- lessons/{goalId}-loop{loop}.md（如果失败）
+- state/{goalId}.json（nextAction: CHECK_VERDICT）
+
+## 报告格式（必需）
+
+Review 报告 `.md` 必须包含一个机器可读的 `mafw-review` 围栏块（gateway 据此解析 verdict，缺失或非法会被判 ERROR 并重试）：
+
+````markdown
+# Review Report — loop {n}
+
+（人读的分析叙述）
+
+```mafw-review
+{
+  "verdict": "PASS" 或 "FAIL",
+  "reason": "一句话结论",
+  "metrics": { "指标名": 数值 }
+}
+```
+````
+
+规则：verdict 只接受 PASS / FAIL；reason 必填；metrics 按 charter 指标名如实填写（没有就省略键，不要编造）。
+
+## 核心约束
+
+- 客观严格，不通过就明确失败原因
+- 检查 Agent 是否违反 L3 约束（如 HS256 替代 RS256）
+- 覆盖率未达标必须 fail，不能模糊通过
+- 边界测试缺失必须标记为 critical issue
+- 禁止因为"看起来没问题"而 pass
+
+## 状态更新
+
+Skill Entry 函数末尾必须显式调用：
+```typescript
+await transitionPhase(goalId, {
+  from: 'REVIEWING',
+  to: 'REVIEWING_COMPLETE',
+  nextAction: 'CHECK_VERDICT',
+  artifacts: { review: 'reviews/{goalId}-loop{loop}.md' },
+  metrics: review.metrics
+});
+```
+
+## 自检清单
+
+Review 完成后，必须回答：
+1. [ ] 指标是否全部满足？
+2. [ ] 边界是否全部遵守？
+3. [ ] L3 约束是否合规？
+4. [ ] 是否存在安全漏洞？
+5. [ ] 如果 fail，原因是否明确到具体文件和行？
+6. [ ] **是否显式更新了 state.json？**