gsd-lite 0.6.4 → 0.6.7

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.6.4",
+      "version": "0.6.7",
       "keywords": [
         "orchestration",
         "mcp",

package/.claude-plugin/plugin.json

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

package/hooks/gsd-session-init.cjs

@@ -94,6 +94,7 @@ setTimeout(() => process.exit(0), 4000).unref();
       if (gsdEntry) {
         const mcpContent = JSON.stringify({
           mcpServers: {
+            // biome-ignore lint/suspicious/noTemplateCurlyInString: intentional — Claude plugin system substitutes this at runtime
             gsd: { command: 'node', args: ['${CLAUDE_PLUGIN_ROOT}/launcher.js'] },
           },
         }, null, 2) + '\n';

package/hooks/hooks.json

@@ -1,41 +1,4 @@
 {
-  "description": "GSD-Lite hooks: statusline + context monitor + session lifecycle",
-  "hooks": {
-    "SessionStart": [
-      {
-        "matcher": "startup",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/gsd-session-init.cjs\"",
-            "timeout": 5
-          }
-        ]
-      }
-    ],
-    "PostToolUse": [
-      {
-        "matcher": "*",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/gsd-context-monitor.cjs\"",
-            "timeout": 3
-          }
-        ]
-      }
-    ],
-    "Stop": [
-      {
-        "matcher": "*",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node \"${CLAUDE_PLUGIN_ROOT}/hooks/gsd-session-stop.cjs\"",
-            "timeout": 5
-          }
-        ]
-      }
-    ]
-  }
+  "description": "GSD-Lite hooks — authoritative registration is in settings.json via install.js. This file cleared to prevent double execution if the plugin system also loads it.",
+  "hooks": {}
 }

package/install.js

