edsger 0.77.0 → 0.79.0

package/dist/phases/code-review/context.d.ts

@@ -1,96 +0,0 @@
-/**
- * Context fetcher for code review phase
- * Fetches GitHub PR data including files, diffs, and commits for review
- */
-import { Octokit } from '@octokit/rest';
-export interface PRFile {
-    filename: string;
-    status: string;
-    additions: number;
-    deletions: number;
-    changes: number;
-    patch?: string;
-    blob_url: string;
-}
-export interface PRCommit {
-    sha: string;
-    commit: {
-        message: string;
-        author: {
-            name: string;
-            date: string;
-        };
-    };
-}
-export interface PRData {
-    number: number;
-    title: string;
-    body: string | null;
-    state: string;
-    head: {
-        ref: string;
-        sha: string;
-    };
-    base: {
-        ref: string;
-        sha: string;
-    };
-    user: {
-        login: string;
-    };
-}
-export interface CodeReviewContext {
-    issueId: string;
-    issueName: string;
-    issueDescription: string | null;
-    pullRequestUrl: string;
-    pullRequestNumber: number;
-    owner: string;
-    repo: string;
-    prData: PRData;
-    files: PRFile[];
-    commits: PRCommit[];
-    technicalDesign?: string;
-    userStories?: any[];
-    testCases?: any[];
-}
-/**
- * Extract owner, repo, and PR number from GitHub PR URL
- */
-export declare function parsePullRequestUrl(pullRequestUrl: string): {
-    owner: string;
-    repo: string;
-    prNumber: number;
-} | null;
-/**
- * Fetch PR details
- */
-export declare function fetchPRDetails(octokit: Octokit, owner: string, repo: string, prNumber: number, verbose?: boolean): Promise<PRData>;
-/**
- * Fetch PR files (changed files with diffs)
- */
-export declare function fetchPRFiles(octokit: Octokit, owner: string, repo: string, prNumber: number, verbose?: boolean): Promise<PRFile[]>;
-/**
- * Fetch PR commits
- */
-export declare function fetchPRCommits(octokit: Octokit, owner: string, repo: string, prNumber: number, verbose?: boolean): Promise<PRCommit[]>;
-/**
- * Fetch user stories via MCP
- */
-export declare function fetchUserStories(issueId: string, verbose?: boolean): Promise<any[]>;
-/**
- * Fetch test cases via MCP
- */
-export declare function fetchTestCases(issueId: string, verbose?: boolean): Promise<any[]>;
-/**
- * Fetch complete code review context
- * @param issueId - The issue ID
- * @param githubToken - GitHub token for API access
- * @param verbose - Enable verbose logging
- * @param pullRequestUrl - Optional PR URL (for multi-branch issues, use branch's PR URL)
- */
-export declare function fetchCodeReviewContext(issueId: string, githubToken: string, verbose?: boolean, pullRequestUrl?: string): Promise<CodeReviewContext>;
-/**
- * Format code review context for prompt
- */
-export declare function formatContextForPrompt(context: CodeReviewContext): string;

package/dist/phases/code-review/context.js

