edsger 0.42.1 → 0.44.0

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

@@ -0,0 +1,123 @@
+/**
+ * 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

@@ -0,0 +1,8 @@
+/**
+ * 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

@@ -0,0 +1,93 @@
+/**
+ * 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/release-sync/github.d.ts

@@ -0,0 +1,54 @@
+/**
+ * GitHub release helpers for smoke-test generation, implemented on top of
+ * @octokit/rest so we share retry / rate-limit / User-Agent behavior with the
+ * rest of the packages/edsger codebase.
+ */
+export interface GithubRelease {
+    id: number;
+    tag_name: string;
+    name: string | null;
+    body: string | null;
+    html_url: string;
+    published_at: string | null;
+    draft: boolean;
+    prerelease: boolean;
+}
+export interface CompareFile {
+    filename: string;
+    status: string;
+    additions: number;
+    deletions: number;
+    changes: number;
+    patch?: string;
+}
+export interface CompareCommit {
+    sha: string;
+    commit: {
+        message: string;
+        author?: {
+            name?: string;
+        } | null;
+    };
+}
+export interface CompareResponse {
+    total_commits: number;
+    files: CompareFile[];
+    commits: CompareCommit[];
+}
+export interface ReleasePair {
+    latest: GithubRelease;
+    previous: GithubRelease | null;
+}
+export declare function fetchLatestTwoReleases(owner: string, repo: string, token: string): Promise<ReleasePair>;
+export declare function getDefaultBranchHead(owner: string, repo: string, token: string): Promise<{
+    branch: string;
+    sha: string;
+}>;
+export declare function fetchCompare(owner: string, repo: string, base: string, head: string, token: string): Promise<CompareResponse>;
+export declare function buildDiffDigest(compare: CompareResponse): string;
+export declare function summariseStats(compare: CompareResponse): {
+    files_changed: number;
+    additions: number;
+    deletions: number;
+    total_commits: number;
+};

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

@@ -0,0 +1,101 @@
+/**
+ * GitHub release helpers for smoke-test generation, implemented on top of
+ * @octokit/rest so we share retry / rate-limit / User-Agent behavior with the
+ * rest of the packages/edsger codebase.
+ */
+import { Octokit } from '@octokit/rest';
+const MAX_PATCH_CHARS = 60_000;
+const MAX_COMMITS_INCLUDED = 40;
+function octokit(token) {
+    return new Octokit({ auth: token, userAgent: 'edsger-cli' });
+}
+export async function fetchLatestTwoReleases(owner, repo, token) {
+    const gh = octokit(token);
+    const { data } = await gh.repos.listReleases({ owner, repo, per_page: 10 });
+    const real = data.filter((r) => !r.draft);
+    if (real.length === 0) {
+        throw new Error(`Repository ${owner}/${repo} has no releases yet. Publish one first.`);
+    }
+    return { latest: real[0], previous: real[1] ?? null };
+}
+export async function getDefaultBranchHead(owner, repo, token) {
+    const gh = octokit(token);
+    const { data: repoInfo } = await gh.repos.get({ owner, repo });
+    const branch = repoInfo.default_branch;
+    const { data: branchInfo } = await gh.repos.getBranch({
+        owner,
+        repo,
+        branch,
+    });
+    return { branch, sha: branchInfo.commit.sha };
+}
+export async function fetchCompare(owner, repo, base, head, token) {
+    const gh = octokit(token);
+    const { data } = await gh.repos.compareCommits({
+        owner,
+        repo,
+        base,
+        head,
+    });
+    // Octokit returns a richer shape; narrow to the subset we consume.
+    return {
+        total_commits: data.total_commits ?? 0,
+        files: (data.files ?? []).map((f) => ({
+            filename: f.filename,
+            status: f.status,
+            additions: f.additions ?? 0,
+            deletions: f.deletions ?? 0,
+            changes: f.changes ?? 0,
+            patch: f.patch,
+        })),
+        commits: (data.commits ?? []).map((c) => ({
+            sha: c.sha,
+            commit: {
+                message: c.commit?.message ?? '',
+                author: c.commit?.author
+                    ? { name: c.commit.author.name ?? undefined }
+                    : null,
+            },
+        })),
+    };
+}
+export function buildDiffDigest(compare) {
+    const lines = [];
+    lines.push(`Total commits: ${compare.total_commits}`);
+    lines.push('\n== Commits ==');
+    for (const c of compare.commits.slice(0, MAX_COMMITS_INCLUDED)) {
+        const subject = (c.commit.message || '').split('\n')[0].slice(0, 200);
+        lines.push(`- ${c.sha.slice(0, 7)} ${subject}`);
+    }
+    lines.push('\n== Files ==');
+    for (const f of compare.files) {
+        lines.push(`- ${f.status} ${f.filename} (+${f.additions}/-${f.deletions})`);
+    }
+    lines.push('\n== Patches (truncated) ==');
+    let budget = MAX_PATCH_CHARS;
+    for (const f of compare.files) {
+        if (budget <= 0 || !f.patch) {
+            continue;
+        }
+        const header = `\n--- ${f.filename} ---\n`;
+        const piece = header + f.patch;
+        if (piece.length > budget) {
+            lines.push(piece.slice(0, budget));
+            lines.push('\n...[truncated]');
+            budget = 0;
+        }
+        else {
+            lines.push(piece);
+            budget -= piece.length;
+        }
+    }
+    return lines.join('\n');
+}
+export function summariseStats(compare) {
+    return {
+        files_changed: compare.files.length,
+        additions: compare.files.reduce((s, f) => s + f.additions, 0),
+        deletions: compare.files.reduce((s, f) => s + f.deletions, 0),
+        total_commits: compare.total_commits,
+    };
+}

