@fluidframework/sequence 2.3.0-288113 → 2.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/sequence",
-  "version": "2.3.0-288113",
+  "version": "2.3.1",
   "description": "Distributed sequence",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -77,34 +77,34 @@
     "temp-directory": "nyc/.nyc_output"
   },
   "dependencies": {
-    "@fluid-internal/client-utils": "2.3.0-288113",
-    "@fluidframework/core-interfaces": "2.3.0-288113",
-    "@fluidframework/core-utils": "2.3.0-288113",
-    "@fluidframework/datastore-definitions": "2.3.0-288113",
-    "@fluidframework/driver-definitions": "2.3.0-288113",
-    "@fluidframework/merge-tree": "2.3.0-288113",
-    "@fluidframework/runtime-definitions": "2.3.0-288113",
-    "@fluidframework/runtime-utils": "2.3.0-288113",
-    "@fluidframework/shared-object-base": "2.3.0-288113",
-    "@fluidframework/telemetry-utils": "2.3.0-288113",
+    "@fluid-internal/client-utils": "~2.3.1",
+    "@fluidframework/core-interfaces": "~2.3.1",
+    "@fluidframework/core-utils": "~2.3.1",
+    "@fluidframework/datastore-definitions": "~2.3.1",
+    "@fluidframework/driver-definitions": "~2.3.1",
+    "@fluidframework/merge-tree": "~2.3.1",
+    "@fluidframework/runtime-definitions": "~2.3.1",
+    "@fluidframework/runtime-utils": "~2.3.1",
+    "@fluidframework/shared-object-base": "~2.3.1",
+    "@fluidframework/telemetry-utils": "~2.3.1",
     "double-ended-queue": "^2.1.0-0",
     "uuid": "^9.0.0"
   },
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.15.2",
     "@biomejs/biome": "~1.8.3",
-    "@fluid-internal/mocha-test-setup": "2.3.0-288113",
-    "@fluid-private/stochastic-test-utils": "2.3.0-288113",
-    "@fluid-private/test-dds-utils": "2.3.0-288113",
+    "@fluid-internal/mocha-test-setup": "~2.3.1",
+    "@fluid-private/stochastic-test-utils": "~2.3.1",
+    "@fluid-private/test-dds-utils": "~2.3.1",
     "@fluid-tools/benchmark": "^0.50.0",
-    "@fluid-tools/build-cli": "^0.43.0",
+    "@fluid-tools/build-cli": "^0.46.0",
     "@fluidframework/build-common": "^2.0.3",
-    "@fluidframework/build-tools": "^0.43.0",
-    "@fluidframework/container-definitions": "2.3.0-288113",
-    "@fluidframework/eslint-config-fluid": "^5.3.0",
+    "@fluidframework/build-tools": "^0.46.0",
+    "@fluidframework/container-definitions": "~2.3.1",
+    "@fluidframework/eslint-config-fluid": "^5.4.0",
     "@fluidframework/sequence-previous": "npm:@fluidframework/sequence@2.2.0",
-    "@fluidframework/test-runtime-utils": "2.3.0-288113",
-    "@microsoft/api-extractor": "^7.45.1",
+    "@fluidframework/test-runtime-utils": "~2.3.1",
+    "@microsoft/api-extractor": "7.47.8",
     "@types/diff": "^3.5.1",
     "@types/double-ended-queue": "^2.1.0",
     "@types/mocha": "^9.1.1",
@@ -135,7 +135,8 @@
     }
   },
   "typeValidation": {
-    "broken": {}
+    "broken": {},
+    "entrypoint": "internal"
   },
   "scripts": {
     "api": "fluid-build . --task api",

package/src/intervalCollection.ts

@@ -597,9 +597,7 @@ class IntervalCollectionIterator<TInterval extends ISerializableInterval>
 	public next(): IteratorResult<TInterval> {
 		if (this.index < this.results.length) {
 			return {
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				value: this.results[this.index++]!,
+				value: this.results[this.index++],
 				done: false,
 			};
 		}

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/sequence";
-export const pkgVersion = "2.3.0-288113";
+export const pkgVersion = "2.3.1";

package/src/revertibles.ts

@@ -324,9 +324,7 @@ export function appendSharedStringDeltaToRevertibles(
 				event: IntervalOpType.POSITION_REMOVE,
 				intervals: [],
 				revertibleRefs,
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				mergeTreeRevertible: removeRevertibles[0]!,
+				mergeTreeRevertible: removeRevertibles[0],
 			};
 
 			// add an interval for each startInterval, accounting for any corresponding endIntervals
@@ -337,9 +335,7 @@ export function appendSharedStringDeltaToRevertibles(
 				});
 				let endOffset: number | undefined;
 				if (endIntervalIndex !== -1) {
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					endOffset = endIntervals[endIntervalIndex]!.offset;
+					endOffset = endIntervals[endIntervalIndex].offset;
 					endIntervals.splice(endIntervalIndex, 1);
 				}
 
@@ -589,9 +585,7 @@ interface RangeInfo {
 // eslint-disable-next-line import/no-deprecated
 class SortedRangeSet extends SortedSet<RangeInfo, string> {
 	protected getKey(item: RangeInfo): string {
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-		return item.ranges[0]!.segment.ordinal;
+		return item.ranges[0].segment.ordinal;
 	}
 }
 

package/src/sequence.ts

@@ -941,9 +941,7 @@ export abstract class SharedSegmentSequence<T extends ISegment>
 			// Do GC every once in a while...
 			if (
 				this.messagesSinceMSNChange.length > 20 &&
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				this.messagesSinceMSNChange[20]!.sequenceNumber < message.minimumSequenceNumber
+				this.messagesSinceMSNChange[20].sequenceNumber < message.minimumSequenceNumber
 			) {
 				this.processMinSequenceNumberChanged(message.minimumSequenceNumber);
 			}
@@ -953,9 +951,7 @@ export abstract class SharedSegmentSequence<T extends ISegment>
 	private processMinSequenceNumberChanged(minSeq: number) {
 		let index = 0;
 		for (; index < this.messagesSinceMSNChange.length; index++) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			if (this.messagesSinceMSNChange[index]!.sequenceNumber > minSeq) {
+			if (this.messagesSinceMSNChange[index].sequenceNumber > minSeq) {
 				break;
 			}
 		}

package/src/sequenceDeltaEvent.ts

@@ -67,15 +67,11 @@ export abstract class SequenceEvent<
 		});
 
 		this.pFirst = new Lazy<ISequenceDeltaRange<TOperation>>(
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			() => this.sortedRanges.value.items[0]!,
+			() => this.sortedRanges.value.items[0],
 		);
 
 		this.pLast = new Lazy<ISequenceDeltaRange<TOperation>>(
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			() => this.sortedRanges.value.items[this.sortedRanges.value.size - 1]!,
+			() => this.sortedRanges.value.items[this.sortedRanges.value.size - 1],
 		);
 	}
 

package/tsconfig.json

@@ -6,6 +6,7 @@
 		"rootDir": "./src",
 		"outDir": "./lib",
 		"noImplicitAny": true,
+		"noUncheckedIndexedAccess": false,
 		"exactOptionalPropertyTypes": false,
 	},
 }