edsger 0.76.0 → 0.78.0

package/dist/phases/pr-review/index.js

@@ -1,209 +0,0 @@
-/**
- * Standalone PR Review
- * Reviews a GitHub PR and posts review comments, without issue lifecycle dependency.
- */
-import { query } from '@anthropic-ai/claude-agent-sdk';
-import { Octokit } from '@octokit/rest';
-import { DEFAULT_MODEL } from '../../constants.js';
-import { logError, logInfo } from '../../utils/logger.js';
-import { buildLineToPositionMap, findClosestPosition, } from '../code-review/diff-utils.js';
-import { createPromptGenerator, extractTextFromContent, tryExtractResult, } from '../pr-shared/agent-utils.js';
-import { fetchStandalonePRContext, formatStandalonePRContextForPrompt, } from '../pr-shared/context.js';
-import { updatePullRequestStatus } from '../pr-shared/status.js';
-import { createStandaloneReviewSystemPrompt, createStandaloneReviewUserPrompt, } from './prompts.js';
-/**
- * Review a standalone PR and post comments to GitHub.
- */
-// eslint-disable-next-line complexity
-export async function reviewStandalonePR(options) {
-    const { pullRequestUrl, githubToken, verbose, prId, repositoryId } = options;
-    logInfo(`Starting standalone PR review: ${pullRequestUrl}`);
-    try {
-        // Fetch PR context
-        const context = await fetchStandalonePRContext(pullRequestUrl, githubToken, verbose);
-        if (context.files.length === 0) {
-            logInfo('No files to review in the PR.');
-            return {
-                status: 'success',
-                message: 'No files to review',
-                summary: 'No changed files found in the PR',
-            };
-        }
-        logInfo(`Found ${context.files.length} files to review across ${context.commits.length} commits`);
-        // Build prompts
-        const systemPrompt = createStandaloneReviewSystemPrompt();
-        const contextInfo = formatStandalonePRContextForPrompt(context);
-        const reviewPrompt = createStandaloneReviewUserPrompt(contextInfo);
-        let lastAssistantResponse = '';
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        let structuredReviewResult = null;
-        logInfo('Starting Claude analysis for code review...');
-        for await (const message of query({
-            prompt: createPromptGenerator(reviewPrompt),
-            options: {
-                systemPrompt: {
-                    type: 'preset',
-                    preset: 'claude_code',
-                    append: systemPrompt,
-                },
-                model: DEFAULT_MODEL,
-                maxTurns: 50,
-                permissionMode: 'bypassPermissions',
-            },
-        })) {
-            if (message.type === 'assistant') {
-                lastAssistantResponse += extractTextFromContent(message.message?.content ?? [], verbose);
-                continue;
-            }
-            if (message.type !== 'result') {
-                continue;
-            }
-            if (message.subtype === 'success') {
-                logInfo('Code review analysis completed, parsing results...');
-                const responseText = message.result || lastAssistantResponse;
-                structuredReviewResult = tryExtractResult(responseText, 'review_result') || {
-                    summary: 'Code review completed',
-                    comments: [],
-                    overall_assessment: 'COMMENT',
-                };
-            }
-            else {
-                logError(`Code review incomplete: ${message.subtype}`);
-                if (lastAssistantResponse) {
-                    structuredReviewResult = tryExtractResult(lastAssistantResponse, 'review_result');
-                }
-            }
-        }
-        if (!structuredReviewResult) {
-            return {
-                status: 'error',
-                message: 'Code review analysis failed or incomplete',
-            };
-        }
-        // Post review to GitHub
-        const { summary, comments, overall_assessment } = structuredReviewResult;
-        const octokit = new Octokit({ auth: githubToken });
-        if (!comments || comments.length === 0) {
-            logInfo('No issues found. PR looks good!');
-            const review = await octokit.pulls.createReview({
-                owner: context.owner,
-                repo: context.repo,
-                pull_number: context.pullRequestNumber,
-                event: 'COMMENT',
-                body: summary ||
-                    overall_assessment ||
-                    'Code review completed. No issues found.',
-            });
-            if (prId) {
-                await updatePullRequestStatus({
-                    prId,
-                    status: 'in_review',
-                    repositoryId,
-                });
-            }
-            return {
-                status: 'success',
-                message: 'Code review completed - no issues found',
-                reviewId: review.data.id,
-                reviewUrl: review.data.html_url,
-                commentsCount: 0,
-                summary: summary || 'No issues found',
-            };
-        }
-        logInfo(`Creating GitHub review with ${comments.length} comments...`);
-        // Build line-to-position maps for all files
-        const fileLineToPosition = new Map();
-        for (const file of context.files) {
-            if (file.patch) {
-                fileLineToPosition.set(file.filename, buildLineToPositionMap(file.patch));
-            }
-        }
-        const commitId = context.prData.head.sha;
-        // Map comments to diff positions
-        const reviewComments = [];
-        for (const comment of comments) {
-            const filePath = comment.file || comment.path;
-            const requestedLine = comment.line;
-            const lineToPosition = fileLineToPosition.get(filePath);
-            if (!lineToPosition) {
-                if (verbose) {
-                    logInfo(`Skipping comment for ${filePath}:${requestedLine} - file has no diff`);
-                }
-                continue;
-            }
-            const positionResult = findClosestPosition(requestedLine, lineToPosition);
-            if (positionResult === null) {
-                if (verbose) {
-                    logInfo(`Skipping comment for ${filePath}:${requestedLine} - line not in diff range`);
-                }
-                continue;
-            }
-            let commentBody = comment.comment || comment.body;
-            if (positionResult.actualLine !== requestedLine) {
-                commentBody = `**Note**: Comment originally for line ${requestedLine}, adjusted to line ${positionResult.actualLine} (nearest line in diff).\n\n${commentBody}`;
-            }
-            reviewComments.push({
-                path: filePath,
-                position: positionResult.position,
-                body: commentBody,
-            });
-        }
-        if (reviewComments.length === 0) {
-            logInfo('All comments were filtered out (invalid line numbers)');
-            const review = await octokit.pulls.createReview({
-                owner: context.owner,
-                repo: context.repo,
-                pull_number: context.pullRequestNumber,
-                event: 'COMMENT',
-                body: `${summary || 'Code review completed.'}\n\n**Note**: Some review comments could not be posted because they referenced lines not present in the diff.`,
-            });
-            if (prId) {
-                await updatePullRequestStatus({
-                    prId,
-                    status: 'in_review',
-                    repositoryId,
-                });
-            }
-            return {
-                status: 'success',
-                message: 'Code review completed - comments filtered',
-                reviewId: review.data.id,
-                reviewUrl: review.data.html_url,
-                commentsCount: 0,
-                summary: summary || 'Code review completed (comments filtered)',
-            };
-        }
-        logInfo(`Posting ${reviewComments.length} validated comments (${comments.length - reviewComments.length} filtered out)`);
-        const review = await octokit.pulls.createReview({
-            owner: context.owner,
-            repo: context.repo,
-            pull_number: context.pullRequestNumber,
-            commit_id: commitId,
-            event: 'COMMENT',
-            body: summary ||
-                overall_assessment ||
-                'Please address the following review comments.',
-            comments: reviewComments,
-        });
-        logInfo(`GitHub review created: ${review.data.html_url}`);
-        if (prId) {
-            await updatePullRequestStatus({ prId, status: 'in_review', repositoryId });
-        }
-        return {
-            status: 'success',
-            message: 'Code review completed and posted to GitHub',
-            reviewId: review.data.id,
-            reviewUrl: review.data.html_url,
-            commentsCount: reviewComments.length,
-            summary: summary || 'Code review completed',
-        };
-    }
-    catch (error) {
-        const errorMessage = error instanceof Error ? error.message : String(error);
-        logError(`PR review failed: ${errorMessage}`);
-        return {
-            status: 'error',
-            message: `PR review failed: ${errorMessage}`,
-        };
-    }
-}

