sillyspec 3.12.4 → 3.12.6

package/package.json

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

package/src/hooks/worktree-guard.js

@@ -14,7 +14,7 @@ import path from 'path'
 
 // ── 常量 ──
 
-const ALLOWED_STAGES = ['execute', 'quick']
+const WORKTREE_STAGES = ['execute'] // 这些阶段必须在 worktree 里
 
 const WORKTREE_SEGMENT = '.sillyspec/.runtime/worktrees/'
 
@@ -48,11 +48,11 @@ const DANGER_PREFIXES = ['sudo', 'rm -rf', 'rm -r', 'rmdir']
 const STAGE_HINTS = {
   '(none)': [
     '没有检测到活跃的 SillySpec 流程。',
-    '你需要先启动一个任务流程才能修改源码：',
+    '你需要先启动一个任务流程才能修改源码（调用对应的 sillyspec skill）：',
     '',
-    '  小改动（≤3 文件）：sillyspec run quick "任务描述"',
-    '  大改动（>3 文件）：sillyspec run brainstorm → plan → execute',
-    '  全自动模式：    sillyspec run auto "任务描述"',
+    '  BUG修复(skill sillyspec-quick)：sillyspec run quick "任务描述"',
+    '  逻辑变更(skill sillyspec-brainstorm)：sillyspec run brainstorm → plan → execute → verify → archive',
+    '  全自动模式(skill sillyspec-auto)：sillyspec run auto "任务描述"',
   ],
   'brainstorm': [
     '当前在 brainstorm（需求分析）阶段，这个阶段只写文档，不写代码。',
@@ -306,12 +306,8 @@ function isSingleCommandReadonly(cmd, extraReadonlyCommands = []) {
     return false
   }
 
-  // sillyspec worktree 命令
-  if (cmdName === 'sillyspec') {
-    const sub = parts[1] || ''
-    if (sub === 'worktree') return true
-    return false
-  }
+  // sillyspec 命令全部放行（CLI 工具本身安全）
+  if (cmdName === 'sillyspec') return true
 
   return false
 }
@@ -414,14 +410,17 @@ export function shouldBlockWrite(filePath, cwd) {
   const effectiveCwd = cwd || process.cwd()
   const stage = readCurrentStage(effectiveCwd) || '(none)'
 
-  if (!ALLOWED_STAGES.includes(stage)) {
+  if (!['execute', 'quick'].includes(stage)) {
     return {
       blocked: true,
       reason: buildStageHint(stage)
     }
   }
 
-  // 3. 位置门禁
+  // quick 阶段直接放行（不要求 worktree）
+  if (stage === 'quick') return { blocked: false }
+
+  // execute 阶段：位置门禁
   if (isInsideWorktree(absPath)) return { blocked: false }
 
   // noWorktree 模式：无隔离环境，禁止源码写入（降级到更严格）
@@ -468,9 +467,8 @@ export function shouldBlockBash(command, cwd) {
 
   // 阶段门禁（使用 fallback 读取）
   const stage = readCurrentStage(effectiveCwd) || '(none)'
-  const stageOk = ALLOWED_STAGES.includes(stage)
 
-  if (!stageOk) {
+  if (!['execute', 'quick'].includes(stage)) {
     // 非 execute/quick 阶段，只允许只读白名单
     const localConfig = loadLocalConfig(effectiveCwd)
     const extraReadonly = localConfig.worktreeHook?.readonlyCommands || localConfig['worktree-hook']?.readonlyCommands || []
@@ -481,7 +479,16 @@ export function shouldBlockBash(command, cwd) {
     }
   }
 
-  // execute/quick 阶段 + 主工作区
+  // quick 阶段直接放行（不要求 worktree）
+  if (stage === 'quick') {
+    // 危险黑名单仍然拦截
+    if (matchDangerBlacklist(command)) {
+      return { blocked: true, reason: `dangerous command blocked: ${command.trim()}` }
+    }
+    return { blocked: false }
+  }
+
+  // execute 阶段 + 主工作区
   const localConfig = loadLocalConfig(effectiveCwd)
   const extraReadonly = localConfig.worktreeHook?.readonlyCommands || localConfig['worktree-hook']?.readonlyCommands || []
 

package/src/index.js

@@ -268,7 +268,8 @@ async function main() {
     case 'worktree': {
       const { WorktreeManager } = await import('./worktree.js');
       const wtSubCmd = filteredArgs[1];
-      const wtName = filteredArgs[2];
+      // 提取第一个非 -- 开头的位置参数作为 wtName
+      const wtName = filteredArgs.slice(2).find(a => !a.startsWith('-'));
       const wm = new WorktreeManager({ cwd: dir });
 
       if (!wtSubCmd || wtSubCmd === 'help' || wtSubCmd === '--help' || wtSubCmd === '-h') {
@@ -338,6 +339,11 @@ SillySpec worktree — git worktree 隔离管理
           } else {
             console.log(`✅ 已应用 ${result.changedFiles.length} 个文件变更`);
           }
+          if (result.warnings && result.warnings.length > 0) {
+            for (const w of result.warnings) {
+              console.log(`⚠️  ${w}`);
+            }
+          }
           break;
         }
         case 'list': {

package/src/run.js

@@ -195,6 +195,28 @@ async function outputStep(stageName, stepIndex, steps, cwd, changeName, dbProjec
   if (projectName && promptText.includes('<project>')) {
     promptText = promptText.replace(/<project>/g, projectName)
   }
+  // 替换 <git-user> 占位符
+  if (promptText.includes('<git-user>')) {
+    const { execSync } = await import('child_process')
+    try {
+      const gitUser = execSync('git config user.name', { cwd, encoding: 'utf8', timeout: 5000 }).trim()
+      promptText = promptText.replace(/<git-user>/g, gitUser)
+    } catch {
+      promptText = promptText.replace(/<git-user>/g, 'unknown')
+    }
+  }
+  // 替换时间戳占位符
+  const now = new Date()
+  const nowDatetime = now.getFullYear() + '-' + String(now.getMonth()+1).padStart(2,'0') + '-' + String(now.getDate()).padStart(2,'0') + ' ' + String(now.getHours()).padStart(2,'0') + ':' + String(now.getMinutes()).padStart(2,'0') + ':' + String(now.getSeconds()).padStart(2,'0')
+  const nowTimestamp = now.getFullYear() + String(now.getMonth()+1).padStart(2,'0') + String(now.getDate()).padStart(2,'0') + '-' + String(now.getHours()).padStart(2,'0') + String(now.getMinutes()).padStart(2,'0') + String(now.getSeconds()).padStart(2,'0')
+  const nowDate = now.getFullYear() + '-' + String(now.getMonth()+1).padStart(2,'0') + '-' + String(now.getDate()).padStart(2,'0')
+  promptText = promptText.replace(/<now-datetime>/g, nowDatetime)
+  promptText = promptText.replace(/<now-timestamp>/g, nowTimestamp)
+  promptText = promptText.replace(/<now-date>/g, nowDate)
+  // 替换 <change-name> 占位符
+  if (changeName && promptText.includes('<change-name>')) {
+    promptText = promptText.replace(/<change-name>/g, changeName)
+  }
   console.log(promptText)
   console.log(`\n### ⚠️ 铁律`)
   console.log('- **文档是核心资产，代码是文档的产物。** 没有文档就没有代码——文档是 AI 的记忆，是团队协作的基础，是后续维护的唯一依据。任何代码产出必须先有对应的设计/规范文档支撑。')

package/src/stages/archive.js

@@ -46,8 +46,8 @@ export const definition = {
 \`\`\`markdown
 # 模块影响分析
 
-author: <git用户名>
-created_at: <YYYY-MM-DD HH:mm:ss>
+author: <git-user>
+created_at: <now-datetime>
 
 ## 变更：<change-name>
 
@@ -90,7 +90,7 @@ module-impact.md 路径 + 影响模块数量 + 未匹配文件数量`,
       - 更新：只改相关章节（当前设计/对外接口/依赖关系等），保持其他章节不变
       - 正文重写为当前状态，不追加历史
       - 底部"变更索引"追加一行：\`| <日期> | <变更名> | <一句话摘要> |\`
-   d. 更新头部元数据：\`> 最后更新：YYYY-MM-DD\`、\`> 最近变更：<change-name>\`
+   d. 更新头部元数据：\`> 最后更新：<now-date>\`、\`> 最近变更：<change-name>\`
 4. 展示所有模块文档的更新内容（diff 摘要），请用户确认
 5. 用户确认后，写入 \`.sillyspec/docs/<project>/modules/*.md\`
 6. 用户拒绝时，不写入模块文档，但提示"module-impact.md 已保留，可稍后手动同步"
@@ -100,7 +100,7 @@ module-impact.md 路径 + 影响模块数量 + 未匹配文件数量`,
 \`\`\`markdown
 # <module-name>
 
-> 最后更新：YYYY-MM-DD
+> 最后更新：<now-date>
 > 最近变更：<change-name>
 > 模块路径：<glob patterns>
 

package/src/stages/quick.js

@@ -24,10 +24,10 @@ export const definition = {
 
 ### 创建任务记录（必须执行）
 理解完任务后，立即创建记录文件：
-1. \`git config user.name\` 获取用户名
-2. 无 \`--change\`：创建 .sillyspec/quicklog/QUICKLOG-<git用户名>.md\`（已存在则追加），写入：
+1. 使用预注入的 git 用户名：\`<git-user>\`
+2. 无 \`--change\`：创建 .sillyspec/quicklog/QUICKLOG-\`<git-user>\`.md\`（已存在则追加），写入：
    \`\`\`
-   ## YYYY-MM-DD HH:mm:ss — <一句话任务描述>
+   ## <now-datetime> — <一句话任务描述>
    状态：进行中
    文件：<预估要改的文件>
    \`\`\`
@@ -40,33 +40,12 @@ export const definition = {
       outputHint: '任务理解',
       optional: false
     },
-    {
-      name: '创建 worktree',
-      prompt: `为本次 quick 任务创建隔离的 git worktree。
-
-### 操作
-1. 确定变更名（change name）：
-   - 如携带 \`--change <变更名>\`，使用该变更名
-   - 否则，生成临时变更名：\`quick-<当前时间戳 YYYYMMDD-HHmmss>\`
-2. 运行 \`sillyspec worktree create <变更名>\`
-3. 记录输出的 worktree 路径（后续步骤需要使用）
-4. 如果创建失败 → 报错并停止（不要在无隔离状态下继续）
-
-### 输出
-worktree 路径 + 变更名 + 分支名`,
-      outputHint: 'worktree 路径',
-      optional: false
-    },
     {
       name: '实现并验证',
-      prompt: `实现任务。
-
-### 工作目录
-你必须在上一步记录的 worktree 路径中工作。
-不要在主工作区修改源码文件。所有代码变更只在 worktree 中进行。
+      prompt: `直接在主工作区实现任务。
 
 ### 操作
-1. 先读后写：调用已有方法前 \`cat\` 源文件确认签名，\`grep\` 确认方法存在（在 worktree 中读取）
+1. 先读后写：调用已有方法前 \`cat\` 源文件确认签名，\`grep\` 确认方法存在
 2. 写代码完成任务
 3. 如涉及逻辑变更，建议写单元测试验证（不强制，纯配置/文档/小改动可跳过）
 4. **不要编译！** 除非用户明确要求或改动量很大
@@ -81,24 +60,6 @@ worktree 路径 + 变更名 + 分支名`,
       outputHint: '实现摘要',
       optional: false
     },
-    {
-      name: 'apply 并 cleanup',
-      prompt: `将 worktree 中的变更应用到主工作区并清理。
-
-### 操作
-1. 运行 \`sillyspec worktree apply --check-only <变更名>\`
-2. 展示 diff 摘要（文件列表 + 变更统计）
-3. 展示检查结果（是否通过文件清单校验）
-4. 用户确认后运行 \`sillyspec worktree apply <变更名>\`
-5. apply 成功 → 自动 cleanup，进入下一步
-6. apply 失败 → 展示错误详情，用户选择重试或手动处理
-7. 如果用户不想 apply → 运行 \`sillyspec worktree cleanup <变更名>\` 丢弃变更
-
-### 输出
-apply 结果 + 下一步建议`,
-      outputHint: 'apply 结果',
-      optional: false
-    },
     {
       name: '暂存和更新记录',
       prompt: `Git 暂存并更新任务记录。

package/src/stages/scan.js

@@ -134,7 +134,7 @@ export const definition = {
 
 ### 每个子代理的共同要求
 - **上下文注入**：主 agent 在启动子代理前，必须将以下信息拼入子代理 prompt：
-  - 项目名（<project>）
+  - 项目名（直接用本次 prompt 中的实际项目名）
   - 断点续扫步骤列出的缺失文档列表（哪些要生成、哪些跳过）
   - 环境探测结果摘要（构建工具、语言框架、关键依赖）
   - _env-detect.md 内容（如存在，直接贴入）
@@ -265,7 +265,7 @@ _module-map.yaml 生成结果（已存在/已生成/模块列表）`,
 3. 按以下模板写入目标文件：
 
 # <module-name>
-> 最后更新：YYYY-MM-DD
+> 最后更新：<now-date>
 > 最近变更：scan（初始生成）
 > 模块路径：<glob patterns>
 

package/src/worktree-apply.js

@@ -251,8 +251,13 @@ export function applyWorktree(changeName, { cwd, checkOnly = false } = {}) {
 
     result.ok = true;
 
-    // --- 8. 成功后自动 cleanup ---
-    wm.cleanup(changeName);
+    // --- 8. 成功后自动 cleanup（失败不影响整体结果） ---
+    try {
+      wm.cleanup(changeName);
+    } catch (cleanupErr) {
+      result.warnings = result.warnings || [];
+      result.warnings.push(`cleanup 失败（不影响应用结果）: ${cleanupErr.message}`);
+    }
 
   } catch (e) {
     result.errors.push(`patch 生成/应用异常: ${e.message}`);

package/src/worktree.js

@@ -106,7 +106,13 @@ export class WorktreeManager {
 
     // 1. 检查 worktree 是否已存在
     if (existsSync(worktreePath)) {
-      throw new Error(`worktree already exists: ${name}. Run cleanup first.`);
+      // 目录在但 meta.json 不存在（幽灵状态），自动清理
+      if (!this.getMeta(name)) {
+        console.log(`⚠️  检测到幽灵 worktree 目录（无 meta.json），自动清理...`);
+        try { rmSync(worktreePath, { recursive: true, force: true }); } catch {}
+      } else {
+        throw new Error(`worktree already exists: ${name}. Run cleanup first.`);
+      }
     }
 
     // 2. 检查分支是否已存在
@@ -223,26 +229,27 @@ export class WorktreeManager {
   cleanup(changeName) {
     const name = validateChangeName(changeName);
     const meta = this.getMeta(name);
+    const worktreePath = this.getWorktreePath(name);
 
-    if (!meta) {
-      throw new Error(`worktree not found: ${name}。meta.json 不存在，可能已被清理或从未创建。`);
+    if (!meta && !existsSync(worktreePath)) {
+      throw new Error(`worktree not found: ${name}。meta.json 不存在，目录也不存在，可能已被清理或从未创建。`);
     }
 
-    const worktreePath = meta.worktreePath || this.getWorktreePath(name);
-    const branch = meta.branch || BRANCH_PREFIX + name;
-
-    // 2. 移除 git worktree
+    // 1. 尝试 git worktree remove
     try {
       git(this.cwd, `worktree remove ${worktreePath} --force`);
     } catch {
       // git worktree remove 失败，尝试直接删除目录
-      try {
-        if (existsSync(worktreePath)) {
-          rmSync(worktreePath, { recursive: true, force: true });
-        }
-      } catch (e) {
-        throw new Error(`清理 worktree 目录失败: ${e.message}`);
+    }
+    const branch = (meta && meta.branch) || BRANCH_PREFIX + name;
+
+    // 2. 确保目录已删除
+    try {
+      if (existsSync(worktreePath)) {
+        rmSync(worktreePath, { recursive: true, force: true });
       }
+    } catch (e) {
+      throw new Error(`清理 worktree 目录失败: ${e.message}`);
     }
 
     // 3. 删除分支（忽略分支不存在的错误）