@fluidframework/tree 2.81.0-374083 → 2.81.1

package/src/feature-libraries/modular-schema/modularChangeFormatV1.ts

@@ -134,7 +134,6 @@ export const EncodedModularChangesetV1 = Type.Object(
 		// This can lead to a less efficient wire representation because of duplicated schema/shape information.
 		builds: Type.Optional(EncodedBuilds),
 		refreshers: Type.Optional(EncodedBuilds),
-
 		/**
 		 * The number of constraints within this changeset that are violated.
 		 */

package/src/feature-libraries/modular-schema/modularChangeTypes.ts

@@ -6,7 +6,6 @@
 import {
 	subtractChangeAtomIds,
 	type ChangeAtomId,
-	type ChangeAtomIdRangeMap,
 	type ChangesetLocalId,
 	type FieldKey,
 	type FieldKindIdentifier,
@@ -18,11 +17,7 @@ import type { TreeChunk } from "../chunked-forest/index.js";
 
 import type { CrossFieldTarget } from "./crossFieldQueries.js";
 
-export type RebaseVersion = 1 | 2;
-
 export interface ModularChangeset extends HasFieldChanges {
-	readonly rebaseVersion: RebaseVersion;
-
 	/**
 	 * The numerically highest `ChangesetLocalId` used in this changeset.
 	 * If undefined then this changeset contains no IDs.
@@ -40,14 +35,12 @@ export interface ModularChangeset extends HasFieldChanges {
 	 * Maps from this changeset's canonical ID for a node (see comment on node aliases) to the changes for that node.
 	 */
 	readonly nodeChanges: ChangeAtomIdBTree<NodeChangeset>;
-	readonly rootNodes: RootNodeTable;
 
 	/**
-	 * Maps from this changeset's canonical ID for a node to the ID for the field which contains that node,
-	 * or the detach ID, if this node is detached.
+	 * Maps from this changeset's canonical ID for a node to the ID for the field which contains that node.
 	 */
 	// TODO: Should this be merged with `nodeChanges`?
-	readonly nodeToParent: ChangeAtomIdBTree<NodeLocation>;
+	readonly nodeToParent: ChangeAtomIdBTree<FieldId>;
 
 	/**
 	 * Maps from a node ID to another ID for the same node.
@@ -81,31 +74,10 @@ export interface ModularChangeset extends HasFieldChanges {
 	readonly refreshers?: ChangeAtomIdBTree<TreeChunk>;
 }
 
-export interface RootNodeTable {
-	// TODO: Include builds, destroys, refreshers, and field changes
-	oldToNewId: ChangeAtomIdRangeMap<ChangeAtomId>;
-	newToOldId: ChangeAtomIdRangeMap<ChangeAtomId>;
-	nodeChanges: ChangeAtomIdBTree<NodeId>;
-
-	/**
-	 * Maps from input context detach ID to the field where the node was last attached.
-	 * There should be an entry for every detach ID referenced in `oldToNewId` or `nodeChanges`.
-	 */
-	detachLocations: ChangeAtomIdRangeMap<FieldId>;
-
-	/**
-	 * Maps from the output root ID of a node to the output detach location of that node.
-	 * This is only guaranteed to contain entries for nodes which have an output detach location
-	 * which is different from their location in the input context.
-	 */
-	outputDetachLocations: ChangeAtomIdRangeMap<FieldId>;
-}
-
-export type CrossFieldRangeTable<T> = RangeMap<CrossFieldKey, T>;
-export type CrossFieldKeyTable = CrossFieldRangeTable<FieldId>;
+export type CrossFieldKeyTable = RangeMap<CrossFieldKey, FieldId>;
 
-export function newCrossFieldRangeTable<V>(): CrossFieldRangeTable<V> {
-	return new RangeMap<CrossFieldKey, V>(offsetCrossFieldKey, subtractCrossFieldKeys);
+export function newCrossFieldKeyTable(): CrossFieldKeyTable {
+	return new RangeMap(offsetCrossFieldKey, subtractCrossFieldKeys);
 }
 
 function offsetCrossFieldKey(key: CrossFieldKey, offset: number): CrossFieldKey {
@@ -138,24 +110,6 @@ export interface FieldId {
 	readonly field: FieldKey;
 }
 
-export interface FieldParent {
-	readonly field: FieldId;
-	readonly root?: undefined;
-}
-
-export interface RootParent {
-	readonly field?: undefined;
-	readonly root: ChangeAtomId;
-}
-
-export type NodeLocation = FieldParent | RootParent;
-export interface DetachLocation {
-	field: FieldId;
-	atomId: ChangeAtomId | undefined;
-}
-
-/**
- */
 export interface NodeExistsConstraint {
 	violated: boolean;
 }
@@ -172,16 +126,15 @@ export interface NoChangeConstraint {
  */
 export interface NodeChangeset extends HasFieldChanges {
 	/** Keeps track of whether node exists constraint has been violated by this change */
-	readonly nodeExistsConstraint?: NodeExistsConstraint;
-
+	nodeExistsConstraint?: NodeExistsConstraint;
 	/** Keeps track of whether node exists constraint will be violated when this change is reverted */
-	readonly nodeExistsConstraintOnRevert?: NodeExistsConstraint;
+	nodeExistsConstraintOnRevert?: NodeExistsConstraint;
 }
 
 export type NodeId = ChangeAtomId;
 
 export interface HasFieldChanges {
-	readonly fieldChanges?: FieldChangeMap;
+	fieldChanges?: FieldChangeMap;
 }
 
 export type FieldChangeMap = Map<FieldKey, FieldChange>;

package/src/feature-libraries/object-forest/objectForest.ts

@@ -50,10 +50,10 @@ import {
 	type WithBreakable,
 } from "../../util/index.js";
 import { chunkField, defaultChunkPolicy } from "../chunked-forest/index.js";
-import { cursorForMapTreeNode, mapTreeFromCursor } from "../mapTreeCursor.js";
-import { type CursorWithNode, SynchronousCursor } from "../treeCursorUtils.js";
 import { defaultSchemaPolicy, FieldKinds } from "../default-schema/index.js";
+import { cursorForMapTreeNode, mapTreeFromCursor } from "../mapTreeCursor.js";
 import { isFieldInSchema, throwOutOfSchema } from "../schemaChecker.js";
+import { type CursorWithNode, SynchronousCursor } from "../treeCursorUtils.js";
 
 /** A `MapTree` with mutable fields */
 interface MutableMapTree extends MapTree {

package/src/feature-libraries/optional-field/index.ts

@@ -3,12 +3,12 @@
  * Licensed under the MIT License.
  */
 
-export { makeOptionalFieldCodecFamily } from "./optionalFieldCodecs.js";
 export type { OptionalChangeset } from "./optionalFieldChangeTypes.js";
 export {
-	optionalChangeHandler,
-	optionalFieldEditor,
-	optionalChangeRebaser,
-	optionalFieldIntoDelta,
 	type OptionalFieldEditor,
+	optional,
 } from "./optionalField.js";
+export {
+	required,
+	type RequiredFieldEditor,
+} from "./requiredField.js";