package/dist/phases/pr-review/prompts.d.ts

@@ -1,5 +0,0 @@
-/**
- * Prompts for standalone PR review (no issue/design context).
- */
-export declare function createStandaloneReviewSystemPrompt(): string;
-export declare function createStandaloneReviewUserPrompt(contextInfo: string): string;

package/dist/phases/pr-review/prompts.js

@@ -1,87 +0,0 @@
-/**
- * Prompts for standalone PR review (no issue/design context).
- */
-export function createStandaloneReviewSystemPrompt() {
-    return `You are an expert code reviewer specializing in thorough, constructive code reviews. Your goal is to analyze pull request code and identify issues, potential bugs, code quality concerns, and areas for improvement.
-
-**Review Focus Areas**:
-1. **Code Quality**: Clean code principles, readability, maintainability
-2. **Best Practices**: Language-specific conventions, design patterns
-3. **Potential Bugs**: Logic errors, edge cases, error handling
-4. **Security**: Security vulnerabilities, input validation
-5. **Performance**: Performance issues, optimization opportunities
-6. **Architecture**: Design decisions, coupling, cohesion
-
-**Important Guidelines**:
-- Be thorough but focus on significant issues
-- Provide constructive, actionable feedback
-- Suggest specific improvements
-- Be respectful and professional
-- Focus on issues that truly matter, not nitpicks
-- If code looks good overall, provide positive feedback
-
-**CRITICAL - Result Format**:
-You MUST end your response with a JSON object containing the review results in this EXACT format:
-
-\`\`\`json
-{
-  "review_result": {
-    "summary": "Overall review summary and assessment",
-    "overall_assessment": "APPROVE | REQUEST_CHANGES | COMMENT",
-    "comments": [
-      {
-        "file": "path/to/file.ts",
-        "line": 42,
-        "comment": "Detailed comment about the issue and suggested fix"
-      }
-    ],
-    "issues_found": {
-      "critical": 0,
-      "major": 0,
-      "minor": 0
-    }
-  }
-}
-\`\`\`
-
-**Comment Guidelines**:
-- Each comment should reference a specific file and line number
-- Be specific about what the issue is
-- Provide a suggested fix or improvement
-
-**Assessment Options**:
-- **APPROVE**: Code looks good, no significant issues
-- **REQUEST_CHANGES**: Issues found that should be addressed
-- **COMMENT**: Minor suggestions or questions, not blocking`;
-}
-export function createStandaloneReviewUserPrompt(contextInfo) {
-    return `Review the following pull request code:
-
-${contextInfo}
-
-## Code Review Instructions
-
-Follow this systematic approach:
-
-1. **Analyze Each File**: For each changed file:
-   - Review the code changes in the diff
-   - Look for code quality issues
-   - Check for potential bugs or logic errors
-   - Verify error handling
-   - Consider security implications
-   - Evaluate performance considerations
-
-2. **Identify Issues**: Categorize by severity:
-   - **Critical**: Security vulnerabilities, data loss risks, major bugs
-   - **Major**: Logic errors, incorrect implementations, missing error handling
-   - **Minor**: Code quality improvements, style issues, optimization suggestions
-
-3. **Provide Actionable Feedback**: For each issue:
-   - Reference the specific file and line number
-   - Explain what the issue is
-   - Suggest how to fix it
-
-4. **Generate Review Result**: Create a structured JSON response with summary, assessment, and comments.
-
-Begin by analyzing the changed files and identifying any issues or improvements.`;
-}

