@andyqiu/codeforge 0.5.1 → 0.5.2

package/agents/discover-challenger.md

@@ -16,7 +16,7 @@ allowed_tools:
   - smart_search
   # 故意不给 save_chat_insight：challenger 是过程角色，不沉淀
   # 故意不给 task：物理禁止派子 agent（避免嵌套对抗）
-  # 故意不给 pending_changes：零写权
+  # 故意不给 edit/write：零写权（worktree 也不允许写）
 model: openai/gpt-5.5
 model_category: ultrabrain
 tier: deep
@@ -60,7 +60,7 @@ fallback_models:
   - "方向是对的，不过..."
 - ❌ 禁止只夸不挑 —— 如果真的全对，必须明示"我没找到问题"并列出**已检查的维度清单**（让 discover 看到你确实查过 7 项）
 - ❌ 禁止派子 agent（工具未给 `task`，物理禁止）
-- ❌ 禁止 stage / apply 任何文件（工具未给 `pending_changes`）
+- ❌ 禁止写任何文件（工具未给 `edit` / `write`）
 - ❌ 禁止 `save_chat_insight`（你是过程角色，沉淀由 discover 做）
 - ❌ 报告里**不允许**出现「伪需求」三字（包括 红旗 reasons 字段）
 

package/agents/discover.md

@@ -10,14 +10,14 @@ permission:
   webfetch: deny
 # CodeForge 自描述字段（phase1:check 校验复数 + 列表形态）
 permissions:
-  edit: deny
   bash: deny
   webfetch: deny
 allowed_tools:
   - read
+  - write
+  - edit
   - smart_search
   - save_chat_insight
-  - pending_changes
   - task
   - skill
 model: anthropic/claude-opus-4-7
@@ -71,14 +71,15 @@ fallback_models:
   - 数值范围：`当前` ∈ [0.0, 1.0]，`加权` = `当前 × 权重`（保留 2 位小数）。**违反格式 = 违反硬约束**。
   - **评分表前必须先输出 1 行 Phase 进度提示**（C5 dogfooding 用户反馈：全程看不到红旗 YES/NO，等价于看不到自己处于哪个 Phase）。模板见附录 D。
 - 进入 Phase E 出 PRD 草稿前**强制展示一次**评分表（让用户拍板）
-- 所有产物必须通过 `pending_changes.stage` 暂存到 `.codeforge/specs/<slug>/`，**不允许** apply
+- 所有产物（PRD.md / handoff.yaml）**直接 `write` 到 session worktree** 的 `.codeforge/specs/<slug>/` 下；session-worktree-guard 隔离主仓，最终由用户 `/merge` 拍板合并
 - 开始新对话时**必须** `smart_search` 三轮并发（业务领域 / 团队历史类似需求 / 反模式清单）
 - 每个 Phase 结束时**必须** `save_chat_insight` 沉淀阶段结论
 
 ### MUST NOT
 
 - ❌ 永不向用户提及以下术语（即便用户先说）：`JTBD` / `Jobs-to-be-Done` / `EARS` / `Pre-mortem` / `Socratic Self-Refine` / `Affinity clustering` / `Divergent phase` / `Convergent phase` / `5 维加权打分` / `Sycophancy` / `Devil's Advocate` / `Example Mapping` / `ambiguity-gate` / **`伪需求`**（红旗用「关键前提未证实」表达，绝不说这三个字）
-- ❌ 不允许出现"按 X 框架"/"用 Y 方法论"/"做一次 Pre-mortem"等领字句式；不允许跳过 pending-changes 直接写文件；不允许调用 `task()` 派出 discover-challenger 以外的任何 agent；不允许 apply / apply_all（用户拍板）
+- ❌ 不允许出现"按 X 框架"/"用 Y 方法论"/"做一次 Pre-mortem"等领字句式；不允许写到 worktree 之外（session-worktree-guard 会拒）；不允许调用 `task()` 派出 discover-challenger 以外的任何 agent；不允许擅自跳过用户 `/merge` 直接动主仓
+- ❌ **不允许 `edit` / `write` 除 `.codeforge/specs/<slug>/PRD.md` 和 `.codeforge/specs/<slug>/handoff.yaml` 之外的任何路径**；禁止修改源码、配置和工作流文件（discover 的产出仅限 spec 两份产物）
 - ❌ 用户说「别给我看分了 / 静默模式 / 不要表格」后，**永久**不再输出评分表（连 Phase E 也不展示）
 - ❌ 输出文档里**禁止**出现「伪需求」三字；允许的替代措辞：「带红旗交付」/「风险标记」/「关键前提未证实」/「多轮反对未回应」
 
