promethios-bridge 1.7.4 → 1.7.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "promethios-bridge",
-  "version": "1.7.4",
+  "version": "1.7.5",
   "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/contextCapture.js

@@ -223,8 +223,10 @@ async function captureLinux(snapshot, log) {
  * approach via PowerShell — but that's expensive, so we only do it when
  * the active process is a known browser.
  */
-function extractUrlFromBrowserTitle(title, process) {
-  const proc = (process || '').toLowerCase();
+// NOTE: parameter renamed from 'process' to 'processName' to avoid shadowing
+// the global Node.js process object (which caused process.pid to be undefined).
+function extractUrlFromBrowserTitle(title, processName) {
+  const proc = (processName || '').toLowerCase();
   const isBrowser = proc.includes('chrome') || proc.includes('msedge') ||
     proc.includes('firefox') || proc.includes('opera') || proc.includes('brave');
   if (!isBrowser) return null;
@@ -232,26 +234,37 @@ function extractUrlFromBrowserTitle(title, process) {
   // Try to get URL from Chrome/Edge via PowerShell UIAutomation (Windows only)
   // This reads the address bar text directly — works even for SPAs
   // NOTE: Must use -File (temp script) not -Command to preserve multi-line PS syntax.
+  // NOTE: Guard $browserProc.MainWindowHandle against IntPtr.Zero (0) — this happens
+  //       when Chrome is minimized or has no foreground window. FromHandle(0) throws.
   if (proc.includes('chrome') || proc.includes('msedge')) {
     try {
       const os = require('os');
       const path = require('path');
       const fs = require('fs');
       const browserName = proc.includes('msedge') ? 'msedge' : 'chrome';
+      // Use global process.pid (Node built-in) for a unique temp file name.
       const tmpFile = path.join(os.tmpdir(), `promethios_url_${process.pid}.ps1`);
       const psLines = [
         'Add-Type -AssemblyName UIAutomationClient',
         'Add-Type -AssemblyName UIAutomationTypes',
-        `$browserProc = Get-Process -Name "${browserName}" -ErrorAction SilentlyContinue | Select-Object -First 1`,
-        'if ($browserProc) {',
-        '  $root = [System.Windows.Automation.AutomationElement]::FromHandle($browserProc.MainWindowHandle)',
-        '  $cond = New-Object System.Windows.Automation.PropertyCondition(',
-        '    [System.Windows.Automation.AutomationElement]::NameProperty,',
-        '    "Address and search bar")',
-        '  $bar = $root.FindFirst([System.Windows.Automation.TreeScope]::Descendants, $cond)',
-        '  if ($bar) {',
-        '    $vp = $bar.GetCurrentPattern([System.Windows.Automation.ValuePattern]::Pattern)',
-        '    Write-Output $vp.Current.Value',
+        // Get all processes with a non-zero MainWindowHandle (i.e. a visible window)
+        `$procs = Get-Process -Name "${browserName}" -ErrorAction SilentlyContinue | Where-Object { $_.MainWindowHandle -ne 0 }`,
+        'if ($procs) {',
+        '  $browserProc = $procs | Select-Object -First 1',
+        '  try {',
+        '    $root = [System.Windows.Automation.AutomationElement]::FromHandle($browserProc.MainWindowHandle)',
+        '    if ($root) {',
+        '      $cond = New-Object System.Windows.Automation.PropertyCondition(',
+        '        [System.Windows.Automation.AutomationElement]::NameProperty,',
+        '        "Address and search bar")',
+        '      $bar = $root.FindFirst([System.Windows.Automation.TreeScope]::Descendants, $cond)',
+        '      if ($bar) {',
+        '        $vp = $bar.GetCurrentPattern([System.Windows.Automation.ValuePattern]::Pattern)',
+        '        Write-Output $vp.Current.Value',
+        '      }',
+        '    }',
+        '  } catch {',
+        '    # Silently ignore UIAutomation errors (window may have closed, etc.)',
         '  }',
         '}',
       ];
@@ -272,8 +285,8 @@ function extractUrlFromBrowserTitle(title, process) {
   return null; // URL not extractable from title alone
 }
 
-function extractTabTitle(windowTitle, process) {
-  const proc = (process || '').toLowerCase();
+function extractTabTitle(windowTitle, processName) {
+  const proc = (processName || '').toLowerCase();
   // Strip browser suffix from title
   return windowTitle
     .replace(/ - Google Chrome$/, '')