@polymorphism-tech/morph-spec 7.3.0 → 7.5.0

package/src/commands/state/approve.js

@@ -1,213 +1,251 @@
-import { Command } from 'commander';
-import chalk from 'chalk';
-import { getFeature, setApprovalGate, getApprovalGate, getApprovalHistory } from '../../core/state/state-manager.js';
-
-/**
- * Unapprove action function - Revoke a previously approved gate
- */
-export async function unapproveCommand(featureName, gateName, options = {}) {
-  try {
-    if (!VALID_GATES.includes(gateName)) {
-      console.error(chalk.red(`\n❌ Invalid gate: ${gateName}`));
-      console.log(chalk.gray(`Valid gates: ${VALID_GATES.join(', ')}\n`));
-      process.exit(1);
-    }
-
-    const feature = getFeature(featureName);
-    if (!feature) {
-      console.error(chalk.red(`\n❌ Feature not found: ${featureName}\n`));
-      process.exit(1);
-    }
-
-    const currentStatus = getApprovalGate(featureName, gateName);
-    if (!currentStatus?.approved) {
-      console.log(chalk.yellow(`\n⚠️  Gate "${gateName}" is not currently approved — nothing to revoke\n`));
-      return;
-    }
-
-    await setApprovalGate(featureName, gateName, false, {
-      revokedBy: options.revoker || process.env.USER || process.env.USERNAME || 'user',
-      revokedAt: new Date().toISOString(),
-      reason: options.reason
-    });
-
-    console.log(chalk.yellow(`\n↩️  Gate "${gateName}" approval revoked for feature: ${featureName}`));
-    console.log(chalk.gray(`Workflow is now blocked at the ${gateName} gate\n`));
-
-  } catch (error) {
-    console.error(chalk.red(`\n❌ Error revoking gate: ${error.message}\n`));
-    process.exit(1);
-  }
-}
-
-/**
- * Approval Commands - Manage approval gates for MORPH workflow
- *
- * Commands:
- * - approve: Mark a gate as approved
- * - unapprove: Revoke a gate approval
- * - approval-status: Show approval status for all gates
- */
-
-const program = new Command();
-
-/**
- * Valid approval gate names
- */
-const VALID_GATES = ['proposal', 'uiux', 'plan', 'review'];
-
-/**
- * Approve action function - Mark a gate as approved
- */
-export async function approveCommand(featureName, gateName, options = {}) {
-  try {
-    // Validate gate name
-    if (!VALID_GATES.includes(gateName)) {
-      console.error(chalk.red(`\n❌ Invalid gate: ${gateName}`));
-      console.log(chalk.gray(`Valid gates: ${VALID_GATES.join(', ')}\n`));
-      process.exit(1);
-    }
-
-    // Get feature state
-    const feature = getFeature(featureName);
-    if (!feature) {
-      console.error(chalk.red(`\n❌ Feature not found: ${featureName}\n`));
-      process.exit(1);
-    }
-
-    // Check if already approved
-    const currentStatus = getApprovalGate(featureName, gateName);
-    if (currentStatus?.approved) {
-      console.log(chalk.yellow(`\n⚠️  Gate "${gateName}" is already approved`));
-      console.log(chalk.gray(`Approved at: ${currentStatus.timestamp}`));
-      console.log(chalk.gray(`Approved by: ${currentStatus.approvedBy || 'Unknown'}\n`));
-      return;
-    }
-
-    // Set approval
-    await setApprovalGate(featureName, gateName, true, {
-      approvedBy: options.approver || process.env.USER || process.env.USERNAME || 'user',
-      approvedAt: new Date().toISOString(),
-      notes: options.notes
-    });
-
-    console.log(chalk.green(`\n✅ Gate "${gateName}" approved for feature: ${featureName}`));
-    console.log(chalk.gray(`Workflow can now proceed past ${gateName} phase\n`));
-
-  } catch (error) {
-    console.error(chalk.red(`\n❌ Error approving gate: ${error.message}\n`));
-    process.exit(1);
-  }
-}
-
-/**
- * Approve command - Mark a gate as approved
- */
-program
-  .command('approve <feature> <gate>')
-  .description('Approve a phase gate to allow workflow progression')
-  .action(approveCommand);
-
-/**
- * Approval status action function - Show all gates and their status
- */
-export async function approvalStatusCommand(featureName, options = {}) {
-  try {
-    // Get feature state
-    const feature = getFeature(featureName);
-    if (!feature) {
-      console.error(chalk.red(`\n❌ Feature not found: ${featureName}\n`));
-      process.exit(1);
-    }
-
-    // JSON output
-    if (options.json) {
-      const status = {
-        feature: featureName,
-        phase: feature.phase,
-        status: feature.status,
-        gates: {}
-      };
-
-      for (const gateName of VALID_GATES) {
-        status.gates[gateName] = getApprovalGate(featureName, gateName) || { approved: false };
-      }
-
-      console.log(JSON.stringify(status, null, 2));
-      return;
-    }
-
-    console.log(chalk.bold(`\n📋 Approval Status for: ${featureName}`));
-    console.log('━'.repeat(60));
-
-    // Show current phase
-    console.log(chalk.cyan(`\nCurrent Phase: ${feature.phase || 'unknown'}`));
-    console.log(chalk.gray(`Feature Status: ${feature.status || 'unknown'}\n`));
-
-    // Show each gate
-    console.log(chalk.bold('Approval Gates:'));
-
-    for (const gateName of VALID_GATES) {
-      const gateStatus = getApprovalGate(featureName, gateName);
-
-      if (!gateStatus) {
-        console.log(`  ${chalk.gray('○')} ${gateName}: ${chalk.gray('Not yet evaluated')}`);
-        continue;
-      }
-
-      if (gateStatus.approved) {
-        console.log(`  ${chalk.green('✓')} ${gateName}: ${chalk.green('Approved')}`);
-        if (gateStatus.timestamp) {
-          console.log(chalk.gray(`      Approved at: ${gateStatus.timestamp}`));
-        }
-        if (gateStatus.approvedBy) {
-          console.log(chalk.gray(`      Approved by: ${gateStatus.approvedBy}`));
-        }
-      } else {
-        console.log(`  ${chalk.red('✗')} ${gateName}: ${chalk.red('Rejected / Not Approved')}`);
-        if (gateStatus.reason) {
-          console.log(chalk.gray(`      Reason: ${gateStatus.reason}`));
-        }
-        if (gateStatus.timestamp) {
-          console.log(chalk.gray(`      Rejected at: ${gateStatus.timestamp}`));
-        }
-      }
-    }
-
-    // Show approval history if available
-    const history = getApprovalHistory(featureName);
-    if (history && history.length > 0) {
-      console.log(chalk.bold('\nApproval History:'));
-      history.slice(-5).forEach(entry => {
-        const status = entry.approved ? chalk.green('✓ Approved') : chalk.red('✗ Rejected');
-        console.log(`  ${status} ${entry.gate} - ${entry.timestamp}`);
-      });
-    }
-
-    console.log('\n' + '━'.repeat(60) + '\n');
-
-  } catch (error) {
-    console.error(chalk.red(`\n❌ Error showing approval status: ${error.message}\n`));
-    process.exit(1);
-  }
-}
-
-/**
- * Approval status command - Show all gates and their status
- */
-program
-  .command('approval-status <feature>')
-  .alias('status')
-  .description('Show approval status for all gates')
-  .action(approvalStatusCommand);
-
-// Only parse if run directly (not imported as module)
-if (import.meta.url === `file://${process.argv[1]}`) {
-  if (process.argv.length > 2) {
-    program.parse(process.argv);
-  } else {
-    program.help();
-  }
-}
-
-export default program;
+import { Command } from 'commander';
+import chalk from 'chalk';
+import { getFeature, setApprovalGate, getApprovalGate, getApprovalHistory } from '../../core/state/state-manager.js';
+
+/**
+ * Unapprove action function - Revoke a previously approved gate
+ */
+export async function unapproveCommand(featureName, gateName, options = {}) {
+  try {
+    if (!VALID_GATES.includes(gateName)) {
+      console.error(chalk.red(`\n❌ Invalid gate: ${gateName}`));
+      console.log(chalk.gray(`Valid gates: ${VALID_GATES.join(', ')}\n`));
+      process.exit(1);
+    }
+
+    const feature = getFeature(featureName);
+    if (!feature) {
+      console.error(chalk.red(`\n❌ Feature not found: ${featureName}\n`));
+      process.exit(1);
+    }
+
+    const currentStatus = getApprovalGate(featureName, gateName);
+    if (!currentStatus?.approved) {
+      console.log(chalk.yellow(`\n⚠️  Gate "${gateName}" is not currently approved — nothing to revoke\n`));
+      return;
+    }
+
+    // Same honesty rule as approveCommand: don't pretend a human revoked
+    // the gate when there's no interactive shell behind the invocation.
+    let revokedBy;
+    if (options.revoker) {
+      revokedBy = options.revoker;
+    } else if (options.mode === 'auto') {
+      revokedBy = 'auto';
+    } else if (process.stdin.isTTY) {
+      revokedBy = process.env.USER || process.env.USERNAME || 'user';
+    } else {
+      revokedBy = 'auto';
+    }
+
+    await setApprovalGate(featureName, gateName, false, {
+      revokedBy,
+      revokedAt: new Date().toISOString(),
+      reason: options.reason
+    });
+
+    console.log(chalk.yellow(`\n↩️  Gate "${gateName}" approval revoked for feature: ${featureName}`));
+    console.log(chalk.gray(`Workflow is now blocked at the ${gateName} gate\n`));
+
+  } catch (error) {
+    console.error(chalk.red(`\n❌ Error revoking gate: ${error.message}\n`));
+    process.exit(1);
+  }
+}
+
+/**
+ * Approval Commands - Manage approval gates for MORPH workflow
+ *
+ * Commands:
+ * - approve: Mark a gate as approved
+ * - unapprove: Revoke a gate approval
+ * - approval-status: Show approval status for all gates
+ */
+
+const program = new Command();
+
+/**
+ * Valid approval gate names
+ */
+const VALID_GATES = ['proposal', 'uiux', 'plan', 'review'];
+
+/**
+ * Approve action function - Mark a gate as approved
+ */
+export async function approveCommand(featureName, gateName, options = {}) {
+  try {
+    // Validate gate name
+    if (!VALID_GATES.includes(gateName)) {
+      console.error(chalk.red(`\n❌ Invalid gate: ${gateName}`));
+      console.log(chalk.gray(`Valid gates: ${VALID_GATES.join(', ')}\n`));
+      process.exit(1);
+    }
+
+    // Get feature state
+    const feature = getFeature(featureName);
+    if (!feature) {
+      console.error(chalk.red(`\n❌ Feature not found: ${featureName}\n`));
+      process.exit(1);
+    }
+
+    // Check if already approved
+    const currentStatus = getApprovalGate(featureName, gateName);
+    if (currentStatus?.approved) {
+      console.log(chalk.yellow(`\n⚠️  Gate "${gateName}" is already approved`));
+      console.log(chalk.gray(`Approved at: ${currentStatus.timestamp}`));
+      console.log(chalk.gray(`Approved by: ${currentStatus.approvedBy || 'Unknown'}\n`));
+      return;
+    }
+
+    // Set approval.
+    //
+    // approvedBy resolution (most explicit wins):
+    //   1. --approver flag (CLI caller signalled who approved)
+    //   2. --mode auto  → "auto" (intentionally non-human signature)
+    //   3. interactive shell (TTY on stdin) → $USER / $USERNAME (a real
+    //      human is at the terminal; their name is the honest answer)
+    //   4. non-interactive (no TTY) → "auto"
+    //
+    // The TTY check exists because morph-spec is increasingly invoked
+    // from headless contexts (claude -p subprocesses, CI, the
+    // morph-e2e-tester). Stamping the host's $USERNAME on a gate that
+    // no human actually approved is misleading — it lies to the audit
+    // trail. "auto" reads as what it is: a deterministic policy gate,
+    // not a human signing off.
+    let approvedBy;
+    if (options.approver) {
+      approvedBy = options.approver;
+    } else if (options.mode === 'auto') {
+      approvedBy = 'auto';
+    } else if (process.stdin.isTTY) {
+      approvedBy = process.env.USER || process.env.USERNAME || 'user';
+    } else {
+      approvedBy = 'auto';
+    }
+
+    await setApprovalGate(featureName, gateName, true, {
+      approvedBy,
+      approvedAt: new Date().toISOString(),
+      notes: options.notes
+    });
+
+    console.log(chalk.green(`\n✅ Gate "${gateName}" approved for feature: ${featureName}`));
+    console.log(chalk.gray(`Workflow can now proceed past ${gateName} phase\n`));
+
+  } catch (error) {
+    console.error(chalk.red(`\n❌ Error approving gate: ${error.message}\n`));
+    process.exit(1);
+  }
+}
+
+/**
+ * Approve command - Mark a gate as approved
+ */
+program
+  .command('approve <feature> <gate>')
+  .description('Approve a phase gate to allow workflow progression')
+  .action(approveCommand);
+
+/**
+ * Approval status action function - Show all gates and their status
+ */
+export async function approvalStatusCommand(featureName, options = {}) {
+  try {
+    // Get feature state
+    const feature = getFeature(featureName);
+    if (!feature) {
+      console.error(chalk.red(`\n❌ Feature not found: ${featureName}\n`));
+      process.exit(1);
+    }
+
+    // JSON output
+    if (options.json) {
+      const status = {
+        feature: featureName,
+        phase: feature.phase,
+        status: feature.status,
+        gates: {}
+      };
+
+      for (const gateName of VALID_GATES) {
+        status.gates[gateName] = getApprovalGate(featureName, gateName) || { approved: false };
+      }
+
+      console.log(JSON.stringify(status, null, 2));
+      return;
+    }
+
+    console.log(chalk.bold(`\n📋 Approval Status for: ${featureName}`));
+    console.log('━'.repeat(60));
+
+    // Show current phase
+    console.log(chalk.cyan(`\nCurrent Phase: ${feature.phase || 'unknown'}`));
+    console.log(chalk.gray(`Feature Status: ${feature.status || 'unknown'}\n`));
+
+    // Show each gate
+    console.log(chalk.bold('Approval Gates:'));
+
+    for (const gateName of VALID_GATES) {
+      const gateStatus = getApprovalGate(featureName, gateName);
+
+      if (!gateStatus) {
+        console.log(`  ${chalk.gray('○')} ${gateName}: ${chalk.gray('Not yet evaluated')}`);
+        continue;
+      }
+
+      if (gateStatus.approved) {
+        console.log(`  ${chalk.green('✓')} ${gateName}: ${chalk.green('Approved')}`);
+        if (gateStatus.timestamp) {
+          console.log(chalk.gray(`      Approved at: ${gateStatus.timestamp}`));
+        }
+        if (gateStatus.approvedBy) {
+          console.log(chalk.gray(`      Approved by: ${gateStatus.approvedBy}`));
+        }
+      } else {
+        console.log(`  ${chalk.red('✗')} ${gateName}: ${chalk.red('Rejected / Not Approved')}`);
+        if (gateStatus.reason) {
+          console.log(chalk.gray(`      Reason: ${gateStatus.reason}`));
+        }
+        if (gateStatus.timestamp) {
+          console.log(chalk.gray(`      Rejected at: ${gateStatus.timestamp}`));
+        }
+      }
+    }
+
+    // Show approval history if available
+    const history = getApprovalHistory(featureName);
+    if (history && history.length > 0) {
+      console.log(chalk.bold('\nApproval History:'));
+      history.slice(-5).forEach(entry => {
+        const status = entry.approved ? chalk.green('✓ Approved') : chalk.red('✗ Rejected');
+        console.log(`  ${status} ${entry.gate} - ${entry.timestamp}`);
+      });
+    }
+
+    console.log('\n' + '━'.repeat(60) + '\n');
+
+  } catch (error) {
+    console.error(chalk.red(`\n❌ Error showing approval status: ${error.message}\n`));
+    process.exit(1);
+  }
+}
+
+/**
+ * Approval status command - Show all gates and their status
+ */
+program
+  .command('approval-status <feature>')
+  .alias('status')
+  .description('Show approval status for all gates')
+  .action(approvalStatusCommand);
+
+// Only parse if run directly (not imported as module)
+if (import.meta.url === `file://${process.argv[1]}`) {
+  if (process.argv.length > 2) {
+    program.parse(process.argv);
+  } else {
+    program.help();
+  }
+}
+
+export default program;

