@dtlabs/ai-toolkit 0.8.0 → 0.10.0

package/.claude/scripts/tests/wb-validate.checks-8-10.test.js

@@ -0,0 +1,243 @@
+'use strict';
+
+const { spawnSync } = require('child_process');
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
+
+const VALIDATE = path.resolve(__dirname, '../wb-validate.js');
+
+/**
+ * Run wb-validate.js as a subprocess and return exit code, parsed JSON report,
+ * and stderr.  No requirements path is needed for checks 8–10.
+ */
+function runValidator(wbPath) {
+  const result = spawnSync('node', [VALIDATE, wbPath], { encoding: 'utf8' });
+  let report = null;
+  try { report = JSON.parse(result.stdout); } catch (_) {}
+  return { exitCode: result.status, report, stderr: result.stderr };
+}
+
+// ── Shared temp-file lifecycle ────────────────────────────────────────────────
+
+let tmpFile;
+
+beforeEach(() => {
+  tmpFile = path.join(os.tmpdir(), `wb-t16-${Date.now()}.json`);
+});
+
+afterEach(() => {
+  try { fs.unlinkSync(tmpFile); } catch (_) {}
+});
+
+function writeFixture(obj) {
+  fs.writeFileSync(tmpFile, JSON.stringify(obj, null, 2));
+  return tmpFile;
+}
+
+// ── Domain builders ───────────────────────────────────────────────────────────
+
+function makeTask(id, dependsOn = []) {
+  return {
+    id,
+    title: `Task ${id}`,
+    outcome: 'Something done',
+    domain: 'BE',
+    agentType: 'developer-backend',
+    dependsOn,
+    acceptanceCriteria: [],
+    verification: { commands: ['node --version'] },
+    estimate: { agentMinutes: 10, tokens: 5000 },
+    outputCount: 1,
+    groupingRationale: 'Single responsibility',
+    commit: { subject: `feat: ${id}` },
+  };
+}
+
+function makePhase(id, tasks, type = 'user-story') {
+  return { id, type, title: `Phase ${id}`, commit: `feat(${id}): done`, tasks };
+}
+
+function makeWB(phases) {
+  return { schemaVersion: 2, phases };
+}
+
+// ─────────────────────────────────────────────────────────────────────────────
+
+describe('wb-validate.js — checks 8–10', () => {
+
+  // ── Check 8: dependency reference existence ────────────────────────────────
+
+  describe('check 8 — dependency reference existence', () => {
+
+    test('exits 1 and reports dependency_not_found for the referencing task when dependsOn contains a non-existent task ID', () => {
+      // Arrange: task depends on US-01-TASK-BE-99 which is not declared anywhere
+      const wb = makeWB([
+        makePhase('US-01', [
+          makeTask('US-01-TASK-BE-01', ['US-01-TASK-BE-99']),
+        ]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(1);
+      expect(report.valid).toBe(false);
+      const err = report.errors.find(e => e.category === 'dependency_not_found');
+      expect(err).toBeDefined();
+      expect(err.taskId).toBe('US-01-TASK-BE-01');
+    });
+
+    test('exits 0 and reports no dependency_not_found errors when all dependsOn IDs resolve to declared tasks', () => {
+      // Arrange: two tasks in the same phase; second depends on first
+      const wb = makeWB([
+        makePhase('US-01', [
+          makeTask('US-01-TASK-BE-01'),
+          makeTask('US-01-TASK-BE-02', ['US-01-TASK-BE-01']),
+        ]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(0);
+      expect(report.valid).toBe(true);
+      expect(report.errors.filter(e => e.category === 'dependency_not_found')).toHaveLength(0);
+    });
+
+    test('exits 0 and reports no dependency_not_found errors when a US task depends on an INFRA task from a preceding phase', () => {
+      // Arrange: INFRA phase declared first; US-01 task resolves its dep cross-phase
+      const wb = makeWB([
+        makePhase('INFRA', [makeTask('INFRA-TASK-BE-01')], 'infra'),
+        makePhase('US-01', [makeTask('US-01-TASK-BE-01', ['INFRA-TASK-BE-01'])]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(0);
+      expect(report.valid).toBe(true);
+      expect(report.errors.filter(e => e.category === 'dependency_not_found')).toHaveLength(0);
+    });
+  });
+
+  // ── Check 9: self-dependency ───────────────────────────────────────────────
+
+  describe('check 9 — self-dependency', () => {
+
+    test('exits 1 and reports self_dependency for the task that lists its own ID in dependsOn', () => {
+      // Arrange: task references itself as a dependency
+      const wb = makeWB([
+        makePhase('US-01', [
+          makeTask('US-01-TASK-BE-01', ['US-01-TASK-BE-01']),
+        ]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(1);
+      expect(report.valid).toBe(false);
+      const err = report.errors.find(e => e.category === 'self_dependency');
+      expect(err).toBeDefined();
+      expect(err.taskId).toBe('US-01-TASK-BE-01');
+    });
+
+    test('exits 0 and reports no self_dependency errors when no task lists its own ID in dependsOn', () => {
+      // Arrange: simple valid WB with no self-references
+      const wb = makeWB([
+        makePhase('US-01', [makeTask('US-01-TASK-BE-01')]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(0);
+      expect(report.valid).toBe(true);
+      expect(report.errors.filter(e => e.category === 'self_dependency')).toHaveLength(0);
+    });
+  });
+
+  // ── Check 10: phase ID / task ID prefix consistency ────────────────────────
+
+  describe('check 10 — phase ID / task ID prefix consistency', () => {
+
+    test('exits 1 and reports phase_id_mismatch when an INFRA-prefixed task ID appears in a non-INFRA phase', () => {
+      // Arrange: task ID starts with INFRA-TASK- but is placed in US-01
+      const wb = makeWB([
+        makePhase('US-01', [
+          makeTask('INFRA-TASK-BE-01'),
+        ]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(1);
+      expect(report.valid).toBe(false);
+      expect(report.errors.some(e => e.category === 'phase_id_mismatch')).toBe(true);
+    });
+
+    test('exits 1 and reports phase_id_mismatch when a task ID carries a US-prefix that does not match its containing phase', () => {
+      // Arrange: task prefixed US-02 but placed in phase US-01
+      const wb = makeWB([
+        makePhase('US-01', [
+          makeTask('US-02-TASK-BE-01'),
+        ]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(1);
+      expect(report.valid).toBe(false);
+      expect(report.errors.some(e => e.category === 'phase_id_mismatch')).toBe(true);
+    });
+
+    test('exits 0 and reports no phase_id_mismatch errors when a US task ID prefix exactly matches its phase ID', () => {
+      // Arrange: prefix US-01 matches phase US-01
+      const wb = makeWB([
+        makePhase('US-01', [makeTask('US-01-TASK-BE-01')]),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(0);
+      expect(report.valid).toBe(true);
+      expect(report.errors.filter(e => e.category === 'phase_id_mismatch')).toHaveLength(0);
+    });
+
+    test('exits 0 and reports no phase_id_mismatch errors when an INFRA-prefixed task is placed in the INFRA phase', () => {
+      // Arrange: INFRA-TASK-BE-01 correctly placed in the INFRA phase
+      const wb = makeWB([
+        makePhase('INFRA', [makeTask('INFRA-TASK-BE-01')], 'infra'),
+      ]);
+      writeFixture(wb);
+
+      // Act
+      const { exitCode, report } = runValidator(tmpFile);
+
+      // Assert
+      expect(exitCode).toBe(0);
+      expect(report.valid).toBe(true);
+      expect(report.errors.filter(e => e.category === 'phase_id_mismatch')).toHaveLength(0);
+    });
+  });
+});

package/.claude/scripts/wb-render.js

@@ -0,0 +1,309 @@
+'use strict'
+
+const fs   = require('fs')
+const path = require('path')
+
+// ── Argument parsing ──────────────────────────────────────────────────────────
+
+const jsonPath = process.argv[2]
+const prefix   = process.argv[3]
+
+if (!jsonPath || !prefix) {
+  process.stderr.write('Usage: node wb-render.js <path-to-work-breakdown.json> <feature-prefix> [destination-dir]\n')
+  process.exit(1)
+}
+
+const destDir = process.argv[4]
+  ? process.argv[4]
+  : path.dirname(path.resolve(jsonPath))
+
+// ── Read and parse JSON ───────────────────────────────────────────────────────
+
+let raw
+try {
+  raw = fs.readFileSync(jsonPath, 'utf8')
+} catch (err) {
+  process.stderr.write(`Error: cannot read file "${jsonPath}": ${err.message}\n`)
+  process.exit(1)
+}
+
+let wb
+try {
+  wb = JSON.parse(raw)
+} catch (err) {
+  process.stderr.write(`Error: "${jsonPath}" is not valid JSON: ${err.message}\n`)
+  process.exit(1)
+}
+
+// ── Basic structure check ─────────────────────────────────────────────────────
+
+if (wb.schemaVersion !== 2) {
+  process.stderr.write(`Error: schemaVersion must be 2, got ${JSON.stringify(wb.schemaVersion)}\n`)
+  process.exit(2)
+}
+
+if (!Array.isArray(wb.phases)) {
+  process.stderr.write('Error: "phases" must be an array\n')
+  process.exit(2)
+}
+
+// ── Output file paths ─────────────────────────────────────────────────────────
+
+const mdPath  = path.join(destDir, `${prefix}-Work-Breakdown.md`)
+const csvPath = path.join(destDir, `${prefix}-Work-Breakdown.csv`)
+
+// ── Phase-level depends_on aggregation ───────────────────────────────────────
+
+function computePhaseDependsOn(wb) {
+  // Build taskId → phaseId index
+  const taskToPhase = new Map()
+  for (const phase of wb.phases) {
+    const tasks = Array.isArray(phase.tasks) ? phase.tasks : []
+    for (const task of tasks) {
+      if (task.id != null) taskToPhase.set(task.id, phase.id)
+    }
+  }
+
+  // For each phase, compute external phase dependencies
+  const result = new Map()
+  for (const phase of wb.phases) {
+    if (phase.id == null) continue
+    const extPhases = new Set()
+    const tasks = Array.isArray(phase.tasks) ? phase.tasks : []
+    for (const task of tasks) {
+      if (!Array.isArray(task.dependsOn)) continue
+      for (const depId of task.dependsOn) {
+        const ownerPhase = taskToPhase.get(depId)
+        if (ownerPhase == null) continue          // unresolved — skip
+        if (ownerPhase === phase.id) continue     // intra-phase — remove
+        extPhases.add(ownerPhase)
+      }
+    }
+    result.set(phase.id, [...extPhases].sort().join(' '))
+  }
+  return result
+}
+
+// ── Compute phase dependencies ────────────────────────────────────────────────
+
+let phaseDepsMap
+try {
+  phaseDepsMap = computePhaseDependsOn(wb)
+} catch (err) {
+  process.stderr.write(`Error: phase dependency aggregation failed: ${err.message}\n`)
+  process.exit(3)
+}
+
+// ── Field helpers ─────────────────────────────────────────────────────────────
+
+function sanitizeField(str) {
+  const s = typeof str === 'string' ? str : String(str ?? '')
+  return s
+    .replace(/\|/g, ' ')
+    .replace(/\r/g, '')
+    .replace(/\n/g, ' ')
+    .trim()
+}
+
+function buildCommitSubject(task) {
+  if (!task.commit || typeof task.commit !== 'object') {
+    return 'chore: implement task'
+  }
+  const type    = sanitizeField(task.commit.type    ?? '')
+  const scope   = sanitizeField(task.commit.scope   ?? '')
+  const subject = sanitizeField(task.commit.subject ?? '')
+  const full = `${type}(${scope}): ${subject}`
+  return full.length > 72 ? full.slice(0, 72) + '…' : full
+}
+
+// ── Render stubs (implemented in T03–T05) ─────────────────────────────────────
+
+function renderMarkdown(wb, prefix, phaseDepsMap) {
+  const DOMAINS = ['BE', 'FE', 'DB', 'DevOps', 'INFRA', 'TEST']
+
+  function getAgentMinutes(task) {
+    const m = task.agentMinutes ?? task.estimate?.agentMinutes
+    return (m !== undefined && m !== null) ? Number(m) : null
+  }
+
+  function getDurationBand(mins) {
+    if (mins === null) return null
+    if (mins <= 15) return 'target'
+    if (mins <= 20) return 'above'
+    if (mins <= 30) return 'warning'
+    return 'split'
+  }
+
+  const allTasks = wb.phases.flatMap(p => Array.isArray(p.tasks) ? p.tasks : [])
+
+  // Summary duration totals — across all tasks
+  let summaryTarget = 0, summaryAbove = 0, summaryWarning = 0, summarySplit = 0
+  for (const task of allTasks) {
+    const band = getDurationBand(getAgentMinutes(task))
+    if (band === 'target')  summaryTarget++
+    else if (band === 'above')   summaryAbove++
+    else if (band === 'warning') summaryWarning++
+    else if (band === 'split')   summarySplit++
+  }
+
+  // Per-domain stats — for Statistics table and domain distribution string
+  const domainStats = {}
+  for (const d of DOMAINS) domainStats[d] = { count: 0, target: 0, above: 0, warning: 0, split: 0 }
+  for (const task of allTasks) {
+    const d = task.domain
+    if (!d || !DOMAINS.includes(d)) continue
+    const band = getDurationBand(getAgentMinutes(task))
+    domainStats[d].count++
+    if (band) domainStats[d][band]++
+  }
+
+  const domainDistStr = DOMAINS.map(d => `${d}: ${domainStats[d].count}`).join(', ')
+
+  const statTotals = DOMAINS.reduce(
+    (acc, d) => {
+      acc.count   += domainStats[d].count
+      acc.target  += domainStats[d].target
+      acc.above   += domainStats[d].above
+      acc.warning += domainStats[d].warning
+      acc.split   += domainStats[d].split
+      return acc
+    },
+    { count: 0, target: 0, above: 0, warning: 0, split: 0 }
+  )
+
+  function phaseCommit(phase) {
+    if (!phase.commit || typeof phase.commit !== 'object') {
+      return `chore(${sanitizeField(prefix)}): implement phase`
+    }
+    const type    = sanitizeField(phase.commit.type    ?? 'chore')
+    const subject = sanitizeField(phase.commit.subject ?? '')
+    return `${type}(${sanitizeField(prefix)}): ${subject}`
+  }
+
+  function taskRow(task) {
+    const id       = sanitizeField(task.id ?? '—')
+    const title    = sanitizeField(task.title ?? '')
+    const outcome  = sanitizeField(task.outcome ?? '')
+    const domain   = sanitizeField(task.domain ?? '—')
+    const mins     = getAgentMinutes(task)
+    const est      = mins !== null ? String(mins) : '—'
+    const depArr   = Array.isArray(task.dependsOn) && task.dependsOn.length > 0 ? task.dependsOn : null
+    const deps     = depArr ? depArr.join(', ') : '—'
+    const verifArr = task.verificationCommands ?? task.verification?.commands ?? []
+    const verif    = verifArr.length > 0 ? sanitizeField(verifArr.join('; ')) : '—'
+    return `| ${id} | ${title} | ${outcome} | ${domain} | ${est} | ${deps} | ${verif} |`
+  }
+
+  const title     = wb.title ?? wb.feature ?? prefix
+  const generated = new Date().toISOString()
+  const L         = []
+
+  // ── Title ──
+  L.push(`# Work Breakdown — ${title}`)
+  L.push('')
+
+  // ── Document Info ──
+  L.push('## Document Info')
+  L.push('| Field | Value |')
+  L.push('|-------|-------|')
+  L.push(`| Feature | ${sanitizeField(prefix)} |`)
+  L.push('| Schema | v2 |')
+  L.push(`| Generated | ${generated} |`)
+  L.push('')
+
+  // ── Summary ──
+  L.push('## Summary')
+  L.push('| Metric | Value |')
+  L.push('|--------|-------|')
+  L.push(`| Total tasks | ${allTasks.length} |`)
+  L.push(`| Total phases | ${wb.phases.length} |`)
+  L.push(`| Within target (≤15 min) | ${summaryTarget} |`)
+  L.push(`| Above target (16–20 min) | ${summaryAbove} |`)
+  L.push(`| Warning (21–30 min) | ${summaryWarning} |`)
+  L.push(`| Split required (>30 min) | ${summarySplit} |`)
+  L.push(`| Domain distribution | ${domainDistStr} |`)
+  L.push('')
+
+  // ── Infrastructure Phase ──
+  const infraPhases     = wb.phases.filter(p => p.id === 'INFRA')
+  const userStoryPhases = wb.phases.filter(p => p.id !== 'INFRA')
+
+  for (const phase of infraPhases) {
+    L.push('## Infrastructure Phase (INFRA)')
+    L.push('')
+    L.push('### Commit')
+    L.push(phaseCommit(phase))
+    L.push('')
+    L.push('### Tasks')
+    L.push('| ID | Title | Outcome | Domain | Est. (min) | Dependencies | Verification |')
+    L.push('|---|---|---|---|---|---|---|')
+    for (const task of (Array.isArray(phase.tasks) ? phase.tasks : [])) {
+      L.push(taskRow(task))
+    }
+    L.push('')
+  }
+
+  // ── User Story Phases ──
+  if (userStoryPhases.length > 0) {
+    L.push('## User Story Phases')
+    L.push('')
+    for (const phase of userStoryPhases) {
+      L.push(`### ${sanitizeField(phase.id ?? '')}: ${sanitizeField(phase.title ?? '')}`)
+      L.push('')
+      L.push('### Commit')
+      L.push(phaseCommit(phase))
+      L.push('')
+      L.push('### Tasks')
+      L.push('| ID | Title | Outcome | Domain | Est. (min) | Dependencies | Verification |')
+      L.push('|---|---|---|---|---|---|---|')
+      for (const task of (Array.isArray(phase.tasks) ? phase.tasks : [])) {
+        L.push(taskRow(task))
+      }
+      L.push('')
+    }
+  }
+
+  // ── Statistics ──
+  L.push('## Statistics')
+  L.push('')
+  L.push('| Domain | Count | Target | Above | Warning | Split |')
+  L.push('|--------|-------|--------|-------|---------|-------|')
+  for (const d of DOMAINS) {
+    const s = domainStats[d]
+    L.push(`| ${d} | ${s.count} | ${s.target} | ${s.above} | ${s.warning} | ${s.split} |`)
+  }
+  L.push(`| **Total** | **${statTotals.count}** | **${statTotals.target}** | **${statTotals.above}** | **${statTotals.warning}** | **${statTotals.split}** |`)
+  L.push('')
+
+  return L.join('\n')
+}
+
+function renderCsv(wb, prefix, phaseDepsMap) {
+  const rows = ['phase_id|phase_title|commit_message|depends_on|task_id|task_title|domain|agent_type']
+  for (const phase of wb.phases) {
+    const phaseId    = sanitizeField(phase.id ?? '')
+    const phaseTitle = sanitizeField(phase.title ?? '')
+    const commitMsg  = phase.commit && typeof phase.commit === 'object'
+      ? sanitizeField(phase.commit.type ?? 'chore') + '(' + sanitizeField(prefix) + '): ' + sanitizeField(phase.commit.subject ?? '')
+      : `chore(${sanitizeField(prefix)}): implement phase`
+    const dependsOn  = sanitizeField(phaseDepsMap.get(phase.id) ?? '')
+    const tasks = Array.isArray(phase.tasks) ? phase.tasks : []
+    for (const task of tasks) {
+      const taskId    = sanitizeField(task.id ?? '')
+      const taskTitle = sanitizeField(task.title ?? '')
+      const domain    = sanitizeField(task.domain ?? '')
+      const agentType = sanitizeField(task.agentType ?? '')
+      rows.push(`${phaseId}|${phaseTitle}|${commitMsg}|${dependsOn}|${taskId}|${taskTitle}|${domain}|${agentType}`)
+    }
+  }
+  return rows.join('\n') + '\n'
+}
+
+// ── Write files and exit ──────────────────────────────────────────────────────
+
+const md  = renderMarkdown(wb, prefix, phaseDepsMap)
+const csv = renderCsv(wb, prefix, phaseDepsMap)
+fs.writeFileSync(mdPath, md, 'utf8')
+fs.writeFileSync(csvPath, csv, 'utf8')
+process.stdout.write(JSON.stringify({ exitCode: 0, markdownPath: mdPath, csvPath }) + '\n')
+process.exit(0)