fix: types

Author: AmirSa12

bin/completion-handlers.ts

@@ -1,62 +1,5 @@
 // TODO: i do not see any completion functionality in this file. nothing is being provided for the defined commands of these package managers. this is a blocker for release. every each of them should be handled.
 import { Completion } from '../src/index.js';
-import { execSync } from 'child_process';
-
-const DEBUG = false; // for debugging purposes
-
-function debugLog(...args: any[]) {
-  if (DEBUG) {
-    console.error('[DEBUG]', ...args);
-  }
-}
-
-async function checkCliHasCompletions(
-  cliName: string,
-  packageManager: string
-): Promise<boolean> {
-  try {
-    debugLog(`Checking if ${cliName} has completions via ${packageManager}`);
-    const command = `${packageManager} ${cliName} complete --`;
-    const result = execSync(command, {
-      encoding: 'utf8',
-      stdio: ['pipe', 'pipe', 'ignore'],
-      timeout: 1000, // AMIR: we still havin issues with this, it still hangs if a cli doesn't have completions. longer timeout needed for shell completion system (shell → node → package manager → cli)
-    });
-    const hasCompletions = !!result.trim();
-    debugLog(`${cliName} supports completions: ${hasCompletions}`);
-    return hasCompletions;
-  } catch (error) {
-    debugLog(`Error checking completions for ${cliName}:`, error);
-    return false;
-  }
-}
-
-async function getCliCompletions(
-  cliName: string,
-  packageManager: string,
-  args: string[]
-): Promise<string[]> {
-  try {
-    const completeArgs = args.map((arg) =>
-      arg.includes(' ') ? `"${arg}"` : arg
-    );
-    const completeCommand = `${packageManager} ${cliName} complete -- ${completeArgs.join(' ')}`;
-    debugLog(`Getting completions with command: ${completeCommand}`);
-
-    const result = execSync(completeCommand, {
-      encoding: 'utf8',
-      stdio: ['pipe', 'pipe', 'ignore'],
-      timeout: 1000, // same: longer timeout needed for shell completion system (shell → node → package manager → cli)
-    });
-
-    const completions = result.trim().split('\n').filter(Boolean);
-    debugLog(`Got ${completions.length} completions from ${cliName}`);
-    return completions;
-  } catch (error) {
-    debugLog(`Error getting completions from ${cliName}:`, error);
-    return [];
-  }
-}
 
 export function setupCompletionForPackageManager(
   packageManager: string,

examples/demo.cac.ts

@@ -1,6 +1,6 @@
 import cac from 'cac';
 import tab from '../src/cac';
-import type { Command, Option, OptionsMap } from '../src/t';
+import type { Option, OptionsMap } from '../src/t';
 
 const cli = cac('vite');
 

examples/demo.t.ts

@@ -96,8 +96,7 @@ t.command('dev build', 'Build project');
 t.command('dev start', 'Start development server');
 
 // Copy command with multiple arguments
-const copyCmd = t
-  .command('copy', 'Copy files')
+t.command('copy', 'Copy files')
   .argument('source', function (complete) {
     complete('src/', 'Source directory');
     complete('dist/', 'Distribution directory');
@@ -110,7 +109,7 @@ const copyCmd = t
   });
 
 // Lint command with variadic arguments
-const lintCmd = t.command('lint', 'Lint project').argument(
+t.command('lint', 'Lint project').argument(
   'files',
   function (complete) {
     complete('main.ts', 'Main file');

src/t.ts

@@ -274,12 +274,10 @@ export class RootCommand extends Command {
   ) {
     // Handle flag value completion
     let optionName: string | undefined;
-    let valueToComplete = toComplete;
 
     if (toComplete.includes('=')) {
-      const [flag, value] = toComplete.split('=');
+      const [flag] = toComplete.split('=');
       optionName = flag;
-      valueToComplete = value || '';
     } else if (lastPrevArg?.startsWith('-')) {
       optionName = lastPrevArg;
     }
@@ -342,7 +340,7 @@ export class RootCommand extends Command {
     if (option) return option;
 
     // Try by short alias
-    for (const [name, opt] of command.options) {
+    for (const [_name, opt] of command.options) {
       if (opt.alias && `-${opt.alias}` === optionName) {
         return opt;
       }
@@ -391,7 +389,7 @@ export class RootCommand extends Command {
 
       if (currentArgIndex < argumentEntries.length) {
         // We're within the defined arguments
-        const [argName, argument] = argumentEntries[currentArgIndex];
+        const [_argName, argument] = argumentEntries[currentArgIndex];
         targetArgument = argument;
       } else {
         // We're beyond the defined arguments, check if the last argument is variadic