gsd-lite 0.5.3 → 0.5.4

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

package/.claude-plugin/plugin.json

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

package/hooks/gsd-statusline.cjs

@@ -74,7 +74,7 @@ process.stdin.on('end', () => {
           let needsWrite = true;
           try {
             const existing = JSON.parse(fs.readFileSync(bridgePath, 'utf8'));
-            if (existing.remaining_percentage === remaining) needsWrite = false;
+            if (existing.remaining_percentage === remaining && existing.has_gsd === hasGsd) needsWrite = false;
           } catch { /* no existing file */ }
           if (needsWrite) {
             const tmpBridge = bridgePath + '.tmp';

package/package.json

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

package/src/schema.js

@@ -17,6 +17,23 @@ export const WORKFLOW_MODES = [
   'failed',
 ];
 
+// Valid workflow_mode transitions — unlisted transitions are rejected by validateStateUpdate.
+// Terminal states (completed, failed) are guarded separately by the FROM-terminal check in state-update.
+export const WORKFLOW_TRANSITIONS = {
+  planning:                 ['executing_task', 'paused_by_user'],
+  executing_task:           ['planning', 'reviewing_task', 'reviewing_phase', 'awaiting_user', 'awaiting_clear', 'paused_by_user', 'reconcile_workspace', 'replan_required', 'research_refresh_needed', 'failed'],
+  reviewing_task:           ['executing_task', 'reviewing_phase', 'awaiting_user', 'awaiting_clear', 'paused_by_user', 'failed'],
+  reviewing_phase:          ['executing_task', 'awaiting_user', 'awaiting_clear', 'paused_by_user', 'completed', 'failed'],
+  awaiting_user:            ['executing_task', 'reviewing_task', 'reviewing_phase', 'paused_by_user', 'awaiting_clear'],
+  awaiting_clear:           ['executing_task', 'paused_by_user'],
+  paused_by_user:           ['executing_task', 'awaiting_user', 'awaiting_clear', 'reconcile_workspace', 'replan_required', 'research_refresh_needed', 'reviewing_task', 'reviewing_phase'],
+  reconcile_workspace:      ['executing_task', 'paused_by_user'],
+  replan_required:          ['executing_task', 'paused_by_user'],
+  research_refresh_needed:  ['executing_task', 'reviewing_task', 'reviewing_phase', 'paused_by_user'],
+  completed:                [],  // terminal — guarded by FROM-terminal check
+  failed:                   [],  // terminal — guarded by FROM-terminal check
+};
+
 export const TASK_LIFECYCLE = {
   pending:              ['running', 'blocked'],
   running:              ['checkpointed', 'blocked', 'failed', 'accepted'], // accepted: auto-accept for L0/review_required=false (atomic, skips checkpointed)
@@ -154,11 +171,28 @@ export function validateStateUpdate(state, updates) {
 
   for (const key of Object.keys(updates)) {
     switch (key) {
-      case 'workflow_mode':
+      case 'workflow_mode': {
         if (!WORKFLOW_MODES.includes(updates.workflow_mode)) {
           errors.push(`Invalid workflow_mode: ${updates.workflow_mode}`);
+          break;
+        }
+        // Transition whitelist — reject unlisted transitions
+        const currentMode = state.workflow_mode;
+        if (currentMode && updates.workflow_mode !== currentMode) {
+          const allowed = WORKFLOW_TRANSITIONS[currentMode];
+          if (allowed && !allowed.includes(updates.workflow_mode)) {
+            errors.push(`Invalid workflow_mode transition: '${currentMode}' → '${updates.workflow_mode}' (allowed: ${allowed.join(', ') || 'none (terminal state)'})`);
+          }
+        }
+        // Guard: 'completed' requires all phases accepted
+        if (updates.workflow_mode === 'completed' && Array.isArray(state.phases)) {
+          const unfinished = state.phases.filter(p => p.lifecycle !== 'accepted');
+          if (unfinished.length > 0) {
+            errors.push(`Cannot set workflow_mode to 'completed': ${unfinished.length} phase(s) not accepted (${unfinished.map(p => `${p.id}:${p.lifecycle}`).join(', ')})`);
+          }
         }
         break;
+      }
       case 'current_phase':
         if (!Number.isFinite(updates.current_phase)) {
           errors.push('current_phase must be a finite number');
@@ -383,8 +417,12 @@ export function validateState(state) {
       }
     }
   }
-  // P2-9: workflow_mode consistency — completed project must not have active/running tasks
+  // P2-9: workflow_mode consistency — completed project requires all phases accepted
   if (state.workflow_mode === 'completed' && Array.isArray(state.phases)) {
+    const unfinishedPhases = state.phases.filter(p => p.lifecycle !== 'accepted');
+    if (unfinishedPhases.length > 0) {
+      errors.push(`Completed project has ${unfinishedPhases.length} unfinished phase(s): ${unfinishedPhases.map(p => `${p.id}:${p.lifecycle}`).join(', ')}`);
+    }
     for (const phase of state.phases) {
       for (const task of (phase.todo || [])) {
         if (task.lifecycle === 'running') {

package/src/server.js

@@ -90,7 +90,7 @@ const TOOLS = [
       properties: {
         updates: {
           type: 'object',
-          description: 'Key-value pairs of canonical fields to update',
+          description: 'Key-value pairs of canonical fields: workflow_mode, current_phase, current_task, current_review, git_head, plan_version, schema_version, total_phases, project, decisions, context, evidence, research',
         },
       },
       required: ['updates'],
@@ -105,7 +105,13 @@ const TOOLS = [
         phase_id: { type: 'number', description: 'Phase number to complete' },
         verification: {
           type: 'object',
-          description: 'Optional precomputed verification: {lint: {exit_code: number}, typecheck: {exit_code: number}, test: {exit_code: number}} — all three keys required, exit_code 0 = passed',
+          description: 'Optional precomputed verification — all three keys required, exit_code 0 = passed',
+          properties: {
+            lint: { type: 'object', properties: { exit_code: { type: 'number' } }, required: ['exit_code'] },
+            typecheck: { type: 'object', properties: { exit_code: { type: 'number' } }, required: ['exit_code'] },
+            test: { type: 'object', properties: { exit_code: { type: 'number' } }, required: ['exit_code'] },
+          },
+          required: ['lint', 'typecheck', 'test'],
         },
         run_verify: {
           type: 'boolean',
@@ -135,7 +141,7 @@ const TOOLS = [
       properties: {
         result: {
           type: 'object',
-          description: 'Executor result: {task_id: string, outcome: "checkpointed"|"blocked"|"failed", summary: string, checkpoint_commit: string|null, files_changed: string[], decisions: string[], blockers: object[], contract_changed: boolean, evidence: object[]}',
+          description: 'Executor result: {task_id: string, outcome: "checkpointed"|"blocked"|"failed", summary: string, checkpoint_commit: string|null, files_changed: string[], decisions: [{id, summary, rationale}], blockers: [{description}], contract_changed: boolean, evidence: string[]}',
         },
       },
       required: ['result'],

package/src/tools/state.js

@@ -341,10 +341,13 @@ function verificationPassed(verification) {
 
 function verificationSummary(verification) {
   if (!verification || typeof verification !== 'object') return 'no verification details';
-  return ['lint', 'typecheck', 'test']
-    .filter((key) => verification[key])
-    .map((key) => `${key}:${verification[key].exit_code}`)
-    .join(', ');
+  const parts = ['lint', 'typecheck', 'test'].map((key) => {
+    const v = verification[key];
+    if (!v) return `${key}:missing`;
+    if (typeof v !== 'object' || !('exit_code' in v)) return `${key}:invalid-format (expected {exit_code: number})`;
+    return `${key}:${v.exit_code === 0 ? 'pass' : `fail(${v.exit_code})`}`;
+  });
+  return parts.join(', ');
 }
 
 export async function phaseComplete({