@fluidframework/container-runtime 2.23.0-325054 → 2.30.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 = exports.isSummariesDisabled = void 0;
+exports.createNewSignalEnvelope = 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 = 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");
@@ -32,6 +32,7 @@ const messageTypes_js_1 = require("./messageTypes.js");
 const index_js_3 = require("./opLifecycle/index.js");
 const packageVersion_js_1 = require("./packageVersion.js");
 const pendingStateManager_js_1 = require("./pendingStateManager.js");
+const signalTelemetryProcessing_js_1 = require("./signalTelemetryProcessing.js");
 const index_js_4 = require("./summary/index.js");
 const throttler_js_1 = require("./throttler.js");
 /**
@@ -53,33 +54,9 @@ function getUnknownMessageTypeError(unknownContainerRuntimeMessageType, codePath
         },
     });
 }
-function isSummariesDisabled(config) {
-    return config.state === "disabled";
-}
-exports.isSummariesDisabled = isSummariesDisabled;
-/**
- * @legacy
- * @alpha
- */
-exports.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
  */
 exports.DeletedResponseHeaderKey = "wasDeleted";
 /**
@@ -198,55 +175,13 @@ const makeLegacySendBatchFn = (submitFn, deltaManager) => (batch) => {
     return clientSequenceNumber;
 };
 exports.makeLegacySendBatchFn = makeLegacySendBatchFn;
-const summarizerRequestUrl = "_summarizer";
-/**
- * Create and retrieve the summmarizer
- */
-async function createSummarizer(loader, url) {
-    const request = {
-        headers: {
-            [internal_1.LoaderHeader.cache]: false,
-            [internal_1.LoaderHeader.clientDetails]: {
-                capabilities: { interactive: false },
-                type: index_js_4.summarizerClientType,
-            },
-            [internal_3.DriverHeader.summarizingClient]: true,
-            [internal_1.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 (0, internal_6.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 internal_7.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;
@@ -280,7 +215,6 @@ async function loadContainerRuntime(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.
@@ -336,15 +270,13 @@ 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),
         ]);
         // read snapshot blobs needed for BlobManager to load
-        const blobManagerSnapshot = await (0, index_js_1.loadBlobManagerLoadInfo)(context);
+        const blobManagerLoadInfo = await (0, index_js_1.loadBlobManagerLoadInfo)(context);
         const messageAtLastSummary = lastMessageFromMetadata(metadata);
         // Verify summary runtime sequence number matches protocol sequence number.
         const runtimeSequenceNumber = messageAtLastSummary?.sequenceNumber;
@@ -448,7 +380,6 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         };
         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,
@@ -476,7 +407,7 @@ class ContainerRuntime extends client_utils_1.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
@@ -605,23 +536,15 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         return layerCompatState_js_1.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
-        ...exports.DefaultSummaryConfiguration,
+        ...index_js_4.DefaultSummaryConfiguration,
         // the runtime configuration overrides
-        ...baseRuntimeOptions.summaryOptions?.summaryConfigOverrides,
+        ...runtimeOptions.summaryOptions?.summaryConfigOverrides,
     }, recentBatchInfo) {
         super();
         this.registry = registry;
@@ -642,17 +565,7 @@ class ContainerRuntime extends client_utils_1.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 signalTelemetryProcessing_js_1.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.
@@ -666,11 +579,6 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         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
-        const runtimeOptions = {
-            flushMode: defaultFlushMode,
-            ...baseRuntimeOptions,
-        };
         this.mc = (0, internal_7.createChildMonitoringContext)({
             logger: this.baseLogger,
             namespace: "ContainerRuntime",
@@ -782,10 +690,10 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         this._deltaManager = outerDeltaManager;
         this.handleContext = new containerHandleContext_js_1.ContainerFluidHandleContext("", this);
         if (summaryConfiguration.state === "enabled") {
-            this.validateSummaryHeuristicConfiguration(summaryConfiguration);
+            (0, index_js_4.validateSummaryHeuristicConfiguration)(summaryConfiguration);
         }
-        this.summariesDisabled = isSummariesDisabled(summaryConfiguration);
-        const { maxOpsSinceLastSummary = 0, initialSummarizerDelayMs = 0 } = isSummariesDisabled(summaryConfiguration)
+        this.summariesDisabled = (0, index_js_4.isSummariesDisabled)(summaryConfiguration);
+        const { maxOpsSinceLastSummary = 0, initialSummarizerDelayMs = 0 } = (0, index_js_4.isSummariesDisabled)(summaryConfiguration)
             ? {}
             : {
                 ...summaryConfiguration,
@@ -880,9 +788,14 @@ class ContainerRuntime extends client_utils_1.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 = (0, channelCollection_js_1.getSummaryForDatastores)(baseSnapshot, metadata);
         if (snapshot !== undefined && snapshotWithContents !== undefined) {
@@ -897,8 +810,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         }), (path) => this.garbageCollector.isNodeDeleted(path), new Map(dataStoreAliasMap), async (runtime) => provideEntryPoint);
         this.blobManager = new index_js_1.BlobManager({
             routeContext: this.handleContext,
-            snapshot: blobManagerSnapshot,
-            getStorage: () => this.storage,
+            blobManagerLoadInfo,
+            storage: this.storage,
             sendBlobAttachOp: (localId, blobId) => {
                 if (!this.disposed) {
                     this.submit({ type: messageTypes_js_1.ContainerMessageType.BlobAttach, contents: undefined }, undefined, {
@@ -986,10 +899,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             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, 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, 
+                this._summarizer = new index_js_4.Summarizer(this /* ISummarizerRuntime */, () => summaryConfiguration, this /* ISummarizerInternalsProvider */, this.handleContext, summaryCollection, 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));
@@ -1017,7 +927,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 summaryCollection.on("default", defaultAction);
                 // Create the SummaryManager and mark the initial state
                 this.summaryManager = new index_js_4.SummaryManager(this.summarizerClientElection, this, // IConnectedState
-                summaryCollection, this.baseLogger, this.formCreateSummarizerFn(loader), new throttler_js_1.Throttler(60 * 1000, // 60 sec delay window
+                summaryCollection, this.baseLogger, (0, index_js_4.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 })), {
@@ -1052,7 +962,7 @@ class ContainerRuntime extends client_utils_1.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),
@@ -1077,7 +987,6 @@ 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.mc.logger.sendTelemetryEvent({
             eventName: "SchemaChangeAccept",
@@ -1254,7 +1163,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         try {
             const parser = internal_6.RequestParser.create(request);
             const id = parser.pathParts[0];
-            if (id === summarizerRequestUrl && parser.pathParts.length === 1) {
+            if (id === index_js_4.summarizerRequestUrl && parser.pathParts.length === 1) {
                 if (this._summarizer !== undefined) {
                     return {
                         status: 200,
@@ -1288,13 +1197,11 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             }
             if (id === index_js_1.blobManagerBasePath && requestParser.isLeaf(2)) {
                 const blob = await this.blobManager.getBlob(requestParser.pathParts[1]);
-                return blob
-                    ? {
-                        status: 200,
-                        mimeType: "fluid/object",
-                        value: blob,
-                    }
-                    : (0, internal_6.create404Response)(request);
+                return {
+                    status: 200,
+                    mimeType: "fluid/object",
+                    value: blob,
+                };
             }
             else if (requestParser.pathParts.length > 0) {
                 return await this.channelCollection.request(request);
@@ -1325,7 +1232,6 @@ 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.
@@ -1339,8 +1245,7 @@ class ContainerRuntime extends client_utils_1.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,
@@ -1562,14 +1467,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         if (connected) {
             (0, internal_2.assert)(this.attachState === container_definitions_1.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
@@ -1868,9 +1766,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 break;
             }
             case messageTypes_js_1.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: {
@@ -1900,90 +1796,6 @@ class ContainerRuntime extends client_utils_1.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 = {
@@ -1994,7 +1806,7 @@ class ContainerRuntime extends client_utils_1.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.
@@ -2162,43 +1974,6 @@ class ContainerRuntime extends client_utils_1.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.
@@ -2213,8 +1988,11 @@ class ContainerRuntime extends client_utils_1.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 === container_definitions_1.AttachState.Attaching) {
@@ -2368,13 +2146,10 @@ 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;
     }
     /**
@@ -2388,13 +2163,10 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             return ["_gcRoot"];
         }
         switch (this.getNodeType(nodePath)) {
-            // 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:
-            // eslint-disable-next-line import/no-deprecated
             case index_js_2.GCNodeType.SubDataStore: {
                 return this.channelCollection.getDataStorePackagePath(nodePath);
             }
@@ -2462,7 +2234,6 @@ 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 { 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
@@ -3064,7 +2835,6 @@ class ContainerRuntime extends client_utils_1.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.
@@ -3223,28 +2993,17 @@ class ContainerRuntime extends client_utils_1.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 internal_7.UsageError(`Summary heuristic configuration property "${prop}" cannot be less than 0`);
-            }
-        }
-        if (configuration.minIdleTime > configuration.maxIdleTime) {
-            throw new internal_7.UsageError(`"minIdleTime" [${configuration.minIdleTime}] cannot be greater than "maxIdleTime" [${configuration.maxIdleTime}]`);
-        }
-    }
     get groupedBatchingEnabled() {
         return this.sessionSchema.opGroupingEnabled === true;
     }
 }
 exports.ContainerRuntime = ContainerRuntime;
+function createNewSignalEnvelope(address, type, content) {
+    const newEnvelope = {
+        address,
+        contents: { type, content },
+    };
+    return newEnvelope;
+}
+exports.createNewSignalEnvelope = createNewSignalEnvelope;
 //# sourceMappingURL=containerRuntime.js.map