@soulcraft/brainy 4.8.5 → 4.8.6

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [4.8.6](https://github.com/soulcraftlabs/brainy/compare/v4.8.5...v4.8.6) (2025-10-28)
+
+- fix: per-sheet column detection in Excel importer (401443a)
+
+
 ### [4.7.4](https://github.com/soulcraftlabs/brainy/compare/v4.7.3...v4.7.4) (2025-10-27)
 
 **CRITICAL SYSTEMIC VFS BUG FIX - Workshop Team Unblocked!**

package/dist/brainy.js

@@ -464,13 +464,6 @@ export class Brainy {
     async convertNounToEntity(noun) {
         // v4.8.0: Storage adapters ALREADY extract standard fields to top-level!
         // Just read from top-level fields of HNSWNounWithMetadata
-        console.log(`[DEBUG convertNounToEntity] Converting noun ${noun.id}:`, {
-            nounMetadataKeys: noun.metadata ? Object.keys(noun.metadata) : [],
-            nounType: noun.type,
-            hasName: !!noun.metadata?.name,
-            hasPath: !!noun.metadata?.path,
-            hasVfsType: !!noun.metadata?.vfsType
-        });
         // v4.8.0: Clean structure with standard fields at top-level
         const entity = {
             id: noun.id,
@@ -487,12 +480,6 @@ export class Brainy {
             // ONLY custom user fields in metadata (v4.8.0: already separated by storage adapter)
             metadata: noun.metadata
         };
-        console.log(`[DEBUG convertNounToEntity] Converted entity metadata:`, {
-            entityMetadataKeys: entity.metadata ? Object.keys(entity.metadata) : [],
-            metadata_name: entity.metadata?.name,
-            metadata_path: entity.metadata?.path,
-            metadata_vfsType: entity.metadata?.vfsType
-        });
         return entity;
     }
     /**

package/dist/importers/SmartExcelImporter.js

@@ -102,8 +102,24 @@ export class SmartExcelImporter {
         if (rows.length === 0) {
             return this.emptyResult(startTime);
         }
-        // Detect column names
-        const columns = this.detectColumns(rows[0], opts);
+        // CRITICAL FIX (v4.8.6): Detect columns per-sheet, not globally
+        // Different sheets may have different column structures (Term vs Name, etc.)
+        // Group rows by sheet and detect columns for each sheet separately
+        const rowsBySheet = new Map();
+        for (const row of rows) {
+            const sheet = row._sheet || 'default';
+            if (!rowsBySheet.has(sheet)) {
+                rowsBySheet.set(sheet, []);
+            }
+            rowsBySheet.get(sheet).push(row);
+        }
+        // Detect columns for each sheet
+        const columnsBySheet = new Map();
+        for (const [sheet, sheetRows] of rowsBySheet) {
+            if (sheetRows.length > 0) {
+                columnsBySheet.set(sheet, this.detectColumns(sheetRows[0], opts));
+            }
+        }
         // Process each row with BATCHED PARALLEL PROCESSING (v3.38.0)
         const extractedRows = [];
         const entityMap = new Map();
@@ -121,6 +137,9 @@ export class SmartExcelImporter {
             // Process chunk in parallel for massive speedup
             const chunkResults = await Promise.all(chunk.map(async (row, chunkIndex) => {
                 const i = chunkStart + chunkIndex;
+                // CRITICAL FIX (v4.8.6): Use sheet-specific column mapping
+                const sheet = row._sheet || 'default';
+                const columns = columnsBySheet.get(sheet) || this.detectColumns(row, opts);
                 // Extract data from row
                 const term = this.getColumnValue(row, columns.term) || `Entity_${i}`;
                 const definition = this.getColumnValue(row, columns.definition) || '';

package/dist/vfs/PathResolver.js

@@ -162,29 +162,16 @@ export class PathResolver {
             from: dirId,
             type: VerbType.Contains
         });
-        console.log(`[DEBUG PathResolver] getChildren(${dirId}): Found ${relations.length} Contains relations`);
         const validChildren = [];
         const childNames = new Set();
         // Fetch all child entities via relationships
         for (const relation of relations) {
             const entity = await this.brain.get(relation.to);
-            console.log(`[DEBUG PathResolver] Retrieved entity ${relation.to}:`, {
-                exists: !!entity,
-                type: entity?.type,
-                hasMetadata: !!entity?.metadata,
-                metadataKeys: entity?.metadata ? Object.keys(entity.metadata) : [],
-                metadata_vfsType: entity?.metadata?.vfsType,
-                metadata_name: entity?.metadata?.name
-            });
             if (entity && entity.metadata?.vfsType && entity.metadata?.name) {
                 validChildren.push(entity);
                 childNames.add(entity.metadata.name);
             }
-            else {
-                console.log(`[DEBUG PathResolver] ❌ FILTERED OUT entity ${relation.to} - missing vfsType or name`);
-            }
         }
-        console.log(`[DEBUG PathResolver] Returning ${validChildren.length} valid children (filtered from ${relations.length})`);
         // Update cache
         this.parentCache.set(dirId, childNames);
         return validChildren;

package/dist/vfs/VirtualFileSystem.js

@@ -683,19 +683,6 @@ export class VirtualFileSystem {
         }
         // Get children
         let children = await this.pathResolver.getChildren(entityId);
-        console.log(`[DEBUG VFS] readdir(${path}): Found ${children.length} children`);
-        // Debug first child
-        if (children.length > 0) {
-            const firstChild = children[0];
-            console.log(`[DEBUG VFS] First child structure:`, {
-                id: firstChild.id,
-                type: firstChild.type,
-                metadataKeys: Object.keys(firstChild.metadata || {}),
-                metadata_name: firstChild.metadata?.name,
-                metadata_path: firstChild.metadata?.path,
-                metadata_vfsType: firstChild.metadata?.vfsType
-            });
-        }
         // Apply filters
         if (options?.filter) {
             children = this.filterDirectoryEntries(children, options.filter);
@@ -715,17 +702,12 @@ export class VirtualFileSystem {
         await this.updateAccessTime(entityId);
         // Return appropriate format
         if (options?.withFileTypes) {
-            const result = children.map(child => ({
+            return children.map(child => ({
                 name: child.metadata.name,
                 path: child.metadata.path,
                 type: child.metadata.vfsType,
                 entityId: child.id
             }));
-            console.log(`[DEBUG VFS] Returning ${result.length} VFSDirent items`);
-            if (result.length > 0) {
-                console.log(`[DEBUG VFS] First VFSDirent:`, result[0]);
-            }
-            return result;
         }
         return children.map(child => child.metadata.name);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@soulcraft/brainy",
-  "version": "4.8.5",
+  "version": "4.8.6",
   "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",