@fluidframework/container-runtime 2.20.0 → 2.21.0

package/dist/channelCollection.js

@@ -18,6 +18,7 @@ const dataStore_js_1 = require("./dataStore.js");
 const dataStoreContext_js_1 = require("./dataStoreContext.js");
 const dataStoreContexts_js_1 = require("./dataStoreContexts.js");
 const dataStoreRegistry_js_1 = require("./dataStoreRegistry.js");
+// eslint-disable-next-line import/no-deprecated
 const index_js_1 = require("./gc/index.js");
 const messageTypes_js_1 = require("./messageTypes.js");
 const storageServiceWithAttachBlobs_js_1 = require("./storageServiceWithAttachBlobs.js");
@@ -165,6 +166,7 @@ class ChannelCollection {
         this.pendingAttach = new Map();
         // 0.24 back-compat attachingBeforeSummary
         this.attachOpFiredForDataStore = new Set();
+        // eslint-disable-next-line unicorn/consistent-function-scoping -- Property is defined once; no need to extract inner lambda
         this.disposeOnce = new internal_1.Lazy(() => this.contexts.dispose());
         this.pendingAliasMap = new Map();
         /**
@@ -193,42 +195,42 @@ class ChannelCollection {
                 unreferencedDataStoreCount++;
             }
             // If we have a detached container, then create local data store contexts.
-            if (this.parentContext.attachState !== container_definitions_1.AttachState.Detached) {
-                let snapshotForRemoteFluidDatastoreContext = value;
-                if ((0, internal_3.isInstanceOfISnapshot)(baseSnapshot)) {
-                    snapshotForRemoteFluidDatastoreContext = {
-                        ...baseSnapshot,
-                        snapshotTree: value,
-                    };
+            if (this.parentContext.attachState === container_definitions_1.AttachState.Detached) {
+                if (typeof value !== "object") {
+                    throw new internal_6.LoggingError("Snapshot should be there to load from!!");
                 }
-                dataStoreContext = new dataStoreContext_js_1.RemoteFluidDataStoreContext({
+                const snapshotTree = value;
+                dataStoreContext = new dataStoreContext_js_1.LocalFluidDataStoreContext({
                     id: key,
-                    snapshot: snapshotForRemoteFluidDatastoreContext,
+                    pkg: undefined,
                     parentContext: this.wrapContextForInnerChannel(key),
                     storage: this.parentContext.storage,
                     scope: this.parentContext.scope,
                     createSummarizerNodeFn: this.parentContext.getCreateChildSummarizerNodeFn(key, {
                         type: internal_4.CreateSummarizerNodeSource.FromSummary,
                     }),
-                    loadingGroupId: value.groupId,
+                    makeLocallyVisibleFn: () => this.makeDataStoreLocallyVisible(key),
+                    snapshotTree,
                 });
             }
             else {
-                if (typeof value !== "object") {
-                    throw new internal_6.LoggingError("Snapshot should be there to load from!!");
+                let snapshotForRemoteFluidDatastoreContext = value;
+                if ((0, internal_3.isInstanceOfISnapshot)(baseSnapshot)) {
+                    snapshotForRemoteFluidDatastoreContext = {
+                        ...baseSnapshot,
+                        snapshotTree: value,
+                    };
                 }
-                const snapshotTree = value;
-                dataStoreContext = new dataStoreContext_js_1.LocalFluidDataStoreContext({
+                dataStoreContext = new dataStoreContext_js_1.RemoteFluidDataStoreContext({
                     id: key,
-                    pkg: undefined,
+                    snapshot: snapshotForRemoteFluidDatastoreContext,
                     parentContext: this.wrapContextForInnerChannel(key),
                     storage: this.parentContext.storage,
                     scope: this.parentContext.scope,
                     createSummarizerNodeFn: this.parentContext.getCreateChildSummarizerNodeFn(key, {
                         type: internal_4.CreateSummarizerNodeSource.FromSummary,
                     }),
-                    makeLocallyVisibleFn: () => this.makeDataStoreLocallyVisible(key),
-                    snapshotTree,
+                    loadingGroupId: value.groupId,
                 });
             }
             this.contexts.addBoundOrRemoted(dataStoreContext);
@@ -491,13 +493,16 @@ class ChannelCollection {
     reSubmit(type, content, localOpMetadata) {
         switch (type) {
             case messageTypes_js_1.ContainerMessageType.Attach:
-            case messageTypes_js_1.ContainerMessageType.Alias:
+            case messageTypes_js_1.ContainerMessageType.Alias: {
                 this.parentContext.submitMessage(type, content, localOpMetadata);
                 return;
-            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp:
+            }
+            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp: {
                 return this.reSubmitChannelOp(type, content, localOpMetadata);
-            default:
+            }
+            default: {
                 (0, internal_1.assert)(false, 0x907 /* unknown op type */);
