@fluidframework/container-runtime 2.20.0 → 2.21.0

package/dist/containerRuntime.js

@@ -4,7 +4,7 @@
  * Licensed under the MIT License.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ContainerRuntime = exports.loadContainerRuntime = exports.getSingleUseLegacyLogCallback = exports.makeLegacySendBatchFn = exports.getDeviceSpec = exports.agentSchedulerId = exports.isUnpackedRuntimeMessage = exports.defaultPendingOpsRetryDelayMs = exports.defaultPendingOpsWaitTimeoutMs = exports.disabledCompressionConfig = exports.CompressionAlgorithms = exports.defaultRuntimeHeaderData = exports.InactiveResponseHeaderKey = exports.TombstoneResponseHeaderKey = exports.DeletedResponseHeaderKey = exports.DefaultSummaryConfiguration = void 0;
+exports.ContainerRuntime = exports.loadContainerRuntime = exports.getSingleUseLegacyLogCallback = exports.makeLegacySendBatchFn = exports.getDeviceSpec = exports.agentSchedulerId = exports.isUnpackedRuntimeMessage = exports.defaultPendingOpsRetryDelayMs = exports.defaultPendingOpsWaitTimeoutMs = exports.disabledCompressionConfig = exports.CompressionAlgorithms = exports.defaultRuntimeHeaderData = exports.InactiveResponseHeaderKey = exports.TombstoneResponseHeaderKey = exports.DeletedResponseHeaderKey = exports.DefaultSummaryConfiguration = exports.isSummariesDisabled = void 0;
 const client_utils_1 = require("@fluid-internal/client-utils");
 const container_definitions_1 = require("@fluidframework/container-definitions");
 const internal_1 = require("@fluidframework/container-definitions/internal");
@@ -27,6 +27,7 @@ const deltaManagerProxies_js_1 = require("./deltaManagerProxies.js");
 const deltaScheduler_js_1 = require("./deltaScheduler.js");
 const index_js_2 = require("./gc/index.js");
 const inboundBatchAggregator_js_1 = require("./inboundBatchAggregator.js");
+const layerCompatState_js_1 = require("./layerCompatState.js");
 const messageTypes_js_1 = require("./messageTypes.js");
 const index_js_3 = require("./opLifecycle/index.js");
 const packageVersion_js_1 = require("./packageVersion.js");
@@ -52,6 +53,10 @@ function getUnknownMessageTypeError(unknownContainerRuntimeMessageType, codePath
         },
     });
 }
+function isSummariesDisabled(config) {
+    return config.state === "disabled";
+}
+exports.isSummariesDisabled = isSummariesDisabled;
 /**
  * @legacy
  * @alpha
@@ -67,13 +72,14 @@ exports.DefaultSummaryConfiguration = {
     maxOpsSinceLastSummary: 7000,
     initialSummarizerDelayMs: 5 * 1000, // 5 secs.
     nonRuntimeOpWeight: 0.1,
-    runtimeOpWeight: 1.0,
+    runtimeOpWeight: 1,
     nonRuntimeHeuristicThreshold: 20,
 };
 /**
  * Error responses when requesting a deleted object will have this header set to true
  * @legacy
  * @alpha
+ * @deprecated This type will be moved to internal in 2.30. External usage is not necessary or supported.
  */
 exports.DeletedResponseHeaderKey = "wasDeleted";
 /**
@@ -113,7 +119,7 @@ var CompressionAlgorithms;
  * @alpha
  */
 exports.disabledCompressionConfig = {
-    minimumBatchSizeInBytes: Infinity,
+    minimumBatchSizeInBytes: Number.POSITIVE_INFINITY,
     compressionAlgorithm: CompressionAlgorithms.lz4,
 };
 const maxConsecutiveReconnectsKey = "Fluid.ContainerRuntime.MaxConsecutiveReconnects";
@@ -168,7 +174,9 @@ function getDeviceSpec() {
             };
         }
     }
-    catch { }
+    catch {
+        // Eat the error
+    }
     return {};
 }
 exports.getDeviceSpec = getDeviceSpec;
