@fluidframework/sequence 0.59.2000 → 0.59.3000-67119

package/src/mapKernel.ts

@@ -54,9 +54,9 @@ interface IMapMessageHandler {
 }
 
 interface IMapMessageLocalMetadata{
-    pendingClearMessageId?: number,
-    pendingMessageId?: number,
-    lastProcessedSeq: number
+    pendingClearMessageId?: number;
+    pendingMessageId?: number;
+    lastProcessedSeq: number;
 }
 
 /**
@@ -510,7 +510,7 @@ export class MapKernel implements IValueTypeCreator {
             const mapLocalMetadata: Partial<IMapMessageLocalMetadata> = localOpMetadata;
             // we don't know how to rebase these operations, so if any other op has come in
             // we will fail.
-            if(this.lastProcessedSeq !== mapLocalMetadata?.lastProcessedSeq) {
+            if (this.lastProcessedSeq !== mapLocalMetadata?.lastProcessedSeq) {
                 throw new Error("SharedInterval does not support reconnect in presence of external changes");
             }
             // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
@@ -777,7 +777,7 @@ export class MapKernel implements IValueTypeCreator {
                     this.eventEmitter.emit("valueChanged", event, local, message, this.eventEmitter);
                 },
                 submit: (op: IMapValueTypeOperation) => {
-                    this.submitMessage(op, {lastProcessedSeq: this.lastProcessedSeq});
+                    this.submitMessage(op, { lastProcessedSeq: this.lastProcessedSeq });
                 },
                 getStashedOpLocalMetadata: (op: IMapValueTypeOperation) => {
                     assert(false, 0x016 /* "apply stashed op not implemented for custom value type ops" */);
