@fluidframework/container-runtime 2.4.0-299707 → 2.5.0-302463

package/dist/containerRuntime.js

@@ -4,7 +4,7 @@
  * Licensed under the MIT License.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ContainerRuntime = exports.loadContainerRuntime = 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.InactiveResponseHeaderKey = exports.TombstoneResponseHeaderKey = exports.DeletedResponseHeaderKey = exports.DefaultSummaryConfiguration = 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");
@@ -90,12 +90,6 @@ exports.TombstoneResponseHeaderKey = "isTombstoned";
  * to this was experimental and is no longer supported.
  */
 exports.InactiveResponseHeaderKey = "isInactive";
-/** Default values for Runtime Headers */
-exports.defaultRuntimeHeaderData = {
-    wait: true,
-    viaHandle: false,
-    allowTombstone: false,
-};
 /**
  * Available compression algorithms for op compression.
  * @legacy
@@ -243,9 +237,10 @@ let getSingleUseLegacyLogCallback = (logger, type) => {
             details: { codePath, type },
         });
         // Now that we've logged, prevent future logging (globally).
-        getSingleUseLegacyLogCallback = () => () => { };
+        exports.getSingleUseLegacyLogCallback = () => () => { };
     };
 };
+exports.getSingleUseLegacyLogCallback = getSingleUseLegacyLogCallback;
 /**
  * This is meant to be used by a {@link @fluidframework/container-definitions#IRuntimeFactory} to instantiate a container runtime.
  * @param params - An object which specifies all required and optional params necessary to instantiate a runtime.
@@ -1523,9 +1518,12 @@ 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 = getSingleUseLegacyLogCallback(this.logger, messageCopy.type);
-        // We expect runtime messages to have JSON contents - deserialize it in place.
-        (0, index_js_3.ensureContentsDeserialized)(messageCopy, hasModernRuntimeMessageEnvelope, logLegacyCase);
+        const logLegacyCase = (0, exports.getSingleUseLegacyLogCallback)(this.logger, messageCopy.type);
+        let runtimeBatch = hasModernRuntimeMessageEnvelope || isUnpackedRuntimeMessage(messageCopy);
+        if (runtimeBatch) {
+            // We expect runtime messages to have JSON contents - deserialize it in place.
+            (0, index_js_3.ensureContentsDeserialized)(messageCopy);
+        }
         if (hasModernRuntimeMessageEnvelope) {
             // If the message has the modern message envelope, then process it here.
             // Here we unpack the message (decompress, unchunk, and/or ungroup) into a batch of messages with ContainerMessageType
@@ -1554,7 +1552,6 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     throw error;
                 }
             }
-            let runtimeBatch = true;
             // Reach out to PendingStateManager, either to zip localOpMetadata into the *local* message list,
             // or to check to ensure the *remote* messages don't match the batchId of a pending local batch.
             // This latter case would indicate that the container has forked - two copies are trying to persist the same local changes.
@@ -1593,8 +1590,17 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             this.processInboundMessages(messagesWithPendingState, locationInBatch, local, savedOp, runtimeBatch);
         }
         else {
+            if (!runtimeBatch) {
+                // The DeltaManager used to do this, but doesn't anymore as of Loader v2.4
+                // Anyone listening to our "op" event would expect the contents to be parsed per this same logic
+                if (typeof messageCopy.contents === "string" &&
+                    messageCopy.contents !== "" &&
+                    messageCopy.type !== internal_3.MessageType.ClientLeave) {
+                    messageCopy.contents = JSON.parse(messageCopy.contents);
+                }
+            }
             this.processInboundMessages([{ message: messageCopy, localOpMetadata: undefined }], { batchStart: true, batchEnd: true }, // Single message
-            local, savedOp, isUnpackedRuntimeMessage(messageCopy) /* runtimeBatch */);
+            local, savedOp, runtimeBatch);
         }
         if (local) {
             // If we have processed a local op, this means that the container is
@@ -1737,11 +1743,13 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         const duration = Date.now() - this._signalTracking.signalTimestamp;
         this.mc.logger.sendPerformanceEvent({
             eventName: "SignalLatency",
-            duration, // Roundtrip duration of the tracked signal in milliseconds.
-            signalsSent: this._signalTracking.totalSignalsSentInLatencyWindow, // Signals sent since the last logged SignalLatency event.
-            signalsLost: this._signalTracking.signalsLost, // Signals lost since the last logged SignalLatency event.
-            outOfOrderSignals: this._signalTracking.signalsOutOfOrder, // Out of order signals since the last logged SignalLatency event.
-            reconnectCount: this.consecutiveReconnects, // Container reconnect count.
+            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;
@@ -1768,9 +1776,11 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 this._signalTracking.signalsLost += signalsLost;
                 this.mc.logger.sendErrorEvent({
                     eventName: "SignalLost",
-                    signalsLost, // Number of lost signals detected.
-                    trackingSequenceNumber: this._signalTracking.trackingSignalSequenceNumber, // The next expected signal sequence number.
-                    clientBroadcastSignalSequenceNumber, // Actual signal sequence number received.
+                    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.
@@ -1782,11 +1792,18 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         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 (envelope.address === undefined) {
+                details.contentsType = envelope.contents.type; // Type of signal that was received out of order.
+            }
             this.mc.logger.sendTelemetryEvent({
                 eventName: "SignalOutOfOrder",
-                type: envelope.contents.type, // Type of signal that was received out of order.
-                trackingSequenceNumber: this._signalTracking.trackingSignalSequenceNumber, // The next expected signal sequence number.
-                clientBroadcastSignalSequenceNumber, // Sequence number of the out of order signal.
+                details,
             });
         }
         if (this._signalTracking.roundTripSignalSequenceNumber !== undefined &&
@@ -1795,7 +1812,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 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 signals sent, signals lost, and out of order signals received.
+                // 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();
             }
@@ -2641,7 +2659,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     contents: idRange,
                 };
                 const idAllocationBatchMessage = {
-                    contents: JSON.stringify(idAllocationMessage),
+                    contents: (0, index_js_3.serializeOpContents)(idAllocationMessage),
                     referenceSequenceNumber: this.deltaManager.lastSequenceNumber,
                 };
                 this.outbox.submitIdAllocation(idAllocationBatchMessage);
@@ -2684,12 +2702,12 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     contents: schemaChangeMessage,
                 };
                 this.outbox.submit({
-                    contents: JSON.stringify(msg),
+                    contents: (0, index_js_3.serializeOpContents)(msg),
                     referenceSequenceNumber: this.deltaManager.lastSequenceNumber,
                 });
             }
             const message = {
-                contents: JSON.stringify(containerRuntimeMessage) /* serialized content */,
+                contents: (0, index_js_3.serializeOpContents)(containerRuntimeMessage),
                 metadata,
                 localOpMetadata,
                 referenceSequenceNumber: this.deltaManager.lastSequenceNumber,