command-stream 0.8.1 → 0.8.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "command-stream",
-  "version": "0.8.1",
+  "version": "0.8.3",
   "description": "Modern $ shell utility library with streaming, async iteration, and EventEmitter support, optimized for Bun runtime",
   "type": "module",
   "main": "src/$.mjs",

package/src/$.mjs

@@ -12,14 +12,30 @@ import { parseShellCommand, needsRealShell } from './shell-parser.mjs';
 
 const isBun = typeof globalThis.Bun !== 'undefined';
 
-const VERBOSE =
-  process.env.COMMAND_STREAM_VERBOSE === 'true' || process.env.CI === 'true';
-
 // Trace function for verbose logging
-function trace(category, messageOrFunc) {
+// Can be controlled via COMMAND_STREAM_VERBOSE or COMMAND_STREAM_TRACE env vars
+// Can be disabled per-command via trace: false option
+// CI environment no longer auto-enables tracing
+function trace(category, messageOrFunc, runner = null) {
+  // Check if runner explicitly disabled tracing
+  if (runner && runner.options && runner.options.trace === false) {
+    return;
+  }
+
+  // Check global trace setting (evaluated dynamically for runtime changes)
+  const TRACE_ENV = process.env.COMMAND_STREAM_TRACE;
+  const VERBOSE_ENV = process.env.COMMAND_STREAM_VERBOSE === 'true';
+
+  // COMMAND_STREAM_TRACE=false explicitly disables tracing even if COMMAND_STREAM_VERBOSE=true
+  // COMMAND_STREAM_TRACE=true explicitly enables tracing
+  // Otherwise, use COMMAND_STREAM_VERBOSE
+  const VERBOSE =
+    TRACE_ENV === 'false' ? false : TRACE_ENV === 'true' ? true : VERBOSE_ENV;
+
   if (!VERBOSE) {
     return;
   }
+
   const message =
     typeof messageOrFunc === 'function' ? messageOrFunc() : messageOrFunc;
   const timestamp = new Date().toISOString();
@@ -42,7 +58,40 @@ function findAvailableShell() {
     return cachedShell;
   }
 
-  const shellsToTry = [
+  const isWindows = process.platform === 'win32';
+
+  // Windows-specific shells
+  const windowsShells = [
+    // Git Bash is the most Unix-compatible option on Windows
+    // Check common installation paths
+    {
+      cmd: 'C:\\Program Files\\Git\\bin\\bash.exe',
+      args: ['-c'],
+      checkPath: true,
+    },
+    {
+      cmd: 'C:\\Program Files\\Git\\usr\\bin\\bash.exe',
+      args: ['-c'],
+      checkPath: true,
+    },
+    {
+      cmd: 'C:\\Program Files (x86)\\Git\\bin\\bash.exe',
+      args: ['-c'],
+      checkPath: true,
+    },
+    // Git Bash via PATH (if added to PATH by user)
+    { cmd: 'bash.exe', args: ['-c'], checkPath: false },
+    // WSL bash as fallback
+    { cmd: 'wsl.exe', args: ['bash', '-c'], checkPath: false },
+    // PowerShell as last resort (different syntax for commands)
+    { cmd: 'powershell.exe', args: ['-Command'], checkPath: false },
+    { cmd: 'pwsh.exe', args: ['-Command'], checkPath: false },
+    // cmd.exe as final fallback
+    { cmd: 'cmd.exe', args: ['/c'], checkPath: false },
+  ];
+
+  // Unix-specific shells
+  const unixShells = [
     // Try absolute paths first (most reliable)
     { cmd: '/bin/sh', args: ['-l', '-c'], checkPath: true },
     { cmd: '/usr/bin/sh', args: ['-l', '-c'], checkPath: true },
@@ -71,6 +120,9 @@ function findAvailableShell() {
     { cmd: 'zsh', args: ['-l', '-c'], checkPath: false },
   ];
 
+  // Select shells based on platform
+  const shellsToTry = isWindows ? windowsShells : unixShells;
+
   for (const shell of shellsToTry) {
     try {
       if (shell.checkPath) {
@@ -84,12 +136,15 @@ function findAvailableShell() {
           return cachedShell;
         }
       } else {
-        // Try to execute 'which' to check if command is in PATH
-        const result = cp.spawnSync('which', [shell.cmd], {
+        // On Windows, use 'where' instead of 'which'
+        const whichCmd = isWindows ? 'where' : 'which';
+        const result = cp.spawnSync(whichCmd, [shell.cmd], {
           encoding: 'utf-8',
+          // On Windows, we need shell: true for 'where' to work
+          shell: isWindows,
         });
         if (result.status === 0 && result.stdout) {
-          const shellPath = result.stdout.trim();
+          const shellPath = result.stdout.trim().split('\n')[0]; // Take first result
           trace(
             'ShellDetection',
             () => `Found shell in PATH: ${shell.cmd} => ${shellPath}`
@@ -100,15 +155,27 @@ function findAvailableShell() {
       }
     } catch (e) {
       // Continue to next shell option
+      trace(
+        'ShellDetection',
+        () => `Failed to check shell ${shell.cmd}: ${e.message}`
+      );
     }
   }
 
-  // Final fallback - use absolute path to sh
-  trace(
-    'ShellDetection',
-    () => 'WARNING: No shell found, using /bin/sh as fallback'
-  );
-  cachedShell = { cmd: '/bin/sh', args: ['-l', '-c'] };
+  // Final fallback based on platform
+  if (isWindows) {
+    trace(
+      'ShellDetection',
+      () => 'WARNING: No shell found, using cmd.exe as fallback on Windows'
+    );
+    cachedShell = { cmd: 'cmd.exe', args: ['/c'] };
+  } else {
+    trace(
+      'ShellDetection',
+      () => 'WARNING: No shell found, using /bin/sh as fallback'
+    );
+    cachedShell = { cmd: '/bin/sh', args: ['-l', '-c'] };
+  }
   return cachedShell;
 }
 

package/src/$.utils.mjs

@@ -1,11 +1,23 @@
 import path from 'path';
 
-const VERBOSE = process.env.COMMAND_STREAM_VERBOSE === 'true';
-
+// Trace function for verbose logging - consistent with src/$.mjs
+// Can be controlled via COMMAND_STREAM_VERBOSE or COMMAND_STREAM_TRACE env vars
+// CI environment no longer auto-enables tracing
 export function trace(category, messageOrFunc) {
+  // Check global trace setting (evaluated dynamically for runtime changes)
+  const TRACE_ENV = process.env.COMMAND_STREAM_TRACE;
+  const VERBOSE_ENV = process.env.COMMAND_STREAM_VERBOSE === 'true';
+
+  // COMMAND_STREAM_TRACE=false explicitly disables tracing even if COMMAND_STREAM_VERBOSE=true
+  // COMMAND_STREAM_TRACE=true explicitly enables tracing
+  // Otherwise, use COMMAND_STREAM_VERBOSE
+  const VERBOSE =
+    TRACE_ENV === 'false' ? false : TRACE_ENV === 'true' ? true : VERBOSE_ENV;
+
   if (!VERBOSE) {
     return;
   }
+
   const message =
     typeof messageOrFunc === 'function' ? messageOrFunc() : messageOrFunc;
   const timestamp = new Date().toISOString();

package/src/commands/$.cp.mjs

@@ -2,7 +2,7 @@ import fs from 'fs';
 import path from 'path';
 import { trace, VirtualUtils } from '../$.utils.mjs';
 
-export default async function cp({ args, stdin, cwd }) {
+export default async function cp({ args, stdin: _stdin, cwd }) {
   const argError = VirtualUtils.validateArgs(args, 2, 'cp');
   if (argError) {
     return VirtualUtils.invalidArgumentError(

package/src/commands/$.env.mjs

@@ -1,6 +1,6 @@
 import { VirtualUtils } from '../$.utils.mjs';
 
-export default async function env({ args, stdin, env }) {
+export default async function env({ args, stdin: _stdin, env }) {
   if (args.length === 0) {
     // Use custom env if provided, otherwise use process.env
     const envVars = env || process.env;

package/src/commands/$.ls.mjs

@@ -2,7 +2,7 @@ import fs from 'fs';
 import path from 'path';
 import { trace, VirtualUtils } from '../$.utils.mjs';
 
-export default async function ls({ args, stdin, cwd }) {
+export default async function ls({ args, stdin: _stdin, cwd }) {
   try {
     // Parse flags
     const flags = new Set();

package/src/commands/$.mkdir.mjs

@@ -1,7 +1,7 @@
 import fs from 'fs';
 import { trace, VirtualUtils } from '../$.utils.mjs';
 
-export default async function mkdir({ args, stdin, cwd }) {
+export default async function mkdir({ args, stdin: _stdin, cwd }) {
   const argError = VirtualUtils.validateArgs(args, 1, 'mkdir');
   if (argError) {
     return argError;

package/src/commands/$.mv.mjs

@@ -2,7 +2,7 @@ import fs from 'fs';
 import path from 'path';
 import { trace, VirtualUtils } from '../$.utils.mjs';
 
-export default async function mv({ args, stdin, cwd }) {
+export default async function mv({ args, stdin: _stdin, cwd }) {
   const argError = VirtualUtils.validateArgs(args, 2, 'mv');
   if (argError) {
     return VirtualUtils.invalidArgumentError(
@@ -62,7 +62,7 @@ export default async function mv({ args, stdin, cwd }) {
       const sourcePath = VirtualUtils.resolvePath(source, cwd);
 
       try {
-        const sourceStats = fs.statSync(sourcePath);
+        const _sourceStats = fs.statSync(sourcePath);
         let finalDestPath = destPath;
 
         if (destIsDir) {

package/src/commands/$.pwd.mjs

@@ -1,6 +1,6 @@
 import { trace, VirtualUtils } from '../$.utils.mjs';
 
-export default async function pwd({ args, stdin, cwd }) {
+export default async function pwd({ args: _args, stdin: _stdin, cwd }) {
   // If cwd option is provided, return that instead of process.cwd()
   const dir = cwd || process.cwd();
   trace(

package/src/commands/$.rm.mjs

@@ -1,7 +1,7 @@
 import fs from 'fs';
 import { trace, VirtualUtils } from '../$.utils.mjs';
 
-export default async function rm({ args, stdin, cwd }) {
+export default async function rm({ args, stdin: _stdin, cwd }) {
   const argError = VirtualUtils.validateArgs(args, 1, 'rm');
   if (argError) {
     return argError;

package/src/commands/$.touch.mjs

@@ -1,7 +1,7 @@
 import fs from 'fs';
 import { trace, VirtualUtils } from '../$.utils.mjs';
 
-export default async function touch({ args, stdin, cwd }) {
+export default async function touch({ args, stdin: _stdin, cwd }) {
   const argError = VirtualUtils.validateArgs(args, 1, 'touch');
   if (argError) {
     return VirtualUtils.missingOperandError(

package/src/commands/$.yes.mjs

@@ -2,7 +2,7 @@ import { trace } from '../$.utils.mjs';
 
 export default async function* yes({
   args,
-  stdin,
+  stdin: _stdin,
   isCancelled,
   abortSignal,
   ...rest