@@ -1,278 +0,0 @@
-/**
- * Context fetcher for code review phase
- * Fetches GitHub PR data including files, diffs, and commits for review
- */
-import { Octokit } from '@octokit/rest';
-import { getIssue } from '../../api/issues/get-issue.js';
-import { getMcpServerUrl, getMcpToken } from '../../auth/auth-store.js';
-import { logDebug } from '../../utils/logger.js';
-/**
- * Extract owner, repo, and PR number from GitHub PR URL
- */
-export function parsePullRequestUrl(pullRequestUrl) {
-    const match = pullRequestUrl.match(/github\.com\/([^\/]+)\/([^\/]+)\/pull\/(\d+)/);
-    if (!match) {
-        return null;
-    }
-    return {
-        owner: match[1],
-        repo: match[2],
-        prNumber: parseInt(match[3], 10),
-    };
-}
-/**
- * Fetch PR details
- */
-export async function fetchPRDetails(octokit, owner, repo, prNumber, verbose) {
-    logDebug(`Fetching PR details for ${owner}/${repo}#${prNumber}...`, verbose);
-    const { data } = await octokit.pulls.get({
-        owner,
-        repo,
-        pull_number: prNumber,
-    });
-    logDebug(`Fetched PR: ${data.title}`, verbose);
-    return data;
-}
-/**
- * Fetch PR files (changed files with diffs)
- */
-export async function fetchPRFiles(octokit, owner, repo, prNumber, verbose) {
-    logDebug(`Fetching PR files for ${owner}/${repo}#${prNumber}...`, verbose);
-    const { data: files } = await octokit.pulls.listFiles({
-        owner,
-        repo,
-        pull_number: prNumber,
-        per_page: 100,
-    });
-    logDebug(`Found ${files.length} changed files`, verbose);
-    return files;
-}
-/**
- * Fetch PR commits
- */
-export async function fetchPRCommits(octokit, owner, repo, prNumber, verbose) {
-    logDebug(`Fetching PR commits for ${owner}/${repo}#${prNumber}...`, verbose);
-    const { data: commits } = await octokit.pulls.listCommits({
-        owner,
-        repo,
-        pull_number: prNumber,
-        per_page: 100,
-    });
-    logDebug(`Found ${commits.length} commits`, verbose);
-    return commits;
-}
-/**
- * Fetch user stories via MCP
- */
-export async function fetchUserStories(issueId, verbose
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-) {
-    try {
-        logDebug(`Fetching user stories for ${issueId}...`, verbose);
-        const mcpServerUrl = getMcpServerUrl();
-        const mcpToken = getMcpToken();
-        const response = await fetch(`${mcpServerUrl}/mcp`, {
-            method: 'POST',
-            headers: {
-                'Content-Type': 'application/json',
-                Authorization: `Bearer ${mcpToken}`,
-            },
-            body: JSON.stringify({
-                jsonrpc: '2.0',
-                id: 1,
-                method: 'user_stories/list',
-                params: {
-                    issue_id: issueId,
-                },
-            }),
-        });
-        if (!response.ok) {
-            logDebug(`Could not fetch user stories: ${response.status}`, verbose);
-            return [];
-        }
-        const data = await response.json();
-        if (data.error || !data.result) {
-            logDebug('User stories not available', verbose);
-            return [];
-        }
-        return data.result.user_stories || [];
-    }
-    catch (error) {
-        logDebug(`Error fetching user stories: ${error}`, verbose);
-        return [];
-    }
-}
-/**
- * Fetch test cases via MCP
- */
-export async function fetchTestCases(issueId, verbose
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-) {
-    try {
-        logDebug(`Fetching test cases for ${issueId}...`, verbose);
-        const mcpServerUrl = getMcpServerUrl();
-        const mcpToken = getMcpToken();
-        const response = await fetch(`${mcpServerUrl}/mcp`, {
-            method: 'POST',
-            headers: {
-                'Content-Type': 'application/json',
-                Authorization: `Bearer ${mcpToken}`,
-            },
-            body: JSON.stringify({
-                jsonrpc: '2.0',
-                id: 1,
-                method: 'test_cases/list',
-                params: {
-                    issue_id: issueId,
-                },
-            }),
-        });
-        if (!response.ok) {
-            logDebug(`Could not fetch test cases: ${response.status}`, verbose);
-            return [];
-        }
-        const data = await response.json();
-        if (data.error || !data.result) {
-            logDebug('Test cases not available', verbose);
-            return [];
-        }
-        return data.result.test_cases || [];
-    }
-    catch (error) {
-        logDebug(`Error fetching test cases: ${error}`, verbose);
-        return [];
-    }
-}
-/**
- * Fetch complete code review context
- * @param issueId - The issue ID
- * @param githubToken - GitHub token for API access
- * @param verbose - Enable verbose logging
- * @param pullRequestUrl - Optional PR URL (for multi-branch issues, use branch's PR URL)
- */
-export async function fetchCodeReviewContext(issueId, githubToken, verbose, pullRequestUrl) {
-    // Fetch issue info using shared API
-    const issue = await getIssue(issueId, verbose);
-    // Use provided PR URL (from branch)
-    const prUrl = pullRequestUrl;
-    if (!prUrl) {
-        throw new Error(`No pull request URL found for issue ${issueId}. Cannot perform code review.`);
-    }
-    // Parse PR URL
-    const prInfo = parsePullRequestUrl(prUrl);
-    if (!prInfo) {
-        throw new Error(`Invalid pull request URL: ${prUrl}. Expected format: https://github.com/owner/repo/pull/123`);
-    }
-    const { owner, repo, prNumber } = prInfo;
-    // Initialize Octokit with GitHub token
-    const octokit = new Octokit({
-        auth: githubToken,
-    });
-    // Fetch PR data, files, commits, and additional context in parallel
-    const [prData, files, commits, userStories, testCases] = await Promise.all([
-        fetchPRDetails(octokit, owner, repo, prNumber, verbose),
-        fetchPRFiles(octokit, owner, repo, prNumber, verbose),
-        fetchPRCommits(octokit, owner, repo, prNumber, verbose),
-        fetchUserStories(issueId, verbose),
-        fetchTestCases(issueId, verbose),
-    ]);
-    logDebug(`Summary: ${files.length} files changed across ${commits.length} commits`, verbose);
-    return {
-        issueId,
-        issueName: issue.name,
-        issueDescription: issue.description ?? null,
-        pullRequestUrl: prUrl,
-        pullRequestNumber: prNumber,
-        owner,
-        repo,
-        prData,
-        files,
-        commits,
-        technicalDesign: issue.technical_design,
-        userStories,
-        testCases,
-    };
-}
-/**
- * Format code review context for prompt
- */
-export function formatContextForPrompt(context) {
-    const sections = [];
-    // Issue information
-    sections.push(`# Issue Information`);
-    sections.push(`**Issue ID**: ${context.issueId}`);
-    sections.push(`**Issue Name**: ${context.issueName}`);
-    if (context.issueDescription) {
-        sections.push(`**Description**: ${context.issueDescription}`);
-    }
-    sections.push(`**Pull Request**: ${context.pullRequestUrl} (#${context.pullRequestNumber})`);
-    sections.push('');
-    // PR details
-    sections.push(`## Pull Request Details`);
-    sections.push(`**Title**: ${context.prData.title}`);
-    sections.push(`**Author**: @${context.prData.user.login}`);
-    sections.push(`**Base Branch**: ${context.prData.base.ref}`);
-    sections.push(`**Head Branch**: ${context.prData.head.ref}`);
-    if (context.prData.body) {
-        sections.push(`**Description**:`);
-        sections.push(context.prData.body);
-    }
-    sections.push('');
-    // Commits
-    if (context.commits.length > 0) {
-        sections.push(`## Commits (${context.commits.length})`);
-        sections.push('');
-        context.commits.forEach((commit) => {
-            sections.push(`- **${commit.sha.substring(0, 7)}**: ${commit.commit.message}`);
-            sections.push(`  by ${commit.commit.author.name} on ${commit.commit.author.date}`);
-        });
-        sections.push('');
-    }
-    // Changed files
-    if (context.files.length > 0) {
-        sections.push(`## Changed Files (${context.files.length})`);
-        sections.push('');
-        context.files.forEach((file) => {
-            sections.push(`### ${file.filename}`);
-            sections.push(`**Status**: ${file.status}`);
-            sections.push(`**Changes**: +${file.additions} -${file.deletions}`);
-            if (file.patch) {
-                sections.push(`**Diff**:`);
-                sections.push('```diff');
-                sections.push(file.patch);
-                sections.push('```');
-            }
-            sections.push('');
-        });
-    }
-    // Technical design
-    if (context.technicalDesign) {
-        sections.push(`## Technical Design`);
-        sections.push('');
-        sections.push(context.technicalDesign);
-        sections.push('');
-    }
-    // User stories
-    if (context.userStories && context.userStories.length > 0) {
-        sections.push(`## User Stories`);
-        sections.push('');
-        context.userStories.forEach((story) => {
-            sections.push(`### ${story.title}`);
-            sections.push(story.description || '');
-            sections.push('');
-        });
-    }
-    // Test cases
-    if (context.testCases && context.testCases.length > 0) {
-        sections.push(`## Test Cases`);
-        sections.push('');
-        context.testCases.forEach((testCase) => {
-            sections.push(`### ${testCase.title}`);
-            if (testCase.description) {
-                sections.push(testCase.description);
-            }
-            sections.push('');
-        });
-    }
-    return sections.join('\n');
-}

