edsger 0.45.0 → 0.46.0

package/dist/phases/pr-splitting/__tests__/import-dep-validator.test.js

@@ -1,331 +0,0 @@
-import assert from 'node:assert';
-import { describe, it } from 'node:test';
-import { autoFixPROrdering, getTransitiveDependencies, parseImportSpecifiers, resolveImportToChangedFile, } from '../import-dep-validator.js';
-// ============================================================
-// parseImportSpecifiers
-// ============================================================
-void describe('parseImportSpecifiers', () => {
-    void it('parses named imports', () => {
-        const result = parseImportSpecifiers(`import { foo } from './utils'`);
-        assert.deepStrictEqual(result, ['./utils']);
-    });
-    void it('parses default imports', () => {
-        const result = parseImportSpecifiers(`import foo from './utils'`);
-        assert.deepStrictEqual(result, ['./utils']);
-    });
-    void it('parses namespace imports', () => {
-        const result = parseImportSpecifiers(`import * as foo from './utils'`);
-        assert.deepStrictEqual(result, ['./utils']);
-    });
-    void it('parses side-effect imports', () => {
-        const result = parseImportSpecifiers(`import './polyfill'`);
-        assert.deepStrictEqual(result, ['./polyfill']);
-    });
-    void it('parses type-only imports', () => {
-        const result = parseImportSpecifiers(`import type { Foo } from './types'`);
-        assert.deepStrictEqual(result, ['./types']);
-    });
-    void it('parses re-exports', () => {
-        const result = parseImportSpecifiers(`export { foo } from './utils'`);
-        assert.deepStrictEqual(result, ['./utils']);
-    });
-    void it('parses star re-exports', () => {
-        const result = parseImportSpecifiers(`export * from './utils'`);
-        assert.deepStrictEqual(result, ['./utils']);
-    });
-    void it('parses dynamic imports', () => {
-        const result = parseImportSpecifiers(`const m = import('./lazy')`);
-        assert.deepStrictEqual(result, ['./lazy']);
-    });
-    void it('parses multiple imports', () => {
-        const source = `
-import { foo } from './utils'
-import Bar from '../components/Bar'
-import type { Baz } from './types'
-`;
-        const result = parseImportSpecifiers(source);
-        assert.deepStrictEqual(result, ['./utils', '../components/Bar', './types']);
-    });
-    void it('filters out non-relative imports', () => {
-        const source = `
-import { X } from 'lodash'
-import React from 'react'
-import { Y } from './local'
-import { Z } from '@scope/package'
-`;
-        const result = parseImportSpecifiers(source);
-        assert.deepStrictEqual(result, ['./local']);
-    });
-    void it('parses multiline imports', () => {
-        const source = `import {
-  foo,
-  bar,
-  baz
-} from './utils'`;
-        const result = parseImportSpecifiers(source);
-        assert.deepStrictEqual(result, ['./utils']);
-    });
-    void it('deduplicates specifiers', () => {
-        const source = `
-import { foo } from './utils'
-import { bar } from './utils'
-`;
-        const result = parseImportSpecifiers(source);
-        assert.deepStrictEqual(result, ['./utils']);
-    });
-    void it('handles .js extension in specifier', () => {
-        const result = parseImportSpecifiers(`import { X } from './utils.js'`);
-        assert.deepStrictEqual(result, ['./utils.js']);
-    });
-    void it('returns empty for empty source', () => {
-        assert.deepStrictEqual(parseImportSpecifiers(''), []);
-    });
-    void it('returns empty for non-relative only', () => {
-        const source = `import express from 'express'`;
-        assert.deepStrictEqual(parseImportSpecifiers(source), []);
-    });
-    void it('parses export default from', () => {
-        const result = parseImportSpecifiers(`export { default as X } from './module'`);
-        assert.deepStrictEqual(result, ['./module']);
-    });
-});
-// ============================================================
-// resolveImportToChangedFile
-// ============================================================
-void describe('resolveImportToChangedFile', () => {
-    void it('resolves exact match with extension', () => {
-        const changed = new Set(['src/utils.ts']);
-        const result = resolveImportToChangedFile('./utils.ts', 'src/app.ts', changed);
-        assert.strictEqual(result, 'src/utils.ts');
-    });
-    void it('infers .ts extension', () => {
-        const changed = new Set(['src/utils.ts']);
-        const result = resolveImportToChangedFile('./utils', 'src/app.ts', changed);
-        assert.strictEqual(result, 'src/utils.ts');
-    });
-    void it('infers .tsx extension', () => {
-        const changed = new Set(['src/Component.tsx']);
-        const result = resolveImportToChangedFile('./Component', 'src/app.ts', changed);
-        assert.strictEqual(result, 'src/Component.tsx');
-    });
-    void it('resolves index file', () => {
-        const changed = new Set(['src/types/index.ts']);
-        const result = resolveImportToChangedFile('./types', 'src/app.ts', changed);
-        assert.strictEqual(result, 'src/types/index.ts');
-    });
-    void it('resolves .js to .ts (ESM convention)', () => {
-        const changed = new Set(['src/utils.ts']);
-        const result = resolveImportToChangedFile('./utils.js', 'src/app.ts', changed);
-        assert.strictEqual(result, 'src/utils.ts');
-    });
-    void it('resolves parent directory traversal', () => {
-        const changed = new Set(['src/shared/helpers.ts']);
-        const result = resolveImportToChangedFile('../shared/helpers', 'src/components/Button.ts', changed);
-        assert.strictEqual(result, 'src/shared/helpers.ts');
-    });
-    void it('resolves double parent traversal', () => {
-        const changed = new Set(['src/utils.ts']);
-        const result = resolveImportToChangedFile('../../utils', 'src/features/explore/App.ts', changed);
-        assert.strictEqual(result, 'src/utils.ts');
-    });
-    void it('returns null for non-changed file', () => {
-        const changed = new Set(['src/other.ts']);
-        const result = resolveImportToChangedFile('./utils', 'src/app.ts', changed);
-        assert.strictEqual(result, null);
-    });
-    void it('returns null for external package import path', () => {
-        // This shouldn't happen since parseImportSpecifiers filters these,
-        // but test defensive behavior
-        const changed = new Set(['src/utils.ts']);
-        const result = resolveImportToChangedFile('./nonexistent', 'src/app.ts', changed);
-        assert.strictEqual(result, null);
-    });
-    void it('prefers .ts over .tsx when both exist', () => {
-        const changed = new Set(['src/utils.ts', 'src/utils.tsx']);
-        const result = resolveImportToChangedFile('./utils', 'src/app.ts', changed);
-        assert.strictEqual(result, 'src/utils.ts');
-    });
-});
-// ============================================================
-// getTransitiveDependencies
-// ============================================================
-void describe('getTransitiveDependencies', () => {
-    void it('returns direct dependency', () => {
-        const graph = new Map([
-            ['A', new Set(['B'])],
-            ['B', new Set()],
-        ]);
-        const result = getTransitiveDependencies('A', graph);
-        assert.deepStrictEqual(result, new Set(['B']));
-    });
-    void it('returns transitive dependencies', () => {
-        const graph = new Map([
-            ['A', new Set(['B'])],
-            ['B', new Set(['C'])],
-            ['C', new Set()],
-        ]);
-        const result = getTransitiveDependencies('A', graph);
-        assert.deepStrictEqual(result, new Set(['B', 'C']));
-    });
-    void it('handles diamond dependency', () => {
-        const graph = new Map([
-            ['A', new Set(['B', 'C'])],
-            ['B', new Set(['D'])],
-            ['C', new Set(['D'])],
-            ['D', new Set()],
-        ]);
-        const result = getTransitiveDependencies('A', graph);
-        assert.deepStrictEqual(result, new Set(['B', 'C', 'D']));
-    });
-    void it('handles cycles without infinite loop', () => {
-        const graph = new Map([
-            ['A', new Set(['B'])],
-            ['B', new Set(['A'])],
-        ]);
-        const result = getTransitiveDependencies('A', graph);
-        assert.deepStrictEqual(result, new Set(['B', 'A']));
-    });
-    void it('returns empty set for no dependencies', () => {
-        const graph = new Map([['A', new Set()]]);
-        const result = getTransitiveDependencies('A', graph);
-        assert.deepStrictEqual(result, new Set());
-    });
-    void it('returns empty set for unknown file', () => {
-        const graph = new Map();
-        const result = getTransitiveDependencies('unknown', graph);
-        assert.deepStrictEqual(result, new Set());
-    });
-});
-// ============================================================
-// autoFixPROrdering
-// ============================================================
-function makePR(sequence, name, files, dependsOn) {
-    return {
-        sequence,
-        name,
-        description: name,
-        branch_name: `pr/feat/${sequence}-${name.toLowerCase().replace(/\s/g, '-')}`,
-        depends_on_branch_name: dependsOn ?? null,
-        files: files.map((f) => ({ path: f, change_type: 'modified' })),
-    };
-}
-void describe('autoFixPROrdering', () => {
-    void it('returns unchanged when no violations', () => {
-        const prs = [
-            makePR(1, 'Foundation', ['src/utils.ts']),
-            makePR(2, 'Components', ['src/app.ts'], 'pr/feat/1-foundation'),
-        ];
-        // app.ts depends on utils.ts, and utils.ts is in PR 1 (earlier) — OK
-        const graph = new Map([
-            ['src/app.ts', new Set(['src/utils.ts'])],
-            ['src/utils.ts', new Set()],
-        ]);
-        const result = autoFixPROrdering(prs, graph);
-        assert.strictEqual(result.movedFiles.length, 0);
-        assert.strictEqual(result.pullRequests[0].files?.length, 1);
-        assert.strictEqual(result.pullRequests[1].files?.length, 1);
-    });
-    void it('moves dependency from later PR to earlier PR', () => {
-        const prs = [
-            makePR(1, 'Components', ['src/app.ts']),
-            makePR(2, 'Utils', ['src/utils.ts'], 'pr/feat/1-components'),
-        ];
-        // app.ts (PR 1) depends on utils.ts (PR 2) — violation!
-        const graph = new Map([
-            ['src/app.ts', new Set(['src/utils.ts'])],
-            ['src/utils.ts', new Set()],
-        ]);
-        const result = autoFixPROrdering(prs, graph);
-        assert.strictEqual(result.movedFiles.length, 1);
-        assert.strictEqual(result.movedFiles[0].file, 'src/utils.ts');
-        // utils.ts should now be in PR 1
-        const pr1Files = result.pullRequests[0].files?.map((f) => f.path) ?? [];
-        assert.ok(pr1Files.includes('src/utils.ts'));
-        assert.ok(pr1Files.includes('src/app.ts'));
-        // PR 2 should be empty
-        assert.strictEqual(result.pullRequests[1].files?.length, 0);
-    });
-    void it('handles transitive dependency moves', () => {
-        const prs = [
-            makePR(1, 'App', ['src/app.ts']),
-            makePR(2, 'Service', ['src/service.ts'], 'pr/feat/1-app'),
-            makePR(3, 'Utils', ['src/utils.ts'], 'pr/feat/2-service'),
-        ];
-        // app.ts → service.ts → utils.ts (chain across 3 PRs)
-        const graph = new Map([
-            ['src/app.ts', new Set(['src/service.ts'])],
-            ['src/service.ts', new Set(['src/utils.ts'])],
-            ['src/utils.ts', new Set()],
-        ]);
-        const result = autoFixPROrdering(prs, graph);
-        // Both service.ts and utils.ts should move to PR 1
-        const pr1Files = result.pullRequests[0].files?.map((f) => f.path) ?? [];
-        assert.ok(pr1Files.includes('src/app.ts'));
-        assert.ok(pr1Files.includes('src/service.ts'));
-        assert.ok(pr1Files.includes('src/utils.ts'));
-    });
-    void it('moves dep to earliest PR that needs it', () => {
-        const prs = [
-            makePR(1, 'PR1', ['src/a.ts']),
-            makePR(2, 'PR2', ['src/b.ts'], 'pr/feat/1-pr1'),
-            makePR(3, 'PR3', ['src/utils.ts'], 'pr/feat/2-pr2'),
-        ];
-        // Both a.ts (PR 1) and b.ts (PR 2) import utils.ts (PR 3)
-        const graph = new Map([
-            ['src/a.ts', new Set(['src/utils.ts'])],
-            ['src/b.ts', new Set(['src/utils.ts'])],
-            ['src/utils.ts', new Set()],
-        ]);
-        const result = autoFixPROrdering(prs, graph);
-        // utils.ts should move to PR 1 (earliest needer)
-        const pr1Files = result.pullRequests[0].files?.map((f) => f.path) ?? [];
-        assert.ok(pr1Files.includes('src/utils.ts'));
-    });
-    void it('handles no dependencies at all', () => {
-        const prs = [
-            makePR(1, 'PR1', ['src/a.ts']),
-            makePR(2, 'PR2', ['src/b.ts'], 'pr/feat/1-pr1'),
-        ];
-        const graph = new Map([
-            ['src/a.ts', new Set()],
-            ['src/b.ts', new Set()],
-        ]);
-        const result = autoFixPROrdering(prs, graph);
-        assert.strictEqual(result.movedFiles.length, 0);
-    });
-    void it('handles single PR with all files', () => {
-        const prs = [makePR(1, 'Everything', ['src/a.ts', 'src/b.ts', 'src/c.ts'])];
-        const graph = new Map([
-            ['src/a.ts', new Set(['src/b.ts'])],
-            ['src/b.ts', new Set(['src/c.ts'])],
-            ['src/c.ts', new Set()],
-        ]);
-        const result = autoFixPROrdering(prs, graph);
-        assert.strictEqual(result.movedFiles.length, 0);
-    });
-    void it('does not mutate original PR files arrays', () => {
-        const originalFiles = [{ path: 'src/app.ts', change_type: 'modified' }];
-        const prs = [
-            {
-                sequence: 1,
-                name: 'PR1',
-                description: 'PR1',
-                files: originalFiles,
-            },
-            {
-                sequence: 2,
-                name: 'PR2',
-                description: 'PR2',
-                files: [{ path: 'src/utils.ts', change_type: 'modified' }],
-            },
-        ];
-        const graph = new Map([
-            ['src/app.ts', new Set(['src/utils.ts'])],
-            ['src/utils.ts', new Set()],
-        ]);
-        autoFixPROrdering(prs, graph);
-        // Original files array should not be modified
-        assert.strictEqual(originalFiles.length, 1);
-        assert.strictEqual(originalFiles[0].path, 'src/app.ts');
-    });
-});

