@fluidframework/container-runtime 2.10.0-304831 → 2.10.0-306579

package/lib/containerRuntime.js

@@ -4,7 +4,7 @@
  */
 import { Trace, TypedEventEmitter } from "@fluid-internal/client-utils";
 import { AttachState, } from "@fluidframework/container-definitions";
-import { LoaderHeader, } from "@fluidframework/container-definitions/internal";
+import { LoaderHeader, 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";
@@ -506,6 +506,14 @@ export class ContainerRuntime extends TypedEventEmitter {
     get IFluidHandleContext() {
         return this.handleContext;
     }
+    /**
+     * This is a proxy to the delta manager provided by the container context (innerDeltaManager). It restricts certain
+     * accesses such as sets "read-only" mode for the summarizer client. This is the default delta manager that should
+     * be used unless the innerDeltaManager is required.
+     */
+    get deltaManager() {
+        return this._deltaManager;
+    }
     /**
      * Invokes the given callback and expects that no ops are submitted
      * until execution finishes. If an op is submitted, an error will be raised.
@@ -637,6 +645,7 @@ export class ContainerRuntime extends TypedEventEmitter {
                 : Number.POSITIVE_INFINITY,
             compressionAlgorithm: CompressionAlgorithms.lz4,
         };
+        assert(isIDeltaManagerFull(deltaManager), "Invalid delta manager");
         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.
@@ -731,7 +740,7 @@ export class ContainerRuntime extends TypedEventEmitter {
             const pendingOpsDeltaManagerProxy = new DeltaManagerPendingOpsProxy(summarizerDeltaManagerProxy, this.pendingStateManager);
             outerDeltaManager = pendingOpsDeltaManagerProxy;
         }
-        this.deltaManager = outerDeltaManager;
+        this._deltaManager = outerDeltaManager;
         this.handleContext = new ContainerFluidHandleContext("", this);
         if (this.summaryConfiguration.state === "enabled") {
             this.validateSummaryHeuristicConfiguration(this.summaryConfiguration);
@@ -992,7 +1001,7 @@ export class ContainerRuntime extends TypedEventEmitter {
             groupedBatchingEnabled: this.groupedBatchingEnabled,
             initialSequenceNumber: this.deltaManager.initialSequenceNumber,
         });
-        ReportOpPerfTelemetry(this.clientId, this.deltaManager, this, this.logger);
+        ReportOpPerfTelemetry(this.clientId, this._deltaManager, this, this.logger);
         BindBatchTracker(this, this.logger);
         this.entryPoint = new LazyPromise(async () => {
             if (this.isSummarizerClient) {
@@ -1136,8 +1145,8 @@ export class ContainerRuntime extends TypedEventEmitter {
             });
             // If the inbound deltas queue is paused or disconnected, we expect a reconnect and unpause
             // as long as it's not a summarizer client.
-            if (this.deltaManager.inbound.paused) {
-                props.inboundPaused = this.deltaManager.inbound.paused; // reusing telemetry
+            if (this._deltaManager.inbound.paused) {
+                props.inboundPaused = this._deltaManager.inbound.paused; // reusing telemetry
             }
             const defP = new Deferred();
             this.deltaManager.on("op", (message) => {
@@ -2423,7 +2432,7 @@ export class ContainerRuntime extends TypedEventEmitter {
         const shouldPauseInboundSignal = this.mc.config.getBoolean("Fluid.ContainerRuntime.SubmitSummary.disableInboundSignalPause") !== true;
         const shouldValidatePreSummaryState = this.mc.config.getBoolean("Fluid.ContainerRuntime.SubmitSummary.shouldValidatePreSummaryState") === true;
         try {
-            await this.deltaManager.inbound.pause();
+            await this._deltaManager.inbound.pause();
             if (shouldPauseInboundSignal) {
                 await this.deltaManager.inboundSignal.pause();
             }
@@ -2643,7 +2652,7 @@ export class ContainerRuntime extends TypedEventEmitter {
             // ! This needs to happen before we resume inbound queues to ensure heuristics are tracked correctly
             this._summarizer?.recordSummaryAttempt?.(summaryRefSeqNum);
             // Restart the delta manager
-            this.deltaManager.inbound.resume();
+            this._deltaManager.inbound.resume();
             if (shouldPauseInboundSignal) {
                 this.deltaManager.inboundSignal.resume();
             }