edsger 0.40.0 → 0.41.0

package/dist/commands/pr-resolve/index.d.ts

@@ -0,0 +1,10 @@
+/**
+ * CLI command: edsger pr-resolve <productId> --pr-url <url>
+ * Resolves PR change requests: fixes what's genuinely better, explains what it won't change.
+ */
+export interface PRResolveCliOptions {
+    prUrl: string;
+    prId?: string;
+    verbose?: boolean;
+}
+export declare function runPRResolve(productId: string, options: PRResolveCliOptions): Promise<void>;

package/dist/commands/pr-resolve/index.js

@@ -0,0 +1,44 @@
+/**
+ * CLI command: edsger pr-resolve <productId> --pr-url <url>
+ * Resolves PR change requests: fixes what's genuinely better, explains what it won't change.
+ */
+import { getGitHubConfigByProduct } from '../../api/github.js';
+import { resolveStandalonePR } from '../../phases/pr-resolve/index.js';
+import { logError, logInfo, logSuccess } from '../../utils/logger.js';
+export async function runPRResolve(productId, options) {
+    const { prUrl, prId, verbose } = options;
+    logInfo(`Starting PR resolve for product ${productId}`);
+    logInfo(`PR URL: ${prUrl}`);
+    // Get GitHub config via product developer settings
+    const githubConfig = await getGitHubConfigByProduct(productId, verbose);
+    if (!githubConfig.configured ||
+        !githubConfig.token ||
+        !githubConfig.owner ||
+        !githubConfig.repo) {
+        logError(`GitHub not configured for product ${productId}: ${githubConfig.message || 'No installation found'}`);
+        process.exit(1);
+    }
+    const result = await resolveStandalonePR({
+        productId,
+        pullRequestUrl: prUrl,
+        githubToken: githubConfig.token,
+        owner: githubConfig.owner,
+        repo: githubConfig.repo,
+        prId,
+        verbose,
+    });
+    if (result.status === 'success') {
+        logSuccess(`PR resolve completed: ${result.message}`);
+        if (result.summary) {
+            logInfo(`Summary: ${result.summary}`);
+        }
+        if (result.threadsAddressed !== undefined) {
+            logInfo(`Threads addressed: ${result.threadsAddressed}`);
+            logInfo(`Threads skipped: ${result.threadsSkipped}`);
+        }
+    }
+    else {
+        logError(`PR resolve failed: ${result.message}`);
+        process.exit(1);
+    }
+}

package/dist/commands/pr-review/index.d.ts

@@ -0,0 +1,10 @@
+/**
+ * CLI command: edsger pr-review <productId> --pr-url <url>
+ * Reviews a standalone GitHub PR and posts comments.
+ */
+export interface PRReviewCliOptions {
+    prUrl: string;
+    prId?: string;
+    verbose?: boolean;
+}
+export declare function runPRReview(productId: string, options: PRReviewCliOptions): Promise<void>;

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

@@ -0,0 +1,43 @@
+/**
+ * CLI command: edsger pr-review <productId> --pr-url <url>
+ * Reviews a standalone GitHub PR and posts comments.
+ */
+import { getGitHubConfigByProduct } from '../../api/github.js';
+import { reviewStandalonePR } from '../../phases/pr-review/index.js';
+import { logError, logInfo, logSuccess } from '../../utils/logger.js';
+export async function runPRReview(productId, options) {
+    const { prUrl, prId, verbose } = options;
+    logInfo(`Starting PR review for product ${productId}`);
+    logInfo(`PR URL: ${prUrl}`);
+    // Get GitHub config via product developer settings
+    const githubConfig = await getGitHubConfigByProduct(productId, verbose);
+    if (!githubConfig.configured ||
+        !githubConfig.token ||
+        !githubConfig.owner ||
+        !githubConfig.repo) {
+        logError(`GitHub not configured for product ${productId}: ${githubConfig.message || 'No installation found'}`);
+        process.exit(1);
+    }
+    const result = await reviewStandalonePR({
+        productId,
+        pullRequestUrl: prUrl,
+        githubToken: githubConfig.token,
+        owner: githubConfig.owner,
+        repo: githubConfig.repo,
+        prId,
+        verbose,
+    });
+    if (result.status === 'success') {
+        logSuccess(`PR review completed: ${result.message}`);
+        if (result.reviewUrl) {
+            logInfo(`Review URL: ${result.reviewUrl}`);
+        }
+        if (result.commentsCount !== undefined) {
+            logInfo(`Comments posted: ${result.commentsCount}`);
+        }
+    }
+    else {
+        logError(`PR review failed: ${result.message}`);
+        process.exit(1);
+    }
+}

package/dist/index.js

@@ -18,6 +18,8 @@ import { runInit } from './commands/init/index.js';
 import { runIntelligence } from './commands/intelligence/index.js';
 import { runRefactor } from './commands/refactor/refactor.js';
 import { runTaskWorker } from './commands/task-worker/index.js';
+import { runPRReview } from './commands/pr-review/index.js';
+import { runPRResolve } from './commands/pr-resolve/index.js';
 import { runWorkflow } from './commands/workflow/index.js';
 import { logError, logInfo } from './utils/logger.js';
 // Get package.json version dynamically