@@ -163,7 +164,7 @@ fallback_models:
    - **不允许**把所有 high-risk-unknown 全塞为 blocker —— 只挑「不解除会导致 coder 实施时翻车或返工」的真阻断点；细节追踪用 `open_issues` 即可
    - 老 v1.1 spec 缺该字段：下游 coder fallback 从 assumptions+open_issues+red_flags 推断；新生成 spec 必走 v1.2.0
 7. **禁止手填 `created_at` 和 `discover_session_id`**：这两个字段由工具运行时注入，留空即可（schema 已 optional）
-8. `pending_changes.stage` 两份产物到 `.codeforge/specs/<slug>/PRD.md` 和 `handoff.yaml`，列 pending id 给用户审批
+8. 用 `write` 工具把两份产物落到 `.codeforge/specs/<slug>/PRD.md` 和 `handoff.yaml`（worktree 内直写，session-worktree-guard 隔离主仓），告诉用户已写入 session worktree，由用户后续 `/merge` 拍板
 
 **人话外壳模板**（递交时）：
 
@@ -403,4 +404,4 @@ related_artifacts: { prd: "PRD.md", transcript: "transcript.md" }
 
 - 用户中途说"算了不做了"：保留 transcript，`save_chat_insight` 沉淀已澄清部分，结束
 - challenger 召唤失败（task 超时 / 模型不可用）：fallback 到 self-critique 模板，并明示用户"对抗这一环降级了"
-- pending_changes.stage 失败：汇报具体错误，**不要直接写文件绕过**
+- `write` 工具失败（例如 worktree-guard 拒绝路径）：汇报具体错误，**不要绕过 worktree 写到主仓**

package/bin/codeforge.mjs

@@ -285,6 +285,52 @@ function cmdRollback(args) {
   return 0
 }
 
+// ────────────────────────────────────────────────────────────────────
+// 子命令：upgrade —— 升级到 npm latest 并重新 install --global
+// ────────────────────────────────────────────────────────────────────
+function cmdUpgrade(args) {
+  const dryRun = !!args.flags["dry-run"] || !!args.flags.dryRun
+  const currentVersion = getVersion()
+
+  log(`CodeForge upgrade`)
+  log(`  当前版本 : ${currentVersion}`)
+  log(`  dry-run  : ${dryRun}`)
+  hr()
+
+  const npmCmd = "npm"
+  const npmArgs = ["install", "-g", "@andyqiu/codeforge@latest"]
+  log(`步骤 1/2：${npmCmd} ${npmArgs.join(" ")}`)
+
+  if (!dryRun) {
+    const r = spawnSync(npmCmd, npmArgs, { stdio: "inherit", shell: true })
+    if (r.status !== 0) {
+      err(`npm install 失败 (exit=${r.status ?? 1})`)
+      err(`提示：可手动跑 npm install -g @andyqiu/codeforge@latest`)
+      return 1
+    }
+  } else {
+    log(`[dry-run] 跳过 npm install`)
+  }
+
+  hr()
+  log(`步骤 2/2：codeforge install --global`)
+
+  if (!dryRun) {
+    const code = installOpencode({ scope: "global", dryRun: false, extraArgs: [] })
+    if (code !== 0) {
+      err(`install --global 失败 (exit=${code})`)
+      return 1
+    }
+  } else {
+    log(`[dry-run] 跳过 install --global`)
+  }
+
+  hr()
+  ok(`升级完成！运行 codeforge -v 查看新版本`)
+  ok(`重启 opencode 后新版本生效`)
+  return 0
+}
+
 // ────────────────────────────────────────────────────────────────────
 // 子命令：runtime —— 转发到 codeforge-runtime-info.mjs
 // ────────────────────────────────────────────────────────────────────
