@andyqiu/codeforge 0.5.1 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andyqiu/codeforge",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "CodeForge — opencode 的零侵入扩展包",
   "type": "module",
   "private": false,

package/workflows/bugfix.yaml

@@ -12,10 +12,10 @@ description: |
   systematic debugging：根因驱动而非症状修补。
     1. reviewer 取证（smart_search 历史、读 log / git blame）
     2. planner 出根因假设 + 修复方案
-    3. coder 先写「复现」测试 stage（红）
-    4. coder 改代码 stage（绿）
-    5. reviewer 审 + 跑全量测试
-    6. coder apply_all（必审批）
+    3. coder 先在 session worktree 写「复现」测试（红）
+    4. coder 在 session worktree 改代码（绿）
+    5. reviewer 审 worktree diff + 跑全量测试
+    6. 用户跑 /merge 触发 review-fix-review 闭环合并到主仓
     7. reviewer 沉淀 bugfix 经验回 KH
 
 trigger: /debug
@@ -49,18 +49,18 @@ steps:
 
   - name: 复现
     agent: coder
-    description: 写一个能稳定复现 bug 的最小测试用例（应该红）
+    description: 在 session worktree 内写一个能稳定复现 bug 的最小测试用例（应该红）
     inject_context:
-      forbid_direct_edit: true
-      hint: 用 ast-edit / pending-changes，不要直接改文件
+      forbid_main_repo_write: true
+      hint: 直接 edit / write / ast_edit 到 session worktree（worktree-guard 隔离主仓），不要绕过
     on_error: retry
     max_retries: 1
 
   - name: 修复
     agent: coder
-    description: 按 planner 选定的根因假设改代码，让复现测试变绿
+    description: 在 session worktree 内按 planner 选定的根因假设改代码，让复现测试变绿
     inject_context:
-      forbid_direct_edit: true
+      forbid_main_repo_write: true
     # ADR:workflow-auto-feedback-loop: 完成 stage 后自动跑 npm test 自纠
     # bugfix 失败本质是假设错——escalate 给 planner 重出根因假设，而不是 reviewer
     auto_feedback:
@@ -73,12 +73,13 @@ steps:
 
   - name: 审阅
     agent: reviewer
-    description: 看全部 pending + 跑全量测试；输出 APPROVE / REQUEST_CHANGES / BLOCK
-    actions:
-      - tool: pending-changes
-        args:
-          action: list
-          status: pending
+    description: 审 worktree diff + 跑全量测试；输出 APPROVE / REQUEST_CHANGES / BLOCK
+    inject_context:
+      review_target: code
+      # ↓ workflow-engine 必须把 [Session Merge Review] 标记 + sessionId / worktreePath / baseSha / plan_id 注入到 reviewer prompt 头部
+      # 这是 reviewer 模式 3（worktree-session 审阅）的判定条件（agents/reviewer.md「模式判定」表）
+      # 若 workflow-engine 暂未实现该注入：fallback 走 reviewer 模式 4（legacy），但代价是不会跑 git diff 对比 baseSha
+    # actions 留空：reviewer 模式 3 自己跑 `bash cd <worktreePath> && git diff <baseSha>..HEAD`
     # ADR:workflow-on-decision-branching: 三档决策驱动；BLOCK 回 planner 重出根因假设（与 auto_feedback escalate_to=planner 同语义）
     on_decision:
       APPROVE: continue           # 进入"落地" step
@@ -91,14 +92,10 @@ steps:
     on_error: abort
 
   - name: 落地
-    agent: coder
-    description: 用户审批通过后把 fix + 复现测试一起 apply
+    # 由用户拍板：跑 `/merge` 触发 review-fix-review 闭环 squash merge 到主仓
+    agent: codeforge
+    description: 提示用户跑 `/merge` 合并 session worktree 到主仓（ADR:worktree-session-isolation Phase 5）
     requires_human_approval: true
-    actions:
-      - tool: pending-changes
-        args:
-          action: apply_all
-        on_error: abort
     on_error: abort
 
   - name: 沉淀

package/workflows/code-review.yaml

@@ -39,12 +39,13 @@ steps:
   - name: 审阅
     agent: reviewer
     description: 按维度清单逐项审，跑测试，输出 APPROVE/REQUEST_CHANGES/BLOCK