@@ -799,7 +799,7 @@ export class MapKernel implements IValueTypeCreator {
      */
     private submitMapClearMessage(op: IMapClearOperation): void {
         const pendingClearMessageId = this.getMapClearMessageLocalMetadata(op);
-        this.submitMessage(op, {pendingClearMessageId, lastProcessedSeq: this.lastProcessedSeq});
+        this.submitMessage(op, { pendingClearMessageId, lastProcessedSeq: this.lastProcessedSeq });
     }
 
     private getMapKeyMessageLocalMetadata(op: IMapKeyOperation): number {
@@ -814,7 +814,7 @@ export class MapKernel implements IValueTypeCreator {
      */
     private submitMapKeyMessage(op: IMapKeyOperation): void {
         const pendingMessageId = this.getMapKeyMessageLocalMetadata(op);
-        this.submitMessage(op, {pendingMessageId, lastProcessedSeq: this.lastProcessedSeq});
+        this.submitMessage(op, { pendingMessageId, lastProcessedSeq: this.lastProcessedSeq });
     }
 
     /**
@@ -839,7 +839,7 @@ export class MapKernel implements IValueTypeCreator {
                 },
             };
             // Send the localOpMetadata as undefined because we don't care about the ack.
-            this.submitMessage(op, {lastProcessedSeq: this.lastProcessedSeq});
+            this.submitMessage(op, { lastProcessedSeq: this.lastProcessedSeq });
 
             const event: IValueChanged = { key, previousValue };
             this.eventEmitter.emit("valueChanged", event, true, null, this.eventEmitter);

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/sequence";
-export const pkgVersion = "0.59.2000";
+export const pkgVersion = "0.59.3000-67119";

package/src/sequence.ts

@@ -175,7 +175,7 @@ export abstract class SharedSegmentSequence<T extends ISegment>
     ) {
         super(id, dataStoreRuntime, attributes);
 
-        this.loadedDeferred.promise.catch((error)=>{
+        this.loadedDeferred.promise.catch((error) => {
             this.logger.sendErrorEvent({ eventName: "SequenceLoadFailed" }, error);
         });
 
@@ -294,6 +294,9 @@ export abstract class SharedSegmentSequence<T extends ISegment>
         return this.client.getRangeExtentsOfPosition(pos);
     }
 
+    /**
+     * @deprecated - use createLocalReferencePosition
+     */
     public createPositionReference(
         segment: T,
         offset: number,
@@ -305,12 +308,27 @@ export abstract class SharedSegmentSequence<T extends ISegment>
         return lref;
     }
 
+    public createLocalReferencePosition(
+        segment: T,
+        offset: number,
+        refType: ReferenceType,
+        properties: PropertySet | undefined): ReferencePosition {
+        return this.client.createLocalReferencePosition(
+            segment,
+            offset,
+            refType,
+            properties);
+    }
+
+    /**
+     * @deprecated - use localReferencePositionToPosition
+     */
     public localRefToPos(localRef: LocalReference) {
-        if (localRef.segment) {
-            return localRef.offset + this.getPosition(localRef.segment);
-        } else {
-            return -1;
-        }
+        return this.client.localReferencePositionToPosition(localRef);
+    }
+
+    public localReferencePositionToPosition(lref: ReferencePosition): number {
+        return this.client.localReferencePositionToPosition(lref);
     }
 
     /**
@@ -355,12 +373,22 @@ export abstract class SharedSegmentSequence<T extends ISegment>
         }
     }
 
+    /**
+     * @deprecated - use createLocalReferencePosition
+     */
     public addLocalReference(lref: LocalReference) {
         return this.client.addLocalReference(lref);
     }
 
+    /**
+     * @deprecated - use removeLocalReferencePosition
+     */
     public removeLocalReference(lref: LocalReference) {
-        return this.client.removeLocalReference(lref);
+        return this.client.removeLocalReferencePosition(lref);
+    }
+
+    public removeLocalReferencePosition(lref: ReferencePosition) {
+        return this.client.removeLocalReferencePosition(lref);
     }
 
     /**
@@ -539,12 +567,12 @@ export abstract class SharedSegmentSequence<T extends ISegment>
                             || m.sequenceNumber <= collabWindow.currentSeq) {
                             throw new Error(`Invalid catchup operations in snapshot: ${
                                 JSON.stringify({
-                                    op:{
+                                    op: {
                                         seq: m.sequenceNumber,
                                         minSeq: m.minimumSequenceNumber,
-                                        refSeq:m.referenceSequenceNumber,
+                                        refSeq: m.referenceSequenceNumber,
                                     },
-                                    collabWindow:{
+                                    collabWindow: {
                                         seq: collabWindow.currentSeq,
                                         minSeq: collabWindow.minSeq,
                                     },
@@ -597,7 +625,10 @@ export abstract class SharedSegmentSequence<T extends ISegment>
         this.loadFinished();
     }
 
-    protected applyStashedOp(content: any) {
+    /**
+     * {@inheritDoc @fluidframework/shared-object-base#SharedObjectCore.applyStashedOp}
+     */
+    protected applyStashedOp(content: any): unknown {
         return this.client.applyStashedOp(content);
     }
 

package/src/sharedIntervalCollection.ts

@@ -160,7 +160,7 @@ export class SharedIntervalCollection<TInterval extends ISerializableInterval =
      */
     protected async loadCore(storage: IChannelStorageService) {
         const blob = await storage.readBlob(snapshotFileName);
-        const header = bufferToString(blob,"utf8");
+        const header = bufferToString(blob, "utf8");
         this.intervalMapKernel.populate(header);
     }
 

package/src/sparsematrix.ts

@@ -349,7 +349,7 @@ export class SparseMatrix extends SharedSegmentSequence<MatrixSegment> {
         const removeColEnd = srcCol + numCols;
         const ops = [];
 
-        for (let r = 0, rowStart = 0; r < this.numRows; r++ , rowStart += maxCols) {
+        for (let r = 0, rowStart = 0; r < this.numRows; r++, rowStart += maxCols) {
             ops.push(this.client.removeRangeLocal(rowStart + removeColStart, rowStart + removeColEnd));
             const insertPos = rowStart + destCol;
             const segment = new PaddingSegment(numCols);