package/dist/phases/pr-shared/agent-utils.d.ts

@@ -1,47 +0,0 @@
-/**
- * Shared utilities for Agent SDK phases.
- * Eliminates duplication of prompt generators, text extractors, and JSON parsers.
- */
-/**
- * Create a user message object for the Agent SDK prompt generator.
- */
-export declare function userMessage(content: string): {
-    type: string;
-    message: {
-        role: string;
-        content: string;
-    };
-};
-/**
- * Create an async generator that yields a single user prompt for Agent SDK query().
- */
-export declare function createPromptGenerator(prompt: string): AsyncGenerator<{
-    type: string;
-    message: {
-        role: string;
-        content: string;
-    };
-}>;
-/**
- * Extract text content from assistant message content array.
- *
- * When `verbose`, also surfaces tool_use / tool_result blocks via
- * logDebug so it's visible whether the agent is making MCP / file /
- * bash calls — without these, a long-running session looks frozen
- * between text emissions.
- */
-export declare function extractTextFromContent(content: any[], verbose?: boolean): string;
-/**
- * Try to parse a JSON result from agent response text.
- * Tries a custom fenceTag (e.g. ```screen_flow) first when provided, then
- * ```json, then falls back to raw JSON parsing. Returns the parsed object or
- * null on failure.
- */
-export declare function tryParseJsonFromResponse(responseText: string, fenceTag?: string): unknown | null;
-/**
- * Extract a specific keyed result from agent response.
- * e.g., tryExtractResult(text, 'review_result') extracts the review_result key.
- * The key is also tried as the fenced code-block tag so phases whose output
- * contract uses a custom fence (e.g. ```screen_flow) parse correctly.
- */
-export declare function tryExtractResult(responseText: string, key: string): unknown | null;