@@ -272,6 +274,42 @@ program
     }
 });
 // ============================================================
+// Subcommand: edsger pr-review <productId>
+// ============================================================
+program
+    .command('pr-review <productId>')
+    .description('AI-review a GitHub PR for a product')
+    .requiredOption('--pr-url <url>', 'GitHub PR URL')
+    .option('--pr-id <id>', 'Pull request record ID in database')
+    .option('-v, --verbose', 'Verbose output')
+    .action(async (productId, opts) => {
+    try {
+        await runPRReview(productId, opts);
+    }
+    catch (error) {
+        logError(error instanceof Error ? error.message : String(error));
+        process.exit(1);
+    }
+});
+// ============================================================
+// Subcommand: edsger pr-resolve <productId>
+// ============================================================
+program
+    .command('pr-resolve <productId>')
+    .description('AI-resolve change requests on a GitHub PR')
+    .requiredOption('--pr-url <url>', 'GitHub PR URL')
+    .option('--pr-id <id>', 'Pull request record ID in database')
+    .option('-v, --verbose', 'Verbose output')
+    .action(async (productId, opts) => {
+    try {
+        await runPRResolve(productId, opts);
+    }
+    catch (error) {
+        logError(error instanceof Error ? error.message : String(error));
+        process.exit(1);
+    }
+});
+// ============================================================
 // Default command (options-based, backward compatible)
 // ============================================================
 program

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

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

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

@@ -0,0 +1,101 @@
+import assert from 'node:assert';
+import { describe, it } from 'node:test';
+import { buildLineToPositionMap, findClosestPosition } from '../diff-utils.js';
+describe('buildLineToPositionMap', () => {
+    it('maps simple additions correctly', () => {
+        const patch = `@@ -1,3 +1,4 @@
+ line 1
++new line
+ line 2
+ line 3`;
+        const map = buildLineToPositionMap(patch);
+        // line 1 is at position 1 (new file line 1)
+        assert.strictEqual(map.get(1), 1);
+        // new line is at position 2 (new file line 2)
+        assert.strictEqual(map.get(2), 2);
+        // line 2 is at position 3 (new file line 3)
+        assert.strictEqual(map.get(3), 3);
+        // line 3 is at position 4 (new file line 4)
+        assert.strictEqual(map.get(4), 4);
+    });
+    it('handles deletions - deleted lines have no new file line number', () => {
+        const patch = `@@ -1,3 +1,2 @@
+ line 1
+-deleted line
+ line 3`;
+        const map = buildLineToPositionMap(patch);
+        assert.strictEqual(map.get(1), 1);
+        // deleted line takes position 2 but no new line
+        // line 3 in old file becomes line 2 in new file, position 3
+        assert.strictEqual(map.get(2), 3);
+    });
+    it('handles multiple hunks', () => {
+        const patch = `@@ -1,2 +1,2 @@
+ line 1
++added
+@@ -10,2 +10,2 @@
+ line 10
++added at 11`;
+        const map = buildLineToPositionMap(patch);
+        // First hunk: line 1 -> pos 1, added -> pos 2
+        assert.strictEqual(map.get(1), 1);
+        assert.strictEqual(map.get(2), 2);
+        // Second hunk: line 10 -> pos 3, added at 11 -> pos 4
+        assert.strictEqual(map.get(10), 3);
+        assert.strictEqual(map.get(11), 4);
+    });
+    it('returns empty map for patch with only hunk header', () => {
+        const patch = '@@ -0,0 +1 @@';
+        const map = buildLineToPositionMap(patch);
+        // Hunk header only, no content lines
+        assert.strictEqual(map.size, 0);
+    });
+    it('handles hunk starting at line other than 1', () => {
+        const patch = `@@ -50,3 +50,3 @@
+ context
+-old
++new`;
+        const map = buildLineToPositionMap(patch);
+        assert.strictEqual(map.get(50), 1);
+        // -old takes position 2 (no new line)
+        // +new takes position 3, new file line 51
+        assert.strictEqual(map.get(51), 3);
+    });
+});
+describe('findClosestPosition', () => {
+    it('returns exact match when available', () => {
+        const map = new Map([
+            [10, 5],
+            [11, 6],
+            [12, 7],
+        ]);
+        const result = findClosestPosition(11, map);
+        assert.deepStrictEqual(result, { position: 6, actualLine: 11 });
+    });
+    it('returns nearby line below first', () => {
+        const map = new Map([
+            [10, 5],
+            [15, 8],
+        ]);
+        // Line 12 not in map, closest below is 15 (offset 3)
+        // closest above is 10 (offset 2) - but we check below first at each offset
+        // offset 1: check 13 (no), check 11 (no)
+        // offset 2: check 14 (no), check 10 (yes!)
+        const result = findClosestPosition(12, map);
+        assert.deepStrictEqual(result, { position: 5, actualLine: 10 });
+    });
+    it('returns null when no line within range', () => {
+        const map = new Map([
+            [1, 1],
+            [100, 50],
+        ]);
+        // Line 50 is too far from both 1 and 100
+        const result = findClosestPosition(50, map);
+        assert.strictEqual(result, null);
+    });
+    it('returns null for empty map', () => {
+        const map = new Map();
+        const result = findClosestPosition(5, map);
+        assert.strictEqual(result, null);
+    });
+});

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

