@fluidframework/datastore 2.43.0 → 2.50.0

package/lib/packageVersion.d.ts

@@ -5,5 +5,5 @@
  * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY
  */
 export declare const pkgName = "@fluidframework/datastore";
-export declare const pkgVersion = "2.43.0";
+export declare const pkgVersion = "2.50.0";
 //# sourceMappingURL=packageVersion.d.ts.map

package/lib/packageVersion.js

@@ -5,5 +5,5 @@
  * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY
  */
 export const pkgName = "@fluidframework/datastore";
-export const pkgVersion = "2.43.0";
+export const pkgVersion = "2.50.0";
 //# sourceMappingURL=packageVersion.js.map

package/lib/packageVersion.js.map

@@ -1 +1 @@
-{"version":3,"file":"packageVersion.js","sourceRoot":"","sources":["../src/packageVersion.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,CAAC,MAAM,OAAO,GAAG,2BAA2B,CAAC;AACnD,MAAM,CAAC,MAAM,UAAU,GAAG,QAAQ,CAAC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n *\n * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY\n */\n\nexport const pkgName = \"@fluidframework/datastore\";\nexport const pkgVersion = \"2.43.0\";\n"]}
+{"version":3,"file":"packageVersion.js","sourceRoot":"","sources":["../src/packageVersion.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,CAAC,MAAM,OAAO,GAAG,2BAA2B,CAAC;AACnD,MAAM,CAAC,MAAM,UAAU,GAAG,QAAQ,CAAC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n *\n * THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY\n */\n\nexport const pkgName = \"@fluidframework/datastore\";\nexport const pkgVersion = \"2.50.0\";\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/datastore",
-  "version": "2.43.0",
+  "version": "2.50.0",
   "description": "Fluid data store implementation",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -69,29 +69,29 @@
     "temp-directory": "nyc/.nyc_output"
   },
   "dependencies": {
-    "@fluid-internal/client-utils": "~2.43.0",
-    "@fluidframework/container-definitions": "~2.43.0",
-    "@fluidframework/core-interfaces": "~2.43.0",
-    "@fluidframework/core-utils": "~2.43.0",
-    "@fluidframework/datastore-definitions": "~2.43.0",
-    "@fluidframework/driver-definitions": "~2.43.0",
-    "@fluidframework/driver-utils": "~2.43.0",
-    "@fluidframework/id-compressor": "~2.43.0",
-    "@fluidframework/runtime-definitions": "~2.43.0",
-    "@fluidframework/runtime-utils": "~2.43.0",
-    "@fluidframework/telemetry-utils": "~2.43.0",
+    "@fluid-internal/client-utils": "~2.50.0",
+    "@fluidframework/container-definitions": "~2.50.0",
+    "@fluidframework/core-interfaces": "~2.50.0",
+    "@fluidframework/core-utils": "~2.50.0",
+    "@fluidframework/datastore-definitions": "~2.50.0",
+    "@fluidframework/driver-definitions": "~2.50.0",
+    "@fluidframework/driver-utils": "~2.50.0",
+    "@fluidframework/id-compressor": "~2.50.0",
+    "@fluidframework/runtime-definitions": "~2.50.0",
+    "@fluidframework/runtime-utils": "~2.50.0",
+    "@fluidframework/telemetry-utils": "~2.50.0",
     "uuid": "^9.0.0"
   },
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.17.1",
     "@biomejs/biome": "~1.9.3",
-    "@fluid-internal/mocha-test-setup": "~2.43.0",
+    "@fluid-internal/mocha-test-setup": "~2.50.0",
     "@fluid-tools/build-cli": "^0.56.0",
     "@fluidframework/build-common": "^2.0.3",
     "@fluidframework/build-tools": "^0.56.0",
-    "@fluidframework/datastore-previous": "npm:@fluidframework/datastore@2.42.0",
+    "@fluidframework/datastore-previous": "npm:@fluidframework/datastore@2.43.0",
     "@fluidframework/eslint-config-fluid": "^5.7.4",
-    "@fluidframework/test-runtime-utils": "~2.43.0",
+    "@fluidframework/test-runtime-utils": "~2.50.0",
     "@microsoft/api-extractor": "7.52.8",
     "@types/lodash": "^4.14.118",
     "@types/mocha": "^10.0.10",
@@ -110,7 +110,20 @@
     "typescript": "~5.4.5"
   },
   "typeValidation": {
-    "broken": {},
+    "broken": {
+      "Class_FluidDataStoreRuntime": {
+        "backCompat": false
+      },
+      "Class_FluidObjectHandle": {
+        "backCompat": false
+      },
+      "ClassStatics_FluidDataStoreRuntime": {
+        "backCompat": false
+      },
+      "ClassStatics_FluidObjectHandle": {
+        "backCompat": false
+      }
+    },
     "entrypoint": "legacy"
   },
   "scripts": {

package/src/dataStoreLayerCompatState.ts

@@ -43,7 +43,7 @@ export const dataStoreCompatDetailsForRuntime: ILayerCompatDetails = {
  * The requirements that the Runtime layer must meet to be compatible with this DataStore.
  * @internal
  */
-export const runtimeSupportRequirements: ILayerCompatSupportRequirements = {
+export const runtimeSupportRequirementsForDataStore: ILayerCompatSupportRequirements = {
 	/**
 	 * Minimum generation that Runtime must be at to be compatible with DataStore. Note that 0 is used here so
 	 * that Runtime layers before the introduction of the layer compatibility enforcement are compatible.
@@ -64,7 +64,7 @@ export function validateRuntimeCompatibility(
 	disposeFn: () => void,
 ): void {
 	const layerCheckResult = checkLayerCompatibility(
-		runtimeSupportRequirements,
+		runtimeSupportRequirementsForDataStore,
 		maybeRuntimeCompatDetails,
 	);
 	if (!layerCheckResult.isCompatible) {
@@ -74,7 +74,7 @@ export function validateRuntimeCompatibility(
 				runtimeVersion: maybeRuntimeCompatDetails?.pkgVersion,
 				dataStoreGeneration: dataStoreCoreCompatDetails.generation,
 				runtimeGeneration: maybeRuntimeCompatDetails?.generation,
-				minSupportedGeneration: runtimeSupportRequirements.minSupportedGeneration,
+				minSupportedGeneration: runtimeSupportRequirementsForDataStore.minSupportedGeneration,
 				isGenerationCompatible: layerCheckResult.isGenerationCompatible,
 				unsupportedFeatures: layerCheckResult.unsupportedFeatures,
 			}),

package/src/dataStoreRuntime.ts

@@ -98,7 +98,6 @@ import { v4 as uuid } from "uuid";
 import { IChannelContext, summarizeChannel } from "./channelContext.js";
 import {
 	dataStoreCompatDetailsForRuntime,
-	// dataStoreCompatDetailsForRuntime,
 	validateRuntimeCompatibility,
 } from "./dataStoreLayerCompatState.js";
 import { FluidObjectHandle } from "./fluidHandle.js";

package/src/fluidHandle.ts

@@ -4,7 +4,6 @@
  */
 
 import { FluidObject } from "@fluidframework/core-interfaces";
-import type { IFluidHandleInternal } from "@fluidframework/core-interfaces/internal";
 import { IFluidHandleContext } from "@fluidframework/core-interfaces/internal";
 import {
 	generateHandleContextPath,
@@ -19,8 +18,6 @@ import {
 export class FluidObjectHandle<
 	T extends FluidObject = FluidObject,
 > extends FluidHandleBase<T> {
-	private readonly pendingHandlesToMakeVisible: Set<IFluidHandleInternal> = new Set();
-
 	/**
 	 * {@inheritDoc @fluidframework/core-interfaces#IFluidHandle.absolutePath}
 	 */
@@ -33,30 +30,6 @@ export class FluidObjectHandle<
 		return this.routeContext.isAttached;
 	}
 
-	/**
-	 * Tells whether the object of this handle is visible in the container locally or globally.
-	 */
-	private get visible(): boolean {
-		/**
-		 * If the object of this handle is attached, it is visible in the container. Ideally, checking local visibility
-		 * should be enough for a handle. However, there are scenarios where the object becomes locally visible but the
-		 * handle does not know this - This will happen is attachGraph is never called on the handle. Couple of examples
-		 * where this can happen:
-		 *
-		 * 1. Handles to DDS other than the default handle won't know if the DDS becomes visible after the handle was
-		 * created.
-		 *
-		 * 2. Handles to root data stores will never know that it was visible because the handle will not be stores in
-		 * another DDS and so, attachGraph will never be called on it.
-		 */
-		return this.isAttached || this.locallyVisible;
-	}
-
-	/**
-	 * Tracks whether this handle is locally visible in the container.
-	 */
-	private locallyVisible: boolean = false;
-
 	/**
 	 * Creates a new `FluidObjectHandle`.
 	 *
@@ -89,29 +62,6 @@ export class FluidObjectHandle<
 	 * {@inheritDoc @fluidframework/core-interfaces#IFluidHandle.attachGraph }
 	 */
 	public attachGraph(): void {
-		if (this.visible) {
-			return;
-		}
-
-		this.locallyVisible = true;
-		for (const handle of this.pendingHandlesToMakeVisible) {
-			handle.attachGraph();
-		}
-		this.pendingHandlesToMakeVisible.clear();
 		this.routeContext.attachGraph();
 	}
-
-	// eslint-disable-next-line jsdoc/require-description
-	/**
-	 * @deprecated No replacement provided. Arbitrary handles may not serve as a bind source.
-	 * @privateRemarks This implementation will be moved to SharedObjectHandle once this is removed.
-	 */
-	public bind(handle: IFluidHandleInternal): void {
-		// If this handle is visible, attach the graph of the incoming handle as well.
-		if (this.visible) {
-			handle.attachGraph();
-			return;
-		}
-		this.pendingHandlesToMakeVisible.add(handle);
-	}
 }

package/src/index.ts

@@ -11,3 +11,8 @@ export {
 	mixinRequestHandler,
 	mixinSummaryHandler,
 } from "./dataStoreRuntime.js";
+export {
+	dataStoreCoreCompatDetails,
+	dataStoreCompatDetailsForRuntime,
+	runtimeSupportRequirementsForDataStore,
+} from "./dataStoreLayerCompatState.js";

package/src/packageVersion.ts

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