sillyspec 3.23.1 → 3.23.3

package/package.json

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

package/src/run.js

@@ -315,7 +315,7 @@ function parseModuleMapSimple(content) {
 /**
  * quick 完成审计：对比 baseline 与实际变更
  */
-async function auditQuickCompletion(cwd, guard, options = {}) {
+export async function auditQuickCompletion(cwd, guard, options = {}) {
   const { baselineFiles, allowedFiles = [], allowNew = false, forceBaseline = false } = guard
   const { isConfirm } = options
   const result = { status: 'safe', reasons: [], changedFiles: [], newFiles: [], deletedFiles: [], baselineHit: [] }
@@ -326,6 +326,10 @@ async function auditQuickCompletion(cwd, guard, options = {}) {
     const currentEntries = gitStatus.trim().split('\n').filter(Boolean)
 
     const normalizeGitPath = (p) => p.replace(/\\/g, '/')
+    // step1 启动时记录的全量脏文件 = 预存改动（非本次 quick 产生）。审计必须排除它们，
+    // 否则脏工作区下预存文件持续留在 git status → 命中 baselineFiles → 误判「覆盖 baseline」
+    // → 永远 blocked（--force-baseline 也救不回来，因为 status 判定看 baselineHit 数组）。
+    const baselineFilesSet = new Set((baselineFiles || []).map(f => normalizeGitPath(f)))
     const isQuickMetadata = (p) => {
       const file = normalizeGitPath(p)
       return file.startsWith('.sillyspec/quicklog/')
@@ -355,6 +359,9 @@ async function auditQuickCompletion(cwd, guard, options = {}) {
       const file = normalizeGitPath(entry.slice(3).trim())
       if (!file || file.startsWith('??. ')) continue
 
+      // 预存脏文件：step1 baseline 已记录，非本次 quick 产生，跳过审计
+      if (baselineFilesSet.has(file)) continue
+
       result.changedFiles.push(file)
       if (status === 'D' || status === ' D') result.deletedFiles.push(file)
       if (status === '??') result.newFiles.push(file)
@@ -404,10 +411,11 @@ async function auditQuickCompletion(cwd, guard, options = {}) {
       }
     }
 
-    // 判定结果
-    if (result.baselineHit.length > 0 || result.deletedFiles.length > 0 || result.reasons.some(r => r.startsWith('危险') || r.startsWith('删除'))) {
+    // 判定结果（force-baseline 降级 baselineHit → 非 blocked；allow-new 降级新增文件 → 非 warning。
+    // reasons 文案本就受这两个 flag 控制，但原判定直接看数组长度，致 flag 对 status 失效。）
+    if ((!forceBaseline && result.baselineHit.length > 0) || result.deletedFiles.length > 0 || result.reasons.some(r => r.startsWith('危险') || r.startsWith('删除'))) {
       result.status = 'blocked'
-    } else if (result.newFiles.length > 0 || (allowedFiles.length > 0 && result.reasons.some(r => r.startsWith('超出')))) {
+    } else if ((!allowNew && result.newFiles.length > 0) || (allowedFiles.length > 0 && result.reasons.some(r => r.startsWith('超出')))) {
       result.status = 'warning'
     }
 

package/src/stages/plan-postcheck.js

@@ -80,6 +80,10 @@ function parseAllowedPaths(content) {
  * @returns {boolean}
  */
 function hasAcceptanceCriteria(content) {
+  // 新模板把 goal/implementation/acceptance/verify/constraints 放在 frontmatter
+  // （acceptance: 作为 fm 字段），老格式或人工写的卡片可能用 body 章节。
+  // 两种都认，避免「模板格式」与「postcheck 判定」不一致导致卡片过不了校验。
+  if (/^acceptance:/m.test(content)) return true
   return /##\s*验收标准/.test(content) || /##\s*Acceptance/.test(content)
 }
 
@@ -89,6 +93,8 @@ function hasAcceptanceCriteria(content) {
  * @returns {boolean}
  */
 function hasTddOrVerify(content) {
+  // 同 hasAcceptanceCriteria：新模板用 frontmatter 的 verify: 字段，老格式用 body 章节。
+  if (/^verify:/m.test(content)) return true
   return /##\s*TDD/.test(content) || /##\s*验证/.test(content) || /##\s*Verify/.test(content)
 }
 
@@ -230,13 +236,13 @@ export function validateBlueprintConsistency(changeDir) {
     taskInfo.set(taskId, { dependsOn, allowedPaths, hasAcceptance, hasTdd, file })
 
     if (allowedPaths.length === 0) {
-      errors.push(`${taskId} (${file}): 缺少 allowed_paths`)
+      errors.push(`${taskId} (${file}): frontmatter 缺少 allowed_paths（需非空数组，列出本 task 真实改动的源文件；回归类 task 无源码改动时填被验证的关键入口文件）`)
     }
     if (!hasAcceptance) {
-      errors.push(`${taskId} (${file}): 缺少「验收标准」章节`)
+      errors.push(`${taskId} (${file}): 缺少验收标准——frontmatter 需有 acceptance: 列表字段，或 body 需有「## 验收标准」/「## Acceptance」章节`)
     }
     if (!hasTdd) {
-      warnings.push(`${taskId} (${file}): 缺少 TDD/验证步骤章节`)
+      warnings.push(`${taskId} (${file}): 缺少验证步骤——frontmatter 需有 verify: 字段，或 body 需有「## TDD」/「## 验证」/「## Verify」章节`)
     }
 
     for (const p of allowedPaths) {

package/src/stages/plan.js

@@ -447,6 +447,10 @@ TaskCard 格式规则（必须严格遵守）：
 - 填写后 plan-postcheck 会做硬对账：consumer 的每个 expects_from[provider].needs 字段必须在对应 provider 的 provides.fields 里，否则 plan 阶段阻断（不进入 execute）
 - 不要把内部实现字段塞进 provides；只暴露给其他 task 的对外契约形状
 - 如果存在 decisions.md，无法覆盖的 D-xxx@vN 在 constraints 中标注
+- **保存前格式自检**（plan-postcheck 会硬校验，不通过到 Step 4 会批量报错，先在这里逐条自查）：
+  - frontmatter 字段齐全：id、title、title_zh、author、created_at、priority、depends_on、blocks、allowed_paths、goal、implementation、acceptance、verify、constraints
+  - allowed_paths 非空（至少一个真实源文件路径；回归类 task 无源码改动时填被验证的关键入口文件）
+  - acceptance 与 verify 都在 frontmatter 内（漏写会被 plan-postcheck 报「缺少验收标准」/「缺少验证步骤」）
 - 写完后用 Write tool 保存到文件
 \`\`\``
   }).join('\n\n')
@@ -478,8 +482,7 @@ ${subagentPrompts}
 
 ## 验收（生成后自查，不另开步骤）
 - 每个 task-N.md 文件存在且非空
-- frontmatter 包含：id、title、author、created_at、priority、depends_on、blocks、allowed_paths
-- body 包含：goal、implementation、acceptance、verify、constraints
+- frontmatter 包含：id、title、author、created_at、priority、depends_on、blocks、allowed_paths、goal、implementation、acceptance、verify、constraints
 - 每个 task 总长度 20~40 行
 - **一致性自查**：
   - allowed_paths 有无冲突

package/src/worktree.js

@@ -1078,8 +1078,12 @@ export class WorktreeManager {
       if (!status) {
         return gitQuiet(worktreePath, 'rev-parse HEAD');
       }
+      // --no-verify：baseline 是锚点不是交付物，只是把主仓库 dirty 文件快照到 worktree
+      // 分支上以便区分「前置 baseline」与「子代理新增改动」。它不该触发项目 pre-commit
+      // hook（如 ruff format），否则主仓库 dirty 文件中任一不达标的会被 hook reformat
+      // 致 commit 失败 → worktree 创建失败 → execute 无法启动。
       execSync(
-        `git commit -m "sillyspec: baseline checkpoint for ${changeName}"`,
+        `git commit --no-verify -m "sillyspec: baseline checkpoint for ${changeName}"`,
         { cwd: worktreePath, encoding: 'utf8', stdio: ['pipe','pipe','pipe'], env }
       );
       const hash = git(worktreePath, 'rev-parse HEAD');