fraim-hub 2.0.318 → 2.0.319

package/dist/src/ai-hub/conversation-store.js

@@ -652,12 +652,12 @@ class AiHubConversationStore {
         if (fs_1.default.existsSync(path_1.default.join(dir, 'pending-head.json')) || fs_1.default.existsSync(path_1.default.join(dir, 'header-repair.json')))
             (0, conversation_store_lock_1.withBucketLock)(this.lockPath(dir), () => this.repairDerived(dir, key));
         const idx = this.loadIndex(dir, key);
-        if (idx.headers.length && !idx.counts) {
+        // A legacy index (headers with no precomputed counts) or a rebuilding one both queue
+        // the same background upgrade, but neither blocks the read: counts are computed
+        // on the fly below from the headers already on hand, same as pre-#1682 behavior.
+        if ((idx.headers.length && !idx.counts) || idx.rebuilding) {
             (0, history_migration_1.queueHistoryMigration)({ stateFile: this.stateFilePath, source: this.indexPath(dir), mode: 'index', bucketKey: key });
-            throw new history_policy_1.HistoryStoreError('history_migrating');
         }
-        if (idx.rebuilding)
-            (0, history_migration_1.queueHistoryMigration)({ stateFile: this.stateFilePath, source: this.indexPath(dir), mode: 'index', bucketKey: key });
         const counts = idx.counts ?? (0, conversation_ui_state_1.emptyConversationCounts)();
         if (!idx.counts)
             for (const header of idx.headers)
@@ -1130,19 +1130,21 @@ class AiHubConversationStore {
                 return cached.index;
             }
             this.hotIndexCache.delete(bucketDir);
-            if (stat.size > history_policy_1.HISTORY_POLICY.indexBytes) {
-                if (this.options.skipLegacyMigration)
-                    return null;
+            // An oversized or over-long legacy index queues its own background upgrade to
+            // bounded headers, but a read must never block on that migration: it still parses
+            // and returns the index directly below, just without the hot in-memory cache
+            // (cacheIndex already refuses to cache anything this large, so nothing leaks).
+            const oversizedFile = stat.size > history_policy_1.HISTORY_POLICY.indexBytes;
+            if (oversizedFile && this.options.skipLegacyMigration)
+                return null;
+            if (oversizedFile)
                 (0, history_migration_1.queueHistoryMigration)({ stateFile: this.stateFilePath, source: file, mode: 'index' });
-                throw new history_policy_1.HistoryStoreError('history_migrating');
-            }
             const parsed = JSON.parse(fs_1.default.readFileSync(file, 'utf8'));
             if (Array.isArray(parsed.headers)) {
                 if (parsed.headers.length > history_policy_1.HISTORY_POLICY.hotHeaders) {
                     if (this.options.skipLegacyMigration)
                         return null;
                     (0, history_migration_1.queueHistoryMigration)({ stateFile: this.stateFilePath, source: file, mode: 'index' });
-                    throw new history_policy_1.HistoryStoreError('history_migrating');
                 }
                 return { ...parsed, activeId: parsed.activeId ?? null, headers: parsed.headers };
             }

package/dist/src/ai-hub/history-migration.js

@@ -32,8 +32,11 @@ function queueHistoryMigration(request) {
     if (running)
         return;
     const priorFailure = failures.get(request.source);
-    if (priorFailure)
-        throw new history_policy_1.HistoryStoreError(priorFailure);
+    if (priorFailure) {
+        if (Date.now() - priorFailure.at < history_policy_1.HISTORY_POLICY.migrationRetryAfterMs)
+            throw new history_policy_1.HistoryStoreError(priorFailure.code);
+        failures.delete(request.source);
+    }
     const worker = new worker_threads_1.Worker(`
     const { workerData, parentPort } = require('worker_threads');
     if (workerData.module.endsWith('.ts')) require('tsx/cjs');
@@ -50,12 +53,12 @@ function queueHistoryMigration(request) {
             }
             if (!message.ok && message.code !== 'history_migrating' && message.code !== 'history_upgrade_required') {
                 workerCounters.failures++;
-                failures.set(request.source, message.code);
+                failures.set(request.source, { code: message.code, at: Date.now() });
                 if (failures.size > 16)
                     failures.delete(failures.keys().next().value);
             }
         });
-        worker.on('error', () => { workerCounters.failures++; failures.set(request.source, 'history_migration_failed'); });
+        worker.on('error', () => { workerCounters.failures++; failures.set(request.source, { code: 'history_migration_failed', at: Date.now() }); });
         worker.on('exit', () => { running = undefined; resolve(); });
     });
     running = { source: request.source, completion };

package/dist/src/ai-hub/history-policy.js

@@ -16,6 +16,14 @@ exports.HISTORY_POLICY = Object.freeze({
     maxPageSize: 100,
     responseBytes: 2 * 1024 * 1024,
     controlDocumentBytes: 16 * 1024 * 1024,
+    // A migration source that just failed (e.g. a transient Windows file lock) is retried
+    // after this window instead of being blacklisted for the life of the process.
+    migrationRetryAfterMs: 30 * 1000,
+    // A writer-compatibility registration this old is treated as stale regardless of
+    // whether its recorded pid currently matches a live process: Windows recycles pids
+    // quickly, and a crashed writer that never removed its own file can otherwise match
+    // an unrelated later process and block migration store-wide, forever.
+    staleRuntimeRegistrationMs: 24 * 60 * 60 * 1000,
 });
 class HistoryStoreError extends Error {
     constructor(code, statusCode = 503) {

package/dist/src/ai-hub/history-writer-compatibility.js

@@ -41,7 +41,12 @@ function assertHistoryMigrationAllowed(stateFile, ownerPid) {
             }
             if (!Number.isSafeInteger(runtime.pid) || runtime.pid <= 0)
                 throw new history_policy_1.HistoryStoreError('history_upgrade_required');
-            if (runtime.pid === process.pid || !(0, process_liveness_1.isPidAlive)(runtime.pid))
+            // A pid match alone is not proof of liveness for an old registration: Windows
+            // recycles pids quickly, so a crashed writer's leftover file can otherwise match
+            // an unrelated process started long afterward.
+            const startedAtMs = typeof runtime.startedAt === 'string' ? Date.parse(runtime.startedAt) : NaN;
+            const stale = Number.isFinite(startedAtMs) && Date.now() - startedAtMs > history_policy_1.HISTORY_POLICY.staleRuntimeRegistrationMs;
+            if (runtime.pid === process.pid || stale || !(0, process_liveness_1.isPidAlive)(runtime.pid))
                 continue;
             if (runtime.historyFormatVersion !== 3)
                 throw new history_policy_1.HistoryStoreError('history_upgrade_required');

package/dist/src/cli/mcp/fraim-mcp-latest-launcher.js

@@ -138,6 +138,16 @@ function runtimeFromEnv(env = process.env) {
         ? { executable: env.FRAIM_PACKAGED_CLI_EXECUTABLE, script: env.FRAIM_PACKAGED_CLI_SCRIPT }
         : null;
 }
+// Regular Node.js (and ELECTRON_RUN_AS_NODE) cannot see inside Electron's
+// .asar archives. For paths like …/app.asar/node_modules/…, verify the
+// archive file itself rather than the internal entry.
+function existsOrAsarExists(filePath) {
+    const normalized = filePath.replace(/\\/g, '/');
+    const asarSep = normalized.indexOf('.asar/');
+    if (asarSep === -1)
+        return fs_1.default.existsSync(filePath);
+    return fs_1.default.existsSync(filePath.slice(0, asarSep + '.asar'.length));
+}
 function getPackagedFraimRuntime(env = process.env) {
     const envRuntime = runtimeFromEnv(env);
     if (envRuntime)
@@ -146,7 +156,7 @@ function getPackagedFraimRuntime(env = process.env) {
         const raw = JSON.parse(fs_1.default.readFileSync(getPackagedFraimRuntimePath(), 'utf8'));
         if (!raw.executable || !raw.script)
             return null;
-        if (!fs_1.default.existsSync(raw.executable) || !fs_1.default.existsSync(raw.script))
+        if (!fs_1.default.existsSync(raw.executable) || !existsOrAsarExists(raw.script))
             return null;
         return { executable: raw.executable, script: raw.script };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim-hub",
-  "version": "2.0.318",
+  "version": "2.0.319",
   "description": "FRAIM Hub local companion package.",
   "author": "Sid Mathur <sid.mathur@gmail.com>",
   "homepage": "https://github.com/mathursrus/FRAIM#readme",
@@ -212,7 +212,7 @@
     "electron-updater": "^6.8.9",
     "express": "^5.2.1",
     "extract-zip": "^2.0.1",
-    "fraim": "2.0.318",
+    "fraim": "2.0.319",
     "mongodb": "^7.0.0",
     "node-cron": "4.2.1",
     "node-edge-tts": "^1.2.10",