@jack200714/mafw 4.5.1 → 4.8.0

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

@@ -1,85 +0,0 @@
-# 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？**