package/src/commands/state/archive.js

@@ -0,0 +1,111 @@
+import chalk from 'chalk';
+import { existsSync, renameSync, mkdirSync, readFileSync, writeFileSync } from 'fs';
+import { join } from 'path';
+import { execSync } from 'child_process';
+import { getFeature, getApprovalGate } from '../../core/state/state-manager.js';
+
+/**
+ * `morph-spec archive <feature>` — atomic, git-aware archive of a completed
+ * feature. Replaces the manual `/morph-archive` mv: it stamps the feature
+ * completed, moves `.morph/features/{name}/` → `.morph/archive/{name}/`
+ * (staging the move in git when possible), and cleans the thin root index.
+ *
+ * Taxonomy: archived = `status: "completed"` + `archivedAt` (per the project
+ * decision — no separate "archived" status value).
+ */
+
+function isGitRepo(cwd) {
+  try {
+    execSync('git rev-parse --is-inside-work-tree', { cwd, stdio: 'pipe' });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+export async function archiveCommand(featureName, options = {}) {
+  try {
+    const feature = getFeature(featureName);
+    if (!feature) {
+      console.error(chalk.red(`\n❌ Feature not found: ${featureName}\n`));
+      process.exit(1);
+    }
+
+    // Precondition: Gate 3 (review) approved — archiving is for completed work.
+    const reviewGate = getApprovalGate(featureName, 'review');
+    if (!reviewGate?.approved && !options.force) {
+      console.error(chalk.red(`\n❌ Gate 3 (review) is not approved for "${featureName}".`));
+      console.log(chalk.gray(`Approve it first (morph-spec approve ${featureName} review) or pass --force.\n`));
+      process.exit(1);
+    }
+
+    const cwd = process.cwd();
+    const featuresPath = join(cwd, '.morph', 'features', featureName);
+    if (!existsSync(featuresPath)) {
+      console.error(chalk.red(`\n❌ Feature folder not found: .morph/features/${featureName}\n`));
+      process.exit(1);
+    }
+    const archiveDir = join(cwd, '.morph', 'archive');
+    const archivePath = join(archiveDir, featureName);
+    if (existsSync(archivePath)) {
+      console.error(chalk.red(`\n❌ Already archived: .morph/archive/${featureName}\n`));
+      process.exit(1);
+    }
+
+    // 1) Stamp the authoritative feature.json (in place) as completed + archived.
+    const fjPath = join(featuresPath, 'feature.json');
+    if (existsSync(fjPath)) {
+      const fj = JSON.parse(readFileSync(fjPath, 'utf8'));
+      fj.status = 'completed';
+      fj.archivedAt = new Date().toISOString();
+      fj.updatedAt = fj.archivedAt;
+      writeFileSync(fjPath, JSON.stringify(fj, null, 2), 'utf8');
+    }
+
+    // 2) Move the folder. Prefer `git mv` (keeps history + stages in one step);
+    //    fall back to a filesystem rename (+ git add) when the folder isn't
+    //    tracked yet or git mv fails.
+    mkdirSync(archiveDir, { recursive: true });
+    const useGit = isGitRepo(cwd) && options.git !== false;
+    let staged = false;
+    if (useGit) {
+      try {
+        execSync(`git mv "${featuresPath}" "${archivePath}"`, { cwd, stdio: 'pipe' });
+        staged = true;
+      } catch {
+        renameSync(featuresPath, archivePath);
+        try {
+          execSync(`git add -A "${featuresPath}" "${archivePath}"`, { cwd, stdio: 'pipe' });
+          staged = true;
+        } catch { /* not fatal — the move succeeded */ }
+      }
+    } else {
+      renameSync(featuresPath, archivePath);
+    }
+
+    // 3) Drop the feature from the thin root index. We edit the index file
+    //    directly rather than calling saveState — saveState would re-create
+    //    .morph/features/{name}/feature.json and undo the archive.
+    const rootPath = join(cwd, '.morph', 'state.json');
+    if (existsSync(rootPath)) {
+      try {
+        const root = JSON.parse(readFileSync(rootPath, 'utf8'));
+        if (root.features && root.features[featureName]) {
+          delete root.features[featureName];
+          root.metadata = root.metadata || {};
+          root.metadata.totalFeatures = Object.keys(root.features).length;
+          writeFileSync(rootPath, JSON.stringify(root, null, 2), 'utf8');
+        }
+      } catch { /* index is derivable — non-fatal */ }
+    }
+
+    console.log(chalk.green(`\n📦 Archived "${featureName}" → .morph/archive/${featureName}/`));
+    console.log(chalk.gray(`   status: completed, archivedAt stamped in feature.json.`));
+    if (staged) console.log(chalk.gray(`   Move staged in git.\n`));
+    else if (useGit) console.log(chalk.gray(`   Move complete; stage it manually if needed.\n`));
+    else console.log('');
+  } catch (error) {
+    console.error(chalk.red(`\n❌ Error archiving feature: ${error.message}\n`));
+    process.exit(1);
+  }
+}