edsger 0.42.1 → 0.44.0

package/dist/phases/pr-resolve/checklist-learner.js

@@ -9,24 +9,48 @@
 import { query } from '@anthropic-ai/claude-agent-sdk';
 import { createChecklistsMcpServer } from '../../commands/checklists/tools.js';
 import { logInfo, logWarning } from '../../utils/logger.js';
-const LEARNER_SYSTEM_PROMPT = `You are a software quality engineer. Your task is to analyse review comments that were addressed during a PR resolve and distil them into actionable checklist items for the code_review phase.
+const LEARNER_SYSTEM_PROMPT = `You are a software quality engineer. Your task is to analyse review comments that were addressed during a PR resolve and distil them into actionable checklist items — assigned to the most appropriate phase(s) so issues are caught as early as possible.
+
+## Available Phases (earliest → latest)
+
+| Phase | When it runs | Good for |
+|-------|-------------|----------|
+| \`feature_analysis\` | Requirements gathering | Requirement gaps, scope issues, edge cases |
+| \`user_stories_analysis\` | User story refinement | Acceptance criteria gaps, missing scenarios |
+| \`test_cases_analysis\` | Test planning | Missing test coverage, untested edge cases |
+| \`technical_design\` | Architecture & design | Design flaws, API contract issues, data modelling |
+| \`branch_planning\` | Work breakdown | Dependency issues, ordering problems |
+| \`code_implementation\` | Writing code | Coding patterns, error handling, security, performance |
+| \`code_testing\` | Unit/integration tests | Test quality, assertion coverage, mocking issues |
+| \`code_refine\` | Code polish | Readability, naming, documentation |
+| \`code_review\` | Final review | Cross-cutting concerns not caught earlier |
 
 ## Workflow
 
 1. **Read** the addressed review comments provided below.
-2. **Query existing checklists** using \`list_checklists\` with phase "code_review" to see what already exists.
+2. **Query existing checklists** using \`list_checklists\` (without a phase filter) to see what already exists across all phases.
 3. **Identify patterns** — group related comments into categories (e.g., error handling, naming, security, performance, testing).
-4. **Update or create checklist items**:
-   - If an existing checklist covers the category, add new items to it (skip if a similar item already exists).
-   - If no suitable checklist exists, create a new one.
-5. **Summarise** what you added or updated.
+4. **Decide the best phase(s)** for each pattern — prefer earlier phases where the issue could have been prevented. A checklist item can belong to multiple phases if appropriate.
+5. **Update or create checklist items**:
+   - If an existing checklist covers the category and phase, add new items to it (skip if a similar item already exists).
+   - If no suitable checklist exists, create a new one with the appropriate phases.
+6. **Summarise** what you added or updated, and explain why you chose each phase.
+
+## Phase Selection Guidelines
+
+- **Shift left**: If an issue could have been prevented in an earlier phase, assign it there. For example, a missing null check found in code review should become a \`code_implementation\` checklist item.
+- **Security & error handling** → typically \`code_implementation\`
+- **Missing test cases or edge cases** → \`test_cases_analysis\` or \`code_testing\`
+- **Design or architecture issues** → \`technical_design\`
+- **Missing requirements or acceptance criteria** → \`feature_analysis\` or \`user_stories_analysis\`
+- **Code style, readability, naming** → \`code_refine\`
+- **Only use \`code_review\`** for cross-cutting concerns that genuinely can't be checked earlier.
 
 ## Rules
 
-- Only create items for **genuine quality patterns** — things that should be checked in every code review.
+- Only create items for **genuine quality patterns** — things that should be checked repeatedly.
 - Skip one-off nits, purely stylistic preferences, or context-specific fixes that won't generalise.
 - Role: \`developer\`
-- Phases: \`["code_review"]\`
 - Item type: \`boolean\` (yes/no checkable)
 - Keep item titles concise (< 80 chars). Use the description for details.
 - Do NOT duplicate items that already exist in the checklists.
@@ -72,7 +96,7 @@ export function buildLearnerPrompt(addressedComments, unresolvedThreads, comment
         sections.push('');
     }
     sections.push('## Instructions');
-    sections.push('Based on the patterns above, query existing code_review checklists and create or update items to prevent these issues from recurring.');
+    sections.push('Based on the patterns above, query existing checklists across all phases and create or update items in the most appropriate phase(s) to prevent these issues from recurring. Prefer earlier phases where the issue could have been caught sooner.');
     return sections.join('\n');
 }
 /**
@@ -112,6 +136,7 @@ export async function learnFromReviewFeedback(input) {
             if (message.type === 'result') {
                 if (message.subtype === 'success') {
                     logInfo('Checklist learning completed.');
+                    // eslint-disable-next-line max-depth
                     if (verbose && message.result) {
                         logInfo(message.result);
                     }

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

@@ -22,6 +22,7 @@ export { isResolveResult } from './types.js';
 /**
  * Resolve PR change requests: evaluate each comment, fix or explain.
  */
+// eslint-disable-next-line complexity
 export async function resolveStandalonePR(options) {
     const { pullRequestUrl, githubToken, owner, repo, verbose, prId } = options;
     logInfo(`Starting PR resolve: ${pullRequestUrl}`);
@@ -69,9 +70,17 @@ export async function resolveStandalonePR(options) {
         // For fork PRs, pass fallback info so prepareWorkspace can fetch the PR ref
         // from upstream if the fork branch is unavailable.
         // For deleted forks, clone upstream directly and always use PR ref fallback.
-        const repoPath = prepareWorkspace(cloneOwner, cloneRepo, headRef, prInfo.prNumber, githubToken, verbose, isFork
-            ? { upstreamOwner: owner, upstreamRepo: repo }
-            : undefined);
+        const repoPath = prepareWorkspace({
+            owner: cloneOwner,
+            repo: cloneRepo,
+            headRef,
+            prNumber: prInfo.prNumber,
+            token: githubToken,
+            verbose,
+            forkFallback: isFork
+                ? { upstreamOwner: owner, upstreamRepo: repo }
+                : undefined,
+        });
         try {
             // Run Claude Agent SDK to evaluate and fix comments
             const systemPrompt = createResolveSystemPrompt();
@@ -104,6 +113,7 @@ export async function resolveStandalonePR(options) {
                     logInfo('Agent completed, parsing results...');
                     const responseText = message.result || lastAssistantResponse;
                     const parsed = tryExtractResult(responseText, 'resolve_result');
+                    // eslint-disable-next-line max-depth
                     if (isResolveResult(parsed)) {
                         resolveResult = parsed;
                     }
@@ -114,17 +124,19 @@ export async function resolveStandalonePR(options) {
                 else {
                     logError(`Agent incomplete: ${message.subtype}`);
                     // Try to salvage partial results from last response
+                    // eslint-disable-next-line max-depth
                     if (lastAssistantResponse) {
                         const salvaged = tryExtractResult(lastAssistantResponse, 'resolve_result');
+                        // eslint-disable-next-line max-depth
                         if (isResolveResult(salvaged)) {
                             resolveResult = salvaged;
                         }
                     }
                 }
             }
-            // Commit and push any changes the agent made
+            // Fallback: commit any leftover uncommitted changes the agent didn't commit itself
             if (hasUncommittedChanges(repoPath)) {
-                logInfo('Committing changes...');
+                logInfo('Committing remaining uncommitted changes...');
                 execSync('git add -A', { cwd: repoPath, stdio: 'pipe' });
                 execSync('git commit -m "Resolve PR review comments\n\nAutomated resolution by Edsger AI"', { cwd: repoPath, stdio: 'pipe' });
             }
@@ -142,13 +154,16 @@ export async function resolveStandalonePR(options) {
                 for (const comment of comments) {
                     // Map comment_id back to real GraphQL thread ID
                     const threadId = commentIdToThreadId.get(comment.comment_id);
+                    // eslint-disable-next-line max-depth
                     if (!threadId) {
                         logError(`Unknown comment_id "${comment.comment_id}", skipping reply`);
                         threadsErrored++;
                         continue;
                     }
+                    // eslint-disable-next-line max-depth
                     try {
                         const replied = await replyToReviewThread(octokit, threadId, comment.reply, verbose);
+                        // eslint-disable-next-line max-depth
                         if (replied && comment.action === 'changed') {
                             // Resolve the thread since the change was made
                             await resolveReviewThread(octokit, threadId, verbose);
@@ -178,6 +193,7 @@ export async function resolveStandalonePR(options) {
                             ? 'Changes were made to address review feedback. Please re-review.'
                             : 'Reviewed this comment. No changes were made at this time.';
                         const replied = await replyToReviewThread(octokit, thread.id, genericReply, verbose);
+                        // eslint-disable-next-line max-depth
                         if (replied) {
                             threadsSkipped++;
                         }
@@ -212,6 +228,14 @@ export async function resolveStandalonePR(options) {
                     // Non-critical
                 }
             }
+            // Clean up workspace on success
+            try {
+                rmSync(repoPath, { recursive: true, force: true });
+                logInfo(`Cleaned up workspace: ${repoPath}`);
+            }
+            catch {
+                // ignore cleanup errors
+            }
             return {
                 status: 'success',
                 message: `Resolved ${threadsAddressed} threads, skipped ${threadsSkipped}, ${threadsErrored} errors`,
@@ -228,14 +252,6 @@ export async function resolveStandalonePR(options) {
             logInfo(`Workspace preserved for inspection: ${repoPath}`);
             throw innerError;
         }
-        // Only clean up on success
-        try {
-            rmSync(repoPath, { recursive: true, force: true });
-            logInfo(`Cleaned up workspace: ${repoPath}`);
-        }
-        catch {
-            // ignore cleanup errors
-        }
     }
     catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);

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

@@ -27,7 +27,8 @@ export function createResolveSystemPrompt() {
 2. For each comment, examine the relevant code
 3. If you agree: make the change in the file
 4. If you disagree: skip it (do NOT modify the file for that comment)
-5. After processing all comments, output a JSON summary
+5. After making all changes, commit them with a descriptive message summarizing what was resolved (do NOT push)
+6. After committing, output a JSON summary
 
 **CRITICAL - Result Format**:
 After making all changes, you MUST output a JSON result. Use the exact comment_id from each comment (comment_1, comment_2, etc.):

package/dist/phases/pr-resolve/workspace.d.ts

@@ -8,19 +8,29 @@
 export declare function buildCredentialArgs(token: string): string[];
 /**
  * Get the workspace path for a PR resolve operation.
+ * Includes owner/repo to avoid collisions when resolving PRs from different repos.
  */
-export declare function getResolveWorkspacePath(prNumber: number): string;
+export declare function getResolveWorkspacePath(owner: string, repo: string, prNumber: number): string;
 export interface ForkFallbackInfo {
     upstreamOwner: string;
     upstreamRepo: string;
 }
+export interface PrepareWorkspaceOptions {
+    owner: string;
+    repo: string;
+    headRef: string;
+    prNumber: number;
+    token: string;
+    verbose?: boolean;
+    forkFallback?: ForkFallbackInfo;
+}
 /**
  * Clone or reuse a repo for PR resolve.
  * For fork PRs, clones from the fork repo. If the fork branch is unavailable,
  * falls back to fetching the PR ref from the upstream repo.
  * Returns the workspace path.
  */
-export declare function prepareWorkspace(owner: string, repo: string, headRef: string, prNumber: number, token: string, verbose?: boolean, forkFallback?: ForkFallbackInfo): string;
+export declare function prepareWorkspace(options: PrepareWorkspaceOptions): string;
 /**
  * Push changes from workspace back to remote.
  */

package/dist/phases/pr-resolve/workspace.js

@@ -21,9 +21,10 @@ export function buildCredentialArgs(token) {
 }
 /**
  * Get the workspace path for a PR resolve operation.
+ * Includes owner/repo to avoid collisions when resolving PRs from different repos.
  */
-export function getResolveWorkspacePath(prNumber) {
-    return join(homedir(), 'edsger', `pr-resolve-${prNumber}`);
+export function getResolveWorkspacePath(owner, repo, prNumber) {
+    return join(homedir(), 'edsger', `pr-resolve-${owner}-${repo}-${prNumber}`);
 }
 /**
  * Clone or reuse a repo for PR resolve.
@@ -31,8 +32,9 @@ export function getResolveWorkspacePath(prNumber) {
  * falls back to fetching the PR ref from the upstream repo.
  * Returns the workspace path.
  */
-export function prepareWorkspace(owner, repo, headRef, prNumber, token, verbose, forkFallback) {
-    const repoPath = getResolveWorkspacePath(prNumber);
+export function prepareWorkspace(options) {
+    const { owner, repo, headRef, prNumber, token, verbose, forkFallback } = options;
+    const repoPath = getResolveWorkspacePath(owner, repo, prNumber);
     const repoUrl = `https://github.com/${owner}/${repo}.git`;
     const gitCredentialArgs = buildCredentialArgs(token);
     if (existsSync(join(repoPath, '.git'))) {

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

@@ -1,46 +1,46 @@
 import assert from 'node:assert';
 import { describe, it } from 'node:test';
 import { createStandaloneReviewSystemPrompt, createStandaloneReviewUserPrompt, } from '../prompts.js';
-describe('createStandaloneReviewSystemPrompt', () => {
-    it('includes review focus areas', () => {
+void describe('createStandaloneReviewSystemPrompt', () => {
+    void it('includes review focus areas', () => {
         const prompt = createStandaloneReviewSystemPrompt();
         assert.ok(prompt.includes('Code Quality'));
         assert.ok(prompt.includes('Security'));
         assert.ok(prompt.includes('Performance'));
     });
-    it('specifies JSON result format', () => {
+    void it('specifies JSON result format', () => {
         const prompt = createStandaloneReviewSystemPrompt();
         assert.ok(prompt.includes('review_result'));
         assert.ok(prompt.includes('"file"'));
         assert.ok(prompt.includes('"line"'));
         assert.ok(prompt.includes('"comment"'));
     });
-    it('includes assessment options', () => {
+    void it('includes assessment options', () => {
         const prompt = createStandaloneReviewSystemPrompt();
         assert.ok(prompt.includes('APPROVE'));
         assert.ok(prompt.includes('REQUEST_CHANGES'));
         assert.ok(prompt.includes('COMMENT'));
     });
-    it('does not include feature/checklist instructions', () => {
+    void it('does not include feature/checklist instructions', () => {
         const prompt = createStandaloneReviewSystemPrompt();
         assert.ok(!prompt.includes('checklist'));
         assert.ok(!prompt.includes('feature_id'));
         assert.ok(!prompt.includes('user stories'));
     });
 });
-describe('createStandaloneReviewUserPrompt', () => {
-    it('includes context info in the prompt', () => {
+void describe('createStandaloneReviewUserPrompt', () => {
+    void it('includes context info in the prompt', () => {
         const contextInfo = '# Pull Request\n**Title**: Fix auth bug\n';
         const prompt = createStandaloneReviewUserPrompt(contextInfo);
         assert.ok(prompt.includes('Fix auth bug'));
     });
-    it('includes review instructions', () => {
+    void it('includes review instructions', () => {
         const prompt = createStandaloneReviewUserPrompt('some context');
         assert.ok(prompt.includes('Analyze Each File'));
         assert.ok(prompt.includes('Identify Issues'));
         assert.ok(prompt.includes('Actionable Feedback'));
     });
-    it('mentions severity categories', () => {
+    void it('mentions severity categories', () => {
         const prompt = createStandaloneReviewUserPrompt('context');
         assert.ok(prompt.includes('Critical'));
         assert.ok(prompt.includes('Major'));

package/dist/phases/pr-review/__tests__/review-comments.test.js

@@ -36,7 +36,7 @@ function mapCommentsToReviewPayload(agentComments, files) {
     }
     return result;
 }
-describe('review comment mapping (integration)', () => {
+void describe('review comment mapping (integration)', () => {
     const samplePatch = `@@ -1,5 +1,7 @@
  import { useState } from 'react'
 
@@ -47,7 +47,7 @@ describe('review comment mapping (integration)', () => {
 +  const [count, setCount] = useState<number>(0)
    return <div>{count}</div>`;
     const files = [{ filename: 'src/App.tsx', patch: samplePatch }];
-    it('maps exact line comments to correct diff positions', () => {
+    void it('maps exact line comments to correct diff positions', () => {
         const comments = [
             {
                 file: 'src/App.tsx',
@@ -63,7 +63,7 @@ describe('review comment mapping (integration)', () => {
         // No "Note" prefix since line was exact
         assert.ok(!result[0].body.includes('**Note**'));
     });
-    it('adjusts line numbers when exact line not in diff', () => {
+    void it('adjusts line numbers when exact line not in diff', () => {
         // Line 2 is blank (in diff) but line 100 is way outside
         const comments = [{ file: 'src/App.tsx', line: 8, comment: 'Fix this' }];
         const result = mapCommentsToReviewPayload(comments, files);
@@ -73,7 +73,7 @@ describe('review comment mapping (integration)', () => {
         }
         // If no match within range, it's filtered out - also valid
     });
-    it('filters out comments for files not in diff', () => {
+    void it('filters out comments for files not in diff', () => {
         const comments = [
             {
                 file: 'src/other.ts',
@@ -84,7 +84,7 @@ describe('review comment mapping (integration)', () => {
         const result = mapCommentsToReviewPayload(comments, files);
         assert.strictEqual(result.length, 0);
     });
-    it('handles multiple comments on same file', () => {
+    void it('handles multiple comments on same file', () => {
         const comments = [
             { file: 'src/App.tsx', line: 3, comment: 'Comment A' },
             { file: 'src/App.tsx', line: 6, comment: 'Comment B' },
@@ -94,7 +94,7 @@ describe('review comment mapping (integration)', () => {
         assert.ok(result[0].body.includes('Comment A'));
         assert.ok(result[1].body.includes('Comment B'));
     });
-    it('handles files with no patch (binary files)', () => {
+    void it('handles files with no patch (binary files)', () => {
         const binaryFiles = [
             { filename: 'image.png' }, // no patch
             { filename: 'src/App.tsx', patch: samplePatch },

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

@@ -14,6 +14,7 @@ import { createStandaloneReviewSystemPrompt, createStandaloneReviewUserPrompt, }
 /**
  * Review a standalone PR and post comments to GitHub.
  */
+// eslint-disable-next-line complexity
 export async function reviewStandalonePR(options) {
     const { pullRequestUrl, githubToken, verbose, prId } = options;
     logInfo(`Starting standalone PR review: ${pullRequestUrl}`);

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

@@ -1,8 +1,8 @@
 import assert from 'node:assert';
 import { describe, it } from 'node:test';
 import { extractTextFromContent, tryExtractResult, tryParseJsonFromResponse, userMessage, } from '../agent-utils.js';
-describe('userMessage', () => {
-    it('creates a user message object', () => {
+void describe('userMessage', () => {
+    void it('creates a user message object', () => {
         const msg = userMessage('hello');
         assert.deepStrictEqual(msg, {
             type: 'user',
@@ -10,8 +10,8 @@ describe('userMessage', () => {
         });
     });
 });
-describe('extractTextFromContent', () => {
-    it('extracts text items from content array', () => {
+void describe('extractTextFromContent', () => {
+    void it('extracts text items from content array', () => {
         const content = [
             { type: 'text', text: 'hello' },
             { type: 'tool_use', id: '123', name: 'read', input: {} },
@@ -20,35 +20,35 @@ describe('extractTextFromContent', () => {
         const result = extractTextFromContent(content);
         assert.strictEqual(result, 'hello\nworld\n');
     });
-    it('returns empty string for no text items', () => {
+    void it('returns empty string for no text items', () => {
         const content = [{ type: 'tool_use', id: '123', name: 'read', input: {} }];
         const result = extractTextFromContent(content);
         assert.strictEqual(result, '');
     });
-    it('returns empty string for empty array', () => {
+    void it('returns empty string for empty array', () => {
         assert.strictEqual(extractTextFromContent([]), '');
     });
 });
-describe('tryParseJsonFromResponse', () => {
-    it('parses JSON from code block', () => {
+void describe('tryParseJsonFromResponse', () => {
+    void it('parses JSON from code block', () => {
         const text = 'Here is the result:\n```json\n{"key": "value"}\n```\nDone.';
         const result = tryParseJsonFromResponse(text);
         assert.deepStrictEqual(result, { key: 'value' });
     });
-    it('parses raw JSON', () => {
+    void it('parses raw JSON', () => {
         const text = '{"key": "value"}';
         const result = tryParseJsonFromResponse(text);
         assert.deepStrictEqual(result, { key: 'value' });
     });
-    it('returns null for invalid JSON', () => {
+    void it('returns null for invalid JSON', () => {
         const result = tryParseJsonFromResponse('not json at all');
         assert.strictEqual(result, null);
     });
-    it('returns null for empty string', () => {
+    void it('returns null for empty string', () => {
         const result = tryParseJsonFromResponse('');
         assert.strictEqual(result, null);
     });
-    it('parses JSON block with surrounding text', () => {
+    void it('parses JSON block with surrounding text', () => {
         const text = `I analyzed the code and here are my findings:
 
 \`\`\`json
@@ -66,22 +66,22 @@ That's my review.`;
         assert.ok(result.review_result);
     });
 });
-describe('tryExtractResult', () => {
-    it('extracts keyed result from JSON code block', () => {
+void describe('tryExtractResult', () => {
+    void it('extracts keyed result from JSON code block', () => {
         const text = '```json\n{"review_result": {"summary": "good"}}\n```';
         const result = tryExtractResult(text, 'review_result');
         assert.deepStrictEqual(result, { summary: 'good' });
     });
-    it('returns whole object if key not found but JSON valid', () => {
+    void it('returns whole object if key not found but JSON valid', () => {
         const text = '{"summary": "good", "comments": []}';
         const result = tryExtractResult(text, 'review_result');
         assert.deepStrictEqual(result, { summary: 'good', comments: [] });
     });
-    it('returns null for unparseable text', () => {
+    void it('returns null for unparseable text', () => {
         const result = tryExtractResult('no json here', 'review_result');
         assert.strictEqual(result, null);
     });
-    it('extracts resolve_result key', () => {
+    void it('extracts resolve_result key', () => {
         const text = '```json\n{"resolve_result": {"comments": [{"comment_id": "comment_1", "action": "changed", "reply": "fixed"}]}}\n```';
         const result = tryExtractResult(text, 'resolve_result');
         assert.ok(result);

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

@@ -1,8 +1,8 @@
 import assert from 'node:assert';
 import { describe, it } from 'node:test';
 import { formatStandalonePRContextForPrompt, parsePullRequestUrl, } from '../context.js';
-describe('parsePullRequestUrl (re-exported)', () => {
-    it('parses a standard GitHub PR URL', () => {
+void describe('parsePullRequestUrl (re-exported)', () => {
+    void it('parses a standard GitHub PR URL', () => {
         const result = parsePullRequestUrl('https://github.com/owner/repo/pull/123');
         assert.deepStrictEqual(result, {
             owner: 'owner',
@@ -10,7 +10,7 @@ describe('parsePullRequestUrl (re-exported)', () => {
             prNumber: 123,
         });
     });
-    it('parses URL with trailing path segments', () => {
+    void it('parses URL with trailing path segments', () => {
         const result = parsePullRequestUrl('https://github.com/my-org/my-repo/pull/456/files');
         assert.deepStrictEqual(result, {
             owner: 'my-org',
@@ -18,14 +18,14 @@ describe('parsePullRequestUrl (re-exported)', () => {
             prNumber: 456,
         });
     });
-    it('returns null for non-PR URL', () => {
+    void it('returns null for non-PR URL', () => {
         assert.strictEqual(parsePullRequestUrl('https://github.com/owner/repo/issues/1'), null);
     });
-    it('returns null for non-GitHub URL', () => {
+    void it('returns null for non-GitHub URL', () => {
         assert.strictEqual(parsePullRequestUrl('https://example.com/pull/1'), null);
     });
 });
-describe('formatStandalonePRContextForPrompt', () => {
+void describe('formatStandalonePRContextForPrompt', () => {
     const context = {
         pullRequestUrl: 'https://github.com/owner/repo/pull/42',
         pullRequestNumber: 42,
@@ -61,33 +61,33 @@ describe('formatStandalonePRContextForPrompt', () => {
             },
         ],
     };
-    it('includes PR URL and number', () => {
+    void it('includes PR URL and number', () => {
         const output = formatStandalonePRContextForPrompt(context);
         assert.ok(output.includes('#42'));
         assert.ok(output.includes('github.com/owner/repo/pull/42'));
     });
-    it('includes PR title and author', () => {
+    void it('includes PR title and author', () => {
         const output = formatStandalonePRContextForPrompt(context);
         assert.ok(output.includes('Fix bug in auth'));
         assert.ok(output.includes('@testuser'));
     });
-    it('includes branch info', () => {
+    void it('includes branch info', () => {
         const output = formatStandalonePRContextForPrompt(context);
         assert.ok(output.includes('fix/auth'));
         assert.ok(output.includes('main'));
     });
-    it('includes file diff', () => {
+    void it('includes file diff', () => {
         const output = formatStandalonePRContextForPrompt(context);
         assert.ok(output.includes('src/auth.ts'));
         assert.ok(output.includes('+added'));
         assert.ok(output.includes('+5 -2'));
     });
-    it('includes commit info', () => {
+    void it('includes commit info', () => {
         const output = formatStandalonePRContextForPrompt(context);
         assert.ok(output.includes('abc1234'));
         assert.ok(output.includes('fix: auth bug'));
     });
-    it('includes PR body/description', () => {
+    void it('includes PR body/description', () => {
         const output = formatStandalonePRContextForPrompt(context);
         assert.ok(output.includes('This fixes the login issue'));
     });

package/dist/phases/pr-splitting/import-dep-validator.js

@@ -155,6 +155,9 @@ export function getTransitiveDependencies(file, graph) {
     const stack = [file];
     while (stack.length > 0) {
         const current = stack.pop();
+        if (current === undefined) {
+            continue;
+        }
         const deps = graph.get(current);
         if (!deps) {
             continue;
@@ -169,11 +172,8 @@ export function getTransitiveDependencies(file, graph) {
     return result;
 }
 const MAX_FIX_ITERATIONS = 100;
-/**
- * Move a dependency file from a later PR to an earlier PR that needs it.
- * Returns true if a file was moved.
- */
-function moveDepToEarlierPR(dep, prIdx, sorted, fileToPRIndex, movedFiles, reason) {
+function moveDepToEarlierPR(options) {
+    const { dep, prIdx, sorted, fileToPRIndex, movedFiles, reason } = options;
     const depPRIdx = fileToPRIndex.get(dep);
     if (depPRIdx === undefined || depPRIdx <= prIdx) {
         return false;
@@ -229,7 +229,15 @@ export function autoFixPROrdering(pullRequests, dependencyGraph) {
             for (const file of sorted[prIdx].files ?? []) {
                 const transitiveDeps = getTransitiveDependencies(file.path, dependencyGraph);
                 for (const dep of transitiveDeps) {
-                    const moved = moveDepToEarlierPR(dep, prIdx, sorted, fileToPRIndex, movedFiles, `imported by ${file.path}`);
+                    const moved = moveDepToEarlierPR({
+                        dep,
+                        prIdx,
+                        sorted,
+                        fileToPRIndex,
+                        movedFiles,
+                        reason: `imported by ${file.path}`,
+                    });
+                    // eslint-disable-next-line max-depth
                     if (moved) {
                         changed = true;
                     }

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

@@ -27,7 +27,9 @@ async function* prompt(analysisPrompt) {
  * then uses AI to produce a PR split plan saved to the database.
  * Human review is expected before running the pr-execution phase.
  */
-export const splitFeatureIntoPRs = async (options, config) => {
+export const splitFeatureIntoPRs = async (options, config
+// eslint-disable-next-line complexity
+) => {
     const { featureId, verbose, replaceExisting } = options;
     if (verbose) {
         logInfo(`Starting PR splitting for feature ID: ${featureId}`);

package/dist/phases/release-sync/__tests__/github.test.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Unit tests for smoke-test github helpers.
+ *
+ * Covers the pure functions that shape GitHub compare data into a
+ * prompt-ready digest. Network-facing functions (fetchLatestTwoReleases,
+ * fetchCompare) are exercised only indirectly — their output shape is
+ * fed through buildDiffDigest / summariseStats here.
+ */
+export {};