Merge pull request #197 from rimbu-org/fix/fix-table-builder-mofidyat

fix(table): modifyAt did not correctly remove reference to immutable instance on change

deno_dist/table/custom/implementation/base.ts

@@ -775,6 +775,10 @@ export class TableBuilder<
       },
     });
 
+    if (changed) {
+      this.source = undefined;
+    }
+
     return changed;
   };
 

packages/table/src/custom/implementation/base.mts

@@ -775,6 +775,10 @@ export class TableBuilder<
       },
     });
 
+    if (changed) {
+      this.source = undefined;
+    }
+
     return changed;
   };
 

packages/table/test-utils/table-standard-test.mts

@@ -701,6 +701,7 @@ export function runTableTestsWith(
           expect(b.modifyAt(1, 'a', { ifExists: () => true })).toBe(false);
           expect(b.modifyAt(1, 'a', { ifExists: () => false })).toBe(true);
           expect(b.modifyAt(1, 'b', { ifNew: true })).toBe(true);
+          expect(b.get(1, 'b')).toBe(true);
           expect(b.modifyAt(1, 'b', { ifExists: () => true })).toBe(false);
           expect(b.modifyAt(1, 'b', { ifExists: () => false })).toBe(true);
           expect(b.modifyAt(10, 'a', { ifNew: (none) => none })).toBe(false);