package/dist/phases/release-sync/index.d.ts

@@ -0,0 +1,24 @@
+/**
+ * Release sync: reconcile the `releases` table with GitHub releases for a
+ * product. Writes a `pending` row for each release we discover (latest +
+ * previous + any unreleased snapshot version). Does NOT generate smoke-test
+ * cases — that is handled per-release by `phases/smoke-test` once the user
+ * opts in from the release detail page.
+ */
+import { type EdsgerConfig } from '../../types/index.js';
+export interface ReleaseSyncOptions {
+    productId: string;
+    verbose?: boolean;
+}
+export interface ReleaseSyncResult {
+    status: 'success' | 'error';
+    productId: string;
+    summary: string;
+    /** Releases that were created or updated during this sync. */
+    syncedReleases?: {
+        id: string;
+        tag: string;
+        isSnapshot: boolean;
+    }[];
+}
+export declare function runReleaseSync(options: ReleaseSyncOptions, _config: EdsgerConfig): Promise<ReleaseSyncResult>;

package/dist/phases/release-sync/index.js

@@ -0,0 +1,147 @@
+/**
+ * Release sync: reconcile the `releases` table with GitHub releases for a
+ * product. Writes a `pending` row for each release we discover (latest +
+ * previous + any unreleased snapshot version). Does NOT generate smoke-test
+ * cases — that is handled per-release by `phases/smoke-test` once the user
+ * opts in from the release detail page.
+ */
+import { getGitHubConfigByProduct } from '../../api/github.js';
+import { getReleaseByTag, upsertRelease, } from '../../api/releases.js';
+import { logInfo, logSuccess, logWarning } from '../../utils/logger.js';
+import { cloneFeatureRepo, ensureWorkspaceDir, syncRepoToRef, } from '../../workspace/workspace-manager.js';
+import { fetchLatestTwoReleases, getDefaultBranchHead, } from './github.js';
+import { detectSnapshotVersion, isPlausibleSnapshotTag } from './snapshot.js';
+/**
+ * Upsert a release row as `pending` only when it doesn't already exist.
+ * Preserves `ready` / `failed` / `generating` status on existing rows so
+ * repeated syncs don't clobber smoke-test progress.
+ */
+async function ensureReleaseRow(args, verbose) {
+    const existing = await getReleaseByTag(args.productId, args.tag, verbose);
+    if (existing) {
+        return { release: existing, created: false };
+    }
+    const release = await upsertRelease({
+        product_id: args.productId,
+        tag: args.tag,
+        name: args.githubRelease?.name ?? null,
+        body: args.githubRelease?.body ?? null,
+        url: args.githubRelease?.html_url ?? null,
+        published_at: args.githubRelease?.published_at ?? null,
+        previous_tag: args.previousTag,
+        previous_published_at: args.previousPublishedAt,
+        status: 'pending',
+    }, verbose);
+    return { release, created: true };
+}
+// eslint-disable-next-line complexity -- orchestration across GitHub + DB + snapshot
+export async function runReleaseSync(options, _config) {
+    const { productId, verbose } = options;
+    if (verbose) {
+        logInfo(`Starting release sync for product: ${productId}`);
+    }
+    const gh = await getGitHubConfigByProduct(productId, verbose);
+    if (!gh.configured || !gh.token || !gh.owner || !gh.repo) {
+        return {
+            status: 'error',
+            productId,
+            summary: gh.message ||
+                'Product is not connected to a GitHub repository. Connect it in Product Settings.',
+        };
+    }
+    const releases = await fetchLatestTwoReleases(gh.owner, gh.repo, gh.token);
+    const { latest, previous } = releases;
+    if (verbose) {
+        logInfo(`Latest GitHub release: ${latest.tag_name}, previous: ${previous?.tag_name ?? '(none)'}`);
+    }
+    const synced = [];
+    // Always reflect the latest GitHub release.
+    const latestRow = await ensureReleaseRow({
+        productId,
+        tag: latest.tag_name,
+        previousTag: previous?.tag_name ?? null,
+        previousPublishedAt: previous?.published_at ?? null,
+        githubRelease: latest,
+    }, verbose);
+    synced.push({
+        id: latestRow.release.id,
+        tag: latestRow.release.tag,
+        isSnapshot: false,
+    });
+    if (latestRow.created) {
+        logInfo(`Synced release ${latest.tag_name}`);
+    }
+    // Reflect the previous GitHub release so the UI can show history, even
+    // if smoke-test was never run against it.
+    if (previous) {
+        const previousRow = await ensureReleaseRow({
+            productId,
+            tag: previous.tag_name,
+            previousTag: null,
+            previousPublishedAt: null,
+            githubRelease: previous,
+        }, verbose);
+        synced.push({
+            id: previousRow.release.id,
+            tag: previousRow.release.tag,
+            isSnapshot: false,
+        });
+        if (previousRow.created) {
+            logInfo(`Synced release ${previous.tag_name}`);
+        }
+    }
+    // Snapshot detection: clone the repo at the default branch tip and ask
+    // the agent whether there's an unreleased version ahead of `latest`.
+    let cwd;
+    try {
+        const workspaceRoot = ensureWorkspaceDir();
+        const { repoPath } = cloneFeatureRepo(workspaceRoot, `release-sync-${productId}`, gh.owner, gh.repo, gh.token);
+        cwd = repoPath;
+    }
+    catch (err) {
+        logWarning(`Could not clone repo for snapshot detection: ${err instanceof Error ? err.message : String(err)}`);
+    }
+    if (cwd) {
+        try {
+            const { branch } = await getDefaultBranchHead(gh.owner, gh.repo, gh.token);
+            syncRepoToRef(cwd, { branch }, gh.token);
+            const detection = await detectSnapshotVersion({
+                cwd,
+                latestReleaseTag: latest.tag_name,
+                config: _config,
+                verbose,
+            });
+            if (detection.snapshot_tag &&
+                isPlausibleSnapshotTag(detection.snapshot_tag, latest.tag_name)) {
+                const snapshotRow = await ensureReleaseRow({
+                    productId,
+                    tag: detection.snapshot_tag,
+                    previousTag: latest.tag_name,
+                    previousPublishedAt: latest.published_at ?? null,
+                    githubRelease: null,
+                }, verbose);
+                synced.push({
+                    id: snapshotRow.release.id,
+                    tag: snapshotRow.release.tag,
+                    isSnapshot: true,
+                });
+                if (snapshotRow.created) {
+                    logInfo(`Detected unreleased snapshot ${detection.snapshot_tag} (${detection.source ?? 'unknown source'})`);
+                }
+            }
+            else if (verbose) {
+                logInfo(`No snapshot version ahead of ${latest.tag_name}`);
+            }
+        }
+        catch (err) {
+            logWarning(`Snapshot detection skipped: ${err instanceof Error ? err.message : String(err)}`);
+        }
+    }
+    logSuccess(`Synced ${synced.length} release${synced.length === 1 ? '' : 's'}`);
+    return {
+        status: 'success',
+        productId,
+        summary: `Synced ${synced.length} release${synced.length === 1 ? '' : 's'} from GitHub`,
+        syncedReleases: synced,
+    };
+}

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