package/dist/phases/release-sync/__tests__/github.test.d.ts

@@ -1,9 +0,0 @@
-/**
- * Unit tests for smoke-test github helpers.
- *
- * Covers the pure functions that shape GitHub compare data into a
- * prompt-ready digest. Network-facing functions (fetchLatestTwoReleases,
- * fetchCompare) are exercised only indirectly — their output shape is
- * fed through buildDiffDigest / summariseStats here.
- */
-export {};

package/dist/phases/release-sync/__tests__/github.test.js

@@ -1,123 +0,0 @@
-/**
- * Unit tests for smoke-test github helpers.
- *
- * Covers the pure functions that shape GitHub compare data into a
- * prompt-ready digest. Network-facing functions (fetchLatestTwoReleases,
- * fetchCompare) are exercised only indirectly — their output shape is
- * fed through buildDiffDigest / summariseStats here.
- */
-import assert from 'node:assert';
-import { describe, it } from 'node:test';
-import { buildDiffDigest, summariseStats, } from '../github.js';
-function makeCompare(over = {}) {
-    return {
-        total_commits: 2,
-        commits: [
-            {
-                sha: '1111111aaaaaaaa',
-                commit: {
-                    message: 'feat: add checkout flow\n\nmore body',
-                    author: null,
-                },
-            },
-            {
-                sha: '2222222bbbbbbbb',
-                commit: { message: 'fix: null deref', author: { name: 'Ada' } },
-            },
-        ],
-        files: [
-            {
-                filename: 'src/checkout.ts',
-                status: 'modified',
-                additions: 10,
-                deletions: 2,
-                changes: 12,
-                patch: '@@ -1 +1 @@\n-old\n+new',
-            },
-            {
-                filename: 'src/util.ts',
-                status: 'added',
-                additions: 5,
-                deletions: 0,
-                changes: 5,
-            },
-        ],
-        ...over,
-    };
-}
-void describe('summariseStats', () => {
-    void it('sums additions, deletions, and file count', () => {
-        const stats = summariseStats(makeCompare());
-        assert.deepStrictEqual(stats, {
-            files_changed: 2,
-            additions: 15,
-            deletions: 2,
-            total_commits: 2,
-        });
-    });
-    void it('handles an empty diff', () => {
-        const stats = summariseStats({
-            total_commits: 0,
-            commits: [],
-            files: [],
-        });
-        assert.deepStrictEqual(stats, {
-            files_changed: 0,
-            additions: 0,
-            deletions: 0,
-            total_commits: 0,
-        });
-    });
-});
-void describe('buildDiffDigest', () => {
-    void it('includes commit subjects, file list, and patches', () => {
-        const digest = buildDiffDigest(makeCompare());
-        assert.match(digest, /Total commits: 2/);
-        assert.match(digest, /1111111 feat: add checkout flow/);
-        assert.match(digest, /2222222 fix: null deref/);
-        assert.match(digest, /modified src\/checkout\.ts \(\+10\/-2\)/);
-        assert.match(digest, /added src\/util\.ts \(\+5\/-0\)/);
-        assert.match(digest, /--- src\/checkout\.ts ---/);
-        assert.match(digest, /\+new/);
-    });
-    void it('truncates only the subject line of multi-line commit messages', () => {
-        const digest = buildDiffDigest(makeCompare());
-        // "more body" from the first commit's message body must not leak in.
-        assert.ok(!digest.includes('more body'));
-    });
-    void it('caps the patch budget and appends a truncation marker', () => {
-        const huge = 'x'.repeat(200_000);
-        const compare = makeCompare({
-            files: [
-                {
-                    filename: 'src/big.ts',
-                    status: 'modified',
-                    additions: 1,
-                    deletions: 0,
-                    changes: 1,
-                    patch: huge,
-                },
-            ],
-        });
-        const digest = buildDiffDigest(compare);
-        assert.match(digest, /\[truncated\]/);
-        // Even with a 200k patch, digest stays bounded by the internal budget.
-        assert.ok(digest.length < 200_000);
-    });
-    void it('skips files without patches in the patches section', () => {
-        const compare = makeCompare({
-            files: [
-                {
-                    filename: 'vendored.min.js',
-                    status: 'added',
-                    additions: 1,
-                    deletions: 0,
-                    changes: 1,
-                },
-            ],
-        });
-        const digest = buildDiffDigest(compare);
-        assert.ok(digest.includes('== Patches (truncated) =='));
-        assert.ok(!digest.includes('--- vendored.min.js ---'));
-    });
-});

