@fluidframework/container-runtime 2.10.0-307060 → 2.10.0-307399

package/lib/packageVersion.d.ts

@@ -5,5 +5,5 @@
  * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY
  */
 export declare const pkgName = "@fluidframework/container-runtime";
-export declare const pkgVersion = "2.10.0-307060";
+export declare const pkgVersion = "2.10.0-307399";
 //# sourceMappingURL=packageVersion.d.ts.map

package/lib/packageVersion.js

@@ -5,5 +5,5 @@
  * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY
  */
 export const pkgName = "@fluidframework/container-runtime";
-export const pkgVersion = "2.10.0-307060";
+export const pkgVersion = "2.10.0-307399";
 //# sourceMappingURL=packageVersion.js.map

package/lib/packageVersion.js.map

@@ -1 +1 @@
-{"version":3,"file":"packageVersion.js","sourceRoot":"","sources":["../src/packageVersion.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,CAAC,MAAM,OAAO,GAAG,mCAAmC,CAAC;AAC3D,MAAM,CAAC,MAAM,UAAU,GAAG,eAAe,CAAC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n *\n * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY\n */\n\nexport const pkgName = \"@fluidframework/container-runtime\";\nexport const pkgVersion = \"2.10.0-307060\";\n"]}
+{"version":3,"file":"packageVersion.js","sourceRoot":"","sources":["../src/packageVersion.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,CAAC,MAAM,OAAO,GAAG,mCAAmC,CAAC;AAC3D,MAAM,CAAC,MAAM,UAAU,GAAG,eAAe,CAAC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n *\n * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY\n */\n\nexport const pkgName = \"@fluidframework/container-runtime\";\nexport const pkgVersion = \"2.10.0-307399\";\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/container-runtime",
-  "version": "2.10.0-307060",
+  "version": "2.10.0-307399",
   "description": "Fluid container runtime",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -129,18 +129,18 @@
     "temp-directory": "nyc/.nyc_output"
   },
   "dependencies": {
-    "@fluid-internal/client-utils": "2.10.0-307060",
-    "@fluidframework/container-definitions": "2.10.0-307060",
-    "@fluidframework/container-runtime-definitions": "2.10.0-307060",
-    "@fluidframework/core-interfaces": "2.10.0-307060",
-    "@fluidframework/core-utils": "2.10.0-307060",
-    "@fluidframework/datastore": "2.10.0-307060",
-    "@fluidframework/driver-definitions": "2.10.0-307060",
-    "@fluidframework/driver-utils": "2.10.0-307060",
-    "@fluidframework/id-compressor": "2.10.0-307060",
-    "@fluidframework/runtime-definitions": "2.10.0-307060",
-    "@fluidframework/runtime-utils": "2.10.0-307060",
-    "@fluidframework/telemetry-utils": "2.10.0-307060",
+    "@fluid-internal/client-utils": "2.10.0-307399",
+    "@fluidframework/container-definitions": "2.10.0-307399",
+    "@fluidframework/container-runtime-definitions": "2.10.0-307399",
+    "@fluidframework/core-interfaces": "2.10.0-307399",
+    "@fluidframework/core-utils": "2.10.0-307399",
+    "@fluidframework/datastore": "2.10.0-307399",
+    "@fluidframework/driver-definitions": "2.10.0-307399",
+    "@fluidframework/driver-utils": "2.10.0-307399",
+    "@fluidframework/id-compressor": "2.10.0-307399",
+    "@fluidframework/runtime-definitions": "2.10.0-307399",
+    "@fluidframework/runtime-utils": "2.10.0-307399",
+    "@fluidframework/telemetry-utils": "2.10.0-307399",
     "@tylerbu/sorted-btree-es6": "^1.8.0",
     "double-ended-queue": "^2.1.0-0",
     "lz4js": "^0.2.0",
@@ -149,16 +149,16 @@
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.16.4",
     "@biomejs/biome": "~1.9.3",
-    "@fluid-internal/mocha-test-setup": "2.10.0-307060",
-    "@fluid-private/stochastic-test-utils": "2.10.0-307060",
-    "@fluid-private/test-pairwise-generator": "2.10.0-307060",
+    "@fluid-internal/mocha-test-setup": "2.10.0-307399",
+    "@fluid-private/stochastic-test-utils": "2.10.0-307399",
+    "@fluid-private/test-pairwise-generator": "2.10.0-307399",
     "@fluid-tools/benchmark": "^0.50.0",
-    "@fluid-tools/build-cli": "^0.50.0",
+    "@fluid-tools/build-cli": "^0.51.0",
     "@fluidframework/build-common": "^2.0.3",
-    "@fluidframework/build-tools": "^0.50.0",
+    "@fluidframework/build-tools": "^0.51.0",
     "@fluidframework/container-runtime-previous": "npm:@fluidframework/container-runtime@~2.5.0",
     "@fluidframework/eslint-config-fluid": "^5.4.0",
-    "@fluidframework/test-runtime-utils": "2.10.0-307060",
+    "@fluidframework/test-runtime-utils": "2.10.0-307399",
     "@microsoft/api-extractor": "7.47.8",
     "@types/double-ended-queue": "^2.1.0",
     "@types/mocha": "^9.1.1",

package/src/blobManager/blobManager.ts

@@ -17,7 +17,7 @@ import {
 	IFluidHandleContext,
 	type IFluidHandleInternal,
 } from "@fluidframework/core-interfaces/internal";
-import { assert, Deferred } from "@fluidframework/core-utils/internal";
+import { assert, Deferred, LazyPromise } from "@fluidframework/core-utils/internal";
 import {
 	IDocumentStorageService,
 	ICreateBlobResponse,
@@ -181,10 +181,9 @@ export class BlobManager extends TypedEventEmitter<IBlobManagerEvents> {
 	private readonly runtime: IBlobManagerRuntime;
 	private readonly closeContainer: (error?: ICriticalContainerError) => void;
 	private readonly localBlobIdGenerator: () => string;
-	// TODO: consider to replace with a lazy promise
 	private readonly pendingStashedBlobs: Map<string, Promise<ICreateBlobResponse | void>> =
 		new Map();
-	private stashedBlobsUploadP: Promise<(void | ICreateBlobResponse)[]> | undefined = undefined;
+	public readonly stashedBlobsUploadP: Promise<(void | ICreateBlobResponse)[]>;
 
 	constructor(props: {
 		readonly routeContext: IFluidHandleContext;
@@ -271,11 +270,18 @@ export class BlobManager extends TypedEventEmitter<IBlobManagerEvents> {
 				uploadP: this.pendingStashedBlobs.get(localId),
 			});
 		});
-		this.waitForStashedBlobs()
-			.catch(() => {})
-			.finally(() => {
-				this.pendingStashedBlobs.clear();
-			});
+
+		this.stashedBlobsUploadP = new LazyPromise(async () =>
+			PerformanceEvent.timedExecAsync(
+				this.mc.logger,
+				{ eventName: "BlobUploadProcessStashedChanges", count: this.pendingStashedBlobs.size },
+				async () => Promise.all(this.pendingStashedBlobs.values()),
+				{ start: true, end: true },
+			),
+		).finally(() => {
+			this.pendingStashedBlobs.clear();
+		});
+
 		this.sendBlobAttachOp = (localId: string, blobId?: string) => {
 			const pendingEntry = this.pendingBlobs.get(localId);
 			assert(
@@ -307,17 +313,6 @@ export class BlobManager extends TypedEventEmitter<IBlobManagerEvents> {
 		};
 	}
 
-	public async waitForStashedBlobs(): Promise<(void | ICreateBlobResponse)[]> {
-		if (!this.stashedBlobsUploadP) {
-			this.stashedBlobsUploadP = Promise.all(this.pendingStashedBlobs.values()).finally(() => {
-				this.stashedBlobsUploadP = undefined;
-				return;
-			});
-		}
-
-		return this.stashedBlobsUploadP;
-	}
-
 	public get allBlobsAttached(): boolean {
 		for (const [, entry] of this.pendingBlobs) {
 			if (entry.attached === false) {
@@ -344,26 +339,6 @@ export class BlobManager extends TypedEventEmitter<IBlobManagerEvents> {
 	public hasPendingStashedUploads(): boolean {
 		return Array.from(this.pendingBlobs.values()).some((e) => e.stashedUpload === true);
 	}
-	/**
-	 * Upload blobs added while offline. This must be completed before connecting and resubmitting ops.
-	 */
-	public async trackPendingStashedUploads(): Promise<void> {
-		const pendingUploads = Array.from(this.pendingBlobs.values())
-			.filter((e) => e.stashedUpload === true)
-			.map(async (e) => e.uploadP);
-		if (pendingUploads.length === 0) {
-			return;
-		}
-		await PerformanceEvent.timedExecAsync(
-			this.mc.logger,
-			{
-				eventName: "BlobUploadProcessStashedChanges",
-				count: pendingUploads.length,
-			},
-			async () => Promise.all(pendingUploads),
-			{ start: true, end: true },
-		);
-	}
 
 	public async getBlob(blobId: string): Promise<ArrayBufferLike> {
 		// Verify that the blob is not deleted, i.e., it has not been garbage collected. If it is, this will throw
@@ -552,6 +527,16 @@ export class BlobManager extends TypedEventEmitter<IBlobManagerEvents> {
 
 	private onUploadResolve(localId: string, response: ICreateBlobResponseWithTTL) {
 		const entry = this.pendingBlobs.get(localId);
+		if (entry === undefined && this.pendingStashedBlobs.has(localId)) {
+			// The blob was already processed and deleted. This can happen if the blob was reuploaded by
+			// the stashing process and the original upload was processed before the stashed upload.
+			this.mc.logger.sendTelemetryEvent({
+				eventName: "StashedBlobAlreadyProcessed",
+				localId,
+			});
+			return;
+		}
+
 		assert(entry !== undefined, 0x6c8 /* pending blob entry not found for uploaded blob */);
 		if ((entry.abortSignal?.aborted === true && !entry.opsent) || this.stopAttaching) {
 			this.mc.logger.sendTelemetryEvent({

package/src/containerRuntime.ts

@@ -1116,7 +1116,7 @@ export class ContainerRuntime
 			undefined, // summaryConfiguration
 		);
 
-		runtime.blobManager.trackPendingStashedUploads().then(
+		runtime.blobManager.stashedBlobsUploadP.then(
 			() => {
 				// make sure we didn't reconnect before the promise resolved
 				if (runtime.delayConnectClientId !== undefined && !runtime.disposed) {

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/container-runtime";
-export const pkgVersion = "2.10.0-307060";
+export const pkgVersion = "2.10.0-307399";