ultra-builder-pro-cli 0.13.0 → 0.15.0

package/mcp-server/tests/telemetry.test.cjs

@@ -1,219 +0,0 @@
-'use strict';
-
-// Phase 6.2 — Telemetry collectors:
-//   • appendTelemetry writes to state.db.telemetry AND .ultra/telemetry/
-//     {YYYY-MM-DD}.jsonl (double-write; jsonl keeps full payload the table
-//     can't hold).
-//   • computeCost(runtime, model, ti, to) returns USD from 2026-04 pricing.
-//   • aggregateTelemetryByRuntime / byTask / bySession drive /ultra-status.
-
-const { test } = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-const { initStateDb, closeStateDb } = require('../lib/state-db.cjs');
-const ops = require('../lib/state-ops.cjs');
-const telemetry = require('../lib/telemetry.cjs');
-const pricing = require('../lib/pricing.cjs');
-
-function freshFixture() {
-  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'ubp-tele-'));
-  const file = path.join(dir, '.ultra', 'state.db');
-  const init = initStateDb(file);
-  return { dir, db: init.db };
-}
-
-function teardown(dir, db) {
-  try { closeStateDb(db); } catch (_) { /* ignore */ }
-  try { fs.rmSync(dir, { recursive: true, force: true }); } catch (_) { /* ignore */ }
-}
-
-function seedSession(db, { sid = 's-1', task_id = 't-1', runtime = 'claude' } = {}) {
-  ops.createTask(db, { id: task_id, title: 'tele target', type: 'feature', priority: 'P1' });
-  ops.createSession(db, {
-    sid, task_id, runtime, pid: null,
-    worktree_path: `/tmp/${sid}/wt`,
-    artifact_dir: `/tmp/${sid}/art`,
-  });
-  return { sid, task_id, runtime };
-}
-
-// ─── pricing ──────────────────────────────────────────────────────────────
-
-test('computeCost: claude opus produces non-zero USD for realistic tokens', () => {
-  const cost = pricing.computeCost('claude', 'claude-opus-4-7', 1000, 500);
-  assert.ok(cost > 0 && cost < 1, `expected reasonable cost, got ${cost}`);
-});
-
-test('computeCost: unknown runtime → null', () => {
-  assert.equal(pricing.computeCost('madeup', 'model-x', 100, 100), null);
-});
-
-test('computeCost: zero tokens → zero', () => {
-  assert.equal(pricing.computeCost('claude', 'claude-sonnet-4-6', 0, 0), 0);
-});
-
-// ─── appendTelemetry ─────────────────────────────────────────────────────
-
-test('appendTelemetry writes to telemetry table', () => {
-  const { dir, db } = freshFixture();
-  try {
-    seedSession(db, { sid: 's-t1' });
-    const out = telemetry.appendTelemetry(db, {
-      event_type: 'tool_call',
-      tool_name: 'task.list',
-      session_id: 's-t1',
-      rootDir: dir,
-    });
-    assert.ok(out.id > 0);
-    const row = db.prepare('SELECT * FROM telemetry WHERE id = ?').get(out.id);
-    assert.equal(row.event_type, 'tool_call');
-    assert.equal(row.tool_name, 'task.list');
-    assert.equal(row.session_id, 's-t1');
-  } finally { teardown(dir, db); }
-});
-
-test('appendTelemetry writes a jsonl line under .ultra/telemetry/YYYY-MM-DD.jsonl', () => {
-  const { dir, db } = freshFixture();
-  try {
-    seedSession(db, { sid: 's-t2' });
-    telemetry.appendTelemetry(db, {
-      event_type: 'tool_call',
-      tool_name: 'task.create',
-      session_id: 's-t2',
-      rootDir: dir,
-      payload: { duration_ms: 42 },
-    });
-    const today = new Date().toISOString().slice(0, 10);
-    const jsonl = path.join(dir, '.ultra', 'telemetry', `${today}.jsonl`);
-    assert.ok(fs.existsSync(jsonl), `expected jsonl at ${jsonl}`);
-    const lines = fs.readFileSync(jsonl, 'utf8').trim().split('\n');
-    assert.equal(lines.length, 1);
-    const parsed = JSON.parse(lines[0]);
-    assert.equal(parsed.tool_name, 'task.create');
-    assert.equal(parsed.payload.duration_ms, 42);
-  } finally { teardown(dir, db); }
-});
-
-test('appendTelemetry auto-computes cost_usd when runtime + tokens supplied', () => {
-  const { dir, db } = freshFixture();
-  try {
-    seedSession(db, { sid: 's-t3' });
-    const out = telemetry.appendTelemetry(db, {
-      event_type: 'token_usage',
-      tool_name: 'session.close',
-      session_id: 's-t3',
-      runtime: 'claude',
-      tokens_input: 2000,
-      tokens_output: 500,
-      rootDir: dir,
-      payload: { model: 'claude-sonnet-4-6' },
-    });
-    const row = db.prepare('SELECT cost_usd FROM telemetry WHERE id = ?').get(out.id);
-    assert.ok(row.cost_usd > 0, 'cost should be computed from tokens');
-  } finally { teardown(dir, db); }
-});
-
-test('appendTelemetry tolerates null session_id (pre-session CLI calls)', () => {
-  const { dir, db } = freshFixture();
-  try {
-    const out = telemetry.appendTelemetry(db, {
-      event_type: 'tool_call',
-      tool_name: 'cli.task.list',
-      session_id: null,
-      rootDir: dir,
-    });
-    const row = db.prepare('SELECT * FROM telemetry WHERE id = ?').get(out.id);
-    assert.equal(row.session_id, null);
-  } finally { teardown(dir, db); }
-});
-
-test('appendTelemetry event_type constraint: invalid value rejected', () => {
-  const { dir, db } = freshFixture();
-  try {
-    assert.throws(
-      () => telemetry.appendTelemetry(db, { event_type: 'bogus', rootDir: dir }),
-      /CHECK|constraint|event_type/,
-    );
-  } finally { teardown(dir, db); }
-});
-
-test('appendTelemetry rejects unsupported runtime labels', () => {
-  const { dir, db } = freshFixture();
-  const retiredRuntime = ['gem', 'ini'].join('');
-  try {
-    assert.throws(
-      () => telemetry.appendTelemetry(db, {
-        event_type: 'tool_call', runtime: retiredRuntime, rootDir: dir,
-      }),
-      /unsupported runtime/,
-    );
-    assert.equal(db.prepare('SELECT COUNT(*) AS n FROM telemetry').get().n, 0);
-  } finally { teardown(dir, db); }
-});
-
-// ─── aggregation ─────────────────────────────────────────────────────────
-
-test('aggregateTelemetryByRuntime: groups calls + sums tokens/cost', () => {
-  const { dir, db } = freshFixture();
-  try {
-    seedSession(db, { sid: 's-a1', task_id: 't-a1', runtime: 'claude' });
-    seedSession(db, { sid: 's-a2', task_id: 't-a2', runtime: 'codex' });
-    telemetry.appendTelemetry(db, { event_type: 'token_usage', tool_name: 'x', session_id: 's-a1', runtime: 'claude', tokens_input: 1000, tokens_output: 200, rootDir: dir, payload: { model: 'claude-sonnet-4-6' } });
-    telemetry.appendTelemetry(db, { event_type: 'token_usage', tool_name: 'y', session_id: 's-a1', runtime: 'claude', tokens_input: 500, tokens_output: 100, rootDir: dir, payload: { model: 'claude-sonnet-4-6' } });
-    telemetry.appendTelemetry(db, { event_type: 'token_usage', tool_name: 'z', session_id: 's-a2', runtime: 'codex', tokens_input: 800, tokens_output: 300, rootDir: dir, payload: { model: 'gpt-5.4' } });
-
-    const summary = ops.aggregateTelemetryByRuntime(db);
-    const byRuntime = Object.fromEntries(summary.map((r) => [r.runtime, r]));
-    assert.equal(byRuntime.claude.calls, 2);
-    assert.equal(byRuntime.codex.calls, 1);
-    assert.equal(byRuntime.claude.tokens_in, 1500);
-    assert.equal(byRuntime.codex.tokens_in, 800);
-    assert.ok(byRuntime.claude.cost_usd > 0);
-  } finally { teardown(dir, db); }
-});
-
-test('aggregateTelemetryByTask: top N tasks by cost', () => {
-  const { dir, db } = freshFixture();
-  try {
-    seedSession(db, { sid: 's-b1', task_id: 't-cheap', runtime: 'claude' });
-    seedSession(db, { sid: 's-b2', task_id: 't-expensive', runtime: 'claude' });
-    telemetry.appendTelemetry(db, { event_type: 'token_usage', tool_name: 'x', session_id: 's-b1', runtime: 'claude', tokens_input: 100, tokens_output: 50, rootDir: dir, payload: { model: 'claude-sonnet-4-6' } });
-    telemetry.appendTelemetry(db, { event_type: 'token_usage', tool_name: 'y', session_id: 's-b2', runtime: 'claude', tokens_input: 10000, tokens_output: 5000, rootDir: dir, payload: { model: 'claude-opus-4-7' } });
-
-    const top = ops.aggregateTelemetryByTask(db, { limit: 1 });
-    assert.equal(top.length, 1);
-    assert.equal(top[0].task_id, 't-expensive');
-    assert.ok(top[0].cost_usd > 0);
-  } finally { teardown(dir, db); }
-});
-
-test('aggregateTelemetryBySession: per-session roll-up', () => {
-  const { dir, db } = freshFixture();
-  try {
-    seedSession(db, { sid: 's-c1', task_id: 't-c1', runtime: 'claude' });
-    telemetry.appendTelemetry(db, { event_type: 'tool_call', tool_name: 'task.list', session_id: 's-c1', rootDir: dir });
-    telemetry.appendTelemetry(db, { event_type: 'tool_call', tool_name: 'task.get', session_id: 's-c1', rootDir: dir });
-    telemetry.appendTelemetry(db, { event_type: 'token_usage', tool_name: 'session.close', session_id: 's-c1', runtime: 'claude', tokens_input: 500, tokens_output: 100, rootDir: dir, payload: { model: 'claude-haiku-4-5' } });
-
-    const summary = ops.aggregateTelemetryBySession(db, 's-c1');
-    assert.equal(summary.tool_calls, 3);
-    assert.equal(summary.tokens_in, 500);
-    assert.equal(summary.tokens_out, 100);
-    assert.ok(summary.cost_usd > 0);
-  } finally { teardown(dir, db); }
-});
-
-test('aggregate filter: since cutoff excludes older rows', () => {
-  const { dir, db } = freshFixture();
-  try {
-    seedSession(db, { sid: 's-d1', task_id: 't-d1', runtime: 'claude' });
-    // Insert one row; then a cutoff in the future filters it out.
-    telemetry.appendTelemetry(db, { event_type: 'tool_call', tool_name: 'x', session_id: 's-d1', rootDir: dir });
-    const futureCutoff = new Date(Date.now() + 60 * 1000).toISOString();
-    const summary = ops.aggregateTelemetryByRuntime(db, { since: futureCutoff });
-    assert.equal(summary.length, 0);
-  } finally { teardown(dir, db); }
-});

