@fluidframework/container-runtime 2.0.0-dev-rc.5.0.0.263932 → 2.0.0-dev-rc.5.0.0.267932

package/dist/containerRuntime.js

@@ -9,9 +9,9 @@ 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");
 const internal_2 = require("@fluidframework/core-utils/internal");
+const driver_definitions_1 = require("@fluidframework/driver-definitions");
 const internal_3 = require("@fluidframework/driver-definitions/internal");
 const internal_4 = require("@fluidframework/driver-utils/internal");
-const protocol_definitions_1 = require("@fluidframework/protocol-definitions");
 const internal_5 = require("@fluidframework/runtime-definitions/internal");
 const internal_6 = require("@fluidframework/runtime-utils/internal");
 const internal_7 = require("@fluidframework/telemetry-utils/internal");
@@ -50,13 +50,13 @@ function compatBehaviorAllowsMessageType(_unknownContainerRuntimeMessageType, co
 exports.DefaultSummaryConfiguration = {
     state: "enabled",
     minIdleTime: 0,
-    maxIdleTime: 30 * 1000,
-    maxTime: 60 * 1000,
-    maxOps: 100,
+    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,
+    maxAckWaitTime: 3 * 60 * 1000, // 3 mins.
     maxOpsSinceLastSummary: 7000,
-    initialSummarizerDelayMs: 5 * 1000,
+    initialSummarizerDelayMs: 5 * 1000, // 5 secs.
     nonRuntimeOpWeight: 0.1,
     runtimeOpWeight: 1.0,
     nonRuntimeHeuristicThreshold: 20,
@@ -155,7 +155,7 @@ exports.getDeviceSpec = getDeviceSpec;
  */
 const makeLegacySendBatchFn = (submitFn, deltaManager) => (batch) => {
     for (const message of batch.content) {
-        submitFn(protocol_definitions_1.MessageType.Operation, 
+        submitFn(internal_3.MessageType.Operation, 
         // For back-compat (submitFn only works on deserialized content)
         message.contents === undefined ? undefined : JSON.parse(message.contents), true, // batch
         message.metadata);
@@ -532,7 +532,9 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         return this.garbageCollector.throwOnTombstoneUsage;
     }
     /***/
-    constructor(context, registry, metadata, electedSummarizerData, chunks, dataStoreAliasMap, runtimeOptions, containerScope, logger, existing, blobManagerSnapshot, _storage, createIdCompressor, documentsSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, summaryConfiguration = {
+    constructor(context, registry, metadata, electedSummarizerData, chunks, dataStoreAliasMap, runtimeOptions, containerScope, 
+    // Create a custom ITelemetryBaseLogger to output telemetry events.
+    baseLogger, existing, blobManagerSnapshot, _storage, createIdCompressor, documentsSchemaController, featureGatesForTelemetry, provideEntryPoint, requestHandler, summaryConfiguration = {
         // the defaults
         ...exports.DefaultSummaryConfiguration,
         // the runtime configuration overrides
@@ -543,7 +545,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         this.metadata = metadata;
         this.runtimeOptions = runtimeOptions;
         this.containerScope = containerScope;
-        this.logger = logger;
+        this.baseLogger = baseLogger;
         this._storage = _storage;
         this.createIdCompressor = createIdCompressor;
         this.documentsSchemaController = documentsSchemaController;
@@ -575,6 +577,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             expiry: { policy: "absolute", durationMs: 60000 },
         });
         const { options, clientDetails, connected, baseSnapshot, submitFn, submitBatchFn, submitSummaryFn, submitSignalFn, disposeFn, closeFn, deltaManager, quorum, audience, loader, pendingLocalState, supportedFeatures, snapshotWithContents, } = context;
+        this.logger = (0, internal_7.createChildLogger)({ logger: this.baseLogger });
         this.mc = (0, internal_7.createChildMonitoringContext)({
             logger: this.logger,
             namespace: "ContainerRuntime",
@@ -857,7 +860,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 namespace: "OrderedClientElection",
             });
             const orderedClientCollection = new index_js_3.OrderedClientCollection(orderedClientLogger, this.innerDeltaManager, this._quorum);
-            const orderedClientElectionForSummarizer = new index_js_3.OrderedClientElection(orderedClientLogger, orderedClientCollection, electedSummarizerData ?? this.innerDeltaManager.lastSequenceNumber, index_js_3.SummarizerClientElection.isClientEligible);
+            const orderedClientElectionForSummarizer = new index_js_3.OrderedClientElection(orderedClientLogger, orderedClientCollection, electedSummarizerData ?? this.innerDeltaManager.lastSequenceNumber, index_js_3.SummarizerClientElection.isClientEligible, this.mc.config.getBoolean("Fluid.ContainerRuntime.OrderedClientElection.EnablePerformanceEvents"));
             this.summarizerClientElection = new index_js_3.SummarizerClientElection(orderedClientLogger, this.summaryCollection, orderedClientElectionForSummarizer, this.maxOpsSinceLastSummary);
             if (this.isSummarizerClient) {
                 this._summarizer = new index_js_3.Summarizer(this /* ISummarizerRuntime */, () => this.summaryConfiguration, this /* ISummarizerInternalsProvider */, this.handleContext, this.summaryCollection, async (runtime) => index_js_3.RunWhileConnectedCoordinator.create(runtime, 
@@ -874,7 +877,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                         // 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(protocol_definitions_1.MessageType.SummaryAck, () => {
+                        this.summaryCollection.once(internal_3.MessageType.SummaryAck, () => {
                             this.mc.logger.sendTelemetryEvent({
                                 eventName: "SummaryStatus:CaughtUp",
                             });
@@ -901,7 +904,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             }
         }
         // logging hardware telemetry
-        logger.sendTelemetryEvent({
+        this.logger.sendTelemetryEvent({
             eventName: "DeviceSpec",
             ...getDeviceSpec(),
         });
@@ -1064,7 +1067,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             const props = {
                 eventName: "GroupIdSnapshotCatchup",
                 loadingGroupIds: sortedLoadingGroupIds.join(","),
-                targetSequenceNumber: snapshotSeqNumber,
+                targetSequenceNumber: snapshotSeqNumber, // This is so we reuse some columns in telemetry
                 sequenceNumber: this.deltaManager.lastSequenceNumber, // This is so we reuse some columns in telemetry
             };
             const event = internal_7.PerformanceEvent.start(this.mc.logger, {
@@ -1182,7 +1185,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             this.messageAtLastSummary;
         const documentSchema = this.documentsSchemaController.summarizeDocumentSchema(this.deltaManager.lastSequenceNumber);
         // Is document schema explicit control on?
-        const explitiSchemaControl = documentSchema?.runtime.explicitSchemaControl;
+        const explicitSchemaControl = documentSchema?.runtime.explicitSchemaControl;
         const metadata = {
             ...this.createContainerMetadata,
             // Increment the summary number for the next summary that will be generated.
@@ -1195,10 +1198,10 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             // runtimes (that preceed document schema control capabilities) to close container on load due to mismatch in
             // last message's sequence number.
             // See also lastMessageFromMetadata()
-            message: explitiSchemaControl
+            message: explicitSchemaControl
                 ? { sequenceNumber: -1 }
                 : message,
-            lastMessage: explitiSchemaControl ? message : undefined,
+            lastMessage: explicitSchemaControl ? message : undefined,
             documentSchema,
         };
         (0, internal_6.addBlobToSummary)(summaryTree, index_js_3.metadataBlobName, JSON.stringify(metadata));
@@ -1453,7 +1456,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // Whether or not the message appears to be a runtime message from an up-to-date client.
         // It may be a legacy runtime message (ie already unpacked and ContainerMessageType)
         // or something different, like a system message.
-        const modernRuntimeMessage = messageArg.type === protocol_definitions_1.MessageType.Operation;
+        const modernRuntimeMessage = messageArg.type === internal_3.MessageType.Operation;
         // Do shallow copy of message, as the processing flow will modify it.
         // There might be multiple container instances receiving the same message.
         // We do not need to make a deep copy. Each layer will just replace message.contents itself,
@@ -1917,7 +1920,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 await this.collectGarbage({ logger: summaryLogger, runSweep, fullGC }, telemetryContext);
             }
             const { stats, summary } = await this.summarizerNode.summarize(fullTree, trackState, telemetryContext);
-            (0, internal_2.assert)(summary.type === protocol_definitions_1.SummaryType.Tree, 0x12f /* "Container Runtime's summarize should always return a tree" */);
+            (0, internal_2.assert)(summary.type === driver_definitions_1.SummaryType.Tree, 0x12f /* "Container Runtime's summarize should always return a tree" */);
             return { stats, summary };
         }
         finally {
@@ -2080,7 +2083,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
      * @param options - options controlling how the summary is generated or submitted
      */
     async submitSummary(options) {
-        const { fullTree = false, finalAttempt = false, refreshLatestAck, summaryLogger, latestSummaryRefSeqNum, } = options;
+        const { 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;
@@ -2095,13 +2098,6 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             },
         });
         (0, internal_2.assert)(this.outbox.isEmpty, 0x3d1 /* Can't trigger summary in the middle of a batch */);
-        // We close the summarizer and download a new snapshot and reload the container
-        if (refreshLatestAck === true) {
-            return this.prefetchLatestSummaryThenClose((0, internal_7.createChildLogger)({
-                logger: summaryNumberLogger,
-                properties: { all: { safeSummary: true } },
-            }));
-        }
         // If the container is dirty, i.e., there are pending unacked ops, the summary will not be eventual consistent
         // and it may even be incorrect. So, wait for the container to be saved with a timeout. If the container is not
         // saved within the timeout, check if it should be failed or can continue.
@@ -2170,7 +2166,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                         stage: "base",
                         referenceSequenceNumber: summaryRefSeqNum,
                         minimumSequenceNumber,
-                        error: new internal_7.LoggingError(`Summarizer node state inconsistent with summarizer state.`),
+                        error: new index_js_3.RetriableSummaryError(`Summarizer node state inconsistent with summarizer state.`),
                     };
                 }
             }
@@ -2213,17 +2209,14 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     stage: "base",
                     referenceSequenceNumber: summaryRefSeqNum,
                     minimumSequenceNumber,
-                    error: new internal_7.LoggingError(continueResult.error),
+                    error: new index_js_3.RetriableSummaryError(continueResult.error),
                 };
             }
             const trace = client_utils_1.Trace.start();
             let summarizeResult;
-            // If the GC state needs to be reset, we need to force a full tree summary and update the unreferenced
-            // state of all the nodes.
-            const forcedFullTree = this.garbageCollector.summaryStateNeedsReset;
             try {
                 summarizeResult = await this.summarize({
-                    fullTree: fullTree || forcedFullTree,
+                    fullTree,
                     trackState: true,
                     summaryLogger: summaryNumberLogger,
                     runGC: this.garbageCollector.shouldRunGC,
@@ -2234,7 +2227,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                     stage: "base",
                     referenceSequenceNumber: summaryRefSeqNum,
                     minimumSequenceNumber,
-                    error: (0, internal_7.wrapError)(error, (msg) => new internal_7.LoggingError(msg)),
+                    error: (0, internal_7.wrapError)(error, (msg) => new index_js_3.RetriableSummaryError(msg)),
                 };
             }
             // Validate that the summary generated by summarizer nodes is correct before uploading.
@@ -2262,8 +2255,8 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             // Because handles are unchanged dataStores in the current logic,
             // summarized dataStore count is total dataStore count minus handle count
             const dataStoreTree = summaryTree.tree[internal_5.channelsTreeName];
-            (0, internal_2.assert)(dataStoreTree.type === protocol_definitions_1.SummaryType.Tree, 0x1fc /* "summary is not a tree" */);
-            const handleCount = Object.values(dataStoreTree.tree).filter((value) => value.type === protocol_definitions_1.SummaryType.Handle).length;
+            (0, internal_2.assert)(dataStoreTree.type === driver_definitions_1.SummaryType.Tree, 0x1fc /* "summary is not a tree" */);
+            const handleCount = Object.values(dataStoreTree.tree).filter((value) => value.type === driver_definitions_1.SummaryType.Handle).length;
             const gcSummaryTreeStats = summaryTree.tree[internal_5.gcTreeKey]
                 ? (0, internal_6.calculateStats)(summaryTree.tree[internal_5.gcTreeKey])
                 : undefined;
@@ -2282,14 +2275,13 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 summaryTree,
                 summaryStats,
                 generateDuration: trace.trace().duration,
-                forcedFullTree,
             };
             continueResult = checkContinue();
             if (!continueResult.continue) {
                 return {
                     stage: "generate",
                     ...generateSummaryData,
-                    error: new internal_7.LoggingError(continueResult.error),
+                    error: new index_js_3.RetriableSummaryError(continueResult.error),
                 };
             }
             const summaryContext = lastAck === undefined
@@ -2311,7 +2303,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 return {
                     stage: "generate",
                     ...generateSummaryData,
-                    error: (0, internal_7.wrapError)(error, (msg) => new internal_7.LoggingError(msg)),
+                    error: (0, internal_7.wrapError)(error, (msg) => new index_js_3.RetriableSummaryError(msg)),
                 };
             }
             const parent = summaryContext.ackHandle;
@@ -2332,7 +2324,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 return {
                     stage: "upload",
                     ...uploadData,
-                    error: new internal_7.LoggingError(continueResult.error),
+                    error: new index_js_3.RetriableSummaryError(continueResult.error),
                 };
             }
             let clientSequenceNumber;
@@ -2343,7 +2335,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 return {
                     stage: "upload",
                     ...uploadData,
-                    error: (0, internal_7.wrapError)(error, (msg) => new internal_7.LoggingError(msg)),
+                    error: (0, internal_7.wrapError)(error, (msg) => new index_js_3.RetriableSummaryError(msg)),
                 };
             }
             const submitData = {
@@ -2359,7 +2351,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
                 return {
                     stage: "upload",
                     ...uploadData,
-                    error: (0, internal_7.wrapError)(error, (msg) => new internal_7.LoggingError(msg)),
+                    error: (0, internal_7.wrapError)(error, (msg) => new index_js_3.RetriableSummaryError(msg)),
                 };
             }
             return submitData;
@@ -2449,13 +2441,13 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         this.verifyNotClosed();
         return this.blobManager.createBlob(blob, signal);
     }
-    submitIdAllocationOpIfNeeded(resubmitOutstandingRanges = false) {
+    submitIdAllocationOpIfNeeded(resubmitOutstandingRanges) {
         if (this._idCompressor) {
             const idRange = resubmitOutstandingRanges
-                ? this.idCompressor?.takeUnfinalizedCreationRange()
+                ? this._idCompressor.takeUnfinalizedCreationRange()
                 : this._idCompressor.takeNextCreationRange();
             // Don't include the idRange if there weren't any Ids allocated
-            if (idRange?.ids !== undefined) {
+            if (idRange.ids !== undefined) {
                 const idAllocationMessage = {
                     type: messageTypes_js_1.ContainerMessageType.IdAllocation,
                     contents: idRange,
@@ -2484,6 +2476,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             });
         }
         const type = containerRuntimeMessage.type;
+        (0, internal_2.assert)(type !== messageTypes_js_1.ContainerMessageType.IdAllocation, "IdAllocation should be submitted directly to outbox.");
         const message = {
             contents: serializedContent,
             metadata,
@@ -2491,44 +2484,35 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             referenceSequenceNumber: this.deltaManager.lastSequenceNumber,
         };
         try {
-            // If `message` is an allocation op, then we are in the resubmit path and we must redirect the allocation
-            // op into the correct batch to avoid ranges being finalized out of order.
-            // Otherwise, submit an IdAllocation op if any IDs have been generated since the last op was submitted, as
-            // any of the other op types may contain those IDs and thus depend on the allocation op being sent first.
-            if (type === messageTypes_js_1.ContainerMessageType.IdAllocation) {
-                this.outbox.submitIdAllocation(message);
+            this.submitIdAllocationOpIfNeeded(false);
+            // Allow document schema controller to send a message if it needs to propose change in document schema.
+            // If it needs to send a message, it will call provided callback with payload of such message and rely
+            // on this callback to do actual sending.
+            const contents = this.documentsSchemaController.maybeSendSchemaMessage();
+            if (contents) {
+                this.logger.sendTelemetryEvent({
+                    eventName: "SchemaChangeProposal",
+                    refSeq: contents.refSeq,
+                    version: contents.version,
+                    newRuntimeSchema: JSON.stringify(contents.runtime),
+                    sessionRuntimeSchema: JSON.stringify(this.sessionSchema),
+                    oldRuntimeSchema: JSON.stringify(this.metadata?.documentSchema?.runtime),
+                });
+                const msg = {
+                    type: messageTypes_js_1.ContainerMessageType.DocumentSchemaChange,
+                    contents,
+                };
+                this.outbox.submit({
+                    contents: JSON.stringify(msg),
+                    referenceSequenceNumber: this.deltaManager.lastSequenceNumber,
+                });
+            }
+            if (type === messageTypes_js_1.ContainerMessageType.BlobAttach) {
+                // BlobAttach ops must have their metadata visible and cannot be grouped (see opGroupingManager.ts)
+                this.outbox.submitBlobAttach(message);
             }
             else {
-                this.submitIdAllocationOpIfNeeded();
-                // Allow document schema controller to send a message if it needs to propose change in document schema.
-                // If it needs to send a message, it will call provided callback with payload of such message and rely
-                // on this callback to do actual sending.
-                const contents = this.documentsSchemaController.maybeSendSchemaMessage();
-                if (contents) {
-                    this.logger.sendTelemetryEvent({
-                        eventName: "SchemaChangeProposal",
-                        refSeq: contents.refSeq,
-                        version: contents.version,
-                        newRuntimeSchema: JSON.stringify(contents.runtime),
-                        sessionRuntimeSchema: JSON.stringify(this.sessionSchema),
-                        oldRuntimeSchema: JSON.stringify(this.metadata?.documentSchema?.runtime),
-                    });
-                    const msg = {
-                        type: messageTypes_js_1.ContainerMessageType.DocumentSchemaChange,
-                        contents,
-                    };
-                    this.outbox.submit({
-                        contents: JSON.stringify(msg),
-                        referenceSequenceNumber: this.deltaManager.lastSequenceNumber,
-                    });
-                }
-                if (type === messageTypes_js_1.ContainerMessageType.BlobAttach) {
-                    // BlobAttach ops must have their metadata visible and cannot be grouped (see opGroupingManager.ts)
-                    this.outbox.submitBlobAttach(message);
-                }
-                else {
-                    this.outbox.submit(message);
-                }
+                this.outbox.submit(message);
             }
             if (!this.currentlyBatching()) {
                 this.flush();
@@ -2586,7 +2570,7 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
         // back-compat: ADO #1385: Make this call unconditional in the future
         return this.submitSummaryFn !== undefined
             ? this.submitSummaryFn(contents, referenceSequenceNumber)
-            : this.submitFn(protocol_definitions_1.MessageType.Summarize, contents, false);
+            : this.submitFn(internal_3.MessageType.Summarize, contents, false);
     }
     /**
      * Throw an error if the runtime is closed.  Methods that are expected to potentially
@@ -2703,51 +2687,23 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
          * and then close as the current main client is likely to be re-elected as the parent summarizer again.
          */
         if (!result.isSummaryTracked && result.isSummaryNewer) {
-            await this.fetchLatestSnapshotFromStorage(summaryLogger, {
+            await this.fetchLatestSnapshotAndClose(summaryLogger, {
                 eventName: "RefreshLatestSummaryAckFetch",
                 ackHandle,
                 targetSequenceNumber: summaryRefSeq,
             }, readAndParseBlob);
-            await this.closeStaleSummarizer();
             return;
         }
         // Notify the garbage collector so it can update its latest summary state.
         await this.garbageCollector.refreshLatestSummary(result);
     }
     /**
-     * Fetches the latest snapshot from storage to refresh the cache as a performance optimization and closes the
-     * summarizer to reload from new state.
-     * @param summaryLogger - logger to use when fetching snapshot from storage
-     * @returns a generic summarization error
+     * Fetches the latest snapshot from storage and closes the container. This is done in cases where
+     * the last known snapshot is older than the latest one. This will ensure that the latest snapshot
+     * is downloaded and we don't end up loading snapshot from cache.
      */
-    async prefetchLatestSummaryThenClose(summaryLogger) {
-        const readAndParseBlob = async (id) => (0, internal_4.readAndParse)(this.storage, id);
-        // This is a performance optimization as the same parent is likely to be elected again, and would use its
-        // cache to fetch the snapshot instead of the network.
-        await this.fetchLatestSnapshotFromStorage(summaryLogger, {
-            eventName: "RefreshLatestSummaryFromServerFetch",
-        }, readAndParseBlob);
-        await this.closeStaleSummarizer();
-        return {
-            stage: "base",
-            error: new internal_7.LoggingError("summary state stale - Unsupported option 'refreshLatestAck'"),
-            referenceSequenceNumber: this.deltaManager.lastSequenceNumber,
-            minimumSequenceNumber: this.deltaManager.minimumSequenceNumber,
-        };
-    }
-    async closeStaleSummarizer() {
-        // Delay before restarting summarizer to prevent the summarizer from restarting too frequently.
-        await (0, internal_2.delay)(this.closeSummarizerDelayMs);
-        this._summarizer?.stop("latestSummaryStateStale");
-        this.disposeFn();
-    }
-    /**
-     * Downloads the latest snapshot from storage.
-     * By default, it also closes the container after downloading the snapshot. However, this may be
-     * overridden via options.
-     */
-    async fetchLatestSnapshotFromStorage(logger, event, readAndParseBlob) {
-        return internal_7.PerformanceEvent.timedExecAsync(logger, event, async (perfEvent) => {
+    async fetchLatestSnapshotAndClose(logger, event, readAndParseBlob) {
+        await internal_7.PerformanceEvent.timedExecAsync(logger, event, async (perfEvent) => {
             const stats = {};
             const trace = client_utils_1.Trace.start();
             const versions = await this.storage.getVersions(null, 1, "prefetchLatestSummaryBeforeClose", internal_3.FetchSource.noCache);
@@ -2760,12 +2716,10 @@ class ContainerRuntime extends client_utils_1.TypedEventEmitter {
             stats.snapshotRefSeq = latestSnapshotRefSeq;
             stats.snapshotVersion = versions[0].id;
             perfEvent.end(stats);
-            return {
-                snapshotTree: maybeSnapshot,
-                versionId: versions[0].id,
-                latestSnapshotRefSeq,
-            };
         });
+        await (0, internal_2.delay)(this.closeSummarizerDelayMs);
+        this._summarizer?.stop("latestSummaryStateStale");
+        this.disposeFn();
     }
     getPendingLocalState(props) {
         this.verifyNotClosed();