Fix CLI entry detection for npx

Ensure the CLI runs when invoked via npx binary names or paths

Author: cezaraugusto

cli.test.ts

@@ -1,5 +1,7 @@
 import { describe, afterEach, test, expect, vi } from 'vitest';
-import cli from './src/cli.js';
+import path from 'path';
+import { pathToFileURL } from 'url';
+import cli, { shouldRunAsCli } from './src/cli.js';
 
 const originalArgv = process.argv;
 
@@ -78,3 +80,38 @@ describe('cli', () => {
     expect(errorMock).toHaveBeenCalled();
   });
 });
+
+describe('shouldRunAsCli', () => {
+  test('returns true for direct absolute entry path', () => {
+    const entryPath = path.join(process.cwd(), 'dist', 'index.js');
+    const importMetaUrl = pathToFileURL(entryPath).href;
+
+    expect(shouldRunAsCli(importMetaUrl, entryPath)).toBe(true);
+  });
+
+  test('returns true for relative entry path', () => {
+    const entryPath = path.join(process.cwd(), 'dist', 'index.js');
+    const importMetaUrl = pathToFileURL(entryPath).href;
+    const relativePath = path.relative(process.cwd(), entryPath);
+
+    expect(shouldRunAsCli(importMetaUrl, relativePath)).toBe(true);
+  });
+
+  test('returns true for binary name', () => {
+    const importMetaUrl = pathToFileURL('/tmp/fake/index.js').href;
+
+    expect(shouldRunAsCli(importMetaUrl, 'go-git-it')).toBe(true);
+  });
+
+  test('returns false for unrelated argv1', () => {
+    const importMetaUrl = pathToFileURL('/tmp/fake/index.js').href;
+
+    expect(shouldRunAsCli(importMetaUrl, '/usr/bin/node')).toBe(false);
+  });
+
+  test('returns false when argv1 is missing', () => {
+    const importMetaUrl = pathToFileURL('/tmp/fake/index.js').href;
+
+    expect(shouldRunAsCli(importMetaUrl, undefined)).toBe(false);
+  });
+});

src/cli.ts

@@ -1,7 +1,31 @@
 #!/usr/bin/env node
 
+import path from 'path';
+import { fileURLToPath } from 'url';
+
 type GoGitIt = (gitURL: string, outputDirectory?: string) => Promise<void>;
 
+export function shouldRunAsCli(
+  importMetaUrl: string,
+  argv1: string | undefined,
+): boolean {
+  if (!argv1) {
+    return false;
+  }
+
+  const entryPath = fileURLToPath(importMetaUrl);
+  const argvPath = path.isAbsolute(argv1)
+    ? argv1
+    : path.resolve(process.cwd(), argv1);
+
+  if (argvPath === entryPath) {
+    return true;
+  }
+
+  const base = path.basename(argv1);
+  return base === 'go-git-it' || base === 'go-git-it.js' || base === 'go-git-it.cjs';
+}
+
 /**
  * Enhanced CLI with proper error handling and help text
  */

src/index.ts

@@ -18,7 +18,7 @@ import {
   createDirectory,
   cleanupTempDirectory,
 } from './utils/cross-platform.js';
-import cli from './cli.js';
+import cli, { shouldRunAsCli } from './cli.js';
 import path from 'path';
 
 /**
@@ -100,8 +100,8 @@ async function goGitIt(
   }
 }
 
-// Execute CLI if requested
-if (import.meta.url === `file://${process.argv[1]}`) {
+// Execute CLI when invoked as a binary
+if (shouldRunAsCli(import.meta.url, process.argv[1])) {
   cli(goGitIt);
 }
 

src/types/progress.d.ts

@@ -0,0 +1 @@
+declare module 'progress';