@polymorphism-tech/morph-spec 8.47.0 → 9.1.0

package/src/lib/tasks/mermaid-renderer.js

@@ -1,132 +0,0 @@
-/**
- * mermaid-renderer.js — pure renderer: tasks.json DAG -> Mermaid flowchart TD.
- *
- * Backs `morph-spec dag <feature> --mermaid` (S8-T1), so the plan's DAG can be
- * visualized at Gate 2 (IndyDevDan rec. 2: draw the workflow in Mermaid before
- * automating it).
- *
- * STRICT Mermaid rules enforced here (see project CLAUDE.md — violating these
- * breaks rendering in Obsidian):
- *   - every node label wrapped in double quotes, no exceptions
- *   - labels never start with a numeric-with-dot prefix ("1. ", "2. ...") —
- *     Obsidian's markdown-list parser chokes on it even inside quotes
- *   - `&` inside labels -> `e`
- *   - line breaks inside labels -> `<br/>`, never a literal `\n`
- *   - double quotes inside titles are stripped so they can't break out of the
- *     surrounding label quotes
- *
- * Pure module: no fs, no Date.now()/Math.random(). Tasks are sorted by
- * numeric id so output is stable regardless of input order.
- */
-
-const MAX_TITLE_LENGTH = 40;
-
-const VALID_STATUS_CLASSES = ['done', 'in_progress', 'blocked', 'skipped', 'pending'];
-
-// Web-safe fill/stroke pairs, one per task status.
-const CLASS_DEFS = [
-  'classDef done fill:#c6f6d5,stroke:#2f855a,color:#22543d',
-  'classDef in_progress fill:#bee3f8,stroke:#2b6cb0,color:#2a4365',
-  'classDef blocked fill:#feb2b2,stroke:#c53030,color:#63171b',
-  'classDef skipped fill:#e2e8f0,stroke:#718096,color:#2d3748',
-  'classDef pending fill:#edf2f7,stroke:#a0aec0,color:#2d3748',
-];
-
-function numericId(id) {
-  const match = /^T(\d+)$/.exec(id ?? '');
-  return match ? parseInt(match[1], 10) : Number.POSITIVE_INFINITY;
-}
-
-/**
- * Sanitize arbitrary text for embedding inside a double-quoted Mermaid node
- * label, per the project's strict Mermaid rules.
- *
- * - double quotes stripped (they would close the label's own quotes)
- * - `&` -> `e`
- * - truncated to MAX_TITLE_LENGTH chars + `…`
- * - line breaks -> `<br/>`
- *
- * Ordering matters: truncation runs on the raw text BEFORE newlines expand to
- * `<br/>`. Newline->`<br/>` is the only length-changing substitution (1 char ->
- * 5); truncating after it could slice a tag mid-way ("...<b…"), so we truncate
- * first — every emitted `<br/>` is then guaranteed whole. The char-for-char
- * substitutions (`"`->`'`, `&`->`e`) don't move the truncation boundary.
- */
-export function sanitizeLabel(text) {
-  let out = String(text ?? '');
-  out = out.replace(/"/g, "'");
-  out = out.replace(/&/g, 'e');
-  if (out.length > MAX_TITLE_LENGTH) {
-    out = `${out.slice(0, MAX_TITLE_LENGTH)}…`;
-  }
-  out = out.replace(/\r\n|\r|\n/g, '<br/>');
-  return out;
-}
-
-/**
- * Same fallback as groupOf() in dispatch-planner.js / groupTasks() in
- * tasks-json-parser.js — tasks without a `group` field are treated as
- * 'backend'.
- */
-function groupOf(task) {
-  return task.group || 'backend';
-}
-
-function statusClass(status) {
-  return VALID_STATUS_CLASSES.includes(status) ? status : 'pending';
-}
-
-/**
- * Render a parsed tasks.json document as a Mermaid `flowchart TD`.
- *
- * @param {{ tasks?: Array<{ id: string, title: string, dependencies?: string[], effort?: string, group?: string, status?: string }> }} tasksData
- * @returns {string}
- */
-export function renderMermaidFlowchart(tasksData) {
-  const tasks = Array.isArray(tasksData?.tasks) ? tasksData.tasks : [];
-
-  if (tasks.length === 0) {
-    return ['flowchart TD', '  empty["sem tasks"]'].join('\n');
-  }
-
-  const sorted = [...tasks].sort((a, b) => numericId(a.id) - numericId(b.id));
-  const knownIds = new Set(sorted.map(t => t.id));
-
-  // Group tasks by `group`, preserving the deterministic sorted-by-id order
-  // in which each group is first encountered.
-  const groups = new Map();
-  for (const task of sorted) {
-    const g = groupOf(task);
-    if (!groups.has(g)) groups.set(g, []);
-    groups.get(g).push(task);
-  }
-
-  const lines = ['flowchart TD'];
-
-  for (const [group, groupTasks] of groups) {
-    lines.push(`  subgraph ${group}["${sanitizeLabel(group)}"]`);
-    for (const task of groupTasks) {
-      const label = `${task.id} · ${sanitizeLabel(task.title)} (${task.effort})`;
-      lines.push(`    ${task.id}["${label}"]`);
-    }
-    lines.push('  end');
-  }
-
-  // Edges — one per dependency. Dependencies pointing at a non-existent id
-  // are silently skipped (never throw on a dangling reference).
-  for (const task of sorted) {
-    const deps = Array.isArray(task.dependencies) ? task.dependencies : [];
-    for (const dep of deps) {
-      if (!knownIds.has(dep)) continue;
-      lines.push(`  ${dep} --> ${task.id}`);
-    }
-  }
-
-  lines.push(...CLASS_DEFS);
-
-  for (const task of sorted) {
-    lines.push(`  class ${task.id} ${statusClass(task.status)}`);
-  }
-
-  return lines.join('\n');
-}

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

@@ -1,30 +0,0 @@
-/**
- * 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 +0,0 @@
-/**
- * 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/tasks-json-parser.js

@@ -1,296 +0,0 @@
-/**
- * tasks-json-parser.js — read, validate, query, mutate tasks.json
- *
- * Replaces the prior Markdown-based tasks.md parser for new features.
- * Legacy features that still use tasks.md are handled by task-parser.js.
- *
- * Uses a minimal hand-rolled JSON Schema validator (no AJV dep) — sufficient
- * for the controlled schema in framework/schemas/tasks.schema.json.
- */
-
-import fs from 'fs-extra';
-import { readFileSync, existsSync } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
-import { UC_ID_RE } from '../specs/acervo-format.js';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const SCHEMA_PATH = join(__dirname, '..', '..', '..', 'framework', 'schemas', 'tasks.schema.json');
-
-let cachedSchema = null;
-export function getSchema() {
-  if (cachedSchema) return cachedSchema;
-  cachedSchema = JSON.parse(readFileSync(SCHEMA_PATH, 'utf8'));
-  return cachedSchema;
-}
-
-const VALID_STATUS = ['pending', 'in_progress', 'done', 'skipped', 'blocked'];
-const VALID_EFFORT = ['S', 'M', 'L', 'XL'];
-const VALID_GROUP = ['backend', 'frontend', 'tests', 'infra', 'docs'];
-/** Veredito de reuso por conceito — ver framework/schemas/tasks.schema.json. */
-const VALID_REUSE_VERDICT = ['REUSAR', 'ESTENDER', 'CRIAR'];
-
-/**
- * Validate a tasks.json structure.
- * @returns {{ valid: boolean, errors: string[] }}
- */
-export function validateTasksJson(data) {
-  const errors = [];
-  if (!data || typeof data !== 'object') {
-    errors.push('tasks.json must be an object');
-    return { valid: false, errors };
-  }
-  if (data.version !== '1.0') errors.push('version must be "1.0"');
-  if (typeof data.feature !== 'string' || !data.feature) {
-    errors.push('feature must be a non-empty string');
-  }
-  if (!Array.isArray(data.tasks)) {
-    errors.push('tasks must be an array');
-    return { valid: false, errors };
-  }
-
-  const ids = new Set();
-  for (const [i, task] of data.tasks.entries()) {
-    const ctx = `tasks[${i}]`;
-    if (!task.id || !/^T\d+$/.test(task.id)) {
-      errors.push(`${ctx}.id must match /^T\\d+$/`);
-    } else if (ids.has(task.id)) {
-      errors.push(`${ctx}.id duplicate: ${task.id}`);
-    }
-    ids.add(task.id);
-
-    if (typeof task.title !== 'string' || !task.title) errors.push(`${ctx}.title required`);
-    if (typeof task.description !== 'string' || !task.description) errors.push(`${ctx}.description required`);
-    if (!Array.isArray(task.dependencies)) {
-      errors.push(`${ctx}.dependencies must be array`);
-    } else {
-      for (const dep of task.dependencies) {
-        if (typeof dep !== 'string' || !/^T\d+$/.test(dep)) {
-          errors.push(`${ctx}.dependencies[${dep}] must match /^T\\d+$/`);
-        }
-      }
-    }
-    if (!VALID_EFFORT.includes(task.effort)) {
-      errors.push(`${ctx}.effort must be one of ${VALID_EFFORT.join(', ')}`);
-    }
-    if (typeof task.doneCriteria !== 'string' || !task.doneCriteria) {
-      errors.push(`${ctx}.doneCriteria required`);
-    }
-    if (!VALID_STATUS.includes(task.status)) {
-      errors.push(`${ctx}.status must be one of ${VALID_STATUS.join(', ')}`);
-    }
-    if (task.group !== undefined && !VALID_GROUP.includes(task.group)) {
-      errors.push(`${ctx}.group must be one of ${VALID_GROUP.join(', ')}`);
-    }
-    // Optional: a task with no observable behaviour references no use case.
-    // When present it must be the bare ID — `morph-spec dag` is the gate that
-    // catches "UC-07-listar.md" or "UC7" before the plan reaches Gate 2.
-    if (task.usecase !== undefined && (typeof task.usecase !== 'string' || !UC_ID_RE.test(task.usecase))) {
-      errors.push(`${ctx}.usecase must match /^UC-\\d+$/ (the ID alone, e.g. "UC-07")`);
-    }
-
-    // Optional: morph-plan may stamp a persona + standards per task (R2 token
-    // cut — `morph-spec standards --find` replaces reading the whole
-    // STANDARDS.json). Both are part of the plan contract and frozen after
-    // Gate 2 by protect-spec-files (any key outside MUTABLE_SPEC_FIELDS is
-    // blocked by construction — see that hook's top comment). Type-check
-    // only here: no cross-check against agents.json, since stack-filtered
-    // installs carry fewer personas than the full set and would false-fail.
-    if (task.agent !== undefined) {
-      if (typeof task.agent !== 'string' || !task.agent.trim()) {
-        errors.push(`${ctx}.agent must be a non-empty string (persona id, e.g. "dotnet-senior")`);
-      } else if (task.agent.startsWith('morph-')) {
-        errors.push(`${ctx}.agent must be the bare persona id — no "morph-" prefix (got "${task.agent}")`);
-      }
-    }
-    // Type-check e enum apenas: quem cobra o veredito contra o código é o
-    // reuse-validator, no `morph-spec verify`.
-    if (task.reuse !== undefined) {
-      if (!Array.isArray(task.reuse)) {
-        errors.push(`${ctx}.reuse must be an array of { concept, verdict, target?, why? }`);
-      } else {
-        task.reuse.forEach((entry, j) => {
-          if (!entry || typeof entry !== 'object' || Array.isArray(entry)) {
-            errors.push(`${ctx}.reuse[${j}] must be an object { concept, verdict, target?, why? }`);
-            return;
-          }
-          if (typeof entry.concept !== 'string' || !entry.concept.trim()) {
-            errors.push(`${ctx}.reuse[${j}].concept must be a non-empty string`);
-          }
-          if (!VALID_REUSE_VERDICT.includes(entry.verdict)) {
-            errors.push(`${ctx}.reuse[${j}].verdict must be one of ${VALID_REUSE_VERDICT.join(', ')}`);
-          }
-          if (entry.target !== undefined && typeof entry.target !== 'string') {
-            errors.push(`${ctx}.reuse[${j}].target must be a string path`);
-          }
-          if (entry.why !== undefined && typeof entry.why !== 'string') {
-            errors.push(`${ctx}.reuse[${j}].why must be a string`);
-          }
-        });
-      }
-    }
-    if (task.standards !== undefined) {
-      if (!Array.isArray(task.standards)) {
-        errors.push(`${ctx}.standards must be an array of paths`);
-      } else {
-        task.standards.forEach((s, i) => {
-          if (typeof s !== 'string' || !s.trim()) {
-            errors.push(`${ctx}.standards[${i}] must be a non-empty string path`);
-          }
-        });
-      }
-    }
-  }
-
-  return { valid: errors.length === 0, errors };
-}
-
-/**
- * Read and parse tasks.json from disk. Throws on validation errors.
- * @param {string} filePath - Absolute path to tasks.json
- * @returns {Promise<object>}
- */
-export async function parseTasksJson(filePath) {
-  const data = await fs.readJson(filePath);
-  const { valid, errors } = validateTasksJson(data);
-  if (!valid) {
-    throw new Error(`Invalid tasks.json:\n  ${errors.join('\n  ')}`);
-  }
-  return data;
-}
-
-/**
- * Lenient variant for read-only analysis commands (e.g. `morph-spec dag`):
- * a missing `version` field is defaulted to "1.0" instead of failing
- * validation — legacy tasks.json predating the field is still analyzable.
- * All other schema errors still throw exactly like `parseTasksJson`.
- *
- * @param {string} filePath - Absolute path to tasks.json
- * @returns {Promise<{ data: object, versionDefaulted: boolean }>}
- */
-export async function parseTasksJsonLenient(filePath) {
-  const data = await fs.readJson(filePath);
-  const versionDefaulted = data.version === undefined;
-  if (versionDefaulted) data.version = '1.0';
-
-  const { valid, errors } = validateTasksJson(data);
-  if (!valid) {
-    throw new Error(`Invalid tasks.json:\n  ${errors.join('\n  ')}`);
-  }
-  return { data, versionDefaulted };
-}
-
-/**
- * Synchronous variant — used by hooks.
- */
-export function parseTasksJsonSync(filePath) {
-  if (!existsSync(filePath)) {
-    throw new Error(`tasks.json not found: ${filePath}`);
-  }
-  const data = JSON.parse(readFileSync(filePath, 'utf8'));
-  const { valid, errors } = validateTasksJson(data);
-  if (!valid) {
-    throw new Error(`Invalid tasks.json:\n  ${errors.join('\n  ')}`);
-  }
-  return data;
-}
-
-/**
- * Teto de `outputs` por task: a task inteira tem de caber num dispatch. Medido
- * em campo (chat-demo-simulacao): tasks de 12 a 30 outputs levaram 6 de 10
- * sub-agents ao teto de turnos, e cada retomada custou um contexto frio.
- */
-export const MAX_TASK_OUTPUTS = 8;
-
-/**
- * A régua única de "task grande demais", lida pelo `dag` e pelo `gate-check`.
- * Fora de `validateTasksJson` de propósito: `outputs` segue mutável depois do
- * Gate 2, e o teto num write de status travaria feature em voo.
- *
- * @returns {{ id: string, outputs: number }[]}
- */
-export function oversizedTasks(data) {
-  const tasks = Array.isArray(data?.tasks) ? data.tasks : [];
-  return tasks
-    .filter(t => Array.isArray(t?.outputs) && t.outputs.length > MAX_TASK_OUTPUTS)
-    .map(t => ({ id: t.id, outputs: t.outputs.length }));
-}
-
-/**
- * @returns {object[]} tasks with status === 'pending'
- */
-export function getPendingTasks(data) {
-  return data.tasks.filter(t => t.status === 'pending');
-}
-
-/**
- * Tasks ready to run: pending AND all dependencies are 'done'.
- */
-export function getReadyTasks(data) {
-  const doneIds = new Set(data.tasks.filter(t => t.status === 'done').map(t => t.id));
-  return data.tasks.filter(t =>
-    t.status === 'pending' && t.dependencies.every(dep => doneIds.has(dep))
-  );
-}
-
-/**
- * Mutate `data` to set task status. Returns `data` for chaining.
- */
-export function updateTaskStatus(data, taskId, newStatus) {
-  if (!VALID_STATUS.includes(newStatus)) {
-    throw new Error(`Invalid status: ${newStatus}`);
-  }
-  const task = data.tasks.find(t => t.id === taskId);
-  if (!task) throw new Error(`Task not found: ${taskId}`);
-  task.status = newStatus;
-  return data;
-}
-
-/**
- * Write tasks.json to disk. Validates before writing.
- */
-export async function writeTasksJson(filePath, data) {
-  const { valid, errors } = validateTasksJson(data);
-  if (!valid) {
-    throw new Error(`Cannot write invalid tasks.json:\n  ${errors.join('\n  ')}`);
-  }
-  await fs.writeJson(filePath, data, { spaces: 2 });
-}
-
-/**
- * Group tasks by `group` field, falling back to 'backend' when absent.
- * Returns `{ [group]: taskId[] }` shape matching the legacy parseAndGroupTasks output.
- */
-export function groupTasks(data) {
-  const groups = {};
-  for (const task of data.tasks) {
-    const group = task.group || 'backend';
-    if (!groups[group]) groups[group] = [];
-    groups[group].push(task.id);
-  }
-  return groups;
-}
-
-/**
- * Convert tasks.json to the legacy taskList shape used in state.json.
- * Lets the rest of the codebase keep using feature.taskList while migration
- * is in progress.
- *
- * BOUNDARY — this is the ONE place `done` becomes `completed`, and it is a
- * one-way translation OUT of the tasks.json vocabulary. In tasks.json the
- * terminal status is `done` (tasks.schema.json's enum has no `completed`);
- * `completed` belongs to the legacy taskList and to Claude Code's native
- * session tasks. Leaking `completed` back into a tasks.json write is rejected
- * by `protect-spec-files` — see framework/rules/morph-task-tracking.md, which
- * spells the split out for the author.
- */
-export function toTaskList(data) {
-  return data.tasks.map(t => ({
-    id: t.id,
-    title: t.title,
-    status: t.status === 'done' ? 'completed' : t.status,
-    dependencies: [...t.dependencies],
-    files: t.outputs || [],
-    checkpoint: null,
-  }));
-}