agentgui 1.0.815 → 1.0.817

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## 2026-04-11
+- refactor: split tool-version.js into tool-version-check.js (sync) and tool-version-fetch.js (async/network)
+
+## [Unreleased]
+### Refactor
+- Strip all standalone comment lines (// and /* */ block lines) from 14 static JS files: event-filter.js, image-loader.js, syntax-highlighter.js, client.js, streaming-renderer.js, ui-components.js, conversations.js, websocket-manager.js, conv-machine.js, ws-machine.js, ws-client.js, state-barrier.js, script-runner.js, theme.js. Total 781 comment lines removed. All files syntax-verified.
+
 ## [Unreleased]
 
 ### Changed

package/lib/tool-manager.js

@@ -1,4 +1,4 @@
-import { checkToolViaBunx, clearVersionCache } from './tool-version.js';
+import { checkToolViaBunx, clearVersionCache } from './tool-version-fetch.js';
 import { createInstaller } from './tool-spawner.js';
 import { autoProvision as _autoProvision, startPeriodicUpdateCheck as _startPeriodicUpdateCheck, stopPeriodicUpdateCheck } from './tool-provisioner.js';
 

package/lib/tool-provisioner.js

@@ -1,4 +1,4 @@
-import { checkToolViaBunx } from './tool-version.js';
+import { checkToolViaBunx } from './tool-version-fetch.js';
 import * as toolInstallMachine from './tool-install-machine.js';
 
 let updateCheckInterval = null;

package/lib/tool-spawner.js

@@ -1,6 +1,7 @@
 import { spawn } from 'child_process';
 import os from 'os';
-import { getCliVersion, getInstalledVersion, clearVersionCache, checkToolViaBunx } from './tool-version.js';
+import { getCliVersion, getInstalledVersion } from './tool-version-check.js';
+import { clearVersionCache, checkToolViaBunx } from './tool-version-fetch.js';
 import * as toolInstallMachine from './tool-install-machine.js';
 
 const isWindows = os.platform() === 'win32';

package/lib/{tool-version.js → tool-version-check.js}

@@ -1,13 +1,12 @@
 import os from 'os';
 import fs from 'fs';
 import path from 'path';
-import https from 'https';
 import { execSync } from 'child_process';
 
 const isWindows = os.platform() === 'win32';
 const homeDir = os.homedir();
 
-const BIN_MAP = {
+export const BIN_MAP = {
   '@anthropic-ai/claude-code': 'claude',
   'opencode-ai': 'opencode',
   '@google/gemini-cli': 'gemini',
@@ -31,7 +30,7 @@ function getClaudeInstalledPluginPath(pluginId) {
   return null;
 }
 
-const FRAMEWORK_PATHS = {
+export const FRAMEWORK_PATHS = {
   claude: {
     pluginDir: (pluginId) => getClaudeInstalledPluginPath(pluginId) || path.join(homeDir, '.claude', 'plugins', pluginId),
     versionFile: (pluginId) => {
@@ -96,15 +95,12 @@ export function getInstalledVersion(pkg, pluginId = null, frameWork = null, tool
     const tool = tools.find(t => t.pkg === pkg);
     const actualPluginId = pluginId || tool?.pluginId || pkg;
     const actualFrameWork = frameWork || tool?.frameWork;
-
     const frameworks = actualFrameWork ? [actualFrameWork] : Object.keys(FRAMEWORK_PATHS);
     for (const fw of frameworks) {
       const fwConfig = FRAMEWORK_PATHS[fw];
       if (!fwConfig) continue;
-
       const version = tryReadVersion(fwConfig, actualPluginId);
       if (version) return version;
-
       if (fwConfig.fallbackInstalled) {
         const hasMarker = fwConfig.markerFile && fs.existsSync(fwConfig.markerFile(actualPluginId));
         const hasDir = fs.existsSync(fwConfig.pluginDir(actualPluginId));
@@ -177,62 +173,3 @@ export function compareVersions(v1, v2) {
   }
   return false;
 }
-
-const versionCache = new Map();
-
-export function getPublishedVersion(pkg) {
-  const cacheKey = `published-${pkg}`;
-  const cached = versionCache.get(cacheKey);
-  if (cached && Date.now() - cached.timestamp < 3600000) {
-    return cached.version;
-  }
-  return null;
-}
-
-export async function fetchPublishedVersion(pkg) {
-  const cacheKey = `published-${pkg}`;
-  const cached = versionCache.get(cacheKey);
-  if (cached && Date.now() - cached.timestamp < 3600000) {
-    return cached.version;
-  }
-  try {
-    const version = await new Promise((resolve, reject) => {
-      const url = `https://registry.npmjs.org/${pkg}/latest`;
-      https.get(url, { headers: { 'User-Agent': 'agentgui-version-check/1.0' } }, (res) => {
-        let data = '';
-        res.on('data', chunk => { data += chunk; });
-        res.on('end', () => {
-          try { resolve(JSON.parse(data).version || null); } catch (e) { reject(e); }
-        });
-      }).on('error', reject);
-    });
-    if (version) versionCache.set(cacheKey, { version, timestamp: Date.now() });
-    return version;
-  } catch (_) {}
-  return null;
-}
-
-export function clearVersionCache() {
-  versionCache.clear();
-}
-
-export async function checkToolViaBunx(pkg, pluginId = null, category = 'plugin', frameWork = null, skipPublishedVersion = false, tools = []) {
-  try {
-    const isCli = category === 'cli';
-    const installed = isCli ? checkCliInstalled(pkg) : checkToolInstalled(pluginId || pkg, frameWork);
-    const installedVersion = isCli ? getCliVersion(pkg) : getInstalledVersion(pkg, pluginId, frameWork, tools);
-    let publishedVersion = null;
-    if (!skipPublishedVersion) {
-      publishedVersion = await fetchPublishedVersion(pkg);
-    }
-    const needsUpdate = installed && publishedVersion && compareVersions(installedVersion, publishedVersion);
-    const isUpToDate = installed && !needsUpdate;
-    return { installed, isUpToDate, upgradeNeeded: needsUpdate, output: 'version-check', installedVersion, publishedVersion };
-  } catch (err) {
-    console.log(`[tool-manager] Error checking ${pkg}:`, err.message);
-    const isCli = category === 'cli';
-    const installed = isCli ? checkCliInstalled(pkg) : checkToolInstalled(pluginId || pkg, frameWork);
-    const installedVersion = isCli ? getCliVersion(pkg) : getInstalledVersion(pkg, pluginId, frameWork, tools);
-    return { installed, isUpToDate: false, upgradeNeeded: false, output: '', installedVersion, publishedVersion: null };
-  }
-}

package/lib/tool-version-fetch.js

@@ -0,0 +1,61 @@
+import https from 'https';
+import { checkCliInstalled, getCliVersion, checkToolInstalled, compareVersions, getInstalledVersion } from './tool-version-check.js';
+
+const versionCache = new Map();
+
+export function getPublishedVersion(pkg) {
+  const cacheKey = `published-${pkg}`;
+  const cached = versionCache.get(cacheKey);
+  if (cached && Date.now() - cached.timestamp < 3600000) {
+    return cached.version;
+  }
+  return null;
+}
+
+export async function fetchPublishedVersion(pkg) {
+  const cacheKey = `published-${pkg}`;
+  const cached = versionCache.get(cacheKey);
+  if (cached && Date.now() - cached.timestamp < 3600000) {
+    return cached.version;
+  }
+  try {
+    const version = await new Promise((resolve, reject) => {
+      const url = `https://registry.npmjs.org/${pkg}/latest`;
+      https.get(url, { headers: { 'User-Agent': 'agentgui-version-check/1.0' } }, (res) => {
+        let data = '';
+        res.on('data', chunk => { data += chunk; });
+        res.on('end', () => {
+          try { resolve(JSON.parse(data).version || null); } catch (e) { reject(e); }
+        });
+      }).on('error', reject);
+    });
+    if (version) versionCache.set(cacheKey, { version, timestamp: Date.now() });
+    return version;
+  } catch (_) {}
+  return null;
+}
+
+export function clearVersionCache() {
+  versionCache.clear();
+}
+
+export async function checkToolViaBunx(pkg, pluginId = null, category = 'plugin', frameWork = null, skipPublishedVersion = false, tools = []) {
+  try {
+    const isCli = category === 'cli';
+    const installed = isCli ? checkCliInstalled(pkg) : checkToolInstalled(pluginId || pkg, frameWork);
+    const installedVersion = isCli ? getCliVersion(pkg) : getInstalledVersion(pkg, pluginId, frameWork, tools);
+    let publishedVersion = null;
+    if (!skipPublishedVersion) {
+      publishedVersion = await fetchPublishedVersion(pkg);
+    }
+    const needsUpdate = installed && publishedVersion && compareVersions(installedVersion, publishedVersion);
+    const isUpToDate = installed && !needsUpdate;
+    return { installed, isUpToDate, upgradeNeeded: needsUpdate, output: 'version-check', installedVersion, publishedVersion };
+  } catch (err) {
+    console.log(`[tool-manager] Error checking ${pkg}:`, err.message);
+    const isCli = category === 'cli';
+    const installed = isCli ? checkCliInstalled(pkg) : checkToolInstalled(pluginId || pkg, frameWork);
+    const installedVersion = isCli ? getCliVersion(pkg) : getInstalledVersion(pkg, pluginId, frameWork, tools);
+    return { installed, isUpToDate: false, upgradeNeeded: false, output: '', installedVersion, publishedVersion: null };
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentgui",
-  "version": "1.0.815",
+  "version": "1.0.817",
   "description": "Multi-agent ACP client with real-time communication",
   "type": "module",
   "main": "electron/main.js",