package/dist/phases/pr-shared/agent-utils.js

@@ -1,112 +0,0 @@
-/**
- * Shared utilities for Agent SDK phases.
- * Eliminates duplication of prompt generators, text extractors, and JSON parsers.
- */
-import { logDebug } from '../../utils/logger.js';
-/**
- * Create a user message object for the Agent SDK prompt generator.
- */
-export function userMessage(content) {
-    return {
-        type: 'user',
-        message: { role: 'user', content },
-    };
-}
-/**
- * Create an async generator that yields a single user prompt for Agent SDK query().
- */
-export async function* createPromptGenerator(prompt) {
-    yield userMessage(prompt);
-    await new Promise((res) => {
-        setTimeout(res, 10000);
-    });
-}
-/**
- * Extract text content from assistant message content array.
- *
- * When `verbose`, also surfaces tool_use / tool_result blocks via
- * logDebug so it's visible whether the agent is making MCP / file /
- * bash calls — without these, a long-running session looks frozen
- * between text emissions.
- */
-export function extractTextFromContent(
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-content, verbose) {
-    let text = '';
-    for (const item of content) {
-        if (item.type === 'text') {
-            text += `${item.text}\n`;
-            logDebug(item.text, verbose);
-        }
-        else if (verbose && item.type === 'tool_use') {
-            logDebug(`→ ${item.name}(${previewJson(item.input)})`, verbose);
-        }
-        else if (verbose && item.type === 'tool_result') {
-            const preview = Array.isArray(item.content)
-                ? item.content
-                    .filter((c) => c?.type === 'text')
-                    .map((c) => c.text ?? '')
-                    .join(' ')
-                : String(item.content ?? '');
-            const flag = item.is_error ? '✗' : '←';
-            logDebug(`${flag} ${truncate(preview, 200)}`, verbose);
-        }
-    }
-    return text;
-}
-function previewJson(value, max = 200) {
-    try {
-        return truncate(JSON.stringify(value), max);
-    }
-    catch {
-        return truncate(String(value), max);
-    }
-}
-function truncate(text, max) {
-    if (text.length <= max) {
-        return text;
-    }
-    return `${text.slice(0, max - 1)}…`;
-}
-/**
- * Try to parse a JSON result from agent response text.
- * Tries a custom fenceTag (e.g. ```screen_flow) first when provided, then
- * ```json, then falls back to raw JSON parsing. Returns the parsed object or
- * null on failure.
- */
-export function tryParseJsonFromResponse(responseText, fenceTag = 'json') {
-    try {
-        if (fenceTag !== 'json') {
-            const taggedMatch = responseText.match(new RegExp(`\`\`\`${escapeRegExp(fenceTag)}\\s*\\n([\\s\\S]*?)\\n\\s*\`\`\``));
-            if (taggedMatch) {
-                return JSON.parse(taggedMatch[1]);
-            }
-        }
-        const jsonBlockMatch = responseText.match(/```json\s*\n([\s\S]*?)\n\s*```/);
-        return jsonBlockMatch
-            ? JSON.parse(jsonBlockMatch[1])
-            : JSON.parse(responseText);
-    }
-    catch {
-        return null;
-    }
-}
-/**
- * Extract a specific keyed result from agent response.
- * e.g., tryExtractResult(text, 'review_result') extracts the review_result key.
- * The key is also tried as the fenced code-block tag so phases whose output
- * contract uses a custom fence (e.g. ```screen_flow) parse correctly.
- */
-export function tryExtractResult(responseText, key) {
-    const parsed = tryParseJsonFromResponse(responseText, key);
-    if (parsed &&
-        typeof parsed === 'object' &&
-        key in parsed) {
-        return parsed[key];
-    }
-    // If top-level has the expected shape, return the whole thing
-    return parsed;
-}
-function escapeRegExp(value) {
-    return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}

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