@@ -21,7 +21,7 @@ const HOOK_FILES = ['gsd-session-init.cjs', 'gsd-auto-update.cjs', 'gsd-context-
 
 // Hook registration config: hookType → { file identifier, matcher, timeout? }
 const HOOK_REGISTRY = [
-  { hookType: 'SessionStart', identifier: 'gsd-session-init', matcher: 'startup', timeout: 5 },
+  { hookType: 'SessionStart', identifier: 'gsd-session-init', matcher: 'startup|clear|compact', timeout: 5 },
   { hookType: 'PostToolUse', identifier: 'gsd-context-monitor', matcher: '*' },
   { hookType: 'Stop', identifier: 'gsd-session-stop', matcher: '*', timeout: 3 },
 ];
@@ -239,33 +239,13 @@ export function main() {
     const statuslinePath = join(CLAUDE_DIR, 'hooks', 'gsd-statusline.cjs');
     let statusLineRegistered = registerStatusLine(settings, statuslinePath);
 
-    // Hooks are managed by hooks.json via the plugin system for plugin installs.
-    // Only register in settings.json for manual installs to avoid double execution.
+    // Always register hooks in settings.json regardless of install method.
+    // The plugin system's hooks.json auto-loading is unreliable — settings.json
+    // is the only reliable hook registration path (consistent with claude-mem-lite).
     let hooksRegistered = false;
-    if (!isPluginInstall) {
-      if (!settings.hooks) settings.hooks = {};
-      for (const config of HOOK_REGISTRY) {
-        if (registerHookEntry(settings.hooks, config)) hooksRegistered = true;
-      }
-    } else {
-      // Clean up stale manual hook entries left from previous install.js runs
-      if (settings.hooks) {
-        let cleaned = false;
-        for (const [hookType, identifier] of [
-          ['PostToolUse', 'gsd-context-monitor'],
-          ['SessionStart', 'gsd-session-init'],
-          ['Stop', 'gsd-session-stop'],
-        ]) {
-          if (Array.isArray(settings.hooks[hookType])) {
-            const before = settings.hooks[hookType].length;
-            settings.hooks[hookType] = settings.hooks[hookType].filter(e =>
-              !e.hooks?.some(h => h.command?.includes(identifier)));
-            if (settings.hooks[hookType].length < before) cleaned = true;
-            if (settings.hooks[hookType].length === 0) delete settings.hooks[hookType];
-          }
-        }
-        if (cleaned) log('  ✓ Removed stale manual hook entries (plugin hooks.json handles registration)');
-      }
+    if (!settings.hooks) settings.hooks = {};
+    for (const config of HOOK_REGISTRY) {
+      if (registerHookEntry(settings.hooks, config)) hooksRegistered = true;
     }
 
     const tmpSettings = settingsPath + `.${process.pid}-${Date.now()}.tmp`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gsd-lite",
-  "version": "0.6.4",
+  "version": "0.6.7",
   "description": "AI orchestration tool for Claude Code — GSD management shell + Superpowers quality core",
   "type": "module",
   "bin": {
@@ -13,6 +13,7 @@
     "lint:fix": "biome check --write src/ tests/ hooks/",
     "start": "node src/server.js",
     "version": "node scripts/sync-versions.js && git add .claude-plugin/plugin.json .claude-plugin/marketplace.json",
+    "prepare": "[ -d .git ] && ln -sf ../../scripts/pre-commit.sh .git/hooks/pre-commit || true",
     "prepublishOnly": "node scripts/sync-versions.js && npm run lint && npm test"
   },
   "repository": {

package/src/server.js

@@ -285,12 +285,33 @@ async function dispatchToolCall(name, args) {
     case 'state-read':
       result = await read(args || {});
       break;
-    case 'state-update':
-      result = await update(args);
+    case 'state-update': {
+      const updateResult = await update(args);
+      // Strip full state from response to save tokens — keep only _version for optimistic concurrency
+      if (updateResult.success && updateResult.state) {
+        result = { success: true, _version: updateResult.state._version };
+      } else {
+        result = updateResult;
+      }
       break;
-    case 'phase-complete':
-      result = await phaseComplete(args);
+    }
+    case 'phase-complete': {
+      const pcResult = await phaseComplete(args);
+      // Enrich sparse success response with progress info
+      if (pcResult.success && !pcResult.action) {
+        const st = await read({ fields: ['current_phase', 'total_phases', 'workflow_mode'] });
+        result = {
+          ...pcResult,
+          phase_completed: args.phase_id,
+          current_phase: st.current_phase,
+          total_phases: st.total_phases,
+          workflow_mode: st.workflow_mode,
+        };
+      } else {
+        result = pcResult;
+      }
       break;
+    }
     case 'state-patch':
       result = await patchPlan(args);
       break;
@@ -316,9 +337,20 @@ async function dispatchToolCall(name, args) {
   return result;
 }
 
+// Strip result_contract from orchestrator responses — it's static reference data
+// already available in MCP tool descriptions. Saves ~200 tokens per call.
+function stripResultContract(result) {
+  if (result && typeof result === 'object' && 'result_contract' in result) {
+    const { result_contract, ...rest } = result;
+    return rest;
+  }
+  return result;
+}
+
 export async function handleToolCall(name, args) {
   try {
-    return await dispatchToolCall(name, args);
+    const result = await dispatchToolCall(name, args);
+    return stripResultContract(result);
   } catch (err) {
     const message = err instanceof Error ? err.message : String(err);
     return { error: true, message: `Tool execution failed: ${message}` };

package/src/tools/orchestrator/helpers.js

@@ -23,7 +23,7 @@ const RESULT_CONTRACTS = {
     summary: 'string — non-empty description of work done',
     checkpoint_commit: 'string — required when outcome="checkpointed"',
     files_changed: 'string[] — list of modified file paths',
-    decisions: '{ id, title, rationale }[] — architectural decisions made',
+    decisions: '{ id, summary|title, rationale }[] — architectural decisions (summary is canonical; title accepted as alias)',
     blockers: '{ description, type }[] — what blocked progress (when outcome="blocked")',
     contract_changed: 'boolean — true if external API/behavior contract changed',
     confidence: '"high" | "medium" | "low" (optional) — executor self-assessed confidence; affects review level',
@@ -336,12 +336,14 @@ function buildDecisionEntries(decisions, phaseId, taskId, existingCount = 0) {
           task: taskId,
         };
       }
-      if (decision && typeof decision === 'object' && typeof decision.summary === 'string') {
+      if (decision && typeof decision === 'object' && (typeof decision.summary === 'string' || typeof decision.title === 'string')) {
+        const summary = decision.summary || decision.title;
         return {
           id: decision.id || `decision:${phaseId}:${taskId}:${existingCount + index + 1}`,
           phase: decision.phase ?? phaseId,
           task: decision.task ?? taskId,
           ...decision,
+          summary,
         };
       }
       return null;

package/src/tools/orchestrator/resume.js

@@ -216,7 +216,8 @@ async function resumeExecutingTask(state, basePath) {
   // signal complete_phase instead of going idle
   const allAccepted = phase.todo.length > 0 && phase.todo.every(t => t.lifecycle === 'accepted');
   const reviewPassed = phase.phase_review?.status === 'accepted'
-    || phase.phase_handoff?.required_reviews_passed === true;
+    || phase.phase_handoff?.required_reviews_passed === true
+    || allAccepted;
   if (allAccepted && reviewPassed) {
     // Auto-advance phase lifecycle to 'reviewing' if currently 'active'
     // (mirrors trigger_review path at line 480-482)
@@ -531,7 +532,7 @@ export async function resumeWorkflow({ basePath = process.cwd(), _depth = 0, unb
   }
 
   // Attach display-ready summary to all successful responses
-  if (result && result.success && !result.summary) {
+  if (result?.success && !result.summary) {
     const summary = _buildResumeSummary(state, result);
     // Use the response's workflow_mode if it differs from state (e.g., preflight override)
     if (result.workflow_mode && result.workflow_mode !== state.workflow_mode) {

package/src/tools/state/crud.js

@@ -23,6 +23,13 @@ import {
 } from './constants.js';
 import { propagateInvalidation } from './logic.js';
 
+function friendlyReadError(rawError) {
+  if (rawError?.includes('ENOENT')) {
+    return 'No GSD project found (state.json missing). Run /gsd:start or /gsd:prd to begin.';
+  }
+  return rawError;
+}
+
 /**
  * Compute SHA-256 content hashes for an array of file paths.
  * Returns an object mapping relative-to-gsdDir paths to hex hashes.
@@ -154,7 +161,7 @@ export async function read({ fields, basePath = process.cwd(), validate = false
 
   const result = await readJson(statePath);
   if (!result.ok) {
-    return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
+    return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: friendlyReadError(result.error) };
   }
   const state = migrateState(result.data);
 
@@ -207,7 +214,7 @@ export async function update({ updates, basePath = process.cwd(), expectedVersio
   return withStateLock(async () => {
     const result = await readJson(statePath);
     if (!result.ok) {
-      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
+      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: friendlyReadError(result.error) };
     }
     const state = migrateState(result.data);
 
@@ -425,7 +432,7 @@ export async function phaseComplete({
   return withStateLock(async () => {
     const result = await readJson(statePath);
     if (!result.ok) {
-      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
+      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: friendlyReadError(result.error) };
     }
     const state = migrateState(result.data);
 
@@ -479,8 +486,11 @@ export async function phaseComplete({
       };
     }
 
+    const allTasksAutoAccepted = phase.lifecycle === 'active'
+      && phase.todo?.length > 0 && phase.todo.every(t => t.lifecycle === 'accepted');
     const reviewPassed = phase.phase_review?.status === 'accepted'
-      || phase.phase_handoff.required_reviews_passed === true;
+      || phase.phase_handoff.required_reviews_passed === true
+      || allTasksAutoAccepted;
     if (!reviewPassed) {
       return {
         error: true,
@@ -572,7 +582,16 @@ export async function phaseComplete({
     }
     state._version = (state._version ?? 0) + 1;
     await writeJson(statePath, state);
-    return { success: true };
+    const isCompleted = state.workflow_mode === 'completed';
+    const nextPhaseInfo = !isCompleted && state.phases.find(p => p.id === state.current_phase);
+    return {
+      success: true,
+      phase_id,
+      phase_name: phase.name,
+      next_phase: nextPhaseInfo ? { id: nextPhaseInfo.id, name: nextPhaseInfo.name, tasks: nextPhaseInfo.todo?.length || 0 } : null,
+      workflow_mode: state.workflow_mode,
+      ...(isCompleted ? { message: 'All phases completed — project finished' } : {}),
+    };
   });
 }
 
@@ -600,7 +619,7 @@ export async function addEvidence({ id, data, basePath = process.cwd() }) {
   return withStateLock(async () => {
     const result = await readJson(statePath);
     if (!result.ok) {
-      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
+      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: friendlyReadError(result.error) };
     }
     const state = migrateState(result.data);
 
@@ -681,7 +700,7 @@ export async function pruneEvidence({ currentPhase, basePath = process.cwd() })
   return withStateLock(async () => {
     const result = await readJson(statePath);
     if (!result.ok) {
-      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
+      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: friendlyReadError(result.error) };
     }
     const state = migrateState(result.data);
 
@@ -721,7 +740,7 @@ export async function patchPlan({ operations, basePath = process.cwd() } = {}) {
   return withStateLock(async () => {
     const result = await readJson(statePath);
     if (!result.ok) {
-      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
+      return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: friendlyReadError(result.error) };
     }
     const state = migrateState(result.data);
 
@@ -887,7 +906,7 @@ function _applyPatchOp(state, op) {
     }
 
     case 'update_task': {
-      const { task_id, ...fields } = op;
+      const { task_id, task: taskObj, ...fields } = op;
       if (typeof task_id !== 'string') return { error: true, message: 'task_id must be a string' };
 
       const phase = state.phases.find(p => p.todo?.some(t => t.id === task_id));
@@ -895,10 +914,12 @@ function _applyPatchOp(state, op) {
 
       const task = phase.todo.find(t => t.id === task_id);
       const allowedFields = ['name', 'level', 'review_required', 'verification_required', 'research_basis'];
+      // Support both flat fields and nested task object (consistent with add_task API)
+      const source = (taskObj && typeof taskObj === 'object') ? { ...taskObj, ...fields } : fields;
       const updates = {};
       for (const key of allowedFields) {
-        if (key in fields) {
-          updates[key] = fields[key];
+        if (key in source) {
+          updates[key] = source[key];
         }
       }