package/orchestrator/planner/expand-prompt.cjs

@@ -1,62 +0,0 @@
-'use strict';
-
-// Phase 8A.3 — Prompt templates for task → subtasks expansion.
-//
-// Parent context (title, type, priority, complexity, files_modified) is
-// serialized into the user prompt so the LLM can produce children aligned
-// with the parent's shape. sub_count defaults to 3 when omitted.
-
-const DEFAULT_SUB_COUNT = 3;
-
-function buildExpandSystemPrompt({ sub_count = DEFAULT_SUB_COUNT } = {}) {
-  return `You are an engineering task decomposer. Break a single complex task into ${sub_count} smaller subtasks that together deliver the parent's scope.
-
-Output STRICT JSON matching this shape — nothing else, no markdown fences, no prose:
-
-{
-  "children": [
-    {
-      "id": "child-1",
-      "title": "Imperative short title (3-80 chars)",
-      "type": "architecture | feature | bugfix",
-      "priority": "P0 | P1 | P2 | P3",
-      "complexity": 1-10,
-      "deps": ["child-2"],
-      "files_modified": ["src/path/to/file.ts"]
-    }
-  ]
-}
-
-Rules:
-- Exactly ${sub_count} children.
-- Each id matches /^[a-zA-Z0-9_\\-]+$/ and is unique in the list.
-- "deps" references siblings in the same list; omit or use [] when none.
-- "files_modified" overlaps with the parent's files_modified where possible.
-- Each child's complexity is < parent's complexity; sum should approximate the parent.
-- Return JSON only.`;
-}
-
-function buildExpandUserPrompt(parent) {
-  const files = Array.isArray(parent.files_modified) ? parent.files_modified : [];
-  const deps = Array.isArray(parent.deps) ? parent.deps : [];
-  return [
-    'Parent task to decompose:',
-    '',
-    `- id: ${parent.id}`,
-    `- title: ${parent.title}`,
-    `- type: ${parent.type}`,
-    `- priority: ${parent.priority}`,
-    parent.complexity != null ? `- complexity: ${parent.complexity}` : null,
-    files.length ? `- files_modified: ${JSON.stringify(files)}` : null,
-    deps.length ? `- existing deps: ${JSON.stringify(deps)}` : null,
-    parent.trace_to ? `- trace_to: ${parent.trace_to}` : null,
-    '',
-    'Return the children list as JSON following the system schema.',
-  ].filter((l) => l !== null).join('\n');
-}
-
-module.exports = {
-  buildExpandSystemPrompt,
-  buildExpandUserPrompt,
-  DEFAULT_SUB_COUNT,
-};