package/dist/phases/code-review/diff-utils.d.ts

@@ -1,36 +0,0 @@
-/**
- * Shared diff utilities for mapping file line numbers to GitHub diff positions.
- * Used by both issue-linked code review and standalone PR review.
- */
-/**
- * Parse unified diff patch to build a mapping from file line numbers to diff positions
- *
- * GitHub API uses "position" parameter which is the line number in the unified diff:
- * - Position starts at 1 for the first line AFTER the @@ hunk header
- * - Increments for every line (context, additions, deletions)
- * - Continues through all hunks in the file
- *
- * Example unified diff:
- * ```
- * @@ -1,3 +1,4 @@        <- not counted in position
- *  context line          <- position 1, file line 1
- * -old line              <- position 2, (deleted line, no file line number)
- * +new line              <- position 3, file line 2
- * +another new          <- position 4, file line 3
- *  context               <- position 5, file line 4
- * ```
- *
- * Returns: Map<file line number, diff position>
- */
-export declare function buildLineToPositionMap(patch: string): Map<number, number>;
-/**
- * Find the diff position for a target line number, or the closest nearby position
- *
- * @param targetLine - The file line number we want to comment on
- * @param lineToPosition - Map of file line numbers to diff positions
- * @returns Object with position and actual line number, or null if not found
- */
-export declare function findClosestPosition(targetLine: number, lineToPosition: Map<number, number>): {
-    position: number;
-    actualLine: number;
-} | null;

package/dist/phases/code-review/diff-utils.js