-    actions:
-      - tool: pending-changes
-        args:
-          action: list
-          status: pending
-        on_error: skip
+    inject_context:
+      review_target: code
+      # ↓ workflow-engine 必须把 [Session Merge Review] 标记 + sessionId / worktreePath / baseSha 注入到 reviewer prompt 头部
+      # 这是 reviewer 模式 3（worktree-session 审阅）的判定条件（agents/reviewer.md「模式判定」表）
+      # 注意：本 workflow 只读，没有 plan_id（不强制 plan_read 对照）
+      # 若 workflow-engine 暂未实现该注入：fallback 走 reviewer 模式 4（legacy），但代价是不会跑 git diff 对比 baseSha
+    # actions 留空：reviewer 模式 3 自己跑 `bash cd <worktreePath> && git diff <baseSha>..HEAD`（本 workflow 只读，不写 worktree、不触发 /merge）
     on_decision:
       APPROVE: continue
       REQUEST_CHANGES: continue   # 仍 continue 进入沉淀，REQUEST_CHANGES 是给提交者看的

package/workflows/feature-dev.yaml

@@ -18,9 +18,9 @@ description: |
   完整的"从需求到落地"流程。串联三个核心 agent：
     1. planner 出方案（自动 smart_search + repo-map）
     2. reviewer 审方案（plan_only，REQUEST_CHANGES 回 planner 重写）
-    3. coder 按方案 stage 改动到 .codeforge/pending/
-    4. reviewer 审代码 + 跑测试
-    5. 用户审批 → apply 落地工作区 → 自动沉淀经验回 KH
+    3. coder 按方案直接写入 session worktree（session-worktree-guard 隔离主仓）
+    4. reviewer 审 worktree diff + 跑测试
+    5. 用户跑 /merge 触发 review-fix-review 闭环合并到主仓 → 自动沉淀经验回 KH
 
 trigger: /ship
 
@@ -52,12 +52,9 @@ steps:
         - 设计合理性
         - 风险评估
         - 涉及文件清单
-    actions:
-      - tool: pending-changes
-        args:
-          action: list
-          status: pending
-        on_error: skip
+      # ↓ plan_id 由 workflow-engine 从上一步 planner 的 boomerang 摘要中解析后注入（reviewer 模式 1 调 plan_read(plan_id=<id>) 必需）
+      # 若 workflow-engine 当前未实现自动解析：planner 必须把 `plan_id: plan-YYYYMMDD-HHmmss-NNN` 独占一行写到 boomerang，workflow-engine 透传到下一步 inject_context.plan_id
+    # actions 留空：reviewer 自己 plan_read(plan_id=<上方注入>) 拿方案（review_target=plan_only）
     on_decision:
       APPROVE: continue
       REQUEST_CHANGES:
@@ -68,9 +65,9 @@ steps:
 
   - name: 执行
     agent: coder
-    description: 严格按方案 stage 改动；测试失败立刻停下汇报
+    description: 严格按方案在 session worktree 内直接写文件（edit / write / ast_edit）；测试失败立刻停下汇报
     inject_context:
-      forbid_direct_edit: true
+      forbid_main_repo_write: true
     # ADR:workflow-auto-feedback-loop: 完成 stage 后自动跑 npm test 自纠（最多 3 轮，失败 escalate reviewer）
     auto_feedback:
       test_cmd: "npm test"
@@ -78,18 +75,19 @@ steps:
       error_excerpt_lines: 5
       escalate_to: reviewer
     # actions 故意为空 —— 由 coder agent 自主决定调哪些工具
-    # （pending-changes / nav-find / bash 等都允许，但 risky 动作会自动审批）
+    # （edit / write / ast_edit / nav-find / bash 等都允许；主仓写操作被 session-worktree-guard 拦截）
     on_error: retry
     max_retries: 1
 
   - name: 审阅
     agent: reviewer
-    description: 只读审阅 + 跑测试，输出 APPROVE/REQUEST_CHANGES/BLOCK
-    actions:
-      - tool: pending-changes
-        args:
-          action: list
-          status: pending
+    description: 只读审阅 worktree diff + 跑测试，输出 APPROVE/REQUEST_CHANGES/BLOCK
+    inject_context:
+      review_target: code
+      # ↓ workflow-engine 必须把 [Session Merge Review] 标记 + sessionId / worktreePath / baseSha / plan_id 注入到 reviewer prompt 头部
+      # 这是 reviewer 模式 3（worktree-session 审阅）的判定条件（agents/reviewer.md「模式判定」表）
+      # 若 workflow-engine 暂未实现该注入：fallback 走 reviewer 模式 4（legacy），但代价是不会跑 git diff 对比 baseSha
+    # actions 留空：reviewer 模式 3 自己跑 `bash cd <worktreePath> && git diff <baseSha>..HEAD`
     # ADR:workflow-on-decision-branching: 三档决策驱动 workflow 跳转
     on_decision:
       APPROVE: continue           # 进入"落地" step
