@andyqiu/codeforge 0.4.0 → 0.5.1

package/agents/codeforge.md

@@ -66,12 +66,13 @@ fallback_models:
 | 场景 | 该做什么 | MUST NOT |
 |---|---|---|
 | 用户问简单问题 / 寻求解释 / 对比方案讨论（≤ 800 字能答完） | **自己直接答**，不派任何 agent | ❌ 派 planner 或 coder |
-| **小改动 short-circuit**：用户指明确切文件位置 + 改动 ≤ 1 文件 + 估算 < 5 行 + 用户已给出修改思路 | 跳过 planner，**直接派 coder**，prompt 自包含改动需求（无 plan_id 路径） | ❌ 派 planner 再让它派 coder |
+| **小改动 short-circuit**：用户指明确切文件位置 + 改动 ≤ 1 文件 + 估算 < 5 行 + 用户已给出修改思路 | 跳过 planner，**直接派 coder**，prompt 自包含改动需求（无 plan_id 路径）；coder 完成后同样走「coder 回报正常完成」行的自动 reviewer 流程 | ❌ 派 planner 再让它派 coder |
 | 复杂多步任务（含设计 / 涉及多文件 / 不确定改哪 / 需要查历史经验） | **派 planner 出方案** → 等 planner 回 boomerang（含 `plan_id: plan-xxx`）→ **自动派 reviewer 审方案**（`review_target=plan_only` + plan_id）：APPROVE → 派 coder 执行（带 plan_id + sessionId）；REQUEST_CHANGES (`plan_review_loop_count` < 3) → 自动派 planner 改方案，loop +1；REQUEST_CHANGES (loop = 3) → 转告用户三选一；BLOCK → 转告用户 + 建议派 planner 重设计 | ❌ 派完 planner 直接派 coder；❌ 绕过方案 BLOCK |
 | **决策 review（Q3-a 范围）** | 先派 reviewer 审决策合理性（`review_target=decision_only`）：APPROVE → 按决策派；REQUEST_CHANGES → reviewer 给替代方案，loop 1 次后转告用户；BLOCK → 转告用户 | ❌ 审用户业务选择本身；❌ 把所有 question 都套 review |
 | 用户要"独立交付物" | 派 coder 子 session 写并直接落到 session worktree；prompt 明示"final response 不要粘回长内容" | ❌ 自己在父对话吐长文档 |
 | 用户要查项目结构 / 历史经验 | 自己调 `smart_search` / `repo_map` / `read` / `plan_read` | ❌ 为此派 subagent |
-| **subagent 回报正常完成** | 拿到 boomerang 摘要后决定下一棒：**询问用户是否 `/merge`** / 派 reviewer / 派下个 phase / 收尾 | ❌ 重新审查 subagent 的代码；❌ 默认 subagent 会自派下一棒 |
+| **coder 回报正常完成** | 自动派 reviewer 审代码改动（`review_target` 按文件类型自动检测）；后续按下方「reviewer 报三档」独立行处理 | ❌ 跳过 reviewer 直接问用户是否 merge；❌ 重新审查代码 |
+| **其他 subagent 回报正常完成**（planner / reviewer 等） | 按上下文决定下一棒：派 reviewer / 派下个 phase / 收尾 | ❌ 默认 subagent 会自派下一棒 |
 | **subagent 报错 / 中断 / 摘要为空** | **立刻停下**，把错误首行原文 + 子 session id 转告用户，问「重试 / 改方案 / 跳过」三选一 | ❌ 盲目"再派一次试试" |
 | **subagent 长时间无回报** | 提醒用户「子 session 仍在跑，按 `Ctrl+→` 可切过去看进度」 | ❌ 主动 Esc 取消；❌ 重派 |
 | **reviewer 报 REQUEST_CHANGES（代码 review，`code_review_loop_count` < 3）** | **自动派 coder 修**（带具体到行的 reviewer 意见 + 原 plan_id + sessionId），loop +1 | ❌ 同时派多个 coder；❌ 不带 reviewer 意见 |

