@fluidframework/container-runtime 2.23.0 → 2.30.0

package/lib/containerRuntime.js

@@ -2,15 +2,15 @@
  * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
  * Licensed under the MIT License.
  */
-import { Trace, TypedEventEmitter, } from "@fluid-internal/client-utils";
-import { AttachState, } from "@fluidframework/container-definitions";
-import { LoaderHeader, isIDeltaManagerFull, } from "@fluidframework/container-definitions/internal";
+import { Trace, TypedEventEmitter } from "@fluid-internal/client-utils";
+import { AttachState } from "@fluidframework/container-definitions";
+import { isIDeltaManagerFull } from "@fluidframework/container-definitions/internal";
 import { assert, Deferred, LazyPromise, PromiseCache, delay, } from "@fluidframework/core-utils/internal";
-import { SummaryType, } from "@fluidframework/driver-definitions";
-import { DriverHeader, FetchSource, MessageType, } from "@fluidframework/driver-definitions/internal";
+import { SummaryType } from "@fluidframework/driver-definitions";
+import { FetchSource, MessageType } from "@fluidframework/driver-definitions/internal";
 import { readAndParse } from "@fluidframework/driver-utils/internal";
 import { FlushMode, FlushModeExperimental, channelsTreeName, gcTreeKey, } from "@fluidframework/runtime-definitions/internal";
-import { GCDataBuilder, RequestParser, TelemetryContext, addBlobToSummary, addSummarizeResultToSummary, calculateStats, create404Response, exceptionToResponse, responseToException, seqFromTree, } from "@fluidframework/runtime-utils/internal";
+import { GCDataBuilder, RequestParser, TelemetryContext, addBlobToSummary, addSummarizeResultToSummary, calculateStats, create404Response, exceptionToResponse, seqFromTree, } from "@fluidframework/runtime-utils/internal";
 import { DataCorruptionError, DataProcessingError, extractSafePropertiesFromMessage, GenericError, LoggingError, PerformanceEvent, 
 // eslint-disable-next-line import/no-deprecated
 TaggedLoggerAdapter, UsageError, createChildLogger, createChildMonitoringContext, createSampledLogger, loggerToMonitoringContext, raiseConnectedEvent, wrapError, tagCodeArtifacts, } from "@fluidframework/telemetry-utils/internal";
@@ -24,20 +24,15 @@ import { channelToDataStore } from "./dataStore.js";
 import { FluidDataStoreRegistry } from "./dataStoreRegistry.js";
 import { DeltaManagerPendingOpsProxy, DeltaManagerSummarizerProxy, } from "./deltaManagerProxies.js";
 import { DeltaScheduler } from "./deltaScheduler.js";
-import { 
-// eslint-disable-next-line import/no-deprecated
-GCNodeType, GarbageCollector, gcGenerationOptionName, } from "./gc/index.js";
+import { GCNodeType, GarbageCollector, gcGenerationOptionName, } from "./gc/index.js";
 import { InboundBatchAggregator } from "./inboundBatchAggregator.js";
 import { RuntimeCompatDetails, validateLoaderCompatibility } from "./layerCompatState.js";
 import { ContainerMessageType, } from "./messageTypes.js";
 import { DuplicateBatchDetector, ensureContentsDeserialized, OpCompressor, OpDecompressor, OpGroupingManager, OpSplitter, Outbox, RemoteMessageProcessor, serializeOpContents, } from "./opLifecycle/index.js";
 import { pkgVersion } from "./packageVersion.js";
 import { PendingStateManager, } from "./pendingStateManager.js";
