edsger 0.77.0 → 0.79.0

package/dist/phases/pr-execution/context.d.ts

@@ -1,32 +0,0 @@
-import { type PullRequest } from '../../services/pull-requests.js';
-import type { IssueInfo } from '../../types/issues.js';
-import { type RepoForkInfo } from '../../utils/github-repo-info.js';
-export interface GitHubConfigInfo {
-    configured: boolean;
-    token?: string;
-    owner?: string;
-    repo?: string;
-    message?: string;
-}
-export interface ChangedFileInfo {
-    path: string;
-    change_type: 'added' | 'modified' | 'deleted' | 'renamed';
-}
-export interface PRExecutionContext {
-    issue: IssueInfo;
-    pullRequests: PullRequest[];
-    devBranchName: string;
-    devBranchHeadSha: string;
-    githubConfig: GitHubConfigInfo;
-    forkInfo: RepoForkInfo;
-    /** Resolved default branch (e.g. main, master). Used for checkout/diff. */
-    defaultBranch: string;
-    isIncrementalSync: boolean;
-    lastSyncedCommit: string | null;
-    diffStat: string;
-    changedFiles: ChangedFileInfo[];
-}
-/**
- * Fetch context for PR execution phase
- */
-export declare function fetchPRExecutionContext(issueId: string, verbose?: boolean): Promise<PRExecutionContext>;

package/dist/phases/pr-execution/context.js

