kiro-mobile-bridge 1.0.13 → 1.0.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kiro-mobile-bridge",
-  "version": "1.0.13",
+  "version": "1.0.14",
   "description": "A simple mobile web interface for monitoring Kiro IDE agent sessions from your phone over LAN",
   "type": "module",
   "main": "src/server.js",

package/src/services/cdp.js

@@ -7,13 +7,27 @@ import { CDP_CALL_TIMEOUT, HTTP_TIMEOUT } from '../utils/constants.js';
 
 /**
  * Fetch JSON from a CDP endpoint
+ * Tries both 127.0.0.1 and localhost for cross-platform compatibility
  * @param {number} port - The port to fetch from
  * @param {string} path - The path to fetch (default: /json/list)
  * @returns {Promise<any>} - Parsed JSON response
  */
 export function fetchCDPTargets(port, path = '/json/list') {
+  // Try 127.0.0.1 first, then localhost as fallback
+  return fetchFromHost('127.0.0.1', port, path)
+    .catch(() => fetchFromHost('localhost', port, path));
+}
+
+/**
+ * Fetch JSON from a specific host
+ * @param {string} host - The host to connect to
+ * @param {number} port - The port to fetch from
+ * @param {string} path - The path to fetch
+ * @returns {Promise<any>} - Parsed JSON response
+ */
+function fetchFromHost(host, port, path) {
   return new Promise((resolve, reject) => {
-    const url = `http://127.0.0.1:${port}${path}`;
+    const url = `http://${host}:${port}${path}`;
     
     const req = http.get(url, { timeout: HTTP_TIMEOUT }, (res) => {
       let data = '';

package/src/utils/network.js

@@ -1,11 +1,19 @@
 /**
- * Network utilities
+ * Network utilities for cross-platform local IP detection
+ * 
+ * Supports: Windows, macOS, Linux
+ * 
+ * Node.js os.networkInterfaces() family property:
+ * - Node < 18.0.0: returns string ('IPv4' or 'IPv6')
+ * - Node 18.0.0 - 18.3.x: returns number (4 or 6)
+ * - Node >= 18.4.0: returns string ('IPv4' or 'IPv6')
  */
 import { networkInterfaces } from 'os';
 
 /**
  * Check if interface is IPv4
  * Handles both string ('IPv4') and number (4) family values
+ * for compatibility across Node.js versions
  * @param {object} iface - Network interface object
  * @returns {boolean}
  */
@@ -13,22 +21,66 @@ function isIPv4(iface) {
   return iface.family === 'IPv4' || iface.family === 4;
 }
 
+/**
+ * Check if interface name is a virtual/container interface to skip
+ * @param {string} name - Interface name
+ * @returns {boolean}
+ */
+function isVirtualInterface(name) {
+  const lowerName = name.toLowerCase();
+  const virtualPatterns = [
+    'vethernet',    // Windows WSL/Hyper-V
+    'docker',       // Docker
+    'vmware',       // VMware
+    'virtualbox',   // VirtualBox
+    'vbox',         // VirtualBox alternate
+    'virbr',        // Linux libvirt bridge
+    'br-',          // Docker bridge
+    'veth',         // Virtual ethernet
+    'tailscale',    // Tailscale VPN
+    'tun',          // VPN tunnel
+    'tap',          // VPN tap
+    'utun',         // macOS VPN
+    'awdl',         // Apple Wireless Direct Link
+    'llw',          // Apple Low Latency WLAN
+    'bridge',       // Bridge interfaces
+    'ham',          // Hamachi VPN
+    'zt',           // ZeroTier
+  ];
+  
+  return virtualPatterns.some(pattern => lowerName.includes(pattern));
+}
+
 /**
  * Get local IP address for LAN access
- * Returns the first non-internal IPv4 address found.
+ * Returns the first non-internal IPv4 address found, prioritizing
+ * physical network interfaces over virtual ones.
  * 
- * NOTE: On systems with multiple network interfaces, this returns the first one found.
- * For more control, consider using environment variables or configuration.
+ * Platform-specific interface names:
+ * - Windows: 'Ethernet', 'Wi-Fi', 'Ethernet 2', 'Local Area Connection'
+ * - macOS: 'en0' (Wi-Fi), 'en1' (Ethernet), 'en2', etc.
+ * - Linux: 'eth0', 'enp0s3', 'ens33', 'wlan0', 'wlp2s0'
  * 
  * @returns {string} - Local IP or 'localhost' if no suitable interface found
  */
 export function getLocalIP() {
   const interfaces = networkInterfaces();
   
-  // Prioritize common interface names (Linux, macOS, Windows)
-  const priorityInterfaces = ['Ethernet', 'Wi-Fi', 'eth0', 'en0', 'wlan0'];
+  // Priority 1: Common physical interface names across platforms
+  // Order matters - check most common first
+  const priorityInterfaces = [
+    // Windows
+    'Ethernet', 'Wi-Fi', 'Ethernet 2', 'Local Area Connection',
+    // macOS
+    'en0', 'en1', 'en2', 'en3', 'en4', 'en5',
+    // Linux (traditional)
+    'eth0', 'eth1', 'wlan0', 'wlan1',
+    // Linux (systemd predictable names)
+    'enp0s3', 'enp0s25', 'enp0s31f6', 'ens33', 'ens160', 'ens192',
+    'wlp2s0', 'wlp3s0', 'wlp0s20f3',
+  ];
   
-  // First, try priority interfaces
+  // First pass: try priority interfaces
   for (const name of priorityInterfaces) {
     const ifaces = interfaces[name];
     if (ifaces) {
@@ -40,15 +92,10 @@ export function getLocalIP() {
     }
   }
   
-  // Fallback: return first non-internal IPv4 (skip virtual/WSL interfaces)
+  // Second pass: any non-virtual interface
   for (const name of Object.keys(interfaces)) {
-    // Skip virtual interfaces (WSL, Docker, VPN, etc.)
-    if (name.toLowerCase().includes('vethernet') || 
-        name.toLowerCase().includes('docker') ||
-        name.toLowerCase().includes('vmware') ||
-        name.toLowerCase().includes('virtualbox')) {
-      continue;
-    }
+    if (isVirtualInterface(name)) continue;
+    
     for (const iface of interfaces[name]) {
       if (isIPv4(iface) && !iface.internal) {
         return iface.address;
@@ -56,7 +103,7 @@ export function getLocalIP() {
     }
   }
   
-  // Last resort: any non-internal IPv4
+  // Last resort: any non-internal IPv4 (including virtual)
   for (const name of Object.keys(interfaces)) {
     for (const iface of interfaces[name]) {
       if (isIPv4(iface) && !iface.internal) {