package/orchestrator/planner/expand-prompt.test.cjs

@@ -1,52 +0,0 @@
-'use strict';
-
-const test = require('node:test');
-const assert = require('node:assert/strict');
-
-const {
-  buildExpandSystemPrompt,
-  buildExpandUserPrompt,
-  DEFAULT_SUB_COUNT,
-} = require('./expand-prompt.cjs');
-
-test('buildExpandSystemPrompt: mentions sub_count explicitly', () => {
-  const s = buildExpandSystemPrompt({ sub_count: 4 });
-  assert.match(s, /Exactly 4 children/);
-  assert.match(s, /STRICT JSON/);
-});
-
-test('buildExpandSystemPrompt: falls back to DEFAULT_SUB_COUNT when omitted', () => {
-  const s = buildExpandSystemPrompt({});
-  assert.match(s, new RegExp(`Exactly ${DEFAULT_SUB_COUNT} children`));
-});
-
-test('buildExpandUserPrompt: serializes parent fields into prompt body', () => {
-  const parent = {
-    id: 'parent-1',
-    title: 'Build search feature',
-    type: 'feature',
-    priority: 'P1',
-    complexity: 8,
-    files_modified: ['src/search/index.ts', 'src/search/query.ts'],
-    deps: ['infra-1'],
-    trace_to: 'spec §3.2',
-  };
-  const body = buildExpandUserPrompt(parent);
-  assert.match(body, /parent-1/);
-  assert.match(body, /Build search feature/);
-  assert.match(body, /complexity: 8/);
-  assert.match(body, /src\/search\/index\.ts/);
-  assert.match(body, /infra-1/);
-  assert.match(body, /spec §3\.2/);
-});
-
-test('buildExpandUserPrompt: omits empty optional fields', () => {
-  const parent = {
-    id: 'p', title: 'Tiny task', type: 'feature', priority: 'P3',
-    complexity: null, files_modified: [], deps: [],
-  };
-  const body = buildExpandUserPrompt(parent);
-  assert.doesNotMatch(body, /files_modified/);
-  assert.doesNotMatch(body, /existing deps/);
-  assert.doesNotMatch(body, /complexity:/);
-});

