sillyspec 3.22.9 → 3.23.1

package/src/worktree-apply.js

@@ -34,6 +34,20 @@ function gitQuiet(cwd, args) {
   }
 }
 
+/**
+ * 过滤掉 worktree 基础设施文件（非交付物），让 apply 只关心真正的变更产出：
+ *   - meta.json：worktree 元数据，baseline commit 中被跟踪、working-tree 被 CLI 改写
+ *     （provisioning→linked 等）。它必须保持 modified（其 baselineCommit 字段是 apply diff
+ *     的锚点，保证 baseline overlay 文件不被误判为变更）。若不排除，modified-tracked 的
+ *     meta.json 会落入 changedFiles，触发「不在 design.md 清单」误判，导致每个 execute 的
+ *     assess 恒 BLOCKED。
+ *   - .sillyspec/：变更文档 / 运行时产物，不属于源码交付。
+ * 对 modified-tracked（git diff）与 untracked（ls-files --others）一视同仁。
+ */
+export function filterDeliverableFiles(files) {
+  return files.filter(f => !f.startsWith('.sillyspec/') && f !== 'meta.json');
+}
+
 /**
  * 获取文件在 git 中的 blob hash（基于某个 commit/tree）
  * @param {string} cwd - git 工作区路径
@@ -59,7 +73,7 @@ function getFileBlobHash(cwd, treeish, filePath) {
  *   errors: string[]
  * }}
  */
