@polymorphism-tech/morph-spec 7.8.1 → 7.9.0

package/src/lib/tasks/task-classifier.js

@@ -1,30 +1,30 @@
-/**
- * Task classifier — identifies test/validation and infrastructure tasks by title.
- */
-
-const TEST_TASK_PATTERN = /\b(tests?|testing|specs?|coverage|e2e|unit|integration|integra[cç][aã]o|valida[cç][aã]o|cobertura|testes?)\b/i;
-
-/**
- * Returns true if the task title indicates a testing or validation task.
- *
- * @param {string} taskTitle
- * @returns {boolean}
- */
-export function isTestTask(taskTitle) {
-  if (!taskTitle) return false;
-  return TEST_TASK_PATTERN.test(taskTitle);
-}
-
-const INFRA_TASK_PATTERN = /\b(infrastructure|infra|cleanup|migration|migra[cç][aã]o|config|setup|ci\/cd|devops|deploy|docker|packages?|dependencies|tags)\b/i;
-
-/**
- * Returns true if the task title indicates an infrastructure/cleanup/migration task.
- * Used to skip contract validation and suppress breaking change warnings.
- *
- * @param {string} taskTitle
- * @returns {boolean}
- */
-export function isInfrastructureTask(taskTitle) {
-  if (!taskTitle) return false;
-  return INFRA_TASK_PATTERN.test(taskTitle);
-}
+/**
+ * Task classifier — identifies test/validation and infrastructure tasks by title.
+ */
+
+const TEST_TASK_PATTERN = /\b(tests?|testing|specs?|coverage|e2e|unit|integration|integra[cç][aã]o|valida[cç][aã]o|cobertura|testes?)\b/i;
+
+/**
+ * Returns true if the task title indicates a testing or validation task.
+ *
+ * @param {string} taskTitle
+ * @returns {boolean}
+ */
+export function isTestTask(taskTitle) {
+  if (!taskTitle) return false;
+  return TEST_TASK_PATTERN.test(taskTitle);
+}
+
+const INFRA_TASK_PATTERN = /\b(infrastructure|infra|cleanup|migration|migra[cç][aã]o|config|setup|ci\/cd|devops|deploy|docker|packages?|dependencies|tags)\b/i;
+
+/**
+ * Returns true if the task title indicates an infrastructure/cleanup/migration task.
+ * Used to skip contract validation and suppress breaking change warnings.
+ *
+ * @param {string} taskTitle
+ * @returns {boolean}
+ */
+export function isInfrastructureTask(taskTitle) {
+  if (!taskTitle) return false;
+  return INFRA_TASK_PATTERN.test(taskTitle);
+}

package/src/lib/tasks/task-parser.js

