edsger 0.42.1 → 0.44.0

package/dist/phases/intelligence-analysis/__tests__/orchestration.test.js

@@ -110,26 +110,26 @@ function makeDeps(aiResult, contextOverride) {
         saveSnapCalls,
         saveRptCalls,
         updateRptCalls,
-        prepareContext: async () => ({
+        prepareContext: () => Promise.resolve({
             context: contextOverride || makeContext(),
             analysisPrompt: 'fake prompt',
         }),
-        executeQuery: async () => aiResult,
-        batchCreate: async (_productId, competitors) => {
+        executeQuery: () => Promise.resolve(aiResult),
+        batchCreate: (_productId, competitors) => {
             batchCreateCalls.push([_productId, competitors]);
-            return competitors.map((c) => makeCompetitor(c.name, `discovered-${batchCreateCounter++}`, 'suggested'));
+            return Promise.resolve(competitors.map((c) => makeCompetitor(c.name, `discovered-${batchCreateCounter++}`, 'suggested')));
         },
-        saveSnap: async (snap) => {
+        saveSnap: (snap) => {
             saveSnapCalls.push([snap]);
-            return makeSnapshot(snap.competitor_id, `snap-${saveSnapCalls.length}`);
+            return Promise.resolve(makeSnapshot(snap.competitor_id, `snap-${saveSnapCalls.length}`));
         },
-        saveRpt: async (report) => {
+        saveRpt: (report) => {
             saveRptCalls.push([report]);
-            return makeReport(`rpt-${saveRptCalls.length}`);
+            return Promise.resolve(makeReport(`rpt-${saveRptCalls.length}`));
         },
-        updateRpt: async (_id, updates) => {
+        updateRpt: (_id, updates) => {
             updateRptCalls.push([_id, updates]);
-            return makeReport(_id);
+            return Promise.resolve(makeReport(_id));
         },
     };
 }
@@ -368,8 +368,8 @@ void describe('analyseIntelligence — guidance', () => {
 void describe('analyseIntelligence — exception in deps', () => {
     void it('should return error when context fetch throws', async () => {
         const deps = makeDeps(null);
-        deps.prepareContext = async () => {
-            throw new Error('Network error');
+        deps.prepareContext = () => {
+            return Promise.reject(new Error('Network error'));
         };
         const result = await analyseIntelligence({ productId: 'prod-001' }, fakeConfig, deps);
         assert.strictEqual(result.status, 'error');

package/dist/phases/intelligence-analysis/agent.js

@@ -261,12 +261,14 @@ function parseIntelligenceResult(responseText) {
         error: 'JSON parsing failed: could not extract structured data from response',
     };
 }
+// eslint-disable-next-line @typescript-eslint/require-await
 async function* prompt(analysisPrompt) {
     yield {
         type: 'user',
         message: { role: 'user', content: analysisPrompt },
     };
 }
+// eslint-disable-next-line complexity
 export async function executeIntelligenceQuery(currentPrompt, systemPrompt, _config, verbose, cwd) {
     let lastAssistantResponse = '';
     let structuredResult = null;

package/dist/phases/intelligence-analysis/index.js

@@ -74,6 +74,7 @@ const defaultDeps = {
  * 2. AI agent researches competitors and market via web search
  * 3. Save discovered competitors, snapshots, and report
  */
+// eslint-disable-next-line complexity
 export async function analyseIntelligence(options, config, deps = defaultDeps) {
     const { productId, reportType = 'competitive', verbose, guidance, reportId, } = options;
     try {

package/dist/phases/intelligence-analysis/prompts.js

@@ -29,9 +29,19 @@ export const createIntelligencePromptWithContext = (productId, contextInfo, repo
 **Competitor Discovery**: While analyzing confirmed competitors, also look for new entrants or products that should be tracked. Include any new finds in the "discovered_competitors" array.`;
     }
     else {
+        let focusLabel;
+        if (reportType === 'research') {
+            focusLabel = 'papers and reports';
+        }
+        else if (reportType === 'market') {
+            focusLabel = 'market signals and trends';
+        }
+        else {
+            focusLabel = 'product-market fit metrics';
+        }
         discoveryInstructions = `
 
-**Note**: This is a ${reportType} analysis — focus on ${reportType === 'research' ? 'papers and reports' : reportType === 'market' ? 'market signals and trends' : 'product-market fit metrics'}. Do NOT discover new competitors. Set "discovered_competitors" to an empty array.`;
+**Note**: This is a ${reportType} analysis — focus on ${focusLabel}. Do NOT discover new competitors. Set "discovered_competitors" to an empty array.`;
     }
     const researchInstructions = reportType === 'research'
         ? `

package/dist/phases/output-contracts.js

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 /**
  * Output format contracts for each phase.
  * These define the JSON structure that LLM responses must follow.

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

@@ -20,8 +20,8 @@ function makePR(overrides) {
         ...overrides,
     };
 }
-function cf(path, change_type = 'modified') {
-    return { path, change_type };
+function cf(path, changeType = 'modified') {
+    return { path, change_type: changeType };
 }
 // ============================================================
 // findUnassignedFiles
@@ -133,8 +133,9 @@ void describe('applyAssignments', () => {
         ];
         const unassigned = [cf('b.ts', 'added')];
         const updates = [];
-        const mockUpdater = async (prId, u) => {
+        const mockUpdater = (prId, u) => {
             updates.push({ prId, files: u.files });
+            return Promise.resolve();
         };
         const count = await applyAssignments([{ file: 'b.ts', pr_id: 'pr-1' }], unassigned, prs, mockUpdater);
         assert.strictEqual(count, 1);
@@ -155,8 +156,9 @@ void describe('applyAssignments', () => {
         ];
         const unassigned = [cf('a.ts')];
         const updates = [];
-        const mockUpdater = async (_prId, u) => {
+        const mockUpdater = (_prId, u) => {
             updates.push({ files: u.files });
+            return Promise.resolve();
         };
         await applyAssignments([{ file: 'a.ts', pr_id: 'pr-1' }], unassigned, prs, mockUpdater);
         // File list should not have duplicates
@@ -168,8 +170,9 @@ void describe('applyAssignments', () => {
         const prs = [makePR({ id: 'pr-1', sequence: 1, files: [] })];
         const unassigned = [cf('x.ts', 'added')];
         let called = false;
-        const mockUpdater = async () => {
+        const mockUpdater = () => {
             called = true;
+            return Promise.resolve();
         };
         const count = await applyAssignments([{ file: 'x.ts', pr_id: 'unknown-id' }], unassigned, prs, mockUpdater);
         assert.strictEqual(count, 0);
@@ -179,8 +182,9 @@ void describe('applyAssignments', () => {
         const prs = [makePR({ id: 'pr-1', sequence: 1, files: [] })];
         const unassigned = [cf('new.ts', 'added')];
         const updates = [];
-        const mockUpdater = async (_prId, u) => {
+        const mockUpdater = (_prId, u) => {
             updates.push({ files: u.files });
+            return Promise.resolve();
         };
         await applyAssignments([{ file: 'new.ts', pr_id: 'pr-1' }], unassigned, prs, mockUpdater);
         assert.strictEqual(updates[0].files[0].change_type, 'added');
@@ -195,8 +199,9 @@ void describe('applyAssignments', () => {
             cf('b.ts', 'modified'),
         ];
         const updates = [];
-        const mockUpdater = async (prId) => {
+        const mockUpdater = (prId) => {
             updates.push({ prId });
+            return Promise.resolve();
         };
         const count = await applyAssignments([
             { file: 'a.ts', pr_id: 'pr-1' },
@@ -208,8 +213,8 @@ void describe('applyAssignments', () => {
     void it('handles updater failure gracefully', async () => {
         const prs = [makePR({ id: 'pr-1', sequence: 1, files: [] })];
         const unassigned = [cf('a.ts')];
-        const mockUpdater = async () => {
-            throw new Error('DB error');
+        const mockUpdater = () => {
+            return Promise.reject(new Error('DB error'));
         };
         const count = await applyAssignments([{ file: 'a.ts', pr_id: 'pr-1' }], unassigned, prs, mockUpdater);
         assert.strictEqual(count, 0);
@@ -235,8 +240,9 @@ void describe('removeDeletedFilesFromPRs', () => {
             cf('gone.ts', 'deleted'),
         ];
         const updates = [];
-        const mockUpdater = async (prId, u) => {
+        const mockUpdater = (prId, u) => {
             updates.push({ prId, files: u.files });
+            return Promise.resolve();
         };
         const count = await removeDeletedFilesFromPRs(changedFiles, prs, mockUpdater);
         assert.strictEqual(count, 1);
@@ -253,8 +259,9 @@ void describe('removeDeletedFilesFromPRs', () => {
             }),
         ];
         let called = false;
-        const mockUpdater = async () => {
+        const mockUpdater = () => {
             called = true;
+            return Promise.resolve();
         };
         const count = await removeDeletedFilesFromPRs([cf('a.ts')], prs, mockUpdater);
         assert.strictEqual(count, 0);
@@ -274,8 +281,9 @@ void describe('removeDeletedFilesFromPRs', () => {
             }),
         ];
         const updates = [];
-        const mockUpdater = async (prId) => {
+        const mockUpdater = (prId) => {
             updates.push(prId);
+            return Promise.resolve();
         };
         await removeDeletedFilesFromPRs([cf('b.ts', 'deleted')], prs, mockUpdater);
         // Only pr-2 should be updated
@@ -284,8 +292,9 @@ void describe('removeDeletedFilesFromPRs', () => {
     void it('handles PRs with null files', async () => {
         const prs = [makePR({ id: 'pr-1', sequence: 1, files: null })];
         let called = false;
-        const mockUpdater = async () => {
+        const mockUpdater = () => {
             called = true;
+            return Promise.resolve();
         };
         const count = await removeDeletedFilesFromPRs([cf('a.ts', 'deleted')], prs, mockUpdater);
         assert.strictEqual(count, 0);

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

@@ -193,7 +193,9 @@ export async function fetchPRExecutionContext(featureId, verbose) {
         // If the branch was rebased, lastSyncedCommit is no longer in the history.
         // Treat this as a full re-execution so downstream logic uses the correct
         // prompt and skips incremental file-assignment against an invalid base.
-        if (isIncrementalSync && !isAncestor(lastSyncedCommit, devRef)) {
+        if (isIncrementalSync &&
+            lastSyncedCommit &&
+            !isAncestor(lastSyncedCommit, devRef)) {
             if (verbose) {
                 logInfo(`⚠️  Last synced commit ${lastSyncedCommit} is no longer in branch history (likely rebased). Treating as full re-execution.`);
             }
@@ -201,7 +203,7 @@ export async function fetchPRExecutionContext(featureId, verbose) {
             isIncrementalSync = false;
         }
         // Get diff info: for incremental, diff from last sync; for first run, diff from main
-        const diffBase = isIncrementalSync ? lastSyncedCommit : 'main';
+        const diffBase = isIncrementalSync && lastSyncedCommit ? lastSyncedCommit : 'main';
         const diffStat = getDiffStat(diffBase, devRef);
         const changedFiles = getChangedFiles(diffBase, devRef);
         if (verbose) {

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

@@ -97,6 +97,7 @@ async function callLLMForAssignment(prompt, verbose) {
         })) {
             if (message.type === 'assistant' && message.message?.content) {
                 for (const item of message.message.content) {
+                    // eslint-disable-next-line max-depth
                     if (item.type === 'text') {
                         responseText += `${item.text}\n`;
                     }

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

@@ -26,8 +26,8 @@ function makeMap(entries) {
     return new Map(entries);
 }
 // ── buildLearnerPrompt ──────────────────────────────────────
-describe('buildLearnerPrompt', () => {
-    it('includes addressed comment count in header', () => {
+void describe('buildLearnerPrompt', () => {
+    void it('includes addressed comment count in header', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Fixed' },
             { comment_id: 'comment_2', action: 'changed', reply: 'Done' },
@@ -40,7 +40,7 @@ describe('buildLearnerPrompt', () => {
         const prompt = buildLearnerPrompt(comments, threads, map);
         assert.ok(prompt.includes('2 review comment(s)'));
     });
-    it('includes reviewer, file path, line, and comment body', () => {
+    void it('includes reviewer, file path, line, and comment body', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Added null check' },
         ];
@@ -60,7 +60,7 @@ describe('buildLearnerPrompt', () => {
         assert.ok(prompt.includes('Missing null check here'));
         assert.ok(prompt.includes('Added null check'));
     });
-    it('includes resolution text for each comment', () => {
+    void it('includes resolution text for each comment', () => {
         const comments = [
             {
                 comment_id: 'comment_1',
@@ -73,7 +73,7 @@ describe('buildLearnerPrompt', () => {
         const prompt = buildLearnerPrompt(comments, threads, map);
         assert.ok(prompt.includes('**Resolution**: Refactored to use try/catch'));
     });
-    it('includes summary when provided', () => {
+    void it('includes summary when provided', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Fixed' },
         ];
@@ -83,7 +83,7 @@ describe('buildLearnerPrompt', () => {
         assert.ok(prompt.includes('## Overall Summary'));
         assert.ok(prompt.includes('Improved error handling across 3 files'));
     });
-    it('omits summary section when not provided', () => {
+    void it('omits summary section when not provided', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Fixed' },
         ];
@@ -92,7 +92,7 @@ describe('buildLearnerPrompt', () => {
         const prompt = buildLearnerPrompt(comments, threads, map);
         assert.ok(!prompt.includes('## Overall Summary'));
     });
-    it('handles comment with no matching thread gracefully', () => {
+    void it('handles comment with no matching thread gracefully', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Fixed' },
         ];
@@ -105,7 +105,7 @@ describe('buildLearnerPrompt', () => {
         // Should NOT include reviewer info since thread was not found
         assert.ok(!prompt.includes('**Reviewer**'));
     });
-    it('handles comment_id not in map gracefully', () => {
+    void it('handles comment_id not in map gracefully', () => {
         const comments = [
             { comment_id: 'comment_99', action: 'changed', reply: 'Fixed' },
         ];
@@ -113,7 +113,7 @@ describe('buildLearnerPrompt', () => {
         assert.ok(prompt.includes('## comment_99'));
         assert.ok(prompt.includes('**Resolution**: Fixed'));
     });
-    it('omits line when null', () => {
+    void it('omits line when null', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Fixed' },
         ];
@@ -123,7 +123,7 @@ describe('buildLearnerPrompt', () => {
         assert.ok(prompt.includes('**File**: src/index.ts'));
         assert.ok(!prompt.includes('**Line**'));
     });
-    it('uses threadById map correctly for multiple comments', () => {
+    void it('uses threadById map correctly for multiple comments', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Fix A' },
             { comment_id: 'comment_3', action: 'changed', reply: 'Fix C' },
@@ -146,7 +146,7 @@ describe('buildLearnerPrompt', () => {
         // comment_2 was not in addressedComments, should not appear
         assert.ok(!prompt.includes('Issue B'));
     });
-    it('includes instructions section', () => {
+    void it('includes instructions section', () => {
         const comments = [
             { comment_id: 'comment_1', action: 'changed', reply: 'Fixed' },
         ];

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

@@ -34,26 +34,26 @@ function makeThread(id, overrides) {
         },
     };
 }
-describe('createResolveSystemPrompt', () => {
-    it('includes decision criteria', () => {
+void describe('createResolveSystemPrompt', () => {
+    void it('includes decision criteria', () => {
         const prompt = createResolveSystemPrompt();
         assert.ok(prompt.includes('Make the change when'));
         assert.ok(prompt.includes('Skip the change when'));
     });
-    it('specifies comment_id format', () => {
+    void it('specifies comment_id format', () => {
         const prompt = createResolveSystemPrompt();
         assert.ok(prompt.includes('comment_id'));
         assert.ok(prompt.includes('comment_1'));
     });
-    it('includes result format', () => {
+    void it('includes result format', () => {
         const prompt = createResolveSystemPrompt();
         assert.ok(prompt.includes('resolve_result'));
         assert.ok(prompt.includes('"action"'));
         assert.ok(prompt.includes('"reply"'));
     });
 });
-describe('createResolveUserPrompt', () => {
-    it('uses sequential comment IDs not thread IDs', () => {
+void describe('createResolveUserPrompt', () => {
+    void it('uses sequential comment IDs not thread IDs', () => {
         const threads = [makeThread('PRRT_kwDOxx_1'), makeThread('PRRT_kwDOxx_2')];
         const { prompt, commentIdToThreadId } = createResolveUserPrompt(threads);
         // Should use comment_1, comment_2 in the prompt
@@ -67,20 +67,20 @@ describe('createResolveUserPrompt', () => {
         assert.strictEqual(commentIdToThreadId.get('comment_2'), 'PRRT_kwDOxx_2');
         assert.strictEqual(commentIdToThreadId.size, 2);
     });
-    it('includes file path and line number', () => {
+    void it('includes file path and line number', () => {
         const threads = [makeThread('t1', { path: 'src/auth.ts', line: 42 })];
         const { prompt } = createResolveUserPrompt(threads);
         assert.ok(prompt.includes('src/auth.ts'));
         assert.ok(prompt.includes('42'));
     });
-    it('includes comment body', () => {
+    void it('includes comment body', () => {
         const threads = [
             makeThread('t1', { body: 'This should use a const instead of let' }),
         ];
         const { prompt } = createResolveUserPrompt(threads);
         assert.ok(prompt.includes('This should use a const instead of let'));
     });
-    it('includes follow-up comments', () => {
+    void it('includes follow-up comments', () => {
         const threads = [
             makeThread('t1', {
                 body: 'Main comment',
@@ -92,12 +92,12 @@ describe('createResolveUserPrompt', () => {
         assert.ok(prompt.includes('I disagree because...'));
         assert.ok(prompt.includes('@dev'));
     });
-    it('includes instruction to use exact comment IDs', () => {
+    void it('includes instruction to use exact comment IDs', () => {
         const threads = [makeThread('t1'), makeThread('t2'), makeThread('t3')];
         const { prompt } = createResolveUserPrompt(threads);
         assert.ok(prompt.includes('comment_1, comment_2, comment_3'));
     });
-    it('handles threads with no comments gracefully', () => {
+    void it('handles threads with no comments gracefully', () => {
         const emptyThread = {
             id: 'empty',
             isResolved: false,
@@ -108,7 +108,7 @@ describe('createResolveUserPrompt', () => {
         // Empty thread should be skipped (no comment nodes to index)
         assert.strictEqual(commentIdToThreadId.size, 0);
     });
-    it('returns correct count in header', () => {
+    void it('returns correct count in header', () => {
         const threads = [makeThread('t1'), makeThread('t2')];
         const { prompt } = createResolveUserPrompt(threads);
         assert.ok(prompt.includes('2 unresolved review comment(s)'));

package/dist/phases/pr-resolve/__tests__/resolve-mapping.test.js

@@ -45,8 +45,8 @@ function makeThread(id, body) {
         },
     };
 }
-describe('resolve comment→thread mapping (integration)', () => {
-    it('correctly maps comment_ids to thread IDs', () => {
+void describe('resolve comment→thread mapping (integration)', () => {
+    void it('correctly maps comment_ids to thread IDs', () => {
         const threads = [
             makeThread('PRRT_aaa', 'Use const'),
             makeThread('PRRT_bbb', 'Add error handling'),
@@ -75,7 +75,7 @@ describe('resolve comment→thread mapping (integration)', () => {
         assert.strictEqual(result.addressed[1].threadId, 'PRRT_bbb');
         assert.strictEqual(result.skipped[0].threadId, 'PRRT_ccc');
     });
-    it('reports errors for unknown comment_ids', () => {
+    void it('reports errors for unknown comment_ids', () => {
         const threads = [makeThread('PRRT_aaa', 'Fix bug')];
         const { commentIdToThreadId } = createResolveUserPrompt(threads);
         const agentResult = [
@@ -87,7 +87,7 @@ describe('resolve comment→thread mapping (integration)', () => {
         assert.strictEqual(result.errors.length, 1);
         assert.ok(result.errors[0].includes('comment_99'));
     });
-    it('handles agent returning partial results', () => {
+    void it('handles agent returning partial results', () => {
         const threads = [
             makeThread('PRRT_aaa', 'Fix A'),
             makeThread('PRRT_bbb', 'Fix B'),
@@ -105,7 +105,7 @@ describe('resolve comment→thread mapping (integration)', () => {
         assert.strictEqual(result.errors.length, 0);
         // comment_2 was not mentioned - no error, just not processed
     });
-    it('handles empty agent result', () => {
+    void it('handles empty agent result', () => {
         const threads = [makeThread('PRRT_aaa', 'Fix')];
         const { commentIdToThreadId } = createResolveUserPrompt(threads);
         const result = processResolveResult([], commentIdToThreadId);
@@ -113,7 +113,7 @@ describe('resolve comment→thread mapping (integration)', () => {
         assert.strictEqual(result.skipped.length, 0);
         assert.strictEqual(result.errors.length, 0);
     });
-    it('preserves reply text in all cases', () => {
+    void it('preserves reply text in all cases', () => {
         const threads = [
             makeThread('PRRT_aaa', 'Fix this'),
             makeThread('PRRT_bbb', 'Change that'),

package/dist/phases/pr-resolve/__tests__/types.test.js

@@ -1,43 +1,43 @@
 import assert from 'node:assert';
 import { describe, it } from 'node:test';
 import { isResolveResult } from '../types.js';
-describe('isResolveResult', () => {
-    it('returns true for valid ResolveResult', () => {
+void describe('isResolveResult', () => {
+    void it('returns true for valid ResolveResult', () => {
         assert.ok(isResolveResult({
             comments: [
                 { comment_id: 'comment_1', action: 'changed', reply: 'Fixed' },
             ],
         }));
     });
-    it('returns true when optional fields are present', () => {
+    void it('returns true when optional fields are present', () => {
         assert.ok(isResolveResult({
             comments: [],
             files_modified: ['a.ts'],
             summary: 'Done',
         }));
     });
-    it('returns true for empty comments array', () => {
+    void it('returns true for empty comments array', () => {
         assert.ok(isResolveResult({ comments: [] }));
     });
-    it('returns false for null', () => {
+    void it('returns false for null', () => {
         assert.ok(!isResolveResult(null));
     });
-    it('returns false for undefined', () => {
+    void it('returns false for undefined', () => {
         assert.ok(!isResolveResult(undefined));
     });
-    it('returns false for string', () => {
+    void it('returns false for string', () => {
         assert.ok(!isResolveResult('not an object'));
     });
-    it('returns false for number', () => {
+    void it('returns false for number', () => {
         assert.ok(!isResolveResult(42));
     });
-    it('returns false when comments is missing', () => {
+    void it('returns false when comments is missing', () => {
         assert.ok(!isResolveResult({ files_modified: ['a.ts'] }));
     });
-    it('returns false when comments is not an array', () => {
+    void it('returns false when comments is not an array', () => {
         assert.ok(!isResolveResult({ comments: 'not-array' }));
     });
-    it('returns false for empty object', () => {
+    void it('returns false for empty object', () => {
         assert.ok(!isResolveResult({}));
     });
 });

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

@@ -16,8 +16,8 @@ function createTempRepo() {
     execSync('git commit -m "init"', { cwd: dir, stdio: 'pipe' });
     return dir;
 }
-describe('buildCredentialArgs', () => {
-    it('returns 4 args with credential helper config', () => {
+void describe('buildCredentialArgs', () => {
+    void it('returns 4 args with credential helper config', () => {
         const args = buildCredentialArgs('my-token');
         assert.strictEqual(args.length, 4);
         assert.strictEqual(args[0], '-c');
@@ -26,12 +26,12 @@ describe('buildCredentialArgs', () => {
         assert.ok(args[3].includes('my-token'));
         assert.ok(args[3].includes('x-access-token'));
     });
-    it('escapes token in credential helper', () => {
+    void it('escapes token in credential helper', () => {
         const args = buildCredentialArgs('token-with-special-chars!@#');
         assert.ok(args[3].includes('token-with-special-chars!@#'));
     });
 });
-describe('hasUncommittedChanges', () => {
+void describe('hasUncommittedChanges', () => {
     let repoPath;
     beforeEach(() => {
         repoPath = createTempRepo();
@@ -39,18 +39,18 @@ describe('hasUncommittedChanges', () => {
     afterEach(() => {
         rmSync(repoPath, { recursive: true, force: true });
     });
-    it('returns false for clean repo', () => {
+    void it('returns false for clean repo', () => {
         assert.strictEqual(hasUncommittedChanges(repoPath), false);
     });
-    it('returns true after modifying a file', () => {
+    void it('returns true after modifying a file', () => {
         writeFileSync(join(repoPath, 'README.md'), '# Modified');
         assert.strictEqual(hasUncommittedChanges(repoPath), true);
     });
-    it('returns true for new untracked file', () => {
+    void it('returns true for new untracked file', () => {
         writeFileSync(join(repoPath, 'new-file.txt'), 'content');
         assert.strictEqual(hasUncommittedChanges(repoPath), true);
     });
-    it('returns false after staging and committing', () => {
+    void it('returns false after staging and committing', () => {
         writeFileSync(join(repoPath, 'new.txt'), 'x');
         execSync('git add . && git commit -m "add"', {
             cwd: repoPath,
@@ -58,11 +58,11 @@ describe('hasUncommittedChanges', () => {
         });
         assert.strictEqual(hasUncommittedChanges(repoPath), false);
     });
-    it('returns false for non-existent path', () => {
+    void it('returns false for non-existent path', () => {
         assert.strictEqual(hasUncommittedChanges('/tmp/nonexistent-repo-xyz'), false);
     });
 });
-describe('hasNewCommits', () => {
+void describe('hasNewCommits', () => {
     let repoPath;
     let bareRemote;
     beforeEach(() => {
@@ -89,10 +89,10 @@ describe('hasNewCommits', () => {
         rmSync(repoPath, { recursive: true, force: true });
         rmSync(bareRemote, { recursive: true, force: true });
     });
-    it('returns false when HEAD matches remote', () => {
+    void it('returns false when HEAD matches remote', () => {
         assert.strictEqual(hasNewCommits(repoPath, 'main'), false);
     });
-    it('returns true after a local commit not pushed', () => {
+    void it('returns true after a local commit not pushed', () => {
         writeFileSync(join(repoPath, 'new.txt'), 'hello');
         execSync('git add . && git commit -m "local"', {
             cwd: repoPath,
@@ -100,7 +100,7 @@ describe('hasNewCommits', () => {
         });
         assert.strictEqual(hasNewCommits(repoPath, 'main'), true);
     });
-    it('returns false after pushing local commit', () => {
+    void it('returns false after pushing local commit', () => {
         writeFileSync(join(repoPath, 'new.txt'), 'hello');
         execSync('git add . && git commit -m "local" && git push origin main', {
             cwd: repoPath,