@polymorphism-tech/morph-spec 8.47.0 → 9.0.0

package/framework/templates/feature/decisions.md

@@ -1,124 +0,0 @@
-# Architecture Decision Records - {{titleCase FEATURE_NAME}}
-
-## ADR-001: {Decision Title}
-
-**Date:** {{DATE}}
-**Status:** Proposed | Accepted | Deprecated | Superseded
-**Deciders:** MORPH Agents, Developer
-**Alcance:** feature
-
-### Context
-
-{Describe the context and problem that led to this decision}
-
-### Decision
-
-{Describe the decision made}
-
-### Consequences
-
-**Pros:**
-- {Positive consequence 1}
-- {Positive consequence 2}
-
-**Cons:**
-- {Negative consequence 1}
-
-**Risks:**
-- {Identified risk}
-
-### Alternatives Considered
-
-#### Option A: {Name}
-- Pros: {advantages}
-- Cons: {disadvantages}
-- **Rejected because:** {reason}
-
-#### Option B: {Name}
-- Pros: {advantages}
-- Cons: {disadvantages}
-- **Rejected because:** {reason}
-
----
-
-{{#if (eq STACK "nextjs-neon")}}
-## ADR-002: Database Access Strategy
-
-**Date:** {{DATE}}
-**Status:** Accepted
-
-### Context
-
-Need to decide how the .NET API accesses Neon PostgreSQL.
-
-### Decision
-
-Use Dapper with Npgsql for direct SQL queries. No EF Core.
-
-### Consequences
-
-**Pros:**
-- Full control over SQL queries
-- Better performance (no ORM overhead)
-- Direct access to PostgreSQL features (jsonb, pgvector, functions)
-- RLS policies apply naturally through connection
-
-**Cons:**
-- No automatic migrations (use Neon CLI / dotnet ef instead)
-- Manual SQL query writing
-- No change tracking
-
----
-
-## ADR-003: Auth Strategy
-
-**Date:** {{DATE}}
-**Status:** Accepted
-
-### Context
-
-Need authentication for both Next.js frontend and .NET API.
-
-### Decision
-
-Use Neon Auth (Better Auth) for authentication. Auth data lives in the Neon database. JWT validated by .NET API middleware.
-
-### Consequences
-
-**Pros:**
-- Auth data in same database — no external provider, no sync delays
-- RLS policies work with Neon Auth JWT via auth.user_id() / pg_session_jwt
-- Built-in social login, email/password via @neondatabase/auth SDK
-- Branch isolation — each Neon branch gets isolated auth data
-- No webhooks needed — query neon_auth schema directly
-
-**Cons:**
-- Neon Auth is currently in Beta
-- JWT must be forwarded to .NET API for backend auth
-{{/if}}
-
----
-
-## Template for New ADRs
-
-```markdown
-## ADR-XXX: {Title}
-
-**Date:**
-**Status:** Proposed | Accepted | Deprecated | Superseded
-**Alcance:** feature | projeto — `projeto` when the rule holds beyond this feature: `morph-spec finish` copies it to `.morph/specs/adrs/`, the folder brainstorming and plan read instead of archived decisions.md
-
-### Context
-{Why is this decision needed?}
-
-### Decision
-{What was decided?}
-
-### Consequences
-**Pros:** {list}
-**Cons:** {list}
-```
-
----
-
-*Generated by MORPH Framework*

package/framework/templates/feature/recap.md

@@ -1,145 +0,0 @@
-# Feature Recap: {{titleCase FEATURE_NAME}}
-
-## Summary
-
-| Field | Value |
-|-------|-------|
-| **Feature ID** | {{FEATURE_NAME}} |
-| **Completed** | {{DATE}} |
-| **Total Tasks** | {X} |
-| **Time Spent** | {X}h |
-| **Agents Used** | {list} |
-{{#if STACK}}| **Stack** | {{STACK}} |{{/if}}
-
----
-
-## Tasks Completed
-
-| ID | Title | Category | Duration |
-|----|-------|----------|----------|
-| T001 | {title} | {{#if (eq STACK "nextjs-neon")}}Database{{else}}{category}{{/if}} | {X}min |
-| T002 | {title} | {{#if (eq STACK "nextjs-neon")}}Backend{{else}}{category}{{/if}} | {X}min |
-| T003 | {title} | {{#if (eq STACK "nextjs-neon")}}Frontend{{else}}{category}{{/if}} | {X}min |
-
----
-
-{{#if (eq STACK "nextjs-neon")}}
-## Validation Results
-
-| Validator | Status | Details |
-|-----------|--------|---------|
-| contract-compliance | Pass/Fail | {details} |
-| architecture | Pass/Fail | {details} |
-| design-system | Pass/Fail | {details} |
-| coding-standards | Pass/Fail | {details} |
-
----
-
-{{/if}}
-## What Was Delivered
-
-### Functionality
-- {Functionality 1 delivered}
-- {Functionality 2 delivered}
-- {Functionality 3 delivered}
-
-{{#if (eq STACK "nextjs-neon")}}
-### Database (Neon PostgreSQL)
-- Tables: {list}
-- RLS Policies: {count} policies
-- Migrations: {count} files
-
-### Backend API (.NET)
-- Endpoints: {count} endpoints
-- Services: {list}
-
-### Frontend (Next.js)
-- Pages: {list}
-- Components: {list}
-{{/if}}
-
----
-
-## Files Created
-
-```
-{{#if (eq STACK "nextjs-neon")}}
-migrations/{files}
-backend/src/{files}
-frontend/src/{files}
-{{else}}
-{list of created files}
-{{/if}}
-```
-
-## Files Modified
-
-```
-{list of modified files}
-```
-
----
-
-## Architecture Decisions
-
-| Decision | Rationale |
-|----------|-----------|
-| {Decision 1} | {Reason} |
-| {Decision 2} | {Reason} |
-
-See full ADRs in `decisions.md`
-
----
-
-## Test Coverage
-
-| Type | Files | Coverage |
-|------|-------|----------|
-| {{#if (eq STACK "nextjs-neon")}}Backend Unit{{else}}Unit Tests{{/if}} | {X} | {X}% |
-| {{#if (eq STACK "nextjs-neon")}}Frontend Component{{else}}Integration Tests{{/if}} | {X} | {X}% |
-
----
-
-## Lessons Learned
-
-### What Went Well
-- {Positive point 1}
-- {Positive point 2}
-
-### What Could Be Improved
-- {Improvement point 1}
-- {Improvement point 2}
-
-### Recommendations for Future
-- {Recommendation 1}
-- {Recommendation 2}
-
----
-
-## Metrics
-
-| Metric | Value |
-|--------|-------|
-| Tasks Completed | {X}/{X} |
-| Checkpoints Passed | {X} |
-{{#if (eq STACK "nextjs-neon")}}| Validation Passes | {X}/{X} |
-{{else}}| Code Reviews | {X} |
-{{/if}}| Bugs Found | {X} |
-| Bugs Fixed | {X} |
-
----
-
-## Related Resources
-
-- **Proposal:** `proposal.md`
-- **Spec:** `spec.md`
-{{#if (eq STACK "nextjs-neon")}}- **Contracts (.NET):** `contracts.cs`
-- **Contracts (TS):** `contracts.ts`
-{{else}}- **Contracts:** `contracts.cs`
-{{/if}}- **Tasks:** `tasks.json`
-- **Decisions:** `decisions.md`
-- **PR:** {link}
-
----
-
-*Generated by MORPH Framework on {{DATE}}*

package/framework/templates/tasks.template.json

@@ -1,17 +0,0 @@
-{
-  "version": "1.0",
-  "feature": "{{featureName}}",
-  "tasks": [
-    {
-      "id": "T1",
-      "title": "Example task title",
-      "description": "What this task accomplishes and why",
-      "dependencies": [],
-      "effort": "M",
-      "doneCriteria": "Verifiable acceptance condition stated in one sentence",
-      "status": "pending",
-      "outputs": [],
-      "group": "backend"
-    }
-  ]
-}

package/src/commands/state/usecase.js

@@ -1,143 +0,0 @@
-/**
- * `morph-spec usecase reserve [--count <n>] [--feature <name>] [--json]` —
- * atomically mint the next N `UC-{nn}` numbers, shared across every worktree
- * of this repo. See `usecase-registry.js` for why this exists (worktrees only
- * ever see their own tree, so an independent max+1 scan collides).
- *
- * `--feature` is more than an audit label: `usecase-promoter.js`'s collision
- * guard uses the reservation record to tell a legitimate cross-feature
- * correction (never reserved — reusing a published number by design) from an
- * accidental collision (both features reserved the same id). Passing it is
- * how `morph-plan` gets that protection; omitting it degrades to "reserved,
- * no owner recorded" — still collision-free, just unable to name the
- * conflicting feature if one ever surfaces.
- *
- * `morph-spec usecase check <feature>` — see `usecaseCheckCommand` below.
- */
-
-import chalk from 'chalk';
-import { existsSync, readdirSync } from 'fs';
-import { join } from 'path';
-import { resolveGitCommonDir } from '../../../framework/hooks/shared/git-paths.js';
-import { reserveUseCases } from '../../lib/specs/usecase-registry.js';
-import { checkUseCases } from '../../lib/specs/usecase-check.js';
-import { checkLifecycle } from '../../lib/specs/acervo-lifecycle.js';
-import { parseTasksJsonLenient } from '../../lib/tasks/tasks-json-parser.js';
-import { isValidFeatureName } from '../project/worktree.js';
-import { printLifecycleWarnings } from '../tasks/dag.js';
-
-/**
- * Best-effort feature-name inference when `--feature` is omitted: only when
- * exactly one active feature exists is the guess unambiguous. Anything else
- * (zero, or more than one) is left null rather than guessed wrong — a wrong
- * guess would poison the collision guard's provenance record.
- */
-function inferActiveFeature(cwd) {
-  const featuresDir = join(cwd, '.morph', 'features');
-  try {
-    if (!existsSync(featuresDir)) return null;
-    const entries = readdirSync(featuresDir, { withFileTypes: true }).filter((e) => e.isDirectory());
-    return entries.length === 1 ? entries[0].name : null;
-  } catch {
-    return null;
-  }
-}
-
-/**
- * @param {{ count?: string, feature?: string, json?: boolean, cwd?: string }} [options]
- */
-export function usecaseReserveCommand(options = {}) {
-  const cwd = options.cwd || process.cwd();
-  const count = Number.parseInt(options.count, 10);
-  const n = Number.isInteger(count) && count > 0 ? count : 1;
-  const feature = options.feature || inferActiveFeature(cwd) || null;
-
-  let gitCommonDir = null;
-  try { gitCommonDir = resolveGitCommonDir(cwd); } catch { gitCommonDir = null; }
-
-  let result;
-  try {
-    result = reserveUseCases(cwd, { feature, count: n, gitCommonDir });
-  } catch (error) {
-    if (options.json) {
-      console.log(JSON.stringify({ reserved: false, error: error.message }));
-      process.exit(1);
-    }
-    console.error(chalk.red(`\n❌ ${error.message}\n`));
-    process.exit(1);
-  }
-
-  const warning = gitCommonDir
-    ? null
-    : 'projeto fora de um repositório git — numeração protegida só localmente; paralelismo entre worktrees NÃO está garantido.';
-
-  if (options.json) {
-    console.log(JSON.stringify({ reserved: true, ...result, feature, warning }));
-    process.exit(0);
-  }
-
-  console.log(chalk.green(`\n✅ Reservado: ${result.numbers.join(' ')}`));
-  if (feature) console.log(chalk.gray(`   feature: ${feature}`));
-  if (warning) console.log(chalk.yellow(`⚠️  ${warning}\n`));
-  else console.log('');
-  process.exit(0);
-}
-
-/**
- * `morph-spec usecase check <feature> [--json]` — the use-case half of `dag`,
- * runnable before `tasks.json` exists: the feature's use cases and the
- * revocations its `spec.md` declares, against the acervo's lifecycle. With a
- * `tasks.json`, also the task → use case plumbing `dag` checks.
- *
- * Exit 1 only for what `dag` also refuses (a task pointing at a use case that
- * does not exist, a tasks.json that does not validate); lifecycle findings are
- * warnings.
- *
- * @param {string} feature
- * @param {{ json?: boolean, cwd?: string }} [options]
- */
-export async function usecaseCheckCommand(feature, options = {}) {
-  const cwd = options.cwd || process.cwd();
-  const fail = (error) => {
-    if (options.json) console.log(JSON.stringify({ feature, error }));
-    else console.error(chalk.red(`\n❌ ${error}\n`));
-    process.exit(1);
-  };
-
-  if (!isValidFeatureName(feature)) fail(`nome de feature inválido: "${feature}"`);
-  const featureDir = join(cwd, '.morph', 'features', feature);
-  if (!existsSync(featureDir)) fail(`feature não encontrada: .morph/features/${feature}`);
-
-  const tasksPath = join(featureDir, '2-plan', 'tasks.json');
-  let tasksData = null;
-  if (existsSync(tasksPath)) {
-    try {
-      ({ data: tasksData } = await parseTasksJsonLenient(tasksPath));
-    } catch (error) {
-      fail(`tasks.json inválido: ${error.message}`);
-    }
-  }
-
-  const useCases = tasksData ? checkUseCases(tasksData, join(featureDir, '2-plan', 'usecases')) : null;
-  const lifecycle = checkLifecycle({ cwd, featureDir, tasksData });
-  const broken = useCases?.errors.length > 0;
-
-  if (options.json) {
-    console.log(JSON.stringify({ feature, useCases, lifecycle }, null, 2));
-    process.exit(broken ? 1 : 0);
-  }
-
-  if (broken) {
-    console.error(chalk.red(`\n❌ Use cases — ${useCases.errors.length} erro(s)\n`));
-    for (const e of useCases.errors) console.error(chalk.red(`  ${e}`));
-  }
-  if (useCases?.warnings.length) {
-    console.error(chalk.yellow(`\n⚠ Use cases — ${useCases.warnings.length} aviso(s)`));
-    for (const w of useCases.warnings) console.error(chalk.yellow(`  ${w}`));
-  }
-  printLifecycleWarnings(lifecycle);
-  if (!broken && !useCases?.warnings.length && !lifecycle.warnings.length) {
-    console.log(chalk.green(`\n✅ ${feature}: use cases e acervo sem achados\n`));
-  }
-  process.exit(broken ? 1 : 0);
-}

package/src/commands/tasks/dag.js

@@ -1,181 +0,0 @@
-import chalk from 'chalk';
-import { existsSync, readFileSync } from 'fs';
-import { join } from 'path';
-import { parseTasksJsonLenient, oversizedTasks, MAX_TASK_OUTPUTS } from '../../lib/tasks/tasks-json-parser.js';
-import { planDispatch } from '../../lib/tasks/dispatch-planner.js';
-import { renderMermaidFlowchart } from '../../lib/tasks/mermaid-renderer.js';
-import { checkUseCases } from '../../lib/specs/usecase-check.js';
-import { checkArtifactCriteria } from '../../lib/specs/done-criteria-check.js';
-import { checkLifecycle } from '../../lib/specs/acervo-lifecycle.js';
-
-/** Shared by `dag` and `usecase check` — the two surfaces of the same finding. */
-export function printLifecycleWarnings(lifecycle) {
-  if (!lifecycle?.warnings?.length) return;
-  const n = lifecycle.warnings.length;
-  console.error(chalk.yellow(`\n⚠ Ciclo de vida do acervo — ${n} ${n === 1 ? 'aviso' : 'avisos'}`));
-  for (const w of lifecycle.warnings) console.error(chalk.yellow(`  ${w}`));
-}
-
-function planGateApproved(featureDir) {
-  try {
-    const feature = JSON.parse(readFileSync(join(featureDir, 'feature.json'), 'utf8'));
-    return feature?.approvalGates?.plan?.approved === true;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * `morph-spec dag <feature>` — analyze the tasks.json DAG and print the
- * sequential vs parallel dispatch decision, weighted by effort.
- *
- * This is the command morph-apply §2 runs instead of re-deriving the rules
- * (and instead of the old inline JS snippets the LLM could not execute).
- *
- * `--mermaid` emits the DAG as a Mermaid `flowchart TD` instead (for
- * visualizing the plan at Gate 2), bypassing the dispatch analysis entirely.
- */
-export async function dagCommand(featureName, options = {}) {
-  try {
-    const featureDir = join(process.cwd(), '.morph', 'features', featureName);
-    const planDir = join(featureDir, '2-plan');
-    const tasksPath = join(planDir, 'tasks.json');
-    if (!existsSync(tasksPath)) {
-      console.error(chalk.red(`\n❌ tasks.json not found for feature "${featureName}"`));
-      console.error(chalk.gray(`Expected at: .morph/features/${featureName}/2-plan/tasks.json\n`));
-      process.exit(1);
-    }
-
-    // parseTasksJsonLenient validates the schema and throws with the error
-    // list, but defaults a missing `version` (legacy plans predating the
-    // field) instead of failing — the DAG analysis doesn't depend on it.
-    const { data, versionDefaulted } = await parseTasksJsonLenient(tasksPath);
-    if (versionDefaulted) {
-      console.error(chalk.yellow(
-        `⚠ tasks.json has no 'version' field — defaulting to "1.0" for this analysis.\n` +
-        `  Consider adding it back explicitly (safe: protect-spec-files allows backfilling it).`
-      ));
-    }
-
-    // --mermaid short-circuits before the dispatch analysis: visualizing the
-    // plan's DAG doesn't need a sequential-vs-parallel decision.
-    if (options.mermaid) {
-      console.log(renderMermaidFlowchart(data));
-      return;
-    }
-
-    // Use-case cross-check. Runs AFTER the --mermaid short-circuit on purpose:
-    // rendering the DAG is pure visualization and must never be blocked by the
-    // state of the spec. A broken `usecase` reference IS fatal here — the file
-    // either exists or it does not, so there is no false positive to weigh
-    // against a plan that points at a use case nobody can read.
-    const ucCheck = checkUseCases(data, join(planDir, 'usecases'));
-
-    // Artifact-postcondition warning (issue #87). Lexical, not wiring — it
-    // runs over ALL tasks even when the feature has no use cases at all
-    // (checkUseCases short-circuits above in that case), because a task's
-    // own doneCriteria can describe an artifact with no use case in sight.
-    const artifactCriteria = checkArtifactCriteria(data, join(planDir, 'usecases'));
-
-    // Acervo lifecycle: a task citing a superseded use case, and the spec.md
-    // revocation rows the finish would fail to apply. Warnings only.
-    const lifecycle = checkLifecycle({ cwd: process.cwd(), featureDir, tasksData: data });
-
-    // Task acima do teto não cabe num dispatch. Antes do Gate 2 é erro (o plano
-    // ainda fatia); depois é aviso, para não travar feature em voo.
-    const oversized = oversizedTasks(data);
-    const sizeBlocks = oversized.length > 0 && !planGateApproved(featureDir);
-    const taskSize = { max: MAX_TASK_OUTPUTS, oversized, blocking: sizeBlocks };
-
-    const plan = planDispatch(data);
-
-    if (options.json) {
-      console.log(JSON.stringify({ ...plan, useCases: ucCheck, artifactCriteria, lifecycle, taskSize }, null, 2));
-      if (ucCheck.errors.length || sizeBlocks) process.exit(1);
-      return;
-    }
-
-    const plural = (n, one, many) => `${n} ${n === 1 ? one : many}`;
-
-    if (ucCheck.errors.length) {
-      console.error(chalk.red(`\n❌ Use cases — ${plural(ucCheck.errors.length, 'erro', 'erros')}\n`));
-      for (const e of ucCheck.errors) console.error(chalk.red(`  ${e}`));
-      console.error(chalk.gray('\n  Corrija a referência no tasks.json ou restaure o use case.\n'));
-      process.exit(1);
-    }
-    if (oversized.length) {
-      const tone = sizeBlocks ? chalk.red : chalk.yellow;
-      const head = sizeBlocks ? '❌' : '⚠';
-      console.error(tone(`\n${head} Tamanho de task — ${plural(oversized.length, 'task', 'tasks')} acima de ${MAX_TASK_OUTPUTS} outputs`));
-      for (const t of oversized) console.error(tone(`  ${t.id}: ${t.outputs} outputs`));
-      console.error(chalk.gray('  Fatie por pós-condição: uma task cabe num dispatch (morph-plan, "Tamanho da task").\n'));
-      if (sizeBlocks) process.exit(1);
-    }
-    if (ucCheck.warnings.length) {
-      console.error(chalk.yellow(`\n⚠ Use cases — ${plural(ucCheck.warnings.length, 'aviso', 'avisos')}`));
-      for (const w of ucCheck.warnings) console.error(chalk.yellow(`  ${w}`));
-    }
-    if (artifactCriteria.warnings.length) {
-      console.error(chalk.yellow(`\n⚠ Critérios de artefato — ${plural(artifactCriteria.warnings.length, 'aviso', 'avisos')}`));
-      for (const w of artifactCriteria.warnings) console.error(chalk.yellow(`  ${w}`));
-      console.error(chalk.gray('  Teste que lê texto de arquivo não prova comportamento (morph-plan §2, "Por que artefato").'));
-    }
-    printLifecycleWarnings(lifecycle);
-
-    console.log(chalk.bold(`\n📊 DAG — ${featureName}`));
-    console.log('━'.repeat(60));
-
-    const { stats } = plan;
-    console.log(`\nTasks: ${stats.total} total, ${stats.pending} pendente(s)`);
-    const groupLine = Object.keys(stats.groups)
-      .map(g => `${g}:${stats.groups[g]} (peso ${stats.weights[g]})`)
-      .join(' | ');
-    if (groupLine) console.log(chalk.gray(`Grupos pendentes: ${groupLine}`));
-
-    const modeLabel = {
-      'sequential': 'SEQUENCIAL (auto-advance)',
-      'parallel-flows': 'PARALELO por fluxo de domínio',
-      'parallel-roots': 'PARALELO por raízes independentes',
-    }[plan.mode];
-    console.log(chalk.cyan(`\nDecisão: ${modeLabel}`));
-    console.log(`  → ${plan.reason}`);
-
-    for (const flow of plan.flows) {
-      console.log(`  ${chalk.green('▶')} Fluxo ${chalk.bold(flow.group)} (peso ${flow.weight}): ${flow.taskIds.join(', ')}`);
-    }
-    if (plan.batch.length > 0) {
-      console.log(`  ${chalk.green('▶')} Batch paralelo: ${plan.batch.join(', ')}`);
-    }
-    if (plan.inline.length > 0) {
-      console.log(`  ${chalk.yellow('○')} Inline (sem dispatch próprio): ${plan.inline.join(', ')}`);
-    }
-    if (plan.sequentialRemainder.length > 0) {
-      console.log(`  ${chalk.gray('→')} Sequencial: ${plan.sequentialRemainder.join(', ')}`);
-    }
-
-    // Isolamento — only meaningful when there IS a parallel batch. In a shared
-    // working tree two agents writing the same path corrupt each other; the
-    // dag recommends `isolation: "worktree"` so each runs in its own worktree
-    // (morph-apply §3b-w).
-    const iso = plan.isolation;
-    if (iso && (iso.recommended === 'worktree' || iso.unknownFootprint.length > 0)) {
-      console.log(chalk.bold('\nIsolamento:'));
-      if (iso.recommended === 'worktree') {
-        console.log(`  ${chalk.yellow('⚠')} recomendado: ${chalk.bold('worktree')} — ${iso.reason}`);
-        for (const c of iso.collisions) {
-          console.log(chalk.gray(`      colisão em "${c.path}": ${c.units.join(' ‖ ')} (${c.tasks.join(', ')})`));
-        }
-      } else {
-        console.log(`  ${chalk.gray('○')} ${iso.reason}`);
-      }
-      if (iso.unknownFootprint.length > 0) {
-        console.log(chalk.gray(`      footprint desconhecido (sem outputs): ${iso.unknownFootprint.join(', ')}`));
-      }
-    }
-
-    console.log('\n' + '━'.repeat(60) + '\n');
-  } catch (error) {
-    console.error(chalk.red(`\n❌ Error analyzing DAG: ${error.message}\n`));
-    process.exit(1);
-  }
-}

package/src/lib/agents/output-collector.js

@@ -1,76 +0,0 @@
-/**
- * output-collector — Collect phase output files for multi-source agent detection
- *
- * Reads all existing phase outputs for a feature and returns them as
- * weighted TextSource[] for use in detectAgents().
- */
-
-import { readFileSync, existsSync } from 'fs';
-import { OUTPUT_SCHEMA, getAbsoluteOutputPath } from '../../core/paths/output-schema.js';
-
-/**
- * Default source weights per output type.
- * Keys match OUTPUT_SCHEMA keys (camelCase).
- */
-export const SOURCE_WEIGHTS = {
-  proposal:       1.0,
-  stackScan:      0.8,
-  spec:           1.5,
-  tasks:          1.3,
-  decisions:      1.0,
-  uiDesignSystem: 1.0,
-  uiMockups:      1.0,
-  uiComponents:   1.0,
-  uiFlows:        1.0,
-  recap:          1.0,
-  reviewReport:   1.0,
-};
-
-/** Phase order for sorting collected outputs (v5 NLH model) */
-const PHASE_ORDER = ['proposal', 'uiux', 'plan', 'implement', 'review'];
-
-/**
- * Collect all existing phase output files for a feature.
- *
- * @param {string} projectRoot - Absolute path to project root
- * @param {string} featureName - Feature name
- * @returns {Array<{source: string, content: string, weight: number}>}
- */
-export function collectPhaseOutputs(projectRoot, featureName) {
-  const results = [];
-
-  for (const [key, entry] of Object.entries(OUTPUT_SCHEMA)) {
-    // Multi-file types resolve to a DIRECTORY — there is no single document to
-    // weigh as one source. Skipped explicitly rather than left to fail as an
-    // EISDIR swallowed by the catch below.
-    if (entry.multi) continue;
-
-    const absPath = getAbsoluteOutputPath(projectRoot, featureName, key);
-    if (!existsSync(absPath)) continue;
-
-    try {
-      const content = readFileSync(absPath, 'utf8');
-      if (!content.trim()) continue; // Skip empty files
-
-      results.push({
-        source: entry.filename,
-        content,
-        weight: SOURCE_WEIGHTS[key] ?? 1.0,
-        _phase: entry.phase,
-      });
-    } catch {
-      // Skip unreadable files — fail-open
-      continue;
-    }
-  }
-
-  // Sort by phase order
-  results.sort((a, b) => {
-    const orderA = PHASE_ORDER.indexOf(a._phase);
-    const orderB = PHASE_ORDER.indexOf(b._phase);
-    return (orderA === -1 ? 99 : orderA) - (orderB === -1 ? 99 : orderB);
-  });
-
-  // Remove internal _phase field
-  return results.map(({ source, content, weight }) => ({ source, content, weight }));
-}