@polymorphism-tech/morph-spec 3.2.0 → 4.3.0

package/src/commands/{update.js → project/update.js}

@@ -1,240 +1,240 @@
-import { join } from 'path';
-import fs from 'fs-extra';
-import ora from 'ora';
-import chalk from 'chalk';
-import { logger } from '../utils/logger.js';
-import {
-  getContentDir,
-  copyDirectory,
-  pathExists,
-  ensureDir,
-  createDirectoryLink,
-  readJson,
-  writeJson
-} from '../utils/file-copier.js';
-import {
-  checkCLIOutdated,
-  checkProjectOutdated,
-  saveProjectMorphVersion,
-  getUpdateInstructions,
-  detectInstallMethod
-} from '../utils/version-checker.js';
-import { AutoContextOrchestrator } from '../orchestrator.js';
-import { detectClaudeCode } from '../llm/environment-detector.js';
-
-export async function updateCommand(options) {
-  const targetPath = process.cwd();
-  const contentDir = getContentDir();
-  const morphPath = join(targetPath, '.morph');
-
-  logger.header('MORPH-SPEC Update');
-
-  // Check if MORPH is initialized
-  if (!(await pathExists(morphPath))) {
-    logger.error('MORPH not initialized in this directory.');
-    logger.dim('Run "morph-spec init" first.');
-    process.exit(1);
-  }
-
-  // Check CLI version
-  const spinner = ora('Checking versions...').start();
-  const cliCheck = await checkCLIOutdated();
-  const projectCheck = await checkProjectOutdated(targetPath);
-
-  spinner.stop();
-
-  // Display version info
-  logger.blank();
-  logger.info('Version Status:');
-  logger.dim(`  CLI version: ${cliCheck.current}`);
-
-  if (projectCheck.current) {
-    logger.dim(`  Project MORPH version: ${projectCheck.current}`);
-  } else {
-    logger.dim('  Project MORPH version: not found (legacy installation)');
-  }
-
-  if (cliCheck.latest) {
-    const latestColor = cliCheck.isOutdated ? chalk.yellow : chalk.green;
-    logger.dim(`  Latest available: ${latestColor(cliCheck.latest)}`);
-  }
-  logger.blank();
-
-  // If CLI is outdated, stop and instruct user
-  if (cliCheck.isOutdated && cliCheck.latest) {
-    logger.warn(`Your CLI is outdated (${cliCheck.current} → ${cliCheck.latest})`);
-    logger.blank();
-    logger.info('Please update the CLI first:');
-    logger.blank();
-
-    const method = detectInstallMethod();
-    const instructions = getUpdateInstructions(method);
-
-    instructions.forEach(line => {
-      if (line === '') {
-        logger.blank();
-      } else if (line.startsWith('Or ')) {
-        logger.dim(line);
-      } else {
-        logger.box([line]);
-      }
-    });
-
-    logger.blank();
-    logger.dim('Then run "morph-spec update" again.');
-    process.exit(0);
-  }
-
-  // Proceed with update
-  const updateSpinner = ora('Updating MORPH-SPEC...').start();
-
-  try {
-    const updateTemplates = !options.standards || options.templates;
-    const updateStandards = !options.templates || options.standards;
-
-    // Update templates
-    if (updateTemplates) {
-      updateSpinner.text = 'Updating templates...';
-      const templatesSrc = join(contentDir, '.morph', 'templates');
-      const templatesDest = join(morphPath, 'templates');
-      await copyDirectory(templatesSrc, templatesDest);
-    }
-
-    // Update standards
-    if (updateStandards) {
-      updateSpinner.text = 'Updating standards...';
-      const standardsSrc = join(contentDir, '.morph', 'standards');
-      const standardsDest = join(morphPath, 'standards');
-      await copyDirectory(standardsSrc, standardsDest);
-    }
-
-    // Update agents.json
-    updateSpinner.text = 'Updating agents configuration...';
-    const agentsSrc = join(contentDir, '.morph', 'config', 'agents.json');
-    const agentsDest = join(morphPath, 'config', 'agents.json');
-    if (await pathExists(agentsSrc)) {
-      await copyDirectory(agentsSrc, agentsDest);
-    }
-
-    // Update .claude commands and skills
-    updateSpinner.text = 'Updating Claude commands and skills...';
-    const claudeSrc = join(contentDir, '.claude');
-    const claudeDest = join(targetPath, '.claude');
-    if (await pathExists(claudeSrc)) {
-      // Copy commands (always copy, these are small)
-      const commandsSrc = join(claudeSrc, 'commands');
-      const commandsDest = join(claudeDest, 'commands');
-      if (await pathExists(commandsSrc)) {
-        await copyDirectory(commandsSrc, commandsDest);
-
-        // Clean up stale command files moved to skills/workflows/ in v2.4+
-        const staleCommands = [
-          'morph-setup.md', 'morph-uiux.md', 'morph-design.md',
-          'morph-clarify.md', 'morph-tasks.md'
-        ];
-        for (const file of staleCommands) {
-          const stalePath = join(commandsDest, file);
-          if (await pathExists(stalePath)) {
-            await fs.remove(stalePath);
-          }
-        }
-      }
-
-      // Update skills using directory links per category
-      const skillsSrc = join(claudeSrc, 'skills');
-      const skillsDest = join(claudeDest, 'skills');
-      if (await pathExists(skillsSrc)) {
-        await ensureDir(skillsDest);
-        const entries = await fs.readdir(skillsSrc, { withFileTypes: true });
-        for (const entry of entries) {
-          if (entry.isDirectory()) {
-            await createDirectoryLink(join(skillsSrc, entry.name), join(skillsDest, entry.name));
-          }
-        }
-      }
-    }
-
-    // Update CLAUDE.md
-    updateSpinner.text = 'Updating CLAUDE.md...';
-    const claudeMdSrc = join(contentDir, 'CLAUDE.md');
-    const claudeMdDest = join(targetPath, 'CLAUDE.md');
-    await copyDirectory(claudeMdSrc, claudeMdDest);
-
-    // Update .morphversion
-    updateSpinner.text = 'Saving version info...';
-    await saveProjectMorphVersion(targetPath, cliCheck.current);
-
-    // Update config.json frameworkVersion (BUG #9 fix)
-    const configPath = join(morphPath, 'config', 'config.json');
-    if (await pathExists(configPath)) {
-      try {
-        const config = await readJson(configPath);
-        config.frameworkVersion = cliCheck.current;
-        await writeJson(configPath, config);
-      } catch { /* preserve existing config if read/write fails */ }
-    }
-
-    updateSpinner.succeed('MORPH-SPEC updated successfully!');
-    logger.blank();
-
-    // Show version change
-    if (projectCheck.current && projectCheck.current !== cliCheck.current) {
-      logger.success(`Updated: ${projectCheck.current} → ${cliCheck.current}`);
-    } else if (!projectCheck.current) {
-      logger.success(`Updated to v${cliCheck.current}`);
-    } else {
-      logger.success(`Already up to date (v${cliCheck.current})`);
-    }
-
-    logger.blank();
-    logger.info('Updated files:');
-    if (updateTemplates) logger.dim('  ✓ .morph/templates/');
-    if (updateStandards) logger.dim('  ✓ .morph/standards/');
-    logger.dim('  ✓ .morph/config/agents.json');
-    logger.dim('  ✓ .claude/commands/ and .claude/skills/');
-    logger.dim('  ✓ CLAUDE.md');
-    logger.blank();
-    logger.info('Your config.json was preserved.');
-    logger.dim('Review the updated files for any new features.');
-    logger.blank();
-
-    // Re-analyze project context (unless --skip-detection)
-    if (!options.skipDetection && detectClaudeCode()) {
-      logger.header('Re-Analyzing Project Context');
-      logger.dim('Using Claude Code LLM to update project detection...');
-      logger.blank();
-
-      try {
-        const orchestrator = new AutoContextOrchestrator();
-        const result = await orchestrator.execute(targetPath, {
-          skipReview: false,
-          fallbackOnError: true,
-          wizardMode: options.wizard || false
-        });
-
-        if (result.success) {
-          logger.success('Project context re-analyzed and updated');
-          logger.dim('  ✓ .morph/project.md updated with latest detection');
-          logger.dim('  ✓ .morph/config/config.json updated with stack info');
-        } else {
-          logger.warn('Context re-analysis incomplete');
-        }
-      } catch (error) {
-        logger.warn(`Auto-detection failed: ${error.message}`);
-        logger.dim('Your existing config.json was preserved.');
-      }
-
-      logger.blank();
-    } else if (options.skipDetection) {
-      logger.dim('Skipped auto-detection (--skip-detection flag)');
-    } else {
-      logger.warn('⚠️  Claude Code not detected - skipping auto-detection');
-      logger.dim('Run with --wizard to configure manually.');
-    }
-
-  } catch (error) {
-    updateSpinner.fail('Update failed');
-    logger.error(error.message);
-    process.exit(1);
-  }
-}
+import { join } from 'path';
+import fs from 'fs-extra';
+import ora from 'ora';
+import chalk from 'chalk';
+import { logger } from '../../utils/logger.js';
+import {
+  getContentDir,
+  copyDirectory,
+  pathExists,
+  ensureDir,
+  createDirectoryLink,
+  readJson,
+  writeJson
+} from '../../utils/file-copier.js';
+import {
+  checkCLIOutdated,
+  checkProjectOutdated,
+  saveProjectMorphVersion,
+  getUpdateInstructions,
+  detectInstallMethod
+} from '../../utils/version-checker.js';
+import { AutoContextOrchestrator } from '../../core/orchestrator.js';
+import { detectClaudeCode } from '../../llm/environment-detector.js';
+
+export async function updateCommand(options) {
+  const targetPath = process.cwd();
+  const contentDir = getContentDir();
+  const morphPath = join(targetPath, '.morph');
+
+  logger.header('MORPH-SPEC Update');
+
+  // Check if MORPH is initialized
+  if (!(await pathExists(morphPath))) {
+    logger.error('MORPH not initialized in this directory.');
+    logger.dim('Run "morph-spec init" first.');
+    process.exit(1);
+  }
+
+  // Check CLI version
+  const spinner = ora('Checking versions...').start();
+  const cliCheck = await checkCLIOutdated();
+  const projectCheck = await checkProjectOutdated(targetPath);
+
+  spinner.stop();
+
+  // Display version info
+  logger.blank();
+  logger.info('Version Status:');
+  logger.dim(`  CLI version: ${cliCheck.current}`);
+
+  if (projectCheck.current) {
+    logger.dim(`  Project MORPH version: ${projectCheck.current}`);
+  } else {
+    logger.dim('  Project MORPH version: not found (legacy installation)');
+  }
+
+  if (cliCheck.latest) {
+    const latestColor = cliCheck.isOutdated ? chalk.yellow : chalk.green;
+    logger.dim(`  Latest available: ${latestColor(cliCheck.latest)}`);
+  }
+  logger.blank();
+
+  // If CLI is outdated, stop and instruct user
+  if (cliCheck.isOutdated && cliCheck.latest) {
+    logger.warn(`Your CLI is outdated (${cliCheck.current} → ${cliCheck.latest})`);
+    logger.blank();
+    logger.info('Please update the CLI first:');
+    logger.blank();
+
+    const method = detectInstallMethod();
+    const instructions = getUpdateInstructions(method);
+
+    instructions.forEach(line => {
+      if (line === '') {
+        logger.blank();
+      } else if (line.startsWith('Or ')) {
+        logger.dim(line);
+      } else {
+        logger.box([line]);
+      }
+    });
+
+    logger.blank();
+    logger.dim('Then run "morph-spec update" again.');
+    process.exit(0);
+  }
+
+  // Proceed with update
+  const updateSpinner = ora('Updating MORPH-SPEC...').start();
+
+  try {
+    const updateTemplates = !options.standards || options.templates;
+    const updateStandards = !options.templates || options.standards;
+
+    // Update templates
+    if (updateTemplates) {
+      updateSpinner.text = 'Updating templates...';
+      const templatesSrc = join(contentDir, '.morph', 'templates');
+      const templatesDest = join(morphPath, 'templates');
+      await copyDirectory(templatesSrc, templatesDest);
+    }
+
+    // Update standards
+    if (updateStandards) {
+      updateSpinner.text = 'Updating standards...';
+      const standardsSrc = join(contentDir, '.morph', 'standards');
+      const standardsDest = join(morphPath, 'standards');
+      await copyDirectory(standardsSrc, standardsDest);
+    }
+
+    // Update agents.json
+    updateSpinner.text = 'Updating agents configuration...';
+    const agentsSrc = join(contentDir, '.morph', 'config', 'agents.json');
+    const agentsDest = join(morphPath, 'config', 'agents.json');
+    if (await pathExists(agentsSrc)) {
+      await copyDirectory(agentsSrc, agentsDest);
+    }
+
+    // Update .claude commands and skills
+    updateSpinner.text = 'Updating Claude commands and skills...';
+    const claudeSrc = join(contentDir, '.claude');
+    const claudeDest = join(targetPath, '.claude');
+    if (await pathExists(claudeSrc)) {
+      // Copy commands (always copy, these are small)
+      const commandsSrc = join(claudeSrc, 'commands');
+      const commandsDest = join(claudeDest, 'commands');
+      if (await pathExists(commandsSrc)) {
+        await copyDirectory(commandsSrc, commandsDest);
+
+        // Clean up stale command files moved to skills/workflows/ in v2.4+
+        const staleCommands = [
+          'morph-setup.md', 'morph-uiux.md', 'morph-design.md',
+          'morph-clarify.md', 'morph-tasks.md'
+        ];
+        for (const file of staleCommands) {
+          const stalePath = join(commandsDest, file);
+          if (await pathExists(stalePath)) {
+            await fs.remove(stalePath);
+          }
+        }
+      }
+
+      // Update skills using directory links per category
+      const skillsSrc = join(claudeSrc, 'skills');
+      const skillsDest = join(claudeDest, 'skills');
+      if (await pathExists(skillsSrc)) {
+        await ensureDir(skillsDest);
+        const entries = await fs.readdir(skillsSrc, { withFileTypes: true });
+        for (const entry of entries) {
+          if (entry.isDirectory()) {
+            await createDirectoryLink(join(skillsSrc, entry.name), join(skillsDest, entry.name));
+          }
+        }
+      }
+    }
+
+    // Update CLAUDE.md
+    updateSpinner.text = 'Updating CLAUDE.md...';
+    const claudeMdSrc = join(contentDir, 'CLAUDE.md');
+    const claudeMdDest = join(targetPath, 'CLAUDE.md');
+    await copyDirectory(claudeMdSrc, claudeMdDest);
+
+    // Update .morphversion
+    updateSpinner.text = 'Saving version info...';
+    await saveProjectMorphVersion(targetPath, cliCheck.current);
+
+    // Update config.json frameworkVersion (BUG #9 fix)
+    const configPath = join(morphPath, 'config', 'config.json');
+    if (await pathExists(configPath)) {
+      try {
+        const config = await readJson(configPath);
+        config.frameworkVersion = cliCheck.current;
+        await writeJson(configPath, config);
+      } catch { /* preserve existing config if read/write fails */ }
+    }
+
+    updateSpinner.succeed('MORPH-SPEC updated successfully!');
+    logger.blank();
+
+    // Show version change
+    if (projectCheck.current && projectCheck.current !== cliCheck.current) {
+      logger.success(`Updated: ${projectCheck.current} → ${cliCheck.current}`);
+    } else if (!projectCheck.current) {
+      logger.success(`Updated to v${cliCheck.current}`);
+    } else {
+      logger.success(`Already up to date (v${cliCheck.current})`);
+    }
+
+    logger.blank();
+    logger.info('Updated files:');
+    if (updateTemplates) logger.dim('  ✓ .morph/templates/');
+    if (updateStandards) logger.dim('  ✓ .morph/standards/');
+    logger.dim('  ✓ .morph/config/agents.json');
+    logger.dim('  ✓ .claude/commands/ and .claude/skills/');
+    logger.dim('  ✓ CLAUDE.md');
+    logger.blank();
+    logger.info('Your config.json was preserved.');
+    logger.dim('Review the updated files for any new features.');
+    logger.blank();
+
+    // Re-analyze project context (unless --skip-detection)
+    if (!options.skipDetection && detectClaudeCode()) {
+      logger.header('Re-Analyzing Project Context');
+      logger.dim('Using Claude Code LLM to update project detection...');
+      logger.blank();
+
+      try {
+        const orchestrator = new AutoContextOrchestrator();
+        const result = await orchestrator.execute(targetPath, {
+          skipReview: false,
+          fallbackOnError: true,
+          wizardMode: options.wizard || false
+        });
+
+        if (result.success) {
+          logger.success('Project context re-analyzed and updated');
+          logger.dim('  ✓ .morph/project.md updated with latest detection');
+          logger.dim('  ✓ .morph/config/config.json updated with stack info');
+        } else {
+          logger.warn('Context re-analysis incomplete');
+        }
+      } catch (error) {
+        logger.warn(`Auto-detection failed: ${error.message}`);
+        logger.dim('Your existing config.json was preserved.');
+      }
+
+      logger.blank();
+    } else if (options.skipDetection) {
+      logger.dim('Skipped auto-detection (--skip-detection flag)');
+    } else {
+      logger.warn('⚠️  Claude Code not detected - skipping auto-detection');
+      logger.dim('Run with --wizard to configure manually.');
+    }
+
+  } catch (error) {
+    updateSpinner.fail('Update failed');
+    logger.error(error.message);
+    process.exit(1);
+  }
+}