@@ -1,24 +0,0 @@
-/**
- * Shared context builder for standalone PR operations.
- * Reuses GitHub API utilities from code-review/context.ts without issue dependencies.
- */
-import { type PRCommit, type PRData, type PRFile } from '../code-review/context.js';
-export type { PRCommit, PRData, PRFile } from '../code-review/context.js';
-export { parsePullRequestUrl } from '../code-review/context.js';
-export interface StandalonePRContext {
-    pullRequestUrl: string;
-    pullRequestNumber: number;
-    owner: string;
-    repo: string;
-    prData: PRData;
-    files: PRFile[];
-    commits: PRCommit[];
-}
-/**
- * Fetch PR context for standalone review/resolve (no issue dependency).
- */
-export declare function fetchStandalonePRContext(pullRequestUrl: string, githubToken: string, verbose?: boolean): Promise<StandalonePRContext>;
-/**
- * Format standalone PR context for prompts (no issue/design context).
- */
-export declare function formatStandalonePRContextForPrompt(context: StandalonePRContext): string;

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

@@ -1,78 +0,0 @@
-/**
- * Shared context builder for standalone PR operations.
- * Reuses GitHub API utilities from code-review/context.ts without issue dependencies.
- */
-import { Octokit } from '@octokit/rest';
-import { logDebug } from '../../utils/logger.js';
-import { fetchPRCommits, fetchPRDetails, fetchPRFiles, parsePullRequestUrl, } from '../code-review/context.js';
-export { parsePullRequestUrl } from '../code-review/context.js';
-/**
- * Fetch PR context for standalone review/resolve (no issue dependency).
- */
-export async function fetchStandalonePRContext(pullRequestUrl, githubToken, verbose) {
-    const prInfo = parsePullRequestUrl(pullRequestUrl);
-    if (!prInfo) {
-        throw new Error(`Invalid pull request URL: ${pullRequestUrl}. Expected format: https://github.com/owner/repo/pull/123`);
-    }
-    const { owner, repo, prNumber } = prInfo;
-    const octokit = new Octokit({ auth: githubToken });
-    const [prData, files, commits] = await Promise.all([
-        fetchPRDetails(octokit, owner, repo, prNumber, verbose),
-        fetchPRFiles(octokit, owner, repo, prNumber, verbose),
-        fetchPRCommits(octokit, owner, repo, prNumber, verbose),
-    ]);
-    logDebug(`Summary: ${files.length} files changed across ${commits.length} commits`, verbose);
-    return {
-        pullRequestUrl,
-        pullRequestNumber: prNumber,
-        owner,
-        repo,
-        prData,
-        files,
-        commits,
-    };
-}
-/**
- * Format standalone PR context for prompts (no issue/design context).
- */
-export function formatStandalonePRContextForPrompt(context) {
-    const sections = [];
-    sections.push(`# Pull Request Information`);
-    sections.push(`**Pull Request**: ${context.pullRequestUrl} (#${context.pullRequestNumber})`);
-    sections.push('');
-    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('');
-    if (context.commits.length > 0) {
-        sections.push(`## Commits (${context.commits.length})`);
-        sections.push('');
-        for (const commit of context.commits) {
-            sections.push(`- **${commit.sha.substring(0, 7)}**: ${commit.commit.message}`);
-        }
-        sections.push('');
-    }
-    if (context.files.length > 0) {
-        sections.push(`## Changed Files (${context.files.length})`);
-        sections.push('');
-        for (const file of context.files) {
-            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('');
-        }
-    }
-    return sections.join('\n');
-}

