@fluidframework/container-loader 2.1.0 → 2.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/container-loader",
-  "version": "2.1.0",
+  "version": "2.2.0",
   "description": "Fluid container loader",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -117,13 +117,13 @@
     "temp-directory": "nyc/.nyc_output"
   },
   "dependencies": {
-    "@fluid-internal/client-utils": "~2.1.0",
-    "@fluidframework/container-definitions": "~2.1.0",
-    "@fluidframework/core-interfaces": "~2.1.0",
-    "@fluidframework/core-utils": "~2.1.0",
-    "@fluidframework/driver-definitions": "~2.1.0",
-    "@fluidframework/driver-utils": "~2.1.0",
-    "@fluidframework/telemetry-utils": "~2.1.0",
+    "@fluid-internal/client-utils": "~2.2.0",
+    "@fluidframework/container-definitions": "~2.2.0",
+    "@fluidframework/core-interfaces": "~2.2.0",
+    "@fluidframework/core-utils": "~2.2.0",
+    "@fluidframework/driver-definitions": "~2.2.0",
+    "@fluidframework/driver-utils": "~2.2.0",
+    "@fluidframework/telemetry-utils": "~2.2.0",
     "@types/events_pkg": "npm:@types/events@^3.0.0",
     "@ungap/structured-clone": "^1.2.0",
     "debug": "^4.3.4",
@@ -134,13 +134,13 @@
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.15.2",
     "@biomejs/biome": "~1.8.3",
-    "@fluid-internal/client-utils": "~2.1.0",
-    "@fluid-internal/mocha-test-setup": "~2.1.0",
-    "@fluid-private/test-loader-utils": "~2.1.0",
-    "@fluid-tools/build-cli": "^0.41.0",
+    "@fluid-internal/client-utils": "~2.2.0",
+    "@fluid-internal/mocha-test-setup": "~2.2.0",
+    "@fluid-private/test-loader-utils": "~2.2.0",
+    "@fluid-tools/build-cli": "^0.43.0",
     "@fluidframework/build-common": "^2.0.3",
-    "@fluidframework/build-tools": "^0.41.0",
-    "@fluidframework/container-loader-previous": "npm:@fluidframework/container-loader@2.0.0",
+    "@fluidframework/build-tools": "^0.43.0",
+    "@fluidframework/container-loader-previous": "npm:@fluidframework/container-loader@2.1.0",
     "@fluidframework/eslint-config-fluid": "^5.3.0",
     "@microsoft/api-extractor": "^7.45.1",
     "@types/debug": "^4.1.5",

package/src/container.ts

@@ -221,6 +221,7 @@ export interface IContainerCreateProps {
 	 * A property bag of options used by various layers
 	 * to control features
 	 */
+	// eslint-disable-next-line import/no-deprecated
 	readonly options: ILoaderOptions;
 
 	/**
@@ -482,6 +483,7 @@ export class Container
 	private readonly urlResolver: IUrlResolver;
 	private readonly serviceFactory: IDocumentServiceFactory;
 	private readonly codeLoader: ICodeDetailsLoader;
+	// eslint-disable-next-line import/no-deprecated
 	private readonly options: ILoaderOptions;
 	private readonly scope: FluidObject;
 	private readonly subLogger: ITelemetryLoggerExt;
@@ -2133,14 +2135,14 @@ export class Container
 		lastProcessedSequenceNumber?: number,
 	): Promise<void> {
 		return this._deltaManager.attachOpHandler(
-			attributes.minimumSequenceNumber,
-			attributes.sequenceNumber,
+			attributes.minimumSequenceNumber /* minimumSequenceNumber */,
+			attributes.sequenceNumber /* snapshotSequenceNumber */,
 			{
 				process: (message) => this.processRemoteMessage(message),
 				processSignal: (message) => {
 					this.processSignal(message);
 				},
-			},
+			} /* handler to process incoming delta messages */,
 			prefetchType,
 			lastProcessedSequenceNumber,
 		);
@@ -2333,6 +2335,10 @@ export class Container
 		);
 	}
 
+	/**
+	 * Processes incoming delta messages
+	 * @param message - delta message received from the server
+	 */
 	private processRemoteMessage(message: ISequencedDocumentMessage): void {
 		const local = this.clientId === message.clientId;
 

package/src/deltaManager.ts

@@ -1076,11 +1076,20 @@ export class DeltaManager<TConnectionManager extends IConnectionManager>
 
 		// Watch the minimum sequence number and be ready to update as needed
 		if (this.minSequenceNumber > message.minimumSequenceNumber) {
-			// pre-0.58 error message: msnMovesBackwards
-			throw new DataCorruptionError(
-				"Found a lower minimumSequenceNumber (msn) than previously recorded",
+			// This indicates that an invalid series of ops was received by this client.
+			// In the unlikely case where these ops have been truly sequenced and persisted to storage,
+			// this document is corrupted - It will fail here on boot every time.
+			// The more likely scenario, based on the realities of production service operation, is that
+			// something has changed out from under the file on the server, such that the service lost some ops
+			// which this client already processed - the very ops that made this _next_ op to appear invalid.
+			// In this case, only this client will fail (and lose this recent data), but others will be able to connect and continue.
+			throw DataProcessingError.create(
+				// error message through v0.57: msnMovesBackwards
+				// error message through v2.1: "Found a lower minimumSequenceNumber (msn) than previously recorded",
+				"Invalid MinimumSequenceNumber from service - document may have been restored to previous state",
+				"DeltaManager.processInboundMessage",
+				message,
 				{
-					...extractSafePropertiesFromMessage(message),
 					clientId: this.connectionManager.clientId,
 				},
 			);

package/src/loader.ts

@@ -93,8 +93,15 @@ export class RelativeLoader implements ILoader {
 /**
  * @legacy
  * @alpha
+ * @deprecated Use {@link @fluidframework/container-definitions#ILoaderOptions} instead
  */
 export interface ILoaderOptions extends ILoaderOptions1 {
+	/**
+	 *
+	 * @deprecated No longer needed or used (initially introduced to test single-commit summaries).
+	 * Driver layer can enable single-commit summaries via document service policies if needed.
+	 * ADO #9098: To remove declaration and usage from code.
+	 */
 	summarizeProtocolTree?: boolean;
 }
 

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/container-loader";
-export const pkgVersion = "2.1.0";
+export const pkgVersion = "2.2.0";