onbuzz 4.9.9 → 4.9.11

package/bin/cli.js

@@ -12,14 +12,16 @@
  *   loxia trigger-schedule <id> - Trigger a scheduled task (wakes server if needed)
  */
 
-import { fileURLToPath } from 'url';
 import { dirname, join } from 'path';
 import { spawn } from 'child_process';
 import { readFileSync } from 'fs';
 import http from 'http';
 import { runUpdateCheck } from '../src/utilities/updateNotifier.js';
+import { resolveModuleFilename } from '../src/utilities/esmCjsPath.js';
 
-const __filename = fileURLToPath(import.meta.url);
+// Dual-mode filename — works as native ESM and inside an esbuild
+// --format=cjs bundle. See src/utilities/esmCjsPath.js.
+const __filename = resolveModuleFilename(import.meta.url);
 const __dirname = dirname(__filename);
 
 // Fire the update-notifier check as soon as the process starts. The

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "onbuzz",
-  "version": "4.9.9",
+  "version": "4.9.11",
   "description": "Loxia OnBuzz - Your AI Fleet",
   "type": "module",
   "main": "src/index.js",

package/src/core/agentPool.js

@@ -673,11 +673,20 @@ class AgentPool {
   }
 
   /**
-   * Resume agent from persisted state
+   * Rehydrate an agent from persisted state on session boot. NOT the
+   * pause-toggle (see `resumeAgent(agentId)` below). The two were
+   * historically both named `resumeAgent`, which made this one dead
+   * code — JS class bodies keep the second definition only, so callers
+   * passing an object ended up in the id-based handler and silently got
+   * "Agent not found".
+   *
+   * Callers: agentPool.restoreAgent, which is in turn called from
+   * orchestrator's boot-time state restoration path.
+   *
    * @param {Object} agentData - Persisted agent data
    * @returns {Promise<Object>} Restored agent object
    */
