@fluidframework/merge-tree 0.51.0 → 0.52.0

package/src/index.ts

@@ -3,21 +3,20 @@
  * Licensed under the MIT License.
  */
 
-export * from "./mergeTree";
-export * from "./ops";
-export * from "./properties";
-export * from "./snapshotlegacy";
-export * from "./collections";
 export * from "./base";
-export { loadSegments } from "./text";
 export * from "./client";
-export * from "./segmentGroupCollection";
+export * from "./collections";
+export * from "./constants";
+export * from "./localReference";
+export * from "./mergeTree";
 export * from "./mergeTreeDeltaCallback";
-export * from "./opBuilder";
 export * from "./mergeTreeTracking";
+export * from "./opBuilder";
+export * from "./ops";
+export * from "./properties";
+export * from "./segmentGroupCollection";
 export * from "./segmentPropertiesManager";
+export * from "./snapshotlegacy";
 export * from "./sortedSegmentSet";
+export { loadSegments } from "./text";
 export * from "./textSegment";
-export * from "./localReference";
-export * from "./snapshotLoader";
-export * from "./constants";

package/src/localReference.ts

@@ -110,7 +110,6 @@ export class LocalReference implements ReferencePosition {
     }
 
     public getOffset() {
-        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
         if (this.segment?.removedSeq) {
             return 0;
         }
@@ -122,7 +121,7 @@ export class LocalReference implements ReferencePosition {
     }
 }
 
-interface IRefsAtOffest {
+interface IRefsAtOffset {
     before?: LocalReference[];
     at?: LocalReference[];
     after?: LocalReference[];
@@ -139,12 +138,12 @@ export class LocalReferenceCollection {
     }
 
     public hierRefCount: number = 0;
-    private readonly refsByOffset: (IRefsAtOffest | undefined)[];
+    private readonly refsByOffset: (IRefsAtOffset | undefined)[];
     private refCount: number = 0;
 
     constructor(
         private readonly segment: ISegment,
-        initialRefsByfOffset = new Array<IRefsAtOffest | undefined>(segment.cachedLength)) {
+        initialRefsByfOffset = new Array<IRefsAtOffset | undefined>(segment.cachedLength)) {
         // Since javascript arrays are sparse the above won't populate any of the
         // indicies, but it will ensure the length property of the array matches
         // the length of the segment.

package/src/snapshotV1.ts

@@ -73,7 +73,7 @@ export class SnapshotV1 {
         this.segmentLengths = [];
     }
 
-    getSeqLengthSegs(
+    private getSeqLengthSegs(
         allSegments: JsonSegmentSpecs[],
         allLengths: number[],
         approxSequenceLength: number,

package/src/snapshotlegacy.ts

@@ -6,7 +6,7 @@
 /* eslint-disable @typescript-eslint/no-non-null-assertion */
 
 import { ITelemetryLogger } from "@fluidframework/common-definitions";
-import { assert, IsoBuffer } from "@fluidframework/common-utils";
+import { assert } from "@fluidframework/common-utils";
 import {
     IFluidHandle,
     IFluidSerializer,
@@ -25,18 +25,7 @@ import {
     serializeAsMinSupportedVersion,
 } from "./snapshotChunks";
 
-// first three are index entry
-export interface SnapChunk {
-    /**
-     * Offset from beginning of segments.
-     */
-    position: number;
-    lengthBytes: number;
-    sequenceLength: number;
-    buffer?: IsoBuffer;
-}
-
-export interface SnapshotHeader {
+interface SnapshotHeader {
     chunkCount?: number;
     segmentsTotalLength: number;
     indexOffset?: number;
@@ -50,7 +39,7 @@ export interface SnapshotHeader {
 export class SnapshotLegacy {
     public static readonly header = "header";
     public static readonly body = "body";
-    public static readonly catchupOps = "catchupOps";
+    private static readonly catchupOps = "catchupOps";
 
     // Split snapshot into two entries - headers (small) and body (overflow) for faster loading initial content
     // Please note that this number has no direct relationship to anything other than size of raw text (characters).
@@ -60,13 +49,11 @@ export class SnapshotLegacy {
     // for very chunky text, blob size can easily be 4x-8x of that number.
     public static readonly sizeOfFirstChunk: number = 10000;
 
-    header: SnapshotHeader | undefined;
-    seq: number | undefined;
-    buffer: IsoBuffer | undefined;
-    pendingChunk: SnapChunk | undefined;
-    segments: IJSONSegment[] | undefined;
-    segmentLengths: number[] | undefined;
-    logger: ITelemetryLogger;
+    private header: SnapshotHeader | undefined;
+    private seq: number | undefined;
+    private segments: IJSONSegment[] | undefined;
+    private segmentLengths: number[] | undefined;
+    private readonly logger: ITelemetryLogger;
     private readonly chunkSize: number;
 
     constructor(public mergeTree: MergeTree, logger: ITelemetryLogger, public filename?: string,
@@ -75,7 +62,7 @@ export class SnapshotLegacy {
         this.chunkSize = mergeTree?.options?.mergeTreeSnapshotChunkSize ?? SnapshotLegacy.sizeOfFirstChunk;
     }
 
-    getSeqLengthSegs(
+    private getSeqLengthSegs(
         allSegments: IJSONSegment[],
         allLengths: number[],
         approxSequenceLength: number,