package/src/commands/{advance-phase.js → state/advance-phase.js}

@@ -10,12 +10,19 @@
  */
 
 import chalk from 'chalk';
-import { loadState, saveState, getFeature, getApprovalGate } from '../lib/state-manager.js';
+import { loadState, saveState, getFeature, getApprovalGate } from '../../core/state/state-manager.js';
 import { PHASES, validatePhase } from './validate-phase.js';
-import { detectDesignSystem, hasUIAgentsActive } from '../lib/design-system-detector.js';
-import { validateSpec } from '../lib/spec-validator.js';
-import { validateTransition, getPhaseDisplayName } from '../lib/phase-state-machine.js';
-import { validateSpecContent, validateTasksContent, validateFeatureOutputs } from '../lib/validators/content-validator.js';
+import { detectDesignSystem, hasUIAgentsActive } from '../../lib/detectors/design-system-detector.js';
+import { shouldAutoApprove, autoCalculateTrust } from '../../lib/trust/trust-manager.js';
+import { validateSpec } from '../../lib/validators/spec-validator.js';
+import { validateTransition, getPhaseDisplayName } from '../../core/state/phase-state-machine.js';
+import { validateSpecContent, validateTasksContent, validateFeatureOutputs } from '../../lib/validators/content/content-validator.js';
+import { getWorkflowConfig } from '../../core/workflows/workflow-detector.js';
+import { readFileSync, existsSync } from 'fs';
+import { join, dirname } from 'path';
+import { fileURLToPath } from 'url';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
 
 // Phase order for advancing (skips optional phases unless active)
 const PHASE_ORDER = ['proposal', 'setup', 'uiux', 'design', 'clarify', 'tasks', 'implement', 'sync'];