@@ -0,0 +1,27 @@
+/**
+ * Snapshot detection: after the latest GitHub release already has a
+ * smoke-test plan, inspect the cloned repo for a yet-to-be-released
+ * "snapshot" version (next package.json version, an [Unreleased] entry
+ * in CHANGELOG, an unreleased git tag, etc.) so we can prepare the
+ * next smoke test before anyone cuts the tag on GitHub.
+ */
+import { type EdsgerConfig } from '../../types/index.js';
+export interface SnapshotDetection {
+    snapshot_tag: string | null;
+    source: string | null;
+    reasoning: string;
+}
+export declare function buildSnapshotDetectionPrompt(latestReleaseTag: string): string;
+export declare function parseSnapshotDetection(raw: string): SnapshotDetection;
+/**
+ * Very loose sanity check on a proposed snapshot tag. We reject tags
+ * that are obviously the same as the latest release or that contain
+ * characters that would break downstream git / GitHub API calls.
+ */
+export declare function isPlausibleSnapshotTag(candidate: string, latestReleaseTag: string): boolean;
+export declare function detectSnapshotVersion(options: {
+    cwd: string;
+    latestReleaseTag: string;
+    config: EdsgerConfig;
+    verbose?: boolean;
+}): Promise<SnapshotDetection>;