-export function applyWorktree(changeName, { cwd, checkOnly = false } = {}) {
+export function applyWorktree(changeName, { cwd, checkOnly = false, merge = false } = {}) {
   const projectRoot = cwd || process.cwd();
   const wm = new WorktreeManager({ cwd: projectRoot });
   const meta = wm.getMeta(changeName);
@@ -70,6 +84,7 @@ export function applyWorktree(changeName, { cwd, checkOnly = false } = {}) {
     hashMismatchFiles: [],
     patchPath: null,
     errors: [],
+    merged: false,
   };
 
   // --- 1. 校验 worktree 存在 + meta.json 有效 ---
@@ -106,11 +121,12 @@ export function applyWorktree(changeName, { cwd, checkOnly = false } = {}) {
 
     // untracked 新文件（diffBase 中不存在的文件）
     const untrackedRaw = gitQuiet(worktreePath, `ls-files --others --exclude-standard`);
-    const untrackedFiles = untrackedRaw
-      ? untrackedRaw.split('\n').filter(Boolean).filter(f => !f.startsWith('.sillyspec/') && f !== 'meta.json')
-      : [];
+    const untrackedFiles = untrackedRaw ? untrackedRaw.split('\n').filter(Boolean) : [];
 
-    changedFiles = [...new Set([...statusFiles, ...untrackedFiles])];
+    // 排除 worktree 基础设施文件（meta.json / .sillyspec/，见 filterDeliverableFiles）。
+    // 对 modified-tracked（statusFiles）与 untracked 一视同仁——否则 modified 的 meta.json
+    // 会被误算入 changedFiles，触发 design.md 清单校验失败（assess 恒 BLOCKED）。
+    changedFiles = filterDeliverableFiles([...new Set([...statusFiles, ...untrackedFiles])]);
   } catch (e) {
     result.errors.push(`获取变更文件列表失败: ${e.message}`);
     return result;
@@ -157,9 +173,13 @@ export function applyWorktree(changeName, { cwd, checkOnly = false } = {}) {
     const raw = `staged:${staged}\nunstaged:${unstaged}\nuntracked:${untracked}`;
     const currentHash = createHash('sha256').update(raw).digest('hex').slice(0, 16);
     if (currentHash !== meta.baselineHash) {
+      if (merge && !checkOnly) {
+        // baseline 漂移 + --merge 降级（D-001）：跳过步骤 5-7（patch 路径），走 git merge
+        return applyByMerge(result, changeName, projectRoot, wm);
+      }
       result.errors.push(
         `主工作区 baseline 已变化（execute 前后不一致），不能直接 apply task.patch。\n` +
-        `建议：重新创建 worktree 或手动检查冲突。\n` +
+        `建议：重新创建 worktree，或用 --merge 降级（git merge sillyspec/${changeName} 替代 patch，会引入合并提交）。\n` +
         `execute 前 baseline: ${meta.baselineHash}\n` +
         `当前 baseline: ${currentHash}`
       );
@@ -305,6 +325,53 @@ export function applyWorktree(changeName, { cwd, checkOnly = false } = {}) {
   return result;
 }
 
+/**
+ * baseline 漂移时的 merge 降级路径（D-001）。
+ *
+ * 当主工作区 baseline 在 execute 期间漂移、patch 无法干净应用时，用
+ * `git merge sillyspec/<change>` 替代 patch apply（BRANCH_PREFIX='sillyspec/'，worktree.js:18）。
+ * git merge 比 patch 鲁棒，能处理 baseline 漂移 + 潜在冲突。会引入合并提交
+ * （D-002：与 worktree.md:84「patch 而非 merge 保持线性历史」架构决策的张力——
+ * 仅作 --merge 显式 opt-in，不改变默认 patch 行为）。
+ *
+ * merge 冲突时不自动解决：git merge --abort 回滚到合并前状态 + 报冲突文件列表。
+ *
+ * @param {object} result - applyWorktree 的 result 对象（mutate 后返回）
+ * @param {string} changeName
+ * @param {string} projectRoot - 主仓库根
+ * @param {object} wm - WorktreeManager 实例
+ * @returns {object} result（merged=true 表示走了 merge 降级）
+ */
+function applyByMerge(result, changeName, projectRoot, wm) {
+  try {
+    git(projectRoot, `merge --no-ff sillyspec/${changeName}`);
+    result.merged = true;
+    result.ok = true;
+    try { result.mergeSummary = git(projectRoot, `log --oneline -1`); } catch {}
+    // 成功后自动 cleanup（与 patch 路径步骤 8 一致；分支已合并，删除安全）
+    try {
+      wm.cleanup(changeName);
+    } catch (cleanupErr) {
+      result.warnings = result.warnings || [];
+      result.warnings.push(`cleanup 失败（不影响 merge 结果）: ${cleanupErr.message}`);
+    }
+  } catch (e) {
+    // merge 冲突：先取冲突文件列表，再 abort 回滚（避免半成品合并）
+    let conflictFiles = [];
+    try {
+      const cf = gitQuiet(projectRoot, `diff --name-only --diff-filter=U`);
+      conflictFiles = cf ? cf.split('\n').filter(Boolean) : [];
+    } catch {}
+    try { gitQuiet(projectRoot, `merge --abort`); } catch {}
+    result.errors.push(
+      `git merge sillyspec/${changeName} 冲突，请手动解决。冲突文件：\n` +
+      (conflictFiles.length ? `  ${conflictFiles.join('\n  ')}\n` : `  (未能获取冲突文件列表)\n`) +
+      `已执行 git merge --abort 回滚到合并前状态。`
+    );
+  }
+  return result;
+}
+
 /**
  * 风险审计：评估 worktree 变更是否可以安全自动 apply
  *

package/src/worktree.js

@@ -453,6 +453,16 @@ export class WorktreeManager {
       return { branch: existingMeta.branch, worktreePath: existingMeta.worktreePath, baseHash: existingMeta.baseHash, mode: existingMeta.mode }
     }
 
+    // 供给 deps（与 native worktree 路径一致）。in-place-fallback 模式漏写 depsStatus 会致
+    // enforceDepsGate（run.js）把 undefined 当 unknown 阻断 execute --done，死锁无法推进。
+    // 见 docs/sillyspec/execute-inplace-deps-gate.md（2026-07-08 发现）。
+    let deps = {}
+    try {
+      deps = provisionDeps(worktreePath, this.cwd, { specBase: join(this.cwd, '.sillyspec') }) || {}
+    } catch (e) {
+      deps = { depsStatus: 'failed', depsError: `provisionDeps crashed: ${e.message}` }
+    }
+
     // 硬规则：禁止 self-overlay（source 和 target 相同时 overlay 必然冲突）
     const resolvedSource = resolve(this.cwd)
     const resolvedTarget = resolve(worktreePath)
@@ -474,6 +484,7 @@ export class WorktreeManager {
         baselineFiles: [],
         baselineCommit: null,
         baselineHash: null,
+        ...deps,
       }
       if (!existsSync(this.worktreeBase)) mkdirSync(this.worktreeBase, { recursive: true })
       const metaDir = join(this.worktreeBase, name)
@@ -510,6 +521,7 @@ export class WorktreeManager {
       baselineFiles,
       baselineCommit,
       baselineHash,
+      ...deps,
     };
 
     // in-place 模式下 meta 写入 worktreeBase（避免污染主工作区）