@fluidframework/datastore 2.0.0-internal.1.2.0.93071 → 2.0.0-internal.1.2.2

package/lib/fluidHandle.js.map

@@ -1 +1 @@
-{"version":3,"file":"fluidHandle.js","sourceRoot":"","sources":["../src/fluidHandle.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAOH,OAAO,EAAE,yBAAyB,EAAE,MAAM,+BAA+B,CAAC;AAE1E,MAAM,OAAO,iBAAiB;IA8B1B;;;;;OAKG;IACH,YACuB,KAAqB,EACxB,IAAY,EACZ,YAAiC;QAF9B,UAAK,GAAL,KAAK,CAAgB;QACxB,SAAI,GAAJ,IAAI,CAAQ;QACZ,iBAAY,GAAZ,YAAY,CAAqB;QAtCpC,gCAA2B,GAAsB,IAAI,GAAG,EAAE,CAAC;QA0B5E,kEAAkE;QAC1D,mBAAc,GAAY,KAAK,CAAC;QAapC,IAAI,CAAC,YAAY,GAAG,yBAAyB,CAAC,IAAI,EAAE,IAAI,CAAC,YAAY,CAAC,CAAC;IAC3E,CAAC;IAtCD,IAAW,YAAY,KAAmB,OAAO,IAAI,CAAC,CAAC,CAAC;IAExD,IAAW,UAAU;QACjB,OAAO,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC;IACxC,CAAC;IAED;;OAEG;IACH,IAAY,OAAO;QACf;;;;;;;;;WASG;QACH,OAAO,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,cAAc,CAAC;IAClD,CAAC;IAmBM,KAAK,CAAC,GAAG;QACZ,yGAAyG;QACzG,OAAO,IAAI,CAAC,KAAK,CAAC;IACtB,CAAC;IAEM,WAAW;QACd,IAAI,IAAI,CAAC,OAAO,EAAE;YACd,OAAO;SACV;QAED,IAAI,CAAC,cAAc,GAAG,IAAI,CAAC;QAC3B,IAAI,CAAC,2BAA2B,CAAC,OAAO,CAAC,CAAC,MAAM,EAAE,EAAE;YAChD,MAAM,CAAC,WAAW,EAAE,CAAC;QACzB,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,2BAA2B,CAAC,KAAK,EAAE,CAAC;QACzC,IAAI,CAAC,YAAY,CAAC,WAAW,EAAE,CAAC;IACpC,CAAC;IAEM,IAAI,CAAC,MAAoB;QAC5B,8EAA8E;QAC9E,IAAI,IAAI,CAAC,OAAO,EAAE;YACd,MAAM,CAAC,WAAW,EAAE,CAAC;YACrB,OAAO;SACV;QACD,IAAI,CAAC,2BAA2B,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;IACjD,CAAC;CACJ","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport {\n    IFluidHandle,\n    IFluidHandleContext,\n    FluidObject,\n} from \"@fluidframework/core-interfaces\";\nimport { generateHandleContextPath } from \"@fluidframework/runtime-utils\";\n\nexport class FluidObjectHandle<T extends FluidObject = FluidObject> implements IFluidHandle {\n    private readonly pendingHandlesToMakeVisible: Set<IFluidHandle> = new Set();\n    public readonly absolutePath: string;\n\n    public get IFluidHandle(): IFluidHandle { return this; }\n\n    public get isAttached(): boolean {\n        return this.routeContext.isAttached;\n    }\n\n    /**\n     * Tells whether the object of this handle is visible in the container locally or globally.\n     */\n    private get visible(): boolean {\n        /**\n         * If the object of this handle is attached, it is visible in the container. Ideally, checking local visibility\n         * should be enough for a handle. However, there are scenarios where the object becomes locally visible but the\n         * handle does not know this - This will happen is attachGraph is never called on the handle. Couple of examples\n         * where this can happen:\n         * 1. Handles to DDS other than the default handle won't know if the DDS becomes visible after the handle was\n         *    created.\n         * 2. Handles to root data stores will never know that it was visible because the handle will not be stores in\n         *    another DDS and so, attachGraph will never be called on it.\n         */\n        return this.isAttached || this.locallyVisible;\n    }\n\n    // Tracks whether this handle is locally visible in the container.\n    private locallyVisible: boolean = false;\n\n    /**\n     * Creates a new FluidObjectHandle.\n     * @param value - The FluidObject object this handle is for.\n     * @param path - The path to this handle relative to the routeContext.\n     * @param routeContext - The parent IFluidHandleContext that has a route to this handle.\n     */\n    constructor(\n        protected readonly value: T | Promise<T>,\n        public readonly path: string,\n        public readonly routeContext: IFluidHandleContext,\n    ) {\n        this.absolutePath = generateHandleContextPath(path, this.routeContext);\n    }\n\n    public async get(): Promise<any> {\n        // Note that this return works whether we received a T or a Promise<T> for this.value in the constructor.\n        return this.value;\n    }\n\n    public attachGraph(): void {\n        if (this.visible) {\n            return;\n        }\n\n        this.locallyVisible = true;\n        this.pendingHandlesToMakeVisible.forEach((handle) => {\n            handle.attachGraph();\n        });\n        this.pendingHandlesToMakeVisible.clear();\n        this.routeContext.attachGraph();\n    }\n\n    public bind(handle: IFluidHandle) {\n        // If this handle is visible, attach the graph of the incoming handle as well.\n        if (this.visible) {\n            handle.attachGraph();\n            return;\n        }\n        this.pendingHandlesToMakeVisible.add(handle);\n    }\n}\n"]}
+{"version":3,"file":"fluidHandle.js","sourceRoot":"","sources":["../src/fluidHandle.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAOH,OAAO,EAAE,yBAAyB,EAAE,MAAM,+BAA+B,CAAC;AAE1E,MAAM,OAAO,iBAAiB;IAkC1B;;;;;OAKG;IACH,YACuB,KAAqB,EACxB,IAAY,EACZ,YAAiC;QAF9B,UAAK,GAAL,KAAK,CAAgB;QACxB,SAAI,GAAJ,IAAI,CAAQ;QACZ,iBAAY,GAAZ,YAAY,CAAqB;QA1CpC,gCAA2B,GAAsB,IAAI,GAAG,EAAE,CAAC;QA4B5E;;WAEG;QACK,mBAAc,GAAY,KAAK,CAAC;QAapC,IAAI,CAAC,YAAY,GAAG,yBAAyB,CAAC,IAAI,EAAE,IAAI,CAAC,YAAY,CAAC,CAAC;IAC3E,CAAC;IA1CD,IAAW,YAAY,KAAmB,OAAO,IAAI,CAAC,CAAC,CAAC;IAExD,IAAW,UAAU;QACjB,OAAO,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC;IACxC,CAAC;IAED;;OAEG;IACH,IAAY,OAAO;QACf;;;;;;;;;;;WAWG;QACH,OAAO,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,cAAc,CAAC;IAClD,CAAC;IAqBM,KAAK,CAAC,GAAG;QACZ,yGAAyG;QACzG,OAAO,IAAI,CAAC,KAAK,CAAC;IACtB,CAAC;IAEM,WAAW;QACd,IAAI,IAAI,CAAC,OAAO,EAAE;YACd,OAAO;SACV;QAED,IAAI,CAAC,cAAc,GAAG,IAAI,CAAC;QAC3B,IAAI,CAAC,2BAA2B,CAAC,OAAO,CAAC,CAAC,MAAM,EAAE,EAAE;YAChD,MAAM,CAAC,WAAW,EAAE,CAAC;QACzB,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,2BAA2B,CAAC,KAAK,EAAE,CAAC;QACzC,IAAI,CAAC,YAAY,CAAC,WAAW,EAAE,CAAC;IACpC,CAAC;IAEM,IAAI,CAAC,MAAoB;QAC5B,8EAA8E;QAC9E,IAAI,IAAI,CAAC,OAAO,EAAE;YACd,MAAM,CAAC,WAAW,EAAE,CAAC;YACrB,OAAO;SACV;QACD,IAAI,CAAC,2BAA2B,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;IACjD,CAAC;CACJ","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport {\n    IFluidHandle,\n    IFluidHandleContext,\n    FluidObject,\n} from \"@fluidframework/core-interfaces\";\nimport { generateHandleContextPath } from \"@fluidframework/runtime-utils\";\n\nexport class FluidObjectHandle<T extends FluidObject = FluidObject> implements IFluidHandle {\n    private readonly pendingHandlesToMakeVisible: Set<IFluidHandle> = new Set();\n    public readonly absolutePath: string;\n\n    public get IFluidHandle(): IFluidHandle { return this; }\n\n    public get isAttached(): boolean {\n        return this.routeContext.isAttached;\n    }\n\n    /**\n     * Tells whether the object of this handle is visible in the container locally or globally.\n     */\n    private get visible(): boolean {\n        /**\n         * If the object of this handle is attached, it is visible in the container. Ideally, checking local visibility\n         * should be enough for a handle. However, there are scenarios where the object becomes locally visible but the\n         * handle does not know this - This will happen is attachGraph is never called on the handle. Couple of examples\n         * where this can happen:\n         *\n         * 1. Handles to DDS other than the default handle won't know if the DDS becomes visible after the handle was\n         * created.\n         *\n         * 2. Handles to root data stores will never know that it was visible because the handle will not be stores in\n         * another DDS and so, attachGraph will never be called on it.\n         */\n        return this.isAttached || this.locallyVisible;\n    }\n\n    /**\n     * Tracks whether this handle is locally visible in the container.\n     */\n    private locallyVisible: boolean = false;\n\n    /**\n     * Creates a new FluidObjectHandle.\n     * @param value - The FluidObject object this handle is for.\n     * @param path - The path to this handle relative to the routeContext.\n     * @param routeContext - The parent IFluidHandleContext that has a route to this handle.\n     */\n    constructor(\n        protected readonly value: T | Promise<T>,\n        public readonly path: string,\n        public readonly routeContext: IFluidHandleContext,\n    ) {\n        this.absolutePath = generateHandleContextPath(path, this.routeContext);\n    }\n\n    public async get(): Promise<any> {\n        // Note that this return works whether we received a T or a Promise<T> for this.value in the constructor.\n        return this.value;\n    }\n\n    public attachGraph(): void {\n        if (this.visible) {\n            return;\n        }\n\n        this.locallyVisible = true;\n        this.pendingHandlesToMakeVisible.forEach((handle) => {\n            handle.attachGraph();\n        });\n        this.pendingHandlesToMakeVisible.clear();\n        this.routeContext.attachGraph();\n    }\n\n    public bind(handle: IFluidHandle) {\n        // If this handle is visible, attach the graph of the incoming handle as well.\n        if (this.visible) {\n            handle.attachGraph();\n            return;\n        }\n        this.pendingHandlesToMakeVisible.add(handle);\n    }\n}\n"]}

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.0.0-internal.1.2.0.93071";
+export declare const pkgVersion = "2.0.0-internal.1.2.2";
 //# sourceMappingURL=packageVersion.d.ts.map

