promethios-bridge 1.7.2 → 1.7.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "promethios-bridge",
-  "version": "1.7.2",
+  "version": "1.7.3",
   "description": "Run Promethios agent frameworks locally on your computer with full file, terminal, browser access, ambient context capture, and the always-on-top floating chat overlay. Native Framework Mode supports OpenClaw and other frameworks via the bridge.",
   "main": "src/index.js",
   "bin": {

package/src/bridge.js

@@ -227,10 +227,16 @@ async function startBridge({ setupToken, apiBase, port, dev }) {
   // Users can toggle watching on/off from the pill without restarting the bridge.
   if (launchOverlay) {
     try {
-      launchOverlay({ authToken, apiBase, dev });
-      console.log(chalk.cyan('  ⬡ Promethios overlay launched — floating chat is ready'));
-      console.log(chalk.gray('    Hotkey: Ctrl+Shift+P (Win/Linux) or Cmd+Shift+P (Mac)'));
-      console.log('');
+      const overlayChild = launchOverlay({ authToken, apiBase, dev });
+      if (overlayChild) {
+        // Give the child process a moment to start — if it fails immediately
+        // (e.g. ENOENT because electron is not installed), the error handler
+        // in launcher.js will silently absorb it. We print the success message
+        // optimistically but it's non-critical if the overlay doesn't appear.
+        console.log(chalk.cyan('  ⬡ Promethios overlay launched — floating chat is ready'));
+        console.log(chalk.gray('    Hotkey: Ctrl+Shift+P (Win/Linux) or Cmd+Shift+P (Mac)'));
+        console.log('');
+      }
     } catch (err) {
       log('Overlay launch failed (non-critical):', err.message);
     }

package/src/contextCapture.js

@@ -53,39 +53,43 @@ async function captureContext(platform, dev) {
 // ─────────────────────────────────────────────────────────────────────────────
 async function captureWindows(snapshot, log) {
   // Active window title + process name
+  // NOTE: PowerShell here-strings (@"..."@) cannot be passed via -Command because
+  // newlines get collapsed. We write a temp .ps1 file and run it with -File instead.
   try {
-    const psScript = `
-Add-Type @"
-using System;
-using System.Runtime.InteropServices;
-using System.Text;
-public class Win32 {
-  [DllImport("user32.dll")] public static extern IntPtr GetForegroundWindow();
-  [DllImport("user32.dll")] public static extern int GetWindowText(IntPtr hWnd, StringBuilder text, int count);
-  [DllImport("user32.dll")] public static extern int GetWindowThreadProcessId(IntPtr hWnd, out int lpdwProcessId);
-}
-"@
-$hwnd = [Win32]::GetForegroundWindow()
-$sb = New-Object System.Text.StringBuilder 512
-[Win32]::GetWindowText($hwnd, $sb, 512) | Out-Null
-$pid = 0
-[Win32]::GetWindowThreadProcessId($hwnd, [ref]$pid) | Out-Null
-$proc = Get-Process -Id $pid -ErrorAction SilentlyContinue
-$result = @{
-  title = $sb.ToString()
-  process = if ($proc) { $proc.Name } else { "unknown" }
-} | ConvertTo-Json -Compress
-Write-Output $result
-`.trim();
-
+    const os = require('os');
+    const tmpFile = require('path').join(os.tmpdir(), `promethios_ctx_${process.pid}.ps1`);
+    const psScript = [
+      'Add-Type @"',
+      'using System;',
+      'using System.Runtime.InteropServices;',
+      'using System.Text;',
+      'public class Win32 {',
+      '  [DllImport("user32.dll")] public static extern IntPtr GetForegroundWindow();',
+      '  [DllImport("user32.dll")] public static extern int GetWindowText(IntPtr hWnd, StringBuilder text, int count);',
+      '  [DllImport("user32.dll")] public static extern int GetWindowThreadProcessId(IntPtr hWnd, out int lpdwProcessId);',
+      '}',
+      '"@',
+      '$hwnd = [Win32]::GetForegroundWindow()',
+      '$sb = New-Object System.Text.StringBuilder 512',
+      '[Win32]::GetWindowText($hwnd, $sb, 512) | Out-Null',
+      '$pid2 = 0',
+      '[Win32]::GetWindowThreadProcessId($hwnd, [ref]$pid2) | Out-Null',
+      '$proc = Get-Process -Id $pid2 -ErrorAction SilentlyContinue',
+      '$result = @{',
+      '  title = $sb.ToString()',
+      '  procName = if ($proc) { $proc.Name } else { "unknown" }',
+      '} | ConvertTo-Json -Compress',
+      'Write-Output $result',
+    ].join('\r\n');
+    require('fs').writeFileSync(tmpFile, psScript, 'utf8');
     const output = execSync(
-      `powershell -NoProfile -NonInteractive -Command "${psScript.replace(/"/g, '\\"').replace(/\n/g, ' ')}"`,
+      `powershell -NoProfile -NonInteractive -ExecutionPolicy Bypass -File "${tmpFile}"`,
       { encoding: 'utf8', timeout: 4000, windowsHide: true }
     ).trim();
-
+    try { require('fs').unlinkSync(tmpFile); } catch { /* cleanup best-effort */ }
     const parsed = JSON.parse(output);
-    snapshot.active_window = { title: parsed.title, process: parsed.process };
-    snapshot.active_app = friendlyAppName(parsed.process, parsed.title);
+    snapshot.active_window = { title: parsed.title, process: parsed.procName };
+    snapshot.active_app = friendlyAppName(parsed.procName, parsed.title);
     log('Active window:', snapshot.active_window);
   } catch (err) {
     log('Windows active window capture failed:', err.message);

package/src/overlay/launcher.js

@@ -17,6 +17,11 @@ const fs = require('fs');
 /**
  * Launch the Promethios overlay Electron window.
  * Returns the child process (or null if Electron is not available).
+ *
+ * IMPORTANT: When running via `npx`, optional dependencies (including electron)
+ * are NOT installed in the npx cache. The spawn() call may therefore fail with
+ * ENOENT. We attach an 'error' handler before unref() to silently absorb this
+ * instead of crashing the bridge process with an unhandled error event.
  */
 function launchOverlay({ authToken, apiBase = 'https://api.promethios.ai', threadId = '', dev = false } = {}) {
   // Find electron binary — try local node_modules first, then global
@@ -33,7 +38,7 @@ function launchOverlay({ authToken, apiBase = 'https://api.promethios.ai', threa
   for (const candidate of candidates) {
     try {
       if (candidate === 'electron') {
-        // Try to resolve globally
+        // Try to resolve globally — throws if not installed
         require.resolve('electron');
         electronBin = candidate;
         break;
@@ -46,7 +51,7 @@ function launchOverlay({ authToken, apiBase = 'https://api.promethios.ai', threa
   }
 
   if (!electronBin) {
-    if (dev) console.log('[overlay] Electron not found — overlay will not launch. Install with: npm install -g electron');
+    if (dev) console.log('[overlay] Electron not found — overlay will not launch.');
     return null;
   }
 
@@ -64,6 +69,14 @@ function launchOverlay({ authToken, apiBase = 'https://api.promethios.ai', threa
     env: { ...process.env, ELECTRON_NO_ATTACH_CONSOLE: '1' },
   });
 
+  // Attach error handler BEFORE unref() to prevent unhandled 'error' event crashes.
+  // When running via npx, optional deps like electron are not installed, so spawn
+  // may emit ENOENT. We catch it silently — the bridge works fine without the overlay.
+  child.on('error', (err) => {
+    if (dev) console.log(`[overlay] Spawn error (non-critical): ${err.message}`);
+    // No-op: overlay is optional, bridge continues running normally
+  });
+
   child.unref(); // Don't keep bridge CLI alive waiting for overlay
 
   if (dev) console.log(`[overlay] Launched (pid ${child.pid})`);