sillyspec 3.11.9 → 3.11.11

package/package.json

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

package/src/hooks/worktree-guard.js

@@ -3,6 +3,10 @@
  *
  * 三重门禁：stageGate × locationGate × fileGate
  * 纯判断模块，不做实际的 hook 注入。
+ *
+ * P0 优化：
+ * - 阶段检测 fallback：gate-status.json → progress.json currentStage
+ * - 拦截提示针对每个阶段给出具体修复建议
  */
 
 import { existsSync, readFileSync } from 'fs'
@@ -39,6 +43,41 @@ const DANGER_STASH_ACTIONS = new Set(['drop', 'clear', 'pop'])
 /** 危险命令前缀 */
 const DANGER_PREFIXES = ['sudo', 'rm -rf', 'rm -r', 'rmdir']
 
+// ── 阶段 → 拦截提示映射 ──
+
+const STAGE_HINTS = {
+  '(none)': [
+    '没有检测到活跃的 SillySpec 流程。',
+    '你需要先启动一个任务流程才能修改源码：',
+    '',
+    '  小改动（≤3 文件）：sillyspec run quick "任务描述"',
+    '  大改动（>3 文件）：sillyspec run brainstorm → plan → execute',
+    '  全自动模式：    sillyspec run auto "任务描述"',
+  ],
+  'brainstorm': [
+    '当前在 brainstorm（需求分析）阶段，这个阶段只写文档，不写代码。',
+    '完成 brainstorm 后，流程会自动推进到 plan → execute。',
+    'execute 阶段才允许写代码。',
+  ],
+  'plan': [
+    '当前在 plan（计划制定）阶段，这个阶段只写计划文档和任务蓝图，不写代码。',
+    '完成 plan 后，运行 sillyspec run execute 进入执行阶段。',
+  ],
+  'verify': [
+    '当前在 verify（验证）阶段，只做代码审查和测试验证，不修改源码。',
+    '如需修改，请先回到 execute 阶段或使用 quick 模式：',
+    '  sillyspec run quick "修改描述"',
+  ],
+  'archive': [
+    '当前在 archive（归档）阶段，不修改源码。',
+    '如需修改，请开启新变更：sillyspec run quick "修改描述"',
+  ],
+  'explore': [
+    '当前在 explore（探索）阶段，只读不写。',
+    '确认方案后使用：sillyspec run brainstorm 或 sillyspec run quick',
+  ],
+}
+
 // ── 辅助函数 ──
 
 function resolveWorktreeDir(cwd) {
@@ -60,6 +99,46 @@ function readGateStatus(cwd) {
   }
 }
 
