gsd-lite 0.6.0 → 0.6.2

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

package/.claude-plugin/plugin.json

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

package/README.md

@@ -213,8 +213,8 @@ All state lives in `.gsd/state.json` — a single source of truth with:
 |-----------|-----|----------|
 | Commands | 32 | **6** |
 | Agents | 12 | **4** |
-| Source files | 100+ | **~35** |
-| Installer | 2465 lines | **~80 lines** |
+| Source files | 100+ | **~48** |
+| Installer | 2465 lines | **~290 lines** |
 | User interactions | 6+ confirmations | **Typically 2** |
 | TDD / Anti-rationalization | No | **Yes** |
 | State machine recovery | Partial | **Full (12 modes)** |
@@ -249,7 +249,7 @@ gsd-lite/
 ├── references/             # 8 reference docs
 ├── hooks/                  # Session lifecycle (StatusLine + PostToolUse + SessionStart + Stop + AutoUpdate)
 │   └── lib/               # Shared hook utilities (gsd-finder)
-├── tests/                  # 826 tests (unit + simulation + E2E)
+├── tests/                  # 866 tests (unit + simulation + E2E)
 ├── cli.js                  # Install/uninstall CLI entry
 ├── install.js              # Installation script
 └── uninstall.js            # Uninstall script
@@ -258,7 +258,7 @@ gsd-lite/
 ## Testing
 
 ```bash
-npm test                    # Run all 826 tests
+npm test                    # Run all 866 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/gsd-auto-update.cjs

@@ -286,6 +286,7 @@ async function fetchLatestRelease(token) {
     if (!res.ok) return null;
 
     const data = await res.json();
+    if (!data.tag_name || !data.tarball_url) return null;
     return {
       version: data.tag_name.replace(/^v/, ''),
       tarballUrl: data.tarball_url,

package/hooks/gsd-context-monitor.cjs

@@ -112,7 +112,7 @@ process.stdin.on('end', () => {
     if (isCritical) {
       message = `CONTEXT CRITICAL: Usage at ${usedPct}%. Remaining: ${remaining}%. `
         + 'Context is nearly exhausted. Complete current task checkpoint immediately, '
-        + 'set workflow_mode = awaiting_clear via gsd-state-update, and tell user to /clear then /gsd:resume.';
+        + 'set workflow_mode = awaiting_clear via state-update, and tell user to /clear then /gsd:resume.';
     } else {
       message = `CONTEXT WARNING: Usage at ${usedPct}%. Remaining: ${remaining}%. `
         + 'Context is getting limited. Avoid starting new complex work. Complete current task then save state.';

package/install.js

@@ -170,8 +170,13 @@ export function main() {
     copyDir(localNM, join(RUNTIME_DIR, 'node_modules'), 'runtime/node_modules (copied)');
   } else if (!DRY_RUN) {
     log('  ⧗ Installing runtime dependencies...');
-    execSync('npm ci --omit=dev', { cwd: RUNTIME_DIR, stdio: 'pipe' });
-    log('  ✓ runtime dependencies installed');
+    try {
+      execSync('npm ci --omit=dev', { cwd: RUNTIME_DIR, stdio: 'pipe' });
+      log('  ✓ runtime dependencies installed');
+    } catch (err) {
+      log(`  ✗ Failed to install runtime dependencies: ${err.message}`);
+      process.exit(1);
+    }
   } else {
     log('  [dry-run] Would install runtime dependencies');
   }

package/package.json

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

package/src/tools/orchestrator/helpers.js

@@ -208,20 +208,11 @@ async function evaluatePreflight(state, basePath) {
     }
   }
 
-  const expired_research = collectExpiredResearch(state);
-  if (expired_research.length > 0) {
-    hints.push({
-      workflow_mode: 'research_refresh_needed',
-      action: 'dispatch_researcher',
-      updates: { workflow_mode: 'research_refresh_needed' },
-      expired_research,
-      message: 'Research cache expired and must be refreshed before execution resumes',
-    });
-  }
-
-  // P0-2: Dirty-phase detection — rollback current_phase to earliest phase
+  // Dirty-phase detection — rollback current_phase to earliest phase
   // that has needs_revalidation tasks, ensuring earlier invalidated work
   // is re-executed before proceeding with later phases.
+  // Priority: placed before research expiry because dirty-phase rollback is a
+  // safety-critical action (prevents executing later phases on stale foundations).
   // Use filter+reduce (not .find) to guarantee lowest-ID match regardless of array order.
   const dirtyPhases = (state.phases || []).filter(p =>
     p.id < state.current_phase
@@ -245,6 +236,17 @@ async function evaluatePreflight(state, basePath) {
     });
   }
 
+  const expired_research = collectExpiredResearch(state);
+  if (expired_research.length > 0) {
+    hints.push({
+      workflow_mode: 'research_refresh_needed',
+      action: 'dispatch_researcher',
+      updates: { workflow_mode: 'research_refresh_needed' },
+      expired_research,
+      message: 'Research cache expired and must be refreshed before execution resumes',
+    });
+  }
+
   if (hints.length === 0) return { override: null };
 
   return {

package/src/tools/orchestrator/resume.js

@@ -53,7 +53,7 @@ function _buildResumeSummary(state, response) {
   };
 }
 
-async function resumeAwaitingClear(state, basePath) {
+async function resumeAwaitingClear(state, basePath, _depth = 0) {
   const health = await readContextHealth(basePath);
   if (health !== null && health < CONTEXT_RESUME_THRESHOLD) {
     const persistError = await persist(basePath, {
@@ -83,7 +83,7 @@ async function resumeAwaitingClear(state, basePath) {
   }
   const persistError = await persist(basePath, updates);
   if (persistError) return persistError;
-  return resumeWorkflow({ basePath });
+  return resumeWorkflow({ basePath, _depth: _depth + 1 });
 }
 
 async function resumeExecutingTask(state, basePath) {
@@ -109,26 +109,28 @@ async function resumeExecutingTask(state, basePath) {
     };
   }
 
-  if (state.current_task) {
-    const currentTask = getTaskById(phase, state.current_task);
-    if (currentTask?.lifecycle === 'running') {
-      const isRetrying = (currentTask.retry_count || 0) > 0;
-      const persistError = await persist(basePath, {
-        workflow_mode: 'executing_task',
-        current_task: currentTask.id,
-        current_review: null,
-      });
-      if (persistError) return persistError;
-      return buildExecutorDispatch(state, phase, currentTask, {
-        resumed: true,
-        interruption_recovered: !isRetrying,
-        ...(isRetrying ? {
-          retry_after_failure: true,
-          retry_count: currentTask.retry_count,
-          last_failure_summary: currentTask.last_failure_summary,
-        } : {}),
-      });
-    }
+  // Find the running task — either from current_task or by scanning (orphan recovery)
+  const runningTask = state.current_task
+    ? getTaskById(phase, state.current_task)
+    : (phase.todo || []).find(t => t.lifecycle === 'running');
+
+  if (runningTask?.lifecycle === 'running') {
+    const isRetrying = (runningTask.retry_count || 0) > 0;
+    const persistError = await persist(basePath, {
+      workflow_mode: 'executing_task',
+      current_task: runningTask.id,
+      current_review: null,
+    });
+    if (persistError) return persistError;
+    return buildExecutorDispatch(state, phase, runningTask, {
+      resumed: true,
+      interruption_recovered: !isRetrying,
+      ...(isRetrying ? {
+        retry_after_failure: true,
+        retry_count: runningTask.retry_count,
+        last_failure_summary: runningTask.last_failure_summary,
+      } : {}),
+    });
   }
 
   const selection = selectRunnableTask(phase, state);
@@ -317,7 +319,7 @@ export async function resumeWorkflow({ basePath = process.cwd(), _depth = 0, unb
         result = await resumeExecutingTask(state, basePath);
         break;
       case 'awaiting_clear':
-        result = await resumeAwaitingClear(state, basePath);
+        result = await resumeAwaitingClear(state, basePath, _depth);
         break;
       case 'awaiting_user': {
         if (state.current_review?.stage === 'direction_drift') {

package/src/tools/orchestrator/reviewer.js

@@ -74,6 +74,23 @@ export async function handleReviewerResult({ result, basePath = process.cwd() }
   const qualityFailed = result.quality_passed === false;
   const needsRework = hasCritical || specFailed || qualityFailed;
 
+  // Safety: if rework is needed but no tasks were targeted for rework,
+  // fall back to marking all non-accepted checkpointed/accepted tasks as needs_revalidation
+  // to prevent infinite review loops (no runnable tasks → trigger_review → same result).
+  if (needsRework && taskPatches.filter(p => p.lifecycle === 'needs_revalidation').length === 0) {
+    for (const task of (phase.todo || [])) {
+      if (task.lifecycle === 'checkpointed' || task.lifecycle === 'accepted') {
+        taskPatches.push({
+          id: task.id,
+          lifecycle: 'needs_revalidation',
+          retry_count: 0,
+          evidence_refs: [],
+          last_review_feedback: ['Reviewer indicated rework needed but did not specify tasks; all completed tasks require revalidation'],
+        });
+      }
+    }
+  }
+
   // Compute retry count once for both exhaustion check and state update
   const currentRetryCount = phase.phase_review?.retry_count || 0;
   const nextRetryCount = needsRework ? currentRetryCount + 1 : 0;
@@ -153,8 +170,8 @@ export async function handleReviewerResult({ result, basePath = process.cwd() }
     workflow_mode: workflowMode,
     phase_id: phase.id,
     review_status: reviewStatus,
-    accepted_count: result.accepted_tasks?.length || 0,
-    rework_count: result.rework_tasks?.length || 0,
+    accepted_count: taskPatches.filter(p => p.lifecycle === 'accepted').length,
+    rework_count: taskPatches.filter(p => p.lifecycle === 'needs_revalidation').length,
     critical_count: result.critical_issues?.length || 0,
   };
 }

package/src/tools/state/crud.js

@@ -251,6 +251,11 @@ export async function update({ updates, basePath = process.cwd(), expectedVersio
     // Deep merge phases by ID instead of shallow replace [I-1]
     const merged = { ...state, ...updates };
 
+    // Deep merge context by key (preserves plan_hashes, last_session, etc.)
+    if (updates.context && isPlainObject(updates.context)) {
+      merged.context = { ...(state.context || {}), ...updates.context };
+    }
+
     // Deep merge evidence by key (preserves existing entries)
     if (updates.evidence && isPlainObject(updates.evidence)) {
       merged.evidence = { ...(state.evidence || {}), ...updates.evidence };
@@ -326,10 +331,13 @@ export async function update({ updates, basePath = process.cwd(), expectedVersio
       await _pruneEvidenceFromState(merged, merged.current_phase, gsdDir);
     }
 
-    // Use incremental validation for simple updates (no phases changes)
-    const validation = !updates.phases
-      ? validateStateUpdate(state, updates)
-      : validateState(merged);
+    // Use incremental validation for simple updates (no phases/propagation/decisions changes)
+    const needsFullValidation = updates.phases
+      || (_append_decisions?.length > 0)
+      || (_propagation_tasks?.length > 0);
+    const validation = needsFullValidation
+      ? validateState(merged)
+      : validateStateUpdate(state, updates);
     if (!validation.valid) {
       return {
         error: true,
@@ -495,6 +503,7 @@ export async function phaseComplete({
       if (!driftValidation.valid) {
         return { error: true, code: ERROR_CODES.VALIDATION_FAILED, message: `Validation failed: ${driftValidation.errors.join('; ')}` };
       }
+      state._version = (state._version ?? 0) + 1;
       await writeJson(statePath, state);
       return {
         success: true,
@@ -525,6 +534,8 @@ export async function phaseComplete({
     } else if (state.current_phase === phase_id && phase_id >= state.total_phases) {
       // Final phase completed — mark workflow as completed
       state.workflow_mode = 'completed';
+      state.current_task = null;
+      state.current_review = null;
     }
 
     // Update git_head to current commit
@@ -539,6 +550,7 @@ export async function phaseComplete({
     if (!finalValidation.valid) {
       return { error: true, code: ERROR_CODES.VALIDATION_FAILED, message: `Validation failed: ${finalValidation.errors.join('; ')}` };
     }
+    state._version = (state._version ?? 0) + 1;
     await writeJson(statePath, state);
     return { success: true };
   });
@@ -570,7 +582,7 @@ export async function addEvidence({ id, data, basePath = process.cwd() }) {
     if (!result.ok) {
       return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
     }
-    const state = result.data;
+    const state = migrateState(result.data);
 
     if (!state.evidence) {
       state.evidence = {};
@@ -585,6 +597,7 @@ export async function addEvidence({ id, data, basePath = process.cwd() }) {
       await _pruneEvidenceFromState(state, state.current_phase, gsdDir);
     }
 
+    state._version = (state._version ?? 0) + 1;
     await writeJson(statePath, state);
     return { success: true };
   });
@@ -650,11 +663,14 @@ export async function pruneEvidence({ currentPhase, basePath = process.cwd() })
     if (!result.ok) {
       return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: result.error };
     }
-    const state = result.data;
+    const state = migrateState(result.data);
 
     const gsdDir = dirname(statePath);
     const archived = await _pruneEvidenceFromState(state, currentPhase, gsdDir);
-    if (archived > 0) await writeJson(statePath, state);
+    if (archived > 0) {
+      state._version = (state._version ?? 0) + 1;
+      await writeJson(statePath, state);
+    }
 
     return { success: true, archived };
   });
@@ -734,6 +750,7 @@ export async function patchPlan({ operations, basePath = process.cwd() } = {}) {
       return { error: true, code: ERROR_CODES.VALIDATION_FAILED, message: `Validation failed: ${validation.errors.join('; ')}` };
     }
 
+    state._version = (state._version ?? 0) + 1;
     await writeJson(statePath, state);
     return { success: true, applied, plan_version: state.plan_version };
   });

package/src/tools/state/logic.js

@@ -5,6 +5,7 @@ import { writeFile, rename, unlink } from 'node:fs/promises';
 import { ensureDir, readJson, writeJson, getStatePath } from '../../utils.js';
 import {
   TASK_LIFECYCLE,
+  migrateState,
   validateResearchArtifacts,
   validateResearchDecisionIndex,
   validateResearcherResult,
@@ -439,7 +440,7 @@ export async function storeResearch({ result, artifacts, decision_index, basePat
       return { error: true, code: ERROR_CODES.NO_PROJECT_DIR, message: current.error };
     }
 
-    const state = current.data;
+    const state = migrateState(current.data);
     const gsdDir = dirname(statePath);
     const researchDir = join(gsdDir, 'research');
     await ensureDir(researchDir);
@@ -503,6 +504,7 @@ export async function storeResearch({ result, artifacts, decision_index, basePat
       return { error: true, code: ERROR_CODES.VALIDATION_FAILED, message: `State validation failed: ${validation.errors.join('; ')}` };
     }
 
+    state._version = (state._version ?? 0) + 1;
     await writeJson(statePath, state);
     return {
       success: true,