update

Author: AmirSa12

bin/cli.ts

@@ -37,19 +37,9 @@ async function main() {
       dashIndex !== -1 && (!isPowerShell || dashIndex < process.argv.length - 1)
         ? process.argv.slice(dashIndex + 1)
         : isPowerShell
-          ? (() => {
-              const fallback = args.slice(2);
-              // PowerShell shim can drop both the separator and empty placeholders.
-              // Add two empty strings to preserve cursor-position intent.
-              fallback.push('', '');
-              return fallback;
-            })()
+          ? args.slice(2)
           : null;
 
-    if (process.env.TAB_DEBUG) {
-      console.error('COMPLETION ARGS:', completionArgs);
-    }
-
     if (!completionArgs) {
       console.error(`Error: Expected '--' followed by command to complete`);
       process.exit(1);

bin/package-manager-completion.ts

@@ -7,16 +7,9 @@ import { RootCommand } from '../src/t.js';
 
 const noop = () => {};
 
-const isTabDebug = Boolean(process.env.TAB_DEBUG);
 const isPowerShellEnv =
   process.platform === 'win32' && Boolean(process.env.PSModulePath);
 
-function tabDebugLog(...args: unknown[]) {
-  if (isTabDebug) {
-    console.error('[TAB_DEBUG]', ...args);
-  }
-}
-
 function debugLog(...args: unknown[]) {
   if (process.env.DEBUG) {
     console.error('[DEBUG]', ...args);
@@ -38,7 +31,6 @@ function runCompletionCommand(
     completionArgs.length === 0 ? [''] : completionArgs;
 
   const args = [...leadingArgs, 'complete', '--', ...safeCompletionArgs];
-  tabDebugLog('spawn completion:', command, args);
 
   // PowerShell: prefer invoking via powershell.exe so .ps1 shims are resolved.
   if (isPowerShellEnv && path.extname(command) === '') {
@@ -47,12 +39,6 @@ function runCompletionCommand(
 
   const result = spawnSync(command, args, completionSpawnOptions);
 
-  tabDebugLog('spawn result:', {
-    status: result.status,
-    error: result.error?.message,
-    stdout: result.stdout,
-  });
-
   // If the direct spawn fails on Windows, retry via PowerShell as a safety net.
   if (
     getErrnoCode(result.error) === 'ENOENT' &&
@@ -84,8 +70,6 @@ function runCompletionViaPowerShell(command: string, args: string[]): string {
   const psArgs = args.map(powerShellQuote);
   const psCommand = `& ${command} ${psArgs.join(' ')}`.trimEnd();
 
-  tabDebugLog('powershell exec:', psCommand);
-
   const psResult = spawnSync(
     'powershell.exe',
     [
@@ -99,12 +83,6 @@ function runCompletionViaPowerShell(command: string, args: string[]): string {
     completionSpawnOptions
   );
 
-  tabDebugLog('powershell exec result:', {
-    status: psResult.status,
-    error: psResult.error?.message,
-    stdout: psResult.stdout,
-  });
-
   if (psResult.error) {
     throw psResult.error;
   }
@@ -132,18 +110,13 @@ async function checkCliHasCompletions(
     const result = runCompletionCommand(cliName, [], []);
     if (result) return true;
   } catch {
-    tabDebugLog('checkCliHasCompletions direct call failed for', cliName);
     noop();
   }
 
   try {
     const result = runCompletionCommand(packageManager, [cliName], []);
     return !!result;
   } catch {
-    tabDebugLog(
-      'checkCliHasCompletions package-manager call failed for',
-      cliName
-    );
     return false;
   }
 }
@@ -159,19 +132,13 @@ async function getCliCompletions(
       return result.split('\n').filter(Boolean);
     }
   } catch {
-    tabDebugLog('getCliCompletions direct call failed for', cliName, args);
     noop();
   }
 
   try {
     const result = runCompletionCommand(packageManager, [cliName], args);
     return result.split('\n').filter(Boolean);
   } catch {
-    tabDebugLog(
-      'getCliCompletions package-manager call failed for',
-      cliName,
-      args
-    );
     return [];
   }
 }