@fluidframework/datastore 2.1.0-276985 → 2.1.0

package/src/dataStoreRuntime.ts

@@ -255,7 +255,7 @@ export class FluidDataStoreRuntime
 
 		// Must always receive the data store type inside of the attributes
 		if (tree?.trees !== undefined) {
-			Object.keys(tree.trees).forEach((path) => {
+			Object.entries(tree.trees).forEach(([path, subtree]) => {
 				// Issue #4414
 				if (path === "_search") {
 					return;
@@ -266,7 +266,7 @@ export class FluidDataStoreRuntime
 				// container from snapshot where we load detached container from a snapshot, isLocalDataStore would be
 				// true. In this case create a RehydratedLocalChannelContext.
 				if (dataStoreContext.isLocalDataStore) {
-					channelContext = this.createRehydratedLocalChannelContext(path, tree.trees[path]);
+					channelContext = this.createRehydratedLocalChannelContext(path, subtree);
 					// This is the case of rehydrating a detached container from snapshot. Now due to delay loading of
 					// data store, if the data store is loaded after the container is attached, then we missed making
 					// the channel visible. So do it now. Otherwise, add it to local channel context queue, so
@@ -284,7 +284,7 @@ export class FluidDataStoreRuntime
 						(content, localOpMetadata) => this.submitChannelOp(path, content, localOpMetadata),
 						(address: string) => this.setChannelDirty(address),
 						path,
-						tree.trees[path],
+						subtree,
 						this.sharedObjectRegistry,
 						undefined /* extraBlobs */,
 						this.dataStoreContext.getCreateChildSummarizerNodeFn(path, {
@@ -362,17 +362,19 @@ export class FluidDataStoreRuntime
 				return await this.request(parser.createSubRequest(1));
 			}
 
-			// Check for a data type reference first
-			const context = this.contexts.get(id);
-			if (context !== undefined && parser.isLeaf(1)) {
-				try {
-					const channel = await context.getChannel();
+			if (id !== undefined) {
+				// Check for a data type reference first
+				const context = this.contexts.get(id);
+				if (context !== undefined && parser.isLeaf(1)) {
+					try {
+						const channel = await context.getChannel();
 
-					return { mimeType: "fluid/object", status: 200, value: channel };
-				} catch (error) {
-					this.mc.logger.sendErrorEvent({ eventName: "GetChannelFailedInRequest" }, error);
+						return { mimeType: "fluid/object", status: 200, value: channel };
+					} catch (error) {
+						this.mc.logger.sendErrorEvent({ eventName: "GetChannelFailedInRequest" }, error);
 
-					return createResponseError(500, `Failed to get Channel: ${error}`, request);
+						return createResponseError(500, `Failed to get Channel: ${error}`, request);
+					}
 				}
 			}
 
@@ -850,7 +852,9 @@ export class FluidDataStoreRuntime
 						0x181 /* "BaseSnapshot should be there as detached container loaded from snapshot" */,
 					);
 					summaryTree = convertSnapshotTreeToSummaryTree(
-						this.dataStoreContext.baseSnapshot.trees[contextId],
+						// TODO why are we non null asserting here?
+						// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+						this.dataStoreContext.baseSnapshot.trees[contextId]!,
 					);
 				}
 				summaryBuilder.addWithStats(contextId, summaryTree);

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/datastore";
-export const pkgVersion = "2.1.0-276985";
+export const pkgVersion = "2.1.0";

package/tsconfig.json

@@ -5,7 +5,6 @@
 	"compilerOptions": {
 		"rootDir": "./src",
 		"outDir": "./lib",
-		"noUncheckedIndexedAccess": false,
 		"exactOptionalPropertyTypes": false,
 	},
 }