@soulcraft/brainy 0.54.2 → 0.54.4

package/dist/brainyData.js

@@ -788,8 +788,9 @@ export class BrainyData {
             catch (e) {
                 // Ignore errors loading existing statistics
             }
-            // Initialize metadata index if not in write-only mode
-            if (!this.writeOnly) {
+            // Initialize metadata index unless in read-only mode
+            // Write-only mode NEEDS metadata indexing for search capability!
+            if (!this.readOnly) {
                 this.metadataIndex = new MetadataIndexManager(this.storage, this.config.metadataIndex);
                 // Check if we need to rebuild the index (for existing data)
                 // Skip rebuild for memory storage (starts empty) or when in read-only mode
@@ -802,20 +803,30 @@ export class BrainyData {
                     try {
                         const testResult = await this.storage.getNouns({ pagination: { offset: 0, limit: 1 } });
                         if (testResult.items.length > 0) {
-                            if (this.loggingConfig?.verbose) {
-                                console.log('Rebuilding metadata index for existing data...');
+                            // Only rebuild metadata index if explicitly requested or if we have very few items
+                            const shouldRebuild = process.env.BRAINY_REBUILD_INDEX === 'true';
+                            if (shouldRebuild) {
+                                if (this.loggingConfig?.verbose) {
+                                    console.log('🔄 Rebuilding metadata index for existing data...');
+                                }
+                                await this.metadataIndex.rebuild();
+                                if (this.loggingConfig?.verbose) {
+                                    const newStats = await this.metadataIndex.getStats();
+                                    console.log(`✅ Metadata index rebuilt: ${newStats.totalEntries} entries, ${newStats.fieldsIndexed.length} fields`);
+                                }
                             }
-                            await this.metadataIndex.rebuild();
-                            if (this.loggingConfig?.verbose) {
-                                const newStats = await this.metadataIndex.getStats();
-                                console.log(`Metadata index rebuilt: ${newStats.totalEntries} entries, ${newStats.fieldsIndexed.length} fields`);
+                            else {
+                                if (this.loggingConfig?.verbose) {
+                                    console.log('⏭️  Skipping metadata index rebuild (set BRAINY_REBUILD_INDEX=true to force)');
+                                }
+                                // Build index incrementally as items are accessed instead
                             }
                         }
                     }
                     catch (error) {
                         // If getNouns fails, skip rebuild
                         if (this.loggingConfig?.verbose) {
-                            console.log('Skipping metadata index rebuild:', error);
+                            console.log('⚠️  Skipping metadata index rebuild due to error:', error);
                         }
                     }
                 }
@@ -1186,8 +1197,8 @@ export class BrainyData {
                         }
                     }
                     await this.storage.saveMetadata(id, metadataToSave);
-                    // Update metadata index
-                    if (this.metadataIndex && !this.readOnly && !this.frozen) {
+                    // Update metadata index (write-only mode should build indices!)
+                    if (this.metadataIndex && !this.frozen) {
                         await this.metadataIndex.addToIndex(id, metadataToSave);
                     }
                     // Track metadata statistics
@@ -2288,8 +2299,8 @@ export class BrainyData {
             try {
                 // Get metadata before removing for index cleanup
                 const existingMetadata = await this.storage.getMetadata(actualId);
-                // Remove from metadata index
-                if (this.metadataIndex && existingMetadata && !this.readOnly && !this.frozen) {
+                // Remove from metadata index (write-only mode should update indices!)
+                if (this.metadataIndex && existingMetadata && !this.frozen) {
                     await this.metadataIndex.removeFromIndex(actualId, existingMetadata);
                 }
                 await this.storage.saveMetadata(actualId, null);
@@ -2377,8 +2388,8 @@ export class BrainyData {
             }
             // Update metadata
             await this.storage.saveMetadata(id, metadata);
-            // Update metadata index
-            if (this.metadataIndex && !this.readOnly && !this.frozen) {
+            // Update metadata index (write-only mode should build indices!)
+            if (this.metadataIndex && !this.frozen) {
                 // Remove old metadata from index if it exists
                 const oldMetadata = await this.storage.getMetadata(id);
                 if (oldMetadata) {