package/orchestrator/planner/plan-builder.test.cjs

@@ -1,165 +0,0 @@
-'use strict';
-
-const test = require('node:test');
-const assert = require('node:assert/strict');
-
-const { buildPlan, listWaveConflicts } = require('./plan-builder.cjs');
-
-function mk(id, deps = [], files = [], complexity = 3) {
-  return { id, deps, files_modified: files, complexity };
-}
-
-test('buildPlan fixture 1: 5 tasks, no file overlap → 2 waves, no conflicts', () => {
-  const tasks = [
-    mk('a', [], ['src/a.ts'], 2),
-    mk('b', ['a'], ['src/b.ts'], 3),
-    mk('c', ['a'], ['src/c.ts'], 4),
-    mk('d', ['b', 'c'], ['src/d.ts'], 5),
-    mk('e', [], ['src/e.ts'], 2),
-  ];
-  const plan = buildPlan(tasks);
-  assert.equal(plan.waves.length, 3);
-  assert.equal(plan.conflict_surface.length, 0);
-  // wave 1 should contain a+e, both parallel-safe
-  assert.deepEqual(new Set(plan.waves[0].tasks), new Set(['a', 'e']));
-  assert.equal(plan.waves[0].parallel, true);
-  assert.deepEqual(plan.cycles, []);
-  assert.ok(plan.estimated_cost_usd > 0);
-  assert.ok(plan.estimated_duration_min > 0);
-});
-
-test('buildPlan fixture 2: same wave, shared file → conflict_surface + parallel=false', () => {
-  const tasks = [
-    mk('a', []),
-    mk('b', ['a'], ['src/utils.ts']),
-    mk('c', ['a'], ['src/utils.ts']),
-  ];
-  const plan = buildPlan(tasks);
-  assert.equal(plan.waves.length, 2);
-  const w2 = plan.waves[1];
-  assert.equal(w2.parallel, false);
-  assert.match(w2.reason, /shared files: src\/utils\.ts/);
-
-  assert.equal(plan.conflict_surface.length, 1);
-  const c = plan.conflict_surface[0];
-  assert.deepEqual(c.files, ['src/utils.ts']);
-  assert.deepEqual(new Set(c.tasks), new Set(['b', 'c']));
-  assert.equal(c.recommend, 'sequentialize');
-});
-
-test('buildPlan fixture 3: cross-wave same file → NOT a conflict', () => {
-  const tasks = [
-    mk('a', [], ['src/shared.ts']),
-    mk('b', ['a'], ['src/shared.ts']),
-  ];
-  const plan = buildPlan(tasks);
-  assert.equal(plan.conflict_surface.length, 0);
-  // wave 1 has only `a`, not parallel; wave 2 has only `b`, not parallel
-  for (const w of plan.waves) {
-    assert.equal(w.parallel, false);
-    assert.ok(!w.reason);
-  }
-});
-
-test('buildPlan fixture 4: empty tasks → empty plan, non-negative numbers', () => {
-  const plan = buildPlan([]);
-  assert.deepEqual(plan.waves, []);
-  assert.deepEqual(plan.conflict_surface, []);
-  assert.deepEqual(plan.ownership_forecast, {});
-  assert.equal(plan.estimated_cost_usd, 0);
-  assert.equal(plan.estimated_duration_min, 0);
-  assert.deepEqual(plan.cycles, []);
-});
-
-test('buildPlan fixture 5: cost + duration math (parallel vs serial waves)', () => {
-  // wave 1 (parallel): a complexity=2, b complexity=4 → max = 4 * 5 = 20 min
-  // wave 2 (1 task, not parallel): c complexity=6 → 6 * 5 = 30 min
-  // total duration = 50 min
-  const tasks = [
-    mk('a', [], ['src/a.ts'], 2),
-    mk('b', [], ['src/b.ts'], 4),
-    mk('c', ['a', 'b'], ['src/c.ts'], 6),
-  ];
-  const plan = buildPlan(tasks);
-  assert.equal(plan.estimated_duration_min, 50);
-  // cost: 3 tasks × sum(complexity)=12 × (5000*3e-6 + 2000*15e-6) per complexity unit
-  //     = 12 × (0.015 + 0.030) = 12 × 0.045 = 0.54 usd
-  assert.ok(Math.abs(plan.estimated_cost_usd - 0.54) < 0.001);
-});
-
-test('buildPlan: serial wave (with conflict) sums complexities instead of max', () => {
-  // both tasks in same wave share a file → wave is serial
-  // wave 1: a(complexity=3), b(complexity=3) → (3+3) * 5 = 30 min
-  const tasks = [
-    mk('a', [], ['src/utils.ts'], 3),
-    mk('b', [], ['src/utils.ts'], 3),
-  ];
-  const plan = buildPlan(tasks);
-  assert.equal(plan.waves[0].parallel, false);
-  assert.equal(plan.estimated_duration_min, 30);
-});
-
-test('buildPlan: ownership_forecast captures per-task files_modified verbatim', () => {
-  const tasks = [
-    mk('a', [], ['src/a.ts', 'src/a.util.ts']),
-    mk('b', [], ['src/b.ts']),
-  ];
-  const plan = buildPlan(tasks);
-  assert.deepEqual(plan.ownership_forecast, {
-    a: ['src/a.ts', 'src/a.util.ts'],
-    b: ['src/b.ts'],
-  });
-});
-
-test('buildPlan: cycle in task graph surfaced in plan.cycles', () => {
-  const tasks = [
-    mk('x', ['y']),
-    mk('y', ['x']),
-  ];
-  const plan = buildPlan(tasks);
-  assert.deepEqual(plan.waves, []);
-  assert.equal(plan.cycles.length, 1);
-  assert.deepEqual(new Set(plan.cycles[0]), new Set(['x', 'y']));
-});
-
-test('buildPlan: missing complexity defaults to DEFAULT_COMPLEXITY for cost calc', () => {
-  const tasks = [{ id: 'a', deps: [], files_modified: [] }];
-  const plan = buildPlan(tasks);
-  // complexity 3 → 15k in, 6k out → cost > 0
-  assert.ok(plan.estimated_cost_usd > 0);
-  assert.equal(plan.estimated_duration_min, 15); // 3 * 5 (single-task wave, not parallel)
-});
-
-test('buildPlan: non-array input → TypeError', () => {
-  assert.throws(() => buildPlan(null), TypeError);
-  assert.throws(() => buildPlan({}), TypeError);
-});
-
-test('listWaveConflicts: no overlap → empty', () => {
-  const byId = new Map([
-    ['a', { id: 'a', files_modified: ['x.ts'] }],
-    ['b', { id: 'b', files_modified: ['y.ts'] }],
-  ]);
-  assert.deepEqual(listWaveConflicts(['a', 'b'], byId), []);
-});
-
-test('listWaveConflicts: pair overlap reported once', () => {
-  const byId = new Map([
-    ['a', { id: 'a', files_modified: ['shared.ts', 'x.ts'] }],
-    ['b', { id: 'b', files_modified: ['shared.ts'] }],
-  ]);
-  const out = listWaveConflicts(['a', 'b'], byId);
-  assert.equal(out.length, 1);
-  assert.deepEqual(out[0].files, ['shared.ts']);
-  assert.deepEqual(out[0].tasks, ['a', 'b']);
-});
-
-test('listWaveConflicts: three-way overlap yields C(3,2)=3 pairs', () => {
-  const byId = new Map([
-    ['a', { id: 'a', files_modified: ['x.ts'] }],
-    ['b', { id: 'b', files_modified: ['x.ts'] }],
-    ['c', { id: 'c', files_modified: ['x.ts'] }],
-  ]);
-  const out = listWaveConflicts(['a', 'b', 'c'], byId);
-  assert.equal(out.length, 3);
-});