@@ -1,246 +0,0 @@
-import { execFileSync } from 'child_process';
-import { getGitHubConfig } from '../../api/github.js';
-import { getIssue } from '../../api/issues/index.js';
-import { getPullRequests, } from '../../services/pull-requests.js';
-import { branchExists, remoteBranchExists, } from '../../utils/git-branch-manager.js';
-import { buildCredentialArgs } from '../../utils/git-push.js';
-import { getRepoForkInfo, resolveDefaultBranch, } from '../../utils/github-repo-info.js';
-import { logError, logInfo } from '../../utils/logger.js';
-/**
- * Get the dev branch name for an issue
- */
-function getDevBranchName(issueId) {
-    return `dev/${issueId}`;
-}
-/**
- * Get the HEAD SHA of a branch
- */
-function getBranchHeadSha(branchName) {
-    return execFileSync('git', ['rev-parse', branchName], {
-        encoding: 'utf-8',
-    }).trim();
-}
-/**
- * Get diff stat between two refs
- */
-function getDiffStat(baseRef, headRef) {
-    try {
-        return execFileSync('git', ['diff', '--stat', `${baseRef}...${headRef}`], {
-            encoding: 'utf-8',
-        }).trim();
-    }
-    catch {
-        return '';
-    }
-}
-/**
- * Map git diff --name-status letter to a readable change type
- */
-function parseGitStatus(status) {
-    switch (status[0]) {
-        case 'A':
-            return 'added';
-        case 'D':
-            return 'deleted';
-        case 'R':
-            return 'renamed';
-        default:
-            return 'modified';
-    }
-}
-/**
- * Get list of changed files with their change types between two refs
- */
-function getChangedFiles(baseRef, headRef) {
-    try {
-        const output = execFileSync('git', ['diff', '--name-status', `${baseRef}...${headRef}`], { encoding: 'utf-8' }).trim();
-        if (!output) {
-            return [];
-        }
-        return output
-            .split('\n')
-            .filter((line) => line.length > 0)
-            .map((line) => {
-            const [status, ...pathParts] = line.split('\t');
-            // For renames (R100\told\tnew), use the new path
-            const path = pathParts.length > 1 ? pathParts[1] : pathParts[0];
-            return { path, change_type: parseGitStatus(status) };
-        });
-    }
-    catch {
-        return [];
-    }
-}
-/**
- * Check if a commit exists in the ancestry of a given ref.
- * Returns false if the commit was orphaned (e.g. after a rebase).
- */
-function isAncestor(commitSha, ref) {
-    try {
-        execFileSync('git', ['merge-base', '--is-ancestor', commitSha, ref], {
-            encoding: 'utf-8',
-            stdio: 'pipe',
-        });
-        return true;
-    }
-    catch {
-        return false;
-    }
-}
-/**
- * Determine the common last_synced_commit from existing PRs
- * Returns null if any PR hasn't been synced yet (first run)
- */
-function getLastSyncedCommit(pullRequests) {
-    if (pullRequests.length === 0) {
-        return null;
-    }
-    const syncedCommits = pullRequests
-        .map((pr) => pr.last_synced_commit)
-        .filter((c) => c !== null);
-    // If not all PRs have been synced, this is a first-time execution
-    if (syncedCommits.length !== pullRequests.length) {
-        return null;
-    }
-    // All should be the same after a successful sync, use the first
-    return syncedCommits[0];
-}
-/**
- * Fetch context for PR execution phase
- */
-// eslint-disable-next-line complexity -- assembles context from many data sources with fallback handling
-export async function fetchPRExecutionContext(issueId, verbose) {
-    try {
-        if (verbose) {
-            logInfo(`Fetching PR execution context for issue: ${issueId}`);
-        }
-        const devBranchName = getDevBranchName(issueId);
-        // Fetch GitHub config and data in parallel (need token before remote branch check)
-        const [issue, pullRequests, githubConfig] = await Promise.all([
-            getIssue(issueId, verbose),
-            getPullRequests({ issueId, verbose }),
-            getGitHubConfig(issueId, verbose),
-        ]);
-        // Detect fork status + default branch (used by the fast-forward step below)
-        let forkInfo = { isFork: false };
-        if (githubConfig.token && githubConfig.owner && githubConfig.repo) {
-            try {
-                forkInfo = await getRepoForkInfo(githubConfig.token, githubConfig.owner, githubConfig.repo);
-                if (verbose) {
-                    logInfo(forkInfo.isFork
-                        ? `📌 Repository is a fork of ${forkInfo.upstream?.owner}/${forkInfo.upstream?.repo}`
-                        : `📌 Repository is not a fork`);
-                }
-            }
-            catch (error) {
-                if (verbose) {
-                    logError(`Failed to detect fork status: ${error instanceof Error ? error.message : String(error)}`);
-                }
-            }
-        }
-        // Resolve the repo's default branch (e.g. main, master, develop). Falls back
-        // through GitHub API → local symbolic-ref → literal 'main'.
-        const defaultBranch = resolveDefaultBranch(forkInfo.defaultBranch);
-        // Fetch latest remote refs and fast-forward the local default branch.
-        // Use execFileSync (not execSync) so defaultBranch — which can flow in from
-        // the GitHub API's default_branch field — is never passed through a shell.
-        try {
-            const credArgs = buildCredentialArgs(githubConfig.token);
-            execFileSync('git', [...credArgs, 'fetch', 'origin'], {
-                encoding: 'utf-8',
-                stdio: 'pipe',
-            });
-            execFileSync('git', ['checkout', defaultBranch], {
-                encoding: 'utf-8',
-                stdio: 'pipe',
-            });
-            execFileSync('git', ['merge', '--ff-only', `origin/${defaultBranch}`], {
-                encoding: 'utf-8',
-                stdio: 'pipe',
-            });
-            if (verbose) {
-                logInfo(`✅ Local ${defaultBranch} synced with origin/${defaultBranch}`);
-            }
-        }
-        catch (error) {
-            if (verbose) {
-                logInfo(`⚠️  Could not sync ${defaultBranch} with origin: ${error instanceof Error ? error.message : String(error)}`);
-            }
-        }
-        // Verify dev branch exists
-        const localExists = branchExists(devBranchName);
-        const remoteExists = !localExists && remoteBranchExists(devBranchName, githubConfig.token);
-        if (!localExists && !remoteExists) {
-            throw new Error(`Development branch '${devBranchName}' does not exist. ` +
-                `The issue must have code on the dev branch before PR execution.`);
-        }
-        // If branch only exists on remote, fetch it (using credential helper)
-        if (!localExists && remoteExists) {
-            if (verbose) {
-                logInfo(`Fetching remote branch ${devBranchName}...`);
-            }
-            const credArgs = buildCredentialArgs(githubConfig.token);
-            execFileSync('git', [
-                ...credArgs,
-                'fetch',
-                'origin',
-                `${devBranchName}:refs/remotes/origin/${devBranchName}`,
-            ], { encoding: 'utf-8', stdio: 'pipe' });
-        }
-        if (pullRequests.length === 0) {
-            throw new Error('No PR plan found. Run the pr-splitting phase first to create a PR plan.');
-        }
-        if (!githubConfig.configured) {
-            throw new Error(`GitHub is not configured. ${githubConfig.message || 'Please configure GitHub integration.'}`);
-        }
-        // Determine sync mode
-        const devRef = localExists ? devBranchName : `origin/${devBranchName}`;
-        const devBranchHeadSha = getBranchHeadSha(devRef);
-        let lastSyncedCommit = getLastSyncedCommit(pullRequests);
-        let isIncrementalSync = lastSyncedCommit !== null;
-        // If the branch was rebased, lastSyncedCommit is no longer in the history.
-        // Treat this as a full re-execution so downstream logic uses the correct
-        // prompt and skips incremental file-assignment against an invalid base.
-        if (isIncrementalSync &&
-            lastSyncedCommit &&
-            !isAncestor(lastSyncedCommit, devRef)) {
-            if (verbose) {
-                logInfo(`⚠️  Last synced commit ${lastSyncedCommit} is no longer in branch history (likely rebased). Treating as full re-execution.`);
-            }
-            lastSyncedCommit = null;
-            isIncrementalSync = false;
-        }
-        // Get diff info: for incremental, diff from last sync; for first run, diff from default branch
-        const diffBase = isIncrementalSync && lastSyncedCommit ? lastSyncedCommit : defaultBranch;
-        const diffStat = getDiffStat(diffBase, devRef);
-        const changedFiles = getChangedFiles(diffBase, devRef);
-        if (verbose) {
-            logInfo(`✅ PR execution context fetched:`);
-            logInfo(`   Issue: ${issue.name}`);
-            logInfo(`   Dev Branch: ${devBranchName} (HEAD: ${devBranchHeadSha})`);
-            logInfo(`   PR Records: ${pullRequests.length}`);
-            logInfo(`   Mode: ${isIncrementalSync ? '🔄 Incremental sync' : '✨ First execution'}`);
-            logInfo(`   Diff Base: ${diffBase}`);
-            logInfo(`   Changed Files: ${changedFiles.length}`);
-            logInfo(`   GitHub: ${githubConfig.configured ? 'Configured' : 'Not configured'}`);
-        }
-        return {
-            issue,
-            pullRequests,
-            devBranchName,
-            devBranchHeadSha,
-            githubConfig,
-            forkInfo,
-            defaultBranch,
-            isIncrementalSync,
-            lastSyncedCommit,
-            diffStat,
-            changedFiles,
-        };
-    }
-    catch (error) {
-        const errorMessage = error instanceof Error ? error.message : String(error);
-        logError(`Failed to fetch PR execution context: ${errorMessage}`);
-        throw new Error(`Context fetch failed: ${errorMessage}`);
-    }
-}