@@ -100,14 +98,11 @@ steps:
     on_error: abort
 
   - name: 落地
-    agent: coder
-    description: 用户审批通过后，把全部 pending 应用到工作区
+    # 由用户拍板：跑 `/merge` 触发 review-fix-review 闭环 squash merge session worktree 到主仓
+    # 本步骤 workflow 不调任何 apply 工具；仅提示用户执行
+    agent: codeforge
+    description: 提示用户跑 `/merge` 合并 session worktree 到主仓（ADR:worktree-session-isolation Phase 5）
     requires_human_approval: true
-    actions:
-      - tool: pending-changes
-        args:
-          action: apply_all
-        on_error: abort
     on_error: abort
 
   - name: 沉淀

package/workflows/refactor.yaml

@@ -67,11 +67,12 @@ steps:
   - name: 审阅
     agent: reviewer
     description: 确认行为不变（tests 绿）+ 代码更清晰
-    actions:
-      - tool: pending-changes
-        args:
-          action: list
-          status: pending
+    inject_context:
+      review_target: code
+      # ↓ workflow-engine 必须把 [Session Merge Review] 标记 + sessionId / worktreePath / baseSha / plan_id 注入到 reviewer prompt 头部
+      # 这是 reviewer 模式 3（worktree-session 审阅）的判定条件（agents/reviewer.md「模式判定」表）
+      # 若 workflow-engine 暂未实现该注入：fallback 走 reviewer 模式 4（legacy），但代价是不会跑 git diff 对比 baseSha
+    # actions 留空：reviewer 模式 3 自己跑 `bash cd <worktreePath> && git diff <baseSha>..HEAD`
     on_decision:
       APPROVE: continue
       REQUEST_CHANGES:
@@ -81,14 +82,10 @@ steps:
     on_error: abort
 
   - name: 落地
-    agent: coder
-    description: 用户审批通过后 apply 全部 pending
+    # 由用户拍板：跑 `/merge` 触发 review-fix-review 闭环 squash merge 到主仓
+    agent: codeforge
+    description: 提示用户跑 `/merge` 合并 session worktree 到主仓（ADR:worktree-session-isolation Phase 5）
     requires_human_approval: true
-    actions:
-      - tool: pending-changes
-        args:
-          action: apply_all
-        on_error: abort
     on_error: abort
 
   - name: 沉淀

package/workflows/tdd.yaml

@@ -74,11 +74,12 @@ steps:
   - name: 审阅
     agent: reviewer
     description: 确认 RED→GREEN→REFACTOR 三步全做，测试覆盖核心
-    actions:
-      - tool: pending-changes
-        args:
-          action: list
-          status: pending
+    inject_context:
+      review_target: code
+      # ↓ workflow-engine 必须把 [Session Merge Review] 标记 + sessionId / worktreePath / baseSha / plan_id 注入到 reviewer prompt 头部
+      # 这是 reviewer 模式 3（worktree-session 审阅）的判定条件（agents/reviewer.md「模式判定」表）
+      # 若 workflow-engine 暂未实现该注入：fallback 走 reviewer 模式 4（legacy），但代价是不会跑 git diff 对比 baseSha
+    # actions 留空：reviewer 模式 3 自己跑 `bash cd <worktreePath> && git diff <baseSha>..HEAD`
     on_decision:
       APPROVE: continue
       REQUEST_CHANGES:
@@ -88,12 +89,8 @@ steps:
     on_error: abort
 
   - name: 落地
-    agent: coder
-    description: 用户审批通过后 apply 全部 pending
+    # 由用户拍板：跑 `/merge` 触发 review-fix-review 闭环 squash merge 到主仓
+    agent: codeforge
+    description: 提示用户跑 `/merge` 合并 session worktree 到主仓（ADR:worktree-session-isolation Phase 5）
     requires_human_approval: true
-    actions:
-      - tool: pending-changes
-        args:
-          action: apply_all
-        on_error: abort
     on_error: abort