package/orchestrator/planner/prd-prompt.cjs

@@ -1,44 +0,0 @@
-'use strict';
-
-// Phase 8A.1 — Prompt templates for PRD → task-list parsing.
-//
-// Kept in a dedicated module so downstream evaluation (fixture regression,
-// prompt A/B testing) can diff the template against commits without touching
-// parser logic.
-
-const SYSTEM_PROMPT = `You are an engineering planner. Decompose a product requirement document (PRD) into a topologically ordered task list that a coding agent can execute sequentially or in parallel.
-
-Output STRICT JSON matching this shape — nothing else, no markdown fences, no prose:
-
-{
-  "tasks": [
-    {
-      "id": "task-1",
-      "title": "Imperative short title (3-80 chars)",
-      "type": "architecture | feature | bugfix",
-      "priority": "P0 | P1 | P2 | P3",
-      "complexity": 1-10,
-      "deps": ["task-2"],
-      "files_modified": ["src/path/to/file.ts"]
-    }
-  ]
-}
-
-Rules:
-- Every task id matches /^task-[0-9]+$/ and is unique in the list.
-- "deps" references other ids in the same list; omit or use [] when none.
-- "files_modified" is your best guess of affected paths; use [] when unsure.
-- "complexity" 1 = trivial, 10 = multi-day. Tasks at 7+ will be auto-expanded later.
-- "priority" — P0 blocks other work, P1 critical path, P2 standard, P3 nice-to-have.
-- "type" — architecture for design/scaffolding, feature for new capability, bugfix for correcting existing behaviour.
-- Return JSON only. No surrounding text, no trailing explanation.`;
-
-function buildSystemPrompt() {
-  return SYSTEM_PROMPT;
-}
-
-function buildUserPrompt(prdText) {
-  return `PRD:\n\n${prdText}\n\nReturn the task list as JSON following the system schema.`;
-}
-
-module.exports = { SYSTEM_PROMPT, buildSystemPrompt, buildUserPrompt };

package/orchestrator/planner/prd-prompt.test.cjs

@@ -1,30 +0,0 @@
-'use strict';
-
-const test = require('node:test');
-const assert = require('node:assert/strict');
-
-const {
-  SYSTEM_PROMPT,
-  buildSystemPrompt,
-  buildUserPrompt,
-} = require('./prd-prompt.cjs');
-
-test('buildSystemPrompt returns the module-level template', () => {
-  assert.equal(buildSystemPrompt(), SYSTEM_PROMPT);
-  assert.match(SYSTEM_PROMPT, /engineering planner/i);
-  assert.match(SYSTEM_PROMPT, /STRICT JSON/);
-  assert.match(SYSTEM_PROMPT, /\/\^task-\[0-9\]\+\$\//);
-});
-
-test('buildUserPrompt embeds PRD text verbatim', () => {
-  const prd = 'As a user I want to search products by keyword';
-  const out = buildUserPrompt(prd);
-  assert.ok(out.includes(prd), 'user prompt must contain the PRD text');
-  assert.match(out, /JSON/);
-});
-
-test('buildUserPrompt tolerates long PRD text without truncation', () => {
-  const long = 'line\n'.repeat(500);
-  const out = buildUserPrompt(long);
-  assert.ok(out.includes(long));
-});