TypeCache returns unexpected data after multiple insertions and deletions (#121)

* feat: get rid of type cache entirely

* tests: ensure multiple inserts and removals do not throw

---------

Co-authored-by: Yury Tsai <y.tsai@smartcat.ai>

Author: Dewarrum

Tests/YDotNet.Tests.Unit/XmlElements/InsertElementTests.cs

@@ -1,5 +1,6 @@
 using NUnit.Framework;
 using YDotNet.Document;
+using YDotNet.Document.Types.XmlElements;
 
 namespace YDotNet.Tests.Unit.XmlElements;
 
@@ -55,4 +56,41 @@ public void InsertMultipleElements()
 
         transaction.Commit();
     }
+
+    [Test]
+    public void InsertAndRemoveMultipleTimes()
+    {
+        // Arrange
+        var doc = new Doc();
+        var xmlFragment = doc.XmlFragment("xml-fragment");
+        XmlElement xmlElement;
+        using (var transaction = doc.WriteTransaction())
+        {
+            xmlElement = xmlFragment.InsertElement(transaction, index: 0, "xml-element");
+        }
+
+        // Act
+        for (var i = 0; i < 100; i++)
+        {
+            // Assert
+            Assert.DoesNotThrow(() =>
+            {
+                using (var transaction = doc.WriteTransaction())
+                {
+                    for (uint j = 0; j < 100; j++)
+                    {
+                        xmlElement
+                            .InsertElement(transaction, index: j, "paragraph")
+                            .InsertText(transaction, index: 0)
+                            .Insert(transaction, index: 0, "Hello world");
+                    }
+                }
+
+                using (var transaction = doc.WriteTransaction())
+                {
+                    xmlElement.RemoveRange(transaction, index: 0, length: 100);
+                }
+            });
+        }
+    }
 }

Tests/YDotNet.Tests.Unit/XmlFragments/InsertElementTests.cs

@@ -47,4 +47,36 @@ public void InsertMultipleElements()
 
         transaction.Commit();
     }
+
+    [Test]
+    public void InsertAndRemoveMultipleTimes()
+    {
+        // Arrange
+        var doc = new Doc();
+        var xmlFragment = doc.XmlFragment("xml-fragment");
+
+        // Act
+        for (var i = 0; i < 100; i++)
+        {
+            // Assert
+            Assert.DoesNotThrow(() =>
+            {
+                using (var transaction = doc.WriteTransaction())
+                {
+                    for (uint j = 0; j < 100; j++)
+                    {
+                        xmlFragment
+                            .InsertElement(transaction, index: j, "paragraph")
+                            .InsertText(transaction, index: 0)
+                            .Insert(transaction, index: 0, "Hello world");
+                    }
+                }
+
+                using (var transaction = doc.WriteTransaction())
+                {
+                    xmlFragment.RemoveRange(transaction, index: 0, length: 100);
+                }
+            });
+        }
+    }
 }

YDotNet/Document/Doc.cs

@@ -39,7 +39,6 @@ public class Doc : UnmanagedResource
     private readonly EventSubscriber<UpdateEvent> onUpdateV1;
     private readonly EventSubscriber<UpdateEvent> onUpdateV2;
     private readonly Doc? parent;
-    private readonly TypeCache typeCache = new();
     private int openTransactions;
 
     /// <summary>
@@ -542,7 +541,7 @@ private T GetOrAdd<T>(nint handle, Func<nint, Doc, T> factory)
     {
         var doc = GetRootDoc();
 
-        return doc.typeCache.GetOrAdd(handle, h => factory(h, doc));
+        return factory(handle, doc);
     }
 
     private void ThrowIfOpenTransaction()