@fluidframework/container-runtime 2.4.0 → 2.5.0-302463

package/src/dataStoreContexts.ts

@@ -7,6 +7,7 @@ import { IDisposable, ITelemetryBaseLogger } from "@fluidframework/core-interfac
 import { assert, Deferred, Lazy } from "@fluidframework/core-utils/internal";
 import {
 	ITelemetryLoggerExt,
+	PerformanceEvent,
 	createChildLogger,
 } from "@fluidframework/telemetry-utils/internal";
 
@@ -40,7 +41,7 @@ export class DataStoreContexts
 				.catch((contextError) => {
 					this._logger.sendErrorEvent(
 						{
-							eventName: "FluidDataStoreContextDisposeError",
+							eventName: "DisposeError",
 							fluidDataStoreId,
 						},
 						contextError,
@@ -52,7 +53,10 @@ export class DataStoreContexts
 	private readonly _logger: ITelemetryLoggerExt;
 
 	constructor(baseLogger: ITelemetryBaseLogger) {
-		this._logger = createChildLogger({ logger: baseLogger });
+		this._logger = createChildLogger({
+			namespace: "FluidDataStoreContexts",
+			logger: baseLogger,
+		});
 	}
 
 	[Symbol.iterator](): Iterator<[string, FluidDataStoreContext]> {
@@ -140,11 +144,20 @@ export class DataStoreContexts
 	): Promise<FluidDataStoreContext | undefined> {
 		const deferredContext = this.ensureDeferred(id);
 
-		if (!wait && !deferredContext.isCompleted) {
-			return undefined;
-		}
-
-		return deferredContext.promise;
+		const existing = deferredContext.isCompleted;
+		return PerformanceEvent.timedExecAsync(
+			this._logger,
+			{
+				eventName: "GetBoundOrRemoted",
+				wait,
+				existing,
+			},
+			async () => (!wait && !existing ? undefined : deferredContext.promise),
+			{
+				start: true,
+				end: true,
+			},
+		);
 	}
 
 	private ensureDeferred(id: string): Deferred<FluidDataStoreContext> {

package/src/opProperties.ts

@@ -12,7 +12,7 @@ export const opSize = (op: ISequencedDocumentMessage): number => {
 	// Some messages may already have string contents,
 	// so stringifying them again will add inaccurate overhead.
 	const content =
-		typeof op.contents === "string" ? op.contents : JSON.stringify(op.contents) ?? "";
+		typeof op.contents === "string" ? op.contents : (JSON.stringify(op.contents) ?? "");
 	const data = opHasData(op) ? op.data : "";
 	return content.length + data.length;
 };

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/container-runtime";
-export const pkgVersion = "2.4.0";
+export const pkgVersion = "2.5.0-302463";

package/src/summary/documentSchema.ts

@@ -489,7 +489,7 @@ export class DocumentsSchemaController {
 		// Latter is importnat sure that's what will go into summary.
 		this.documentSchema = !existing
 			? this.desiredSchema
-			: (documentMetadataSchema as IDocumentSchemaCurrent) ??
+			: ((documentMetadataSchema as IDocumentSchemaCurrent) ??
 				({
 					version: currentDocumentVersionSchema,
 					// see comment in summarizeDocumentSchema() on why it has to stay zero
@@ -499,7 +499,7 @@ export class DocumentsSchemaController {
 					runtime: {
 						explicitSchemaControl: boolToProp(!existing && features.explicitSchemaControl),
 					},
-				} satisfies IDocumentSchemaCurrent);
+				} satisfies IDocumentSchemaCurrent));
 
 		checkRuntimeCompatibility(this.documentSchema, "document");
 		this.validateSeqNumber(this.documentSchema.refSeq, snapshotSequenceNumber, "summary");