package/lib/packageVersion.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"packageVersion.d.ts","sourceRoot":"","sources":["../src/packageVersion.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,eAAO,MAAM,OAAO,8BAA8B,CAAC;AACnD,eAAO,MAAM,UAAU,+BAA+B,CAAC"}
+{"version":3,"file":"packageVersion.d.ts","sourceRoot":"","sources":["../src/packageVersion.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,eAAO,MAAM,OAAO,8BAA8B,CAAC;AACnD,eAAO,MAAM,UAAU,yBAAyB,CAAC"}

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.0.0-internal.1.2.0.93071";
+export const pkgVersion = "2.0.0-internal.1.2.2";
 //# 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,4BAA4B,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.0.0-internal.1.2.0.93071\";\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,sBAAsB,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.0.0-internal.1.2.2\";\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/datastore",
-  "version": "2.0.0-internal.1.2.0.93071",
+  "version": "2.0.0-internal.1.2.2",
   "description": "Fluid data store implementation",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -63,18 +63,18 @@
   "dependencies": {
     "@fluidframework/common-definitions": "^0.20.1",
     "@fluidframework/common-utils": "^1.0.0",
-    "@fluidframework/container-definitions": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/container-utils": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/core-interfaces": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/datastore-definitions": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/driver-definitions": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/driver-utils": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/garbage-collector": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/protocol-base": "^0.1037.2000-91174",
+    "@fluidframework/container-definitions": "^2.0.0-internal.1.2.2",
+    "@fluidframework/container-utils": "^2.0.0-internal.1.2.2",
+    "@fluidframework/core-interfaces": "^2.0.0-internal.1.2.2",
+    "@fluidframework/datastore-definitions": "^2.0.0-internal.1.2.2",
+    "@fluidframework/driver-definitions": "^2.0.0-internal.1.2.2",
+    "@fluidframework/driver-utils": "^2.0.0-internal.1.2.2",
+    "@fluidframework/garbage-collector": "^2.0.0-internal.1.2.2",
+    "@fluidframework/protocol-base": "^0.1037.2001",
     "@fluidframework/protocol-definitions": "^1.0.0",
-    "@fluidframework/runtime-definitions": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/runtime-utils": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/telemetry-utils": "2.0.0-internal.1.2.0.93071",
+    "@fluidframework/runtime-definitions": "^2.0.0-internal.1.2.2",
+    "@fluidframework/runtime-utils": "^2.0.0-internal.1.2.2",
+    "@fluidframework/telemetry-utils": "^2.0.0-internal.1.2.2",
     "lodash": "^4.17.21",
     "uuid": "^8.3.1"
   },
@@ -82,9 +82,9 @@
     "@fluidframework/build-common": "^1.0.0",
     "@fluidframework/build-tools": "^0.4.4000",
     "@fluidframework/datastore-previous": "npm:@fluidframework/datastore@^1.0.0",
-    "@fluidframework/eslint-config-fluid": "^0.28.2000",
-    "@fluidframework/mocha-test-setup": "2.0.0-internal.1.2.0.93071",
-    "@fluidframework/test-runtime-utils": "2.0.0-internal.1.2.0.93071",
+    "@fluidframework/eslint-config-fluid": "^1.0.0",
+    "@fluidframework/mocha-test-setup": "^2.0.0-internal.1.2.2",
+    "@fluidframework/test-runtime-utils": "^2.0.0-internal.1.2.2",
     "@microsoft/api-extractor": "^7.22.2",
     "@rushstack/eslint-config": "^2.5.1",
     "@types/mocha": "^9.1.1",
@@ -102,10 +102,6 @@
   },
   "typeValidation": {
     "version": "2.0.0",
-    "broken": {
-      "ClassDeclaration_FluidDataStoreRuntime": {
-        "backCompat": false
-      }
-    }
+    "broken": {}
   }
 }

