evals 2.2.5 → 2.2.7

package/cli.js

@@ -7,6 +7,9 @@ import { exec } from 'child_process';
 
 const e = React.createElement;
 
+// Diagnostic: Uncomment to verify script is running
+console.error('CLI script started, platform:', process.platform, 'isTTY:', process.stdin.isTTY);
+
 // "ARIZE EVALS" - EXACTLY matched width (both 44 chars)
 const largeLogo = `
  █████╗   ██████╗    ██╗  ███████╗  ███████╗
@@ -206,4 +209,38 @@ function App() {
 }
 
 // Run the app
-render(e(App), { patchConsole: false });
+// Check if we have an interactive terminal (required for Ink)
+if (!process.stdin.isTTY || !process.stdout.isTTY) {
+  console.error('Error: This command requires an interactive terminal.');
+  console.error('Please run `npx evals` from your terminal (not from a script or non-interactive environment).');
+  process.exit(1);
+}
+
+// Handle uncaught errors
+process.on('uncaughtException', (error) => {
+  console.error('Uncaught exception:', error.message);
+  console.error(error.stack);
+  process.exit(1);
+});
+
+process.on('unhandledRejection', (reason, promise) => {
+  console.error('Unhandled rejection at:', promise);
+  console.error('Reason:', reason);
+  process.exit(1);
+});
+
+// Ensure stdout is not buffered (important for Windows)
+if (process.stdout.isTTY) {
+  process.stdout.setEncoding('utf8');
+}
+
+try {
+  render(e(App), { patchConsole: false });
+} catch (error) {
+  console.error('Failed to render interactive CLI:', error.message);
+  console.error(error.stack);
+  if (process.platform === 'win32') {
+    console.error('\nNote: If you\'re using cmd.exe, try running in PowerShell or Git Bash instead.');
+  }
+  process.exit(1);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "evals",
-  "version": "2.2.5",
+  "version": "2.2.7",
   "description": "Arize evals package",
   "type": "module",
   "main": "cli.js",

package/spawn-interactive.js

@@ -2,30 +2,50 @@ import { spawn } from 'child_process';
 import { openSync } from 'fs';
 
 function tryOpenTTY() {
-  try {
-    const device = process.platform === 'win32' ? 'CON' : '/dev/tty';
-    openSync(device, 'r');
-    return { device, available: true };
-  } catch (err) {
+  if (process.platform === 'win32') {
+    // On Windows, avoid CON device (issues in Node.js 22)
+    // Use isTTY check instead, and stdio: 'inherit' for spawning
+    if (process.stdin.isTTY && process.stdout.isTTY && process.stderr.isTTY) {
+      return { available: true, useInherit: true };
+    }
     return { available: false };
+  } else {
+    // On macOS/Linux, use direct /dev/tty access
+    try {
+      const device = '/dev/tty';
+      openSync(device, 'r');
+      return { device, available: true, useInherit: false };
+    } catch (err) {
+      return { available: false };
+    }
   }
 }
 
 const tty = tryOpenTTY();
 
 if (tty.available) {
-  const child = spawn('node', ['./cli.js'], {
-    stdio: [
-      openSync(tty.device, 'r'),
-      openSync(tty.device, 'w'),
-      openSync(tty.device, 'w')
-    ]
-  });
+  const spawnOptions = tty.useInherit
+    ? { stdio: 'inherit', shell: true }
+    : {
+        stdio: [
+          openSync(tty.device, 'r'),
+          openSync(tty.device, 'w'),
+          openSync(tty.device, 'w')
+        ]
+      };
+
+  const child = spawn('node', ['./cli.js'], spawnOptions);
 
   child.on('close', (code) => {
-    process.exit(code);
+    process.exit(code || 0);
+  });
+
+  child.on('error', (err) => {
+    console.error('Failed to start interactive CLI:', err.message);
+    console.log('Run `npx evals` to complete setup.');
+    process.exit(1);
   });
 } else {
   console.log('Run `npx evals` to complete setup.');
-  process.exit(1)
+  process.exit(1);
 }