@@ -390,6 +436,7 @@ function cmdHelp() {
   codeforge list
   codeforge version
   codeforge rollback   [--target=<path>] [--dry-run]   # 恢复最近 backup（auto_install 失败救场）
+  codeforge upgrade           [--dry-run]   # 升级到 npm latest 并重新 install --global
   codeforge runtime where [<path>]   # 打印当前项目的全局运行时目录
   codeforge adr-init   [--force] [--dry-run] [--write-prepare] [--no-pre-push]
                                      # 把 ADR 校验体系（hooks + scripts + 模板）下发到当前 git 项目
@@ -453,6 +500,8 @@ async function main() {
       return 0
     case "rollback":
       return cmdRollback(args)
+    case "upgrade":
+      return cmdUpgrade(args)
     case "runtime":
       return cmdRuntime(args)
     case "adr-init":

package/commands/{diff.md → changes.md}

@@ -4,7 +4,7 @@ description: 查看当前 session worktree 的改动（文件列表+统计，或
 
 <!--
 codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）：
-  name: diff
+  name: changes
   version: 1.0.0
   requires_human_approval: false
   说明：底层调用 session_merge 工具的 action=diff
@@ -14,9 +14,9 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 -->
 
 
-# /diff — 查看当前 session worktree 改动
+# /changes — 查看当前 session worktree 改动
 
-**ADR: worktree-session-isolation Phase 3**
+**ADR: worktree-session-isolation Phase 3** <!-- ADR:worktree-session-isolation -->
 
 快速查看当前 session worktree 中的代码改动，无需触发 merge 或 review 流程。
 
@@ -27,23 +27,23 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 ## 用法
 
 ```
-/diff                     # 显示改动文件列表 + 统计（git diff --stat 风格）
-/diff --full              # 显示完整 diff 内容
+/changes                     # 显示改动文件列表 + 统计（git diff --stat 风格）
+/changes --full              # 显示完整 diff 内容
 ```
 
 ## 行为说明
 
 | 模式 | 触发条件 | 输出内容 |
 |------|----------|----------|
-| **stat 模式**（默认） | `/diff` | 改动文件列表 + 增删行数统计 |
-| **full 模式** | `/diff --full` | 完整 diff 内容（含每行改动） |
+| **stat 模式**（默认） | `/changes` | 改动文件列表 + 增删行数统计 |
+| **full 模式** | `/changes --full` | 完整 diff 内容（含每行改动） |
 | **无 worktree** | session 无写操作 | 友好提示：当前 session 没有写操作，无 worktree 改动 |
 
 ## 底层调用
 
 ```
-/diff         → session_merge(action="diff", stat=true)
-/diff --full  → session_merge(action="diff", stat=false)
+/changes         → session_merge(action="diff", stat=true)
+/changes --full  → session_merge(action="diff", stat=false)
 ```
 
 ## 何时使用
@@ -56,7 +56,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 
 | 命令 | 用途 |
 |------|------|
-| `/diff` | 本命令：只读查看 worktree 改动，不触发任何合并 |
+| `/changes` | 本命令：只读查看 worktree 改动，不触发任何合并 |
 | `/merge` | 触发 review-fix-review 审批闭环后 squash merge 入主仓 |
 | `/discard-session` | 放弃当前 session 所有改动（不可恢复） |
 | `session_merge action=status` | 查看 worktree 绑定状态（非 diff 内容） |

package/commands/debug.md

@@ -8,7 +8,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   version: 1.0.0
   trigger_workflow: bugfix
   requires_human_approval: true
-  allowed_tools: smart_search, save_chat_insight, pending-changes, repo-map, ast-edit, nav-find, nav-goto, bash
+  allowed_tools: smart_search, save_chat_insight, repo-map, ast-edit, nav-find, nav-goto, bash, edit, write
   说明：入口 agent 由 workflow 控制（先取证用 reviewer，后续步骤切换）
   ADR-0008（gstack 单 markdown skill）/ ADR-0010（plan-act 模式分离）
 -->
@@ -50,8 +50,8 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 | 2. 假设 | planner | 出最多 3 个根因假设 + 验证方法 | 否 |
 | 3. 复现 | coder | 写最小测试用例（应红）| 风险动作自动审批 |
 | 4. 修复 | coder | 改代码让复现测试变绿 | 风险动作自动审批 |
-| 5. 审阅 | reviewer | 列 pending + 跑全量测试 | 否 |
-| 6. 落地 | coder | `pending-changes apply_all` | **必审批** |
+| 5. 审阅 | reviewer | 审 worktree diff + 跑全量测试 | 否 |
+| 6. 落地 | 用户 | 跑 `/merge` 触发 review-fix-review 闭环合并 session worktree 到主仓 | **必审批** |
 | 7. 沉淀 | reviewer | `save_chat_insight category=bugfix` | 否（写失败 skip） |
 
 ## 与 /ship 的差异
@@ -79,7 +79,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 | step 3 (复现) | retry 1 次。仍失败 → abort（也许不是真 bug） |
 | step 4 (修复) | retry 2 次。仍失败 → abort（假设可能错了） |
 | step 5 (审阅 BLOCK) | abort：reviewer 否决，回 step 2 重新假设 |
-| step 6 (apply 漂移) | abort：pending-changes 拒绝脏覆盖 |
+| step 6 (/merge 失败) | abort：review-fix-review 闭环未通过或主仓冲突 |
 | step 7 (save_chat_insight) | skip |
 
 ## 上下文注入

package/commands/deep.md

@@ -1,5 +1,5 @@
 ---
-description: 临时把当前任务的 agent 模型升到 deep 档（Opus + 大 thinking）；改动 stage 到 pending-changes 等待用户审批
+description: 临时把当前任务的 agent 模型升到 deep 档（Opus + 大 thinking）；改动直接写入 session worktree，由用户 /merge 拍板
 agent: planner
 ---
 
@@ -8,8 +8,8 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   name: deep
   version: 1.0.0
   trigger_workflow: null
-  allowed_tools: model_chain, ast_edit, pending_changes
-  说明：方案 A 用户显式 override 入口；与 /quick 对称；与 /model-switch 共享 ast_edit + pending_changes 落地路径
+  allowed_tools: model_chain, ast_edit
+  说明：方案 A 用户显式 override 入口；与 /quick 对称；与 /model-switch 共享 ast_edit 落地路径（写入 session worktree，由用户 /merge 拍板）
   ADR-0060（model-tier-three-layer-escalation，Phase 2d）
 -->
 
@@ -17,7 +17,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 # /deep — 临时升档到 deep（Opus + 大 thinking）
 
 把指定 agent 的模型临时升到 `deep` 档（默认 `anthropic/claude-opus-4-7` + 大 thinking budget）。
-改动会 stage 到 pending-changes，**由用户审批后 apply 才会真正生效**。
+改动会直接写入当前 session worktree（由 session-worktree-guard 隔离主仓），**由用户 `/merge` 拍板合并到主仓后才会真正生效**。
 
 ## 输入
 
@@ -45,11 +45,11 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
    - 若当前已是 `ultra` 档（更高档）→ 提示「<agent> 当前是 ultra 档（比 deep 更高），如确需降到 deep 请显式确认」，**不 stage**
    - 否则进入切换流程
 
-4. **执行切换**（复用 `/model-switch` 的等价路径，不直接调它，按 ast_edit + pending_changes 标准模式自行 stage）：
+4. **执行切换**（复用 `/model-switch` 的等价路径，不直接调它，按 ast_edit 标准模式直接写入 session worktree）：
    1. 从 `codeforge.json` 的 `models.categories.deep`（或 `models.tiers.deep`，若已落地 Phase 2a）读出目标 model id 和 thinking 配置
    2. 用 `ast_edit` 修改 `codeforge.json`：把 `models.agents.<agent>.model` 改为 deep 档 model；旧主模型自动 prepend 到该 agent 的 fallback 链头部（避免重启后被切回去）
-   3. 改动用 `pending_changes.stage` 暂存；**不调 apply**
-   4. 输出 pending-changes ID 给用户审批
+   3. `ast_edit` 直接写入 session worktree（session-worktree-guard 隔离主仓，不影响主工作区）
+   4. 告诉用户改动已落到 session worktree，跑 `/merge` 触发 review-fix-review 闭环合并
 
 5. **审批后提示**：用户 apply 之后，主动提示跑 `npm run sync:models` 把新配置同步到 `agents/<agent>.md` 的 frontmatter
 
@@ -65,15 +65,15 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 
 - `/quick` ←→ `/deep`：完全对称，方向相反
 - `/model-switch agent <model-id>`：底层通用切换；`/deep` 是它的「按档位语义糖」，自动算出 deep 档 model 不用用户记 id
-- 三者最终都走相同的 `ast_edit + pending_changes` 路径，没有"另一套"写文件机制
+- 三者最终都走相同的 `ast_edit` 写入 session worktree 路径，没有"另一套"写文件机制
 
 ## 安全约束
 
-- ❌ 禁止直接 apply pending-changes（必须用户审批）
+- ❌ 禁止绕过 `/merge` 直接把改动写进主仓（必须用户拍板）
 - ❌ 禁止改 `codeforge.json` 之外的文件
 - ❌ 禁止在没有 `models.categories.deep`（或 `models.tiers.deep`）配置时硬编码 model id —— 必须从配置读
-- ✅ stage 完一定要输出 pending id 给用户
-- ✅ apply 后一定要提示 `npm run sync:models`
+- ✅ 写入 session worktree 后，明确告知用户「改动已落到 session worktree，请跑 `/merge`」
+- ✅ `/merge` 后一定要提示 `npm run sync:models`
 
 ## 失败回退
 

package/commands/model-switch.md

@@ -8,8 +8,8 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   name: model-switch
   version: 1.0.0
   trigger_workflow: null
-  allowed_tools: model_chain, ast_edit, pending_changes, repo_map
-  说明：单 agent 命令，不触发 workflow；切换走 ast_edit + pending_changes 暂存
+  allowed_tools: model_chain, ast_edit, repo_map
+  说明：单 agent 命令，不触发 workflow；切换走 ast_edit 直接写入 session worktree（由用户 /merge 拍板）
   ADR-0008（gstack 单 markdown skill）
 -->
 
@@ -39,10 +39,10 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
    - **只给 agent 名** → 输出该 agent 的链 + 当前主模型 + 完整 fallback 顺序
    - **agent + model id** → 校验目标 model 在该 agent 的 chain 里，然后：
      1. 用 `ast_edit` 把 `codeforge.json` 中 `models.agents.<name>.model` 改成新值（旧主模型自动追加到 fallback 链头部，避免下次重启又被切回去）
-     2. 用 `pending_changes` 暂存改动，**等用户确认**
+     2. `ast_edit` 直接写入 session worktree（session-worktree-guard 隔离主仓），**告知用户跑 `/merge` 拍板**
      3. 提示用户跑 `npm run sync:models` 把新配置同步到 `agents/<name>.md`
    - **agent + `next`** → 调用 `model_chain({agent, current})`，拿到下一档后等同于第 3 种用法
-3. **永远不直接写文件**：所有切换都走 `pending_changes`
+3. **永远不直接写主仓**：所有切换都走 session worktree，最终由用户 `/merge` 合并
 
 ## 何时用
 

package/commands/quick.md

@@ -1,5 +1,5 @@
 ---
-description: 临时把当前任务的 agent 模型降到 quick 档（Sonnet/Haiku 档，省 token）；改动 stage 到 pending-changes 等待用户审批
+description: 临时把当前任务的 agent 模型降到 quick 档（Sonnet/Haiku 档，省 token）；改动直接写入 session worktree，由用户 /merge 拍板
 agent: planner
 ---
 
@@ -8,8 +8,8 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   name: quick
   version: 1.0.0
   trigger_workflow: null
-  allowed_tools: model_chain, ast_edit, pending_changes
-  说明：方案 A 用户显式 override 入口；与 /deep 对称；与 /model-switch 共享 ast_edit + pending_changes 落地路径
+  allowed_tools: model_chain, ast_edit
+  说明：方案 A 用户显式 override 入口；与 /deep 对称；与 /model-switch 共享 ast_edit 落地路径（写入 session worktree，由用户 /merge 拍板）
   ADR-0060（model-tier-three-layer-escalation，Phase 2d）
 -->
 
@@ -17,7 +17,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 # /quick — 临时降档到 quick（Sonnet/Haiku 档，省 token）
 
 把指定 agent 的模型临时降到 `quick` 档（默认 `anthropic/claude-sonnet-4-5` 或更轻量的 Haiku，按 `codeforge.json` 配置）。
-改动会 stage 到 pending-changes，**由用户审批后 apply 才会真正生效**。
+改动会直接写入当前 session worktree（由 session-worktree-guard 隔离主仓），**由用户 `/merge` 拍板合并到主仓后才会真正生效**。
 
 ## 输入
 
@@ -49,11 +49,11 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
    - 若用户降的是 `planner` 或 `reviewer`，额外提示「⚠️ <agent> 是质量关键节点，降档可能导致方案/评审失误」
    - 仍按用户意图执行 stage（提示是软约束，不阻断）
 
-5. **执行切换**（复用 `/model-switch` 的等价路径，按 ast_edit + pending_changes 标准模式自行 stage）：
+5. **执行切换**（复用 `/model-switch` 的等价路径，按 ast_edit 标准模式直接写入 session worktree）：
    1. 从 `codeforge.json` 的 `models.categories.quick`（或 `models.tiers.quick`，若已落地 Phase 2a）读出目标 model id 和 thinking 配置
    2. 用 `ast_edit` 修改 `codeforge.json`：把 `models.agents.<agent>.model` 改为 quick 档 model；旧主模型自动 prepend 到该 agent 的 fallback 链头部（避免重启后被切回去，方便升回去）
-   3. 改动用 `pending_changes.stage` 暂存；**不调 apply**
-   4. 输出 pending-changes ID 给用户审批
+   3. `ast_edit` 直接写入 session worktree（session-worktree-guard 隔离主仓，不影响主工作区）
+   4. 告诉用户改动已落到 session worktree，跑 `/merge` 触发 review-fix-review 闭环合并
 
 6. **审批后提示**：用户 apply 之后，主动提示跑 `npm run sync:models` 把新配置同步到 `agents/<agent>.md` 的 frontmatter
 
@@ -69,16 +69,16 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 
 - `/deep` ←→ `/quick`：完全对称，方向相反
 - `/model-switch agent <model-id>`：底层通用切换；`/quick` 是它的「按档位语义糖」，自动算出 quick 档 model 不用用户记 id
-- 三者最终都走相同的 `ast_edit + pending_changes` 路径，没有"另一套"写文件机制
+- 三者最终都走相同的 `ast_edit` 写入 session worktree 路径，没有"另一套"写文件机制
 
 ## 安全约束
 
-- ❌ 禁止直接 apply pending-changes（必须用户审批）
+- ❌ 禁止绕过 `/merge` 直接把改动写进主仓（必须用户拍板）
 - ❌ 禁止改 `codeforge.json` 之外的文件
 - ❌ 禁止在没有 `models.categories.quick`（或 `models.tiers.quick`）配置时硬编码 model id —— 必须从配置读
 - ✅ 降档前必须输出质量风险提示
-- ✅ stage 完一定要输出 pending id 给用户
-- ✅ apply 后一定要提示 `npm run sync:models`
+- ✅ 改动写入 session worktree 后，明确告知用户「已落到 session worktree，请跑 `/merge`」
+- ✅ `/merge` 后一定要提示 `npm run sync:models`
 
 ## 失败回退
 

package/commands/refactor.md

@@ -9,7 +9,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   version: 1.0.0
   trigger_workflow: refactor
   requires_human_approval: true
-  allowed_tools: smart_search, save_chat_insight, pending-changes, repo-map, nav-find, nav-goto, bash
+  allowed_tools: smart_search, save_chat_insight, repo-map, nav-find, nav-goto, bash, edit, write
   说明：workflow-engine 看到 trigger_workflow=refactor 拦截 /refactor 触发对应 YAML
 -->
 
@@ -45,7 +45,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 | 3. 确认覆盖 | reviewer | 确认测试覆盖待重构代码路径 | 否 |
 | 4. 重构 | coder | 实施重构，每步保证测试仍绿 | 否，auto_feedback 自纠 |
 | 5. 审阅 | reviewer | 确认行为不变 + 代码更清晰 | 否 |
-| 6. 落地 | coder | `pending-changes apply_all` | **必审批**（YAML 写死） |
+| 6. 落地 | 用户 | 跑 `/merge` 触发 review-fix-review 闭环合并 session worktree 到主仓 | **必审批**（YAML 写死） |
 | 7. 沉淀 | reviewer | `save_chat_insight` 把经验回写 KH | 否（写失败 skip） |
 
 ## 与 /tdd 的区别

package/commands/review.md

@@ -8,14 +8,14 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   name: review
   version: 1.0.0
   trigger_workflow: code-review
-  allowed_tools: smart_search, save_chat_insight, pending-changes, bash, repo-map, nav-find
+  allowed_tools: smart_search, save_chat_insight, bash, repo-map, nav-find
   说明：workflow-engine 看到 trigger_workflow=code-review 拦截 /review 触发对应 YAML
 -->
 
 
 # /review — 代码审阅工作流
 
-触发 `workflows/code-review.yaml`。**只读流程**：不 stage 任何改动，不 apply 任何 pending。
+触发 `workflows/code-review.yaml`。**只读流程**：不写任何文件、不触发 `/merge`。
 
 ## 输入
 

package/commands/ship.md

@@ -8,7 +8,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   version: 1.0.0
   trigger_workflow: feature-dev
   requires_human_approval: true
-  allowed_tools: smart_search, save_chat_insight, pending-changes, repo-map, nav-find, nav-goto, bash
+  allowed_tools: smart_search, save_chat_insight, repo-map, nav-find, nav-goto, bash, edit, write
   说明：workflow-engine 看到 trigger_workflow=feature-dev 拦截 /ship 触发对应 YAML
 -->
 
@@ -39,15 +39,15 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 | 步骤 | Agent | 动作 | 是否需审批 |
 |------|------|------|-----------|
 | 1. 规划 | planner | smart_search + repo-map → 出方案 | 否 |
-| 2. 执行 | coder | 按方案 stage 改动到 `.codeforge/pending/` | 风险动作自动审批 |
-| 3. 审阅 | reviewer | 列 pending + 跑测试 + 给 APPROVE/REQUEST_CHANGES | 否 |
-| 4. 落地 | coder | `pending-changes apply_all` | **必审批**（YAML 写死） |
+| 2. 执行 | coder | 按方案直接写入 session worktree（session-worktree-guard 隔离主仓） | 风险动作自动审批 |
+| 3. 审阅 | reviewer | 审 worktree diff + 跑测试 + 给 APPROVE/REQUEST_CHANGES | 否 |
+| 4. 落地 | 用户 | 跑 `/merge` 触发 review-fix-review 闭环合并 session worktree 到主仓 | **必审批**（YAML 写死） |
 | 5. 沉淀 | reviewer | `save_chat_insight` 把会话经验回写 KH | 否（写失败 skip） |
 
 ## 自主度（AGENTS.md C18）
 
 `/ship` 默认 `semi` 模式：
-- 风险动作（pending-changes apply / bash / write）自动弹审批
+- 风险动作（bash / 写到 worktree 外）自动弹审批；最终合并由用户跑 `/merge`
 - planner / reviewer 的只读动作不打扰
 
 要全自动跑，启动 opencode 时加 `--mode=full`，**且必须开 worktree 隔离**（自动启用）。
@@ -69,7 +69,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 | step 1 (planner) | abort：方案出不来，后续没意义 |
 | step 2 (coder)   | retry 1 次，仍失败则 abort |
 | step 3 (reviewer) BLOCK | abort：用户决定要不要继续 |
-| step 4 (apply) hash 漂移 | abort：避免脏覆盖（pending-changes 会拒） |
+| step 4 (/merge 失败) | abort：review-fix-review 闭环未通过或主仓冲突 |
 | step 5 (save_chat_insight) | skip：KH 写失败不能让用户已 ship 的功能"看起来失败" |
 
 ## 与 /plan 的关系

package/commands/tdd.md

@@ -9,7 +9,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
   version: 1.0.0
   trigger_workflow: tdd
   requires_human_approval: true
-  allowed_tools: smart_search, save_chat_insight, pending-changes, repo-map, nav-find, nav-goto, bash
+  allowed_tools: smart_search, save_chat_insight, repo-map, nav-find, nav-goto, bash, edit, write
   说明：workflow-engine 看到 trigger_workflow=tdd 拦截 /tdd 触发对应 YAML
 -->
 
@@ -46,7 +46,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 | 4. 最小实现-GREEN | coder | 写最小实现让测试变绿（不超出需要） | 否，auto_feedback 自纠 |
 | 5. 重构-REFACTOR | coder | 可选 refactor，测试必须仍绿 | 否，可 skip |
 | 6. 审阅 | reviewer | 确认三步全做 + 测试覆盖核心 | 否 |
-| 7. 落地 | coder | `pending-changes apply_all` | **必审批**（YAML 写死） |
+| 7. 落地 | 用户 | 跑 `/merge` 触发 review-fix-review 闭环合并 session worktree 到主仓 | **必审批**（YAML 写死） |
 
 ## TDD 三色循环
 
@@ -82,7 +82,7 @@ codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）
 | step 4 (最小实现) | auto_feedback 5 轮 |
 | step 5 (REFACTOR 失败) | skip：refactor 是可选 |
 | step 6 (审阅 REQUEST_CHANGES) | 回 step 2，max_loops=5 |
-| step 7 (apply) | hash 漂移则拒 |
+| step 7 (/merge) | review-fix-review 闭环未通过或主仓冲突则拒 |
 
 ## 元数据