@@ -1,100 +0,0 @@
-/**
- * Shared diff utilities for mapping file line numbers to GitHub diff positions.
- * Used by both issue-linked code review and standalone PR review.
- */
-/**
- * Parse unified diff patch to build a mapping from file line numbers to diff positions
- *
- * GitHub API uses "position" parameter which is the line number in the unified diff:
- * - Position starts at 1 for the first line AFTER the @@ hunk header
- * - Increments for every line (context, additions, deletions)
- * - Continues through all hunks in the file
- *
- * Example unified diff:
- * ```
- * @@ -1,3 +1,4 @@        <- not counted in position
- *  context line          <- position 1, file line 1
- * -old line              <- position 2, (deleted line, no file line number)
- * +new line              <- position 3, file line 2
- * +another new          <- position 4, file line 3
- *  context               <- position 5, file line 4
- * ```
- *
- * Returns: Map<file line number, diff position>
- */
-export function buildLineToPositionMap(patch) {
-    const lineToPosition = new Map();
-    const lines = patch.split('\n');
-    let position = 0; // Position in the diff (starts at 1 after first line after @@)
-    let currentNewLine = 0; // Current line number in the new file (RIGHT side)
-    for (const line of lines) {
-        // Parse hunk headers like @@ -1,7 +1,7 @@
-        // This extracts the starting line number for the new file (+side)
-        const hunkMatch = line.match(/^@@\s+-\d+(?:,\d+)?\s+\+(\d+)(?:,\d+)?\s+@@/);
-        if (hunkMatch) {
-            currentNewLine = parseInt(hunkMatch[1], 10);
-            // Don't increment position for the @@ line itself
-            continue;
-        }
-        // Every line after @@ increments the position counter
-        position++;
-        // Map line numbers based on line type
-        if (line.startsWith('+')) {
-            // Addition: this line exists in the new file at currentNewLine
-            lineToPosition.set(currentNewLine, position);
-            currentNewLine++;
-        }
-        else if (line.startsWith('-')) {
-            // Deletion: this line existed in old file, not in new file
-            // Position still increments, but currentNewLine does not
-        }
-        else if (!line.startsWith('\\')) {
-            // Context line (no prefix): exists in both old and new file
-            lineToPosition.set(currentNewLine, position);
-            currentNewLine++;
-        }
-        // Lines starting with '\' (like "") are metadata
-    }
-    return lineToPosition;
-}
-/**
- * Find the diff position for a target line number, or the closest nearby position
- *
- * @param targetLine - The file line number we want to comment on
- * @param lineToPosition - Map of file line numbers to diff positions
- * @returns Object with position and actual line number, or null if not found
- */
-export function findClosestPosition(targetLine, lineToPosition) {
-    // Check if exact line exists in the diff
-    const exactPosition = lineToPosition.get(targetLine);
-    if (exactPosition !== undefined) {
-        return {
-            position: exactPosition,
-            actualLine: targetLine,
-        };
-    }
-    // Try to find a nearby line within ±10 lines range
-    const range = 10;
-    for (let offset = 1; offset <= range; offset++) {
-        // Try below first (more likely to be relevant for review comments)
-        const lineBelow = targetLine + offset;
-        const belowPosition = lineToPosition.get(lineBelow);
-        if (belowPosition !== undefined) {
-            return {
-                position: belowPosition,
-                actualLine: lineBelow,
-            };
-        }
-        // Try above
-        const lineAbove = targetLine - offset;
-        const abovePosition = lineToPosition.get(lineAbove);
-        if (abovePosition !== undefined) {
-            return {
-                position: abovePosition,
-                actualLine: lineAbove,
-            };
-        }
-    }
-    // No valid position found within range
-    return null;
-}

package/dist/phases/code-review/index.d.ts

@@ -1,40 +0,0 @@
-/**
- * Code Review Analyzer
- * Reviews GitHub PR code and creates review comments with REQUEST_CHANGES
- */
-import { type ChecklistPhaseContext } from '../../services/checklist.js';
-import { type EdsgerConfig } from '../../types/index.js';
-export interface CodeReviewOptions {
-    issueId: string;
-    githubToken: string;
-    verbose?: boolean;
-    checklistContext?: ChecklistPhaseContext | null;
-}
-export interface ReviewComment {
-    path: string;
-    line: number;
-    side: 'LEFT' | 'RIGHT';
-    body: string;
-}
-export interface CodeReviewResult {
-    issueId: string;
-    status: 'success' | 'error';
-    message: string;
-    reviewId?: number;
-    reviewUrl?: string;
-    commentsCount?: number;
-    summary?: string;
-    data?: {
-        checklist_item_results?: {
-            checklist_item_id: string;
-            is_passed: boolean;
-            value?: unknown;
-            notes?: string;
-        }[];
-        [key: string]: any;
-    };
-}
-/**
- * Main code review function
- */
-export declare const reviewPullRequest: (options: CodeReviewOptions, config: EdsgerConfig, checklistContext?: ChecklistPhaseContext | null) => Promise<CodeReviewResult>;