@polymorphism-tech/morph-spec 7.9.3 → 8.0.0

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

@@ -3,12 +3,13 @@
  *
  * Resolution order for test command:
  *   1. .morph/config/config.json → project.testCommand
- *   2. package.json present → "npm test"
- *   3. null (no test suite detected)
+ *   2. .sln or .csproj present at project root → "dotnet test"
+ *   3. package.json present AND has scripts.test → "npm test"
+ *   4. null (no test suite detected)
  */
 
 import { execSync } from 'child_process';
-import { existsSync, readFileSync } from 'fs';
+import { existsSync, readFileSync, readdirSync } from 'fs';
 import { join } from 'path';
 
 /**
@@ -32,13 +33,33 @@ export function resolveTestCommand(projectPath = '.') {
     }
   }
 
-  // 2. Detect package.json → npm test
+  // 2. Detect .NET project (sln/csproj at root) → dotnet test
+  try {
+    const entries = readdirSync(projectPath);
+    const hasDotnetProject = entries.some(
+      (entry) => entry.endsWith('.sln') || entry.endsWith('.csproj')
+    );
+    if (hasDotnetProject) {
+      return 'dotnet test';
+    }
+  } catch {
+    // Unreadable project directory — fall through to next detection
+  }
+
+  // 3. package.json present AND has scripts.test → npm test
   const packageJsonPath = join(projectPath, 'package.json');
   if (existsSync(packageJsonPath)) {
-    return 'npm test';
+    try {
+      const pkg = JSON.parse(readFileSync(packageJsonPath, 'utf8'));
+      if (pkg?.scripts?.test) {
+        return 'npm test';
+      }
+    } catch {
+      // Malformed package.json — fall through to null
+    }
   }
 
-  // 3. Nothing found
+  // 4. Nothing found
   return null;
 }