project-scan 1.0.1 → 1.0.3

package/lib/test-runner.js

@@ -9,6 +9,8 @@ const { execSync, spawnSync } = require('child_process');
 const fs   = require('fs');
 const path = require('path');
 
+const IS_WIN = process.platform === 'win32';
+
 // ─── Per-language runner configs ─────────────────────────────────────────────
 
 /**
@@ -100,9 +102,9 @@ const RUNNERS = [
              hasSourceExt(root, '.py');
     },
     run(root) {
-      const cmd = commandExists('pytest') ? 'pytest -v --tb=short 2>&1'
-               : commandExists('python3') ? 'python3 -m pytest -v --tb=short 2>&1'
-               : commandExists('python')  ? 'python -m pytest -v --tb=short 2>&1'
+      const cmd = commandExists('pytest')  ? 'pytest -v --tb=short'
+               : commandExists('python3')  ? 'python3 -m pytest -v --tb=short'
+               : commandExists('python')   ? 'python -m pytest -v --tb=short'
                : null;
       return [runCommand({ dir: root, label: '(root)', cmd, suite: 'pytest', parseOutput: parsePytestOutput })];
     }
@@ -115,7 +117,7 @@ const RUNNERS = [
     run(root) {
       return [runCommand({
         dir: root, label: '(root)',
-        cmd: commandExists('go') ? 'go test ./... -v 2>&1' : null,
+        cmd: commandExists('go') ? 'go test ./... -v' : null,
         suite: 'Go test',
         parseOutput: parseGoTestOutput,
       })];
@@ -129,7 +131,7 @@ const RUNNERS = [
     run(root) {
       return [runCommand({
         dir: root, label: '(root)',
-        cmd: commandExists('cargo') ? 'cargo test 2>&1' : null,
+        cmd: commandExists('cargo') ? 'cargo test' : null,
         suite: 'Cargo test',
         parseOutput: parseCargoOutput,
       })];
@@ -141,7 +143,7 @@ const RUNNERS = [
     name: 'Maven',
     detect(root) { return fs.existsSync(path.join(root, 'pom.xml')); },
     run(root) {
-      const cmd = commandExists('mvn') ? 'mvn test -B 2>&1' : null;
+      const cmd = commandExists('mvn') ? 'mvn test -B' : null;
       return [runCommand({ dir: root, label: '(root)', cmd, suite: 'Maven', parseOutput: parseMavenOutput })];
     }
   },
@@ -154,8 +156,13 @@ const RUNNERS = [
              fs.existsSync(path.join(root, 'build.gradle.kts'));
     },
     run(root) {
-      const gradleCmd = fs.existsSync(path.join(root, 'gradlew')) ? './gradlew' : (commandExists('gradle') ? 'gradle' : null);
-      return [runCommand({ dir: root, label: '(root)', cmd: gradleCmd ? `${gradleCmd} test 2>&1` : null, suite: 'Gradle', parseOutput: parseGradleOutput })];
+      // Windows uses gradlew.bat; Unix uses ./gradlew
+      const wrapperWin  = path.join(root, 'gradlew.bat');
+      const wrapperUnix = path.join(root, 'gradlew');
+      const gradleCmd = fs.existsSync(IS_WIN ? wrapperWin : wrapperUnix)
+        ? (IS_WIN ? 'gradlew.bat' : './gradlew')
+        : (commandExists('gradle') ? 'gradle' : null);
+      return [runCommand({ dir: root, label: '(root)', cmd: gradleCmd ? `${gradleCmd} test` : null, suite: 'Gradle', parseOutput: parseGradleOutput })];
     }
   },
 
@@ -163,14 +170,16 @@ const RUNNERS = [
   {
     name: 'PHPUnit',
     detect(root) {
-      return fs.existsSync(path.join(root, 'phpunit.xml'))       ||
-             fs.existsSync(path.join(root, 'phpunit.xml.dist'))  ||
-             fs.existsSync(path.join(root, 'vendor/bin/phpunit')) ||
+      return fs.existsSync(path.join(root, 'phpunit.xml'))           ||
+             fs.existsSync(path.join(root, 'phpunit.xml.dist'))      ||
+             fs.existsSync(path.join(root, 'vendor', 'bin', 'phpunit')) ||
              hasSourceExt(root, '.php');
     },
     run(root) {
-      const cmd = fs.existsSync(path.join(root, 'vendor/bin/phpunit')) ? './vendor/bin/phpunit --testdox 2>&1'
-               : commandExists('phpunit') ? 'phpunit --testdox 2>&1' : null;
+      const vendorBin = path.join(root, 'vendor', 'bin', 'phpunit');
+      const cmd = fs.existsSync(vendorBin)
+        ? `"${vendorBin}" --testdox`
+        : commandExists('phpunit') ? 'phpunit --testdox' : null;
       return [runCommand({ dir: root, label: '(root)', cmd, suite: 'PHPUnit', parseOutput: parsePHPUnitOutput })];
     }
   },
@@ -184,7 +193,7 @@ const RUNNERS = [
              fs.existsSync(path.join(root, 'spec'));
     },
     run(root) {
-      const cmd = commandExists('rspec') ? 'rspec --format progress 2>&1' : (commandExists('bundle') ? 'bundle exec rspec --format progress 2>&1' : null);
+      const cmd = commandExists('rspec') ? 'rspec --format progress' : (commandExists('bundle') ? 'bundle exec rspec --format progress' : null);
       return [runCommand({ dir: root, label: '(root)', cmd, suite: 'RSpec', parseOutput: parseRSpecOutput })];
     }
   },
@@ -194,7 +203,7 @@ const RUNNERS = [
     name: 'Flutter/Dart test',
     detect(root) { return fs.existsSync(path.join(root, 'pubspec.yaml')); },
     run(root) {
-      const cmd = commandExists('flutter') ? 'flutter test 2>&1' : (commandExists('dart') ? 'dart test 2>&1' : null);
+      const cmd = commandExists('flutter') ? 'flutter test' : (commandExists('dart') ? 'dart test' : null);
       return [runCommand({ dir: root, label: '(root)', cmd, suite: 'Flutter/Dart', parseOutput: parseDartOutput })];
     }
   },
@@ -208,7 +217,7 @@ const RUNNERS = [
       } catch { return false; }
     },
     run(root) {
-      const cmd = commandExists('dotnet') ? 'dotnet test --logger "console;verbosity=normal" 2>&1' : null;
+      const cmd = commandExists('dotnet') ? 'dotnet test --logger "console;verbosity=normal"' : null;
       return [runCommand({ dir: root, label: '(root)', cmd, suite: 'dotnet test', parseOutput: parseDotnetOutput })];
     }
   },
@@ -252,17 +261,18 @@ function runAllTests(rootDir) {
 
 function runJest(dir, label) {
   const start = Date.now();
-  if (!fs.existsSync(path.join(dir, 'node_modules', '.bin', 'jest')) &&
+  if (!fs.existsSync(path.join(dir, 'node_modules', '.bin', 'jest'))    &&
+      !fs.existsSync(path.join(dir, 'node_modules', '.bin', 'jest.cmd')) &&
       !fs.existsSync(path.join(dir, 'node_modules', 'jest'))) {
-    // Try npm test fallback
     const pkg = tryReadJSON(path.join(dir, 'package.json'));
     if (!pkg?.scripts?.test) {
       return { suite: `Jest (${label})`, status: 'skipped', passed: 0, failed: 0, skipped: 0, output: 'Jest not installed. Run npm install first.', duration: 0 };
     }
   }
 
+  // shell:true is required on Windows so npx.cmd is resolved correctly
   const result = spawnSync('npx', ['jest', '--no-coverage', '--json', '--forceExit'], {
-    cwd: dir, encoding: 'utf8', timeout: 120000,
+    cwd: dir, encoding: 'utf8', timeout: 120000, shell: true,
     env: { ...process.env, FORCE_COLOR: '0' }
   });
 
@@ -329,7 +339,11 @@ function runCommand({ dir, label, cmd, suite, parseOutput }) {
   }
   const start = Date.now();
   try {
-    const output = execSync(cmd, { cwd: dir, encoding: 'utf8', timeout: 120000, env: { ...process.env, FORCE_COLOR: '0' }, stdio: 'pipe' });
+    // shell:true is required on Windows; harmless on Unix
+    const output = execSync(cmd, {
+      cwd: dir, encoding: 'utf8', timeout: 120000, shell: true,
+      env: { ...process.env, FORCE_COLOR: '0' }, stdio: 'pipe'
+    });
     const duration = Date.now() - start;
     return { suite: `${suite} (${label})`, duration, ...parseOutput(output, true) };
   } catch (err) {
@@ -432,12 +446,28 @@ function parseDotnetOutput(output, exitOk) {
   return { status: exitOk ? 'passed' : 'failed', passed: p ? parseInt(p[1]) : 0, failed: f ? parseInt(f[1]) : 0, skipped: 0, output: output.slice(-3000) };
 }
 
-// ─── Utility helpers ──────────────────────────────────────────────────────────
+// ─── Utility helpers ─────────────────────────────────────────────────────────
 
 function tryReadJSON(p) { try { return JSON.parse(fs.readFileSync(p, 'utf8')); } catch { return null; } }
 function allDeps(pkg) { return [...Object.keys(pkg.dependencies || {}), ...Object.keys(pkg.devDependencies || {}), ...Object.keys(pkg.peerDependencies || {})]; }
-function commandExists(cmd) { try { execSync(`which ${cmd} 2>/dev/null || command -v ${cmd} 2>/dev/null`, { encoding: 'utf8' }).trim(); return true; } catch { return false; } }
-function hasBin(dir, bin) { return fs.existsSync(path.join(dir, 'node_modules', '.bin', bin)); }
+
+/** Works on Windows (where and Get-Command) and Unix (which / command -v). */
+function commandExists(cmd) {
+  try {
+    if (IS_WIN) {
+      execSync(`where ${cmd}`, { encoding: 'utf8', stdio: 'pipe' });
+    } else {
+      execSync(`command -v ${cmd}`, { encoding: 'utf8', stdio: 'pipe', shell: '/bin/sh' });
+    }
+    return true;
+  } catch { return false; }
+}
+
+function hasBin(dir, bin) {
+  // On Windows the bin is a .cmd wrapper
+  return fs.existsSync(path.join(dir, 'node_modules', '.bin', bin)) ||
+         fs.existsSync(path.join(dir, 'node_modules', '.bin', bin + '.cmd'));
+}
 function hasSourceExt(root, ext) {
   try { return fs.readdirSync(root).some(f => f.endsWith(ext)); } catch { return false; }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "project-scan",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Universal project summarizer & test runner — scans any project for languages, frameworks, databases, cloud services, CI/CD pipelines and runs tests. Works with JavaScript, TypeScript, Python, Go, Rust, Java, Kotlin, PHP, Ruby, Dart, C#, and more.",
   "main": "lib/scanner.js",
   "bin": {