sillyspec 3.24.4 → 3.24.6

package/src/hooks/worktree-guard.js

@@ -11,6 +11,9 @@
 
 import { existsSync, readFileSync, readdirSync } from 'fs'
 import path from 'path'
+import { execFileSync } from 'child_process'
+import { createRequire } from 'module'
+import { pathToFileURL } from 'url'
 
 // ── 常量 ──
 
@@ -236,6 +239,35 @@ function readGateStatus(cwd) {
   }
 }
 
+/**
+ * 同步只读查询 sillyspec.db 第一行第一列（不依赖外部 sqlite3 CLI）。
+ * sql.js 的 init 是 async，故用子进程跑（execFileSync 等待）；sql.js 模块经 hook 进程的
+ * require.resolve 解析出绝对路径传入，避免子进程 cwd（项目目录）找不到 sql.js。
+ * 全平台一致（node 是 SillySpec 自身运行时）。返回字符串值或 null。
+ */
+function queryDbFirstCell(cwd, sql) {
+  const dbPath = path.join(cwd, '.sillyspec', '.runtime', 'sillyspec.db')
+  if (!existsSync(dbPath)) return null
+  let sqlJsPath
+  try { sqlJsPath = createRequire(import.meta.url).resolve('sql.js') } catch { return null }
+  // Windows 上 dynamic import 不接受裸 "C:\..." 路径，必须转 file:// URL（全平台合法）
+  const libUrl = pathToFileURL(sqlJsPath).href
+  const script =
+    "const fs=require('fs');" +
+    "const dbPath=" + JSON.stringify(dbPath) + ",sql=" + JSON.stringify(sql) + ",lib=" + JSON.stringify(libUrl) + ";" +
+    "if(!fs.existsSync(dbPath))process.exit(0);" +
+    "import(lib).then(async m=>{const SQL=await m.default();" +
+    "const db=new SQL.Database(fs.readFileSync(dbPath));const r=db.exec(sql);" +
+    "if(r.length&&r[0].values.length)process.stdout.write(String(r[0].values[0][0]??''));db.close()}).catch(()=>{});"
+  try {
+    return execFileSync(process.execPath, ['-e', script], {
+      encoding: 'utf8', timeout: 5000, stdio: ['ignore', 'pipe', 'pipe'],
+    }).trim() || null
+  } catch { return null }
+}
+
+export { queryDbFirstCell as _queryDbFirstCellForTest };
+
 /**
  * 从 sillyspec.db 读取 currentStage
  * 优先级：gate-status.json > sillyspec.db
@@ -247,19 +279,9 @@ function readCurrentStage(cwd) {
   const gateStatus = readGateStatus(cwd)
   if (gateStatus && gateStatus.stage) return gateStatus.stage
 
-  // 2. 从 sillyspec.db 读取（通过 sqlite3 CLI 同步调用）
-  const dbPath = path.join(cwd, '.sillyspec', '.runtime', 'sillyspec.db')
-  if (!existsSync(dbPath)) return null
-  try {
-    const { execSync } = require('child_process')
-    const result = execSync(
-      `sqlite3 "${dbPath}" "SELECT current_stage FROM changes WHERE status='active' AND current_stage IN ('execute','quick') ORDER BY last_active DESC LIMIT 1"`,
-      { encoding: 'utf8', timeout: 2000 }
-    ).trim()
-    return result || null
-  } catch { /* sqlite3 CLI 不可用或查询失败 */ }
-
-  return null
+  // 2. 从 sillyspec.db 读取（node + sql.js 子进程，不依赖外部 sqlite3 CLI——Windows 默认没有）
+  const v = queryDbFirstCell(cwd, "SELECT current_stage FROM changes WHERE status='active' AND current_stage IN ('execute','quick') ORDER BY last_active DESC LIMIT 1")
+  return v || null
 }
 
 /**
@@ -272,19 +294,8 @@ function isNoWorktreeMode(cwd) {
   const gateStatus = readGateStatus(cwd)
   if (gateStatus && gateStatus.noWorktree) return true
 
-  // 2. 从 sillyspec.db 读取
-  const dbPath = path.join(cwd, '.sillyspec', '.runtime', 'sillyspec.db')
-  if (!existsSync(dbPath)) return false
-  try {
-    const { execSync } = require('child_process')
-    const result = execSync(
-      `sqlite3 "${dbPath}" "SELECT no_worktree FROM changes WHERE status='active' AND current_stage IN ('execute','quick') LIMIT 1"`,
-      { encoding: 'utf8', timeout: 2000 }
-    ).trim()
-    return result === '1'
-  } catch { /* sqlite3 CLI 不可用或查询失败 */ }
-
-  return false
+  // 2. 从 sillyspec.db 读取（node + sql.js，同 readCurrentStage）
+  return queryDbFirstCell(cwd, "SELECT no_worktree FROM changes WHERE status='active' AND current_stage IN ('execute','quick') LIMIT 1") === '1'
 }
 
 /**

package/src/run.js

@@ -632,11 +632,26 @@ async function getStageSteps(stageName, cwd, progress, specDir = null) {
   if (stageName === 'execute') {
     const changeDir = resolveChangeDir(cwd, progress, specDir)
     let planFile = null
+    let worktreePath = null
     if (changeDir) {
       const p = join(changeDir, 'plan.md')
       if (existsSync(p)) planFile = p
+      // 自动检测 worktree 路径，注入 Wave prompt 的 workdir 指令
+      // 修复：之前未传 worktreePath 给 buildExecuteSteps，导致 Wave prompt 缺失工作目录指令，
+      // 子代理可能把文件写到主工作区而非 worktree 内，破坏隔离。
+      try {
+        const changeName = basename(changeDir)
+        const { WorktreeManager } = await import('./worktree.js')
+        const wm = new WorktreeManager({ cwd })
+        const meta = wm.getMeta(changeName)
+        if (meta?.worktreePath && existsSync(meta.worktreePath)) {
+          worktreePath = meta.worktreePath
+        }
+      } catch {
+        // 无 worktree meta 不阻断——可能是首次启动或 in-place 模式
+      }
     }
-    return buildExecuteSteps(planFile)
+    return buildExecuteSteps(planFile, { worktreePath })
   }
   if (stageName === 'plan') {
     const changeDir = resolveChangeDir(cwd, progress, specDir)
@@ -879,7 +894,12 @@ async function outputStep(stageName, stepIndex, steps, cwd, changeName, dbProjec
     if (platformOpts.workspaceId) {
       platformDirectives.push(`workspace_id: ${platformOpts.workspaceId}`)
     }
-    promptText = platformDirectives.join('\n') + '\n\n' + promptText
+    // 平台 directives 仅 step0 前置（对齐 persona 仅 step0 策略）：路径列表/Write 规则/
+    // 占位符映射一次建立即可；后续 step 靠 agent 跨步记忆 + 每步 changeDir header +
+    // footer 平台规则（下方 L1077+）维持写入约束，避免每步重复注入 ~500 tokens。
+    if (stepIndex === 0) {
+      promptText = platformDirectives.join('\n') + '\n\n' + promptText
+    }
   } else {
     // 常规模式（无平台 specRoot）：占位符替换为 cwd/.sillyspec 下对应路径
     // 让用 {SPEC_ROOT}/{DOCS_ROOT} 等占位符的 prompt（如 quick/scan）在常规模式也写到正确位置
@@ -1048,7 +1068,7 @@ async function outputStep(stageName, stepIndex, steps, cwd, changeName, dbProjec
 
   console.log(promptText)
   console.log(`\n### ⚠️ 铁律`)
-  console.log('- **文档是核心资产，代码是文档的产物。** 没有文档就没有代码——文档是 AI 的记忆，是团队协作的基础，是后续维护的唯一依据。任何代码产出必须先有对应的设计/规范文档支撑。')
+  console.log('- 文档优先：代码产出必须先有对应的设计/规范文档支撑。')
   console.log('- 只做本步骤描述的操作，不得自行扩展或跳过')
   console.log('- 不要回头修改已完成的步骤')
   console.log('- 不要编造不存在的 CLI 子命令')
@@ -1332,7 +1352,10 @@ export async function runCommand(args, cwd, specDir = null) {
   // ── cwd 纠正：向上查找真实项目根 ──
   // 防止多 project 工作区中 cwd 停在子目录（如 backend/）时
   // 状态写入子目录下误建的 .sillyspec，导致状态分裂
-  if (!specDir) {
+  if (!specDir && !existsSync(join(cwd, '.sillyspec-platform.json'))) {
+    // 平台模式（cwd 有 .sillyspec-platform.json 指针）不做 cwd 纠正：指针已明确 specDir，
+    // 向上找 .sillyspec 反而会撞到无关项目（如用户 home 的 .sillyspec），导致 --done 恢复
+    // 读写 ~/.sillyspec-platform.json 出错的 specRoot。
     const resolvedRoot = resolveSpecDir(cwd)
     if (resolvedRoot && resolvedRoot !== join(cwd, '.sillyspec')) {
       const realRoot = dirname(resolvedRoot)
@@ -2106,18 +2129,22 @@ async function runStage(pm, progress, stageName, cwd, changeName, skipApproval =
   }
 
   if (currentIdx === -1) {
-    // 已完成 → 自动重置，重新开始
-    const freshSteps = await getStageSteps(stageName, cwd, progress, platformOpts?.specRoot || null)
-    stageData.steps = freshSteps
-      ? freshSteps.map(s => ({ name: s.name, status: 'pending' }))
-      : []
-    stageData.status = 'in-progress'
-    stageData.startedAt = new Date().toLocaleString('zh-CN', { hour12: false })
-    stageData.completedAt = null
-    await pm._write(cwd, progress, changeName)
-    triggerSync(cwd, changeName, platformOpts)
-    currentIdx = 0
-    console.log(`🔄 ${stageName} 阶段已自动重置，重新开始。\n`)
+    // 所有步骤已 completed/skipped，但阶段未盖到 completed —— 这是上次最后一步完成后、
+    // stage 升级事务未提交的崩溃中间态（正常完成会被上方 stageStatus==='completed' 守卫拦下）。
+    // 旧逻辑无条件清空 steps 为 pending 会丢掉已完成的进度且不可恢复。改为走正式
+    // completeStage：产物齐则补盖完成戳，不齐则给出 actionable 提示，永不静默清空步骤。
+    console.log(`\nℹ️  ${stageName} 所有步骤已完成，但阶段未标记完成（上次可能中断）。尝试补盖完成戳…`)
+    await pm.completeStage(cwd, stageName, changeName)
+    const after = await pm.read(cwd, changeName)
+    if (after?.stages?.[stageName]?.status === 'completed') {
+      console.log(`   ✅ 已补盖完成戳。下一步: sillyspec run <下一阶段>，或 sillyspec run ${stageName} --status 查看。`)
+      process.exit(0)
+    }
+    // completeStage 已打印产物校验失败的明细；这里只补充恢复指引
+    console.error(`\n   ⚠️ 已保留步骤进度（未清空）。修复产物后重跑，或：`)
+    console.error(`   - 重新开始: sillyspec run ${stageName} --reset`)
+    console.error(`   - 强制补盖: sillyspec progress complete-stage ${stageName}${changeName ? ` --change ${changeName}` : ''} --force`)
+    process.exit(1)
   }
 
   // quick 阶段：记录 baselineFiles + 分配 ql-ID（CLI 接管 QUICKLOG 写入）

package/src/stages/doctor.js

@@ -12,6 +12,8 @@ export const definition = {
       name: 'SillySpec 内部检查',
       prompt: `运行 SillySpec 内部检查。逐项执行以下命令并汇总结果：
 
+> ⚠️ Windows 兼容：下列 sqlite3 查询依赖外部 sqlite3 CLI（Windows 默认没有）。若某条 sqlite3 失败，改用 sillyspec 自身命令替代——sillyspec progress show 查看当前变更与各阶段状态（等价于查 changes/stages 表），sillyspec doctor --json 查看 db 健康与活跃变更列表。
+
 ### 1. 目录结构完整性
 \`\`\`bash
 # 检查 .sillyspec/ 及子目录
@@ -21,7 +23,8 @@ done
 # 检查 sillyspec.db（SQLite 权威状态源）
 DB_FILE='.sillyspec/.runtime/sillyspec.db'
 [ -f "$DB_FILE" ] && echo "✅ sillyspec.db 存在" || echo "❌ sillyspec.db 不存在"
-sqlite3 "$DB_FILE" "SELECT count(*) FROM project" 2>/dev/null && echo "✅ sillyspec.db 可查询" || echo "⚠️ sillyspec.db 不可查询"
+# 用 sillyspec 自身命令验证 db 可读（fallback 不依赖外部 sqlite3 CLI，兼容 Windows）
+sqlite3 "$DB_FILE" "SELECT count(*) FROM project" 2>/dev/null && echo "✅ sillyspec.db 可查询" || (sillyspec progress show >/dev/null 2>&1 && echo "✅ sillyspec.db 可查询（经 sillyspec 命令）" || echo "⚠️ sillyspec.db 不可查询")
 \`\`\`
 
 ### 2. 项目配置检查