@fluidframework/sequence 2.100.0 → 2.101.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/sequence",
-  "version": "2.100.0",
+  "version": "2.101.0",
   "description": "Distributed sequence",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -91,33 +91,33 @@
     "temp-directory": "nyc/.nyc_output"
   },
   "dependencies": {
-    "@fluid-internal/client-utils": "~2.100.0",
-    "@fluidframework/core-interfaces": "~2.100.0",
-    "@fluidframework/core-utils": "~2.100.0",
-    "@fluidframework/datastore-definitions": "~2.100.0",
-    "@fluidframework/driver-definitions": "~2.100.0",
-    "@fluidframework/merge-tree": "~2.100.0",
-    "@fluidframework/runtime-definitions": "~2.100.0",
-    "@fluidframework/runtime-utils": "~2.100.0",
-    "@fluidframework/shared-object-base": "~2.100.0",
-    "@fluidframework/telemetry-utils": "~2.100.0",
+    "@fluid-internal/client-utils": "~2.101.0",
+    "@fluidframework/core-interfaces": "~2.101.0",
+    "@fluidframework/core-utils": "~2.101.0",
+    "@fluidframework/datastore-definitions": "~2.101.0",
+    "@fluidframework/driver-definitions": "~2.101.0",
+    "@fluidframework/merge-tree": "~2.101.0",
+    "@fluidframework/runtime-definitions": "~2.101.0",
+    "@fluidframework/runtime-utils": "~2.101.0",
+    "@fluidframework/shared-object-base": "~2.101.0",
+    "@fluidframework/telemetry-utils": "~2.101.0",
     "double-ended-queue": "^2.1.0-0",
     "uuid": "^11.1.0"
   },
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.18.2",
     "@biomejs/biome": "~2.4.5",
-    "@fluid-internal/mocha-test-setup": "~2.100.0",
-    "@fluid-private/stochastic-test-utils": "~2.100.0",
-    "@fluid-private/test-dds-utils": "~2.100.0",
-    "@fluid-tools/benchmark": "^0.58.0",
+    "@fluid-internal/mocha-test-setup": "~2.101.0",
+    "@fluid-private/stochastic-test-utils": "~2.101.0",
+    "@fluid-private/test-dds-utils": "~2.101.0",
+    "@fluid-tools/benchmark": "^0.59.0",
     "@fluid-tools/build-cli": "^0.65.0",
     "@fluidframework/build-common": "^2.0.3",
     "@fluidframework/build-tools": "^0.65.0",
-    "@fluidframework/container-definitions": "~2.100.0",
+    "@fluidframework/container-definitions": "~2.101.0",
     "@fluidframework/eslint-config-fluid": "^9.0.0",
     "@fluidframework/sequence-previous": "npm:@fluidframework/sequence@2.92.0",
-    "@fluidframework/test-runtime-utils": "~2.100.0",
+    "@fluidframework/test-runtime-utils": "~2.101.0",
     "@microsoft/api-extractor": "7.58.1",
     "@types/diff": "^3.5.1",
     "@types/double-ended-queue": "^2.1.0",

package/src/intervalCollection.ts

@@ -27,6 +27,7 @@ import {
 	refTypeIncludesFlag,
 	reservedRangeLabelsKey,
 	Side,
+	defaultSide,
 	endpointPosAndSide,
 	type ISegmentInternal,
 	createLocalReconnectingPerspective,
@@ -945,21 +946,30 @@ export class IntervalCollection
 	public applyStashedOp(op: IIntervalCollectionTypeOperationValue): void {
 		const { opName, value } = op;
 		const { id, properties } = getSerializedProperties(value);
+		// Serialized ops always include startSide/endSide for future-proofing,
+		// but the original add()/change() call may have used plain numeric
+		// positions. Drop the side when it equals the default so the replayed
+		// call matches the original shape — otherwise assertStickinessEnabled
+		// would treat a plain-number interval as sticky. Stashed-op values
+		// can be deep-frozen by upstream layers (e.g. test mocks), so this is
+		// computed into locals rather than mutating value.
+		const startSide = value.startSide === defaultSide ? undefined : value.startSide;
+		const endSide = value.endSide === defaultSide ? undefined : value.endSide;
 		switch (opName) {
 			case "add": {
 				this.add({
 					id,
 					// Todo: we should improve typing so we know add ops always have start and end
-					start: toSequencePlace(value.start, value.startSide),
-					end: toSequencePlace(value.end, value.endSide),
+					start: toSequencePlace(value.start, startSide),
+					end: toSequencePlace(value.end, endSide),
 					props: properties,
 				});
 				break;
 			}
 			case "change": {
 				this.change(id, {
-					start: toOptionalSequencePlace(value.start, value.startSide),
-					end: toOptionalSequencePlace(value.end, value.endSide),
+					start: toOptionalSequencePlace(value.start, startSide),
+					end: toOptionalSequencePlace(value.end, endSide),
 					props: properties,
 				});
 				break;
@@ -1427,8 +1437,8 @@ export class IntervalCollection
 				const { start, end, startSide, endSide } = serializedInterval;
 				newInterval = intervalToChange.modify(
 					"",
-					toOptionalSequencePlace(start, startSide ?? Side.Before),
-					toOptionalSequencePlace(end, endSide ?? Side.Before),
+					toOptionalSequencePlace(start, startSide ?? defaultSide),
+					toOptionalSequencePlace(end, endSide ?? defaultSide),
 					op,
 				);
 				if (isLatestInterval) {
@@ -1671,8 +1681,8 @@ export class IntervalCollection
 
 		const interval: SequenceIntervalClass = this.localCollection.addInterval(
 			id,
-			toSequencePlace(serializedInterval.start, serializedInterval.startSide ?? Side.Before),
-			toSequencePlace(serializedInterval.end, serializedInterval.endSide ?? Side.Before),
+			toSequencePlace(serializedInterval.start, serializedInterval.startSide ?? defaultSide),
+			toSequencePlace(serializedInterval.end, serializedInterval.endSide ?? defaultSide),
 			properties,
 			op,
 		);

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/sequence";
-export const pkgVersion = "2.100.0";
+export const pkgVersion = "2.101.0";