+/**
+ * 从 progress.json fallback 读取 currentStage
+ * 优先级：gate-status.json > 全局 progress.json > 变更级 progress.json
+ * @param {string} cwd
+ * @returns {string|null} 阶段名，null 表示无法确定
+ */
+function readCurrentStage(cwd) {
+  // 1. gate-status.json（权威来源）
+  const gateStatus = readGateStatus(cwd)
+  if (gateStatus && gateStatus.stage) return gateStatus.stage
+
+  // 2. 全局 progress.json
+  const globalProgress = path.join(cwd, '.sillyspec', '.runtime', 'progress.json')
+  if (existsSync(globalProgress)) {
+    try {
+      const data = JSON.parse(readFileSync(globalProgress, 'utf8'))
+      if (data.currentStage) return data.currentStage
+    } catch { /* skip */ }
+  }
+
+  // 3. 变更级 progress.json（从 global.json 找到活跃变更）
+  const globalFile = path.join(cwd, '.sillyspec', '.runtime', 'global.json')
+  if (existsSync(globalFile)) {
+    try {
+      const global = JSON.parse(readFileSync(globalFile, 'utf8'))
+      const changesDir = path.join(cwd, '.sillyspec', 'changes')
+      for (const cn of (global.activeChanges || [])) {
+        const pp = path.join(changesDir, cn, 'progress.json')
+        if (!existsSync(pp)) continue
+        try {
+          const data = JSON.parse(readFileSync(pp, 'utf8'))
+          if (data.currentStage) return data.currentStage
+        } catch { /* skip */ }
+      }
+    } catch { /* skip */ }
+  }
+
+  return null
+}
+
 /**
  * 检查当前变更是否处于 noWorktree 模式
  * @param {string} cwd
@@ -153,7 +232,6 @@ function loadLocalConfig(cwd) {
   for (const p of candidates) {
     if (!existsSync(p)) continue
     try {
-      // 简单 YAML 解析（只处理顶层键值对和简单数组）
       const content = readFileSync(p, 'utf8')
       return parseSimpleYaml(content)
     } catch {
@@ -327,6 +405,16 @@ function matchDangerBlacklist(command) {
   return parts.some(p => isSingleCommandDangerous(p))
 }
 
+/**
+ * 构建阶段拦截提示
+ * @param {string} stage
+ * @returns {string}
+ */
+function buildStageHint(stage) {
+  const hint = STAGE_HINTS[stage] || STAGE_HINTS['(none)']
+  return hint.join('\n')
+}
+
 // ── 公共接口 ──
 
 /**
@@ -336,6 +424,10 @@ function matchDangerBlacklist(command) {
  * - noWorktree 模式下，execute/quick 阶段不允许源码写入（没有隔离环境）
  * - 除非同时设置 SILLYSPEC_DISABLE_HOOKS=1
  *
+ * P0 优化：
+ * - 使用 readCurrentStage() fallback 读取阶段（gate-status → progress）
+ * - 拦截提示按阶段给出具体修复建议
+ *
  * @param {string} filePath - 目标文件绝对路径
  * @param {string} cwd - 当前工作目录
  * @returns {{ blocked: boolean, reason?: string }}
@@ -348,11 +440,15 @@ export function shouldBlockWrite(filePath, cwd) {
   // 1. 文件门禁：文档类/配置类始终放行
   if (matchFileWhitelist(absPath)) return { blocked: false }
 
-  // 2. 阶段门禁
+  // 2. 阶段门禁（使用 fallback 读取）
   const effectiveCwd = cwd || process.cwd()
-  const gateStatus = readGateStatus(effectiveCwd)
-  if (!gateStatus || !ALLOWED_STAGES.includes(gateStatus.stage)) {
-    return { blocked: true, reason: `stage "${gateStatus?.stage || '(none)'}" does not allow source code writes` }
+  const stage = readCurrentStage(effectiveCwd) || '(none)'
+
+  if (!ALLOWED_STAGES.includes(stage)) {
+    return {
+      blocked: true,
+      reason: buildStageHint(stage)
+    }
   }
 
   // 3. 位置门禁
@@ -360,10 +456,30 @@ export function shouldBlockWrite(filePath, cwd) {
 
   // noWorktree 模式：无隔离环境，禁止源码写入（降级到更严格）
   if (isNoWorktreeMode(effectiveCwd)) {
-    return { blocked: true, reason: 'noWorktree mode: source code writes require isolation (use SILLYSPEC_DISABLE_HOOKS=1 to override)' }
+    return {
+      blocked: true,
+      reason: [
+        '当前处于 --no-worktree 降级模式，不允许源码写入。',
+        '如需修改源码，请移除 --no-worktree 标志重新执行。',
+        '紧急情况可设置 SILLYSPEC_DISABLE_HOOKS=1 绕过限制。',
+      ].join('\n')
+    }
   }
 
-  return { blocked: true, reason: 'source code write blocked outside worktree' }
+  return {
+    blocked: true,
+    reason: [
+      '源码修改只能在 worktree 隔离环境中进行。',
+      '',
+      '你可能需要：',
+      '  1. 确认 worktree 已创建：sillyspec worktree list',
+      '  2. 如未创建，先创建：sillyspec worktree create <变更名>',
+      '  3. 在 worktree 目录中工作（子代理的 cwd 设为 worktree 路径）',
+      '',
+      '如果你在 execute 阶段，通常会自动创建 worktree。',
+      '检查是否跳过了 "创建 worktree" 步骤。',
+    ].join('\n')
+  }
 }
 
 /**
@@ -380,16 +496,19 @@ export function shouldBlockBash(command, cwd) {
   // cwd 在 worktree 内 → 全部放行
   if (isInsideWorktree(effectiveCwd)) return { blocked: false }
 
-  // 阶段门禁
-  const gateStatus = readGateStatus(effectiveCwd)
-  const stageOk = gateStatus && ALLOWED_STAGES.includes(gateStatus.stage)
+  // 阶段门禁（使用 fallback 读取）
+  const stage = readCurrentStage(effectiveCwd) || '(none)'
+  const stageOk = ALLOWED_STAGES.includes(stage)
 
   if (!stageOk) {
     // 非 execute/quick 阶段，只允许只读白名单
     const localConfig = loadLocalConfig(effectiveCwd)
     const extraReadonly = localConfig.worktreeHook?.readonlyCommands || localConfig['worktree-hook']?.readonlyCommands || []
     if (matchReadonlyWhitelist(command, extraReadonly)) return { blocked: false }
-    return { blocked: true, reason: `Bash command blocked in stage "${gateStatus?.stage || '(none)'}"` }
+    return {
+      blocked: true,
+      reason: buildStageHint(stage)
+    }
   }
 
   // execute/quick 阶段 + 主工作区

package/src/worktree.js

@@ -141,12 +141,42 @@ export class WorktreeManager {
       throw new Error(`git worktree add 失败: ${e.stderr || e.message}`);
     }
 
+    // 5.5 自动同步远程最新代码（防止 worktree 基于过时的 commit）
+    try {
+      // 先 fetch origin
+      gitQuiet(worktreePath, 'fetch origin');
+
+      // 尝试 merge origin/main（或 origin/master）到 worktree 分支
+      const defaultBranch = gitQuiet(this.cwd, 'symbolic-ref refs/remotes/origin/HEAD --short')?.replace('origin/', '')
+        || gitQuiet(this.cwd, 'rev-parse --abbrev-ref origin/main') ? 'main'
+        : gitQuiet(this.cwd, 'rev-parse --abbrev-ref origin/master') ? 'master'
+        : null;
+
+      if (defaultBranch) {
+        // 检查 worktree 是否落后于远程
+        const localHead = gitQuiet(worktreePath, 'rev-parse HEAD');
+        const remoteHead = gitQuiet(worktreePath, `rev-parse origin/${defaultBranch}`);
+
+        if (localHead && remoteHead && localHead !== remoteHead) {
+ // 检查是否有共同祖先（避免完全不相关的分支强行 merge）
+          const mergeBase = gitQuiet(worktreePath, `merge-base ${localHead} origin/${defaultBranch}`);
+          if (mergeBase) {
+            git(worktreePath, `merge origin/${defaultBranch} --ff-only`);
+          }
+        }
+      }
+    } catch {
+ // fetch/merge 失败不影响 worktree 创建，只记录警告
+    }
+
     // 6. 写入 meta.json
     const meta = {
       changeName: name,
       branch,
       baseBranch,
       baseHash,
+      // actualBaseHash 记录 fetch+merge 后的实际 HEAD（可能与 baseHash 不同）
+      actualBaseHash: gitQuiet(worktreePath, 'rev-parse HEAD') || baseHash,
       createdAt: new Date().toISOString(),
       worktreePath,
     };