package/dist/phases/pr-execution/file-assigner.d.ts

@@ -1,57 +0,0 @@
-/**
- * File assignment for incremental sync.
- *
- * When new changes on the dev branch include files not covered by any
- * existing PR's file list, this module asks an LLM to assign them to
- * the most appropriate PR, then updates the PR records in the database.
- */
-import { type PullRequest } from '../../services/pull-requests.js';
-import type { ChangedFileInfo } from './context.js';
-export interface FileAssignment {
-    file: string;
-    pr_id: string;
-}
-export interface FileAssignmentResult {
-    assignments: FileAssignment[];
-}
-/**
- * Find changed files that are not covered by any PR's file list.
- */
-export declare function findUnassignedFiles(changedFiles: ChangedFileInfo[], pullRequests: PullRequest[]): ChangedFileInfo[];
-/**
- * Update PR records in the database with newly assigned files.
- * Exported for testing via dependency injection of the updater function.
- */
-export declare function applyAssignments(assignments: FileAssignment[], unassignedFiles: ChangedFileInfo[], pullRequests: PullRequest[], updater?: (prId: string, updates: {
-    files: {
-        path: string;
-        change_type: string;
-    }[];
-}, verbose?: boolean) => Promise<unknown>, verbose?: boolean): Promise<number>;
-/**
- * Remove files that were deleted on the dev branch from PR file lists.
- * Returns the number of PRs updated.
- */
-export declare function removeDeletedFilesFromPRs(changedFiles: ChangedFileInfo[], pullRequests: PullRequest[], updater?: (prId: string, updates: {
-    files: {
-        path: string;
-        change_type: string;
-    }[];
-}, verbose?: boolean) => Promise<unknown>, verbose?: boolean): Promise<number>;
-/**
- * Remove files from PR file lists that are no longer in the current diff.
- * Used after rebase when the full diff against main may differ from what
- * was previously recorded in PR file lists.
- * Returns the number of PRs updated.
- */
-export declare function removeStaleFilesFromPRs(changedFiles: ChangedFileInfo[], pullRequests: PullRequest[], updater?: (prId: string, updates: {
-    files: {
-        path: string;
-        change_type: string;
-    }[];
-}, verbose?: boolean) => Promise<unknown>, verbose?: boolean): Promise<number>;
-/**
- * Detect unassigned files and use LLM to assign them to existing PRs.
- * Returns the number of files assigned, or 0 if none needed assignment.
- */
-export declare function assignNewFilesToPRs(changedFiles: ChangedFileInfo[], pullRequests: PullRequest[], verbose?: boolean): Promise<number>;