+            }
         }
     }
     reSubmitChannelOp(type, content, localOpMetadata) {
@@ -534,14 +539,18 @@ class ChannelCollection {
     async applyStashedOp(content) {
         const opContents = content;
         switch (opContents.type) {
-            case messageTypes_js_1.ContainerMessageType.Attach:
+            case messageTypes_js_1.ContainerMessageType.Attach: {
                 return this.applyStashedAttachOp(opContents.contents);
-            case messageTypes_js_1.ContainerMessageType.Alias:
+            }
+            case messageTypes_js_1.ContainerMessageType.Alias: {
                 return;
-            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp:
+            }
+            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp: {
                 return this.applyStashedChannelChannelOp(opContents.contents);
-            default:
+            }
+            default: {
                 (0, internal_1.assert)(false, 0x908 /* unknon type of op */);
+            }
         }
     }
     async applyStashedChannelChannelOp(envelope) {
@@ -589,7 +598,7 @@ class ChannelCollection {
             // if the client is not detached put in the pending attach list
             // so that on ack of the stashed op, the context is found.
             // detached client don't send ops, so should not expect and ack.
-            this.pendingAttach.set(message.id, message);
+            this.pendingAttach.set(id, message);
         }
     }
     /**
@@ -598,17 +607,21 @@ class ChannelCollection {
      */
     processMessages(messageCollection) {
         switch (messageCollection.envelope.type) {
-            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp:
+            case messageTypes_js_1.ContainerMessageType.FluidDataStoreOp: {
                 this.processChannelMessages(messageCollection);
                 break;
-            case messageTypes_js_1.ContainerMessageType.Attach:
+            }
+            case messageTypes_js_1.ContainerMessageType.Attach: {
                 this.processAttachMessages(messageCollection);
                 break;
-            case messageTypes_js_1.ContainerMessageType.Alias:
+            }
+            case messageTypes_js_1.ContainerMessageType.Alias: {
                 this.processAliasMessages(messageCollection);
                 break;
-            default:
+            }
+            default: {
                 (0, internal_1.assert)(false, 0x8e9 /* unreached */);
+            }
         }
     }
     /**
@@ -635,7 +648,7 @@ class ChannelCollection {
      * @param messageCollection - The collection of messages to process.
      */
     processChannelMessages(messageCollection) {
-        const { messagesContent, local } = messageCollection;
+        const { envelope, messagesContent, local } = messageCollection;
         let currentMessageState;
         let currentMessagesContent = [];
         // Helper that sends the current bunch of messages to the data store. It validates that the data stores exists.
@@ -647,7 +660,7 @@ class ChannelCollection {
             const currentContext = this.contexts.get(currentMessageState.address);
             (0, internal_1.assert)(!!currentContext, 0xa66 /* Context not found */);
             currentContext.processMessages({
-                envelope: { ...messageCollection.envelope, type: currentMessageState.type },
+                envelope: { ...envelope, type: currentMessageState.type },
                 messagesContent: currentMessagesContent,
                 local,
             });
@@ -669,10 +682,10 @@ class ChannelCollection {
             }
             if (context === undefined) {
                 // Former assert 0x162
-                throw internal_6.DataProcessingError.create("No context for op", "processFluidDataStoreOp", messageCollection.envelope, {
+                throw internal_6.DataProcessingError.create("No context for op", "processFluidDataStoreOp", envelope, {
                     local,
                     messageDetails: JSON.stringify({
-                        type: messageCollection.envelope.type,
+                        type: envelope.type,
                         contentType: typeof contents,
                     }),
                     ...(0, internal_6.tagCodeArtifacts)({ address }),
@@ -694,10 +707,10 @@ class ChannelCollection {
             this.gcNodeUpdated({
                 node: { type: "DataStore", path: `/${address}` },
                 reason: "Changed",
-                timestampMs: messageCollection.envelope.timestamp,
+                timestampMs: envelope.timestamp,
                 packagePath: context.isLoaded ? context.packagePath : undefined,
             });
-            detectOutboundReferences(address, contextContents, (fromPath, toPath) => this.parentContext.addedGCOutboundRoute(fromPath, toPath, messageCollection.envelope.timestamp));
+            detectOutboundReferences(address, contextContents, (fromPath, toPath) => this.parentContext.addedGCOutboundRoute(fromPath, toPath, envelope.timestamp));
         }
         // Process the last bunch of messages, if any. Note that there may not be any messages in case all of them are
         // ignored because the data store is deleted.
@@ -708,6 +721,7 @@ class ChannelCollection {
         if (this.checkAndLogIfDeleted(id, this.contexts.get(id), "Requested", "getDataStore", requestHeaderData, originalRequest)) {
             // The requested data store has been deleted by gc. Create a 404 response exception.
             throw (0, internal_5.responseToException)((0, internal_5.createResponseError)(404, "DataStore was deleted", originalRequest, {
+                // eslint-disable-next-line import/no-deprecated
                 [containerRuntime_js_1.DeletedResponseHeaderKey]: true,
             }), originalRequest);
         }
@@ -749,7 +763,7 @@ class ChannelCollection {
         if (!this.isDataStoreDeleted(dataStoreNodePath)) {
             return false;
         }
-        const idToLog = originalRequest !== undefined ? (0, index_js_1.urlToGCNodePath)(originalRequest.url) : dataStoreNodePath;
+        const idToLog = originalRequest === undefined ? dataStoreNodePath : (0, index_js_1.urlToGCNodePath)(originalRequest.url);
         // Log the package details asynchronously since getInitialSnapshotDetails is async
         const recentlyDeletedContext = this.contexts.getRecentlyDeletedContext(id);
         if (recentlyDeletedContext !== undefined) {
@@ -872,7 +886,7 @@ class ChannelCollection {
             builder.prefixAndAddNodes(contextId, contextGCData.gcNodes);
         });
         // Get the outbound routes (aliased data stores) and add a GC node for this channel.
-        builder.addNode("/", Array.from(this.aliasedDataStores));
+        builder.addNode("/", [...this.aliasedDataStores]);
         return builder.getGCData();
     }
     /**
@@ -1037,7 +1051,7 @@ class ChannelCollection {
             }
             this.deleteChild(dataStoreId);
         }
-        return Array.from(sweepReadyDataStoreRoutes);
+        return [...sweepReadyDataStoreRoutes];
     }
     /**
      * This is called to update objects whose routes are tombstones.
@@ -1086,12 +1100,14 @@ class ChannelCollection {
         // If the node belongs to a data store, return its package path. For DDSes, we return the package path of the
         // data store that contains it.
         const context = this.contexts.get(nodePath.split("/")[1]);
-        return (await context?.getInitialSnapshotDetails())?.pkg;
+        const initialSnapshotDetails = await context?.getInitialSnapshotDetails();
+        return initialSnapshotDetails?.pkg;
     }
     /**
      * Called by GC to determine if a node is for a data store or for an object within a data store (for e.g. DDS).
      * @returns the GC node type if the node belongs to a data store or object within data store, undefined otherwise.
      */
+    // eslint-disable-next-line import/no-deprecated
     getGCNodeType(nodePath) {
         const pathParts = nodePath.split("/");
         if (!this.contexts.has(pathParts[1])) {
@@ -1100,8 +1116,10 @@ class ChannelCollection {
         // Data stores paths are of the format "/dataStoreId".
         // Sub data store paths are of the format "/dataStoreId/subPath/...".
         if (pathParts.length === 2) {
+            // eslint-disable-next-line import/no-deprecated
             return index_js_1.GCNodeType.DataStore;
         }
+        // eslint-disable-next-line import/no-deprecated
         return index_js_1.GCNodeType.SubDataStore;
     }
     internalId(maybeAlias) {
@@ -1150,7 +1168,9 @@ class ChannelCollection {
     }
 }
 exports.ChannelCollection = ChannelCollection;
-function getSummaryForDatastores(snapshot, metadata) {
+function getSummaryForDatastores(snapshot, 
+// eslint-disable-next-line import/no-deprecated
+metadata) {
     if (!snapshot) {
         return undefined;
     }
@@ -1204,7 +1224,9 @@ function detectOutboundReferences(address, contents, addedOutboundReference) {
     // GC node paths are all absolute paths, hence the "" prefix.
     // e.g. this will yield "/dataStoreId/ddsId"
     const fromPath = ["", address, ddsAddress].join("/");
-    outboundPaths.forEach((toPath) => addedOutboundReference(fromPath, toPath));
+    for (const toPath of outboundPaths) {
+        addedOutboundReference(fromPath, toPath);
+    }
 }
 exports.detectOutboundReferences = detectOutboundReferences;
 /**