@soulcraft/brainy 3.37.7 → 3.37.8

package/dist/storage/adapters/gcsStorage.js

@@ -342,8 +342,14 @@ export class GcsStorage extends BaseStorage {
                 contentType: 'application/json',
                 resumable: false // For small objects, non-resumable is faster
             });
-            // Update cache
-            this.nounCacheManager.set(node.id, node);
+            // CRITICAL FIX (v3.37.8): Only cache nodes with non-empty vectors
+            // This prevents cache pollution from HNSW's lazy-loading nodes (vector: [])
+            if (node.vector && Array.isArray(node.vector) && node.vector.length > 0) {
+                this.nounCacheManager.set(node.id, node);
+            }
+            else {
+                prodLog.warn(`[saveNode] Not caching node ${node.id.substring(0, 8)}... with empty vector (HNSW lazy mode)`);
+            }
             // Increment noun count
             const metadata = await this.getNounMetadata(node.id);
             if (metadata && metadata.type) {
@@ -387,7 +393,7 @@ export class GcsStorage extends BaseStorage {
     async getNode(id) {
         await this.ensureInitialized();
         // Check cache first WITH LOGGING
-        const cached = this.nounCacheManager.get(id);
+        const cached = await this.nounCacheManager.get(id);
         // DIAGNOSTIC LOGGING: Reveal cache poisoning
         prodLog.info(`[getNode] 🔍 Cache check for ${id.substring(0, 8)}...:`, {
             hasCached: cached !== undefined,
@@ -395,11 +401,38 @@ export class GcsStorage extends BaseStorage {
             isObject: cached !== null && typeof cached === 'object',
             type: typeof cached
         });
-        // CRITICAL FIX: Only return cached value if it's valid (not null/undefined)
+        // CRITICAL FIX (v3.37.8): Validate cached object before returning
         if (cached !== undefined && cached !== null) {
-            prodLog.info(`[getNode] ✅ Cache HIT - returning cached node for ${id.substring(0, 8)}...`);
-            this.logger.trace(`Cache hit for noun ${id}`);
-            return cached;
+            // Log cached object structure to diagnose incomplete objects
+            prodLog.info(`[getNode] Cached object structure:`, {
+                hasId: !!cached.id,
+                idMatches: cached.id === id,
+                hasVector: !!cached.vector,
+                vectorLength: cached.vector?.length,
+                hasConnections: !!cached.connections,
+                connectionsType: typeof cached.connections,
+                hasLevel: cached.level !== undefined,
+                level: cached.level,
+                objectKeys: Object.keys(cached || {})
+            });
+            // Validate cached object has required fields (including non-empty vector!)
+            if (!cached.id || !cached.vector || !Array.isArray(cached.vector) || cached.vector.length === 0) {
+                prodLog.error(`[getNode] ❌ INVALID cached object for ${id.substring(0, 8)}...:`, {
+                    reason: !cached.id ? 'missing id' :
+                        !cached.vector ? 'missing vector' :
+                            !Array.isArray(cached.vector) ? 'vector not array' :
+                                cached.vector.length === 0 ? 'vector is empty array' :
+                                    'unknown'
+                });
+                prodLog.error(`[getNode] Removing invalid object from cache and loading from GCS`);
+                this.nounCacheManager.delete(id);
+                // Fall through to load from GCS
+            }
+            else {
+                prodLog.info(`[getNode] ✅ Valid cached object - returning`);
+                this.logger.trace(`Cache hit for noun ${id}`);
+                return cached;
+            }
         }
         else if (cached === null) {
             prodLog.warn(`[getNode] ⚠️ Cache contains NULL for ${id.substring(0, 8)}... - ignoring and loading from GCS`);
@@ -441,13 +474,13 @@ export class GcsStorage extends BaseStorage {
                 level: data.level || 0
                 // NO metadata field - retrieved separately for scalability
             };
-            // CRITICAL FIX: Only cache valid nodes (never cache null)
-            if (node && node.id && node.vector && Array.isArray(node.vector)) {
+            // CRITICAL FIX: Only cache valid nodes with non-empty vectors (never cache null or empty)
+            if (node && node.id && node.vector && Array.isArray(node.vector) && node.vector.length > 0) {
                 this.nounCacheManager.set(id, node);
                 prodLog.info(`[getNode] 💾 Cached node ${id.substring(0, 8)}... successfully`);
             }
             else {
-                prodLog.warn(`[getNode] ⚠️ NOT caching invalid node for ${id.substring(0, 8)}...`);
+                prodLog.warn(`[getNode] ⚠️ NOT caching invalid node for ${id.substring(0, 8)}... (missing id/vector or empty vector)`);
             }
             this.logger.trace(`Successfully retrieved node ${id}`);
             this.releaseBackpressure(true, requestId);

package/dist/storage/adapters/s3CompatibleStorage.js

@@ -827,11 +827,36 @@ export class S3CompatibleStorage extends BaseStorage {
             isObject: cached !== null && typeof cached === 'object',
             type: typeof cached
         });
-        // CRITICAL FIX: Only return cached value if it's valid (not null/undefined)
+        // CRITICAL FIX (v3.37.8): Validate cached object before returning
         if (cached !== undefined && cached !== null) {
-            prodLog.info(`[getNode] ✅ Cache HIT - returning cached node for ${id.substring(0, 8)}...`);
-            this.logger.trace(`Cache hit for node ${id}`);
-            return cached;
+            // Log cached object structure to diagnose incomplete objects
+            prodLog.info(`[getNode] Cached object structure:`, {
+                hasId: !!cached.id,
+                idMatches: cached.id === id,
+                hasVector: !!cached.vector,
+                vectorLength: cached.vector?.length,
+                hasConnections: !!cached.connections,
+                connectionsType: typeof cached.connections,
+                objectKeys: Object.keys(cached || {})
+            });
+            // Validate cached object has required fields (including non-empty vector!)
+            if (!cached.id || !cached.vector || !Array.isArray(cached.vector) || cached.vector.length === 0) {
+                prodLog.error(`[getNode] ❌ INVALID cached object for ${id.substring(0, 8)}...:`, {
+                    reason: !cached.id ? 'missing id' :
+                        !cached.vector ? 'missing vector' :
+                            !Array.isArray(cached.vector) ? 'vector not array' :
+                                cached.vector.length === 0 ? 'vector is empty array' :
+                                    'unknown'
+                });
+                prodLog.error(`[getNode] Removing invalid object from cache and loading from S3`);
+                this.nodeCache.delete(id);
+                // Fall through to load from S3
+            }
+            else {
+                prodLog.info(`[getNode] ✅ Valid cached object - returning`);
+                this.logger.trace(`Cache hit for node ${id}`);
+                return cached;
+            }
         }
         else if (cached === null) {
             prodLog.warn(`[getNode] ⚠️ Cache contains NULL for ${id.substring(0, 8)}... - ignoring and loading from S3`);
@@ -889,13 +914,13 @@ export class S3CompatibleStorage extends BaseStorage {
                 connections,
                 level: parsedNode.level || 0
             };
-            // CRITICAL FIX: Only cache valid nodes (never cache null)
-            if (node && node.id && node.vector && Array.isArray(node.vector)) {
+            // CRITICAL FIX: Only cache valid nodes with non-empty vectors (never cache null or empty)
+            if (node && node.id && node.vector && Array.isArray(node.vector) && node.vector.length > 0) {
                 this.nodeCache.set(id, node);
                 prodLog.info(`[getNode] 💾 Cached node ${id.substring(0, 8)}... successfully`);
             }
             else {
-                prodLog.warn(`[getNode] ⚠️ NOT caching invalid node for ${id.substring(0, 8)}...`);
+                prodLog.warn(`[getNode] ⚠️ NOT caching invalid node for ${id.substring(0, 8)}... (missing id/vector or empty vector)`);
             }
             this.logger.trace(`Successfully retrieved node ${id}`);
             return node;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@soulcraft/brainy",
-  "version": "3.37.7",
+  "version": "3.37.8",
   "description": "Universal Knowledge Protocol™ - World's first Triple Intelligence database unifying vector, graph, and document search in one API. 31 nouns × 40 verbs for infinite expressiveness.",
   "main": "dist/index.js",
   "module": "dist/index.js",