@@ -1,209 +1,209 @@
-/**
- * Task Parser — Shared utilities for reading, parsing, and grouping tasks.
- *
- * Extracted from bin/task-manager.cjs and dispatch-agents.js so all
- * consumers share the same parsing logic without duplication.
- */
-
-import { readFile } from 'fs/promises';
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
-
-// ============================================================================
-// TASK_CATEGORY_TO_GROUP — maps task categories to parallel groups
-// ============================================================================
-
-const TASK_CATEGORY_TO_GROUP = {
-  'domain': 'backend',
-  'domain-bc': 'backend',
-  'infrastructure': 'backend',
-  'application': 'backend',
-  'presentation': 'frontend',
-  'tests': 'tests',
-  'infra': 'infra',
-  'docs': 'docs',
-};
-
-// ============================================================================
-// parseTaskList — structured task parsing with error reporting
-// ============================================================================
-
-/**
- * Parse tasks.md content into structured task objects with error reporting.
- *
- * @param {string} content - Raw tasks.md content
- * @returns {{ tasks: ParsedTask[], errors: TaskParseError[], groups: Object<string, string[]> }}
- */
-export function parseTaskList(content) {
-  const tasks = [];
-  const errors = [];
-  const groups = {};
-
-  const taskHeadingRe = /^###\s+(T\d{3,})\s*[—–:\-]\s*(.+)$/gm;
-  const groupRe = /(?:\*\*Group\*\*\s*:\s*`?([a-z-]+)`?|- \*\*Group:\*\*\s*`?([a-z-]+)`?)/i;
-  const categoryRe = /(?:\*\*category\*\*\s*:\s*`?([a-z-]+)`?|category:\s*([a-z-]+))/i;
-
-  let headingMatch;
-  const taskPositions = [];
-
-  while ((headingMatch = taskHeadingRe.exec(content)) !== null) {
-    taskPositions.push({ id: headingMatch[1], title: headingMatch[2].trim(), pos: headingMatch.index });
-  }
-
-  for (let i = 0; i < taskPositions.length; i++) {
-    const { id, title, pos } = taskPositions[i];
-    const end = i + 1 < taskPositions.length ? taskPositions[i + 1].pos : content.length;
-    const block = content.slice(pos, end);
-
-    // Try group field first (v4), fall back to category field (v3)
-    const grpMatch = groupRe.exec(block);
-    const catMatch = categoryRe.exec(block);
-    let group;
-    if (grpMatch) {
-      group = (grpMatch[1] || grpMatch[2]).toLowerCase();
-    } else if (catMatch) {
-      const category = (catMatch[1] || catMatch[2]).toLowerCase();
-      group = TASK_CATEGORY_TO_GROUP[category] || 'backend';
-    } else {
-      group = 'backend';
-    }
-
-    tasks.push({ id, title, group });
-
-    if (!groups[group]) groups[group] = [];
-    groups[group].push(id);
-  }
-
-  return { tasks, errors, groups };
-}
-
-// ============================================================================
-// parseAndGroupTasks — drop-in replacement for dispatch-agents.js
-// ============================================================================
-
-/**
- * Parse tasks.md and return a map of { group -> taskId[] }.
- * Gracefully returns {} if the file doesn't exist or is unparseable.
- *
- * @param {string} tasksPath - Absolute path to tasks.md
- * @returns {Object<string, string[]>}
- */
-export function parseAndGroupTasks(tasksPath) {
-  if (!existsSync(tasksPath)) return {};
-
-  try {
-    const content = readFileSync(tasksPath, 'utf8');
-    return parseTaskList(content).groups;
-  } catch {
-    return {};
-  }
-}
-
-// ============================================================================
-// parseTasksMd
-// ============================================================================
-
-/**
- * Parse tasks.md to extract task stubs for the v3 state format.
- * Accepts headings in either format:
- *   ### T001 — Task title   (em-dash / en-dash / hyphen)
- *   ### T001: Task title    (colon)
- *
- * @param {string} featureName
- * @param {string} [cwd] - Project root (defaults to process.cwd())
- * @returns {Promise<Array<{id, title, status, dependencies, files, checkpoint}>>}
- */
-export async function parseTasksMd(featureName, cwd = process.cwd()) {
-  const tasksPath = join(cwd, `.morph/features/${featureName}/2-plan/tasks.md`);
-  let content = '';
-  try {
-    content = await readFile(tasksPath, 'utf-8');
-  } catch {
-    return [];
-  }
-
-  // Phase 1: collect heading positions
-  const headingRe = /^###\s+(T\d+)\s*[—–:\-]\s*(.+)$/gm;
-  const depRe = /\*\*Dependenc(?:y|ies):\*\*\s*(.+)/i;
-  const positions = [];
-  let match;
-  while ((match = headingRe.exec(content)) !== null) {
-    positions.push({ id: match[1], title: match[2].trim(), pos: match.index });
-  }
-
-  // Phase 2: extract body and parse dependencies
-  const tasks = [];
-  for (let i = 0; i < positions.length; i++) {
-    const { id, title, pos } = positions[i];
-    const end = i + 1 < positions.length ? positions[i + 1].pos : content.length;
-    const block = content.slice(pos, end);
-
-    let dependencies = [];
-    const depMatch = depRe.exec(block);
-    if (depMatch) {
-      const val = depMatch[1].trim();
-      if (val.toLowerCase() !== 'none') {
-        dependencies = val.split(/[,\s]+/).filter(s => /^T\d+$/.test(s));
-      }
-    }
-
-    tasks.push({ id, title, status: 'pending', dependencies, files: [], checkpoint: null });
-  }
-  return tasks;
-}
-
-// ============================================================================
-// ensureTaskList
-// ============================================================================
-
-/**
- * Ensure feature.taskList exists (array of individual task objects).
- *
- * In v3 state, feature.tasks is a counter object {total, completed, …}.
- * Individual task objects live in feature.taskList.
- *
- * @param {Object} feature - Feature state object (mutated in place)
- * @param {string} featureName
- * @param {string} [cwd]
- * @returns {Promise<Array>} The task list
- */
-export async function ensureTaskList(feature, featureName, cwd = process.cwd()) {
-  if (Array.isArray(feature.tasks)) {
-    // v2 format: tasks IS the array
-    return feature.tasks;
-  }
-  // v3 format: use taskList or build from tasks.md
-  if (!feature.taskList || feature.taskList.length === 0) {
-    feature.taskList = await parseTasksMd(featureName, cwd);
-  }
-  return feature.taskList;
-}
-
-// ============================================================================
-// syncCounters
-// ============================================================================
-
-/**
- * After modifying taskList, sync the counter fields back to feature.tasks.
- * tasks.total is derived from the actual taskList length (not trusted from state).
- *
- * @deprecated Prefer reading directly from feature.taskList. Counter fields
- * (feature.tasks) duplicate native Claude Code task tracking. This function
- * is kept for backward compatibility but will be removed in a future version.
- *
- * @param {Object} feature - Feature state object (mutated in place)
- */
-export function syncCounters(feature) {
-  if (Array.isArray(feature.tasks)) return; // v2 format — nothing to sync
-  // Safety: ensure feature.tasks is an object (may be undefined after migration or corruption)
-  if (!feature.tasks || typeof feature.tasks !== 'object') {
-    feature.tasks = { total: 0, completed: 0, inProgress: 0, pending: 0 };
-  }
-  const list = feature.taskList || [];
-  // Exclude expanded tasks — they are replaced by their sub-tasks
-  const active = list.filter(t => t.status !== 'expanded');
-  feature.tasks.total = active.length;
-  feature.tasks.completed = active.filter(t => t.status === 'completed').length;
-  feature.tasks.inProgress = active.filter(t => t.status === 'in_progress').length;
-  feature.tasks.pending = active.filter(t => t.status === 'pending').length;
-}
+/**
+ * Task Parser — Shared utilities for reading, parsing, and grouping tasks.
+ *
+ * Extracted from bin/task-manager.cjs and dispatch-agents.js so all
+ * consumers share the same parsing logic without duplication.
+ */
+
+import { readFile } from 'fs/promises';
+import { readFileSync, existsSync } from 'fs';
+import { join } from 'path';
+
+// ============================================================================
+// TASK_CATEGORY_TO_GROUP — maps task categories to parallel groups
+// ============================================================================
+
+const TASK_CATEGORY_TO_GROUP = {
+  'domain': 'backend',
+  'domain-bc': 'backend',
+  'infrastructure': 'backend',
+  'application': 'backend',
+  'presentation': 'frontend',
+  'tests': 'tests',
+  'infra': 'infra',
+  'docs': 'docs',
+};
+
+// ============================================================================
+// parseTaskList — structured task parsing with error reporting
+// ============================================================================
+
+/**
+ * Parse tasks.md content into structured task objects with error reporting.
+ *
+ * @param {string} content - Raw tasks.md content
+ * @returns {{ tasks: ParsedTask[], errors: TaskParseError[], groups: Object<string, string[]> }}
+ */
+export function parseTaskList(content) {
+  const tasks = [];
+  const errors = [];
+  const groups = {};
+
+  const taskHeadingRe = /^###\s+(T\d{3,})\s*[—–:\-]\s*(.+)$/gm;
+  const groupRe = /(?:\*\*Group\*\*\s*:\s*`?([a-z-]+)`?|- \*\*Group:\*\*\s*`?([a-z-]+)`?)/i;
+  const categoryRe = /(?:\*\*category\*\*\s*:\s*`?([a-z-]+)`?|category:\s*([a-z-]+))/i;
+
+  let headingMatch;
+  const taskPositions = [];
+
+  while ((headingMatch = taskHeadingRe.exec(content)) !== null) {
+    taskPositions.push({ id: headingMatch[1], title: headingMatch[2].trim(), pos: headingMatch.index });
+  }
+
+  for (let i = 0; i < taskPositions.length; i++) {
+    const { id, title, pos } = taskPositions[i];
+    const end = i + 1 < taskPositions.length ? taskPositions[i + 1].pos : content.length;
+    const block = content.slice(pos, end);
+
+    // Try group field first (v4), fall back to category field (v3)
+    const grpMatch = groupRe.exec(block);
+    const catMatch = categoryRe.exec(block);
+    let group;
+    if (grpMatch) {
+      group = (grpMatch[1] || grpMatch[2]).toLowerCase();
+    } else if (catMatch) {
+      const category = (catMatch[1] || catMatch[2]).toLowerCase();
+      group = TASK_CATEGORY_TO_GROUP[category] || 'backend';
+    } else {
+      group = 'backend';
+    }
+
+    tasks.push({ id, title, group });
+
+    if (!groups[group]) groups[group] = [];
+    groups[group].push(id);
+  }
+
+  return { tasks, errors, groups };
+}
+
+// ============================================================================
+// parseAndGroupTasks — drop-in replacement for dispatch-agents.js
+// ============================================================================
+
+/**
+ * Parse tasks.md and return a map of { group -> taskId[] }.
+ * Gracefully returns {} if the file doesn't exist or is unparseable.
+ *
+ * @param {string} tasksPath - Absolute path to tasks.md
+ * @returns {Object<string, string[]>}
+ */
+export function parseAndGroupTasks(tasksPath) {
+  if (!existsSync(tasksPath)) return {};
+
+  try {
+    const content = readFileSync(tasksPath, 'utf8');
+    return parseTaskList(content).groups;
+  } catch {
+    return {};
+  }
+}
+
+// ============================================================================
+// parseTasksMd
+// ============================================================================
+
+/**
+ * Parse tasks.md to extract task stubs for the v3 state format.
+ * Accepts headings in either format:
+ *   ### T001 — Task title   (em-dash / en-dash / hyphen)
+ *   ### T001: Task title    (colon)
+ *
+ * @param {string} featureName
+ * @param {string} [cwd] - Project root (defaults to process.cwd())
+ * @returns {Promise<Array<{id, title, status, dependencies, files, checkpoint}>>}
+ */
+export async function parseTasksMd(featureName, cwd = process.cwd()) {
+  const tasksPath = join(cwd, `.morph/features/${featureName}/2-plan/tasks.md`);
+  let content = '';
+  try {
+    content = await readFile(tasksPath, 'utf-8');
+  } catch {
+    return [];
+  }
+
+  // Phase 1: collect heading positions
+  const headingRe = /^###\s+(T\d+)\s*[—–:\-]\s*(.+)$/gm;
+  const depRe = /\*\*Dependenc(?:y|ies):\*\*\s*(.+)/i;
+  const positions = [];
+  let match;
+  while ((match = headingRe.exec(content)) !== null) {
+    positions.push({ id: match[1], title: match[2].trim(), pos: match.index });
+  }
+
+  // Phase 2: extract body and parse dependencies
+  const tasks = [];
+  for (let i = 0; i < positions.length; i++) {
+    const { id, title, pos } = positions[i];
+    const end = i + 1 < positions.length ? positions[i + 1].pos : content.length;
+    const block = content.slice(pos, end);
+
+    let dependencies = [];
+    const depMatch = depRe.exec(block);
+    if (depMatch) {
+      const val = depMatch[1].trim();
+      if (val.toLowerCase() !== 'none') {
+        dependencies = val.split(/[,\s]+/).filter(s => /^T\d+$/.test(s));
+      }
+    }
+
+    tasks.push({ id, title, status: 'pending', dependencies, files: [], checkpoint: null });
+  }
+  return tasks;
+}
+
+// ============================================================================
+// ensureTaskList
+// ============================================================================
+
+/**
+ * Ensure feature.taskList exists (array of individual task objects).
+ *
+ * In v3 state, feature.tasks is a counter object {total, completed, …}.
+ * Individual task objects live in feature.taskList.
+ *
+ * @param {Object} feature - Feature state object (mutated in place)
+ * @param {string} featureName
+ * @param {string} [cwd]
+ * @returns {Promise<Array>} The task list
+ */
+export async function ensureTaskList(feature, featureName, cwd = process.cwd()) {
+  if (Array.isArray(feature.tasks)) {
+    // v2 format: tasks IS the array
+    return feature.tasks;
+  }
+  // v3 format: use taskList or build from tasks.md
+  if (!feature.taskList || feature.taskList.length === 0) {
+    feature.taskList = await parseTasksMd(featureName, cwd);
+  }
+  return feature.taskList;
+}
+
+// ============================================================================
+// syncCounters
+// ============================================================================
+
+/**
+ * After modifying taskList, sync the counter fields back to feature.tasks.
+ * tasks.total is derived from the actual taskList length (not trusted from state).
+ *
+ * @deprecated Prefer reading directly from feature.taskList. Counter fields
+ * (feature.tasks) duplicate native Claude Code task tracking. This function
+ * is kept for backward compatibility but will be removed in a future version.
+ *
+ * @param {Object} feature - Feature state object (mutated in place)
+ */
+export function syncCounters(feature) {
+  if (Array.isArray(feature.tasks)) return; // v2 format — nothing to sync
+  // Safety: ensure feature.tasks is an object (may be undefined after migration or corruption)
+  if (!feature.tasks || typeof feature.tasks !== 'object') {
+    feature.tasks = { total: 0, completed: 0, inProgress: 0, pending: 0 };
+  }
+  const list = feature.taskList || [];
+  // Exclude expanded tasks — they are replaced by their sub-tasks
+  const active = list.filter(t => t.status !== 'expanded');
+  feature.tasks.total = active.length;
+  feature.tasks.completed = active.filter(t => t.status === 'completed').length;
+  feature.tasks.inProgress = active.filter(t => t.status === 'in_progress').length;
+  feature.tasks.pending = active.filter(t => t.status === 'pending').length;
+}

