reactoradar 1.5.3 → 1.5.4

package/app.js

@@ -209,7 +209,7 @@ if (window.electronAPI) {
     const btn = $('btnCDP');
     if (btn) {
       const hasCDP = targets?.length > 0;
-      const port = getStoredMetroPort();
+      const port = state.ports?.METRO || getStoredMetroPort();
       btn.textContent = hasCDP
         ? `JS Debugger (:${port}) [${targets.length}] ↗`
         : `JS Debugger (:${port}) ↗`;
@@ -268,21 +268,7 @@ if (window.electronAPI) {
   window.electronAPI.on('update-available', ({ current, latest }) => {
     // Show in settings only, not as a banner
     state._updateAvailable = { current, latest };
-    const el = $('aboutVersion');
-    if (el) el.innerHTML = `v${current} <span style="color:var(--green);font-size:10px;margin-left:6px">v${latest} available</span>`;
-    // Add update button in settings if not already there
-    if (!$('updateBtn')) {
-      const aboutEl = document.querySelector('.settings-about');
-      if (aboutEl) {
-        const btn = document.createElement('div');
-        btn.style.cssText = 'margin-top:10px';
-        btn.innerHTML = '<button id="updateBtn" class="tb-btn primary" style="font-size:11px">Download v' + latest + '</button>';
-        aboutEl.appendChild(btn);
-        $('updateBtn')?.addEventListener('click', () => {
-          window.electronAPI?.openExternal('https://github.com/sharanagouda/react-native-debugger/releases');
-        });
-      }
-    }
+    _applyUpdateBanner();
   });
 
   window.electronAPI.on('trigger-open-cdp', () => {
@@ -299,6 +285,31 @@ if (window.electronAPI) {
 }
 
 // ─── Device Connection Status (inline in titlebar) ───────────────────────────
+// Reusable — called from IPC handler AND from initSettingsPanel
+function _applyUpdateBanner() {
+  const info = state._updateAvailable;
+  if (!info) return;
+  const { current, latest } = info;
+  const el = $('aboutVersion');
+  if (el && !el.dataset.updateApplied) {
+    el.innerHTML = `v${current} <span style="color:var(--green);font-size:10px;margin-left:6px">v${latest} available</span>`;
+    el.dataset.updateApplied = '1';
+  }
+  // Add update button in settings if not already there
+  if (!$('updateBtn')) {
+    const aboutEl = document.querySelector('.settings-about');
+    if (aboutEl) {
+      const btn = document.createElement('div');
+      btn.style.cssText = 'margin-top:10px';
+      btn.innerHTML = '<button id="updateBtn" class="tb-btn primary" style="font-size:11px;padding:6px 16px">Download v' + latest + '</button>';
+      aboutEl.appendChild(btn);
+      $('updateBtn')?.addEventListener('click', () => {
+        window.electronAPI?.openExternal('https://github.com/sharanagouda/react-native-debugger/releases');
+      });
+    }
+  }
+}
+
 function updateDeviceBanner(service, connected) {
   state.connections[service] = connected;
   const el = $('deviceStatus');
@@ -1273,9 +1284,13 @@ function renderNetwork() {
   rows.appendChild(frag);
 }
 
+function _isHttpError(r) {
+  return r.phase === 'error' || (r.status && r.status >= 400);
+}
+
 function buildNetRow(r, wfMin, wfRange) {
   const row = document.createElement('div');
-  row.className = 'net-row' + (r.id === state.network.selectedId ? ' selected' : '') + (r.phase === 'error' ? ' error' : '');
+  row.className = 'net-row' + (r.id === state.network.selectedId ? ' selected' : '') + (_isHttpError(r) ? ' error' : '');
   row.dataset.id = r.id;
 
   const urlObj = tryURL(r.url);
@@ -1291,7 +1306,9 @@ function buildNetRow(r, wfMin, wfRange) {
   const method = r.method || '?';
   const mClass = ['GET','POST','PUT','PATCH','DELETE'].includes(method) ? `m-${method}` : 'm-other';
   const fullPath = urlObj ? urlObj.pathname + urlObj.search : r.url || '';
-  nameCell.innerHTML = `<span class="method-badge ${mClass}">${method}</span> <span class="net-path" title="${esc(r.url)}">${esc(fullPath)}</span><span class="net-host">${esc(host)}</span>`;
+  const isErr = _isHttpError(r);
+  const pathCls = isErr ? ' net-path-error' : '';
+  nameCell.innerHTML = `<span class="method-badge ${mClass}">${method}</span> <span class="net-path${pathCls}" title="${esc(r.url)}">${esc(fullPath)}</span><span class="net-host">${esc(host)}</span>`;
   row.appendChild(nameCell);
 
   // Status
@@ -1301,7 +1318,13 @@ function buildNetRow(r, wfMin, wfRange) {
   statusCell.style.width = NET_COLS[1].width + 'px';
   let statusStr = '...', sCls = 's-pending';
   if (r.phase === 'error') { statusStr = 'ERR'; sCls = 's-err'; }
-  else if (r.status) { statusStr = String(r.status); sCls = `s-${Math.floor(r.status/100)}`; }
+  else if (r.status) {
+    statusStr = String(r.status);
+    const group = Math.floor(r.status / 100);
+    // 1xx info, 2xx success, 3xx redirect, 4xx client error, 5xx server error
+    if (group >= 4) sCls = 's-err';
+    else sCls = `s-${group}`;
+  }
   statusCell.className += ` ${sCls}`;
   statusCell.textContent = statusStr;
   row.appendChild(statusCell);
@@ -1350,6 +1373,7 @@ function buildNetRow(r, wfMin, wfRange) {
     const width = Math.max(2, ((r.duration || 50) / wfRange) * 100);
     let barCls = 'pending';
     if (r.phase === 'error') barCls = 'err';
+    else if (r.status && r.status >= 400) barCls = 'err';
     else if (r.status) barCls = `s${Math.floor(r.status/100)}`;
     wfCell.innerHTML = `<div class="wf-bar ${barCls}" style="left:${left}%;width:${width}%"></div>`;
   }
@@ -1416,8 +1440,12 @@ function renderNetDetailTabs(r) {
 }
 
 function renderNetDetailContent(r) {
-  const body = $('netDetailContent');
+  let body = $('netDetailContent');
   if (!body) return;
+  // Clone-replace to remove all stale event listeners (prevents contextmenu leak)
+  const fresh = body.cloneNode(false);
+  body.parentNode.replaceChild(fresh, body);
+  body = fresh;
   const tab = r._tab || 'headers';
 
   if (tab === 'headers') {
@@ -1436,7 +1464,7 @@ function renderNetDetailContent(r) {
       <div class="kv-grid">
         <span class="kv-key">Request URL</span><span class="kv-val">${esc(r.url)}</span>
         <span class="kv-key">Method</span><span class="kv-val">${esc(r.method)}</span>
-        <span class="kv-key">Status</span><span class="kv-val ${r.status ? 's-' + Math.floor(r.status/100) : 's-pending'}">${r.status || 'Pending'} ${r.statusText || ''}</span>
+        <span class="kv-key">Status</span><span class="kv-val ${r.phase === 'error' ? 's-err' : r.status ? (r.status >= 400 ? 's-err' : 's-' + Math.floor(r.status/100)) : 's-pending'}">${r.phase === 'error' ? (r.status || 'ERR') : (r.status || 'Pending')} ${r.statusText || (r.phase === 'error' ? r.error || 'Network Error' : '')}</span>
       </div>
       ${renderH('Response Headers', rsH)}
       ${renderH('Request Headers', rqH)}`;
@@ -1460,16 +1488,27 @@ function renderNetDetailContent(r) {
       }
     }
   } else if (tab === 'preview') {
-    if (r.phase === 'error') { body.innerHTML = `<span style="color:var(--red)">${esc(r.error || 'Request failed')}</span>`; return; }
+    const isErrStatus = _isHttpError(r);
+    if (r.phase === 'error' && !r.responseBody) { body.innerHTML = `<span style="color:var(--red)">${esc(r.error || 'Request failed')}</span>`; return; }
     if (!r.responseBody && r.phase !== 'response') { body.innerHTML = '<span style="color:var(--text-dim)">Pending...</span>'; return; }
     // Render as collapsible JSON tree with right-click copy
     const val = r.responseBody;
     let treeData = val;
     if (typeof val === 'string') {
-      try { treeData = JSON.parse(val); } catch { body.textContent = val; return; }
+      try { treeData = JSON.parse(val); } catch {
+        body.innerHTML = `<span style="color:${isErrStatus ? 'var(--red)' : 'inherit'}">${esc(val)}</span>`;
+        return;
+      }
     }
     if (treeData && typeof treeData === 'object') {
       body.innerHTML = '';
+      // Show error status banner above the response body
+      if (isErrStatus) {
+        const errBanner = document.createElement('div');
+        errBanner.style.cssText = 'color:var(--red);font-weight:600;padding:4px 0 8px;font-size:11px;border-bottom:1px solid rgba(255,94,114,.15);margin-bottom:8px';
+        errBanner.textContent = `${r.status || 'ERR'} ${r.statusText || r.error || 'Error'}`;
+        body.appendChild(errBanner);
+      }
       body.appendChild(createTreeNode(null, treeData, false));
       // Right-click on preview to copy the whole object or clicked node value
       body.addEventListener('contextmenu', (e) => {
@@ -1477,12 +1516,27 @@ function renderNetDetailContent(r) {
         showPreviewCopyMenu(e, treeData);
       });
     } else {
-      body.innerHTML = '<span style="color:var(--text-dim)">No preview available</span>';
+      body.innerHTML = isErrStatus
+        ? `<span style="color:var(--red)">${esc(String(r.responseBody))}</span>`
+        : '<span style="color:var(--text-dim)">No preview available</span>';
     }
   } else if (tab === 'response') {
-    if (r.phase === 'error') { body.innerHTML = `<span style="color:var(--red)">${esc(r.error || 'Request failed')}</span>`; return; }
+    const isErrStatus = _isHttpError(r);
+    if (r.phase === 'error' && !r.responseBody) { body.innerHTML = `<span style="color:var(--red)">${esc(r.error || 'Request failed')}</span>`; return; }
     if (!r.responseBody && r.phase !== 'response') { body.innerHTML = '<span style="color:var(--text-dim)">Pending...</span>'; return; }
-    body.innerHTML = renderJSON(r.responseBody);
+    if (isErrStatus) {
+      const errBanner = document.createElement('div');
+      errBanner.style.cssText = 'color:var(--red);font-weight:600;padding:4px 0 8px;font-size:11px;border-bottom:1px solid rgba(255,94,114,.15);margin-bottom:8px';
+      errBanner.textContent = `${r.status || 'ERR'} ${r.statusText || r.error || 'Error'}`;
+      body.innerHTML = '';
+      body.appendChild(errBanner);
+      const raw = document.createElement('div');
+      raw.style.color = 'var(--red)';
+      raw.innerHTML = renderJSON(r.responseBody);
+      body.appendChild(raw);
+    } else {
+      body.innerHTML = renderJSON(r.responseBody);
+    }
   }
 }
 
@@ -2433,6 +2487,9 @@ function initSettingsPanel() {
     setStoredFontSize(size);
     applyFontSize(size);
   });
+
+  // Apply update banner if update info arrived before settings panel was created
+  _applyUpdateBanner();
 }
 
 // Apply saved theme + font size + app name on load

package/main.js

@@ -107,6 +107,18 @@ async function createMainWindow() {
 }
 
 // ─── Update Checker ──────────────────────────────────────────────────────────
+function _semverCompare(a, b) {
+  // Returns 1 if a > b, -1 if a < b, 0 if equal
+  const pa = (a || '').split('.').map(Number);
+  const pb = (b || '').split('.').map(Number);
+  for (let i = 0; i < 3; i++) {
+    const va = pa[i] || 0, vb = pb[i] || 0;
+    if (va > vb) return 1;
+    if (va < vb) return -1;
+  }
+  return 0;
+}
+
 function checkForUpdates() {
   const currentVersion = require('./package.json').version;
   https.get('https://registry.npmjs.org/reactoradar/latest', (res) => {
@@ -115,9 +127,16 @@ function checkForUpdates() {
     res.on('end', () => {
       try {
         const latest = JSON.parse(data).version;
-        if (latest && latest !== currentVersion) {
-          // Notify the renderer to show an update banner
-          mainWindow?.webContents.send('update-available', { current: currentVersion, latest });
+        if (latest && _semverCompare(latest, currentVersion) > 0) {
+          // Send with retries to ensure renderer catches it after did-finish-load
+          const payload = { current: currentVersion, latest };
+          [500, 2000, 5000].forEach(delay => {
+            setTimeout(() => {
+              if (mainWindow && !mainWindow.isDestroyed()) {
+                mainWindow.webContents.send('update-available', payload);
+              }
+            }, delay);
+          });
           console.log(`[Update] New version available: ${latest} (current: ${currentVersion})`);
         }
       } catch {}
@@ -336,7 +355,9 @@ function setupIPC() {
   // clear-all is handled by renderer via clear-all-ui IPC from menu
 
   ipcMain.on('set-metro-port', (_, port) => {
-    PORTS.METRO = port;
+    const p = parseInt(port);
+    if (isNaN(p) || p < 1024 || p > 65535) return;
+    PORTS.METRO = p;
     fetchCDPTargets();
     mainWindow?.webContents.send('ports', PORTS);
   });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "reactoradar",
   "productName": "ReactoRadar",
-  "version": "1.5.3",
+  "version": "1.5.4",
   "description": "macOS debugger for React Native — Console, Sources, Network, Performance, Memory, Redux, AsyncStorage, React tree. Supports RN 0.74+ with Hermes and New Architecture.",
   "main": "main.js",
   "bin": {

package/styles.css

@@ -741,7 +741,11 @@ body {
 }
 .net-row:hover { background: var(--bg3); }
 .net-row.selected { background: var(--bg4); }
-.net-row.error { background: rgba(255,94,114,.04); }
+.net-row.error { background: rgba(255,94,114,.06); border-bottom-color: rgba(255,94,114,.10); }
+.net-row.error:hover { background: rgba(255,94,114,.10); }
+.net-row.error .net-path { color: var(--red); }
+.net-row.error .net-host { color: rgba(255,94,114,.6); }
+.net-path-error { color: var(--red) !important; }
 .net-cell {
   padding: 6px 8px;
   overflow: hidden;
@@ -757,12 +761,13 @@ body {
 .net-cell-name .method-badge { flex-shrink: 0; vertical-align: middle; }
 
 .net-status { font-weight: 700; }
-.s-2 { color: var(--green); }
-.s-3 { color: var(--yellow); }
-.s-4 { color: var(--orange); }
-.s-5 { color: var(--red); }
+.s-1 { color: var(--text-mid); }   /* 1xx informational */
+.s-2 { color: var(--green); }      /* 2xx success */
+.s-3 { color: var(--yellow); }     /* 3xx redirect */
+.s-4 { color: var(--red); }        /* 4xx client error */
+.s-5 { color: var(--red); }        /* 5xx server error */
 .s-pending { color: var(--text-dim); }
-.s-err { color: var(--red); }
+.s-err { color: var(--red); font-weight: 700; }
 
 .net-type { color: var(--text-dim); font-size: 10px; }
 .net-initiator { color: var(--text-dim); font-size: 10px; }