package/src/dataStoreRuntime.ts

@@ -40,6 +40,7 @@ import {
     IQuorumClients,
 } from "@fluidframework/protocol-definitions";
 import {
+    BindState,
     CreateSummarizerNodeSource,
     IAttachMessage,
     IEnvelope,
@@ -154,6 +155,7 @@ IFluidDataStoreChannel, IFluidDataStoreRuntime, IFluidHandleContext {
     private readonly contextsDeferred = new Map<string, Deferred<IChannelContext>>();
     private readonly pendingAttach = new Map<string, IAttachMessage>();
 
+    private bindState: BindState;
     private readonly deferredAttached = new Deferred<void>();
     private readonly localChannelContextQueue = new Map<string, LocalChannelContextBase>();
     private readonly notBoundedChannelContextSet = new Set<string>();
@@ -264,15 +266,19 @@ IFluidDataStoreChannel, IFluidDataStoreRuntime, IFluidHandleContext {
         }
 
         this.attachListener();
+        // If exists on storage or loaded from a snapshot, it should already be bound.
+        this.bindState = existing ? BindState.Bound : BindState.NotBound;
         this._attachState = dataStoreContext.attachState;
 
         /**
          * If existing flag is false, this is a new data store and is not visible. The existing flag can be true in two
          * conditions:
+         *
          * 1. It's a local data store that is created when a detached container is rehydrated. In this case, the data
-         *    store is locally visible because the snapshot it is loaded from contains locally visible data stores only.
+         * store is locally visible because the snapshot it is loaded from contains locally visible data stores only.
+         *
          * 2. It's a remote data store that is created when an attached container is loaded is loaded from snapshot or
-         *    when an attach op comes in. In both these cases, the data store is already globally visible.
+         * when an attach op comes in. In both these cases, the data store is already globally visible.
          */
         if (existing) {
             this.visibilityState = dataStoreContext.attachState === AttachState.Detached
@@ -418,11 +424,14 @@ IFluidDataStoreChannel, IFluidDataStoreRuntime, IFluidHandleContext {
 
     /**
      * This function is called when a data store becomes root. It does the following:
+     *
      * 1. Marks the data store locally visible in the container.
+     *
      * 2. Attaches the graph of all the handles bound to it.
+     *
      * 3. Calls into the data store context to mark it visible in the container too. If the container is globally
-     *    visible, it will mark us globally visible. Otherwise, it will mark us globally visible when it becomes
-     *    globally visible.
+     * visible, it will mark us globally visible. Otherwise, it will mark us globally visible when it becomes
+     * globally visible.
      */
     public makeVisibleAndAttachGraph() {
         if (this.visibilityState !== VisibilityState.NotVisible) {
@@ -434,7 +443,7 @@ IFluidDataStoreChannel, IFluidDataStoreRuntime, IFluidHandleContext {
             handle.attachGraph();
         });
         this.pendingHandlesToMakeVisible.clear();
-        this.dataStoreContext.makeLocallyVisible();
+        this.bindToContext();
     }
 
     /**
@@ -444,6 +453,22 @@ IFluidDataStoreChannel, IFluidDataStoreRuntime, IFluidHandleContext {
         this.makeVisibleAndAttachGraph();
     }
 
+    /**
+     * @deprecated - Not necessary if consumers add a new dataStore to the container by storing its handle.
+     * Binds this runtime to the container
+     * This includes the following:
+     * 1. Sending an Attach op that includes all existing state
+     * 2. Attaching the graph if the data store becomes attached.
+     */
+    public bindToContext() {
+        if (this.bindState !== BindState.NotBound) {
+            return;
+        }
+        this.bindState = BindState.Binding;
+        this.dataStoreContext.bindToContext();
+        this.bindState = BindState.Bound;
+    }
+
     public bind(handle: IFluidHandle): void {
         // If visible, attach the incoming handle's graph. Else, this will be done when we become visible.
         if (this.visibilityState !== VisibilityState.NotVisible) {
@@ -599,11 +624,15 @@ IFluidDataStoreChannel, IFluidDataStoreRuntime, IFluidHandleContext {
      * Generates data used for garbage collection. This includes a list of GC nodes that represent this channel
      * including any of its child channel contexts. Each node has a set of outbound routes to other GC nodes in the
      * document. It does the following:
+     *
      * 1. Calls into each child context to get its GC data.
+     *
      * 2. Prefixes the child context's id to the GC nodes in the child's GC data. This makes sure that the node can be
-     *    identified as belonging to the child.
+     * identified as belonging to the child.
+     *
      * 3. Adds a GC node for this channel to the nodes received from the children. All these nodes together represent
-     *    the GC data of this channel.
+     * the GC data of this channel.
+     *
      * @param fullGC - true to bypass optimizations and force full generation of GC data.
      */
     public async getGCData(fullGC: boolean = false): Promise<IGarbageCollectionData> {
@@ -715,21 +744,21 @@ IFluidDataStoreChannel, IFluidDataStoreRuntime, IFluidHandleContext {
          * back-compat 0.59.1000 - getAttachSummary() is called when making a data store globally visible (previously
          * attaching state). Ideally, attachGraph() should have already be called making it locally visible. However,
          * before visibility state was added, this may not have been the case and getAttachSummary() could be called:
-         * 1) Before attaching the data store - When a detached container is attached.
-         * 2) After attaching the data store - When a data store is created and bound in an attached container.
+         *
+         * 1. Before attaching the data store - When a detached container is attached.
+         *
+         * 2. After attaching the data store - When a data store is created and bound in an attached container.
          *
          * The basic idea is that all local object should become locally visible before they are globally visible.
          */
         this.attachGraph();
 
-        /**
-         * This assert cannot be added now due to back-compat. To be uncommented when the following issue is fixed -
-         * https://github.com/microsoft/FluidFramework/issues/9688.
-         *
-         * assert(this.visibilityState === VisibilityState.LocallyVisible,
-         *   "The data store should be locally visible when generating attach summary",
-         * );
-         */
+        // This assert cannot be added now due to back-compat. To be uncommented when the following issue is fixed -
+        // https://github.com/microsoft/FluidFramework/issues/9688.
+        //
+        // assert(this.visibilityState === VisibilityState.LocallyVisible,
+        //  "The data store should be locally visible when generating attach summary",
+        // );
 
         const summaryBuilder = new SummaryTreeBuilder();
 

package/src/fluidHandle.ts

@@ -29,15 +29,19 @@ export class FluidObjectHandle<T extends FluidObject = FluidObject> implements I
          * 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.
+         * 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.
+         * 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.
+    /**
+     * Tracks whether this handle is locally visible in the container.
+     */
     private locallyVisible: boolean = false;
 
     /**

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/datastore";
-export const pkgVersion = "2.0.0-internal.1.2.0.93071";
+export const pkgVersion = "2.0.0-internal.1.2.2";