-import { 
-// eslint-disable-next-line import/no-deprecated
-DocumentsSchemaController, OrderedClientCollection, OrderedClientElection, RetriableSummaryError, RunWhileConnectedCoordinator, 
-// eslint-disable-next-line import/no-deprecated
-Summarizer, SummarizerClientElection, SummaryCollection, SummaryManager, aliasBlobName, chunksBlobName, recentBatchInfoBlobName, createRootSummarizerNodeWithGC, electedSummarizerBlobName, extractSummaryMetadataMessage, idCompressorBlobName, metadataBlobName, rootHasIsolatedChannels, summarizerClientType, wrapSummaryInChannelsTree, } from "./summary/index.js";
+import { SignalTelemetryManager } from "./signalTelemetryProcessing.js";
+import { DocumentsSchemaController, OrderedClientCollection, OrderedClientElection, RetriableSummaryError, RunWhileConnectedCoordinator, Summarizer, SummarizerClientElection, SummaryCollection, SummaryManager, aliasBlobName, chunksBlobName, recentBatchInfoBlobName, createRootSummarizerNodeWithGC, electedSummarizerBlobName, extractSummaryMetadataMessage, idCompressorBlobName, metadataBlobName, rootHasIsolatedChannels, summarizerClientType, wrapSummaryInChannelsTree, formCreateSummarizerFn, summarizerRequestUrl, validateSummaryHeuristicConfiguration, DefaultSummaryConfiguration, isSummariesDisabled, } from "./summary/index.js";
 import { Throttler, formExponentialFn } from "./throttler.js";
 /**
  * Creates an error object to be thrown / passed to Container's close fn in case of an unknown message type.
@@ -58,32 +53,9 @@ function getUnknownMessageTypeError(unknownContainerRuntimeMessageType, codePath
         },
     });
 }
-export function isSummariesDisabled(config) {
-    return config.state === "disabled";
-}
-/**
- * @legacy
- * @alpha
- */
-export const DefaultSummaryConfiguration = {
-    state: "enabled",
-    minIdleTime: 0,
-    maxIdleTime: 30 * 1000, // 30 secs.
-    maxTime: 60 * 1000, // 1 min.
-    maxOps: 100, // Summarize if 100 weighted ops received since last snapshot.
-    minOpsForLastSummaryAttempt: 10,
-    maxAckWaitTime: 3 * 60 * 1000, // 3 mins.
-    maxOpsSinceLastSummary: 7000,
-    initialSummarizerDelayMs: 5 * 1000, // 5 secs.
-    nonRuntimeOpWeight: 0.1,
-    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.
+ * @internal
  */
 export const DeletedResponseHeaderKey = "wasDeleted";
 /**
@@ -199,55 +171,13 @@ export const makeLegacySendBatchFn = (submitFn, deltaManager) => (batch) => {
     deltaManager.flush();
     return clientSequenceNumber;
 };
-const summarizerRequestUrl = "_summarizer";
-/**
- * Create and retrieve the summmarizer
- */
-async function createSummarizer(loader, url) {
-    const request = {
-        headers: {
-            [LoaderHeader.cache]: false,
-            [LoaderHeader.clientDetails]: {
-                capabilities: { interactive: false },
-                type: summarizerClientType,
-            },
-            [DriverHeader.summarizingClient]: true,
-            [LoaderHeader.reconnect]: false,
-        },
-        url,
-    };
-    const resolvedContainer = await loader.resolve(request);
-    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) {
-        // 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}`,
-        }));
-        if (response.status !== 200 || response.mimeType !== "fluid/object") {
-            throw responseToException(response, request);
-        }
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-        fluidObject = response.value;
-    }
-    else {
-        fluidObject = await resolvedContainer.getEntryPoint();
-    }
-    if (fluidObject?.ISummarizer === undefined) {
-        throw new UsageError("Fluid object does not implement ISummarizer");
-    }
-    return fluidObject.ISummarizer;
-}
 /**
  * Extract last message from the snapshot metadata.
  * Uses legacy property if not using explicit schema control, otherwise uses the new property.
  * 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(
-// eslint-disable-next-line import/no-deprecated
-metadata) {
+function lastMessageFromMetadata(metadata) {
     return metadata?.documentSchema?.runtime?.explicitSchemaControl
         ? metadata?.lastMessage
         : metadata?.message;
@@ -279,7 +209,6 @@ export async function loadContainerRuntime(params) {
     return ContainerRuntime.loadRuntime(params);
 }
 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.
@@ -335,15 +264,13 @@ export class ContainerRuntime extends TypedEventEmitter {
         const [chunks, recentBatchInfo, metadata, electedSummarizerData, aliases, serializedIdCompressor,] = await Promise.all([
             tryFetchBlob(chunksBlobName),
             tryFetchBlob(recentBatchInfoBlobName),
-            // eslint-disable-next-line import/no-deprecated
             tryFetchBlob(metadataBlobName),
-            // eslint-disable-next-line import/no-deprecated
             tryFetchBlob(electedSummarizerBlobName),
             tryFetchBlob(aliasBlobName),
             tryFetchBlob(idCompressorBlobName),
         ]);
         // read snapshot blobs needed for BlobManager to load
-        const blobManagerSnapshot = await loadBlobManagerLoadInfo(context);
+        const blobManagerLoadInfo = await loadBlobManagerLoadInfo(context);
         const messageAtLastSummary = lastMessageFromMetadata(metadata);
         // Verify summary runtime sequence number matches protocol sequence number.
         const runtimeSequenceNumber = messageAtLastSummary?.sequenceNumber;
@@ -447,7 +374,6 @@ export class ContainerRuntime extends TypedEventEmitter {
         };
         const compressionLz4 = compressionOptions.minimumBatchSizeInBytes !== Number.POSITIVE_INFINITY &&
             compressionOptions.compressionAlgorithm === "lz4";
-        // eslint-disable-next-line import/no-deprecated
         const documentSchemaController = new DocumentsSchemaController(existing, protocolSequenceNumber, metadata?.documentSchema, {
             explicitSchemaControl,
             compressionLz4,
@@ -475,7 +401,7 @@ export class ContainerRuntime extends TypedEventEmitter {
             enableGroupedBatching,
             explicitSchemaControl,
         };
-        const runtime = new containerRuntimeCtor(context, registry, metadata, electedSummarizerData, chunks ?? [], aliases ?? [], internalRuntimeOptions, containerScope, logger, existing, blobManagerSnapshot, context.storage, createIdCompressorFn, documentSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, undefined, // summaryConfiguration
+        const runtime = new containerRuntimeCtor(context, registry, metadata, electedSummarizerData, chunks ?? [], aliases ?? [], internalRuntimeOptions, containerScope, logger, existing, blobManagerLoadInfo, context.storage, createIdCompressorFn, documentSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, undefined, // summaryConfiguration
         recentBatchInfo);
         runtime.blobManager.stashedBlobsUploadP.then(() => {
             // make sure we didn't reconnect before the promise resolved
@@ -604,23 +530,15 @@ export class ContainerRuntime extends TypedEventEmitter {
         return RuntimeCompatDetails;
     }
     /***/