package/dist/phases/pr-execution/file-assigner.js

@@ -1,287 +0,0 @@
-/**
- * File assignment for incremental sync.
- *
- * When new changes on the dev branch include files not covered by any
- * existing PR's file list, this module asks an LLM to assign them to
- * the most appropriate PR, then updates the PR records in the database.
- */
-import { query } from '@anthropic-ai/claude-agent-sdk';
-import { DEFAULT_MODEL } from '../../constants.js';
-import { updatePullRequest, } from '../../services/pull-requests.js';
-import { extractJsonFromResponse } from '../../utils/json-extractor.js';
-import { logError, logInfo } from '../../utils/logger.js';
-/**
- * Find changed files that are not covered by any PR's file list.
- */
-export function findUnassignedFiles(changedFiles, pullRequests) {
-    const assignedPaths = new Set();
-    for (const pr of pullRequests) {
-        if (pr.files) {
-            for (const f of pr.files) {
-                assignedPaths.add(f.path);
-            }
-        }
-    }
-    return changedFiles.filter((f) => !assignedPaths.has(f.path));
-}
-/**
- * Build a prompt asking the LLM to assign unassigned files to PRs.
- */
-function buildAssignmentPrompt(unassignedFiles, pullRequests) {
-    const prList = pullRequests
-        .map((pr) => {
-        const files = pr.files
-            ? pr.files.map((f) => `    - ${f.path} (${f.change_type})`).join('\n')
-            : '    (no files)';
-        return `### PR ${pr.sequence}: ${pr.name} (id: ${pr.id})
-- Description: ${pr.description || 'No description'}
-- Files:
-${files}`;
-    })
-        .join('\n\n');
-    const fileList = unassignedFiles
-        .map((f) => `- ${f.path} (${f.change_type})`)
-        .join('\n');
-    return `# Assign New Files to Existing PRs
-
-The following files were changed on the dev branch but are not assigned to any existing PR.
-Assign each file to the most appropriate PR based on the PR's purpose and existing file list.
-
-## Unassigned Files (${unassignedFiles.length})
-${fileList}
-
-## Existing PRs
-${prList}
-
-## Instructions
-For each unassigned file, decide which PR it belongs to based on:
-1. Directory proximity (same package/module as existing PR files)
-2. Logical cohesion (related functionality)
-3. Dependency order (if file B imports from file A, they may belong together)
-
-Return ONLY a JSON object:
-
-\`\`\`json
-{
-  "assignments": [
-    {
-      "file": "path/to/file.ts",
-      "pr_id": "uuid-of-target-pr"
-    }
-  ]
-}
-\`\`\`
-
-Every file in the unassigned list MUST appear exactly once in the assignments array.
-Do NOT create new PRs — assign all files to existing PRs only.`;
-}
-/**
- * Call LLM to assign files, returning structured assignments.
- */
-async function callLLMForAssignment(prompt, verbose) {
-    let responseText = '';
-    function* userMessage() {
-        yield {
-            type: 'user',
-            message: { role: 'user', content: prompt },
-        };
-    }
-    try {
-        for await (const message of query({
-            prompt: userMessage(),
-            options: {
-                model: DEFAULT_MODEL,
-                maxTurns: 5,
-                permissionMode: 'bypassPermissions',
-            },
-        })) {
-            if (message.type === 'assistant' && message.message?.content) {
-                for (const item of message.message.content) {
-                    // eslint-disable-next-line max-depth
-                    if (item.type === 'text') {
-                        responseText += `${item.text}\n`;
-                    }
-                }
-            }
-        }
-    }
-    catch (error) {
-        if (verbose) {
-            logError(`LLM query failed during file assignment: ${error instanceof Error ? error.message : String(error)}`);
-        }
-        return null;
-    }
-    const parsed = extractJsonFromResponse(responseText, '"assignments"');
-    if (!parsed || !Array.isArray(parsed.assignments)) {
-        if (verbose) {
-            logError('Failed to parse file assignment response from LLM');
-        }
-        return null;
-    }
-    return parsed;
-}
-/**
- * Update PR records in the database with newly assigned files.
- * Exported for testing via dependency injection of the updater function.
- */
-export async function applyAssignments(assignments, unassignedFiles, pullRequests, updater = updatePullRequest, verbose) {
-    // Build a lookup from file path to its git change_type
-    const changeTypeByPath = new Map();
-    for (const f of unassignedFiles) {
-        changeTypeByPath.set(f.path, f.change_type);
-    }
-    // Group assignments by PR id
-    const byPR = new Map();
-    for (const a of assignments) {
-        const list = byPR.get(a.pr_id) || [];
-        list.push(a);
-        byPR.set(a.pr_id, list);
-    }
-    let updatedCount = 0;
-    for (const [prId, newFiles] of byPR) {
-        const pr = pullRequests.find((p) => p.id === prId);
-        if (!pr) {
-            if (verbose) {
-                logError(`Assignment references unknown PR id: ${prId}`);
-            }
-            continue;
-        }
-        const existingFiles = pr.files || [];
-        const existingPaths = new Set(existingFiles.map((f) => f.path));
-        const mergedFiles = [...existingFiles];
-        for (const nf of newFiles) {
-            if (!existingPaths.has(nf.file)) {
-                const changeType = changeTypeByPath.get(nf.file) || 'modified';
-                mergedFiles.push({ path: nf.file, change_type: changeType });
-            }
-        }
-        try {
-            await updater(prId, { files: mergedFiles }, verbose);
-            updatedCount++;
-            if (verbose) {
-                logInfo(`   Updated PR "${pr.name}": +${newFiles.length} files (${newFiles.map((f) => f.file).join(', ')})`);
-            }
-        }
-        catch (error) {
-            logError(`Failed to update PR ${prId}: ${error instanceof Error ? error.message : String(error)}`);
-        }
-    }
-    return updatedCount;
-}
-/**
- * Remove files that were deleted on the dev branch from PR file lists.
- * Returns the number of PRs updated.
- */
-export async function removeDeletedFilesFromPRs(changedFiles, pullRequests, updater = updatePullRequest, verbose) {
-    const deletedPaths = new Set(changedFiles.filter((f) => f.change_type === 'deleted').map((f) => f.path));
-    if (deletedPaths.size === 0) {
-        return 0;
-    }
-    let updatedCount = 0;
-    for (const pr of pullRequests) {
-        if (!pr.files || pr.files.length === 0) {
-            continue;
-        }
-        const filtered = pr.files.filter((f) => !deletedPaths.has(f.path));
-        if (filtered.length === pr.files.length) {
-            continue;
-        }
-        const removed = pr.files.length - filtered.length;
-        try {
-            await updater(pr.id, { files: filtered }, verbose);
-            updatedCount++;
-            if (verbose) {
-                logInfo(`   Cleaned PR "${pr.name}": removed ${removed} deleted file(s)`);
-            }
-        }
-        catch (error) {
-            logError(`Failed to clean PR ${pr.id}: ${error instanceof Error ? error.message : String(error)}`);
-        }
-    }
-    if (verbose && updatedCount > 0) {
-        logInfo(`🗑️  Cleaned deleted files from ${updatedCount} PR(s)`);
-    }
-    return updatedCount;
-}
-/**
- * Remove files from PR file lists that are no longer in the current diff.
- * Used after rebase when the full diff against main may differ from what
- * was previously recorded in PR file lists.
- * Returns the number of PRs updated.
- */
-export async function removeStaleFilesFromPRs(changedFiles, pullRequests, updater = updatePullRequest, verbose) {
-    const currentPaths = new Set(changedFiles.map((f) => f.path));
-    let updatedCount = 0;
-    for (const pr of pullRequests) {
-        if (!pr.files || pr.files.length === 0) {
-            continue;
-        }
-        const filtered = pr.files.filter((f) => currentPaths.has(f.path));
-        if (filtered.length === pr.files.length) {
-            continue;
-        }
-        const removed = pr.files.length - filtered.length;
-        try {
-            await updater(pr.id, { files: filtered }, verbose);
-            updatedCount++;
-            if (verbose) {
-                logInfo(`   Cleaned PR "${pr.name}": removed ${removed} stale file(s)`);
-            }
-        }
-        catch (error) {
-            logError(`Failed to clean PR ${pr.id}: ${error instanceof Error ? error.message : String(error)}`);
-        }
-    }
-    if (verbose && updatedCount > 0) {
-        logInfo(`🧹 Removed stale files from ${updatedCount} PR(s)`);
-    }
-    return updatedCount;
-}
-/**
- * Detect unassigned files and use LLM to assign them to existing PRs.
- * Returns the number of files assigned, or 0 if none needed assignment.
- */
-export async function assignNewFilesToPRs(changedFiles, pullRequests, verbose) {
-    const unassigned = findUnassignedFiles(changedFiles, pullRequests);
-    if (unassigned.length === 0) {
-        if (verbose) {
-            logInfo('All changed files are already assigned to PRs.');
-        }
-        return 0;
-    }
-    if (verbose) {
-        logInfo(`\n📂 ${unassigned.length} changed file(s) not assigned to any PR:`);
-        for (const f of unassigned) {
-            logInfo(`   - ${f.path} (${f.change_type})`);
-        }
-        logInfo('🤖 Asking LLM to assign them...');
-    }
-    const prompt = buildAssignmentPrompt(unassigned, pullRequests);
-    const result = await callLLMForAssignment(prompt, verbose);
-    if (!result) {
-        logError('LLM file assignment failed. Proceeding with existing file lists.');
-        return 0;
-    }
-    // Validate: every unassigned file should appear in assignments
-    const assignedFiles = new Set(result.assignments.map((a) => a.file));
-    const missing = unassigned.filter((f) => !assignedFiles.has(f.path));
-    if (missing.length > 0 && verbose) {
-        logInfo(`⚠️  LLM did not assign ${missing.length} file(s): ${missing.map((f) => f.path).join(', ')}`);
-    }
-    // Validate: all pr_ids reference existing PRs
-    const validPRIds = new Set(pullRequests.map((pr) => pr.id));
-    const validAssignments = result.assignments.filter((a) => {
-        if (!validPRIds.has(a.pr_id)) {
-            if (verbose) {
-                logError(`Assignment for ${a.file} references unknown PR: ${a.pr_id}`);
-            }
-            return false;
-        }
-        return true;
-    });
-    const updatedCount = await applyAssignments(validAssignments, unassigned, pullRequests, updatePullRequest, verbose);
-    if (verbose) {
-        logInfo(`✅ Assigned ${validAssignments.length} file(s) to ${updatedCount} PR(s)`);
-    }
-    return validAssignments.length;
-}

package/dist/phases/pr-execution/index.d.ts

@@ -1,22 +0,0 @@
-import { type EdsgerConfig } from '../../types/index.js';
-export interface PRExecutionOptions {
-    issueId: string;
-    verbose?: boolean;
-}
-/**
- * PR Execution Phase: Create git branches, push code, and generate compare URLs
- *
- * Prerequisites: pr-splitting phase must have been run and human-approved.
- * PR plan records must exist in the database.
- *
- * Supports incremental re-sync: if PR branches already exist and dev branch
- * has new changes, only the new changes are synced to existing branches.
- *
- * GitHub PRs are created manually by the user via compare URLs in the UI.
- */
-export declare const executeIssuePRs: (options: PRExecutionOptions, config: EdsgerConfig) => Promise<{
-    status: "success" | "error";
-    message?: string;
-    summary?: string;
-    [key: string]: any;
-}>;