gsd-lite 0.7.4 → 0.7.6

package/.claude-plugin/marketplace.json

@@ -13,7 +13,7 @@
       "name": "gsd",
       "source": "./",
       "description": "AI orchestration tool — GSD management shell + Superpowers quality core. 5 commands, 4 agents, 5 workflows, MCP server, context monitoring.",
-      "version": "0.7.4",
+      "version": "0.7.6",
       "keywords": [
         "orchestration",
         "mcp",

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "gsd",
-  "version": "0.7.4",
+  "version": "0.7.6",
   "description": "AI orchestration tool for Claude Code — GSD management shell + Superpowers quality core",
   "author": {
     "name": "sdsrss",

package/README.md

@@ -291,7 +291,7 @@ gsd-lite/
 │   ├── gsd-session-stop.cjs   # Graceful shutdown with crash markers
 │   ├── gsd-statusline.cjs     # StatusLine display (composite-aware)
 │   └── lib/                   # Shared hook utilities (gsd-finder, composite statusline, semver)
-├── tests/                  # 966 tests (unit + simulation + E2E integration)
+├── tests/                  # 972 tests (unit + simulation + E2E integration)
 ├── cli.js                  # Install/uninstall CLI entry
 ├── install.js              # Installation script (plugin-aware, idempotent)
 └── uninstall.js            # Uninstall script
@@ -300,7 +300,7 @@ gsd-lite/
 ## Testing
 
 ```bash
-npm test                    # Run all 966 tests
+npm test                    # Run all 972 tests
 npm run test:coverage       # Tests + coverage report (94%+ lines, 83%+ branches)
 npm run lint                # Biome lint
 node --test tests/file.js   # Run a single test file

package/hooks/lib/statusline-composite.cjs

@@ -63,33 +63,17 @@ function runChainCLI(args) {
 }
 
 /**
- * Register GSD as a provider in the composite statusline registry.
- * Prefers code-graph's statusline-chain.js CLI when available; falls back to
- * writing the cache registry directly for older code-graph versions.
- * Idempotent: updates existing entry or inserts before code-graph.
- * @param {string} statuslineScriptPath - Absolute path to gsd-statusline.cjs
- * @returns {boolean} true if registered/updated
+ * Rewrite a single registry file to its desired state: exactly one canonical
+ * `{id: 'gsd', ...}` entry with the given command, dropping every other entry
+ * whose command references gsd-statusline (e.g. ghost `_previous` entries
+ * left by code-graph's composite-takeover). Canonical entry is placed before
+ * `code-graph` for display priority.
+ *
+ * Returns true if the registry is now in the desired state (including
+ * idempotent no-op), false if the file can't be parsed as an array.
  */
-function registerProvider(statuslineScriptPath) {
-  const command = `node ${JSON.stringify(statuslineScriptPath)}`;
-  if (runChainCLI(['register', 'gsd', command, '--stdin'])) return true;
-
-  let registryPath = findCompositeRegistry();
-
-  // If composite statusLine is configured but registry file is missing,
-  // create it if the parent directory exists (e.g., code-graph installed
-  // but registry was deleted or not yet created).
-  if (!registryPath) {
-    for (const candidate of REGISTRY_PATHS) {
-      const dir = path.dirname(candidate);
-      if (fs.existsSync(dir)) {
-        registryPath = candidate;
-        break;
-      }
-    }
-    if (!registryPath) return false;
-  }
-
+function normalizeRegistryFile(registryPath, canonicalCommand) {
+  const canonical = { id: 'gsd', command: canonicalCommand, needsStdin: true };
   try {
     let registry;
     try {
@@ -99,24 +83,24 @@ function registerProvider(statuslineScriptPath) {
     }
     if (!Array.isArray(registry)) return false;
 
-    const provider = { id: 'gsd', command, needsStdin: true };
+    // Drop every entry pointing at gsd-statusline, regardless of id. This
+    // catches the canonical `gsd` slot AND ghosts like `_previous` that
+    // code-graph's id-scoped chain CLI `register` can't see.
+    const nonGsd = registry.filter(
+      e => !(e.command || '').includes('gsd-statusline'),
+    );
 
-    // Find existing GSD entry (by id or command)
-    const idx = registry.findIndex(p =>
-      p.id === 'gsd' || p.command?.includes('gsd-statusline'));
+    const cgIdx = nonGsd.findIndex(e => e.id === 'code-graph');
+    if (cgIdx >= 0) nonGsd.splice(cgIdx, 0, canonical);
+    else nonGsd.unshift(canonical);
 
-    if (idx >= 0) {
-      registry[idx] = provider;
-    } else {
-      // Insert before code-graph for display priority
-      const cgIdx = registry.findIndex(p => p.id === 'code-graph');
-      if (cgIdx >= 0) registry.splice(cgIdx, 0, provider);
-      else registry.unshift(provider);
-    }
+    // Skip write if already in desired state (idempotent re-install).
+    const before = JSON.stringify(registry);
+    const after = JSON.stringify(nonGsd);
+    if (before === after) return true;
 
-    // Atomic write
     const tmp = registryPath + `.${process.pid}-${Date.now()}.tmp`;
-    fs.writeFileSync(tmp, JSON.stringify(registry, null, 2) + '\n');
+    fs.writeFileSync(tmp, JSON.stringify(nonGsd, null, 2) + '\n');
     fs.renameSync(tmp, registryPath);
     return true;
   } catch {
@@ -124,6 +108,37 @@ function registerProvider(statuslineScriptPath) {
   }
 }
 
+/**
+ * Register GSD as a provider in the composite statusline registry.
+ *
+ * Calls code-graph's statusline-chain.js CLI when available, THEN post-scrubs
+ * every known registry path. The CLI's `register gsd <cmd>` is id-scoped and
+ * silently leaves ghost entries (e.g. `_previous` whose command is our
+ * gsd-statusline but whose id isn't `gsd`) in place — that caused
+ * double-rendering after upgrades where code-graph had previously promoted a
+ * top-level GSD statusLine to `_previous`. Post-normalization guarantees
+ * exactly one canonical `gsd` entry per registry regardless of which path
+ * succeeded.
+ *
+ * @param {string} statuslineScriptPath - Absolute path to gsd-statusline.cjs
+ * @returns {boolean} true if registered or normalized in at least one registry
+ */
+function registerProvider(statuslineScriptPath) {
+  const command = `node ${JSON.stringify(statuslineScriptPath)}`;
+  const cliOk = runChainCLI(['register', 'gsd', command, '--stdin']);
+
+  let anyNormalized = false;
+  for (const candidate of REGISTRY_PATHS) {
+    // Only touch paths whose parent dir exists — don't create arbitrary
+    // ~/.cache/ subtrees on machines without code-graph.
+    const dir = path.dirname(candidate);
+    if (!fs.existsSync(dir)) continue;
+    if (normalizeRegistryFile(candidate, command)) anyNormalized = true;
+  }
+
+  return cliOk || anyNormalized;
+}
+
 /**
  * Remove GSD entry from composite statusline registry.
  * Prefers code-graph's statusline-chain.js CLI when available; falls back to

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gsd-lite",
-  "version": "0.7.4",
+  "version": "0.7.6",
   "description": "AI orchestration tool for Claude Code — GSD management shell + Superpowers quality core",
   "type": "module",
   "bin": {

package/src/server.js

@@ -144,7 +144,7 @@ const TOOLS = [
         },
         run_verify: {
           type: 'boolean',
-          description: 'When true, run lint/typecheck/test during handoff evaluation',
+          description: 'Assert that verification was run externally; if true without a verification object, returns INVALID_INPUT. The state layer does not execute external tools.',
         },
         direction_ok: {
           type: 'boolean',

package/src/tools/orchestrator/executor.js

@@ -1,4 +1,4 @@
-import { read, reclassifyReviewLevel } from '../state/index.js';
+import { read, reclassifyReviewLevel, selectRunnableTask } from '../state/index.js';
 import { validateExecutorResult } from '../../schema.js';
 import {
   MAX_DEBUG_RETRY,
@@ -101,8 +101,20 @@ export async function handleExecutorResult({ result, basePath = process.cwd() }
 
   if (result.outcome === 'blocked') {
     const { blocked_reason, unblock_condition } = getBlockedReasonFromResult(result);
+    // Probe whether other tasks remain runnable after this one is blocked.
+    // Design (docs/gsd-lite-design.md §1399): awaiting_user fires only when
+    // 0 runnable tasks remain — blocked-with-others-runnable continues execution.
+    const probePhase = {
+      ...phase,
+      todo: phase.todo.map((t) => (t.id === task.id
+        ? { ...t, lifecycle: 'blocked', blocked_reason, unblock_condition }
+        : t)),
+    };
+    const probe = selectRunnableTask(probePhase, state);
+    const hasOtherRunnable = !!probe?.task;
+
     const persistError = await persist(basePath, {
-      workflow_mode: 'awaiting_user',
+      workflow_mode: hasOtherRunnable ? 'executing_task' : 'awaiting_user',
       current_task: null,
       current_review: null,
       phases: [{
@@ -120,8 +132,8 @@ export async function handleExecutorResult({ result, basePath = process.cwd() }
 
     return {
       success: true,
-      action: 'awaiting_user',
-      workflow_mode: 'awaiting_user',
+      action: hasOtherRunnable ? 'continue_execution' : 'awaiting_user',
+      workflow_mode: hasOtherRunnable ? 'executing_task' : 'awaiting_user',
       task_id: task.id,
       blockers: getBlockedTasks({ todo: [{ id: task.id, lifecycle: 'blocked', blocked_reason, unblock_condition }] }),
     };

package/src/tools/state/crud.js

@@ -511,6 +511,13 @@ export async function phaseComplete({
       ? verificationPassed(verificationResult)
       : phase.phase_handoff.tests_passed === true;
     if (!testsPassed) {
+      if (!verificationResult) {
+        return {
+          error: true,
+          code: ERROR_CODES.HANDOFF_GATE,
+          message: 'Handoff gate not met: verification required. Run lint/typecheck/test externally, then call phase-complete with verification: { lint: { exit_code }, typecheck: { exit_code }, test: { exit_code } }',
+        };
+      }
       return {
         error: true,
         code: ERROR_CODES.HANDOFF_GATE,

package/workflows/execution-flow.md

@@ -163,21 +163,16 @@
 | `await_recovery_decision` | 工作流处于 failed 状态。向用户展示失败信息和恢复选项 (retry/skip/replan) |
 
 **`phase-complete` 参数:**
+
+编排器必须先用 Bash 外部执行 lint/typecheck/test (state 层不自动跑外部工具)，然后把结果作为 `verification` 传入:
 ```
 phase-complete({
   phase_id: <当前 phase 编号>,
-  run_verify: true,          // 自动运行 lint/typecheck/test
-  direction_ok: true         // 方向校验通过 (如有偏差设为 false)
-})
-```
-如果没有 lint/typecheck/test 工具，可改用 `verification` 参数传入预计算结果:
-```
-phase-complete({
-  phase_id: <phase>,
   verification: { lint: {exit_code: 0}, typecheck: {exit_code: 0}, test: {exit_code: 0} },
-  direction_ok: true
+  direction_ok: true         // 方向校验通过 (如有偏差设为 false)
 })
 ```
+`run_verify: true` 只是声明"已外部跑过"，不提供 `verification` 时会返回 INVALID_INPUT，用于防止调用方忘记传结果。如果项目缺少某项脚本 (例如无 typecheck)，跑对应工具时返回 0 视为通过即可 (约定: "命令不存在" = 不适用 = 通过)。
 </execution_loop>
 
 ## STEP 12 — 最终报告