@fluidframework/tree 2.10.0-306579 → 2.10.0-307399

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/tree",
-  "version": "2.10.0-306579",
+  "version": "2.10.0-307399",
   "description": "Distributed tree",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -89,17 +89,17 @@
     "temp-directory": "nyc/.nyc_output"
   },
   "dependencies": {
-    "@fluid-internal/client-utils": "2.10.0-306579",
-    "@fluidframework/container-runtime": "2.10.0-306579",
-    "@fluidframework/core-interfaces": "2.10.0-306579",
-    "@fluidframework/core-utils": "2.10.0-306579",
-    "@fluidframework/datastore-definitions": "2.10.0-306579",
-    "@fluidframework/driver-definitions": "2.10.0-306579",
-    "@fluidframework/id-compressor": "2.10.0-306579",
-    "@fluidframework/runtime-definitions": "2.10.0-306579",
-    "@fluidframework/runtime-utils": "2.10.0-306579",
-    "@fluidframework/shared-object-base": "2.10.0-306579",
-    "@fluidframework/telemetry-utils": "2.10.0-306579",
+    "@fluid-internal/client-utils": "2.10.0-307399",
+    "@fluidframework/container-runtime": "2.10.0-307399",
+    "@fluidframework/core-interfaces": "2.10.0-307399",
+    "@fluidframework/core-utils": "2.10.0-307399",
+    "@fluidframework/datastore-definitions": "2.10.0-307399",
+    "@fluidframework/driver-definitions": "2.10.0-307399",
+    "@fluidframework/id-compressor": "2.10.0-307399",
+    "@fluidframework/runtime-definitions": "2.10.0-307399",
+    "@fluidframework/runtime-utils": "2.10.0-307399",
+    "@fluidframework/shared-object-base": "2.10.0-307399",
+    "@fluidframework/telemetry-utils": "2.10.0-307399",
     "@sinclair/typebox": "^0.32.29",
     "@tylerbu/sorted-btree-es6": "^1.8.0",
     "@types/ungap__structured-clone": "^1.2.0",
@@ -109,19 +109,19 @@
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.16.4",
     "@biomejs/biome": "~1.9.3",
-    "@fluid-internal/mocha-test-setup": "2.10.0-306579",
-    "@fluid-private/stochastic-test-utils": "2.10.0-306579",
-    "@fluid-private/test-dds-utils": "2.10.0-306579",
-    "@fluid-private/test-drivers": "2.10.0-306579",
+    "@fluid-internal/mocha-test-setup": "2.10.0-307399",
+    "@fluid-private/stochastic-test-utils": "2.10.0-307399",
+    "@fluid-private/test-dds-utils": "2.10.0-307399",
+    "@fluid-private/test-drivers": "2.10.0-307399",
     "@fluid-tools/benchmark": "^0.50.0",
-    "@fluid-tools/build-cli": "^0.50.0",
+    "@fluid-tools/build-cli": "^0.51.0",
     "@fluidframework/build-common": "^2.0.3",
-    "@fluidframework/build-tools": "^0.50.0",
-    "@fluidframework/container-definitions": "2.10.0-306579",
-    "@fluidframework/container-loader": "2.10.0-306579",
+    "@fluidframework/build-tools": "^0.51.0",
+    "@fluidframework/container-definitions": "2.10.0-307399",
+    "@fluidframework/container-loader": "2.10.0-307399",
     "@fluidframework/eslint-config-fluid": "^5.4.0",
-    "@fluidframework/test-runtime-utils": "2.10.0-306579",
-    "@fluidframework/test-utils": "2.10.0-306579",
+    "@fluidframework/test-runtime-utils": "2.10.0-307399",
+    "@fluidframework/test-utils": "2.10.0-307399",
     "@fluidframework/tree-previous": "npm:@fluidframework/tree@2.5.0",
     "@microsoft/api-extractor": "7.47.8",
     "@types/diff": "^3.5.1",
@@ -163,15 +163,7 @@
     }
   },
   "typeValidation": {
-    "broken": {
-      "TypeAlias_Input": {
-        "backCompat": false,
-        "forwardCompat": false
-      },
-      "Interface_InternalTypes_TreeArrayNodeBase": {
-        "backCompat": false
-      }
-    },
+    "broken": {},
     "entrypoint": "public"
   },
   "scripts": {

package/src/feature-libraries/chunked-forest/basicChunk.ts

@@ -30,14 +30,22 @@ export class BasicChunk extends ReferenceCountedBase implements TreeChunk {
 	/**
 	 * Create a tree chunk with ref count 1.
 	 *
-	 * @param fields - provides exclusive deep ownership of this map to this object (which might mutate it in the future).
-	 * The caller must have already accounted for this reference to the children in this map (via `referenceAdded`),
-	 * and any edits to this must update child reference counts.
-	 * @param value - the value on this node, if any.
+	 * Caller must have already accounted for references via `fields` to the children in the fields map (via `referenceAdded`).
 	 */
 	public constructor(
 		public type: TreeNodeSchemaIdentifier,
+		/**
+		 * Fields of this node.
+		 * @remarks
+		 * This object has exclusive deep ownership of this map (which might mutate it in the future).
+		 * Any code editing this map must update child reference counts.
+		 *
+		 * Like with {@link MapTree}, fields with no nodes must be removed from the map.
+		 */
 		public fields: Map<FieldKey, TreeChunk[]>,
+		/**
+		 * The value on this node, if any.
+		 */
 		public value?: TreeValue,
 	) {
 		super();

package/src/feature-libraries/chunked-forest/chunkedForest.ts

@@ -195,6 +195,11 @@ export class ChunkedForest implements IEditableForest {
 					newContentSource !== oldContentDestination,
 					0x7b0 /* Replace detached source field and detached destination field must be different */,
 				);
+				// TODO: optimize this to: perform in-place replace in uniform chunks when possible.
+				// This should result in 3 cases:
+				// 1. In-place update of uniform chunk. No allocations, no ref count changes, no new TreeChunks.
+				// 2. Uniform chunk is shared: copy it (and parent path as needed), and update the copy.
+				// 3. Fallback to detach then attach (Which will copy parents and convert to basic chunks as needed).
 				this.detachEdit(range, oldContentDestination);
 				this.attachEdit(newContentSource, range.end - range.start, range.start);
 			},

package/src/feature-libraries/index.ts

@@ -74,7 +74,7 @@ export {
 	type FieldChangeEncodingContext,
 	type FieldKindConfiguration,
 	type FieldKindConfigurationEntry,
-	getAllowedContentIncompatibilities,
+	getAllowedContentDiscrepancies,
 	isRepoSuperset,
 	isNeverTree,
 } from "./modular-schema/index.js";