neoagent 2.4.0 → 2.4.1-beta.11

package/extensions/chrome-browser/background.mjs

@@ -1,7 +1,7 @@
 import { createBrowserProtocol } from './protocol.mjs';
 import { DEFAULT_SERVER_URL } from './config.mjs';
 
-const STORAGE_KEYS = ['serverUrl', 'configuredServerUrl', 'token', 'pairingId', 'pairingSecret', 'approvalUrl', 'status'];
+const STORAGE_KEYS = ['serverUrl', 'configuredServerUrl', 'token', 'pairingId', 'pairingSecret', 'approvalUrl', 'status', 'extensionName'];
 const protocol = createBrowserProtocol(chrome);
 let socket = null;
 let reconnectTimer = null;
@@ -188,10 +188,12 @@ async function handleSocketMessage(raw) {
 async function startPairing(serverUrl) {
   const normalized = await resolveServerUrl(serverUrl);
   if (!normalized) throw new Error('NeoAgent server URL required.');
+  const { extensionName } = await getStorage(['extensionName']);
+  const nameToUse = String(extensionName || 'Chrome Extension').trim() || 'Chrome Extension';
   const response = await fetchWithTimeout(`${normalized}/api/browser-extension/pairing/request`, {
     method: 'POST',
     headers: { 'content-type': 'application/json' },
-    body: JSON.stringify({ extensionName: 'NeoAgent Browser' }),
+    body: JSON.stringify({ extensionName: nameToUse }),
   });
   const payload = await response.json().catch(() => ({}));
   if (!response.ok) throw new Error(payload.error || `Pairing failed: ${response.status}`);
@@ -212,14 +214,15 @@ async function startPairing(serverUrl) {
 }
 
 async function claimPairing() {
-  const { serverUrl, pairingId, pairingSecret } = await getStorage(['serverUrl', 'pairingId', 'pairingSecret']);
+  const { serverUrl, pairingId, pairingSecret, extensionName } = await getStorage(['serverUrl', 'pairingId', 'pairingSecret', 'extensionName']);
   if (!serverUrl || !pairingId || !pairingSecret) {
     throw new Error('No pending pairing request.');
   }
+  const nameToUse = String(extensionName || 'Chrome Extension').trim() || 'Chrome Extension';
   const response = await fetchWithTimeout(`${serverUrl}/api/browser-extension/pairing/${encodeURIComponent(pairingId)}/claim`, {
     method: 'POST',
     headers: { 'content-type': 'application/json' },
-    body: JSON.stringify({ pairingSecret, extensionName: 'NeoAgent Browser' }),
+    body: JSON.stringify({ pairingSecret, extensionName: nameToUse }),
   });
   const payload = await response.json().catch(() => ({}));
   if (!response.ok) throw new Error(payload.error || `Claim failed: ${response.status}`);
@@ -250,12 +253,18 @@ async function checkForUpdates(preferredServerUrl) {
   const response = await fetchWithTimeout(`${serverUrl}/api/browser-extension/latest`);
   const latest = await response.json().catch(() => ({}));
   if (!response.ok) throw new Error(latest.error || `Update check failed: ${response.status}`);
-  const currentVersion = chrome.runtime.getManifest().version;
+  const manifest = chrome.runtime.getManifest();
+  const currentVersion = manifest.version;
+  const currentVersionName = manifest.version_name || currentVersion;
+  const latestVersion = latest.version || currentVersion;
+  const latestVersionName = latest.versionName || latestVersion;
   return {
     currentVersion,
-    latestVersion: latest.version || currentVersion,
+    currentVersionName,
+    latestVersion,
+    latestVersionName,
     downloadUrl: latest.downloadUrl || `${serverUrl}/api/browser-extension/download`,
-    updateAvailable: compareVersions(latest.version, currentVersion) > 0,
+    updateAvailable: compareVersions(latestVersion, currentVersion) > 0,
   };
 }
 
@@ -279,6 +288,9 @@ chrome.runtime.onMessage.addListener((message, _sender, sendResponse) => {
         return disconnect();
       case 'checkForUpdates':
         return checkForUpdates(message.serverUrl);
+      case 'saveExtensionName':
+        await setStorage({ extensionName: message.extensionName });
+        return { success: true };
       case 'openDownload':
         return openDownload(message.serverUrl);
       case 'getState':

package/extensions/chrome-browser/icons/logo.svg

@@ -0,0 +1,12 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 32 32">
+  <defs>
+    <linearGradient id="bg" x1="0%" y1="0%" x2="100%" y2="100%">
+      <stop offset="0%" stop-color="#8f6d3e"/>
+      <stop offset="100%" stop-color="#2f7d6e"/>
+    </linearGradient>
+  </defs>
+  <rect x="5.76" y="5.76" width="20.48" height="20.48" rx="6.96" fill="url(#bg)" stroke="#ffffff" stroke-opacity="0.16" stroke-width="1"/>
+  <polygon points="16,9.76 9.35,13.12 16,16.48 22.65,13.12" fill="white"/>
+  <polyline points="9.35,16.48 16,19.68 22.65,16.48" fill="none" stroke="white" stroke-width="1.44" stroke-linecap="round" stroke-linejoin="round"/>
+  <polyline points="9.35,19.68 16,22.72 22.65,19.68" fill="none" stroke="white" stroke-width="1.44" stroke-linecap="round" stroke-linejoin="round"/>
+</svg>

package/extensions/chrome-browser/manifest.json

@@ -1,14 +1,25 @@
 {
   "manifest_version": 3,
   "name": "NeoAgent Browser",
-  "version": "1.0.0",
+  "version": "2.4.1.7",
+  "version_name": "2.4.1-beta.7",
   "description": "Connect this Chrome browser to NeoAgent for browser automation.",
   "minimum_chrome_version": "118",
   "permissions": ["debugger", "storage", "tabs"],
   "host_permissions": ["http://*/*", "https://*/*"],
+  "icons": {
+    "16": "icons/icon16.png",
+    "48": "icons/icon48.png",
+    "128": "icons/icon128.png"
+  },
   "action": {
     "default_title": "NeoAgent Browser",
-    "default_popup": "popup.html"
+    "default_popup": "popup.html",
+    "default_icon": {
+      "16": "icons/icon16.png",
+      "48": "icons/icon48.png",
+      "128": "icons/icon128.png"
+    }
   },
   "background": {
     "service_worker": "background.mjs",

package/extensions/chrome-browser/popup.css

@@ -46,6 +46,11 @@ header {
   gap: 6px;
 }
 
+.logo {
+  width: 40px;
+  height: 40px;
+}
+
 h1,
 h2,
 p {

package/extensions/chrome-browser/popup.html

@@ -8,6 +8,7 @@
 <body>
   <main>
     <header>
+      <img src="icons/logo.svg" alt="NeoAgent" class="logo">
       <p class="eyebrow">NeoAgent Browser</p>
       <h1>Connect this Chrome</h1>
       <p class="intro">Pair once, then let NeoAgent control this browser when you ask it to.</p>
@@ -31,15 +32,17 @@
       </div>
     </section>
 
-    <button id="openApp" type="button" class="link-button">Open NeoAgent web page</button>
-
     <details id="settings" class="settings">
       <summary>Settings &amp; updates</summary>
       <label>
         Server URL
         <input id="serverUrl" type="url" placeholder="https://neoagent.example.com">
       </label>
-      <div class="settings-actions">
+      <label style="margin-top: 8px;">
+        Extension Name
+        <input id="extensionName" type="text" placeholder="e.g. Work Laptop, Personal Mac" value="Chrome Extension">
+      </label>
+      <div class="settings-actions" style="margin-top: 12px;">
         <button id="checkUpdate" type="button" class="secondary">Check for update</button>
         <button id="download" type="button" class="secondary">Download latest ZIP</button>
         <button id="disconnect" type="button" class="secondary danger">Disconnect</button>
@@ -47,8 +50,7 @@
       <p class="hint">The server URL is usually filled in by the ZIP downloaded from NeoAgent.</p>
     </details>
 
-    <p class="hint">Pairing opens NeoAgent in a tab. This extension never stores your NeoAgent password.</p>
-    <p id="message" class="message"></p>
+<p id="message" class="message"></p>
   </main>
   <script src="popup.js" type="module"></script>
 </body>

package/extensions/chrome-browser/popup.js

@@ -1,6 +1,7 @@
 const statusEl = document.querySelector('#status');
 const statusDotEl = document.querySelector('#statusDot');
 const serverUrlEl = document.querySelector('#serverUrl');
+const extensionNameEl = document.querySelector('#extensionName');
 const serverLabelEl = document.querySelector('#serverLabel');
 const messageEl = document.querySelector('#message');
 const settingsEl = document.querySelector('#settings');
@@ -9,7 +10,6 @@ const flowTitleEl = document.querySelector('#flowTitle');
 const flowDescriptionEl = document.querySelector('#flowDescription');
 const primaryActionEl = document.querySelector('#primaryAction');
 const secondaryActionEl = document.querySelector('#secondaryAction');
-const openAppEl = document.querySelector('#openApp');
 const disconnectEl = document.querySelector('#disconnect');
 const checkUpdateEl = document.querySelector('#checkUpdate');
 const downloadEl = document.querySelector('#download');
@@ -59,7 +59,7 @@ function setBusy(isBusy, label = 'Working...') {
   }
   const busy = pendingActions > 0;
 
-  [primaryActionEl, secondaryActionEl, openAppEl, disconnectEl, checkUpdateEl, downloadEl].forEach((button) => {
+  [primaryActionEl, secondaryActionEl, disconnectEl, checkUpdateEl, downloadEl].forEach((button) => {
     if (!button || button.hidden) return;
     if (busy) {
       if (!Object.prototype.hasOwnProperty.call(button.dataset, 'wasDisabled')) {
@@ -99,8 +99,6 @@ function updateFlow() {
   const hasToken = Boolean(currentState.token || currentState.tokenId);
   const approvalUrl = currentState.approvalUrl || '';
 
-  openAppEl.disabled = !hasServerUrl;
-
   if (!hasServerUrl) {
     stepLabelEl.textContent = 'Step 1 of 3';
     flowTitleEl.textContent = 'Add your NeoAgent server';
@@ -217,6 +215,10 @@ async function refresh() {
   if (serverUrl && document.activeElement !== serverUrlEl) {
     serverUrlEl.value = serverUrl;
   }
+  const extensionName = currentState.extensionName || 'Chrome Extension';
+  if (document.activeElement !== extensionNameEl) {
+    extensionNameEl.value = extensionName;
+  }
   if (!serverUrl) {
     settingsEl.open = true;
   }
@@ -240,10 +242,17 @@ function bindAsyncClick(element, handler) {
 }
 
 serverUrlEl.addEventListener('input', updateFlow);
+extensionNameEl.addEventListener('input', async () => {
+  const name = String(extensionNameEl.value || '').trim();
+  try {
+    await send('saveExtensionName', { extensionName: name });
+  } catch (err) {
+    console.error('Failed to save extension name', err);
+  }
+});
 
 bindAsyncClick(primaryActionEl, () => runAction(primaryActionEl.dataset.action));
 bindAsyncClick(secondaryActionEl, () => runAction(secondaryActionEl.dataset.action));
-bindAsyncClick(openAppEl, () => runAction('openApp'));
 bindAsyncClick(disconnectEl, async () => {
   await send('disconnect');
   await refresh();
@@ -253,8 +262,8 @@ bindAsyncClick(checkUpdateEl, async () => {
   const result = await send('checkForUpdates', { serverUrl: effectiveServerUrl() });
   setMessage(
     result.updateAvailable
-      ? `Update available: ${result.currentVersion} -> ${result.latestVersion}.`
-      : `Current version ${result.currentVersion} is up to date.`,
+      ? `Update available: ${result.currentVersionName || result.currentVersion} -> ${result.latestVersionName || result.latestVersion}.`
+      : `Current version ${result.currentVersionName || result.currentVersion} is up to date.`,
     result.updateAvailable ? '' : 'success',
   );
 });