-  async resumeAgent(agentData) {
+  async rehydrateFromState(agentData) {
     const agent = {
       ...agentData,
       status: agentData.status === 'paused' && this._isPauseExpired(agentData) ? 'active' : agentData.status
@@ -846,12 +855,13 @@ class AgentPool {
   }
 
   /**
-   * Restore agent from saved state
+   * Restore agent from saved state. Thin wrapper around rehydrateFromState
+   * kept for the public-API name orchestrator + stateManager still use.
    * @param {Object} agentState - Saved agent state
    * @returns {Promise<Object>} Restored agent
    */
   async restoreAgent(agentState) {
-    return await this.resumeAgent(agentState);
+    return await this.rehydrateFromState(agentState);
   }
 
   /**

package/src/index.js

@@ -9,9 +9,9 @@
  */
 
 import path from 'path';
-import { fileURLToPath } from 'url';
 import { exec } from 'child_process';
 import { createLogger } from './utilities/logger.js';
+import { resolveModuleFilename } from './utilities/esmCjsPath.js';
 import { createConfigManager } from './utilities/configManager.js';
 import Orchestrator from './core/orchestrator.js';
 import AgentPool from './core/agentPool.js';
@@ -152,7 +152,7 @@ class LoxiaApplication {
    * @private
    */
   async initializeConfig(options) {
-    const __filename = fileURLToPath(import.meta.url);
+    const __filename = resolveModuleFilename(import.meta.url);
     const __dirname = path.dirname(__filename);
     
     const configPaths = [
@@ -1208,8 +1208,8 @@ async function main() {
   return app;
 }
 
-// Start the application if this file is run directly
-const __filename = fileURLToPath(import.meta.url);
+// Start the application if this file is run directly.
+const __filename = resolveModuleFilename(import.meta.url);
 if (process.argv[1] === __filename) {
   console.log('🚀 Starting Loxia Autopilot One...');
   main().catch(error => {

package/src/interfaces/terminal/utils/debugLogger.js

@@ -5,9 +5,9 @@
 
 import fs from 'fs';
 import path from 'path';
-import { fileURLToPath } from 'url';
+import { resolveModuleFilename } from '../../../utilities/esmCjsPath.js';
 
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const __dirname = path.dirname(resolveModuleFilename(import.meta.url));
 const LOG_FILE = path.join(__dirname, '../../../../debug-terminal-ui.log');
 
 // Clear log file on startup

package/src/interfaces/webServer.js

@@ -14,7 +14,7 @@ import { WebSocketServer } from 'ws';
 import path from 'path';
 import { promises as fs, createReadStream } from 'fs';
 import { execSync, spawn } from 'child_process';
-import { fileURLToPath } from 'url';
+import { resolveModuleFilename } from '../utilities/esmCjsPath.js';
 
 import {
   INTERFACE_TYPES,
@@ -64,7 +64,7 @@ import { registerSchedulerRoutes } from './schedulerRoutes.js';
 import { registerAgentContextRoutes } from './agentContextRoutes.js';
 import { getMemoryService } from '../services/memoryService.js';
 
-const __filename = fileURLToPath(import.meta.url);
+const __filename = resolveModuleFilename(import.meta.url);
 const __dirname = path.dirname(__filename);
 
 // Self-contained HTML for the /scheduler debug viewer. No external deps;

package/src/services/scheduleService.js

@@ -14,6 +14,7 @@ import { promises as fs } from 'fs';
 import path from 'path';
 import { execSync } from 'child_process';
 import { getUserDataDir } from '../utilities/userDataDir.js';
+import { resolveModuleFilename } from '../utilities/esmCjsPath.js';
 
 // ========================
 // Lightweight cron parser (no external dependency)
@@ -531,7 +532,7 @@ class ScheduleService {
       return execSync(which, { encoding: 'utf-8' }).trim().split('\n')[0];
     } catch {
       // Fallback to local bin
-      const __dirname = path.dirname(new URL(import.meta.url).pathname);
+      const __dirname = path.dirname(resolveModuleFilename(import.meta.url));
       const localBin = path.resolve(__dirname, '../../bin/cli.js');
       return `node "${localBin}"`;
     }

package/src/services/visualEditorServer.js

@@ -13,7 +13,7 @@ import express from 'express';
 import { createProxyMiddleware } from 'http-proxy-middleware';
 import { WebSocketServer } from 'ws';
 import path from 'path';
-import { fileURLToPath } from 'url';
+import { resolveModuleFilename } from '../utilities/esmCjsPath.js';
 import { Transform } from 'stream';
 import https from 'https';
 import http from 'http';
@@ -36,7 +36,7 @@ function getBridge() {
   return bridgeGetter();
 }
 
-const __filename = fileURLToPath(import.meta.url);
+const __filename = resolveModuleFilename(import.meta.url);
 const __dirname = path.dirname(__filename);
 
 // Service name for registry

package/src/tools/office/officePres.js

@@ -21,7 +21,7 @@
 
 import fs from 'fs/promises';
 import path from 'path';
-import { createRequire } from 'module';
+import { resolveLocalRequire } from '../../utilities/esmCjsPath.js';
 
 // Themed-presentation system. Opt-in via `theme: '<preset-name>'` on
 // the spec — when present we route to the new builder for stunning
@@ -35,7 +35,7 @@ import { fillPres } from './pres/edit/fill.js';
 // `import JSZip from 'jszip'` which jszip (CJS-only) does not satisfy
 // cleanly under jest's experimental-vm-modules. Loading via createRequire
 // resolves the CJS build, which is well-behaved in both Node and Jest.
-const localRequire = createRequire(import.meta.url);
+const localRequire = resolveLocalRequire(import.meta.url);
 
 let pptxgen = null;
 let JSZip = null;

package/src/tools/office/pres/edit/fill.js

@@ -20,9 +20,9 @@
  * Returns { path, size, replaced: {token:count}, unresolved: [...] }.
  */
 import fs from 'fs/promises';
-import { createRequire } from 'module';
+import { resolveLocalRequire } from '../../../../utilities/esmCjsPath.js';
 
-const localRequire = createRequire(import.meta.url);
+const localRequire = resolveLocalRequire(import.meta.url);
 let JSZip = null;
 function loadZip() {
   if (JSZip) return JSZip;

package/src/tools/office/pres/edit/read.js

@@ -10,9 +10,9 @@
  * Used by the `pres.read` action.
  */
 import fs from 'fs/promises';
-import { createRequire } from 'module';
+import { resolveLocalRequire } from '../../../../utilities/esmCjsPath.js';
 
-const localRequire = createRequire(import.meta.url);
+const localRequire = resolveLocalRequire(import.meta.url);
 let JSZip = null;
 function loadZip() {
   if (JSZip) return JSZip;

package/src/tools/office/pres/index.js

@@ -21,14 +21,14 @@
  * by the caller in officePres.js.
  */
 import fs from 'fs/promises';
-import { createRequire } from 'module';
+import { resolveLocalRequire } from '../../../utilities/esmCjsPath.js';
 
 import { resolveTheme, listThemes } from './themes/index.js';
 import { renderArchetype, listArchetypes } from './archetypes/index.js';
 import { listBackgrounds } from './backgrounds/index.js';
 import { defineMasters, buildDeckContext, usesChrome } from './master.js';
 
-const localRequire = createRequire(import.meta.url);
+const localRequire = resolveLocalRequire(import.meta.url);
 let pptxgen = null;
 function loadPptx() {
   if (pptxgen) return pptxgen;

package/src/utilities/attachmentValidator.js

@@ -4,10 +4,10 @@
  */
 
 import path from 'path';
-import { fileURLToPath } from 'url';
 import { getSystemRestrictedPaths, getPlatformBlockedExtensions } from './platformUtils.js';
+import { resolveModuleFilename } from './esmCjsPath.js';
 
-const __filename = fileURLToPath(import.meta.url);
+const __filename = resolveModuleFilename(import.meta.url);
 const __dirname = path.dirname(__filename);
 
 // Constants

package/src/utilities/brand.js

@@ -20,9 +20,9 @@
 
 import { readFileSync } from 'fs';
 import path from 'path';
-import { fileURLToPath } from 'url';
+import { resolveModuleFilename } from './esmCjsPath.js';
 
-const __filename = fileURLToPath(import.meta.url);
+const __filename = resolveModuleFilename(import.meta.url);
 const __dirname  = path.dirname(__filename);
 
 let cached = null;

package/src/utilities/esmCjsPath.js

@@ -0,0 +1,51 @@
+/**
+ * ESM/CJS dual-mode path + require helpers.
+ *
+ * Required because we ship the same source two ways:
+ *   1. Native ESM via `node bin/cli.js` / `npx loxia` — `import.meta` is
+ *      a real object and `module`/`require` are not defined.
+ *   2. Bundled CJS via `esbuild --format=cjs` for the standalone binary
+ *      build — `import.meta.url` is empty (esbuild emits a warning and
+ *      sets it to an empty object), so calling `fileURLToPath` on it
+ *      throws ERR_INVALID_ARG_TYPE at module load. Meanwhile Node's
+ *      CJS wrapper provides `module.filename` and `require` directly.
+ *
+ * `typeof` on an undeclared identifier is the one expression that's
+ * safe in both contexts — it returns `'undefined'` without throwing —
+ * so we use it to discriminate.
+ *
+ * Sibling-lookup math note: in the bundle, `module.filename` is the
+ * bundled file's path (e.g. `dist-bundle/cli.cjs`), not the original
+ * source file's path. That's intentional — pkg's snapshot packages the
+ * bundle, not the source tree, so paths derived from `__dirname` need
+ * to resolve relative to the bundle.
+ */
+
+import { fileURLToPath } from 'url';
+import { createRequire } from 'module';
+
+/**
+ * Pick the right `__filename` for the calling module.
+ *
+ * @param {string|undefined} metaUrl - pass `import.meta.url` from the call site
+ * @returns {string} absolute path of the module's file
+ */
+export function resolveModuleFilename(metaUrl) {
+  if (typeof require !== 'undefined' && typeof module !== 'undefined' && module.filename) {
+    return module.filename;
+  }
+  return fileURLToPath(metaUrl);
+}
+
+/**
+ * Pick the right `require` for the calling module.
+ *
+ * @param {string|undefined} metaUrl - pass `import.meta.url` from the call site
+ * @returns {NodeRequire}
+ */
+export function resolveLocalRequire(metaUrl) {
+  if (typeof require !== 'undefined') {
+    return require;
+  }
+  return createRequire(metaUrl);
+}

package/src/utilities/updateNotifier.js

@@ -32,7 +32,7 @@
 
 import { readFileSync } from 'fs';
 import { dirname, join } from 'path';
-import { fileURLToPath } from 'url';
+import { resolveModuleFilename } from './esmCjsPath.js';
 
 /**
  * Env vars that disable the check. All are case-sensitive string
@@ -65,8 +65,13 @@ export function isUpdateCheckDisabled(env = process.env) {
  *
  * @returns {{ name: string, version: string } | null}
  */
+// Module-load-time default for startDir, in a way that works both as
+// native ESM and inside the esbuild --format=cjs bundle. See
+// src/utilities/esmCjsPath.js.
+const _defaultStartDir = dirname(resolveModuleFilename(import.meta.url));
+
 export function loadPackageJson({
-  startDir = dirname(fileURLToPath(import.meta.url)),
+  startDir = _defaultStartDir,
   fs = { readFileSync },
 } = {}) {
   // Walk up from this file looking for package.json with a `bin`