sillyspec 3.16.0 → 3.16.2

package/test/stage-definitions.test.mjs

@@ -0,0 +1,43 @@
+import assert from 'node:assert/strict'
+import { stageRegistry } from '../src/stages/index.js'
+import { buildPlanSteps } from '../src/stages/plan.js'
+import { buildExecuteSteps } from '../src/stages/execute.js'
+
+const stageSteps = {
+  brainstorm: stageRegistry.brainstorm.steps,
+  propose: stageRegistry.propose.steps,
+  scan: stageRegistry.scan.steps,
+  quick: stageRegistry.quick.steps,
+  archive: stageRegistry.archive.steps,
+  verify: stageRegistry.verify.steps,
+  plan: buildPlanSteps(null),
+  execute: buildExecuteSteps(null),
+}
+
+function names(stage) {
+  return stageSteps[stage].map(step => step.name)
+}
+
+function assertContains(stage, expectedNames) {
+  const actual = names(stage)
+  for (const name of expectedNames) {
+    assert.ok(actual.includes(name), `${stage} should include step "${name}". Actual: ${actual.join(', ')}`)
+  }
+}
+
+assert.equal(stageSteps.brainstorm.length, 11, 'brainstorm should expose design self-review and final confirmation separately')
+assertContains('brainstorm', ['写设计文档并自审', '用户确认并生成规范文件'])
+
+assert.equal(stageSteps.propose.length, 7, 'propose should expose generation and self-check separately')
+assertContains('propose', ['生成规范文件', '自检门控', '展示并更新进度'])
+
+assert.equal(stageSteps.scan.length, 10, 'scan base definition should stay at 10 steps before per-project expansion')
+assertContains('scan', ['构建扫描项目列表', '生成本地配置', '生成模块映射'])
+
+assert.equal(stageSteps.quick.length, 3, 'quick should remain a short auxiliary workflow')
+assertContains('quick', ['理解任务', '实现并验证', '暂存和更新记录'])
+
+assert.equal(stageSteps.archive.length, 5, 'archive should keep its five-step lifecycle')
+assertContains('archive', ['extract-module-impact', 'sync-module-docs', '确认归档'])
+
+console.log('✅ stage definition regression checks passed')

package/test/worktree-guard.test.mjs

@@ -0,0 +1,78 @@
+import assert from 'node:assert/strict'
+import { mkdirSync, rmSync, writeFileSync } from 'node:fs'
+import { join } from 'node:path'
+import { tmpdir } from 'node:os'
+import { shouldBlock } from '../src/hooks/worktree-guard.js'
+
+const root = join(tmpdir(), `sillyspec-guard-test-${Date.now()}`)
+const changeName = '2026-06-04-guard-test'
+const runtimeDir = join(root, '.sillyspec', '.runtime')
+const registeredWorktree = join(runtimeDir, 'worktrees', changeName)
+const unregisteredWorktree = join(runtimeDir, 'worktrees', 'other-change')
+
+mkdirSync(registeredWorktree, { recursive: true })
+mkdirSync(unregisteredWorktree, { recursive: true })
+writeFileSync(join(runtimeDir, 'gate-status.json'), JSON.stringify({
+  stage: 'execute',
+  changes: [changeName],
+  updatedAt: new Date().toISOString(),
+}, null, 2))
+writeFileSync(join(registeredWorktree, 'meta.json'), JSON.stringify({
+  changeName,
+  worktreePath: registeredWorktree,
+  mode: 'worktree',
+}, null, 2))
+
+try {
+  assert.equal(
+    shouldBlock({ tool: 'Write', filePath: join(registeredWorktree, 'src', 'ok.js'), cwd: root }).blocked,
+    false,
+    'registered worktree writes should be allowed'
+  )
+
+  assert.equal(
+    shouldBlock({ tool: 'Bash', command: 'npm run build', cwd: registeredWorktree }).blocked,
+    false,
+    'bash commands from a registered worktree cwd should be allowed'
+  )
+
+  assert.equal(
+    shouldBlock({ tool: 'Write', filePath: join(unregisteredWorktree, 'src', 'blocked.js'), cwd: root }).blocked,
+    true,
+    'unregistered worktree storage writes should be blocked'
+  )
+
+  assert.equal(
+    shouldBlock({ tool: 'Write', filePath: join(root, '.sillyspec', 'docs', 'note.md'), cwd: root }).blocked,
+    false,
+    'ordinary .sillyspec docs should remain writable'
+  )
+
+  rmSync(join(runtimeDir, 'gate-status.json'), { force: true })
+  writeFileSync(join(root, '.sillyspec', 'local.yaml'), [
+    'worktreeHook:',
+    '  readonlyCommands:',
+    '    - custom-read',
+    '',
+  ].join('\n'))
+  assert.equal(
+    shouldBlock({ tool: 'Bash', command: 'custom-read status', cwd: root }).blocked,
+    false,
+    '.sillyspec/local.yaml readonlyCommands should extend the bash whitelist'
+  )
+
+  writeFileSync(join(runtimeDir, 'gate-status.json'), JSON.stringify({
+    stage: 'quick',
+    changes: [changeName],
+    updatedAt: new Date().toISOString(),
+  }, null, 2))
+  assert.equal(
+    shouldBlock({ tool: 'Write', filePath: join(root, 'src', 'quick.js'), cwd: root }).blocked,
+    false,
+    'quick writes should still be allowed in the main workspace'
+  )
+
+  console.log('✅ worktree guard regression checks passed')
+} finally {
+  rmSync(root, { recursive: true, force: true })
+}