fix: invoke by cli name

Author: AmirSa12

src/cac.ts

@@ -9,19 +9,10 @@ import t, { type RootCommand } from './t';
 
 const execPath = process.execPath;
 const processArgs = process.argv.slice(1);
-const quotedExecPath = quoteIfNeeded(execPath);
-const quotedProcessArgs = processArgs.map(quoteIfNeeded);
-const quotedProcessExecArgs = process.execArgv.map(quoteIfNeeded);
-
-const x = `${quotedExecPath} ${quotedProcessExecArgs.join(' ')} ${quotedProcessArgs[0]}`;
 
 // Regex to detect if an option takes a value (has <required> or [optional] parameters)
 const VALUE_OPTION_RE = /<[^>]+>|\[[^\]]+\]/;
 
-function quoteIfNeeded(path: string): string {
-  return path.includes(' ') ? `'${path}'` : path;
-}
-
 export default async function tab(
   instance: CAC,
   completionConfig?: CompletionConfig
@@ -122,24 +113,27 @@ export default async function tab(
   }
 
   instance.command('complete [shell]').action(async (shell, extra) => {
+    // invoke the cli by its program name so completion works regardless of how
+    // the cli is launched (including compiled binaries). see the related issue #135
+    const execCommand = instance.name;
     switch (shell) {
       case 'zsh': {
-        const script = zsh.generate(instance.name, x);
+        const script = zsh.generate(instance.name, execCommand);
         console.log(script);
         break;
       }
       case 'bash': {
-        const script = bash.generate(instance.name, x);
+        const script = bash.generate(instance.name, execCommand);
         console.log(script);
         break;
       }
       case 'fish': {
-        const script = fish.generate(instance.name, x);
+        const script = fish.generate(instance.name, execCommand);
         console.log(script);
         break;
       }
       case 'powershell': {
-        const script = powershell.generate(instance.name, x);
+        const script = powershell.generate(instance.name, execCommand);
         console.log(script);
         break;
       }

src/citty.ts

@@ -15,17 +15,6 @@ import { assertDoubleDashes } from './shared';
 import type { CompletionConfig } from './shared';
 import t, { type RootCommand } from './t';
 
-function quoteIfNeeded(path: string) {
-  return path.includes(' ') ? `'${path}'` : path;
-}
-
-const execPath = process.execPath;
-const processArgs = process.argv.slice(1);
-const quotedExecPath = quoteIfNeeded(execPath);
-const quotedProcessArgs = processArgs.map(quoteIfNeeded);
-const quotedProcessExecArgs = process.execArgv.map(quoteIfNeeded);
-const x = `${quotedExecPath} ${quotedProcessExecArgs.join(' ')} ${quotedProcessArgs[0]}`;
-
 function isConfigPositional<T extends ArgsDef>(config: CommandDef<T>) {
   return (
     config.args &&
@@ -200,24 +189,27 @@ export default async function tab<TArgs extends ArgsDef>(
         shell = undefined;
       }
 
+      // the cli by its program name so completion works regardless of
+      // how the cli is launched (including compiled binaries). see the related issue #135
+      const execCommand = name;
       switch (shell) {
         case 'zsh': {
-          const script = zsh.generate(name, x);
+          const script = zsh.generate(name, execCommand);
           console.log(script);
           break;
         }
         case 'bash': {
-          const script = bash.generate(name, x);
+          const script = bash.generate(name, execCommand);
           console.log(script);
           break;
         }
         case 'fish': {
-          const script = fish.generate(name, x);
+          const script = fish.generate(name, execCommand);
           console.log(script);
           break;
         }
         case 'powershell': {
-          const script = powershell.generate(name, x);
+          const script = powershell.generate(name, execCommand);
           console.log(script);
           break;
         }

src/commander.ts

@@ -6,18 +6,6 @@ interface CommandWithRawArgs extends CommanderCommand {
   rawArgs: string[];
 }
 
-const execPath = process.execPath;
-const processArgs = process.argv.slice(1);
-const quotedExecPath = quoteIfNeeded(execPath);
-const quotedProcessArgs = processArgs.map(quoteIfNeeded);
-const quotedProcessExecArgs = process.execArgv.map(quoteIfNeeded);
-
-const x = `${quotedExecPath} ${quotedProcessExecArgs.join(' ')} ${quotedProcessArgs[0]}`;
-
-function quoteIfNeeded(path: string): string {
-  return path.includes(' ') ? `'${path}'` : path;
-}
-
 export default function tab(
   instance: CommanderCommand,
   completionConfig?: { completionCommandName?: string }
@@ -36,7 +24,8 @@ export default function tab(
         .choices(['zsh', 'bash', 'fish', 'powershell'])
     )
     .action((shell) => {
-      t.setup(programName, x, shell);
+      // invoke the cli by its program name and not the runtime launch path. see the related issue #135.
+      t.setup(programName, programName, shell);
     });
   completionCommand.copyInheritedSettings(instance);