@soulcraft/brainy 5.0.0 → 5.1.0

package/dist/storage/adapters/memoryStorage.js

@@ -65,6 +65,7 @@ export class MemoryStorage extends BaseStorage {
     }
     /**
      * Save a noun to storage (v4.0.0: pure vector only, no metadata)
+     * v5.0.1: COW-aware - uses branch-prefixed paths for fork isolation
      */
     async saveNoun_internal(noun) {
         const isNew = !this.nouns.has(noun.id);
@@ -82,7 +83,12 @@ export class MemoryStorage extends BaseStorage {
         for (const [level, connections] of noun.connections.entries()) {
             nounCopy.connections.set(level, new Set(connections));
         }
-        // Save the noun directly in the nouns map
+        // v5.0.1: COW-aware write using branch-prefixed path
+        // Use synthetic path for vector storage (nouns don't have types in standalone mode)
+        const path = `hnsw/nouns/${noun.id}.json`;
+        await this.writeObjectToBranch(path, nounCopy);
+        // ALSO store in nouns Map for fast iteration (getNouns, initializeCounts)
+        // This is redundant but maintains backward compatibility
         this.nouns.set(noun.id, nounCopy);
         // Count tracking happens in baseStorage.saveNounMetadata_internal (v4.1.2)
         // This fixes the race condition where metadata didn't exist yet
@@ -90,10 +96,12 @@ export class MemoryStorage extends BaseStorage {
     /**
      * Get a noun from storage (v4.0.0: returns pure vector only)
      * Base class handles combining with metadata
+     * v5.0.1: COW-aware - reads from branch-prefixed paths with inheritance
      */
     async getNoun_internal(id) {
-        // Get the noun directly from the nouns map
-        const noun = this.nouns.get(id);
+        // v5.0.1: COW-aware read using branch-prefixed path with inheritance
+        const path = `hnsw/nouns/${id}.json`;
+        const noun = await this.readWithInheritance(path);
         // If not found, return null
         if (!noun) {
             return null;
@@ -107,9 +115,10 @@ export class MemoryStorage extends BaseStorage {
             level: noun.level || 0
             // ✅ NO metadata field in v4.0.0
         };
-        // Copy connections
-        for (const [level, connections] of noun.connections.entries()) {
-            nounCopy.connections.set(level, new Set(connections));
+        // Copy connections (handle both Map and plain object from JSON)
+        const connections = noun.connections instanceof Map ? noun.connections : new Map(Object.entries(noun.connections || {}));
+        for (const [level, conns] of connections.entries()) {
+            nounCopy.connections.set(Number(level), new Set(conns));
         }
         return nounCopy;
     }
@@ -252,6 +261,7 @@ export class MemoryStorage extends BaseStorage {
     }
     /**
      * Delete a noun from storage (v4.0.0)
+     * v5.0.1: COW-aware - deletes from branch-prefixed paths
      */
     async deleteNoun_internal(id) {
         // v4.0.0: Get type from separate metadata storage
@@ -260,10 +270,15 @@ export class MemoryStorage extends BaseStorage {
             const type = metadata.noun || 'default';
             this.decrementEntityCount(type);
         }
+        // v5.0.1: COW-aware delete using branch-prefixed path
+        const path = `hnsw/nouns/${id}.json`;
+        await this.deleteObjectFromBranch(path);
+        // Also remove from nouns Map for fast iteration
         this.nouns.delete(id);
     }
     /**
      * Save a verb to storage (v4.0.0: pure vector + core fields, no metadata)
+     * v5.0.1: COW-aware - uses branch-prefixed paths for fork isolation
      */
     async saveVerb_internal(verb) {
         const isNew = !this.verbs.has(verb.id);
@@ -283,17 +298,22 @@ export class MemoryStorage extends BaseStorage {
         for (const [level, connections] of verb.connections.entries()) {
             verbCopy.connections.set(level, new Set(connections));
         }
-        // Save the verb directly in the verbs map
+        // v5.0.1: COW-aware write using branch-prefixed path
+        const path = `hnsw/verbs/${verb.id}.json`;
+        await this.writeObjectToBranch(path, verbCopy);
+        // ALSO store in verbs Map for fast iteration (getVerbs, initializeCounts)
         this.verbs.set(verb.id, verbCopy);
         // Note: Count tracking happens in saveVerbMetadata since metadata is separate
     }
     /**
      * Get a verb from storage (v4.0.0: returns pure vector + core fields)
      * Base class handles combining with metadata
+     * v5.0.1: COW-aware - reads from branch-prefixed paths with inheritance
      */
     async getVerb_internal(id) {
-        // Get the verb directly from the verbs map
-        const verb = this.verbs.get(id);
+        // v5.0.1: COW-aware read using branch-prefixed path with inheritance
+        const path = `hnsw/verbs/${id}.json`;
+        const verb = await this.readWithInheritance(path);
         // If not found, return null
         if (!verb) {
             return null;
@@ -310,9 +330,10 @@ export class MemoryStorage extends BaseStorage {
             targetId: verb.targetId
             // ✅ NO metadata field in v4.0.0
         };
-        // Copy connections
-        for (const [level, connections] of verb.connections.entries()) {
-            verbCopy.connections.set(level, new Set(connections));
+        // Copy connections (handle both Map and plain object from JSON)
+        const connections = verb.connections instanceof Map ? verb.connections : new Map(Object.entries(verb.connections || {}));
+        for (const [level, conns] of connections.entries()) {
+            verbCopy.connections.set(Number(level), new Set(conns));
         }
         return verbCopy;
     }
@@ -474,10 +495,9 @@ export class MemoryStorage extends BaseStorage {
     }
     /**
      * Delete a verb from storage
+     * v5.0.1: COW-aware - deletes from branch-prefixed paths
      */
     async deleteVerb_internal(id) {
-        // Delete the HNSWVerb from the verbs map
-        this.verbs.delete(id);
         // CRITICAL: Also delete verb metadata - this is what getVerbs() uses to find verbs
         // Without this, getVerbsBySource() will still find "deleted" verbs via their metadata
         const metadata = await this.getVerbMetadata(id);
@@ -487,6 +507,11 @@ export class MemoryStorage extends BaseStorage {
             // Delete the metadata using the base storage method
             await this.deleteVerbMetadata(id);
         }
+        // v5.0.1: COW-aware delete using branch-prefixed path
+        const path = `hnsw/verbs/${id}.json`;
+        await this.deleteObjectFromBranch(path);
+        // Also remove from verbs Map for fast iteration
+        this.verbs.delete(id);
     }
     /**
      * Primitive operation: Write object to path

package/dist/storage/adapters/typeAwareStorageAdapter.d.ts

@@ -36,7 +36,7 @@
  * @since Phase 1 - Type-First Implementation
  */
 import { BaseStorage } from '../baseStorage.js';
-import { HNSWNoun, HNSWVerb, HNSWVerbWithMetadata, NounMetadata, VerbMetadata, StatisticsData } from '../../coreTypes.js';
+import { HNSWNoun, HNSWVerb, HNSWNounWithMetadata, HNSWVerbWithMetadata, NounMetadata, VerbMetadata, StatisticsData } from '../../coreTypes.js';
 import { NounType, VerbType } from '../../types/graphTypes.js';
 /**
  * Options for TypeAwareStorageAdapter
@@ -238,6 +238,36 @@ export declare class TypeAwareStorageAdapter extends BaseStorage {
         level: number;
         connections: Record<string, string[]>;
     } | null>;
+    /**
+     * Get nouns with pagination (v5.0.1: COW-aware)
+     * Required for find() to work with TypeAwareStorage
+     */
+    getNounsWithPagination(options: {
+        limit?: number;
+        offset?: number;
+        cursor?: string;
+        filter?: any;
+    }): Promise<{
+        items: HNSWNounWithMetadata[];
+        totalCount: number;
+        hasMore: boolean;
+        nextCursor?: string;
+    }>;
+    /**
+     * Get verbs with pagination (v5.0.1: COW-aware)
+     * Required for GraphAdjacencyIndex rebuild and find() to work
+     */
+    getVerbsWithPagination(options: {
+        limit?: number;
+        offset?: number;
+        cursor?: string;
+        filter?: any;
+    }): Promise<{
+        items: HNSWVerbWithMetadata[];
+        totalCount: number;
+        hasMore: boolean;
+        nextCursor?: string;
+    }>;
     /**
      * Save HNSW system data (entry point, max level)
      */