package/dist/phases/release-sync/__tests__/snapshot.test.d.ts

@@ -1,8 +0,0 @@
-/**
- * Unit tests for snapshot-detection pure helpers.
- *
- * The network / SDK path (`detectSnapshotVersion`) is not covered here
- * — it's exercised end-to-end when the CLI runs — but the parser and
- * the plausibility check are critical and easy to cover.
- */
-export {};

package/dist/phases/release-sync/__tests__/snapshot.test.js

@@ -1,93 +0,0 @@
-/**
- * Unit tests for snapshot-detection pure helpers.
- *
- * The network / SDK path (`detectSnapshotVersion`) is not covered here
- * — it's exercised end-to-end when the CLI runs — but the parser and
- * the plausibility check are critical and easy to cover.
- */
-import assert from 'node:assert';
-import { describe, it } from 'node:test';
-import { buildSnapshotDetectionPrompt, isPlausibleSnapshotTag, parseSnapshotDetection, } from '../snapshot.js';
-void describe('parseSnapshotDetection', () => {
-    void it('parses a clean JSON object', () => {
-        const raw = '{"snapshot_tag":"v2.0.0","source":"package.json","reasoning":"next ver"}';
-        const parsed = parseSnapshotDetection(raw);
-        assert.strictEqual(parsed.snapshot_tag, 'v2.0.0');
-        assert.strictEqual(parsed.source, 'package.json');
-        assert.match(parsed.reasoning, /next ver/);
-    });
-    void it('strips ```json fences', () => {
-        const raw = '```json\n{"snapshot_tag":"v1.0.1","source":"VERSION","reasoning":"x"}\n```';
-        const parsed = parseSnapshotDetection(raw);
-        assert.strictEqual(parsed.snapshot_tag, 'v1.0.1');
-    });
-    void it('tolerates leading and trailing prose', () => {
-        const raw = `I checked package.json:
-{"snapshot_tag":"v3.0.0","source":"package.json","reasoning":"bumped"}
-Done.`;
-        const parsed = parseSnapshotDetection(raw);
-        assert.strictEqual(parsed.snapshot_tag, 'v3.0.0');
-    });
-    void it('returns null snapshot_tag when model reports no snapshot', () => {
-        const raw = '{"snapshot_tag": null, "source": null, "reasoning": "version matches latest release"}';
-        const parsed = parseSnapshotDetection(raw);
-        assert.strictEqual(parsed.snapshot_tag, null);
-        assert.strictEqual(parsed.source, null);
-    });
-    void it('treats empty string snapshot_tag as null', () => {
-        const raw = '{"snapshot_tag":"","source":null,"reasoning":"nothing found"}';
-        const parsed = parseSnapshotDetection(raw);
-        assert.strictEqual(parsed.snapshot_tag, null);
-    });
-    void it('falls back to balanced-brace extraction for surrounding prose', () => {
-        const raw = 'Analysis: { could be v1 } — but the real answer is ' +
-            '{"snapshot_tag":"v1.2.0","source":"CHANGELOG","reasoning":"unreleased section present"}';
-        const parsed = parseSnapshotDetection(raw);
-        assert.strictEqual(parsed.snapshot_tag, 'v1.2.0');
-        assert.strictEqual(parsed.source, 'CHANGELOG');
-    });
-    void it('throws when no JSON object is present', () => {
-        assert.throws(() => parseSnapshotDetection('no json at all'), /JSON/);
-    });
-    void it('supplies default reasoning when missing', () => {
-        const raw = '{"snapshot_tag":"v1","source":"package.json"}';
-        const parsed = parseSnapshotDetection(raw);
-        assert.strictEqual(parsed.snapshot_tag, 'v1');
-        assert.strictEqual(parsed.reasoning, '(no reasoning given)');
-    });
-});
-void describe('isPlausibleSnapshotTag', () => {
-    void it('accepts normal version tags', () => {
-        assert.strictEqual(isPlausibleSnapshotTag('v2.0.0', 'v1.9.0'), true);
-        assert.strictEqual(isPlausibleSnapshotTag('2.0.0-rc.1', '1.9.0'), true);
-        assert.strictEqual(isPlausibleSnapshotTag('v1.2.3-SNAPSHOT', 'v1.2.2'), true);
-        assert.strictEqual(isPlausibleSnapshotTag('release/2.0', 'release/1.9'), true);
-    });
-    void it('rejects tags identical to the latest release', () => {
-        assert.strictEqual(isPlausibleSnapshotTag('v1.0.0', 'v1.0.0'), false);
-    });
-    void it('rejects empty or overlong tags', () => {
-        assert.strictEqual(isPlausibleSnapshotTag('', 'v1'), false);
-        assert.strictEqual(isPlausibleSnapshotTag('x'.repeat(101), 'v1'), false);
-    });
-    void it('rejects whitespace and shell metacharacters', () => {
-        assert.strictEqual(isPlausibleSnapshotTag('v 2', 'v1'), false);
-        assert.strictEqual(isPlausibleSnapshotTag('v2;rm -rf', 'v1'), false);
-        assert.strictEqual(isPlausibleSnapshotTag('v2$PATH', 'v1'), false);
-    });
-    void it('rejects leading dot or dash, double dot, @{ reflog', () => {
-        assert.strictEqual(isPlausibleSnapshotTag('.v2', 'v1'), false);
-        assert.strictEqual(isPlausibleSnapshotTag('-v2', 'v1'), false);
-        assert.strictEqual(isPlausibleSnapshotTag('v2..rc', 'v1'), false);
-        assert.strictEqual(isPlausibleSnapshotTag('v2@{1}', 'v1'), false);
-    });
-});
-void describe('buildSnapshotDetectionPrompt', () => {
-    void it('injects the latest release tag into the prompt', () => {
-        const prompt = buildSnapshotDetectionPrompt('v1.4.2');
-        assert.match(prompt, /v1\.4\.2/);
-        // Must ask for JSON output
-        assert.match(prompt, /JSON/);
-        assert.match(prompt, /snapshot_tag/);
-    });
-});