@@ -0,0 +1,36 @@
+/**
+ * Shared diff utilities for mapping file line numbers to GitHub diff positions.
+ * Used by both feature-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

@@ -0,0 +1,100 @@
+/**
+ * Shared diff utilities for mapping file line numbers to GitHub diff positions.
+ * Used by both feature-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.js

@@ -11,172 +11,36 @@ import { getBaseBranchInfo, getBranches, updateBranch, } from '../../services/br
 import { formatChecklistsForContext, } from '../../services/checklist.js';
 import { formatFeedbacksForContext, getFeedbacksForPhase, } from '../../services/feedbacks.js';
 import { prepareCustomBranchGitEnvironmentAsync, preparePhaseGitEnvironmentAsync, syncFeatBranchWithMain, } from '../../utils/git-branch-manager.js';
-import { logDebug, logError, logInfo } from '../../utils/logger.js';
+import { logError, logInfo } from '../../utils/logger.js';
+import { createPromptGenerator, extractTextFromContent, tryExtractResult, } from '../pr-shared/agent-utils.js';
 import { fetchCodeReviewContext, formatContextForPrompt, } from './context.js';
-function userMessage(content) {
-    return {
-        type: 'user',
-        message: { role: 'user', content },
-    };
-}
-async function* prompt(reviewPrompt) {
-    yield userMessage(reviewPrompt);
-    await new Promise((res) => {
-        setTimeout(res, 10000);
-    });
-}
-/**
- * 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>
- */
-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
-            // We can't comment on this line using file line numbers
-        }
-        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
- */
-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;
-}
-/**
- * Extract text content from assistant message content array
- */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-function extractTextFromContent(content, verbose) {
-    let text = '';
-    for (const item of content) {
-        if (item.type === 'text') {
-            text += `${item.text}\n`;
-            logDebug(`🔍 ${item.text}`, verbose);
-        }
-    }
-    return text;
-}
-/**
- * Parse review result from a JSON response string
- */
-function tryParseReviewResult(responseText) {
-    const jsonBlockMatch = responseText.match(/```json\s*\n([\s\S]*?)\n\s*```/);
-    const jsonResult = jsonBlockMatch
-        ? JSON.parse(jsonBlockMatch[1])
-        : JSON.parse(responseText);
-    if (jsonResult && jsonResult.review_result) {
-        return jsonResult.review_result;
-    }
-    throw new Error('Invalid JSON structure');
-}
+import { buildLineToPositionMap, findClosestPosition } from './diff-utils.js';
 /**
  * Process a query result message and extract the structured review result
  */
 function parseQueryResultMessage(message, lastAssistantResponse) {
     if (message.subtype === 'success') {
-        logInfo('\n🛠️ Code review analysis completed, parsing results...');
-        try {
-            return tryParseReviewResult(message.result || lastAssistantResponse);
-        }
-        catch (error) {
-            logError(`Failed to parse structured review result: ${error}`);
-            return parseCodeReviewResponse(message.result || lastAssistantResponse);
+        logInfo('\n Code review analysis completed, parsing results...');
+        const responseText = message.result || lastAssistantResponse;
+        const result = tryExtractResult(responseText, 'review_result');
+        if (result) {
+            return result;
         }
+        logError('Failed to parse structured review result');
+        return parseCodeReviewResponse(responseText);
     }
-    logError(`\n⚠️  Code review incomplete: ${message.subtype}`);
+    logError(`\n Code review incomplete: ${message.subtype}`);
     if (message.subtype === 'error_max_turns') {
-        logError('💡 Try simplifying the review scope');
+        logError('Try simplifying the review scope');
     }
     if (!lastAssistantResponse) {
         return null;
     }
-    try {
-        return tryParseReviewResult(lastAssistantResponse);
-    }
-    catch (error) {
-        logError(`Failed to parse assistant response: ${error}`);
-        return parseCodeReviewResponse(lastAssistantResponse);
+    const result = tryExtractResult(lastAssistantResponse, 'review_result');
+    if (result) {
+        return result;
     }
+    return parseCodeReviewResponse(lastAssistantResponse);
 }
 /**
  * Log base branch rebase info for verbose mode
@@ -374,7 +238,7 @@ export const reviewPullRequest = async (options, config, checklistContext
         // Use Claude to analyze the code (not using Claude Code SDK for actual implementation)
         // This is a simplified approach - just analyze the code
         for await (const message of query({
-            prompt: prompt(reviewPrompt),
+            prompt: createPromptGenerator(reviewPrompt),
             options: {
                 systemPrompt: {
                     type: 'preset',

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

@@ -38,6 +38,18 @@ export declare function removeDeletedFilesFromPRs(changedFiles: ChangedFileInfo[
         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.

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

@@ -194,6 +194,40 @@ export async function removeDeletedFilesFromPRs(changedFiles, pullRequests, upda
     }
     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.