gsd-lite 0.6.1 → 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.1",
+      "version": "0.6.2",
       "keywords": [
         "orchestration",
         "mcp",

package/.claude-plugin/plugin.json

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

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/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.1",
+  "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) {
@@ -319,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;

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,