@@ -24,6 +31,12 @@ const PHASE_ORDER = ['proposal', 'setup', 'uiux', 'design', 'clarify', 'tasks',
  * Get the next phase after the current one
  */
 function getNextPhase(currentPhase, feature, skipOptional) {
+  // Load workflow config if workflow is set
+  let workflowConfig = null;
+  if (feature.workflow && feature.workflow !== 'auto') {
+    workflowConfig = getWorkflowConfig(feature.workflow);
+  }
+
   const currentIndex = PHASE_ORDER.indexOf(currentPhase);
   if (currentIndex === -1 || currentIndex >= PHASE_ORDER.length - 1) {
     return null;
@@ -33,26 +46,73 @@ function getNextPhase(currentPhase, feature, skipOptional) {
     const candidate = PHASE_ORDER[i];
     const phaseDef = PHASES[candidate];
 
-    // Skip optional phases if flag set or no relevant agents/outputs
-    if (phaseDef?.optional && skipOptional) {
-      continue;
-    }
-
-    // Skip uiux if no UI agents are active
-    if (candidate === 'uiux') {
-      const uiAgents = ['blazor-builder', 'uiux-designer'];
-      const hasUiAgent = feature.activeAgents?.some(a => uiAgents.includes(a));
-      if (!hasUiAgent && skipOptional !== false) {
+    // === Workflow-Based Phase Skipping ===
+    if (workflowConfig && workflowConfig.phases) {
+      // Check if workflow explicitly skips this phase
+      if (workflowConfig.phases.skip && workflowConfig.phases.skip.includes(candidate)) {
+        console.log(chalk.gray(`  ⏩ Skipping ${candidate}: workflow ${feature.workflow} skips this phase`));
         continue;
       }
+
+      // Check if this phase is part of a combined phase
+      if (workflowConfig.phases.combined) {
+        for (const [combinedName, phases] of Object.entries(workflowConfig.phases.combined)) {
+          if (phases.includes(candidate) && candidate !== combinedName) {
+            console.log(chalk.gray(`  ⏩ Skipping ${candidate}: combined into ${combinedName} phase`));
+            continue;
+          }
+        }
+      }
+
+      // Check if workflow only runs specific phases
+      if (workflowConfig.phases.run && !workflowConfig.phases.run.includes(candidate)) {
+        // Also check if it's not a combined phase
+        const isCombinedPhase = workflowConfig.phases.combined &&
+          Object.keys(workflowConfig.phases.combined).includes(candidate);
+        if (!isCombinedPhase) {
+          console.log(chalk.gray(`  ⏩ Skipping ${candidate}: not in workflow run list`));
+          continue;
+        }
+      }
+
+      // Check conditional skips
+      if (workflowConfig.phases.skipIfCondition) {
+        const condition = workflowConfig.phases.skipIfCondition[candidate];
+        if (condition) {
+          // Evaluate condition (simple conditions for now)
+          if (condition === '!hasUIAgents' && !hasUIAgentsActive(feature)) {
+            console.log(chalk.gray(`  ⏩ Skipping ${candidate}: no UI agents active`));
+            continue;
+          }
+        }
+      }
     }
 
-    // Skip sync for simple workflows
-    if (candidate === 'sync') {
-      const isSimple = feature.workflow === 'fast-track';
-      if (isSimple && skipOptional !== false) {
+    // === Legacy Phase Skipping (fallback when no workflow config) ===
+    if (!workflowConfig) {
+      // Skip optional phases if flag set or no relevant agents/outputs
+      if (phaseDef?.optional && skipOptional) {
         continue;
       }
+
+      // Skip uiux if no UI agents are active
+      if (candidate === 'uiux') {
+        const uiAgents = ['blazor-builder', 'uiux-designer', 'nextjs-expert', 'ui-ux-designer'];
+        const hasUiAgent = feature.activeAgents?.some(a => uiAgents.includes(a));
+        if (!hasUiAgent && skipOptional !== false) {
+          console.log(chalk.gray(`  ⏩ Skipping ${candidate}: no UI agents active`));
+          continue;
+        }
+      }
+
+      // Skip sync for simple workflows
+      if (candidate === 'sync') {
+        const isSimple = feature.workflow === 'fast-track';
+        if (isSimple && skipOptional !== false) {
+          console.log(chalk.gray(`  ⏩ Skipping ${candidate}: fast-track workflow`));
+          continue;
+        }
+      }
     }
 
     return candidate;
@@ -123,14 +183,30 @@ export async function advancePhaseCommand(feature, options = {}) {
   if (requiredGate && !options.skipApproval) {
     const gateStatus = getApprovalGate(feature, requiredGate);
 
+    // Check trust-based auto-approval before blocking
     if (!gateStatus || !gateStatus.approved) {
-      console.log(chalk.red(`\n✗ Phase "${currentPhase}" requires approval before advancing`));
-      console.log(chalk.yellow(`\nRun: morph-spec approve ${feature} ${requiredGate}`));
-      console.log(chalk.gray('Or use --skip-approval to bypass (not recommended)\n'));
-      process.exit(1);
+      try {
+        const trustResult = shouldAutoApprove(feature, requiredGate);
+        if (trustResult.autoApprove) {
+          console.log(chalk.green(`\n✓ Auto-approved gate "${requiredGate}" (${trustResult.reason})`));
+          // Proceed — trust bypasses manual gate requirement
+        } else {
+          console.log(chalk.red(`\n✗ Phase "${currentPhase}" requires approval before advancing`));
+          console.log(chalk.gray(`   Trust level: ${trustResult.level} (${trustResult.reason})`));
+          console.log(chalk.yellow(`\nRun: morph-spec approve ${feature} ${requiredGate}`));
+          console.log(chalk.gray('Or use --skip-approval to bypass (not recommended)\n'));
+          process.exit(1);
+        }
+      } catch {
+        // Trust manager unavailable — fall through to manual gate check
+        console.log(chalk.red(`\n✗ Phase "${currentPhase}" requires approval before advancing`));
+        console.log(chalk.yellow(`\nRun: morph-spec approve ${feature} ${requiredGate}`));
+        console.log(chalk.gray('Or use --skip-approval to bypass (not recommended)\n'));
+        process.exit(1);
+      }
+    } else {
+      console.log(chalk.green(`✓ Approval gate "${requiredGate}" passed`));
     }
-
-    console.log(chalk.green(`✓ Approval gate "${requiredGate}" passed`));
   }
 
   // === GATE 3: Output Requirements ===