@@ -211,21 +219,20 @@ async function createSummarizer(loader, url) {
     let fluidObject;
     // Older containers may not have the "getEntryPoint" API
     // ! This check will need to stay until LTS of loader moves past 2.0.0-internal.7.0.0
-    if (resolvedContainer.getEntryPoint !== undefined) {
-        fluidObject = await resolvedContainer.getEntryPoint();
-    }
-    else {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment
-        const response = await resolvedContainer.request({
+    if (resolvedContainer.getEntryPoint === undefined) {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-explicit-any
+        const response = (await resolvedContainer.request({
             url: `/${summarizerRequestUrl}`,
-        });
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+        }));
         if (response.status !== 200 || response.mimeType !== "fluid/object") {
             throw (0, internal_6.responseToException)(response, request);
         }
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-assignment
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
         fluidObject = response.value;
     }
+    else {
+        fluidObject = await resolvedContainer.getEntryPoint();
+    }
     if (fluidObject?.ISummarizer === undefined) {
         throw new internal_7.UsageError("Fluid object does not implement ISummarizer");
     }
@@ -237,7 +244,9 @@ async function createSummarizer(loader, url) {
  * This allows new runtime to make documents not openable for old runtimes, one explicit document schema control is enabled.
  * Please see addMetadataToSummary() as well
  */
-function lastMessageFromMetadata(metadata) {
+function lastMessageFromMetadata(
+// eslint-disable-next-line import/no-deprecated
+metadata) {
     return metadata?.documentSchema?.runtime?.explicitSchemaControl
         ? metadata?.lastMessage
         : metadata?.message;
@@ -254,6 +263,7 @@ let getSingleUseLegacyLogCallback = (logger, type) => {
             details: { codePath, type },
         });
         // Now that we've logged, prevent future logging (globally).
+        // eslint-disable-next-line unicorn/consistent-function-scoping
         exports.getSingleUseLegacyLogCallback = () => () => { };
     };
 };
@@ -269,6 +279,8 @@ async function loadContainerRuntime(params) {
     return ContainerRuntime.loadRuntime(params);
 }
 exports.loadContainerRuntime = loadContainerRuntime;
+const defaultMaxConsecutiveReconnects = 7;
+const defaultTelemetrySignalSampleCount = 100;
 /**
  * Represents the runtime of the container. Contains helper functions/state of the container.
  * It will define the store level mappings.
@@ -324,7 +336,9 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         const [chunks, recentBatchInfo, metadata, electedSummarizerData, aliases, serializedIdCompressor,] = await Promise.all([
             tryFetchBlob(index_js_4.chunksBlobName),
             tryFetchBlob(index_js_4.recentBatchInfoBlobName),
+            // eslint-disable-next-line import/no-deprecated
             tryFetchBlob(index_js_4.metadataBlobName),
+            // eslint-disable-next-line import/no-deprecated
             tryFetchBlob(index_js_4.electedSummarizerBlobName),
             tryFetchBlob(index_js_4.aliasBlobName),
             tryFetchBlob(index_js_4.idCompressorBlobName),
@@ -338,6 +352,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // When we load with pending state, we reuse an old snapshot so we don't expect these numbers to match
         if (!context.pendingLocalState && runtimeSequenceNumber !== undefined) {
             // Unless bypass is explicitly set, then take action when sequence numbers mismatch.
+            // eslint-disable-next-line unicorn/no-lonely-if -- Separate if statements make flow easier to parse
             if (loadSequenceNumberVerification !== "bypass" &&
                 runtimeSequenceNumber !== protocolSequenceNumber) {
                 // Message to OCEs:
@@ -360,15 +375,18 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         }
         let desiredIdCompressorMode;
         switch (mc.config.getBoolean("Fluid.ContainerRuntime.IdCompressorEnabled")) {
-            case true:
+            case true: {
                 desiredIdCompressorMode = "on";
                 break;
-            case false:
+            }
+            case false: {
                 desiredIdCompressorMode = undefined;
                 break;
-            default:
+            }
+            default: {
                 desiredIdCompressorMode = enableRuntimeIdCompressor;
                 break;
+            }
         }
         // Enabling the IdCompressor is a one-way operation and we only want to
         // allow new containers to turn it on.
@@ -421,15 +439,16 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             if (pendingLocalState?.pendingIdCompressorState !== undefined) {
                 return deserializeIdCompressor(pendingLocalState.pendingIdCompressorState, compressorLogger);
             }
-            else if (serializedIdCompressor !== undefined) {
-                return deserializeIdCompressor(serializedIdCompressor, createSessionId(), compressorLogger);
+            else if (serializedIdCompressor === undefined) {
+                return createIdCompressor(compressorLogger);
             }
             else {
-                return createIdCompressor(compressorLogger);
+                return deserializeIdCompressor(serializedIdCompressor, createSessionId(), compressorLogger);
             }
         };
-        const compressionLz4 = compressionOptions.minimumBatchSizeInBytes !== Infinity &&
+        const compressionLz4 = compressionOptions.minimumBatchSizeInBytes !== Number.POSITIVE_INFINITY &&
             compressionOptions.compressionAlgorithm === "lz4";
+        // eslint-disable-next-line import/no-deprecated
         const documentSchemaController = new index_js_4.DocumentsSchemaController(existing, protocolSequenceNumber, metadata?.documentSchema, {
             explicitSchemaControl,
             compressionLz4,
@@ -507,9 +526,6 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
     get sessionSchema() {
         return this.documentsSchemaController.sessionSchema.runtime;
     }
-    get idCompressorMode() {
-        return this.sessionSchema.idCompressorMode;
-    }
     /**
      * {@inheritDoc @fluidframework/runtime-definitions#IContainerRuntimeBase.idCompressor}
      */
@@ -518,7 +534,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // If container uses delayed mode, then we can only expose generateDocumentUniqueId() and nothing else.
         // That's because any other usage will require immidiate loading of ID Compressor in next sessions in order
         // to reason over such things as session ID space.
-        if (this.idCompressorMode === "on") {
+        if (this.sessionSchema.idCompressorMode === "on") {
             (0, internal_2.assert)(this._idCompressor !== undefined, 0x8ea /* compressor should have been loaded */);
             return this._idCompressor;
         }
@@ -569,28 +585,6 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
     get disposed() {
         return this._disposed;
     }
-    get summarizer() {
-        (0, internal_2.assert)(this._summarizer !== undefined, 0x257 /* "This is not summarizing container" */);
-        return this._summarizer;
-    }
-    isSummariesDisabled() {
-        return this.summaryConfiguration.state === "disabled";
-    }
-    getMaxOpsSinceLastSummary() {
-        return this.summaryConfiguration.state !== "disabled"
-            ? this.summaryConfiguration.maxOpsSinceLastSummary
-            : 0;
-    }
-    getInitialSummarizerDelayMs() {
-        // back-compat: initialSummarizerDelayMs was moved from ISummaryRuntimeOptions
-        //   to ISummaryConfiguration in 0.60.
-        if (this.runtimeOptions.summaryOptions.initialSummarizerDelayMs !== undefined) {
-            return this.runtimeOptions.summaryOptions.initialSummarizerDelayMs;
-        }
-        return this.summaryConfiguration.state !== "disabled"
-            ? this.summaryConfiguration.initialSummarizerDelayMs
-            : 0;
-    }
     /**
      * If false, loading or using a Tombstoned object should merely log, not fail.
      * @deprecated NOT SUPPORTED - hardcoded to return false since it's deprecated.
@@ -607,14 +601,27 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
     get gcThrowOnTombstoneUsage() {
         return false;
     }
+    get ILayerCompatDetails() {
+        return layerCompatState_js_1.RuntimeCompatDetails;
+    }
     /***/
-    constructor(context, registry, metadata, electedSummarizerData, chunks, dataStoreAliasMap, runtimeOptions, containerScope, 
+    constructor(context, registry, 
+    // eslint-disable-next-line import/no-deprecated
+    metadata, 
+    // eslint-disable-next-line import/no-deprecated
+    electedSummarizerData, chunks, dataStoreAliasMap, baseRuntimeOptions, containerScope, 
     // Create a custom ITelemetryBaseLogger to output telemetry events.
-    baseLogger, existing, blobManagerSnapshot, _storage, createIdCompressor, documentsSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, summaryConfiguration = {
+    baseLogger, existing, 
+    // eslint-disable-next-line import/no-deprecated
+    blobManagerSnapshot, _storage, createIdCompressor, 
+    // eslint-disable-next-line import/no-deprecated
+    documentsSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, 
+    // eslint-disable-next-line unicorn/no-object-as-default-parameter
+    summaryConfiguration = {
         // the defaults
         ...exports.DefaultSummaryConfiguration,
         // the runtime configuration overrides
-        ...runtimeOptions.summaryOptions?.summaryConfigOverrides,
+        ...baseRuntimeOptions.summaryOptions?.summaryConfigOverrides,
     }, recentBatchInfo) {
         super();
         this.registry = registry;
@@ -625,19 +632,16 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         this.createIdCompressor = createIdCompressor;
         this.documentsSchemaController = documentsSchemaController;
         this.requestHandler = requestHandler;
-        this.summaryConfiguration = summaryConfiguration;
         this.imminentClosure = false;
         // We accumulate Id compressor Ops while Id compressor is not loaded yet (only for "delayed" mode)
         // Once it loads, it will process all such ops and we will stop accumulating further ops - ops will be processes as they come in.
         this.pendingIdCompressorOps = [];
-        this.defaultMaxConsecutiveReconnects = 7;
         this._orderSequentiallyCalls = 0;
         this.flushTaskExists = false;
         this.consecutiveReconnects = 0;
         this.ensureNoDataModelChangesCalls = 0;
         this._disposed = false;
         this.emitDirtyDocumentEvent = true;
-        this.defaultTelemetrySignalSampleCount = 100;
         this._signalTracking = {
             totalSignalsSentInLatencyWindow: 0,
             signalsLost: 0,
@@ -656,15 +660,19 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         this.snapshotCacheForLoadingGroupIds = new internal_2.PromiseCache({
             expiry: { policy: "absolute", durationMs: 60000 },
         });
+        this.readAndParseBlob = async (id) => (0, internal_4.readAndParse)(this.storage, id);
         const { options, clientDetails, connected, baseSnapshot, submitFn, submitBatchFn, submitSummaryFn, submitSignalFn, disposeFn, closeFn, deltaManager, quorum, audience, loader, pendingLocalState, supportedFeatures, snapshotWithContents, } = context;
+        // In old loaders without dispose functionality, closeFn is equivalent but will also switch container to readonly mode
+        this.disposeFn = disposeFn ?? closeFn;
+        const maybeLoaderCompatDetails = context;
+        (0, layerCompatState_js_1.validateLoaderCompatibility)(maybeLoaderCompatDetails.ILayerCompatDetails, this.disposeFn);
         // Backfill in defaults for the internal runtimeOptions, since they may not be present on the provided runtimeOptions object
-        this.runtimeOptions = {
+        const runtimeOptions = {
             flushMode: defaultFlushMode,
-            ...runtimeOptions,
+            ...baseRuntimeOptions,
         };
-        this.logger = (0, internal_7.createChildLogger)({ logger: this.baseLogger });
         this.mc = (0, internal_7.createChildMonitoringContext)({
-            logger: this.logger,
+            logger: this.baseLogger,
             namespace: "ContainerRuntime",
         });
         // If we support multiple algorithms in the future, then we would need to manage it here carefully.
@@ -681,19 +689,23 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         this.innerDeltaManager = deltaManager;
         // Here we could wrap/intercept on these functions to block/modify outgoing messages if needed.
         // This makes ContainerRuntime the final gatekeeper for outgoing messages.
-        this.submitFn = submitFn;
-        this.submitBatchFn = submitBatchFn;
-        this.submitSummaryFn = submitSummaryFn;
+        // back-compat: ADO #1385: Make this call unconditional in the future
+        this.submitSummaryFn =
+            submitSummaryFn ??
+                ((summaryOp, refseq) => submitFn(internal_3.MessageType.Summarize, summaryOp, false));
         this.submitSignalFn = submitSignalFn;
         // TODO: After IContainerContext.options is removed, we'll just create a new blank object {} here.
         // Values are generally expected to be set from the runtime side.
         this.options = options ?? {};
         this.clientDetails = clientDetails;
-        this.isSummarizerClient = this.clientDetails.type === index_js_4.summarizerClientType;
+        const isSummarizerClient = this.clientDetails.type === index_js_4.summarizerClientType;
         this.loadedFromVersionId = context.getLoadedFromVersion()?.id;
+        // eslint-disable-next-line unicorn/consistent-destructuring
         this._getClientId = () => context.clientId;
+        // eslint-disable-next-line unicorn/consistent-destructuring
         this._getAttachState = () => context.attachState;
         this.getAbsoluteUrl = async (relativeUrl) => {
+            // eslint-disable-next-line unicorn/consistent-destructuring
             if (context.getAbsoluteUrl === undefined) {
                 throw new Error("Driver does not implement getAbsoluteUrl");
             }
@@ -706,10 +718,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // customer should observe dirty state on the runtime (the owner of dirty state) directly, rather than on the IContainer.
         this.on("dirty", () => context.updateDirtyContainerState(true));
         this.on("saved", () => context.updateDirtyContainerState(false));
-        // In old loaders without dispose functionality, closeFn is equivalent but will also switch container to readonly mode
-        this.disposeFn = disposeFn ?? closeFn;
         // In cases of summarizer, we want to dispose instead since consumer doesn't interact with this container
-        this.closeFn = this.isSummarizerClient ? this.disposeFn : closeFn;
+        this.closeFn = isSummarizerClient ? this.disposeFn : closeFn;
         let loadSummaryNumber;
         // Get the container creation metadata. For new container, we initialize these. For existing containers,
         // get the values from the metadata blob.
@@ -739,16 +749,14 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             metadataValue: JSON.stringify(metadata?.gcFeatureMatrix),
             inputs: JSON.stringify({
                 // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-                gcOptions_gcGeneration: this.runtimeOptions.gcOptions[index_js_2.gcGenerationOptionName],
+                gcOptions_gcGeneration: runtimeOptions.gcOptions[index_js_2.gcGenerationOptionName],
             }),
         });
         this.telemetryDocumentId = metadata?.telemetryDocumentId ?? (0, uuid_1.v4)();
-        this.disableAttachReorder = this.mc.config.getBoolean("Fluid.ContainerRuntime.disableAttachOpReorder");
         const opGroupingManager = new index_js_3.OpGroupingManager({
             groupedBatchingEnabled: this.groupedBatchingEnabled,
-            opCountThreshold: this.mc.config.getNumber("Fluid.ContainerRuntime.GroupedBatchingOpCount") ?? 2,
         }, this.mc.logger);
-        const opSplitter = new index_js_3.OpSplitter(chunks, this.submitBatchFn, runtimeOptions.chunkSizeInBytes, runtimeOptions.maxBatchSizeInBytes, this.mc.logger);
+        const opSplitter = new index_js_3.OpSplitter(chunks, submitBatchFn, runtimeOptions.chunkSizeInBytes, runtimeOptions.maxBatchSizeInBytes, this.mc.logger);
         this.remoteMessageProcessor = new index_js_3.RemoteMessageProcessor(opSplitter, new index_js_3.OpDecompressor(this.mc.logger), opGroupingManager);
         const pendingRuntimeState = pendingLocalState;
         this.pendingStateManager = new pendingStateManager_js_1.PendingStateManager({
@@ -758,7 +766,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             reSubmitBatch: this.reSubmitBatch.bind(this),
             isActiveConnection: () => this.innerDeltaManager.active,
             isAttached: () => this.attachState !== container_definitions_1.AttachState.Detached,
-        }, pendingRuntimeState?.pending, this.logger);
+        }, pendingRuntimeState?.pending, this.baseLogger);
         let outerDeltaManager;
         this.useDeltaManagerOpsProxy =
             this.mc.config.getBoolean("Fluid.ContainerRuntime.DeltaManagerOpsProxy") === true;
@@ -773,24 +781,35 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         }
         this._deltaManager = outerDeltaManager;
         this.handleContext = new containerHandleContext_js_1.ContainerFluidHandleContext("", this);
-        if (this.summaryConfiguration.state === "enabled") {
-            this.validateSummaryHeuristicConfiguration(this.summaryConfiguration);
-        }
-        this.summariesDisabled = this.isSummariesDisabled();
-        this.maxOpsSinceLastSummary = this.getMaxOpsSinceLastSummary();
-        this.initialSummarizerDelayMs = this.getInitialSummarizerDelayMs();
+        if (summaryConfiguration.state === "enabled") {
+            this.validateSummaryHeuristicConfiguration(summaryConfiguration);
+        }
+        this.summariesDisabled = isSummariesDisabled(summaryConfiguration);
+        const { maxOpsSinceLastSummary = 0, initialSummarizerDelayMs = 0 } = isSummariesDisabled(summaryConfiguration)
+            ? {}
+            : {
+                ...summaryConfiguration,
+                initialSummarizerDelayMs: 
+                // back-compat: initialSummarizerDelayMs was moved from ISummaryRuntimeOptions
+                //   to ISummaryConfiguration in 0.60.
+                runtimeOptions.summaryOptions.initialSummarizerDelayMs ??
+                    summaryConfiguration.initialSummarizerDelayMs,
+            };
         this.maxConsecutiveReconnects =
-            this.mc.config.getNumber(maxConsecutiveReconnectsKey) ??
-                this.defaultMaxConsecutiveReconnects;
-        if (this.runtimeOptions.flushMode ===
-            internal_5.FlushModeExperimental.Async &&
-            supportedFeatures?.get("referenceSequenceNumbers") !== true) {
+            this.mc.config.getNumber(maxConsecutiveReconnectsKey) ?? defaultMaxConsecutiveReconnects;
+        // If the context has ILayerCompatDetails, it supports referenceSequenceNumbers since that features
+        // predates ILayerCompatDetails.
+        const referenceSequenceNumbersSupported = maybeLoaderCompatDetails.ILayerCompatDetails === undefined
+            ? supportedFeatures?.get("referenceSequenceNumbers") === true
+            : true;
+        if (runtimeOptions.flushMode === internal_5.FlushModeExperimental.Async &&
+            !referenceSequenceNumbersSupported) {
             // The loader does not support reference sequence numbers, falling back on FlushMode.TurnBased
             this.mc.logger.sendErrorEvent({ eventName: "FlushModeFallback" });
             this._flushMode = internal_5.FlushMode.TurnBased;
         }
         else {
-            this._flushMode = this.runtimeOptions.flushMode;
+            this._flushMode = runtimeOptions.flushMode;
         }
         this.offlineEnabled =
             this.mc.config.getBoolean("Fluid.Container.enableOfflineLoad") ?? false;
@@ -805,6 +824,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         if (this.offlineEnabled) {
             this.duplicateBatchDetector = new index_js_3.DuplicateBatchDetector(recentBatchInfo);
         }
+        // eslint-disable-next-line unicorn/consistent-destructuring
         if (context.attachState === container_definitions_1.AttachState.Attached) {
             const maxSnapshotCacheDurationMs = this._storage?.policies?.maximumCacheDurationMs;
             if (maxSnapshotCacheDurationMs !== undefined &&
@@ -817,13 +837,13 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         }
         this.garbageCollector = index_js_2.GarbageCollector.create({
             runtime: this,
-            gcOptions: this.runtimeOptions.gcOptions,
+            gcOptions: runtimeOptions.gcOptions,
             baseSnapshot,
             baseLogger: this.mc.logger,
             existing,
             metadata,
             createContainerMetadata: this.createContainerMetadata,
-            isSummarizerClient: this.isSummarizerClient,
+            isSummarizerClient,
             getNodePackagePath: async (nodePath) => this.getGCNodePackagePath(nodePath),
             getLastSummaryTimestampMs: () => this.messageAtLastSummary?.timestamp,
             readAndParseBlob: async (id) => (0, internal_4.readAndParse)(this.storage, id),
@@ -837,7 +857,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         const summaryReferenceSequenceNumber = baseSnapshot === undefined || metadata?.disableIsolatedChannels === true
             ? undefined
             : loadedFromSequenceNumber;
-        this.summarizerNode = (0, index_js_4.createRootSummarizerNodeWithGC)((0, internal_7.createChildLogger)({ logger: this.logger, namespace: "SummarizerNode" }), 
+        this.summarizerNode = (0, index_js_4.createRootSummarizerNodeWithGC)((0, internal_7.createChildLogger)({ logger: this.baseLogger, namespace: "SummarizerNode" }), 
         // Summarize function to call when summarize is called. Summarizer node always tracks summary state.
         async (fullTree, trackState, telemetryContext) => this.summarizeInternal(fullTree, trackState, telemetryContext), 
         // Latest change sequence number, no changes since summary applied yet
@@ -897,14 +917,14 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             stashedBlobs: pendingRuntimeState?.pendingAttachmentBlobs,
             closeContainer: (error) => this.closeFn(error),
         });
-        this.deltaScheduler = new deltaScheduler_js_1.DeltaScheduler(this.innerDeltaManager, this, (0, internal_7.createChildLogger)({ logger: this.logger, namespace: "DeltaScheduler" }));
-        this.inboundBatchAggregator = new inboundBatchAggregator_js_1.InboundBatchAggregator(this.innerDeltaManager, () => this.clientId, (0, internal_7.createChildLogger)({ logger: this.logger, namespace: "InboundBatchAggregator" }));
+        this.deltaScheduler = new deltaScheduler_js_1.DeltaScheduler(this.innerDeltaManager, this, (0, internal_7.createChildLogger)({ logger: this.baseLogger, namespace: "DeltaScheduler" }));
+        this.inboundBatchAggregator = new inboundBatchAggregator_js_1.InboundBatchAggregator(this.innerDeltaManager, () => this.clientId, (0, internal_7.createChildLogger)({ logger: this.baseLogger, namespace: "InboundBatchAggregator" }));
         const disablePartialFlush = this.mc.config.getBoolean("Fluid.ContainerRuntime.DisablePartialFlush");
-        const legacySendBatchFn = (0, exports.makeLegacySendBatchFn)(this.submitFn, this.innerDeltaManager);
+        const legacySendBatchFn = (0, exports.makeLegacySendBatchFn)(submitFn, this.innerDeltaManager);
         this.outbox = new index_js_3.Outbox({
             shouldSend: () => this.canSendOps(),
             pendingStateManager: this.pendingStateManager,
-            submitBatchFn: this.submitBatchFn,
+            submitBatchFn,
             legacySendBatchFn,
             compressor: new index_js_3.OpCompressor(this.mc.logger),
             splitter: opSplitter,
@@ -951,7 +971,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         const closeSummarizerDelayOverride = this.mc.config.getNumber("Fluid.ContainerRuntime.Test.CloseSummarizerDelayOverrideMs");
         this.closeSummarizerDelayMs =
             closeSummarizerDelayOverride ?? defaultCloseSummarizerDelayMs;
-        this.summaryCollection = new index_js_4.SummaryCollection(this.deltaManager, this.logger);
+        const summaryCollection = new index_js_4.SummaryCollection(this.deltaManager, this.baseLogger);
         this.dirtyContainer =
             this.attachState !== container_definitions_1.AttachState.Attached || this.hasPendingMessages();
         context.updateDirtyContainerState(this.dirtyContainer);
@@ -960,14 +980,17 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         }
         else {
             const orderedClientLogger = (0, internal_7.createChildLogger)({
-                logger: this.logger,
+                logger: this.baseLogger,
                 namespace: "OrderedClientElection",
             });
             const orderedClientCollection = new index_js_4.OrderedClientCollection(orderedClientLogger, this.innerDeltaManager, this._quorum);
             const orderedClientElectionForSummarizer = new index_js_4.OrderedClientElection(orderedClientLogger, orderedClientCollection, electedSummarizerData ?? this.innerDeltaManager.lastSequenceNumber, index_js_4.SummarizerClientElection.isClientEligible, this.mc.config.getBoolean("Fluid.ContainerRuntime.OrderedClientElection.EnablePerformanceEvents"));
-            this.summarizerClientElection = new index_js_4.SummarizerClientElection(orderedClientLogger, this.summaryCollection, orderedClientElectionForSummarizer, this.maxOpsSinceLastSummary);
-            if (this.isSummarizerClient) {
-                this._summarizer = new index_js_4.Summarizer(this /* ISummarizerRuntime */, () => this.summaryConfiguration, this /* ISummarizerInternalsProvider */, this.handleContext, this.summaryCollection, async (runtime) => index_js_4.RunWhileConnectedCoordinator.create(runtime, 
+            this.summarizerClientElection = new index_js_4.SummarizerClientElection(orderedClientLogger, summaryCollection, orderedClientElectionForSummarizer, maxOpsSinceLastSummary);
+            if (isSummarizerClient) {
+                // eslint-disable-next-line import/no-deprecated
+                this._summarizer = new index_js_4.Summarizer(this /* ISummarizerRuntime */, () => summaryConfiguration, this /* ISummarizerInternalsProvider */, this.handleContext, summaryCollection, 
+                // eslint-disable-next-line import/no-deprecated
+                async (runtime) => index_js_4.RunWhileConnectedCoordinator.create(runtime, 
                 // Summarization runs in summarizer client and needs access to the real (non-proxy) active
                 // information. The proxy delta manager would always return false for summarizer client.
                 () => this.innerDeltaManager.active));
@@ -976,48 +999,49 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 // Only create a SummaryManager and SummarizerClientElection
                 // if summaries are enabled and we are not the summarizer client.
                 const defaultAction = () => {
-                    if (this.summaryCollection.opsSinceLastAck > this.maxOpsSinceLastSummary) {
+                    if (summaryCollection.opsSinceLastAck > maxOpsSinceLastSummary) {
                         this.mc.logger.sendTelemetryEvent({ eventName: "SummaryStatus:Behind" });
                         // unregister default to no log on every op after falling behind
                         // and register summary ack handler to re-register this handler
                         // after successful summary
-                        this.summaryCollection.once(internal_3.MessageType.SummaryAck, () => {
+                        summaryCollection.once(internal_3.MessageType.SummaryAck, () => {
                             this.mc.logger.sendTelemetryEvent({
                                 eventName: "SummaryStatus:CaughtUp",
                             });
                             // we've caught up, so re-register the default action to monitor for
                             // falling behind, and unregister ourself
-                            this.summaryCollection.on("default", defaultAction);
+                            summaryCollection.on("default", defaultAction);
                         });
-                        this.summaryCollection.off("default", defaultAction);
+                        summaryCollection.off("default", defaultAction);
                     }
                 };
-                this.summaryCollection.on("default", defaultAction);
+                summaryCollection.on("default", defaultAction);
                 // Create the SummaryManager and mark the initial state
                 this.summaryManager = new index_js_4.SummaryManager(this.summarizerClientElection, this, // IConnectedState
-                this.summaryCollection, this.logger, this.formCreateSummarizerFn(loader), new throttler_js_1.Throttler(60 * 1000, // 60 sec delay window
+                summaryCollection, this.baseLogger, this.formCreateSummarizerFn(loader), new throttler_js_1.Throttler(60 * 1000, // 60 sec delay window
                 30 * 1000, // 30 sec max delay
                 // throttling function increases exponentially (0ms, 40ms, 80ms, 160ms, etc)
                 (0, throttler_js_1.formExponentialFn)({ coefficient: 20, initialDelay: 0 })), {
-                    initialDelayMs: this.initialSummarizerDelayMs,
+                    initialDelayMs: initialSummarizerDelayMs,
                 });
                 // Forward events from SummaryManager
-                [
+                for (const eventName of [
                     "summarize",
                     "summarizeAllAttemptsFailed",
                     "summarizerStop",
                     "summarizerStart",
                     "summarizerStartupFailed",
-                ].forEach((eventName) => {
+                ]) {
                     this.summaryManager?.on(eventName, (...args) => {
                         this.emit(eventName, ...args);
                     });
-                });
+                }
                 this.summaryManager.start();
             }
         }
         // logging hardware telemetry
-        this.logger.sendTelemetryEvent({
+        this.baseLogger.send({
+            category: "generic",
             eventName: "DeviceSpec",
             ...getDeviceSpec(),
         });
@@ -1029,13 +1053,12 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             summaryFormatVersion: metadata?.summaryFormatVersion,
             disableIsolatedChannels: metadata?.disableIsolatedChannels,
             gcVersion: metadata?.gcFeature,
-            options: JSON.stringify(runtimeOptions),
+            options: JSON.stringify(baseRuntimeOptions),
             idCompressorModeMetadata: metadata?.documentSchema?.runtime?.idCompressorMode,
-            idCompressorMode: this.idCompressorMode,
+            idCompressorMode: this.sessionSchema.idCompressorMode,
             sessionRuntimeSchema: JSON.stringify(this.sessionSchema),
             featureGates: JSON.stringify({
                 ...featureGatesForTelemetry,
-                disableAttachReorder: this.disableAttachReorder,
                 disablePartialFlush,
                 closeSummarizerDelayOverride,
             }),
@@ -1043,11 +1066,10 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             groupedBatchingEnabled: this.groupedBatchingEnabled,
             initialSequenceNumber: this.deltaManager.initialSequenceNumber,
         });
-        (0, connectionTelemetry_js_1.ReportOpPerfTelemetry)(this.clientId, this._deltaManager, this, this.logger);
-        (0, batchTracker_js_1.BindBatchTracker)(this, this.logger);
+        (0, connectionTelemetry_js_1.ReportOpPerfTelemetry)(this.clientId, this._deltaManager, this, this.baseLogger);
+        (0, batchTracker_js_1.BindBatchTracker)(this, this.baseLogger);
         this.entryPoint = new internal_2.LazyPromise(async () => {
-            if (this.isSummarizerClient) {
-                (0, internal_2.assert)(this._summarizer !== undefined, 0x5bf /* Summarizer object is undefined in a summarizer client */);
+            if (this._summarizer !== undefined) {
                 return this._summarizer;
             }
             return provideEntryPoint(this);
@@ -1056,8 +1078,9 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // saved state, i.e. all the ops marked by Loader layer sa savedOp === true.
         this.skipSavedCompressorOps = pendingRuntimeState?.pendingIdCompressorState !== undefined;
     }
+    // eslint-disable-next-line import/no-deprecated
     onSchemaChange(schema) {
-        this.logger.sendTelemetryEvent({
+        this.mc.logger.sendTelemetryEvent({
             eventName: "SchemaChangeAccept",
             sessionRuntimeSchema: JSON.stringify(schema),
         });
@@ -1092,8 +1115,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
      * Initializes the state from the base snapshot this container runtime loaded from.
      */
     async initializeBaseState() {
-        if (this.idCompressorMode === "on" ||
-            (this.idCompressorMode === "delayed" && this.connected)) {
+        if (this.sessionSchema.idCompressorMode === "on" ||
+            (this.sessionSchema.idCompressorMode === "delayed" && this.connected)) {
             this._idCompressor = await this.createIdCompressor();
             // This is called from loadRuntime(), long before we process any ops, so there should be no ops accumulated yet.
             (0, internal_2.assert)(this.pendingIdCompressorOps.length === 0, 0x8ec /* no pending ops */);
@@ -1136,7 +1159,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // Lookup up in the cache, if not present then make the network call as multiple datastores could
         // be in same loading group. So, once we have fetched the snapshot for that loading group on
         // any request, then cache that as same group could be requested in future too.
-        const snapshot = await this.snapshotCacheForLoadingGroupIds.addOrGet(sortedLoadingGroupIds.join(), async () => {
+        const snapshot = await this.snapshotCacheForLoadingGroupIds.addOrGet(sortedLoadingGroupIds.join(","), async () => {
             (0, internal_2.assert)(this.storage.getSnapshot !== undefined, 0x8ee /* getSnapshot api should be defined if used */);
             loadedFromCache = false;
             return this.storage.getSnapshot({
@@ -1145,7 +1168,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 loadingGroupIds: sortedLoadingGroupIds,
             });
         });
-        this.logger.sendTelemetryEvent({
+        this.mc.logger.sendTelemetryEvent({
             eventName: "GroupIdSnapshotFetched",
             details: JSON.stringify({
                 fromCache: loadedFromCache,
@@ -1175,7 +1198,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             // another snapshot from which the summarizer loaded and it is behind, then just give up as
             // the summarizer state is not up to date.
             // This should be a recoverable scenario and shouldn't happen as we should process the ack first.
-            if (this.isSummarizerClient) {
+            if (this._summarizer !== undefined) {
                 throw new Error("Summarizer client behind, loaded newer snapshot with loadingGroupId");
             }
             // We want to catchup from sequenceNumber to targetSequenceNumber
@@ -1237,7 +1260,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     return {
                         status: 200,
                         mimeType: "fluid/object",
-                        value: this.summarizer,
+                        value: this._summarizer,
                     };
                 }
                 return (0, internal_6.create404Response)(request);
@@ -1303,6 +1326,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         const documentSchema = this.documentsSchemaController.summarizeDocumentSchema(this.deltaManager.lastSequenceNumber);
         // Is document schema explicit control on?
         const explicitSchemaControl = documentSchema?.runtime.explicitSchemaControl;
+        // eslint-disable-next-line import/no-deprecated
         const metadata = {
             ...this.createContainerMetadata,
             // Increment the summary number for the next summary that will be generated.
@@ -1316,7 +1340,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             // last message's sequence number.
             // See also lastMessageFromMetadata()
             message: explicitSchemaControl
-                ? { sequenceNumber: -1 }
+                ? // eslint-disable-next-line import/no-deprecated
+                    { sequenceNumber: -1 }
                 : message,
             lastMessage: explicitSchemaControl ? message : undefined,
             documentSchema,
@@ -1432,9 +1457,10 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         switch (opContents.type) {
             case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp:
             case messageTypes_js_1.ContainerMessageType.Attach:
-            case messageTypes_js_1.ContainerMessageType.Alias:
+            case messageTypes_js_1.ContainerMessageType.Alias: {
                 return this.channelCollection.applyStashedOp(opContents);
-            case messageTypes_js_1.ContainerMessageType.IdAllocation:
+            }
+            case messageTypes_js_1.ContainerMessageType.IdAllocation: {
                 // IDs allocation ops in stashed state are ignored because the tip state of the compressor
                 // is serialized into the pending state. This is done because generation of new IDs during
                 // stashed op application (or, later, resubmit) must generate new IDs and if the compressor
@@ -1444,17 +1470,22 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 // and the runtime could filter out all ID allocation ops from the stashed state and apply them
                 // before applying the rest of the stashed ops. This would accomplish the same thing but with
                 // better performance in future incremental stashed state creation.
-                (0, internal_2.assert)(this.idCompressorMode !== undefined, 0x8f1 /* ID compressor should be in use */);
+                (0, internal_2.assert)(this.sessionSchema.idCompressorMode !== undefined, 0x8f1 /* ID compressor should be in use */);
                 return;
-            case messageTypes_js_1.ContainerMessageType.DocumentSchemaChange:
+            }
+            case messageTypes_js_1.ContainerMessageType.DocumentSchemaChange: {
                 return;
-            case messageTypes_js_1.ContainerMessageType.BlobAttach:
+            }
+            case messageTypes_js_1.ContainerMessageType.BlobAttach: {
                 return;
-            case messageTypes_js_1.ContainerMessageType.Rejoin:
+            }
+            case messageTypes_js_1.ContainerMessageType.Rejoin: {
                 throw new Error("rejoin not expected here");
-            case messageTypes_js_1.ContainerMessageType.GC:
+            }
+            case messageTypes_js_1.ContainerMessageType.GC: {
                 // GC op is only sent in summarizer which should never have stashed ops.
                 throw new internal_7.LoggingError("GC op not expected to be stashed in summarizer");
+            }
             default: {
                 const error = getUnknownMessageTypeError(opContents.type, "applyStashedOp" /* codePath */);
                 this.closeFn(error);
@@ -1464,7 +1495,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
     }
     async loadIdCompressor() {
         if (this._idCompressor === undefined &&
-            this.idCompressorMode !== undefined &&
+            this.sessionSchema.idCompressorMode !== undefined &&
             this._loadIdCompressor === undefined) {
             this._loadIdCompressor = this.createIdCompressor()
                 .then((compressor) => {
@@ -1478,7 +1509,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 this._idCompressor = compressor;
             })
                 .catch((error) => {
-                this.logger.sendErrorEvent({ eventName: "IdCompressorDelayedLoad" }, error);
+                this.mc.logger.sendErrorEvent({ eventName: "IdCompressorDelayedLoad" }, error);
                 throw error;
             });
         }
@@ -1489,7 +1520,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         const currentClientId = this._audience.getSelf()?.clientId;
         (0, internal_2.assert)(clientId === currentClientId, 0x977 /* input clientId does not match Audience */);
         (0, internal_2.assert)(this.clientId === currentClientId, 0x978 /* this.clientId does not match Audience */);
-        if (connected && this.idCompressorMode === "delayed") {
+        if (connected && this.sessionSchema.idCompressorMode === "delayed") {
             // eslint-disable-next-line @typescript-eslint/no-floating-promises
             this.loadIdCompressor();
         }
@@ -1578,7 +1609,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // or something different, like a system message.
         const hasModernRuntimeMessageEnvelope = messageCopy.type === internal_3.MessageType.Operation;
         const savedOp = messageCopy.metadata?.savedOp;
-        const logLegacyCase = (0, exports.getSingleUseLegacyLogCallback)(this.logger, messageCopy.type);
+        const logLegacyCase = (0, exports.getSingleUseLegacyLogCallback)(this.mc.logger, messageCopy.type);
         let runtimeBatch = hasModernRuntimeMessageEnvelope || isUnpackedRuntimeMessage(messageCopy);
         if (runtimeBatch) {
             // We expect runtime messages to have JSON contents - deserialize it in place.
@@ -1680,11 +1711,11 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         let error;
         try {
             if (!runtimeBatch) {
-                messagesWithMetadata.forEach(({ message }) => {
+                for (const { message } of messagesWithMetadata) {
                     this.ensureNoDataModelChanges(() => {
                         this.observeNonRuntimeMessage(message);
                     });
-                });
+                }
                 return;
             }
             // Updates a message's minimum sequence number to the minimum sequence number that container
@@ -1752,8 +1783,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 this.emit("op", message, true /* runtimeMessage */);
             }
         }
-        catch (e) {
-            error = e;
+        catch (error_) {
+            error = error_;
             throw error;
         }
         finally {
@@ -1810,30 +1841,39 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         switch (message.type) {
             case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp:
             case messageTypes_js_1.ContainerMessageType.Attach:
-            case messageTypes_js_1.ContainerMessageType.Alias:
+            case messageTypes_js_1.ContainerMessageType.Alias: {
                 // Remove the metadata from the message before sending it to the channel collection. The metadata
                 // is added by the container runtime and is not part of the message that the channel collection and
                 // layers below it expect.
                 this.channelCollection.processMessages({ envelope: message, messagesContent, local });
                 break;
-            case messageTypes_js_1.ContainerMessageType.BlobAttach:
+            }
+            case messageTypes_js_1.ContainerMessageType.BlobAttach: {
                 this.blobManager.processBlobAttachMessage(message, local);
                 break;
-            case messageTypes_js_1.ContainerMessageType.IdAllocation:
+            }
+            case messageTypes_js_1.ContainerMessageType.IdAllocation: {
                 this.processIdCompressorMessages(contents, savedOp);
                 break;
-            case messageTypes_js_1.ContainerMessageType.GC:
+            }
+            case messageTypes_js_1.ContainerMessageType.GC: {
                 this.garbageCollector.processMessages(contents, message.timestamp, local);
                 break;
-            case messageTypes_js_1.ContainerMessageType.ChunkedOp:
+            }
+            case messageTypes_js_1.ContainerMessageType.ChunkedOp: {
                 // From observability POV, we should not expose the rest of the system (including "op" events on object) to these messages.
                 // Also resetReconnectCount() would be wrong - see comment that was there before this change was made.
                 (0, internal_2.assert)(false, 0x93d /* should not even get here */);
-            case messageTypes_js_1.ContainerMessageType.Rejoin:
+            }
+            case messageTypes_js_1.ContainerMessageType.Rejoin: {
                 break;
-            case messageTypes_js_1.ContainerMessageType.DocumentSchemaChange:
-                this.documentsSchemaController.processDocumentSchemaMessages(contents, local, message.sequenceNumber);
+            }
+            case messageTypes_js_1.ContainerMessageType.DocumentSchemaChange: {
+                this.documentsSchemaController.processDocumentSchemaMessages(
+                // eslint-disable-next-line import/no-deprecated
+                contents, local, message.sequenceNumber);
                 break;
+            }
             default: {
                 const error = getUnknownMessageTypeError(message.type, "validateAndProcessRuntimeMessage" /* codePath */, message);
                 this.closeFn(error);
@@ -1851,7 +1891,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 // Some other client turned on the id compressor. If we have not turned it on,
                 // put it in a pending queue and delay finalization.
                 if (this._idCompressor === undefined) {
-                    (0, internal_2.assert)(this.idCompressorMode !== undefined, 0x93c /* id compressor should be enabled */);
+                    (0, internal_2.assert)(this.sessionSchema.idCompressorMode !== undefined, 0x93c /* id compressor should be enabled */);
                     this.pendingIdCompressorOps.push(range);
                 }
                 else {
@@ -1885,7 +1925,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
      * Updates signal telemetry including emitting telemetry events.
      */
     processSignalForTelemetry(envelope) {
-        const { clientBroadcastSignalSequenceNumber } = envelope;
+        const { clientBroadcastSignalSequenceNumber, contents: envelopeContents, address: envelopeAddress, } = envelope;
         if (clientBroadcastSignalSequenceNumber === undefined) {
             return;
         }
@@ -1923,8 +1963,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             };
             // Only log `contents.type` when address is for container to avoid
             // chance that contents type is customer data.
-            if (envelope.address === undefined) {
-                details.contentsType = envelope.contents.type; // Type of signal that was received out of order.
+            if (envelopeAddress === undefined) {
+                details.contentsType = envelopeContents.type; // Type of signal that was received out of order.
             }
             this.mc.logger.sendTelemetryEvent({
                 eventName: "SignalOutOfOrder",
@@ -2005,8 +2045,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 try {
                     checkpoint.rollback((message) => this.rollback(message.contents, message.localOpMetadata));
                 }
-                catch (err) {
-                    const error2 = (0, internal_7.wrapError)(err, (message) => {
+                catch (error_) {
+                    const error2 = (0, internal_7.wrapError)(error_, (message) => {
                         return internal_7.DataProcessingError.create(`RollbackError: ${message}`, "checkpointRollback", undefined);
                     });
                     this.closeFn(error2);
@@ -2117,8 +2157,9 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             case messageTypes_js_1.ContainerMessageType.GC: {
                 return false;
             }
-            default:
+            default: {
                 break;
+            }
         }
         return true;
     }
@@ -2147,7 +2188,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     clientBroadcastSignalSequenceNumber;
             }
             // We should not track the round trip of a new signal in the case we are already tracking one.
-            if (clientBroadcastSignalSequenceNumber % this.defaultTelemetrySignalSampleCount === 1 &&
+            if (clientBroadcastSignalSequenceNumber % defaultTelemetrySignalSampleCount === 1 &&
                 this._signalTracking.roundTripSignalSequenceNumber === undefined) {
                 this._signalTracking.signalTimestamp = Date.now();
                 this._signalTracking.roundTripSignalSequenceNumber =
@@ -2213,6 +2254,12 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         this.addContainerStateToSummary(summarizeResult, true /* fullTree */, false /* trackState */, telemetryContext);
         return summarizeResult.summary;
     }
+    /**
+     * Builds the Summary tree including all the channels and the container state.
+     *
+     * @remarks - Unfortunately, this function is accessed in a non-typesafe way by a legacy first-party partner,
+     * so until we can provide a proper API for their scenario, we need to ensure this function doesn't change.
+     */
     async summarizeInternal(fullTree, trackState, telemetryContext) {
         const summarizeResult = await this.channelCollection.summarize(fullTree, trackState, telemetryContext);
         // Wrap data store summaries in .channels subtree.
@@ -2293,8 +2340,10 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
      */
     deleteSweepReadyNodes(sweepReadyRoutes) {
         const { dataStoreRoutes, blobManagerRoutes } = this.getDataStoreAndBlobManagerRoutes(sweepReadyRoutes);
-        const deletedRoutes = this.channelCollection.deleteSweepReadyNodes(dataStoreRoutes);
-        return deletedRoutes.concat(this.blobManager.deleteSweepReadyNodes(blobManagerRoutes));
+        return [
+            ...this.channelCollection.deleteSweepReadyNodes(dataStoreRoutes),
+            ...this.blobManager.deleteSweepReadyNodes(blobManagerRoutes),
+        ];
     }
     /**
      * This is called to update objects that are tombstones.
@@ -2320,10 +2369,13 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
      * Returns the type of the GC node. Currently, there are nodes that belong to the root ("/"), data stores or
      * blob manager.
      */
+    // eslint-disable-next-line import/no-deprecated
     getNodeType(nodePath) {
         if ((0, index_js_1.isBlobPath)(nodePath)) {
+            // eslint-disable-next-line import/no-deprecated
             return index_js_2.GCNodeType.Blob;
         }
+        // eslint-disable-next-line import/no-deprecated
         return this.channelCollection.getGCNodeType(nodePath) ?? index_js_2.GCNodeType.Other;
     }
     /**
@@ -2337,13 +2389,19 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             return ["_gcRoot"];
         }
         switch (this.getNodeType(nodePath)) {
-            case index_js_2.GCNodeType.Blob:
+            // eslint-disable-next-line import/no-deprecated
+            case index_js_2.GCNodeType.Blob: {
                 return [index_js_1.blobManagerBasePath];
+            }
+            // eslint-disable-next-line import/no-deprecated
             case index_js_2.GCNodeType.DataStore:
-            case index_js_2.GCNodeType.SubDataStore:
+            // eslint-disable-next-line import/no-deprecated
+            case index_js_2.GCNodeType.SubDataStore: {
                 return this.channelCollection.getDataStorePackagePath(nodePath);
-            default:
+            }
+            default: {
                 (0, internal_2.assert)(false, 0x2de /* "Package path requested for unsupported node type." */);
+            }
         }
     }
     /**
@@ -2405,8 +2463,9 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
      * op processing, updating SummarizerNode state tracking, and garbage collection.
      * @param options - options controlling how the summary is generated or submitted
      */
+    // eslint-disable-next-line import/no-deprecated
     async submitSummary(options) {
-        const { fullTree = false, finalAttempt = false, summaryLogger, latestSummaryRefSeqNum, } = options;
+        const { cancellationToken, fullTree = false, finalAttempt = false, summaryLogger, latestSummaryRefSeqNum, } = options;
         // The summary number for this summary. This will be updated during the summary process, so get it now and
         // use it for all events logged during this summary.
         const summaryNumber = this.nextSummaryNumber;
@@ -2490,7 +2549,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     eventName: "LatestSummaryRefSeqNumMismatch",
                     details: {
                         ...startSummaryResult,
-                        mismatchNumbers: Array.from(startSummaryResult.mismatchNumbers),
+                        mismatchNumbers: [...startSummaryResult.mismatchNumbers],
                     },
                 });
                 if (shouldValidatePreSummaryState && !finalAttempt) {
@@ -2510,7 +2569,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 // summarizer to reconnect in the future.
                 // Also checking for cancellation is a must as summary process may be abandoned for other reasons,
                 // like loss of connectivity for main (interactive) client.
-                if (options.cancellationToken.cancelled) {
+                if (cancellationToken.cancelled) {
                     return { continue: false, error: "disconnected" };
                 }
                 // That said, we rely on submitSystemMessage() that today only works in connected state.
@@ -2623,7 +2682,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             };
             let handle;
             try {
-                handle = await this.storage.uploadSummaryWithContext(summarizeResult.summary, summaryContext);
+                handle = await this.storage.uploadSummaryWithContext(summaryTree, summaryContext);
             }
             catch (error) {
                 return {
@@ -2745,13 +2804,13 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         return this.pendingMessagesCount !== 0;
     }
     updateDocumentDirtyState(dirty) {
-        if (this.attachState !== container_definitions_1.AttachState.Attached) {
-            (0, internal_2.assert)(dirty, 0x3d2 /* Non-attached container is dirty */);
-        }
-        else {
+        if (this.attachState === container_definitions_1.AttachState.Attached) {
             // Other way is not true = see this.isContainerMessageDirtyable()
             (0, internal_2.assert)(!dirty || this.hasPendingMessages(), 0x3d3 /* if doc is dirty, there has to be pending ops */);
         }
+        else {
+            (0, internal_2.assert)(dirty, 0x3d2 /* Non-attached container is dirty */);
+        }
         if (this.dirtyContainer === dirty) {
             return;
         }
@@ -2813,7 +2872,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             // on this callback to do actual sending.
             const schemaChangeMessage = this.documentsSchemaController.maybeSendSchemaMessage();
             if (schemaChangeMessage) {
-                this.logger.sendTelemetryEvent({
+                this.mc.logger.sendTelemetryEvent({
                     eventName: "SchemaChangeProposal",
                     refSeq: schemaChangeMessage.refSeq,
                     version: schemaChangeMessage.version,
@@ -2865,6 +2924,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             return;
         }
         this.flushTaskExists = true;
+        // TODO: hoist this out of the function scope to save unnecessary allocations
+        // eslint-disable-next-line unicorn/consistent-function-scoping -- Separate `flush` method already exists in outer scope
         const flush = () => {
             this.flushTaskExists = false;
             try {
@@ -2875,22 +2936,25 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             }
         };
         switch (this.flushMode) {
-            case internal_5.FlushMode.TurnBased:
+            case internal_5.FlushMode.TurnBased: {
                 // When in TurnBased flush mode the runtime will buffer operations in the current turn and send them as a single
                 // batch at the end of the turn
                 // eslint-disable-next-line @typescript-eslint/no-floating-promises
                 Promise.resolve().then(flush);
                 break;
+            }
             // FlushModeExperimental is experimental and not exposed directly in the runtime APIs
-            case internal_5.FlushModeExperimental.Async:
+            case internal_5.FlushModeExperimental.Async: {
                 // When in Async flush mode, the runtime will accumulate all operations across JS turns and send them as a single
                 // batch when all micro-tasks are complete.
                 // Compared to TurnBased, this flush mode will capture more ops into the same batch.
                 setTimeout(flush, 0);
                 break;
-            default:
+            }
+            default: {
                 (0, internal_2.assert)(this._orderSequentiallyCalls > 0, 0x587 /* Unreachable unless running under orderSequentially */);
                 break;
+            }
         }
     }
     submitSummaryMessage(contents, referenceSequenceNumber) {
@@ -2898,10 +2962,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         (0, internal_2.assert)(this.connected, 0x133 /* "Container disconnected when trying to submit system message" */);
         // System message should not be sent in the middle of the batch.
         (0, internal_2.assert)(this.outbox.isEmpty, 0x3d4 /* System op in the middle of a batch */);
-        // back-compat: ADO #1385: Make this call unconditional in the future
-        return this.submitSummaryFn !== undefined
-            ? this.submitSummaryFn(contents, referenceSequenceNumber)
-            : this.submitFn(internal_3.MessageType.Summarize, contents, false);
+        return this.submitSummaryFn(contents, referenceSequenceNumber);
     }
     /**
      * Throw an error if the runtime is closed.  Methods that are expected to potentially
@@ -2941,15 +3002,16 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
      * @param localOpMetadata - The local metadata associated with the original message.
      */
     reSubmitCore(message, localOpMetadata, opMetadata) {
-        (0, internal_2.assert)(!this.isSummarizerClient, 0x8f2 /* Summarizer never reconnects so should never resubmit */);
+        (0, internal_2.assert)(this._summarizer === undefined, 0x8f2 /* Summarizer never reconnects so should never resubmit */);
         switch (message.type) {
             case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp:
             case messageTypes_js_1.ContainerMessageType.Attach:
-            case messageTypes_js_1.ContainerMessageType.Alias:
+            case messageTypes_js_1.ContainerMessageType.Alias: {
                 // For Operations, call resubmitDataStoreOp which will find the right store
                 // and trigger resubmission on it.
                 this.channelCollection.reSubmit(message.type, message.contents, localOpMetadata);
                 break;
+            }
             case messageTypes_js_1.ContainerMessageType.IdAllocation: {
                 // Allocation ops are never resubmitted/rebased. This is because they require special handling to
                 // avoid being submitted out of order. For example, if the pending state manager contained
@@ -2960,20 +3022,24 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 // all pending IDs. The resubmitted allocation ops are then ignored here.
                 break;
             }
-            case messageTypes_js_1.ContainerMessageType.BlobAttach:
+            case messageTypes_js_1.ContainerMessageType.BlobAttach: {
                 this.blobManager.reSubmit(opMetadata);
                 break;
-            case messageTypes_js_1.ContainerMessageType.Rejoin:
+            }
+            case messageTypes_js_1.ContainerMessageType.Rejoin: {
                 this.submit(message);
                 break;
-            case messageTypes_js_1.ContainerMessageType.GC:
+            }
+            case messageTypes_js_1.ContainerMessageType.GC: {
                 this.submit(message);
                 break;
-            case messageTypes_js_1.ContainerMessageType.DocumentSchemaChange:
+            }
+            case messageTypes_js_1.ContainerMessageType.DocumentSchemaChange: {
                 // There is no need to resend this message. Document schema controller will properly resend it again (if needed)
                 // on a first occasion (any ops sent after reconnect). There is a good chance, though, that it will not want to
                 // send any ops, as some other client already changed schema.
                 break;
+            }
             default: {
                 const error = getUnknownMessageTypeError(message.type, "reSubmitCore" /* codePath */);
                 this.closeFn(error);
@@ -2985,18 +3051,21 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // Need to parse from string for back-compat
         const { type, contents } = this.parseLocalOpContent(content);
         switch (type) {
-            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp:
+            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp: {
                 // For operations, call rollbackDataStoreOp which will find the right store
                 // and trigger rollback on it.
                 this.channelCollection.rollback(type, contents, localOpMetadata);
                 break;
-            default:
+            }
+            default: {
                 throw new Error(`Can't rollback ${type}`);
+            }
         }
     }
     /**
      * Implementation of ISummarizerInternalsProvider.refreshLatestSummaryAck
      */
+    // eslint-disable-next-line import/no-deprecated
     async refreshLatestSummaryAck(options) {
         const { proposalHandle, ackHandle, summaryRefSeq, summaryLogger } = options;
         // proposalHandle is always passed from RunningSummarizer.
@@ -3066,15 +3135,16 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 snapshotTree = snapshot.snapshotTree;
             }
             else {
-                const versions = await this.storage.getVersions(null, 1, scenarioName, internal_3.FetchSource.noCache);
+                const versions = await this.storage.getVersions(
+                // eslint-disable-next-line unicorn/no-null
+                null, 1, scenarioName, internal_3.FetchSource.noCache);
                 (0, internal_2.assert)(!!versions && !!versions[0], 0x137 /* "Failed to get version from storage" */);
                 snapshotTree = await this.storage.getSnapshotTree(versions[0]);
                 (0, internal_2.assert)(!!snapshotTree, 0x138 /* "Failed to get snapshot from storage" */);
                 props.snapshotVersion = versions[0].id;
             }
             props.getSnapshotDuration = trace.trace().duration;
-            const readAndParseBlob = async (id) => (0, internal_4.readAndParse)(this.storage, id);
-            const snapshotRefSeq = await (0, internal_6.seqFromTree)(snapshotTree, readAndParseBlob);
+            const snapshotRefSeq = await (0, internal_6.seqFromTree)(snapshotTree, this.readAndParseBlob);
             props.snapshotRefSeq = snapshotRefSeq;
             props.newerSnapshotPresent = snapshotRefSeq >= targetRefSeq;
             perfEvent.end({ details: props });
@@ -3127,32 +3197,32 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             : internal_7.PerformanceEvent.timedExec(this.mc.logger, perfEvent, (event) => logAndReturnPendingState(event, getSyncState()));
     }
     summarizeOnDemand(options) {
-        if (this.isSummarizerClient) {
-            return this.summarizer.summarizeOnDemand(options);
-        }
-        else if (this.summaryManager !== undefined) {
-            return this.summaryManager.summarizeOnDemand(options);
+        if (this._summarizer !== undefined) {
+            return this._summarizer.summarizeOnDemand(options);
         }
-        else {
+        else if (this.summaryManager === undefined) {
             // If we're not the summarizer, and we don't have a summaryManager, we expect that
             // disableSummaries is turned on. We are throwing instead of returning a failure here,
             // because it is a misuse of the API rather than an expected failure.
             throw new internal_7.UsageError(`Can't summarize, disableSummaries: ${this.summariesDisabled}`);
         }
+        else {
+            return this.summaryManager.summarizeOnDemand(options);
+        }
     }
     enqueueSummarize(options) {
-        if (this.isSummarizerClient) {
-            return this.summarizer.enqueueSummarize(options);
+        if (this._summarizer !== undefined) {
+            return this._summarizer.enqueueSummarize(options);
         }
-        else if (this.summaryManager !== undefined) {
-            return this.summaryManager.enqueueSummarize(options);
-        }
-        else {
+        else if (this.summaryManager === undefined) {
             // If we're not the summarizer, and we don't have a summaryManager, we expect that
             // generateSummaries is turned off. We are throwing instead of returning a failure here,
             // because it is a misuse of the API rather than an expected failure.
             throw new internal_7.UsageError(`Can't summarize, disableSummaries: ${this.summariesDisabled}`);
         }
+        else {
+            return this.summaryManager.enqueueSummarize(options);
+        }
     }
     /**
      * Forms a function that will create and retrieve a Summarizer.