package/bin/codeforge.mjs

@@ -70,7 +70,10 @@ function getVersion() {
 // 上下文；process.argv[1] 保留用户实际调用路径，不会被解析。
 function isFromNpm() {
   const entry = process.argv[1] ?? ""
-  return entry.includes(`${path.sep}node_modules${path.sep}`)
+  if (entry.includes(`${path.sep}node_modules${path.sep}`)) return true
+  // fallback：npm 包里没有 build-with-version.mjs（只有 dist/），检测它不存在即为 npm 场景
+  const buildScript = path.join(REPO_ROOT, "scripts", "build-with-version.mjs")
+  return !existsSync(buildScript)
 }
 
 // ────────────────────────────────────────────────────────────────────

package/commands/diff.md

@@ -0,0 +1,62 @@
+---
+description: 查看当前 session worktree 的改动（文件列表+统计，或完整 diff）
+---
+
+<!--
+codeforge 元数据（opencode 不读，由 plugins / workflow-engine 解析）：
+  name: diff
+  version: 1.0.0
+  requires_human_approval: false
+  说明：底层调用 session_merge 工具的 action=diff
+        默认显示改动文件列表 + 统计（stat=true）
+        --full 显示完整 diff 内容（stat=false）
+        没有 worktree 时友好提示
+-->
+
+
+# /diff — 查看当前 session worktree 改动
+
+**ADR: worktree-session-isolation Phase 3**
+
+快速查看当前 session worktree 中的代码改动，无需触发 merge 或 review 流程。
+
+## 输入
+
+用户参数：$ARGUMENTS
+
+## 用法
+
+```
+/diff                     # 显示改动文件列表 + 统计（git diff --stat 风格）
+/diff --full              # 显示完整 diff 内容
+```
+
+## 行为说明
+
+| 模式 | 触发条件 | 输出内容 |
+|------|----------|----------|
+| **stat 模式**（默认） | `/diff` | 改动文件列表 + 增删行数统计 |
+| **full 模式** | `/diff --full` | 完整 diff 内容（含每行改动） |
+| **无 worktree** | session 无写操作 | 友好提示：当前 session 没有写操作，无 worktree 改动 |
+
+## 底层调用
+
+```
+/diff         → session_merge(action="diff", stat=true)
+/diff --full  → session_merge(action="diff", stat=false)
+```
+
+## 何时使用
+
+- 在 `/merge` 之前确认改动范围
+- 向用户或 reviewer 展示本次 session 做了什么
+- 调试时快速确认某文件是否已被修改
+
+## 与其他命令的关系
+
+| 命令 | 用途 |
+|------|------|
+| `/diff` | 本命令：只读查看 worktree 改动，不触发任何合并 |
+| `/merge` | 触发 review-fix-review 审批闭环后 squash merge 入主仓 |
+| `/discard-session` | 放弃当前 session 所有改动（不可恢复） |
+| `session_merge action=status` | 查看 worktree 绑定状态（非 diff 内容） |

package/dist/index.js

@@ -13763,6 +13763,7 @@ var description26 = [
   "- action=status：查询当前 session worktree 状态（任意 agent 可调）",
   "- action=merge：codeforge orchestrator 在用户触发 /merge 后调（**subagent 禁止**）",
   "- action=discard：用户决定放弃当前 session 改动时调",
+  "- action=diff：查看当前 session worktree 相对主仓（baseSha）的改动；stat=true 只看文件列表",
   "**注意**：",
   "- merge 走 review-fix-review 闭环（默认 3 轮）；force=true 跳过 review 直接 squash",
   "- subagent (coder/reviewer/planner) 调 action=merge 会被 Phase 2 guard plugin 拒绝",
@@ -13784,6 +13785,11 @@ var ArgsSchema26 = z27.discriminatedUnion("action", [
   z27.object({
     action: z27.literal("discard"),
     session_id: z27.string().optional().describe("默认当前 session")
+  }),
+  z27.object({
+    action: z27.literal("diff"),
+    session_id: z27.string().optional().describe("默认当前 session"),
+    stat: z27.boolean().optional().describe("true=只显示文件列表+统计，false=完整 diff（默认 false）")
   })
 ]);
 var _ctx = {};
@@ -13833,6 +13839,63 @@ async function execute26(input) {
         data: { discarded: true, sessionId }
       };
     }
+    if (args.action === "diff") {
+      const entry = await getSessionWorktree(sessionId, mainRoot);
+      if (!entry) {
+        return { ok: false, action: "diff", error: `session ${sessionId} 没有绑定 worktree` };
+      }
+      const { execFile: execFile4 } = await import("node:child_process");
+      const { promisify: promisify2 } = await import("node:util");
+      const execFileAsync = promisify2(execFile4);
+      const diffArgs = args.stat ? ["diff", "--stat", entry.baseSha] : ["diff", entry.baseSha];
+      let diffOutput = "";
+      let changedFiles = [];
+      try {
+        const { stdout } = await execFileAsync("git", diffArgs, {
+          cwd: entry.worktreePath,
+          maxBuffer: 5242880
+        });
+        diffOutput = stdout;
+        const { stdout: nameOnly } = await execFileAsync("git", ["diff", "--name-only", entry.baseSha], { cwd: entry.worktreePath, maxBuffer: 1048576 });
+        changedFiles = nameOnly.trim().split(`
+`).filter(Boolean);
+      } catch (e) {
+        const msg = e instanceof Error ? e.message : String(e);
+        if (!msg.includes("unknown revision") && !msg.includes("bad object")) {
+          return { ok: false, action: "diff", error: `git diff 失败: ${msg}` };
+        }
+        try {
+          const fallbackArgs = args.stat ? ["diff", "--stat", "HEAD"] : ["diff", "HEAD"];
+          const { stdout } = await execFileAsync("git", fallbackArgs, {
+            cwd: entry.worktreePath,
+            maxBuffer: 5242880
+          });
+          diffOutput = stdout;
+          const { stdout: nameOnly } = await execFileAsync("git", ["diff", "--name-only", "HEAD"], { cwd: entry.worktreePath, maxBuffer: 1048576 });
+          changedFiles = nameOnly.trim().split(`
+`).filter(Boolean);
+        } catch (e2) {
+          return {
+            ok: false,
+            action: "diff",
+            error: `git diff 失败: ${e2 instanceof Error ? e2.message : String(e2)}`
+          };
+        }
+      }
+      return {
+        ok: true,
+        action: "diff",
+        data: {
+          sessionId,
+          worktreePath: entry.worktreePath,
+          branch: entry.branch,
+          baseSha: entry.baseSha,
+          changedFiles,
+          fileCount: changedFiles.length,
+          diff: diffOutput
+        }
+      };
+    }
     if (!_ctx.spawner) {
       return {
         ok: false,
@@ -13840,11 +13903,12 @@ async function execute26(input) {
         error: "session_merge: SubagentSpawner 未注入（Phase 1 runtime 暂未接 wire-up；" + "Phase 3 commands/merge.md 落地后会通过 __setContext 注入）"
       };
     }
+    const mergeArgs = args;
     const result = await runMergeLoop({
       sessionId,
       mainRoot,
-      ...args.plan_id ? { planId: args.plan_id } : {},
-      ...args.force ? { force: true } : {},
+      ...mergeArgs.plan_id ? { planId: mergeArgs.plan_id } : {},
+      ...mergeArgs.force ? { force: true } : {},
       spawner: _ctx.spawner
     });
     return { ok: true, action: "merge", data: result };
@@ -20596,7 +20660,7 @@ import * as zlib from "node:zlib";
 // lib/version-injected.ts
 function getInjectedVersion() {
   try {
-    const v = "0.4.0";
+    const v = "0.5.1";
     if (typeof v === "string" && /^\d+\.\d+\.\d+/.test(v)) {
       return v;
     }