package/dist/phases/smoke-test/__tests__/agent.test.d.ts

@@ -1,4 +0,0 @@
-/**
- * Unit tests for smoke-test agent response parsing.
- */
-export {};

package/dist/phases/smoke-test/__tests__/agent.test.js

@@ -1,85 +0,0 @@
-/**
- * Unit tests for smoke-test agent response parsing.
- */
-import assert from 'node:assert';
-import { describe, it } from 'node:test';
-import { findBalancedJsonObject } from '../../../utils/json-extract.js';
-import { extractJson } from '../agent.js';
-void describe('extractJson', () => {
-    void it('parses a plain JSON object', () => {
-        const raw = `{
-      "summary": "one thing changed",
-      "test_cases": [
-        { "name": "login still works", "description": "step 1", "is_critical": true }
-      ]
-    }`;
-        const parsed = extractJson(raw);
-        assert.strictEqual(parsed.summary, 'one thing changed');
-        assert.strictEqual(parsed.test_cases.length, 1);
-        assert.strictEqual(parsed.test_cases[0].is_critical, true);
-    });
-    void it('strips ```json fences', () => {
-        const raw = '```json\n{"summary":"x","test_cases":[{"name":"a","description":"b"}]}\n```';
-        const parsed = extractJson(raw);
-        assert.strictEqual(parsed.summary, 'x');
-        assert.strictEqual(parsed.test_cases[0].name, 'a');
-    });
-    void it('strips unlabeled fences', () => {
-        const raw = '```\n{"summary":"x","test_cases":[]}\n```';
-        const parsed = extractJson(raw);
-        assert.deepStrictEqual(parsed.test_cases, []);
-    });
-    void it('tolerates leading and trailing prose', () => {
-        const raw = `Here you go:
-{"summary":"x","test_cases":[{"name":"a","description":"b"}]}
-
-Hope that helps.`;
-        const parsed = extractJson(raw);
-        assert.strictEqual(parsed.summary, 'x');
-    });
-    void it('throws when test_cases is missing', () => {
-        assert.throws(() => extractJson('{"summary":"x"}'), /test_cases/);
-    });
-    void it('throws on invalid JSON', () => {
-        assert.throws(() => extractJson('not json'));
-    });
-    void it('picks the first balanced object when prose contains decoy braces', () => {
-        const raw = 'Note: { pseudo-json example } but the real answer is:\n' +
-            '{"summary":"x","test_cases":[{"name":"a","description":"b"}]}\n' +
-            'Let me know if you want changes.';
-        const parsed = extractJson(raw);
-        assert.strictEqual(parsed.summary, 'x');
-        assert.strictEqual(parsed.test_cases[0].name, 'a');
-    });
-    void it('preserves braces inside JSON string values', () => {
-        const raw = '{"summary":"changes to {pricing} and {checkout}","test_cases":[{"name":"n","description":"d"}]}';
-        const parsed = extractJson(raw);
-        assert.strictEqual(parsed.summary, 'changes to {pricing} and {checkout}');
-    });
-    void it('throws when test_cases is not an array', () => {
-        assert.throws(() => extractJson('{"summary":"x","test_cases":"nope"}'), /test_cases/);
-    });
-});
-void describe('findBalancedJsonObject', () => {
-    void it('returns null when there is no opening brace', () => {
-        assert.strictEqual(findBalancedJsonObject('no braces here'), null);
-    });
-    void it('returns the first balanced top-level object', () => {
-        assert.strictEqual(findBalancedJsonObject('prefix {"a": 1} and {"b": 2} suffix'), '{"a": 1}');
-    });
-    void it('handles nested braces', () => {
-        const text = 'xxx {"a": {"b": {"c": 1}}} yyy';
-        assert.strictEqual(findBalancedJsonObject(text), '{"a": {"b": {"c": 1}}}');
-    });
-    void it('ignores braces inside strings', () => {
-        const text = '{"msg": "this has { and } inside", "ok": true}';
-        assert.strictEqual(findBalancedJsonObject(text), text);
-    });
-    void it('handles escaped quotes inside strings', () => {
-        const text = '{"msg": "she said \\"hi\\" {not object}"}';
-        assert.strictEqual(findBalancedJsonObject(text), text);
-    });
-    void it('returns null on unbalanced input', () => {
-        assert.strictEqual(findBalancedJsonObject('{"a": 1'), null);
-    });
-});

package/dist/services/coaching/__tests__/coaching-agent.test.d.ts

@@ -1 +0,0 @@
-export {};