-    constructor(context, registry, 
-    // eslint-disable-next-line import/no-deprecated
-    metadata, 
-    // eslint-disable-next-line import/no-deprecated
-    electedSummarizerData, chunks, dataStoreAliasMap, baseRuntimeOptions, containerScope, 
+    constructor(context, registry, metadata, electedSummarizerData, chunks, dataStoreAliasMap, runtimeOptions, containerScope, 
     // Create a custom ITelemetryBaseLogger to output telemetry events.
-    baseLogger, existing, 
-    // eslint-disable-next-line import/no-deprecated
-    blobManagerSnapshot, _storage, createIdCompressor, 
-    // eslint-disable-next-line import/no-deprecated
-    documentsSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, 
+    baseLogger, existing, blobManagerLoadInfo, _storage, createIdCompressor, documentsSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, 
     // eslint-disable-next-line unicorn/no-object-as-default-parameter
     summaryConfiguration = {
         // the defaults
         ...DefaultSummaryConfiguration,
         // the runtime configuration overrides
-        ...baseRuntimeOptions.summaryOptions?.summaryConfigOverrides,
+        ...runtimeOptions.summaryOptions?.summaryConfigOverrides,
     }, recentBatchInfo) {
         super();
         this.registry = registry;
@@ -641,17 +559,7 @@ export class ContainerRuntime extends TypedEventEmitter {
         this.ensureNoDataModelChangesCalls = 0;
         this._disposed = false;
         this.emitDirtyDocumentEvent = true;
-        this._signalTracking = {
-            totalSignalsSentInLatencyWindow: 0,
-            signalsLost: 0,
-            signalsOutOfOrder: 0,
-            signalsSentSinceLastLatencyMeasurement: 0,
-            broadcastSignalSequenceNumber: 0,
-            signalTimestamp: 0,
-            roundTripSignalSequenceNumber: undefined,
-            trackingSignalSequenceNumber: undefined,
-            minimumTrackingSignalSequenceNumber: undefined,
-        };
+        this.signalTelemetryManager = new SignalTelemetryManager();
         /**
          * It a cache for holding mapping for loading groupIds with its snapshot from the service. Add expiry policy of 1 minute.
          * Starting with 1 min and based on recorded usage we can tweak it later on.
@@ -665,11 +573,6 @@ export class ContainerRuntime extends TypedEventEmitter {
         this.disposeFn = disposeFn ?? closeFn;
         const maybeLoaderCompatDetails = context;
         validateLoaderCompatibility(maybeLoaderCompatDetails.ILayerCompatDetails, this.disposeFn);
-        // Backfill in defaults for the internal runtimeOptions, since they may not be present on the provided runtimeOptions object
-        const runtimeOptions = {
-            flushMode: defaultFlushMode,
-            ...baseRuntimeOptions,
-        };
         this.mc = createChildMonitoringContext({
             logger: this.baseLogger,
             namespace: "ContainerRuntime",
@@ -781,7 +684,7 @@ export class ContainerRuntime extends TypedEventEmitter {
         this._deltaManager = outerDeltaManager;
         this.handleContext = new ContainerFluidHandleContext("", this);
         if (summaryConfiguration.state === "enabled") {
-            this.validateSummaryHeuristicConfiguration(summaryConfiguration);
+            validateSummaryHeuristicConfiguration(summaryConfiguration);
         }
         this.summariesDisabled = isSummariesDisabled(summaryConfiguration);
         const { maxOpsSinceLastSummary = 0, initialSummarizerDelayMs = 0 } = isSummariesDisabled(summaryConfiguration)
@@ -879,9 +782,14 @@ export class ContainerRuntime extends TypedEventEmitter {
         // what is the interface of passing signals, we need the
         // downstream stores to wrap the signal.
         parentContext.submitSignal = (type, content, targetClientId) => {
+            // Future: Can the `content` argument type be IEnvelope?
+            // verifyNotClosed is called in FluidDataStoreContext, which is *the* expected caller.
             const envelope1 = content;
-            const envelope2 = this.createNewSignalEnvelope(envelope1.address, type, envelope1.contents);
-            return this.submitEnvelopedSignal(envelope2, targetClientId);
+            const envelope2 = createNewSignalEnvelope(envelope1.address, type, envelope1.contents);
+            if (targetClientId === undefined) {
+                this.signalTelemetryManager.applyTrackingToBroadcastSignalEnvelope(envelope2);
+            }
+            this.submitSignalFn(envelope2, targetClientId);
         };
         let snapshot = getSummaryForDatastores(baseSnapshot, metadata);
         if (snapshot !== undefined && snapshotWithContents !== undefined) {
@@ -896,8 +804,8 @@ export class ContainerRuntime extends TypedEventEmitter {
         }), (path) => this.garbageCollector.isNodeDeleted(path), new Map(dataStoreAliasMap), async (runtime) => provideEntryPoint);
         this.blobManager = new BlobManager({
             routeContext: this.handleContext,
-            snapshot: blobManagerSnapshot,
-            getStorage: () => this.storage,
+            blobManagerLoadInfo,
+            storage: this.storage,
             sendBlobAttachOp: (localId, blobId) => {
                 if (!this.disposed) {
                     this.submit({ type: ContainerMessageType.BlobAttach, contents: undefined }, undefined, {
@@ -985,10 +893,7 @@ export class ContainerRuntime extends TypedEventEmitter {
             const orderedClientElectionForSummarizer = new OrderedClientElection(orderedClientLogger, orderedClientCollection, electedSummarizerData ?? this.innerDeltaManager.lastSequenceNumber, SummarizerClientElection.isClientEligible, this.mc.config.getBoolean("Fluid.ContainerRuntime.OrderedClientElection.EnablePerformanceEvents"));
             this.summarizerClientElection = new SummarizerClientElection(orderedClientLogger, summaryCollection, orderedClientElectionForSummarizer, maxOpsSinceLastSummary);
             if (isSummarizerClient) {
-                // eslint-disable-next-line import/no-deprecated
-                this._summarizer = new Summarizer(this /* ISummarizerRuntime */, () => summaryConfiguration, this /* ISummarizerInternalsProvider */, this.handleContext, summaryCollection, 
-                // eslint-disable-next-line import/no-deprecated
-                async (runtime) => RunWhileConnectedCoordinator.create(runtime, 
+                this._summarizer = new Summarizer(this /* ISummarizerRuntime */, () => summaryConfiguration, this /* ISummarizerInternalsProvider */, this.handleContext, summaryCollection, async (runtime) => 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));
@@ -1016,7 +921,7 @@ export class ContainerRuntime extends TypedEventEmitter {
                 summaryCollection.on("default", defaultAction);
                 // Create the SummaryManager and mark the initial state
                 this.summaryManager = new SummaryManager(this.summarizerClientElection, this, // IConnectedState
-                summaryCollection, this.baseLogger, this.formCreateSummarizerFn(loader), new Throttler(60 * 1000, // 60 sec delay window
+                summaryCollection, this.baseLogger, formCreateSummarizerFn(loader), new Throttler(60 * 1000, // 60 sec delay window
                 30 * 1000, // 30 sec max delay
                 // throttling function increases exponentially (0ms, 40ms, 80ms, 160ms, etc)
                 formExponentialFn({ coefficient: 20, initialDelay: 0 })), {
@@ -1051,7 +956,7 @@ export class ContainerRuntime extends TypedEventEmitter {
             summaryFormatVersion: metadata?.summaryFormatVersion,
             disableIsolatedChannels: metadata?.disableIsolatedChannels,
             gcVersion: metadata?.gcFeature,
-            options: JSON.stringify(baseRuntimeOptions),
+            options: JSON.stringify(runtimeOptions),
             idCompressorModeMetadata: metadata?.documentSchema?.runtime?.idCompressorMode,
             idCompressorMode: this.sessionSchema.idCompressorMode,
             sessionRuntimeSchema: JSON.stringify(this.sessionSchema),
@@ -1076,7 +981,6 @@ export class ContainerRuntime extends 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.mc.logger.sendTelemetryEvent({
             eventName: "SchemaChangeAccept",
@@ -1287,13 +1191,11 @@ export class ContainerRuntime extends TypedEventEmitter {
             }
             if (id === blobManagerBasePath && requestParser.isLeaf(2)) {
                 const blob = await this.blobManager.getBlob(requestParser.pathParts[1]);
-                return blob
-                    ? {
-                        status: 200,
-                        mimeType: "fluid/object",
-                        value: blob,
-                    }
-                    : create404Response(request);
+                return {
+                    status: 200,
+                    mimeType: "fluid/object",
+                    value: blob,
+                };
             }
             else if (requestParser.pathParts.length > 0) {
                 return await this.channelCollection.request(request);
@@ -1324,7 +1226,6 @@ export class ContainerRuntime extends 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.
@@ -1338,8 +1239,7 @@ export class ContainerRuntime extends TypedEventEmitter {
             // last message's sequence number.
             // See also lastMessageFromMetadata()
             message: explicitSchemaControl
-                ? // eslint-disable-next-line import/no-deprecated
-                    { sequenceNumber: -1 }
+                ? { sequenceNumber: -1 }
                 : message,
             lastMessage: explicitSchemaControl ? message : undefined,
             documentSchema,
@@ -1561,14 +1461,7 @@ export class ContainerRuntime extends TypedEventEmitter {
         if (connected) {
             assert(this.attachState === AttachState.Attached, 0x3cd /* Connection is possible only if container exists in storage */);
             if (changeOfState) {
-                this._signalTracking.signalsLost = 0;
-                this._signalTracking.signalsOutOfOrder = 0;
-                this._signalTracking.signalTimestamp = 0;
-                this._signalTracking.signalsSentSinceLastLatencyMeasurement = 0;
-                this._signalTracking.totalSignalsSentInLatencyWindow = 0;
-                this._signalTracking.roundTripSignalSequenceNumber = undefined;
-                this._signalTracking.trackingSignalSequenceNumber = undefined;
-                this._signalTracking.minimumTrackingSignalSequenceNumber = undefined;
+                this.signalTelemetryManager.resetTracking();
             }
         }
         // Fail while disconnected
@@ -1867,9 +1760,7 @@ export class ContainerRuntime extends TypedEventEmitter {
                 break;
             }
             case ContainerMessageType.DocumentSchemaChange: {
-                this.documentsSchemaController.processDocumentSchemaMessages(
-                // eslint-disable-next-line import/no-deprecated
-                contents, local, message.sequenceNumber);
+                this.documentsSchemaController.processDocumentSchemaMessages(contents, local, message.sequenceNumber);
                 break;
             }
             default: {
@@ -1899,90 +1790,6 @@ export class ContainerRuntime extends TypedEventEmitter {
             }
         }
     }
-    /**
-     * Emits the Signal event and update the perf signal data.
-     */
-    sendSignalTelemetryEvent() {
-        const duration = Date.now() - this._signalTracking.signalTimestamp;
-        this.mc.logger.sendPerformanceEvent({
-            eventName: "SignalLatency",
-            details: {
-                duration, // Roundtrip duration of the tracked signal in milliseconds.
-                sent: this._signalTracking.totalSignalsSentInLatencyWindow, // Signals sent since the last logged SignalLatency event.
-                lost: this._signalTracking.signalsLost, // Signals lost since the last logged SignalLatency event.
-                outOfOrder: this._signalTracking.signalsOutOfOrder, // Out of order signals since the last logged SignalLatency event.
-                reconnectCount: this.consecutiveReconnects, // Container reconnect count.
-            },
-        });
-        this._signalTracking.signalsLost = 0;
-        this._signalTracking.signalsOutOfOrder = 0;
-        this._signalTracking.signalTimestamp = 0;
-        this._signalTracking.totalSignalsSentInLatencyWindow = 0;
-    }
-    /**
-     * Updates signal telemetry including emitting telemetry events.
-     */
-    processSignalForTelemetry(envelope) {
-        const { clientBroadcastSignalSequenceNumber, contents: envelopeContents, address: envelopeAddress, } = envelope;
-        if (clientBroadcastSignalSequenceNumber === undefined) {
-            return;
-        }
-        if (this._signalTracking.trackingSignalSequenceNumber === undefined ||
-            this._signalTracking.minimumTrackingSignalSequenceNumber === undefined) {
-            return;
-        }
-        if (clientBroadcastSignalSequenceNumber >= this._signalTracking.trackingSignalSequenceNumber) {
-            // Calculate the number of signals lost and log the event.
-            const signalsLost = clientBroadcastSignalSequenceNumber -
-                this._signalTracking.trackingSignalSequenceNumber;
-            if (signalsLost > 0) {
-                this._signalTracking.signalsLost += signalsLost;
-                this.mc.logger.sendErrorEvent({
-                    eventName: "SignalLost",
-                    details: {
-                        signalsLost, // Number of lost signals detected.
-                        expectedSequenceNumber: this._signalTracking.trackingSignalSequenceNumber, // The next expected signal sequence number.
-                        clientBroadcastSignalSequenceNumber, // Actual signal sequence number received.
-                    },
-                });
-            }
-            // Update the tracking signal sequence number to the next expected signal in the sequence.
-            this._signalTracking.trackingSignalSequenceNumber =
-                clientBroadcastSignalSequenceNumber + 1;
-        }
-        else if (
-        // Check if this is a signal in range of interest.
-        clientBroadcastSignalSequenceNumber >=
-            this._signalTracking.minimumTrackingSignalSequenceNumber) {
-            this._signalTracking.signalsOutOfOrder++;
-            const details = {
-                expectedSequenceNumber: this._signalTracking.trackingSignalSequenceNumber, // The next expected signal sequence number.
-                clientBroadcastSignalSequenceNumber, // Sequence number of the out of order signal.
-            };
-            // Only log `contents.type` when address is for container to avoid
-            // chance that contents type is customer data.
-            if (envelopeAddress === undefined) {
-                details.contentsType = envelopeContents.type; // Type of signal that was received out of order.
-            }
-            this.mc.logger.sendTelemetryEvent({
-                eventName: "SignalOutOfOrder",
-                details,
-            });
-        }
-        if (this._signalTracking.roundTripSignalSequenceNumber !== undefined &&
-            clientBroadcastSignalSequenceNumber >= this._signalTracking.roundTripSignalSequenceNumber) {
-            if (clientBroadcastSignalSequenceNumber ===
-                this._signalTracking.roundTripSignalSequenceNumber) {
-                // Latency tracked signal has been received.
-                // We now log the roundtrip duration of the tracked signal.
-                // This telemetry event also logs metrics for broadcast signals
-                // sent, lost, and out of order.
-                // These metrics are reset after logging the telemetry event.
-                this.sendSignalTelemetryEvent();
-            }
-            this._signalTracking.roundTripSignalSequenceNumber = undefined;
-        }
-    }
     processSignal(message, local) {
         const envelope = message.content;
         const transformed = {
@@ -1993,7 +1800,7 @@ export class ContainerRuntime extends TypedEventEmitter {
         };
         // Only collect signal telemetry for broadcast messages sent by the current client.
         if (message.clientId === this.clientId) {
-            this.processSignalForTelemetry(envelope);
+            this.signalTelemetryManager.trackReceivedSignal(envelope, this.mc.logger, this.consecutiveReconnects);
         }
         if (envelope.address === undefined) {
             // No address indicates a container signal message.
@@ -2161,43 +1968,6 @@ export class ContainerRuntime extends TypedEventEmitter {
         }
         return true;
     }
-    createNewSignalEnvelope(address, type, content) {
-        const newEnvelope = {
-            address,
-            contents: { type, content },
-        };
-        return newEnvelope;
-    }
-    submitEnvelopedSignal(envelope, targetClientId) {
-        const isBroadcastSignal = targetClientId === undefined;
-        if (isBroadcastSignal) {
-            const clientBroadcastSignalSequenceNumber = ++this._signalTracking
-                .broadcastSignalSequenceNumber;
-            // Stamp with the broadcast signal sequence number.
-            envelope.clientBroadcastSignalSequenceNumber = clientBroadcastSignalSequenceNumber;
-            this._signalTracking.signalsSentSinceLastLatencyMeasurement++;
-            if (this._signalTracking.minimumTrackingSignalSequenceNumber === undefined ||
-                this._signalTracking.trackingSignalSequenceNumber === undefined) {
-                // Signal monitoring window is undefined
-                // Initialize tracking to expect the next signal sent by the connected client.
-                this._signalTracking.minimumTrackingSignalSequenceNumber =
-                    clientBroadcastSignalSequenceNumber;
-                this._signalTracking.trackingSignalSequenceNumber =
-                    clientBroadcastSignalSequenceNumber;
-            }
-            // We should not track the round trip of a new signal in the case we are already tracking one.
-            if (clientBroadcastSignalSequenceNumber % defaultTelemetrySignalSampleCount === 1 &&
-                this._signalTracking.roundTripSignalSequenceNumber === undefined) {
-                this._signalTracking.signalTimestamp = Date.now();
-                this._signalTracking.roundTripSignalSequenceNumber =
-                    clientBroadcastSignalSequenceNumber;
-                this._signalTracking.totalSignalsSentInLatencyWindow +=
-                    this._signalTracking.signalsSentSinceLastLatencyMeasurement;
-                this._signalTracking.signalsSentSinceLastLatencyMeasurement = 0;
-            }
-        }
-        this.submitSignalFn(envelope, targetClientId);
-    }
     /**
      * Submits the signal to be sent to other clients.
      * @param type - Type of the signal.
@@ -2212,8 +1982,11 @@ export class ContainerRuntime extends TypedEventEmitter {
      */
     submitSignal(type, content, targetClientId) {
         this.verifyNotClosed();
-        const envelope = this.createNewSignalEnvelope(undefined /* address */, type, content);
-        return this.submitEnvelopedSignal(envelope, targetClientId);
+        const envelope = createNewSignalEnvelope(undefined /* address */, type, content);
+        if (targetClientId === undefined) {
+            this.signalTelemetryManager.applyTrackingToBroadcastSignalEnvelope(envelope);
+        }
+        this.submitSignalFn(envelope, targetClientId);
     }
     setAttachState(attachState) {
         if (attachState === AttachState.Attaching) {
@@ -2367,13 +2140,10 @@ export class ContainerRuntime extends 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 (isBlobPath(nodePath)) {
-            // eslint-disable-next-line import/no-deprecated
             return GCNodeType.Blob;
         }
-        // eslint-disable-next-line import/no-deprecated
         return this.channelCollection.getGCNodeType(nodePath) ?? GCNodeType.Other;
     }
     /**
@@ -2387,13 +2157,10 @@ export class ContainerRuntime extends TypedEventEmitter {
             return ["_gcRoot"];
         }
         switch (this.getNodeType(nodePath)) {
-            // eslint-disable-next-line import/no-deprecated
             case GCNodeType.Blob: {
                 return [blobManagerBasePath];
             }
-            // eslint-disable-next-line import/no-deprecated
             case GCNodeType.DataStore:
-            // eslint-disable-next-line import/no-deprecated
             case GCNodeType.SubDataStore: {
                 return this.channelCollection.getDataStorePackagePath(nodePath);
             }
@@ -2461,7 +2228,6 @@ export class ContainerRuntime extends 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 { 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
@@ -3063,7 +2829,6 @@ export class ContainerRuntime extends TypedEventEmitter {
     /**
      * 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.
@@ -3222,27 +2987,15 @@ export class ContainerRuntime extends TypedEventEmitter {
             return this.summaryManager.enqueueSummarize(options);
         }
     }
-    /**
-     * Forms a function that will create and retrieve a Summarizer.
-     */
-    formCreateSummarizerFn(loader) {
-        return async () => {
-            return createSummarizer(loader, `/${summarizerRequestUrl}`);
-        };
-    }
-    validateSummaryHeuristicConfiguration(configuration) {
-        // eslint-disable-next-line no-restricted-syntax
-        for (const prop in configuration) {
-            if (typeof configuration[prop] === "number" && configuration[prop] < 0) {
-                throw new UsageError(`Summary heuristic configuration property "${prop}" cannot be less than 0`);
-            }
-        }
-        if (configuration.minIdleTime > configuration.maxIdleTime) {
-            throw new UsageError(`"minIdleTime" [${configuration.minIdleTime}] cannot be greater than "maxIdleTime" [${configuration.maxIdleTime}]`);
-        }
-    }
     get groupedBatchingEnabled() {
         return this.sessionSchema.opGroupingEnabled === true;
     }
 }
+export function createNewSignalEnvelope(address, type, content) {
+    const newEnvelope = {
+        address,
+        contents: { type, content },
+    };
+    return newEnvelope;
+}
 //# sourceMappingURL=containerRuntime.js.map