sillyspec 3.23.0 → 3.23.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sillyspec",
-  "version": "3.23.0",
+  "version": "3.23.1",
   "description": "SillySpec CLI — 流程状态机，让 AI 严格按步骤来",
   "icon": "logo.jpg",
   "homepage": "https://sillyspec.ppdmq.top/",

package/src/index.js

@@ -632,12 +632,13 @@ SillySpec worktree — git worktree 隔离管理
         }
         case 'apply': {
           if (!wtName) {
-            console.error('❌ 用法: sillyspec worktree apply <change-name> [--check-only]');
+            console.error('❌ 用法: sillyspec worktree apply <change-name> [--check-only] [--merge]');
             process.exit(1);
           }
           const checkOnly = args.includes('--check-only');
+          const merge = args.includes('--merge');
           const { applyWorktree } = await import('./worktree-apply.js');
-          const result = applyWorktree(wtName, { cwd: dir, checkOnly });
+          const result = applyWorktree(wtName, { cwd: dir, checkOnly, merge });
 
           if (result.errors.length > 0) {
             console.error(`❌ 校验失败:`);
@@ -657,6 +658,8 @@ SillySpec worktree — git worktree 隔离管理
             for (const f of result.changedFiles) {
               console.log(`   ${f}`);
             }
+          } else if (result.merged) {
+            console.log(`✅ 已通过 git merge 应用变更（baseline 漂移降级，引入合并提交）${result.mergeSummary ? '：' + result.mergeSummary : ''}`);
           } else {
             console.log(`✅ 已应用 ${result.changedFiles.length} 个文件变更`);
           }
@@ -710,6 +713,7 @@ SillySpec worktree — git worktree 隔离管理
             console.log('Action: blocked');
             console.log('   → 检查变更: sillyspec worktree diff ' + wtName);
             console.log('   → 丢弃变更: sillyspec worktree cleanup ' + wtName);
+            console.log('   → baseline 漂移降级: sillyspec worktree apply ' + wtName + ' --merge');
           }
           break;
         }

package/src/run.js

@@ -3321,7 +3321,7 @@ async function completeStep(pm, progress, stageName, cwd, outputText, inputText
             } catch {}
 
             const reviewResult = validateTaskReviews({ planContent, runtimeRoot, executeRunId, changeDir: planFile, gitDir: reviewGitDir })
-            printReviewResult(reviewResult)
+            printReviewResult(reviewResult, { runtimeRoot, executeRunId })
 
             if (!reviewResult.ok) {
               // Task review 校验失败，阻断 execute 完成
@@ -3329,7 +3329,7 @@ async function completeStep(pm, progress, stageName, cwd, outputText, inputText
               const uncheckedTasks = reviewResult.errors.filter(e => e.includes('缺少 review.json'))
               if (uncheckedTasks.length > 0) {
                 console.error('\n⚠️  部分任务已在 plan.md 中勾选，但 review.json 不存在。')
-                console.error('   请取消勾选这些任务的 checkbox，或补充对应的 review.json。')
+                console.error(`   请取消勾选这些任务的 checkbox，或补充对应的 review.json（execute run ID: ${executeRunId}）。`)
               }
               rollbackStageCompletion(stageData, steps, currentIdx)
               progress.lastActive = new Date().toLocaleString('zh-CN',{hour12:false})

package/src/stages/execute.js

@@ -620,7 +620,7 @@ ${taskList}
 
 **review.json 路径：**
 
-task-XX 对应：.sillyspec/.runtime/execute-runs/{EXECUTE_RUN_ID}/tasks/task-XX/review.json
+task-XX 对应：{SPEC_ROOT}/.runtime/execute-runs/{EXECUTE_RUN_ID}/tasks/task-XX/review.json
 
 本 execute run 的固定 ID 是：{EXECUTE_RUN_ID}
 **所有 task 的 review.json 必须使用这个 ID，不要自行创建新目录。**
@@ -641,7 +641,7 @@ task-XX 对应：.sillyspec/.runtime/execute-runs/{EXECUTE_RUN_ID}/tasks/task-XX
 ### 完成后
 1. 为每个后端 router task，扫描变更文件提取 API 端点 artifact：
    - 在变更文件中搜索所有 router 注册路径（@router.get/post/put/delete）
-   - 将端点清单写入 .sillyspec/.runtime/contract-artifacts/<task-name>/endpoints.json
+   - 将端点清单写入 {SPEC_ROOT}/.runtime/contract-artifacts/<task-name>/endpoints.json
    - 格式: { "task": "task-XX", "type": "backend_endpoints", "endpoints": [{ "method": "GET", "path": "/api/ppm/xxx" }] }
 2. 运行 sillyspec run execute --done --input "用户原始反馈" --output "Wave ${waveIndex} 结果摘要"`
 }

package/src/task-review.js

@@ -179,7 +179,7 @@ export function validateTaskReviews(opts) {
         if (isTaskLowRisk(changeDir, taskId)) {
           warnings.push(`${taskId}: 缺少 review.json — task 声明 low_risk: true，已豁免评审（type-only/机械迁移）`)
         } else {
-          errors.push(`${taskId}: 缺少 review.json — task 未经过评审`)
+          errors.push(`${taskId}: 缺少 review.json — task 未经过评审（期望路径：${reviewPath}，execute run ID: ${executeRunId}）`)
         }
       }
       continue
@@ -447,7 +447,7 @@ export function ensureTaskReviewDir(runtimeRoot, executeRunId, taskId) {
  * 打印校验结果
  * @param {{ ok: boolean, errors: string[], warnings: string[], requiredEvidence: Array }} result
  */
-export function printReviewResult(result) {
+export function printReviewResult(result, context = {}) {
   if (result.ok && result.warnings.length === 0) {
     console.log('\n✅ Task Review Gate — 所有任务评审通过')
     return
@@ -458,7 +458,10 @@ export function printReviewResult(result) {
     for (const err of result.errors) {
       console.error(`   - ${err}`)
     }
-    console.error('\n   提示：为缺失/失败的任务补充 review.json，然后重新 --done')
+    const hint = context.runtimeRoot && context.executeRunId
+      ? `期望路径：${context.runtimeRoot}/execute-runs/${context.executeRunId}/tasks/<task-XX>/review.json`
+      : '为缺失/失败的任务补充 review.json'
+    console.error(`\n   提示：${hint}，然后重新 --done`)
   }
 
   if (result.warnings.length > 0) {

package/src/worktree-apply.js

@@ -73,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);
@@ -84,6 +84,7 @@ export function applyWorktree(changeName, { cwd, checkOnly = false } = {}) {
     hashMismatchFiles: [],
     patchPath: null,
     errors: [],
+    merged: false,
   };
 
   // --- 1. 校验 worktree 存在 + meta.json 有效 ---
@@ -172,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}`
       );
@@ -320,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
  *