@boyingliu01/xp-gate 0.9.3 → 0.9.4

package/lib/sprint-status.js

@@ -9,22 +9,7 @@
 
 const fs = require('fs');
 const path = require('path');
-
-const PHASE_NAMES = {
-  '-1': 'ISOLATE',
-  '-0.5': 'AUTO-ESTIMATE',
-  '0': 'THINK',
-  '1': 'PLAN',
-  '2': 'BUILD',
-  '3': 'REVIEW',
-  '4': 'USER ACCEPT',
-  '5': 'FEEDBACK',
-  '6': 'SHIP',
-  '7': 'LAND',
-  '8': 'CLEANUP',
-};
-
-const PHASE_ORDER = ['-1', '-0.5', '0', '1', '2', '3', '4', '5', '6', '7', '8'];
+const { PHASE_NAMES, PHASE_ORDER, getLatestTimestamp, isStale } = require('./shared-phase-constants');
 
 /**
  * Read and parse sprint-state.json from a project directory.
@@ -70,29 +55,78 @@ function formatDuration(seconds) {
 }
 
 /**
- * Check if the state is stale (>1h since last activity).
+ * Format the sprint header lines (task description, ID, branch).
+ * @param {object} state - Sprint state object
+ * @param {string} branch - Branch name
+ * @returns {string[]} Header lines
+ */
+function buildHeader(state, branch) {
+  return [
+    `Sprint: ${state.task_description}`,
+    `ID: ${state.id || 'unknown'}  |  Branch: ${branch}`,
+  ];
+}
+
+/**
+ * Format the metrics line (tests, coverage).
+ * @param {object} metrics - Sprint metrics object
+ * @returns {string|null} Metrics line or null if no metrics
+ */
+function buildMetricsLine(metrics) {
+  const parts = [];
+  if (metrics.tests_passed != null) {
+    parts.push(`Tests: ${metrics.tests_passed} passed${metrics.tests_failed ? `, ${metrics.tests_failed} failed` : ''}`);
+  }
+  if (metrics.coverage_pct != null) {
+    parts.push(`Coverage: ${metrics.coverage_pct}%`);
+  }
+  return parts.length > 0 ? `Metrics: ${parts.join('  |  ')}` : null;
+}
+
+/**
+ * Build a phase lookup map from phase_history array.
  * @param {object} state - Sprint state object
- * @returns {boolean}
+ * @returns {object} Map of phase key → phase history entry
  */
