gsd-lite 0.5.3 → 0.5.5

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.5",
       "keywords": [
         "orchestration",
         "mcp",

package/.claude-plugin/plugin.json

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

package/agents/debugger.md

@@ -67,9 +67,11 @@ Phase 4 修复方向建议:
   "task_id": "2.3",
   "outcome": "root_cause_found | fix_suggested | failed",
   "root_cause": "Description of the identified root cause",
-  "evidence": ["ev:repro:error-xyz", "ev:trace:data-flow"],
+  "evidence": [
+    { "id": "ev:repro:error-xyz", "scope": "task:2.3", "command": "npm test", "exit_code": 1, "stdout": "...", "stderr": "...", "timestamp": "ISO8601" }
+  ],
   "hypothesis_tested": [
-    { "hypothesis": "X causes Y", "result": "confirmed | rejected", "evidence": "..." }
+    { "hypothesis": "X causes Y", "result": "confirmed | rejected", "evidence": "non-empty string (required)" }
   ],
   "fix_direction": "Suggested fix approach for executor",
   "fix_attempts": 0,

package/agents/reviewer.md

@@ -93,7 +93,7 @@ Minor = 建议修复 (命名/风格)
 {
   "scope": "task | phase",
   "scope_id": "2.3 (task scope: string ID) | 2 (phase scope: number ID)",
-  "review_level": "L2 | L1-batch",
+  "review_level": "L2 | L1-batch | L1",
   "spec_passed": true,
   "quality_passed": false,
   "critical_issues": [

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.5",
   "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') {
@@ -560,7 +598,7 @@ export function validateReviewerResult(r) {
   if (!(typeof r.scope_id === 'string' || typeof r.scope_id === 'number') || r.scope_id === '') {
     errors.push('missing scope_id');
   }
-  if (!['L2', 'L1-batch'].includes(r.review_level)) errors.push('invalid review_level');
+  if (!['L2', 'L1-batch', 'L1'].includes(r.review_level)) errors.push('invalid review_level (expected L2, L1-batch, or L1)');
   if (typeof r.spec_passed !== 'boolean') errors.push('spec_passed must be boolean');
   if (typeof r.quality_passed !== 'boolean') errors.push('quality_passed must be boolean');
   if (!Array.isArray(r.critical_issues)) errors.push('critical_issues must be array');

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/orchestrator.js

@@ -450,6 +450,13 @@ async function resumeExecutingTask(state, basePath) {
 
   if (selection.task) {
     const task = selection.task;
+    // Two-step transition for needs_revalidation: must go through pending first
+    if (task.lifecycle === 'needs_revalidation') {
+      const resetError = await persist(basePath, {
+        phases: [{ id: phase.id, todo: [{ id: task.id, lifecycle: 'pending' }] }],
+      });
+      if (resetError) return resetError;
+    }
     const persistError = await persist(basePath, {
       workflow_mode: 'executing_task',
       current_task: task.id,

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({