fix: refine server adaptor api and fix a loro mem leak

Author: zxch3n

packages/loro-adaptors/src/server/server-loro-adaptor.ts

@@ -28,12 +28,13 @@ export class LoroServerAdaptor implements CrdtServerAdaptor {
     updates?: Uint8Array[];
   } {
     const doc = new LoroDoc();
+    let serverVersion: undefined | VersionVector;
     try {
       if (documentData.length > 0) {
         doc.import(documentData);
       }
 
-      const serverVersion = doc.version();
+      serverVersion = doc.version();
       let updates: Uint8Array[] | undefined;
 
       if (clientVersion.length > 0) {
@@ -68,6 +69,7 @@ export class LoroServerAdaptor implements CrdtServerAdaptor {
       return { response, updates };
     } finally {
       doc.free();
+      serverVersion?.free();
     }
   }
 
@@ -135,20 +137,19 @@ export class LoroServerAdaptor implements CrdtServerAdaptor {
 
   getVersion(documentData: Uint8Array): Uint8Array {
     const doc = new LoroDoc();
+    let version: undefined | VersionVector;
     try {
       if (documentData.length > 0) {
         doc.import(documentData);
       }
-      return doc.version().encode();
+      version = doc.version();
+      return version.encode();
     } finally {
       doc.free();
+      version?.free();
     }
   }
 
-  getSize(documentData: Uint8Array): number {
-    return documentData.length;
-  }
-
   merge(documents: Uint8Array[]): Uint8Array {
     const doc = new LoroDoc();
     try {

packages/loro-adaptors/src/server/server-loro-ephemeral-adaptor.ts

@@ -116,10 +116,6 @@ export class LoroEphemeralServerAdaptor implements CrdtServerAdaptor {
     return new Uint8Array();
   }
 
-  getSize(documentData: Uint8Array): number {
-    return documentData.length;
-  }
-
   merge(documents: Uint8Array[]): Uint8Array {
     const store = new EphemeralStore(this.timeout);
     for (const data of documents) {

packages/loro-adaptors/src/server/server-yjs-awareness-adaptor.ts

@@ -98,10 +98,6 @@ export class YjsAwarenessServerAdaptor implements CrdtServerAdaptor {
     return new Uint8Array();
   }
 
-  getSize(documentData: Uint8Array): number {
-    return documentData.length;
-  }
-
   merge(documents: Uint8Array[]): Uint8Array {
     let total = 0;
     for (const d of documents) total += d.length;

packages/loro-adaptors/src/types.ts

@@ -87,8 +87,6 @@ export interface CrdtServerAdaptor {
 
   getVersion(documentData: Uint8Array): Uint8Array;
 
-  getSize(documentData: Uint8Array): number;
-
   merge(documents: Uint8Array[]): Uint8Array;
 }