-function isStale(state) {
-  if (!state || !state.started_at) return false;
-  const started = new Date(state.started_at).getTime();
-  if (isNaN(started)) return false;
-  // Check latest phase_history timestamp (started_at or completed_at)
-  let latest = started;
-  if (Array.isArray(state.phase_history) && state.phase_history.length > 0) {
+function buildPhaseLookup(state) {
+  const map = {};
+  if (Array.isArray(state.phase_history)) {
     for (const ph of state.phase_history) {
-      if (ph.completed_at) {
-        const t = new Date(ph.completed_at).getTime();
-        if (!isNaN(t) && t > latest) latest = t;
-      }
-      if (ph.started_at) {
-        const t = new Date(ph.started_at).getTime();
-        if (!isNaN(t) && t > latest) latest = t;
-      }
+      map[String(ph.phase)] = ph;
     }
   }
-  return Date.now() - latest > 3600000; // 1h
+  return map;
+}
+
+/**
+ * Format a single phase row line.
+ * @param {string} key - Phase key (e.g. '0', '1')
+ * @param {object|undefined} history - Phase history entry
+ * @param {number} maxNameLen - Max phase name length for padding
+ * @returns {string} Formatted phase line
+ */
+function formatPhaseLine(key, history, maxNameLen) {
+  const name = history?.phase_name || PHASE_NAMES[key] || key;
+  const status = history?.status || 'pending';
+  const icon = statusIcon(status);
+  const dur = formatDuration(history?.duration_seconds);
+  const statusLabel = status === 'completed' ? 'Completed' :
+    status === 'in_progress' ? 'In Progress' : 'Pending';
+  return `  Phase ${key.padStart(4)}  ${name.padEnd(maxNameLen + 1)} ${icon} ${dur.padEnd(5)} ${statusLabel}`;
+}
+
+/**
+ * Format REQ-level progress sub-lines for a phase (typically BUILD).
+ * @param {object|undefined} history - Phase history entry with optional reqs
+ * @returns {string[]} Array of REQ progress lines (may be empty)
+ */
+function formatReqLines(history) {
+  if (!history?.reqs) return [];
+  const lines = [];
+  for (const [reqId, req] of Object.entries(history.reqs)) {
+    lines.push(`    ${statusIcon(req.status)} ${reqId}  ${req.name}`);
+  }
+  return lines;
 }
 
 /**
@@ -105,77 +139,35 @@ function formatSprintTable(state) {
     return 'No active sprint in this directory';
   }
 
-  const lines = [];
   const branch = state.isolation?.branch || 'unknown';
   const metrics = state.metrics || {};
+  const lines = buildHeader(state, branch);
 
-  // Header
-  lines.push(`Sprint: ${state.task_description}`);
-  lines.push(`ID: ${state.id || 'unknown'}  |  Branch: ${branch}`);
-
-  // Metrics line (if available)
-  const metricParts = [];
-  if (metrics.tests_passed != null) {
-    metricParts.push(`Tests: ${metrics.tests_passed} passed${metrics.tests_failed ? `, ${metrics.tests_failed} failed` : ''}`);
-  }
-  if (metrics.coverage_pct != null) {
-    metricParts.push(`Coverage: ${metrics.coverage_pct}%`);
-  }
-  if (metricParts.length > 0) {
-    lines.push(`Metrics: ${metricParts.join('  |  ')}`);
-  }
+  const metricsLine = buildMetricsLine(metrics);
+  if (metricsLine) lines.push(metricsLine);
 
-  // Stale warning
   if (isStale(state)) {
     lines.push('⚠️  State may be stale (last updated >1h ago)');
   }
-
   lines.push('');
 
-  // Build phase lookup from phase_history
-  const historyByPhase = {};
-  if (Array.isArray(state.phase_history)) {
-    for (const ph of state.phase_history) {
-      historyByPhase[String(ph.phase)] = ph;
-    }
-  }
+  const historyByPhase = buildPhaseLookup(state);
 
-  // Calculate column widths dynamically
   let maxNameLen = 'Phase'.length;
   for (const key of PHASE_ORDER) {
-    const history = historyByPhase[key];
-    const name = history?.phase_name || PHASE_NAMES[key] || key;
+    const name = historyByPhase[key]?.phase_name || PHASE_NAMES[key] || key;
     if (name.length > maxNameLen) maxNameLen = name.length;
   }
 
-  // Separator line
   const sep = '─'.repeat(maxNameLen + 40);
-
   lines.push(sep);
 
   for (const key of PHASE_ORDER) {
-    const history = historyByPhase[key];
-    const name = history?.phase_name || PHASE_NAMES[key] || key;
-    const status = history?.status || 'pending';
-    const icon = statusIcon(status);
-    const dur = formatDuration(history?.duration_seconds);
-    const statusLabel = status === 'completed' ? 'Completed' :
-      status === 'in_progress' ? 'In Progress' : 'Pending';
-
-    const line = `  Phase ${key.padStart(4)}  ${name.padEnd(maxNameLen + 1)} ${icon} ${dur.padEnd(5)} ${statusLabel}`;
-    lines.push(line);
-
-    // REQ-level progress for BUILD phase
-    if (history?.reqs) {
-      for (const [reqId, req] of Object.entries(history.reqs)) {
-        const reqIcon = statusIcon(req.status);
-        lines.push(`    ${reqIcon} ${reqId}  ${req.name}`);
-      }
-    }
+    lines.push(formatPhaseLine(key, historyByPhase[key], maxNameLen));
+    lines.push(...formatReqLines(historyByPhase[key]));
   }
 
   lines.push(sep);
-
   return lines.join('\n');
 }
 

package/lib/upgrade.js

@@ -1,105 +1,116 @@
-const { execSync } = require('child_process');
+const { execSync, spawn } = require('child_process');
 const { checkUpgrade, formatUpgradeMsg, clearCache, getLocalVersion, getPackageName } = require('./check-version.js');
 
 /**
- * xp-gate upgrade command handler.
- *
- * Modes:
- *   xp-gate upgrade           — human-readable output
- *   xp-gate upgrade --preview — single-line JSON for plugin consumption
- *   xp-gate upgrade --apply   — auto-upgrade via npm install -g
- *
- * @param {string[]} args
- * @returns {Promise<number>} exit code
+ * Handle checkUpgrade() failure.
+ * @param {Error} err
+ * @param {boolean} isPreview
+ * @returns {number} exit code (always 1)
  */
-async function upgrade(args) {
-  const isPreview = args.includes('--preview');
-  const isApply = args.includes('--apply');
-  const pkgName = getPackageName();
-
-  // --- check for upgrade ---
-  let result;
-  try {
-    result = await checkUpgrade(pkgName);
-  } catch (err) {
-    if (isPreview) {
-      console.log(JSON.stringify({ error: 'check failed', detail: err.message }));
-    } else {
-      console.error('Unable to check for updates (check error).');
-    }
-    return 1;
-  }
-
-  // Guard: if checkUpgrade returned null remote (network issue)
-  if (!result.remote) {
-    if (isPreview) {
-      console.log(JSON.stringify({
-        local: result.local || 'unknown',
-        remote: null,
-        outdated: false,
-        lagDays: 0,
-        error: 'Unable to check for updates (network issue)',
-      }));
-    } else {
-      console.error('Unable to check for updates (network issue).');
-    }
-    return 0;
+function handleCheckError(err, isPreview) {
+  if (isPreview) {
+    console.log(JSON.stringify({ error: 'check failed', detail: err.message }));
+  } else {
+    console.error('Unable to check for updates (check error).');
   }
+  return 1;
+}
 
-  // --- --preview: JSON output ---
+/**
+ * Handle null remote (network issue) after checkUpgrade().
+ * @param {{ local: string|null }} result
+ * @param {boolean} isPreview
+ * @returns {number} exit code (always 0)
+ */
+function handleNullRemote(result, isPreview) {
   if (isPreview) {
-    const releaseUrl = result.outdated
-      ? `https://github.com/boyingliu01/xp-gate/releases/tag/v${result.remote}`
-      : null;
     console.log(JSON.stringify({
-      local: result.local,
-      remote: result.remote,
-      outdated: result.outdated,
-      lagDays: result.lagDays,
-      releaseUrl,
-      publishedAt: result.publishedAt || null,
+      local: result.local || 'unknown',
+      remote: null,
+      outdated: false,
+      lagDays: 0,
+      error: 'Unable to check for updates (network issue)',
     }));
-    return 0;
+  } else {
+    console.error('Unable to check for updates (network issue).');
   }
+  return 0;
+}
+
+/**
+ * Handle --preview mode: emit single-line JSON.
+ * @param {{ local: string, remote: string, outdated: boolean, lagDays: number, publishedAt?: string }} result
+ * @returns {number} exit code (always 0)
+ */
+function handlePreviewMode(result) {
+  const releaseUrl = result.outdated
+    ? `https://github.com/boyingliu01/xp-gate/releases/tag/v${result.remote}`
+    : null;
+  console.log(JSON.stringify({
+    local: result.local,
+    remote: result.remote,
+    outdated: result.outdated,
+    lagDays: result.lagDays,
+    releaseUrl,
+    publishedAt: result.publishedAt || null,
+  }));
+  return 0;
+}
 
-  // --- --apply: auto-upgrade ---
-  if (isApply) {
-    if (!result.outdated) {
-      if (result.local) {
-        console.log(`\u2713 xp-gate v${result.local} is up to date`);
-      } else {
-        console.log('xp-gate is up to date.');
-      }
-      return 0;
+/**
+ * Handle --apply mode: auto-upgrade via npm install -g.
+ * @param {{ local: string, remote: string, outdated: boolean }} result
+ * @param {string} pkgName
+ * @returns {Promise<number>} exit code
+ */
+async function handleApplyMode(result, pkgName) {
+  if (!result.outdated) {
+    if (result.local) {
+      console.log(`\u2713 xp-gate v${result.local} is up to date`);
+    } else {
+      console.log('xp-gate is up to date.');
     }
+    return 0;
+  }
 
-    console.log(`Upgrading xp-gate from v${result.local} to v${result.remote}...`);
-    try {
-      execSync(`npm install -g ${pkgName}@${result.remote}`, {
-        stdio: 'inherit',
-        timeout: 120000, // 2 minute timeout
+  console.log(`Upgrading xp-gate from v${result.local} to v${result.remote}...`);
+  try {
+    const child = spawn('npm', ['install', '-g', `${pkgName}@${result.remote}`], {
+      stdio: 'inherit',
+      timeout: 120000,
+    });
+    await new Promise((resolve, reject) => {
+      child.on('close', (code) => {
+        if (code === 0) resolve();
+        else reject(new Error(`npm install exited with code ${code}`));
       });
-      // clear cache so next check picks up the new version
-      clearCache();
-      console.log(`\u2713 Upgraded to v${result.remote}`);
-      return 0;
-    } catch (err) {
-      const stderr = (err.stderr || '').toString();
-      if (stderr.includes('EACCES') || stderr.includes('permission') || stderr.includes('EPERM')) {
-        console.error(`Permission denied. Try: sudo npm install -g ${pkgName}`);
-      } else if (err.code === 'ETIMEDOUT' || stderr.includes('ETIMEDOUT')) {
-        console.error('npm install timed out. Check your network and try again.');
-        console.error(`  Retry: npm install -g ${pkgName}@latest`);
-      } else {
-        console.error('Upgrade failed:');
-        console.error(`  ${err.message}`);
-        console.error(`  Retry manually: npm install -g ${pkgName}@latest`);
-      }
-      return 1;
+      child.on('error', reject);
+    });
+    clearCache();
+    console.log(`\u2713 Upgraded to v${result.remote}`);
+    return 0;
+  } catch (err) {
+    const msg = err.message || '';
+    if (msg.includes('EACCES') || msg.includes('permission') || msg.includes('EPERM')) {
+      console.error(`Permission denied. Try: sudo npm install -g ${pkgName}`);
+    } else if (msg.includes('ETIMEDOUT')) {
+      console.error('npm install timed out. Check your network and try again.');
+      console.error(`  Retry: npm install -g ${pkgName}@latest`);
+    } else {
+      console.error('Upgrade failed:');
+      console.error(`  ${msg}`);
+      console.error(`  Retry manually: npm install -g ${pkgName}@latest`);
     }
+    return 1;
   }
+}
 
-  // --- default mode: human-readable ---
+/**
+ * Handle default mode: human-readable output.
+ * @param {{ local: string, remote: string, outdated: boolean, lagDays: number, publishedAt?: string }} result
+ * @returns {number} exit code (always 0)
+ */
+function handleDefaultMode(result) {
   if (!result.outdated) {
     if (result.local) {
       console.log(`\u2713 xp-gate v${result.local} is up to date`);
@@ -113,4 +124,36 @@ async function upgrade(args) {
   return 0;
 }
 
+/**
+ * xp-gate upgrade command handler.
+ *
+ * Modes:
+ *   xp-gate upgrade           — human-readable output
+ *   xp-gate upgrade --preview — single-line JSON for plugin consumption
+ *   xp-gate upgrade --apply   — auto-upgrade via npm install -g
+ *
+ * @param {string[]} args
+ * @returns {Promise<number>} exit code
+ */
+async function upgrade(args) {
+  const isPreview = args.includes('--preview');
+  const isApply = args.includes('--apply');
+  const pkgName = getPackageName();
+
+  let result;
+  try {
+    result = await checkUpgrade(pkgName);
+  } catch (err) {
+    return handleCheckError(err, isPreview);
+  }
+
+  if (!result.remote) {
+    return handleNullRemote(result, isPreview);
+  }
+
+  if (isPreview) return handlePreviewMode(result);
+  if (isApply) return handleApplyMode(result, pkgName);
+  return handleDefaultMode(result);
+}
+
 module.exports = { upgrade };

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Mock layering policy enforcement — Gate M3 of pre-push hook. Ensures integration tests use real implementations for internal dependencies, mock external dependencies, and annotate pending mocks with removal plans. Combines project scope scanning, mock decision engine, and per-file validation into a single pipeline.

package/mutation/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MUTATION KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 **Gate M** (incremental mutation testing) + **Gate M2** helpers (test-layer detection used by `src/mock-policy/`). Pre-push quality gate. TypeScript-only; uses Stryker.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/xp-gate",
-  "version": "0.9.3",
+  "version": "0.9.4",
   "description": "AI-driven development workflow: 6 quality gates + Delphi review + Sprint Flow",
   "bin": {
     "xp-gate": "./bin/xp-gate.js"

package/plugins/claude-code/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.9.3",
+  "version": "0.9.4",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Claude Code. Includes 10 quality gates (Gate 0-9), Sprint Flow (11 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/claude-code/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/plugins/claude-code/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 **11-phase** development pipeline: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → USER ACCEPTANCE → FEEDBACK → SHIP → LAND → CLEANUP. Phase 2 default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE in Phase 1: design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/claude-code/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-06-18
-**Commit:** b67fc85
+**Generated:** 2026-06-19
+**Commit:** b67f7f9
 **Branch:** main
-**Version:** 0.9.3.0
+**Version:** 0.9.4.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.