fix: skip auto-update in headless mode

Author: riccardogiorato

src/cli.ts

@@ -12,16 +12,18 @@ import modelCommand from './commands/model.js';
 import hookCommand, { isCalledFromGitHook } from './commands/hook.js';
 import prCommand from './commands/pr.js';
 import { checkAndAutoUpdate } from './utils/auto-update.js';
+import { isHeadless } from './utils/headless.js';
 
 // Auto-update check - runs in production to update under the hood
 // Skip during git hooks to avoid breaking commit flow
-if (!isCalledFromGitHook && version !== '0.0.0-semantic-release') {
+if (!isCalledFromGitHook && !isHeadless() && version !== '0.0.0-semantic-release') {
 	const distTag = version.includes('-') ? 'develop' : 'latest';
 
 	// Check for updates and auto-update if available
 	checkAndAutoUpdate({
 		pkg,
 		distTag,
+		headless: false,
 	});
 }
 

src/commands/aicommits.ts

@@ -24,8 +24,7 @@ import {
 import { KnownError, handleCommandError } from '../utils/error.js';
 
 import { getCommitMessage } from '../utils/commit-helpers.js';
-
-const isHeadless = () => !process.stdout.isTTY;
+import { isHeadless } from '../utils/headless.js';
 
 export default async (
 	generate: number | undefined,

src/utils/auto-update.ts

@@ -6,6 +6,7 @@ const execAsync = promisify(exec);
 export interface AutoUpdateOptions {
 	pkg: { name: string; version: string };
 	distTag?: string;
+	headless?: boolean;
 }
 
 // Parse version string into comparable parts
@@ -128,7 +129,11 @@ async function runBackgroundUpdate(
 export async function checkAndAutoUpdate(
 	options: AutoUpdateOptions
 ): Promise<void> {
-	const { pkg, distTag = 'latest' } = options;
+	const { pkg, distTag = 'latest', headless = false } = options;
+
+	if (headless) {
+		return;
+	}
 
 	// Skip for development/semantic-release versions
 	if (

src/utils/headless.ts

@@ -0,0 +1 @@
+export const isHeadless = () => !process.stdout.isTTY;

tests/index.ts

@@ -2,6 +2,7 @@ import { describe } from 'manten';
 
 describe('aicommits', ({ runTestSuite }) => {
 	runTestSuite(import('./specs/cli/index.js'));
+	runTestSuite(import('./specs/auto-update.js'));
 	runTestSuite(import('./specs/openai/index.js'));
 	runTestSuite(import('./specs/togetherai/index.js'));
 	runTestSuite(import('./specs/config.js'));

tests/specs/auto-update.ts

@@ -0,0 +1,38 @@
+import { testSuite, expect } from 'manten';
+import { checkAndAutoUpdate } from '../../src/utils/auto-update.js';
+
+export default testSuite(({ describe }) => {
+	describe('Auto update', ({ test }) => {
+		test('skips update checks entirely in headless mode', async () => {
+			const originalFetch = globalThis.fetch;
+			const originalConsoleLog = console.log;
+			const fetchCalls: string[] = [];
+			const consoleCalls: string[] = [];
+
+			globalThis.fetch = (async (input: string | URL | Request) => {
+				fetchCalls.push(String(input));
+				throw new Error('fetch should not be called in headless mode');
+			}) as typeof fetch;
+
+			console.log = (...args: unknown[]) => {
+				consoleCalls.push(args.join(' '));
+			};
+
+			try {
+				await checkAndAutoUpdate({
+					pkg: {
+						name: 'aicommits',
+						version: '1.0.0',
+					},
+					headless: true,
+				});
+			} finally {
+				globalThis.fetch = originalFetch;
+				console.log = originalConsoleLog;
+			}
+
+			expect(fetchCalls).toEqual([]);
+			expect(consoleCalls).toEqual([]);
+		});
+	});
+});