@xulthekl/team-flow 0.30.0 → 0.32.1

package/tests/lib/solutions-inject.test.mjs

@@ -1,115 +0,0 @@
-// tests/lib/solutions-inject.test.mjs
-// Tests for scripts/lib/solutions-inject.mjs — compound injection mechanism
-
-import { describe, it, before, after, beforeEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, writeFileSync, existsSync, rmSync, mkdirSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { fileURLToPath } from 'node:url';
-import { dirname } from 'node:path';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const ROOT = join(__dirname, '..', '..');
-
-const { run: injectRun } = await import(join(ROOT, 'scripts', 'lib', 'solutions-inject.mjs'));
-
-function makeDir() {
-  return mkdtempSync(join(tmpdir(), 'tf-sol-inject-'));
-}
-
-function cleanup(dir) {
-  if (existsSync(dir)) rmSync(dir, { recursive: true, force: true });
-}
-
-const INDEX_HEADER = `# Solutions Index
-<!-- 每条一行，按 severity 降序，≤150 行硬上限 -->
-| date | phase | domain | type | severity | summary | file |
-|------|-------|--------|------|----------|---------|------|
-`;
-
-function writeIndex(dir, rows) {
-  mkdirSync(dir, { recursive: true });
-  writeFileSync(join(dir, 'INDEX.md'), INDEX_HEADER + rows.join('\n') + '\n', 'utf-8');
-}
-
-describe('solutions-inject', () => {
-  let dir;
-
-  beforeEach(() => { dir = makeDir(); });
-  after(() => { cleanup(dir); });
-
-  it('returns empty when no INDEX.md exists', () => {
-    const result = injectRun({ phase: 'prd', dir });
-    assert.deepEqual(result.entries, []);
-  });
-
-  it('filters by exact phase match', () => {
-    writeIndex(dir, [
-      '| 2026-07-01 | prd | auth | pitfall | high | PRD auth issue | prd/f1.md |',
-      '| 2026-07-01 | build | auth | pitfall | high | Build auth issue | build/f2.md |',
-    ]);
-    const result = injectRun({ phase: 'prd', dir });
-    assert.equal(result.entries.length, 1);
-    assert.equal(result.entries[0].summary, 'PRD auth issue');
-  });
-
-  it('includes cross-phase entries for any phase query', () => {
-    writeIndex(dir, [
-      '| 2026-07-01 | cross-phase | general | pattern | high | Global pattern | cross-phase/f1.md |',
-      '| 2026-07-01 | prd | auth | pitfall | high | PRD issue | prd/f2.md |',
-    ]);
-    const result = injectRun({ phase: 'build', dir });
-    // cross-phase matches for any phase
-    assert.equal(result.entries.length, 1);
-    assert.equal(result.entries[0].phase, 'cross-phase');
-  });
-
-  it('filters by domain (exact match)', () => {
-    writeIndex(dir, [
-      '| 2026-07-01 | prd | auth | pitfall | high | Auth issue | prd/f1.md |',
-      '| 2026-07-01 | prd | billing | pitfall | high | Billing issue | prd/f2.md |',
-    ]);
-    const result = injectRun({ phase: 'prd', domain: 'auth', dir });
-    assert.equal(result.entries.length, 1);
-    assert.equal(result.entries[0].domain, 'auth');
-  });
-
-  it('domain "general" matches any query', () => {
-    writeIndex(dir, [
-      '| 2026-07-01 | prd | general | pattern | high | General pattern | prd/f1.md |',
-    ]);
-    const result = injectRun({ phase: 'prd', domain: 'auth', dir });
-    assert.equal(result.entries.length, 1);
-  });
-
-  it('sorts by severity descending (high > medium > low)', () => {
-    writeIndex(dir, [
-      '| 2026-07-01 | prd | auth | pitfall | low | Low issue | prd/f1.md |',
-      '| 2026-07-01 | prd | auth | pitfall | high | High issue | prd/f2.md |',
-      '| 2026-07-01 | prd | auth | pitfall | medium | Medium issue | prd/f3.md |',
-    ]);
-    const result = injectRun({ phase: 'prd', dir });
-    assert.equal(result.entries[0].severity, 'high');
-    assert.equal(result.entries[1].severity, 'medium');
-    assert.equal(result.entries[2].severity, 'low');
-  });
-
-  it('limits to top-5 entries', () => {
-    const rows = [];
-    for (let i = 0; i < 10; i++) {
-      rows.push(`| 2026-07-0${(i % 9) + 1} | prd | auth | pitfall | high | Issue ${i} | prd/f${i}.md |`);
-    }
-    writeIndex(dir, rows);
-    const result = injectRun({ phase: 'prd', dir });
-    assert.equal(result.entries.length, 5, 'should cap at top-5');
-  });
-
-  it('returns empty for no matching entries', () => {
-    writeIndex(dir, [
-      '| 2026-07-01 | build | ci | pattern | high | Build issue | build/f1.md |',
-    ]);
-    const result = injectRun({ phase: 'prd', dir });
-    assert.deepEqual(result.entries, []);
-  });
-});

package/tests/lib/solutions-promote.test.mjs

@@ -1,200 +0,0 @@
-// tests/lib/solutions-promote.test.mjs
-// Tests for scripts/lib/solutions-promote.mjs — change→product promotion
-
-import { describe, it, before, after, beforeEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, writeFileSync, existsSync, rmSync, mkdirSync, readFileSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { fileURLToPath } from 'node:url';
-import { dirname } from 'node:path';
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const ROOT = join(__dirname, '..', '..');
-
-// NODE_ENV=test is set below before import — promote's CLI guard skips when test.
-process.env.NODE_ENV = 'test';
-const { run: promoteRun } = await import(join(ROOT, 'scripts', 'lib', 'solutions-promote.mjs'));
-
-function makeDir(prefix = '') {
-  return mkdtempSync(join(tmpdir(), `tf-sol-promote-${prefix}-`));
-}
-
-function cleanup(dir) {
-  if (existsSync(dir)) rmSync(dir, { recursive: true, force: true });
-}
-
-function writeLearnings(dir, content) {
-  writeFileSync(join(dir, 'learnings.md'), content, 'utf-8');
-}
-
-const INDEX_HEADER = `# Solutions Index
-| date | phase | domain | type | severity | summary | file |
-|------|-------|--------|------|----------|---------|------|
-`;
-
-function writeGlobalIndex(solDir, rows = []) {
-  mkdirSync(solDir, { recursive: true });
-  writeFileSync(join(solDir, 'INDEX.md'), INDEX_HEADER + rows.join('\n') + '\n', 'utf-8');
-}
-
-describe('solutions-promote', () => {
-  let changeDir, solDir;
-
-  beforeEach(() => {
-    changeDir = makeDir('change');
-    solDir = makeDir('solutions');
-  });
-
-  after(() => {
-    cleanup(changeDir);
-    cleanup(solDir);
-  });
-
-  it('returns zero when no learnings.md exists', () => {
-    const result = promoteRun({ changeDir, dir: solDir });
-    assert.equal(result.promoted, 0);
-    assert.equal(result.updated, 0);
-  });
-
-  it('promotes entry with severity=high and type=pitfall', () => {
-    writeLearnings(changeDir, `## Auth token refresh failure
----
-phase: build
-domain: auth
-type: pitfall
-severity: high
-date: 2026-07-15
----
-
-When OAuth token refresh fails, the system silently swallows the error.
-`);
-    const result = promoteRun({ changeDir, dir: solDir });
-    assert.equal(result.promoted, 1);
-    assert.ok(existsSync(join(solDir, 'build')), 'build phase dir should exist');
-  });
-
-  it('promotes entry with severity=medium and type=pattern', () => {
-    writeLearnings(changeDir, `## Retry with backoff pattern
----
-phase: build
-domain: networking
-type: pattern
-severity: medium
-date: 2026-07-15
----
-
-Use exponential backoff for API retry logic.
-`);
-    const result = promoteRun({ changeDir, dir: solDir });
-    assert.equal(result.promoted, 1);
-  });
-
-  it('skips entry with severity=low', () => {
-    writeLearnings(changeDir, `## Minor formatting issue
----
-phase: build
-domain: style
-type: pitfall
-severity: low
-date: 2026-07-15
----
-
-Some low severity issue.
-`);
-    const result = promoteRun({ changeDir, dir: solDir });
-    assert.equal(result.promoted, 0);
-  });
-
-  it('skips entry with type=insight (not pitfall/pattern)', () => {
-    writeLearnings(changeDir, `## Interesting observation
----
-phase: build
-domain: auth
-type: insight
-severity: high
-date: 2026-07-15
----
-
-An insight that should not be promoted.
-`);
-    const result = promoteRun({ changeDir, dir: solDir });
-    assert.equal(result.promoted, 0);
-  });
-
-  it('detects duplicate domain+type and marks as confirmed', () => {
-    writeGlobalIndex(solDir, [
-      '| 2026-07-01 | build | auth | pitfall | medium | Existing auth pitfall | build/existing.md |',
-    ]);
-    writeLearnings(changeDir, `## Another auth pitfall
----
-phase: build
-domain: auth
-type: pitfall
-severity: high
-date: 2026-07-15
----
-
-Duplicate domain+type with existing index entry.
-`);
-    const result = promoteRun({ changeDir, dir: solDir });
-    assert.equal(result.updated, 1, 'should detect duplicate and mark as confirmed');
-    assert.equal(result.promoted, 0, 'should not create new file for duplicate');
-  });
-
-  it('promotes multiple entries from single learnings.md', () => {
-    writeLearnings(changeDir, `## Issue one
----
-phase: spec
-domain: api
-type: pitfall
-severity: high
-date: 2026-07-15
----
-
-First pitfall.
-
-## Issue two
----
-phase: spec
-domain: db
-type: pattern
-severity: medium
-date: 2026-07-15
----
-
-Second pattern.
-
-## Issue three (low sev, skipped)
----
-phase: spec
-domain: style
-type: pitfall
-severity: low
-date: 2026-07-15
----
-
-Skipped due to low severity.
-`);
-    const result = promoteRun({ changeDir, dir: solDir });
-    assert.equal(result.promoted, 2, 'should promote 2 entries (high+medium, skip low)');
-  });
-
-  it('appends promoted entry to global INDEX.md', () => {
-    writeGlobalIndex(solDir);
-    writeLearnings(changeDir, `## DB migration ordering
----
-phase: build
-domain: db
-type: pitfall
-severity: high
-date: 2026-07-15
----
-
-Migration files must be ordered by timestamp.
-`);
-    promoteRun({ changeDir, dir: solDir });
-    const indexContent = readFileSync(join(solDir, 'INDEX.md'), 'utf-8');
-    assert.match(indexContent, /DB migration ordering/);
-  });
-});

package/tests/lib/spec-paths.test.mjs

@@ -1,80 +0,0 @@
-import { describe, it, before, after } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, mkdirSync, writeFileSync, rmSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-
-let mod;
-let tempDir;
-
-describe('spec-paths', () => {
-  before(async () => {
-    tempDir = mkdtempSync(join(tmpdir(), 'tf-spec-paths-'));
-    mod = await import(join(process.cwd(), 'scripts/lib/spec-paths.mjs'));
-  });
-
-  after(() => {
-    if (tempDir) rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('finds canonical specs in sorted order', () => {
-    const dir = mkdtempSync(join(tempDir, 'canonical-'));
-    mkdirSync(join(dir, 'specs', 'z-auth'), { recursive: true });
-    mkdirSync(join(dir, 'specs', 'a-ui'), { recursive: true });
-    writeFileSync(join(dir, 'specs', 'z-auth', 'spec.md'), 'z');
-    writeFileSync(join(dir, 'specs', 'a-ui', 'spec.md'), 'a');
-
-    const files = mod.findCanonicalSpecFiles(dir).map(f => f.replace(dir + '/', ''));
-    assert.deepEqual(files, ['specs/a-ui/spec.md', 'specs/z-auth/spec.md']);
-  });
-
-  it('ignores nested spec.md files when finding canonical specs', () => {
-    const dir = mkdtempSync(join(tempDir, 'nested-'));
-    mkdirSync(join(dir, 'specs', 'ui-theme', 'nested'), { recursive: true });
-    writeFileSync(join(dir, 'specs', 'ui-theme', 'nested', 'spec.md'), 'nested');
-
-    assert.deepEqual(mod.findCanonicalSpecFiles(dir), []);
-
-    const result = mod.validateSpecPathLayout(dir, { requireSpecs: true });
-    assert.equal(result.pass, false);
-    assert.deepEqual(result.specFiles, []);
-    assert.ok(result.failures.some(f => f.includes('No canonical spec files found')));
-  });
-
-  it('reports flat capability specs with expected canonical path', () => {
-    const dir = mkdtempSync(join(tempDir, 'flat-'));
-    mkdirSync(join(dir, 'specs'), { recursive: true });
-    writeFileSync(join(dir, 'specs', 'ui-theme.md'), 'flat');
-
-    const invalid = mod.findInvalidSpecFiles(dir);
-    assert.deepEqual(invalid.map(i => i.path), ['specs/ui-theme.md']);
-    assert.deepEqual(invalid.map(i => i.expected), ['specs/ui-theme/spec.md']);
-  });
-
-  it('reports root specs/spec.md as invalid', () => {
-    const dir = mkdtempSync(join(tempDir, 'root-spec-'));
-    mkdirSync(join(dir, 'specs'), { recursive: true });
-    writeFileSync(join(dir, 'specs', 'spec.md'), 'root');
-
-    const invalid = mod.findInvalidSpecFiles(dir);
-    assert.deepEqual(invalid, [{ path: 'specs/spec.md', expected: 'specs/<capability>/spec.md' }]);
-  });
-
-  it('does not reject README.md inside specs', () => {
-    const dir = mkdtempSync(join(tempDir, 'readme-'));
-    mkdirSync(join(dir, 'specs'), { recursive: true });
-    writeFileSync(join(dir, 'specs', 'README.md'), 'notes');
-
-    assert.deepEqual(mod.findInvalidSpecFiles(dir), []);
-  });
-
-  it('fails layout validation when canonical specs are required but missing', () => {
-    const dir = mkdtempSync(join(tempDir, 'missing-'));
-    mkdirSync(join(dir, 'specs'), { recursive: true });
-
-    const result = mod.validateSpecPathLayout(dir, { requireSpecs: true });
-    assert.equal(result.pass, false);
-    assert.deepEqual(result.specFiles, []);
-    assert.ok(result.failures.some(f => f.includes('No canonical spec files found')));
-  });
-});

package/tests/lib/state-loader.test.mjs

@@ -1,251 +0,0 @@
-// tests/lib/state-loader.test.mjs
-// Tests for scripts/lib/state-loader.mjs
-import { describe, it, before, after } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, writeFileSync, readFileSync, rmSync, existsSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-
-let tempDir;
-
-describe('state-loader: readState()', () => {
-  let stateLoader;
-
-  before(async () => {
-    tempDir = mkdtempSync(join(tmpdir(), 'tf-state-test-'));
-    const modulePath = join(process.cwd(), 'scripts/lib/state-loader.mjs');
-    stateLoader = await import(modulePath);
-  });
-
-  after(() => {
-    if (tempDir) rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('returns defaults when no state file exists', () => {
-    const state = stateLoader.readState(tempDir);
-    assert.equal(state.state, 'exploring');
-    assert.equal(state.workflow, 'auto');
-    assert.equal(state.batches_completed, 0);
-    assert.equal(state.execution_mode, null);
-    assert.equal(state.execution_plan_hash, null);
-    assert.equal(state.execution_plan_revision, null);
-  });
-
-  it('infers change_name from directory when no state file', () => {
-    const changeDir = join(tempDir, 'my-feature');
-    const state = stateLoader.readState(changeDir);
-    assert.equal(state.change_name, 'my-feature');
-  });
-
-  it('parses existing state file correctly', () => {
-    const stateYaml = [
-      'state: executing',
-      'workflow: full',
-      'artifacts_hash: sha256:abc123',
-      'batches_completed: 3',
-      'test_result: pass',
-      'change_name: export-csv',
-      'dp_0_confirmed: true',
-      'dp_1_result: confirmed: add csv export',
-    ].join('\n');
-
-    writeFileSync(join(tempDir, '.team-flow.yaml'), stateYaml);
-
-    const state = stateLoader.readState(tempDir);
-
-    assert.equal(state.state, 'executing');
-    assert.equal(state.workflow, 'full');
-    assert.equal(state.artifacts_hash, 'sha256:abc123');
-    assert.equal(state.batches_completed, 3);
-    assert.equal(state.test_result, 'pass');
-    assert.equal(state.change_name, 'export-csv');
-    assert.equal(state.dp_0_confirmed, 'true');
-    assert.equal(state.dp_1_result, 'confirmed: add csv export');
-  });
-
-  it('handles null values in YAML', () => {
-    const stateYaml = [
-      'state: exploring',
-      'artifacts_hash: null',
-      'contract_hash: null',
-      'dp_0_timestamp: null',
-    ].join('\n');
-
-    writeFileSync(join(tempDir, '.team-flow.yaml'), stateYaml);
-
-    const state = stateLoader.readState(tempDir);
-
-    assert.equal(state.artifacts_hash, null);
-    assert.equal(state.contract_hash, null);
-    assert.equal(state.dp_0_timestamp, null);
-  });
-
-  it('ignores comments and empty lines in YAML', () => {
-    const stateYaml = [
-      '# This is a comment',
-      '',
-      'state: specifying',
-      '  ',
-      '# Another comment',
-      'workflow: tweak',
-    ].join('\n');
-
-    writeFileSync(join(tempDir, '.team-flow.yaml'), stateYaml);
-
-    const state = stateLoader.readState(tempDir);
-
-    assert.equal(state.state, 'specifying');
-    assert.equal(state.workflow, 'tweak');
-  });
-
-  it('merges partial state file with defaults', () => {
-    const stateYaml = 'state: approved-for-build\nworkflow: hotfix';
-
-    writeFileSync(join(tempDir, '.team-flow.yaml'), stateYaml);
-
-    const state = stateLoader.readState(tempDir);
-    assert.equal(state.state, 'approved-for-build');
-    assert.equal(state.workflow, 'hotfix');
-    // Unspecified fields should use defaults
-    assert.equal(state.batches_completed, 0);
-    assert.equal(state.test_result, null);
-  });
-});
-
-describe('state-loader: writeState()', () => {
-  let stateLoader;
-
-  before(async () => {
-    tempDir = mkdtempSync(join(tmpdir(), 'tf-state-write-'));
-    const modulePath = join(process.cwd(), 'scripts/lib/state-loader.mjs');
-    stateLoader = await import(modulePath);
-  });
-
-  after(() => {
-    if (tempDir) rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('writes state file with correct structure', () => {
-    const state = {
-      state: 'executing',
-      workflow: 'full',
-      artifacts_hash: 'sha256:def456',
-      contract_hash: null,
-      batches_completed: 2,
-      test_result: null,
-      change_name: 'test-change',
-      dp_0_confirmed: 'true',
-    };
-
-    stateLoader.writeState(tempDir, state);
-
-    const filePath = join(tempDir, '.team-flow.yaml');
-    assert.ok(existsSync(filePath));
-
-    const content = readFileSync(filePath, 'utf-8');
-    assert.ok(content.includes('state: executing'));
-    assert.ok(content.includes('workflow: full'));
-    assert.ok(content.includes('artifacts_hash: sha256:def456'));
-    assert.ok(content.includes('contract_hash: null'));
-    assert.ok(content.includes('batches_completed: 2'));
-    assert.ok(content.includes('execution_plan_hash: null'));
-    assert.ok(content.includes('execution_plan_revision: null'));
-    assert.ok(content.includes('change_name: test-change'));
-    assert.ok(content.includes('dp_0_confirmed: true'));
-  });
-
-  it('writes all DP fields (0-7)', () => {
-    const state = {
-      state: 'closing',
-      dp_0_decisions: 'scope: csv export only, tech: nestjs',
-      dp_0_result: 'confirmed: scope ok',
-      dp_0_confirmed: 'true',
-      dp_0_timestamp: '2026-07-01T08:00:00Z',
-      dp_1_result: 'confirmed: csv export',
-      dp_2_result: 'approved: artifacts ok',
-      dp_3_result: 'contract signed',
-      dp_4_result: 'execution started',
-      dp_5_result: null,
-      dp_6_result: null,
-      dp_7_result: null,
-    };
-
-    stateLoader.writeState(tempDir, state);
-
-    const content = readFileSync(join(tempDir, '.team-flow.yaml'), 'utf-8');
-    assert.ok(content.includes('dp_0_decisions: scope: csv export only, tech: nestjs'));
-    assert.ok(content.includes('dp_0_result: confirmed: scope ok'));
-    assert.ok(content.includes('dp_0_confirmed: true'));
-    assert.ok(content.includes('dp_1_result: confirmed: csv export'));
-    assert.ok(content.includes('dp_5_result: null'));
-  });
-
-  it('round-trips dp_0_result when updated through state loader', () => {
-    stateLoader.writeState(tempDir, {
-      state: 'exploring',
-      dp_0_decisions: 'scope confirmed',
-      dp_0_confirmed: 'true',
-    });
-
-    stateLoader.updateField(tempDir, 'dp_0_result', 'confirmed: manual');
-
-    const content = readFileSync(join(tempDir, '.team-flow.yaml'), 'utf-8');
-    const state = stateLoader.readState(tempDir);
-
-    assert.ok(content.includes('dp_0_result: confirmed: manual'));
-    assert.equal(state.dp_0_result, 'confirmed: manual');
-  });
-
-  it('round-trips state through write then read', () => {
-    const original = {
-      state: 'bridging',
-      workflow: 'hotfix',
-      artifacts_hash: 'sha256:xyz789',
-      batches_completed: 0,
-      dp_1_result: 'confirmed: refactor auth',
-      dp_1_timestamp: '2026-07-01T10:00:00Z',
-    };
-
-    stateLoader.writeState(tempDir, original);
-    const read = stateLoader.readState(tempDir);
-
-    assert.equal(read.state, original.state);
-    assert.equal(read.workflow, original.workflow);
-    assert.equal(read.artifacts_hash, original.artifacts_hash);
-    assert.equal(read.batches_completed, original.batches_completed);
-    assert.equal(read.dp_1_result, original.dp_1_result);
-    assert.equal(read.dp_1_timestamp, original.dp_1_timestamp);
-  });
-});
-
-describe('state-loader: updateField()', () => {
-  let stateLoader;
-
-  before(async () => {
-    tempDir = mkdtempSync(join(tmpdir(), 'tf-state-update-'));
-    const modulePath = join(process.cwd(), 'scripts/lib/state-loader.mjs');
-    stateLoader = await import(modulePath);
-  });
-
-  after(() => {
-    if (tempDir) rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  it('updates a single field without affecting others', () => {
-    // Initialize with some state
-    stateLoader.writeState(tempDir, {
-      state: 'exploring',
-      workflow: 'full',
-      batches_completed: 0,
-    });
-
-    stateLoader.updateField(tempDir, 'state', 'specifying');
-    stateLoader.updateField(tempDir, 'batches_completed', 3);
-
-    const state = stateLoader.readState(tempDir);
-    assert.equal(state.state, 'specifying');
-    assert.equal(state.batches_completed, 3);
-    // workflow should still be 'full'
-    assert.equal(state.workflow, 'full');
-  });
-});