@yeaft/webchat-agent 1.0.412 → 1.0.414

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "1.0.412",
+  "version": "1.0.414",
   "description": "Remote worker agent for Yeaft Web Code Agent — connects the native Yeaft engine, CLI providers, and workbench tools",
   "main": "index.js",
   "type": "module",
@@ -41,6 +41,8 @@
     "docker-entrypoint.sh",
     "yeaft/templates/**/*.md",
     "yeaft/dream/prompts/**/*.md",
+    "browser-runtime/extension/**/*",
+    "browser-runtime/*.ps1",
     "local-runtime/**/*"
   ],
   "license": "MIT",
@@ -62,6 +64,7 @@
   },
   "dependencies": {
     "@homebridge/node-pty-prebuilt-multiarch": "^0.13.1",
+    "@puppeteer/browsers": "2.13.0",
     "bcrypt": "^6.0.0",
     "compression": "^1.8.1",
     "dotenv": "^16.3.1",
@@ -69,6 +72,7 @@
     "jsonwebtoken": "^9.0.2",
     "multer": "^2.2.0",
     "nodemailer": "^9.0.3",
+    "puppeteer-core": "24.41.0",
     "qrcode": "^1.5.4",
     "speakeasy": "^2.0.0",
     "tweetnacl": "^1.0.3",

package/service/config.js

@@ -113,7 +113,7 @@ export function resolveServiceInstanceId(args = [], env = process.env, options =
 }
 
 /** Resolve the CLI/env/default Yeaft data root for one Agent instance. */
-export function resolveYeaftDir(args = [], env = process.env, instanceId = DEFAULT_INSTANCE_ID) {
+function explicitYeaftDirFromArgs(args = []) {
   let explicitYeaftDir = null;
   for (let index = 0; index < args.length; index += 1) {
     if (args[index] !== '--yeaft-dir') continue;
@@ -122,7 +122,28 @@ export function resolveYeaftDir(args = [], env = process.env, instanceId = DEFAU
     explicitYeaftDir = value;
     index += 1;
   }
-  return explicitYeaftDir || env.YEAFT_DIR || getDefaultYeaftDir(instanceId);
+  return explicitYeaftDir;
+}
+
+export function resolveYeaftDir(args = [], env = process.env, instanceId = DEFAULT_INSTANCE_ID) {
+  return explicitYeaftDirFromArgs(args) || env.YEAFT_DIR || getDefaultYeaftDir(instanceId);
+}
+
+/** Resolve the persisted data root for an explicitly selected management target. */
+export function resolveManagedYeaftDir(
+  args = [],
+  env = process.env,
+  instanceId = DEFAULT_INSTANCE_ID,
+  dependencies = {},
+) {
+  const explicitYeaftDir = explicitYeaftDirFromArgs(args);
+  if (explicitYeaftDir) return explicitYeaftDir;
+  const hasExplicitIdentity = args.includes('--name') || args.includes('--instance');
+  if (!hasExplicitIdentity && env.YEAFT_DIR) return env.YEAFT_DIR;
+  const readServiceConfig = dependencies.loadServiceConfig || loadServiceConfig;
+  const defaultYeaftDir = dependencies.getDefaultYeaftDir || getDefaultYeaftDir;
+  const persisted = readServiceConfig(instanceId)?.yeaftDir;
+  return persisted || defaultYeaftDir(instanceId);
 }
 
 /** Legacy alias for resolveServiceInstanceId(). */

package/service/index.js

@@ -29,6 +29,7 @@ export {
   resolveDisplayName,
   resolveRuntimeIdentity,
   resolveServiceInstanceId,
+  resolveManagedYeaftDir,
   shouldLoadLegacyLocalConfig,
   normalizeInstanceId,
   isDefaultInstance,

package/service/linux.js

@@ -12,7 +12,7 @@ export function getSystemdServicePath(instanceId = DEFAULT_INSTANCE_ID) {
   return join(homedir(), '.config', 'systemd', 'user', `${getServiceName(instanceId)}.service`);
 }
 
-function generateSystemdUnit(config) {
+export function generateSystemdUnit(config) {
   const nodePath = getNodePath();
   const cliPath = getCliPath();
   const logDir = getLogDir(config.instanceId);
@@ -38,7 +38,8 @@ ExecStart=${nodePath} ${cliPath}
 WorkingDirectory=${config.workDir || homedir()}
 Restart=on-failure
 RestartSec=10
-KillMode=process
+KillMode=mixed
+TimeoutStopSec=15
 ${envLines.join('\n')}
 Environment=PATH=${nodeBinDir}:${homedir()}/.local/bin:${homedir()}/.npm-global/bin:/usr/local/bin:/usr/bin:/bin
 

package/yeaft/config-api.js

@@ -8,12 +8,14 @@
  * like maxContinueTurns or debug that don't belong in the UI.
  */
 
-import { existsSync, readFileSync, writeFileSync } from 'fs';
+import { existsSync, readFileSync } from 'fs';
 import { join } from 'path';
 import { DEFAULT_YEAFT_DIR } from './init.js';
 import { normalizeProviderModels, parseModelRef, serializeModelForPersistence } from './models.js';
 import { normaliseTelemetrySection, normaliseYeaftSection } from './config.js';
+import { normaliseBrowserRuntimeSection, validateBrowserRuntimeUpdate } from '../browser-runtime/config.js';
 import { normalizePluginConfig } from './plugins.js';
+import { mutateAgentConfig, readAgentConfigForWrite } from './config-store.js';
 import { isGitHubCopilotProvider, serializeKnownProviderForPersistence } from './llm/known-providers.js';
 
 /**
@@ -28,16 +30,7 @@ import { isGitHubCopilotProvider, serializeKnownProviderForPersistence } from '.
  * @throws {Error} when an existing config cannot be safely preserved
  */
 function readConfigForWrite(configPath) {
-  if (!existsSync(configPath)) return {};
-  const json = JSON.parse(readFileSync(configPath, 'utf8'));
-  if (!json || typeof json !== 'object' || Array.isArray(json)
-    || Object.getPrototypeOf(json) !== Object.prototype) {
-    throw new Error('config.json must contain an object');
-  }
-  if (Object.prototype.hasOwnProperty.call(json, 'plugins')) {
-    normalizePluginConfig(json.plugins);
-  }
-  return json;
+  return readAgentConfigForWrite(configPath);
 }
 
 /**
@@ -142,85 +135,57 @@ function normalizeManagedModelDefaults(config) {
  */
 export function updateLlmConfig(update, dir) {
   const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
-  const configPath = join(root, 'config.json');
-
-  // Preserve all existing fields only when the on-disk document and its
-  // Plugins policy are valid. Never turn a failed read into a fresh config.
-  let existing;
-  try {
-    existing = readConfigForWrite(configPath);
-  } catch (err) {
-    return { error: `Failed to read config.json: ${err?.message || err}` };
-  }
 
-  // Validate providers structure
   if (update.providers !== undefined) {
-    if (!Array.isArray(update.providers)) {
-      return { error: 'providers must be an array' };
-    }
-    for (const p of update.providers) {
-      if (!p.name || typeof p.name !== 'string') {
+    if (!Array.isArray(update.providers)) return { error: 'providers must be an array' };
+    for (const provider of update.providers) {
+      if (!provider.name || typeof provider.name !== 'string') {
         return { error: 'Each provider must have a name' };
       }
-      if (isGitHubCopilotProvider(p)) continue;
-      if (!p.baseUrl || typeof p.baseUrl !== 'string') {
-        return { error: `Provider "${p.name}" must have a baseUrl` };
+      if (isGitHubCopilotProvider(provider)) continue;
+      if (!provider.baseUrl || typeof provider.baseUrl !== 'string') {
+        return { error: `Provider "${provider.name}" must have a baseUrl` };
       }
-      if (!Array.isArray(p.models) || p.models.length === 0) {
-        return { error: `Provider "${p.name}" must have at least one model` };
+      if (!Array.isArray(provider.models) || provider.models.length === 0) {
+        return { error: `Provider "${provider.name}" must have at least one model` };
       }
     }
-    // Normalize + re-serialize each provider's models so that:
-    //   - id-only entries are persisted as plain strings (back-compat)
-    //   - entries with ctx / maxOutput are persisted as objects
-    //   - empty / 0 / NaN values get stripped
-    existing.providers = update.providers.map(p => {
-      const managed = serializeKnownProviderForPersistence(p);
-      if (managed) return managed;
-      const normalized = normalizeProviderModels(p);
-      return {
-        ...p,
-        models: normalized.map(serializeModelForPersistence),
-      };
-    });
   }
 
-  // Update model selections. A managed provider catalog is authoritative:
-  // when it drops the old Agent default, keep no hidden reference to that
-  // model in primaryModel or fastModel.
-  if (update.primaryModel !== undefined) {
-    existing.primaryModel = update.primaryModel || null;
-  }
-  if (update.fastModel !== undefined) {
-    existing.fastModel = update.fastModel || null;
-  }
-  if (update.providers !== undefined) normalizeManagedModelDefaults(existing);
-  if (update.language !== undefined) {
-    existing.language = update.language;
-  }
-  if (update.debug !== undefined) {
-    existing.debug = update.debug === true;
-  }
-
-  // Write back
   try {
-    writeFileSync(configPath, JSON.stringify(existing, null, 2) + '\n', 'utf8');
-  } catch (e) {
-    return { error: `Failed to write config.json: ${e.message}` };
+    return mutateAgentConfig(root, existing => {
+      if (update.providers !== undefined) {
+        existing.providers = update.providers.map(provider => {
+          const managed = serializeKnownProviderForPersistence(provider);
+          if (managed) return managed;
+          return {
+            ...provider,
+            models: normalizeProviderModels(provider).map(serializeModelForPersistence),
+          };
+        });
+      }
+      if (update.primaryModel !== undefined) existing.primaryModel = update.primaryModel || null;
+      if (update.fastModel !== undefined) existing.fastModel = update.fastModel || null;
+      if (update.providers !== undefined) normalizeManagedModelDefaults(existing);
+      if (update.language !== undefined) existing.language = update.language;
+      if (update.debug !== undefined) existing.debug = update.debug === true;
+
+      const agentConfig = {
+        providers: Array.isArray(existing.providers) ? existing.providers : [],
+        primaryModel: existing.primaryModel || null,
+        fastModel: existing.fastModel || null,
+        language: existing.language || 'en',
+        debug: existing.debug === true,
+      };
+      return {
+        ...agentConfig,
+        agentConfig,
+        effectiveConfig: agentConfig,
+      };
+    });
+  } catch (error) {
+    return { error: `Failed to read config.json or persist update: ${error?.message || error}` };
   }
-
-  const agentConfig = {
-    providers: Array.isArray(existing.providers) ? existing.providers : [],
-    primaryModel: existing.primaryModel || null,
-    fastModel: existing.fastModel || null,
-    language: existing.language || 'en',
-    debug: existing.debug === true,
-  };
-  return {
-    ...agentConfig,
-    agentConfig,
-    effectiveConfig: agentConfig,
-  };
 }
 
 // ─── Yeaft runtime settings (task-318) ────────────────────────────
@@ -261,7 +226,6 @@ export function getYeaftSettings(dir) {
  */
 export function updateYeaftSettings(update, dir) {
   const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
-  const configPath = join(root, 'config.json');
 
   if (!update || typeof update !== 'object') {
     return { error: 'update payload required' };
@@ -288,37 +252,26 @@ export function updateYeaftSettings(update, dir) {
     }
   }
 
-  // Preserve all existing fields only when the on-disk document and its
-  // Plugins policy are valid. A Settings update must not repair bad JSON into
-  // a config whose missing Plugins fields inherit all capabilities.
-  let existing;
-  try {
-    existing = readConfigForWrite(configPath);
-  } catch (err) {
-    return { error: `Failed to read config.json: ${err?.message || err}` };
-  }
-
-  const prev = normaliseYeaftSection(existing.yeaft);
-  const merged = {
-    maxConcurrentThreads: update.maxConcurrentThreads !== undefined
-      ? Math.floor(Number(update.maxConcurrentThreads))
-      : prev.maxConcurrentThreads,
-    autoArchiveIdleDays: update.autoArchiveIdleDays !== undefined
-      ? Math.floor(Number(update.autoArchiveIdleDays))
-      : prev.autoArchiveIdleDays,
-    recentTurnsLimit: update.recentTurnsLimit !== undefined
-      ? Math.floor(Number(update.recentTurnsLimit))
-      : prev.recentTurnsLimit,
-  };
-  existing.yeaft = merged;
-
   try {
-    writeFileSync(configPath, JSON.stringify(existing, null, 2) + '\n', 'utf8');
-  } catch (e) {
-    return { error: `Failed to write config.json: ${e.message}` };
+    return mutateAgentConfig(root, existing => {
+      const prev = normaliseYeaftSection(existing.yeaft);
+      const merged = {
+        maxConcurrentThreads: update.maxConcurrentThreads !== undefined
+          ? Math.floor(Number(update.maxConcurrentThreads))
+          : prev.maxConcurrentThreads,
+        autoArchiveIdleDays: update.autoArchiveIdleDays !== undefined
+          ? Math.floor(Number(update.autoArchiveIdleDays))
+          : prev.autoArchiveIdleDays,
+        recentTurnsLimit: update.recentTurnsLimit !== undefined
+          ? Math.floor(Number(update.recentTurnsLimit))
+          : prev.recentTurnsLimit,
+      };
+      existing.yeaft = merged;
+      return merged;
+    });
+  } catch (error) {
+    return { error: `Failed to read config.json or persist update: ${error?.message || error}` };
   }
-
-  return merged;
 }
 
 /**
@@ -357,25 +310,50 @@ export function updateTelemetrySettings(update, dir) {
   if (Object.keys(update).some(key => !allowed.has(key))) {
     return { error: 'unknown telemetry setting' };
   }
+  const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
+  try {
+    return mutateAgentConfig(root, existing => {
+      const merged = normaliseTelemetrySection({
+        ...(existing.telemetry && typeof existing.telemetry === 'object' ? existing.telemetry : {}),
+        ...update,
+      });
+      existing.telemetry = merged;
+      return merged;
+    });
+  } catch (error) {
+    return { error: `Failed to read config.json or persist update: ${error?.message || error}` };
+  }
+}
+
+// ─── Browser Runtime settings ───────────────────────────────────────
+
+export function getBrowserRuntimeSettings(dir) {
   const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
   const configPath = join(root, 'config.json');
-  let existing;
+  if (!existsSync(configPath)) return normaliseBrowserRuntimeSection(null);
   try {
-    existing = readConfigForWrite(configPath);
-  } catch (err) {
-    return { error: `Failed to read config.json: ${err?.message || err}` };
+    const json = JSON.parse(readFileSync(configPath, 'utf8'));
+    return normaliseBrowserRuntimeSection(json.browserRuntime);
+  } catch (error) {
+    return { error: `Failed to read config.json: ${error.message}` };
   }
-  const merged = normaliseTelemetrySection({
-    ...(existing.telemetry && typeof existing.telemetry === 'object' ? existing.telemetry : {}),
-    ...update,
-  });
-  existing.telemetry = merged;
+}
+
+export function updateBrowserRuntimeSettings(update, dir) {
+  const validationError = validateBrowserRuntimeUpdate(update);
+  if (validationError) return { error: validationError };
+  const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
   try {
-    writeFileSync(configPath, JSON.stringify(existing, null, 2) + '\n', 'utf8');
-  } catch (e) {
-    return { error: `Failed to write config.json: ${e.message}` };
+    return mutateAgentConfig(root, existing => {
+      const previous = existing.browserRuntime && typeof existing.browserRuntime === 'object'
+        ? existing.browserRuntime
+        : {};
+      existing.browserRuntime = normaliseBrowserRuntimeSection({ ...previous, ...update });
+      return existing.browserRuntime;
+    });
+  } catch (error) {
+    return { error: `Failed to read config.json or persist update: ${error?.message || error}` };
   }
-  return merged;
 }
 
 // ─── Search settings (web-search backend selection + Tavily key) ────
@@ -443,7 +421,6 @@ export function getSearchSettings(dir) {
  */
 export function updateSearchSettings(update, dir) {
   const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
-  const configPath = join(root, 'config.json');
 
   if (!update || typeof update !== 'object') {
     return { error: 'update payload required' };
@@ -455,23 +432,17 @@ export function updateSearchSettings(update, dir) {
     return { error: 'tavilyApiKey must be a string' };
   }
 
-  let existing;
   try {
-    existing = readConfigForWrite(configPath);
-  } catch (err) {
-    return { error: `Failed to read config.json: ${err?.message || err}` };
-  }
-  const prev = (existing && typeof existing.search === 'object' && existing.search) || {};
-  const merged = { ...prev };
-  if (update.backend !== undefined) merged.backend = update.backend;
-  if (update.tavilyApiKey !== undefined) merged.tavilyApiKey = update.tavilyApiKey;
-  if (update.disableHtmlFallback !== undefined) merged.disableHtmlFallback = !!update.disableHtmlFallback;
-  existing.search = merged;
-
-  try {
-    writeFileSync(configPath, JSON.stringify(existing, null, 2) + '\n', 'utf8');
-  } catch (e) {
-    return { error: `Failed to write config.json: ${e.message}` };
+    mutateAgentConfig(root, existing => {
+      const prev = (existing && typeof existing.search === 'object' && existing.search) || {};
+      const merged = { ...prev };
+      if (update.backend !== undefined) merged.backend = update.backend;
+      if (update.tavilyApiKey !== undefined) merged.tavilyApiKey = update.tavilyApiKey;
+      if (update.disableHtmlFallback !== undefined) merged.disableHtmlFallback = !!update.disableHtmlFallback;
+      existing.search = merged;
+    });
+  } catch (error) {
+    return { error: `Failed to read config.json or persist update: ${error?.message || error}` };
   }
   return getSearchSettings(root);
 }
@@ -549,24 +520,17 @@ export function getPluginConfig(dir) {
  */
 export function updatePluginConfig(plugins, dir) {
   const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
-  const configPath = join(root, 'config.json');
   let normalized;
-  let existing;
   try {
-    existing = readConfigForWrite(configPath);
     normalized = normalizePluginConfig(plugins);
-  } catch (err) {
-    return { error: `Failed to read plugin config: ${err?.message || err}` };
-  }
-
-  if (Object.keys(normalized).length === 0) delete existing.plugins;
-  else existing.plugins = normalized;
-  try {
-    writeFileSync(configPath, JSON.stringify(existing, null, 2) + '\n', 'utf8');
-  } catch (err) {
-    return { error: `Failed to write plugin config: ${err?.message || err}` };
+    return mutateAgentConfig(root, existing => {
+      if (Object.keys(normalized).length === 0) delete existing.plugins;
+      else existing.plugins = normalized;
+      return { plugins: normalized };
+    });
+  } catch (error) {
+    return { error: `Failed to read plugin config or persist update: ${error?.message || error}` };
   }
-  return { plugins: normalized };
 }
 
 // ─── MCP server config (mcpServers array in config.json) ──
@@ -684,33 +648,21 @@ export function upsertMcpServer(server, dir) {
   if (err) return { error: err };
 
   const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
-  const configPath = join(root, 'config.json');
-  let existing;
-  try {
-    existing = readConfigForWrite(configPath);
-  } catch (err) {
-    return { error: `Failed to read config.json: ${err?.message || err}` };
-  }
-  const list = Array.isArray(existing.mcpServers) ? existing.mcpServers.slice() : [];
-
   const normalised = normaliseMcpServer(server);
   if (!normalised) return { error: 'invalid server payload' };
 
-  const idx = list.findIndex(s => s && typeof s === 'object' && s.name === normalised.name);
-  if (idx >= 0) {
-    list[idx] = normalised;
-  } else {
-    list.push(normalised);
-  }
-  existing.mcpServers = list;
-
   try {
-    writeFileSync(configPath, JSON.stringify(existing, null, 2) + '\n', 'utf8');
-  } catch (e) {
-    return { error: `Failed to write config.json: ${e.message}` };
+    return mutateAgentConfig(root, existing => {
+      const list = Array.isArray(existing.mcpServers) ? existing.mcpServers.slice() : [];
+      const index = list.findIndex(entry => entry && typeof entry === 'object' && entry.name === normalised.name);
+      if (index >= 0) list[index] = normalised;
+      else list.push(normalised);
+      existing.mcpServers = list;
+      return { servers: list.map(normaliseMcpServer).filter(Boolean), server: normalised };
+    });
+  } catch (error) {
+    return { error: `Failed to read config.json or persist update: ${error?.message || error}` };
   }
-
-  return { servers: list.map(normaliseMcpServer).filter(Boolean), server: normalised };
 }
 
 /**
@@ -732,24 +684,18 @@ export function removeMcpServer(name, dir) {
   // matching against the padded string.
   const target = name.trim();
   const root = dir || process.env.YEAFT_DIR || DEFAULT_YEAFT_DIR;
-  const configPath = join(root, 'config.json');
-  let existing;
-  try {
-    existing = readConfigForWrite(configPath);
-  } catch (err) {
-    return { error: `Failed to read config.json: ${err?.message || err}` };
-  }
-  const list = Array.isArray(existing.mcpServers) ? existing.mcpServers.slice() : [];
-  const next = list.filter(s => !(s && typeof s === 'object' && s.name === target));
-  const removed = next.length !== list.length;
-  existing.mcpServers = next;
-
   try {
-    writeFileSync(configPath, JSON.stringify(existing, null, 2) + '\n', 'utf8');
-  } catch (e) {
-    return { error: `Failed to write config.json: ${e.message}` };
+    return mutateAgentConfig(root, existing => {
+      const list = Array.isArray(existing.mcpServers) ? existing.mcpServers.slice() : [];
+      const next = list.filter(entry => !(entry && typeof entry === 'object' && entry.name === target));
+      existing.mcpServers = next;
+      return {
+        servers: next.map(normaliseMcpServer).filter(Boolean),
+        removed: next.length !== list.length,
+      };
+    });
+  } catch (error) {
+    return { error: `Failed to read config.json or persist update: ${error?.message || error}` };
   }
-
-  return { servers: next.map(normaliseMcpServer).filter(Boolean), removed };
 }