package/dist/phases/pr-shared/status.d.ts

@@ -1,18 +0,0 @@
-/**
- * Update a pull_requests row's status, choosing the right transport for the
- * scope:
- *
- *   - Repo-scoped PRs (repo-only mode) have no product, so the MCP
- *     pull_requests/update endpoint — which authorises by product/issue —
- *     can't see them. Write straight through the CLI's RLS-scoped Supabase
- *     session instead (team-membership RLS authorises the repo PR).
- *   - Product/issue PRs keep using the MCP endpoint.
- *
- * Best-effort: a status update is never allowed to fail the review/resolve.
- */
-export declare function updatePullRequestStatus(opts: {
-    prId: string;
-    status: string;
-    /** When set, the PR is repo-scoped and updated directly via Supabase. */
-    repositoryId?: string;
-}): Promise<void>;

package/dist/phases/pr-shared/status.js

@@ -1,37 +0,0 @@
-/**
- * Update a pull_requests row's status, choosing the right transport for the
- * scope:
- *
- *   - Repo-scoped PRs (repo-only mode) have no product, so the MCP
- *     pull_requests/update endpoint — which authorises by product/issue —
- *     can't see them. Write straight through the CLI's RLS-scoped Supabase
- *     session instead (team-membership RLS authorises the repo PR).
- *   - Product/issue PRs keep using the MCP endpoint.
- *
- * Best-effort: a status update is never allowed to fail the review/resolve.
- */
-import { callMcpEndpoint } from '../../api/mcp-client.js';
-import { ensureSupabaseSession, getSupabase } from '../../supabase/client.js';
-export async function updatePullRequestStatus(opts) {
-    const { prId, status, repositoryId } = opts;
-    try {
-        if (repositoryId) {
-            const ready = await ensureSupabaseSession();
-            if (!ready) {
-                return;
-            }
-            await getSupabase()
-                .from('pull_requests')
-                .update({ status, updated_at: new Date().toISOString() })
-                .eq('id', prId);
-            return;
-        }
-        await callMcpEndpoint('pull_requests/update', {
-            pull_request_id: prId,
-            status,
-        });
-    }
-    catch {
-        // Non-critical — don't fail the review/resolve on a status write.
-    }
-}

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

@@ -1,28 +0,0 @@
-import { type ProductInfo } from '../../api/products.js';
-import { type Branch } from '../../services/branches.js';
-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 PRSplittingContext {
-    issue: IssueInfo;
-    product: ProductInfo;
-    existing_branches: Branch[];
-    existing_pull_requests: PullRequest[];
-    diffStat: string;
-    changedFiles: string[];
-    devBranchName: string;
-    devBranchHeadSha: string;
-    githubConfig: GitHubConfigInfo;
-    forkInfo: RepoForkInfo;
-}
-/**
- * Fetch all context information needed for PR splitting
- */
-export declare function fetchPRSplittingContext(issueId: string, verbose?: boolean, replaceExisting?: boolean): Promise<PRSplittingContext>;