@yeaft/webchat-agent 0.1.780 → 0.1.781

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "0.1.780",
+  "version": "0.1.781",
   "description": "Remote agent for Yeaft WebChat — connects worker machines to the central server",
   "main": "index.js",
   "type": "module",

package/unify/session.js

@@ -50,7 +50,7 @@ import { openSegmentIndex } from './memory/index-db.js';
 import { syncAll as syncSegmentIndex } from './memory/segment-sync.js';
 import { openAmsRegistry } from './memory/ams-registry.js';
 import { join } from 'path';
-import { existsSync as existsSyncSafe, readFileSync as readFileSyncSafe } from 'fs';
+import { existsSync as existsSyncSafe, readFileSync as readFileSyncSafe, mkdirSync as mkdirSyncSafe } from 'fs';
 
 /**
  * @typedef {Object} SessionOptions
@@ -80,6 +80,33 @@ import { existsSync as existsSyncSafe, readFileSync as readFileSyncSafe } from '
  * @property {() => Promise<void>} shutdown — Graceful shutdown
  */
 
+/**
+ * Eagerly create `<yeaftDir>/stats/` and surface failures as a warn.
+ *
+ * Returns the resolved path either way — the caller can still pass it
+ * to `ToolUsageStats`, which keeps an in-memory counter path even when
+ * the disk is read-only.
+ *
+ * NOTE: this knowledge ("stats lives under `stats/`") belongs inside
+ * `ToolUsageStats.init()`. Once that exists, delete this helper and
+ * the call site collapses to `await toolStats.init(yeaftDir)`.
+ *
+ * @param {string} yeaftDir
+ * @returns {string} statsDir
+ */
+function prepareToolStatsDir(yeaftDir) {
+  const statsDir = join(yeaftDir, 'stats');
+  try {
+    mkdirSyncSafe(statsDir, { recursive: true });
+  } catch (err) {
+    console.warn(
+      `[Unify] Could not create stats dir ${statsDir}: ${err?.message || err}. ` +
+      `Tool-usage counters will live in memory only.`
+    );
+  }
+  return statsDir;
+}
+
 /**
  * Load (or initialize) a Yeaft session.
  *
@@ -320,8 +347,24 @@ export async function loadSession(options = {}) {
   // Tool-call usage statistics: persisted to <yeaftDir>/stats/tool-usage.json.
   // Loaded synchronously at boot so the first turn already sees prior counts.
   // Threaded into the engine so it can `record` each tool_exec event.
+  //
+  // 2026-05-16: eagerly create the `stats/` directory at boot. The
+  // ToolUsageStats writer does `fsp.mkdir(..., {recursive:true})` lazily
+  // inside `#doFlush()` and swallows any mkdir error, which meant an
+  // unwritable parent (perm denied, ENOSPC) was silently invisible
+  // until the user filed a support ticket. Doing it here surfaces the
+  // failure as a console.warn while still leaving the in-memory
+  // counter path functional — the engine keeps recording even if the
+  // disk is read-only.
+  //
+  // FOLLOW-UP: this leaks `ToolUsageStats`'s storage layout (its
+  // directory name) into the session orchestrator. The right home is
+  // a `ToolUsageStats.init()` that owns the mkdir + the warn + a
+  // `writesDisabled` flag. Tracking as future work; for now the helper
+  // below visually quarantines the leak so the migration is one delete.
+  const statsDir = prepareToolStatsDir(yeaftDir);
   const toolStats = new ToolUsageStats({
-    path: join(yeaftDir, 'stats', 'tool-usage.json'),
+    path: join(statsDir, 'tool-usage.json'),
   });
   toolStats.loadSync();
   const engine = new Engine({