package/src/lib/tasks/test-runner.js

@@ -1,81 +1,81 @@
-/**
- * Test Runner — Resolves and executes the project's test suite.
- *
- * Resolution order for test command:
- *   1. .morph/config/config.json → project.testCommand
- *   2. package.json present → "npm test"
- *   3. null (no test suite detected)
- */
-
-import { execSync } from 'child_process';
-import { existsSync, readFileSync } from 'fs';
-import { join } from 'path';
-
-/**
- * Resolve the test command for a project.
- *
- * @param {string} [projectPath] - Project root directory (defaults to '.')
- * @returns {string | null} The test command to run, or null if none found
- */
-export function resolveTestCommand(projectPath = '.') {
-  // 1. Read from .morph/config/config.json
-  const configPath = join(projectPath, '.morph', 'config', 'config.json');
-  if (existsSync(configPath)) {
-    try {
-      const config = JSON.parse(readFileSync(configPath, 'utf8'));
-      const cmd = config?.project?.testCommand;
-      if (cmd && typeof cmd === 'string') {
-        return cmd;
-      }
-    } catch {
-      // Malformed config — fall through to detection
-    }
-  }
-
-  // 2. Detect package.json → npm test
-  const packageJsonPath = join(projectPath, 'package.json');
-  if (existsSync(packageJsonPath)) {
-    return 'npm test';
-  }
-
-  // 3. Nothing found
-  return null;
-}
-
-/**
- * Run the project's test suite.
- *
- * @param {string} [projectPath] - Project root directory (defaults to '.')
- * @param {object} [opts] - Optional overrides for testing
- * @param {Function} [opts._execSync] - Override execSync for unit testing
- * @returns {{ passed?: boolean, output?: string, exitCode?: number, skipped?: boolean, reason?: string }}
- */
-export function runTestSuite(projectPath = '.', opts = {}) {
-  const cmd = resolveTestCommand(projectPath);
-
-  if (!cmd) {
-    return { skipped: true, reason: 'No testCommand configured and no known project file detected' };
-  }
-
-  const exec = opts._execSync ?? execSync;
-
-  try {
-    const output = exec(cmd, {
-      cwd: projectPath,
-      timeout: 120_000,
-      encoding: 'utf8',
-      stdio: 'pipe'
-    });
-    return { passed: true, output: output ?? '', exitCode: 0 };
-  } catch (err) {
-    const output = (err.stdout || '') + (err.stderr || '');
-    const BUILD_FAIL = /Build FAILED|error MSB\d+|error CS\d+|Could not find.*\.csproj/i;
-    const buildFailed = BUILD_FAIL.test(output);
-    return {
-      passed: false,
-      buildFailed,
-      output,
-      exitCode: err.status ?? 1
-    };
-  }
-}
+/**
+ * Test Runner — Resolves and executes the project's test suite.
+ *
+ * Resolution order for test command:
+ *   1. .morph/config/config.json → project.testCommand
+ *   2. package.json present → "npm test"
+ *   3. null (no test suite detected)
+ */
+
+import { execSync } from 'child_process';
+import { existsSync, readFileSync } from 'fs';
+import { join } from 'path';
+
+/**
+ * Resolve the test command for a project.
+ *
+ * @param {string} [projectPath] - Project root directory (defaults to '.')
+ * @returns {string | null} The test command to run, or null if none found
+ */
+export function resolveTestCommand(projectPath = '.') {
+  // 1. Read from .morph/config/config.json
+  const configPath = join(projectPath, '.morph', 'config', 'config.json');
+  if (existsSync(configPath)) {
+    try {
+      const config = JSON.parse(readFileSync(configPath, 'utf8'));
+      const cmd = config?.project?.testCommand;
+      if (cmd && typeof cmd === 'string') {
+        return cmd;
+      }
+    } catch {
+      // Malformed config — fall through to detection
+    }
+  }
+
+  // 2. Detect package.json → npm test
+  const packageJsonPath = join(projectPath, 'package.json');
+  if (existsSync(packageJsonPath)) {
+    return 'npm test';
+  }
+
+  // 3. Nothing found
+  return null;
+}
+
+/**
+ * Run the project's test suite.
+ *
+ * @param {string} [projectPath] - Project root directory (defaults to '.')
+ * @param {object} [opts] - Optional overrides for testing
+ * @param {Function} [opts._execSync] - Override execSync for unit testing
+ * @returns {{ passed?: boolean, output?: string, exitCode?: number, skipped?: boolean, reason?: string }}
+ */
+export function runTestSuite(projectPath = '.', opts = {}) {
+  const cmd = resolveTestCommand(projectPath);
+
+  if (!cmd) {
+    return { skipped: true, reason: 'No testCommand configured and no known project file detected' };
+  }
+
+  const exec = opts._execSync ?? execSync;
+
+  try {
+    const output = exec(cmd, {
+      cwd: projectPath,
+      timeout: 120_000,
+      encoding: 'utf8',
+      stdio: 'pipe'
+    });
+    return { passed: true, output: output ?? '', exitCode: 0 };
+  } catch (err) {
+    const output = (err.stdout || '') + (err.stderr || '');
+    const BUILD_FAIL = /Build FAILED|error MSB\d+|error CS\d+|Could not find.*\.csproj/i;
+    const buildFailed = BUILD_FAIL.test(output);
+    return {
+      passed: false,
+      buildFailed,
+      output,
+      exitCode: err.status ?? 1
+    };
+  }
+}