@fluidframework/tree 2.80.0 → 2.81.0-374083

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

@@ -3,68 +3,85 @@
  * Licensed under the MIT License.
  */
 
-import { assert, oob, fail } from "@fluidframework/core-utils/internal";
-import type { TAnySchema } from "@sinclair/typebox";
-
+import { assert, fail, oob } from "@fluidframework/core-utils/internal";
 import {
+	extractJsonValidator,
+	withSchemaValidation,
 	type ICodecOptions,
 	type IJsonCodec,
 	type IMultiFormatCodec,
 	type SchemaValidationFunction,
-	extractJsonValidator,
-	withSchemaValidation,
 } from "../../codec/index.js";
-import type {
-	ChangeEncodingContext,
-	ChangesetLocalId,
-	EncodedRevisionTag,
-	FieldKey,
-	FieldKindIdentifier,
-	ITreeCursorSynchronous,
-	RevisionInfo,
-	RevisionTag,
+import {
+	newChangeAtomIdTransform,
+	type ChangeAtomId,
+	type ChangeAtomIdRangeMap,
+	type ChangeEncodingContext,
+	type ChangesetLocalId,
+	type EncodedRevisionTag,
+	type FieldKey,
+	type FieldKindIdentifier,
+	type ITreeCursorSynchronous,
+	type RevisionInfo,
+	type RevisionTag,
 } from "../../core/index.js";
 import {
-	type IdAllocator,
-	type JsonCompatibleReadOnly,
-	type Mutable,
 	brand,
 	idAllocatorFromMaxId,
 	newTupleBTree,
+	type IdAllocator,
+	type JsonCompatibleReadOnly,
+	type Mutable,
+	type RangeQueryEntry,
+	type RangeQueryResult,
+	type TupleBTree,
 } from "../../util/index.js";
 import {
-	type FieldBatchCodec,
-	type TreeChunk,
 	chunkFieldSingle,
 	defaultChunkPolicy,
+	type FieldBatchCodec,
+	type TreeChunk,
 } from "../chunked-forest/index.js";
 import { TreeCompressionStrategy } from "../treeCompressionUtils.js";
-
-import type { FieldChangeEncodingContext, FieldChangeHandler } from "./fieldChangeHandler.js";
 import type {
 	FieldKindConfiguration,
 	FieldKindConfigurationEntry,
 } from "./fieldKindConfiguration.js";
-import { genericFieldKind } from "./genericFieldKind.js";
 import {
-	type EncodedBuilds,
-	type EncodedBuildsArray,
-	type EncodedFieldChange,
-	type EncodedFieldChangeMap,
-	EncodedModularChangesetV1,
-	type EncodedNodeChangeset,
-	type EncodedRevisionInfo,
-} from "./modularChangeFormatV1.js";
+	addNodeRename,
+	getFirstAttachField,
+	getFirstDetachField,
+	newRootTable,
+	normalizeFieldId,
+	validateChangeset,
+	type FieldIdKey,
+} from "./modularChangeFamily.js";
 import {
-	type FieldChangeset,
-	newCrossFieldKeyTable,
+	newCrossFieldRangeTable,
+	type CrossFieldKeyTable,
 	type FieldChangeMap,
+	type FieldChangeset,
 	type FieldId,
 	type ModularChangeset,
 	type NodeChangeset,
 	type NodeId,
+	type NodeLocation,
+	type RootNodeTable,
 } from "./modularChangeTypes.js";
-import type { ChangeAtomIdBTree } from "../changeAtomIdBTree.js";
+import type {
+	EncodedBuilds,
+	EncodedBuildsArray,
+	EncodedFieldChange,
+	EncodedFieldChangeMap,
+	EncodedNodeChangeset,
+	EncodedRevisionInfo,
+} from "./modularChangeFormatV1.js";
+import { EncodedModularChangesetV1 } from "./modularChangeFormatV1.js";
+import type { FieldChangeEncodingContext, FieldChangeHandler } from "./fieldChangeHandler.js";
+import { genericFieldKind } from "./genericFieldKind.js";
+import type { TAnySchema } from "@sinclair/typebox";
+import { setInChangeAtomIdMap, type ChangeAtomIdBTree } from "../changeAtomIdBTree.js";
+import type { FlexFieldKind } from "./fieldKind.js";
 
 type ModularChangeCodec = IJsonCodec<
 	ModularChangeset,
@@ -80,6 +97,13 @@ type FieldCodec = IMultiFormatCodec<
 	FieldChangeEncodingContext
 >;
 
+interface FieldRootChanges {
+	readonly nodeChanges: ChangeAtomIdBTree<NodeId>;
+	readonly renames: ChangeAtomIdRangeMap<ChangeAtomId>;
+}
+
+type FieldRootMap = TupleBTree<FieldIdKey, FieldRootChanges>;
+
 type FieldChangesetCodecs = Map<
 	FieldKindIdentifier,
 	{
@@ -88,42 +112,64 @@ type FieldChangesetCodecs = Map<
 	}
 >;
 
-export function getFieldChangesetCodec(
-	fieldKind: FieldKindIdentifier,
-	fieldChangesetCodecs: FieldChangesetCodecs,
-): {
-	compiledSchema?: SchemaValidationFunction<TAnySchema>;
-	codec: FieldCodec;
-} {
-	const entry = fieldChangesetCodecs.get(fieldKind);
-	assert(entry !== undefined, 0x5ea /* Tried to encode unsupported fieldKind */);
-	return entry;
-}
-
-export function encodeFieldChangesForJson(
-	change: FieldChangeMap,
-	context: ChangeEncodingContext,
-	nodeChanges: ChangeAtomIdBTree<NodeChangeset>,
-	fieldChangesetCodecs: FieldChangesetCodecs,
-): EncodedFieldChangeMap {
-	const fieldContext: FieldChangeEncodingContext = {
-		baseContext: context,
+export function getFieldChangesetCodecs(
+	fieldKinds: FieldKindConfiguration,
+	revisionTagCodec: IJsonCodec<
+		RevisionTag,
+		EncodedRevisionTag,
+		EncodedRevisionTag,
+		ChangeEncodingContext
+	>,
+	codecOptions: ICodecOptions,
+): Map<
+	FieldKindIdentifier,
+	{ compiledSchema?: SchemaValidationFunction<TAnySchema>; codec: FieldCodec }
+> {
+	// eslint-disable-next-line @typescript-eslint/explicit-function-return-type
+	const getMapEntry = ({ kind, formatVersion }: FieldKindConfigurationEntry) => {
+		const codec = kind.changeHandler.codecsFactory(revisionTagCodec).resolve(formatVersion);
+		return {
+			codec,
+			compiledSchema: codec.json.encodedSchema
+				? extractJsonValidator(codecOptions.jsonValidator).compile(codec.json.encodedSchema)
+				: undefined,
+		};
+	};
 
-		encodeNode: (nodeId: NodeId): EncodedNodeChangeset => {
-			const node = nodeChanges.get([nodeId.revision, nodeId.localId]);
-			assert(node !== undefined, 0x92e /* Unknown node ID */);
-			return encodeNodeChangesForJson(node, fieldContext, fieldChangesetCodecs);
-		},
+	/**
+	 * The codec version for the generic field kind.
+	 */
+	const genericFieldKindFormatVersion = 1;
+	const fieldChangesetCodecs: Map<
+		FieldKindIdentifier,
+		{
+			compiledSchema?: SchemaValidationFunction<TAnySchema>;
+			codec: FieldCodec;
+		}
+	> = new Map([
+		[
+			genericFieldKind.identifier,
+			getMapEntry({ kind: genericFieldKind, formatVersion: genericFieldKindFormatVersion }),
+		],
+	]);
 
-		decodeNode: () => fail(0xb1e /* Should not decode nodes during field encoding */),
-	};
+	// eslint-disable-next-line unicorn/no-array-for-each -- Map.forEach with (value, key) signature; codec initialization
+	fieldKinds.forEach((entry, identifier) => {
+		fieldChangesetCodecs.set(identifier, getMapEntry(entry));
+	});
 
-	return encodeFieldChangesForJsonI(change, fieldContext, fieldChangesetCodecs);
+	return fieldChangesetCodecs;
 }
 
-export function encodeFieldChangesForJsonI(
+function encodeFieldChangesForJson(
 	change: FieldChangeMap,
-	context: FieldChangeEncodingContext,
+	parentId: NodeId | undefined,
+	fieldToRoots: FieldRootMap,
+	context: ChangeEncodingContext,
+	encodeNode: NodeEncoder,
+	getInputRootId: ChangeAtomMappingQuery,
+	isAttachId: ChangeAtomIdRangeQuery,
+	isDetachId: ChangeAtomIdRangeQuery,
 	fieldChangesetCodecs: FieldChangesetCodecs,
 ): EncodedFieldChangeMap {
 	const encodedFields: EncodedFieldChangeMap = [];
@@ -133,7 +179,25 @@ export function encodeFieldChangesForJsonI(
 			fieldChange.fieldKind,
 			fieldChangesetCodecs,
 		);
-		const encodedChange = codec.json.encode(fieldChange.change, context);
+		const rootChanges = fieldToRoots.get([parentId?.revision, parentId?.localId, field]);
+
+		const fieldContext: FieldChangeEncodingContext = {
+			baseContext: context,
+			rootNodeChanges: rootChanges?.nodeChanges ?? newTupleBTree(),
+			rootRenames: rootChanges?.renames ?? newChangeAtomIdTransform(),
+
+			encodeNode,
+			getInputRootId,
+			isAttachId,
+			isDetachId,
+			decodeNode: () => fail(0xb1e /* Should not decode nodes during field encoding */),
+			decodeRootNodeChange: () => fail("Should not be called during encoding"),
+			decodeRootRename: () => fail("Should not be called during encoding"),
+			decodeMoveAndDetach: () => fail("Should not be called during encoding"),
+			generateId: () => fail("Should not be called during encoding"),
+		};
+
+		const encodedChange = codec.json.encode(fieldChange.change, fieldContext);
 		if (compiledSchema !== undefined && !compiledSchema.check(encodedChange)) {
 			fail(0xb1f /* Encoded change didn't pass schema validation. */);
 		}
@@ -151,9 +215,24 @@ export function encodeFieldChangesForJsonI(
 	return encodedFields;
 }
 
-export function encodeNodeChangesForJson(
+type ChangeAtomMappingQuery = (
+	id: ChangeAtomId,
+	count: number,
+) => RangeQueryResult<ChangeAtomId | undefined>;
+
+type ChangeAtomIdRangeQuery = (id: ChangeAtomId, count: number) => RangeQueryResult<boolean>;
+type NodeEncoder = (nodeId: NodeId) => EncodedNodeChangeset;
+type NodeDecoder = (encoded: EncodedNodeChangeset, fieldId: NodeLocation) => NodeId;
+
+function encodeNodeChangesForJson(
 	change: NodeChangeset,
-	context: FieldChangeEncodingContext,
+	id: NodeId,
+	fieldToRoots: FieldRootMap,
+	context: ChangeEncodingContext,
+	encodeNode: NodeEncoder,
+	getInputRootId: ChangeAtomMappingQuery,
+	isAttachId: ChangeAtomIdRangeQuery,
+	isDetachId: ChangeAtomIdRangeQuery,
 	fieldChangesetCodecs: FieldChangesetCodecs,
 ): EncodedNodeChangeset {
 	const encodedChange: EncodedNodeChangeset = {};
@@ -161,9 +240,15 @@ export function encodeNodeChangesForJson(
 	const { fieldChanges, nodeExistsConstraint } = change;
 
 	if (fieldChanges !== undefined) {
-		encodedChange.fieldChanges = encodeFieldChangesForJsonI(
+		encodedChange.fieldChanges = encodeFieldChangesForJson(
 			fieldChanges,
+			id,
+			fieldToRoots,
 			context,
+			encodeNode,
+			getInputRootId,
+			isAttachId,
+			isDetachId,
 			fieldChangesetCodecs,
 		);
 	}
@@ -175,11 +260,25 @@ export function encodeNodeChangesForJson(
 	return encodedChange;
 }
 
-export function decodeFieldChangesFromJson(
+function getFieldChangesetCodec(
+	fieldKind: FieldKindIdentifier,
+	fieldChangesetCodecs: FieldChangesetCodecs,
+): {
+	compiledSchema?: SchemaValidationFunction<TAnySchema>;
+	codec: FieldCodec;
+} {
+	const entry = fieldChangesetCodecs.get(fieldKind);
+	assert(entry !== undefined, 0x5ea /* Tried to encode unsupported fieldKind */);
+	return entry;
+}
+
+function decodeFieldChangesFromJson(
 	encodedChange: EncodedFieldChangeMap,
 	parentId: NodeId | undefined,
-	decoded: ModularChangeset,
+	decodedCrossFieldKeys: CrossFieldKeyTable,
+	decodedRootTable: RootNodeTable,
 	context: ChangeEncodingContext,
+	decodeNode: NodeDecoder,
 	idAllocator: IdAllocator,
 	fieldKinds: FieldKindConfiguration,
 	fieldChangesetCodecs: FieldChangesetCodecs,
@@ -201,29 +300,39 @@ export function decodeFieldChangesFromJson(
 
 		const fieldContext: FieldChangeEncodingContext = {
 			baseContext: context,
+			rootNodeChanges: newTupleBTree(),
+			rootRenames: newChangeAtomIdTransform(),
 
 			encodeNode: () => fail(0xb21 /* Should not encode nodes during field decoding */),
+			getInputRootId: () => fail("Should not query during decoding"),
+			isAttachId: () => fail("Should not query during decoding"),
+			isDetachId: () => fail("Should not query during decoding"),
 
 			decodeNode: (encodedNode: EncodedNodeChangeset): NodeId => {
-				const nodeId: NodeId = {
-					revision: context.revision,
-					localId: brand(idAllocator.allocate()),
-				};
-
-				const node = decodeNodeChangesetFromJson(
-					encodedNode,
-					nodeId,
-					decoded,
-					context,
-					idAllocator,
-					fieldKinds,
-					fieldChangesetCodecs,
+				return decodeNode(encodedNode, { field: fieldId });
+			},
+
+			decodeRootNodeChange: (detachId, encodedNode): void => {
+				setInChangeAtomIdMap(
+					decodedRootTable.nodeChanges,
+					detachId,
+					decodeNode(encodedNode, { root: detachId }),
 				);
+				decodedRootTable.detachLocations.set(detachId, 1, fieldId);
+			},
+
+			decodeRootRename: (oldId, newId, count): void => {
+				addNodeRename(decodedRootTable, oldId, newId, count, fieldId);
+			},
 
-				decoded.nodeChanges.set([nodeId.revision, nodeId.localId], node);
-				decoded.nodeToParent.set([nodeId.revision, nodeId.localId], fieldId);
-				return nodeId;
+			decodeMoveAndDetach: (detachId, count): void => {
+				decodedRootTable.outputDetachLocations.set(detachId, count, fieldId);
 			},
+
+			generateId: (): ChangeAtomId => ({
+				revision: context.revision,
+				localId: brand(idAllocator.allocate()),
+			}),
 		};
 
 		const fieldChangeset = codec.json.decode(field.change, fieldContext);
@@ -233,7 +342,7 @@ export function decodeFieldChangesFromJson(
 		);
 
 		for (const { key, count } of crossFieldKeys) {
-			decoded.crossFieldKeys.set(key, count, fieldId);
+			decodedCrossFieldKeys.set(key, count, fieldId);
 		}
 
 		const fieldKey: FieldKey = brand<FieldKey>(field.fieldKey);
@@ -247,24 +356,28 @@ export function decodeFieldChangesFromJson(
 	return decodedFields;
 }
 
-export function decodeNodeChangesetFromJson(
+function decodeNodeChangesetFromJson(
 	encodedChange: EncodedNodeChangeset,
 	id: NodeId,
-	decoded: ModularChangeset,
+	decodedCrossFieldKeys: CrossFieldKeyTable,
+	decodedRootTable: RootNodeTable,
 	context: ChangeEncodingContext,
+	decodeNode: NodeDecoder,
 	idAllocator: IdAllocator,
 	fieldKinds: FieldKindConfiguration,
 	fieldChangesetCodecs: FieldChangesetCodecs,
 ): NodeChangeset {
-	const decodedChange: NodeChangeset = {};
+	const decodedChange: Mutable<NodeChangeset> = {};
 	const { fieldChanges, nodeExistsConstraint } = encodedChange;
 
 	if (fieldChanges !== undefined) {
 		decodedChange.fieldChanges = decodeFieldChangesFromJson(
 			fieldChanges,
 			id,
-			decoded,
+			decodedCrossFieldKeys,
+			decodedRootTable,
 			context,
+			decodeNode,
 			idAllocator,
 			fieldKinds,
 			fieldChangesetCodecs,
@@ -278,64 +391,7 @@ export function decodeNodeChangesetFromJson(
 	return decodedChange;
 }
 
-export function encodeDetachedNodes(
-	detachedNodes: ChangeAtomIdBTree<TreeChunk> | undefined,
-	context: ChangeEncodingContext,
-	revisionTagCodec: IJsonCodec<
-		RevisionTag,
-		EncodedRevisionTag,
-		EncodedRevisionTag,
-		ChangeEncodingContext
-	>,
-	fieldsCodec: FieldBatchCodec,
-	chunkCompressionStrategy: TreeCompressionStrategy,
-): EncodedBuilds | undefined {
-	if (detachedNodes === undefined) {
-		return undefined;
-	}
-
-	const treesToEncode: ITreeCursorSynchronous[] = [];
-	const buildsArray: EncodedBuildsArray = [];
-
-	let buildsForRevision:
-		| [[ChangesetLocalId, number][], EncodedRevisionTag]
-		| [[ChangesetLocalId, number][]]
-		| undefined;
-
-	for (const [[revision, id], chunk] of detachedNodes.entries()) {
-		const encodedRevision = encodeRevisionOpt(revisionTagCodec, revision, context);
-
-		if (buildsForRevision === undefined || buildsForRevision[1] !== encodedRevision) {
-			if (buildsForRevision !== undefined) {
-				buildsArray.push(buildsForRevision);
-			}
-
-			buildsForRevision = encodedRevision === undefined ? [[]] : [[], encodedRevision];
-		}
-
-		treesToEncode.push(chunk.cursor());
-		const treeIndexInBatch = treesToEncode.length - 1;
-		buildsForRevision?.[0].push([id, treeIndexInBatch]);
-	}
-
-	if (buildsForRevision !== undefined) {
-		buildsArray.push(buildsForRevision);
-	}
-
-	return buildsArray.length === 0
-		? undefined
-		: {
-				builds: buildsArray,
-				trees: fieldsCodec.encode(treesToEncode, {
-					encodeType: chunkCompressionStrategy,
-					schema: context.schema,
-					originatorId: context.originatorId,
-					idCompressor: context.idCompressor,
-				}),
-			};
-}
-
-export function decodeDetachedNodes(
+function decodeDetachedNodes(
 	encoded: EncodedBuilds | undefined,
 	context: ChangeEncodingContext,
 	revisionTagCodec: IJsonCodec<
@@ -365,6 +421,7 @@ export function decodeDetachedNodes(
 	};
 
 	const map: ModularChangeset["builds"] = newTupleBTree();
+	// eslint-disable-next-line unicorn/no-array-for-each -- Codec internals: minimizing changes to serialization logic
 	encoded.builds.forEach((build) => {
 		// EncodedRevisionTag cannot be an array so this ensures that we can isolate the tuple
 		const revision =
@@ -383,45 +440,7 @@ export function decodeDetachedNodes(
 	return map;
 }
 
-export function encodeRevisionInfos(
-	revisions: readonly RevisionInfo[],
-	context: ChangeEncodingContext,
-	revisionTagCodec: IJsonCodec<
-		RevisionTag,
-		EncodedRevisionTag,
-		EncodedRevisionTag,
-		ChangeEncodingContext
-	>,
-): EncodedRevisionInfo[] | undefined {
-	if (context.revision !== undefined) {
-		assert(
-			revisions.length === 1 &&
-				revisions[0] !== undefined &&
-				revisions[0].revision === context.revision &&
-				revisions[0].rollbackOf === undefined,
-			0x964 /* A tagged change should only contain the tagged revision */,
-		);
-
-		return undefined;
-	}
-
-	const encodedRevisions = [];
-	for (const revision of revisions) {
-		const encodedRevision: Mutable<EncodedRevisionInfo> = {
-			revision: revisionTagCodec.encode(revision.revision, context),
-		};
-
-		if (revision.rollbackOf !== undefined) {
-			encodedRevision.rollbackOf = revisionTagCodec.encode(revision.rollbackOf, context);
-		}
-
-		encodedRevisions.push(encodedRevision);
-	}
-
-	return encodedRevisions;
-}
-
-export function decodeRevisionInfos(
+function decodeRevisionInfos(
 	revisions: readonly EncodedRevisionInfo[] | undefined,
 	context: ChangeEncodingContext,
 	revisionTagCodec: IJsonCodec<
@@ -451,16 +470,8 @@ export function decodeRevisionInfos(
 	return decodedRevisions;
 }
 
-export function encodeChange(
-	change: ModularChangeset,
-	context: ChangeEncodingContext,
-	fieldChangesetCodecs: Map<
-		FieldKindIdentifier,
-		{
-			compiledSchema?: SchemaValidationFunction<TAnySchema>;
-			codec: FieldCodec;
-		}
-	>,
+export function makeModularChangeCodecV1(
+	fieldKinds: FieldKindConfiguration,
 	revisionTagCodec: IJsonCodec<
 		RevisionTag,
 		EncodedRevisionTag,
@@ -468,51 +479,49 @@ export function encodeChange(
 		ChangeEncodingContext
 	>,
 	fieldsCodec: FieldBatchCodec,
-	chunkCompressionStrategy: TreeCompressionStrategy,
-): EncodedModularChangesetV1 {
-	// Destroys only exist in rollback changesets, which are never sent.
-	assert(change.destroys === undefined, 0x899 /* Unexpected changeset with destroys */);
-	return {
-		maxId: change.maxId,
-		revisions:
-			change.revisions === undefined
-				? undefined
-				: encodeRevisionInfos(change.revisions, context, revisionTagCodec),
-		changes: encodeFieldChangesForJson(
-			change.fieldChanges,
-			context,
-			change.nodeChanges,
-			fieldChangesetCodecs,
-		),
-		builds: encodeDetachedNodes(
-			change.builds,
-			context,
-			revisionTagCodec,
-			fieldsCodec,
-			chunkCompressionStrategy,
-		),
-		refreshers: encodeDetachedNodes(
-			change.refreshers,
-			context,
-			revisionTagCodec,
-			fieldsCodec,
-			chunkCompressionStrategy,
-		),
-		violations: change.constraintViolationCount,
+	codecOptions: ICodecOptions,
+	chunkCompressionStrategy: TreeCompressionStrategy = TreeCompressionStrategy.Compressed,
+): ModularChangeCodec {
+	const fieldChangesetCodecs = getFieldChangesetCodecs(
+		fieldKinds,
+		revisionTagCodec,
+		codecOptions,
+	);
+
+	const modularChangeCodec: ModularChangeCodec = {
+		encode: (change, context) =>
+			encodeChange(
+				change,
+				context,
+				fieldChangesetCodecs,
+				revisionTagCodec,
+				fieldsCodec,
+				chunkCompressionStrategy,
+			),
+
+		decode: (encodedChange, context) =>
+			decodeChange(
+				encodedChange,
+				context,
+				fieldKinds,
+				fieldChangesetCodecs,
+				revisionTagCodec,
+				fieldsCodec,
+				chunkCompressionStrategy,
+			),
 	};
+
+	return withSchemaValidation(
+		EncodedModularChangesetV1,
+		modularChangeCodec,
+		codecOptions.jsonValidator,
+	);
 }
 
-export function decodeChange(
-	encodedChange: EncodedModularChangesetV1,
+export function encodeChange(
+	change: ModularChangeset,
 	context: ChangeEncodingContext,
-	fieldKinds: FieldKindConfiguration,
-	fieldChangesetCodecs: Map<
-		FieldKindIdentifier,
-		{
-			compiledSchema?: SchemaValidationFunction<TAnySchema>;
-			codec: FieldCodec;
-		}
-	>,
+	fieldChangesetCodecs: FieldChangesetCodecs,
 	revisionTagCodec: IJsonCodec<
 		RevisionTag,
 		EncodedRevisionTag,
@@ -521,112 +530,127 @@ export function decodeChange(
 	>,
 	fieldsCodec: FieldBatchCodec,
 	chunkCompressionStrategy: TreeCompressionStrategy,
-): Mutable<ModularChangeset> {
-	const decoded: Mutable<ModularChangeset> = {
-		fieldChanges: new Map(),
-		nodeChanges: newTupleBTree(),
-		nodeToParent: newTupleBTree(),
-		nodeAliases: newTupleBTree(),
-		crossFieldKeys: newCrossFieldKeyTable(),
+): EncodedModularChangesetV1 {
+	const fieldToRoots = getFieldToRoots(change.rootNodes, change.nodeAliases);
+	const isAttachId = (id: ChangeAtomId, count: number): RangeQueryResult<boolean> => {
+		const attachEntry = getFirstAttachField(change.crossFieldKeys, id, count);
+		return { ...attachEntry, value: attachEntry.value !== undefined };
 	};
 
-	decoded.fieldChanges = decodeFieldChangesFromJson(
-		encodedChange.changes,
-		undefined,
-		decoded,
-		context,
-		idAllocatorFromMaxId(encodedChange.maxId),
-		fieldKinds,
-		fieldChangesetCodecs,
-	);
+	const isDetachId = (
+		id: ChangeAtomId,
+		count: number,
+	): RangeQueryEntry<ChangeAtomId, boolean> => {
+		const detachEntry = getFirstDetachField(change.crossFieldKeys, id, count);
+		const renameEntry = change.rootNodes.oldToNewId.getFirst(id, detachEntry.length);
+		const isDetach = (detachEntry.value ?? renameEntry.value) !== undefined;
+		return { start: id, value: isDetach, length: renameEntry.length };
+	};
 
-	if (encodedChange.builds !== undefined) {
-		decoded.builds = decodeDetachedNodes(
-			encodedChange.builds,
+	const getInputRootId = (
+		id: ChangeAtomId,
+		count: number,
+	): RangeQueryResult<ChangeAtomId | undefined> => {
+		return change.rootNodes.newToOldId.getFirst(id, count);
+	};
+
+	const encodeNode = (nodeId: NodeId): EncodedNodeChangeset => {
+		// TODO: Handle node aliasing.
+		const node = change.nodeChanges.get([nodeId.revision, nodeId.localId]);
+		assert(node !== undefined, 0x92e /* Unknown node ID */);
+		return encodeNodeChangesForJson(
+			node,
+			nodeId,
+			fieldToRoots,
+			context,
+			encodeNode,
+			getInputRootId,
+			isAttachId,
+			isDetachId,
+			fieldChangesetCodecs,
+		);
+	};
+
+	// Destroys only exist in rollback changesets, which are never sent.
+	assert(change.destroys === undefined, 0x899 /* Unexpected changeset with destroys */);
+	const encoded: EncodedModularChangesetV1 = {
+		maxId: change.maxId,
+		revisions:
+			change.revisions === undefined
+				? undefined
+				: encodeRevisionInfos(change.revisions, context, revisionTagCodec),
+		changes: encodeFieldChangesForJson(
+			change.fieldChanges,
+			undefined,
+			fieldToRoots,
+			context,
+			encodeNode,
+			getInputRootId,
+			isAttachId,
+			isDetachId,
+			fieldChangesetCodecs,
+		),
+		builds: encodeDetachedNodes(
+			change.builds,
 			context,
 			revisionTagCodec,
 			fieldsCodec,
 			chunkCompressionStrategy,
-		);
-	}
-	if (encodedChange.refreshers !== undefined) {
-		decoded.refreshers = decodeDetachedNodes(
-			encodedChange.refreshers,
+		),
+		refreshers: encodeDetachedNodes(
+			change.refreshers,
 			context,
 			revisionTagCodec,
 			fieldsCodec,
 			chunkCompressionStrategy,
-		);
-	}
-
-	if (encodedChange.violations !== undefined) {
-		decoded.constraintViolationCount = encodedChange.violations;
-	}
+		),
+		violations: change.constraintViolationCount,
+	};
 
-	const decodedRevInfos = decodeRevisionInfos(
-		encodedChange.revisions,
-		context,
-		revisionTagCodec,
-	);
-	if (decodedRevInfos !== undefined) {
-		decoded.revisions = decodedRevInfos;
-	}
-	if (encodedChange.maxId !== undefined) {
-		decoded.maxId = encodedChange.maxId;
-	}
-	return decoded;
+	return encoded;
 }
 
-export function getFieldChangesetCodecs(
-	fieldKinds: FieldKindConfiguration,
+function encodeRevisionInfos(
+	revisions: readonly RevisionInfo[],
+	context: ChangeEncodingContext,
 	revisionTagCodec: IJsonCodec<
 		RevisionTag,
 		EncodedRevisionTag,
 		EncodedRevisionTag,
 		ChangeEncodingContext
 	>,
-	codecOptions: ICodecOptions,
-): Map<
-	FieldKindIdentifier,
-	{ compiledSchema?: SchemaValidationFunction<TAnySchema>; codec: FieldCodec }
-> {
-	// eslint-disable-next-line @typescript-eslint/explicit-function-return-type
-	const getMapEntry = ({ kind, formatVersion }: FieldKindConfigurationEntry) => {
-		const codec = kind.changeHandler.codecsFactory(revisionTagCodec).resolve(formatVersion);
-		return {
-			codec,
-			compiledSchema: codec.json.encodedSchema
-				? extractJsonValidator(codecOptions.jsonValidator).compile(codec.json.encodedSchema)
-				: undefined,
+): EncodedRevisionInfo[] | undefined {
+	if (context.revision !== undefined) {
+		assert(
+			revisions.length === 1 &&
+				revisions[0] !== undefined &&
+				revisions[0].revision === context.revision &&
+				revisions[0].rollbackOf === undefined,
+			0x964 /* A tagged change should only contain the tagged revision */,
+		);
+
+		return undefined;
+	}
+
+	const encodedRevisions = [];
+	for (const revision of revisions) {
+		const encodedRevision: Mutable<EncodedRevisionInfo> = {
+			revision: revisionTagCodec.encode(revision.revision, context),
 		};
-	};
 
-	/**
-	 * The codec version for the generic field kind.
-	 */
-	const genericFieldKindFormatVersion = 1;
-	const fieldChangesetCodecs: Map<
-		FieldKindIdentifier,
-		{
-			compiledSchema?: SchemaValidationFunction<TAnySchema>;
-			codec: FieldCodec;
+		if (revision.rollbackOf !== undefined) {
+			encodedRevision.rollbackOf = revisionTagCodec.encode(revision.rollbackOf, context);
 		}
-	> = new Map([
-		[
-			genericFieldKind.identifier,
-			getMapEntry({ kind: genericFieldKind, formatVersion: genericFieldKindFormatVersion }),
-		],
-	]);
 
-	fieldKinds.forEach((entry, identifier) => {
-		fieldChangesetCodecs.set(identifier, getMapEntry(entry));
-	});
+		encodedRevisions.push(encodedRevision);
+	}
 
-	return fieldChangesetCodecs;
+	return encodedRevisions;
 }
 
-export function makeModularChangeCodecV1(
-	fieldKinds: FieldKindConfiguration,
+function encodeDetachedNodes(
+	detachedNodes: ChangeAtomIdBTree<TreeChunk> | undefined,
+	context: ChangeEncodingContext,
 	revisionTagCodec: IJsonCodec<
 		RevisionTag,
 		EncodedRevisionTag,
@@ -634,42 +658,51 @@ export function makeModularChangeCodecV1(
 		ChangeEncodingContext
 	>,
 	fieldsCodec: FieldBatchCodec,
-	codecOptions: ICodecOptions,
-	chunkCompressionStrategy: TreeCompressionStrategy = TreeCompressionStrategy.Compressed,
-): ModularChangeCodec {
-	const fieldChangesetCodecs = getFieldChangesetCodecs(
-		fieldKinds,
-		revisionTagCodec,
-		codecOptions,
-	);
+	chunkCompressionStrategy: TreeCompressionStrategy,
+): EncodedBuilds | undefined {
+	if (detachedNodes === undefined) {
+		return undefined;
+	}
 
-	const modularChangeCodec: ModularChangeCodec = {
-		encode: (change, context) =>
-			encodeChange(
-				change,
-				context,
-				fieldChangesetCodecs,
-				revisionTagCodec,
-				fieldsCodec,
-				chunkCompressionStrategy,
-			),
-		decode: (encodedChange, context) =>
-			decodeChange(
-				encodedChange,
-				context,
-				fieldKinds,
-				fieldChangesetCodecs,
-				revisionTagCodec,
-				fieldsCodec,
-				chunkCompressionStrategy,
-			),
-	};
+	const treesToEncode: ITreeCursorSynchronous[] = [];
+	const buildsArray: EncodedBuildsArray = [];
 
-	return withSchemaValidation(
-		EncodedModularChangesetV1,
-		modularChangeCodec,
-		codecOptions.jsonValidator,
-	);
+	let buildsForRevision:
+		| [[ChangesetLocalId, number][], EncodedRevisionTag]
+		| [[ChangesetLocalId, number][]]
+		| undefined;
+
+	for (const [[revision, id], chunk] of detachedNodes.entries()) {
+		const encodedRevision = encodeRevisionOpt(revisionTagCodec, revision, context);
+
+		if (buildsForRevision === undefined || buildsForRevision[1] !== encodedRevision) {
+			if (buildsForRevision !== undefined) {
+				buildsArray.push(buildsForRevision);
+			}
+
+			buildsForRevision = encodedRevision === undefined ? [[]] : [[], encodedRevision];
+		}
+
+		treesToEncode.push(chunk.cursor());
+		const treeIndexInBatch = treesToEncode.length - 1;
+		buildsForRevision?.[0].push([id, treeIndexInBatch]);
+	}
+
+	if (buildsForRevision !== undefined) {
+		buildsArray.push(buildsForRevision);
+	}
+
+	return buildsArray.length === 0
+		? undefined
+		: {
+				builds: buildsArray,
+				trees: fieldsCodec.encode(treesToEncode, {
+					encodeType: chunkCompressionStrategy,
+					schema: context.schema,
+					originatorId: context.originatorId,
+					idCompressor: context.idCompressor,
+				}),
+			};
 }
 
 function getChangeHandler(
@@ -701,3 +734,179 @@ function encodeRevisionOpt(
 
 	return revision === context.revision ? undefined : revisionCodec.encode(revision, context);
 }
+
+function getFieldToRoots(
+	rootTable: RootNodeTable,
+	aliases: ChangeAtomIdBTree<NodeId>,
+): FieldRootMap {
+	const fieldToRoots: FieldRootMap = newTupleBTree();
+	for (const [[revision, localId], nodeId] of rootTable.nodeChanges.entries()) {
+		const detachId: ChangeAtomId = { revision, localId };
+		const fieldId = rootTable.detachLocations.getFirst(detachId, 1).value;
+		if (fieldId === undefined) {
+			fail("Untracked root change");
+		} else {
+			setInChangeAtomIdMap(
+				getOrAddInFieldRootMap(fieldToRoots, normalizeFieldId(fieldId, aliases)).nodeChanges,
+				detachId,
+				nodeId,
+			);
+		}
+	}
+
+	for (const entry of rootTable.oldToNewId.entries()) {
+		const fieldId = rootTable.detachLocations.getFirst(entry.start, 1).value;
+		if (fieldId === undefined) {
+			fail("Untracked root change");
+		} else {
+			getOrAddInFieldRootMap(fieldToRoots, normalizeFieldId(fieldId, aliases)).renames.set(
+				entry.start,
+				entry.length,
+				entry.value,
+			);
+		}
+	}
+
+	return fieldToRoots;
+}
+
+function getOrAddInFieldRootMap(map: FieldRootMap, fieldId: FieldId): FieldRootChanges {
+	const key: FieldIdKey = [fieldId.nodeId?.revision, fieldId.nodeId?.localId, fieldId.field];
+	const rootChanges = map.get(key);
+	if (rootChanges !== undefined) {
+		return rootChanges;
+	}
+
+	const newRootChanges: FieldRootChanges = {
+		nodeChanges: newTupleBTree(),
+		renames: newChangeAtomIdTransform(),
+	};
+	map.set(key, newRootChanges);
+	return newRootChanges;
+}
+
+export function decodeChange(
+	encodedChange: EncodedModularChangesetV1,
+	context: ChangeEncodingContext,
+	fieldKinds: FieldKindConfiguration,
+	fieldChangesetCodecs: Map<
+		FieldKindIdentifier,
+		{
+			compiledSchema?: SchemaValidationFunction<TAnySchema>;
+			codec: FieldCodec;
+		}
+	>,
+	revisionTagCodec: IJsonCodec<
+		RevisionTag,
+		EncodedRevisionTag,
+		EncodedRevisionTag,
+		ChangeEncodingContext
+	>,
+	fieldsCodec: FieldBatchCodec,
+	chunkCompressionStrategy: TreeCompressionStrategy,
+): Mutable<ModularChangeset> {
+	const idAllocator = idAllocatorFromMaxId(encodedChange.maxId);
+	const nodeChanges: ChangeAtomIdBTree<NodeChangeset> = newTupleBTree();
+	const nodeToParent: ChangeAtomIdBTree<NodeLocation> = newTupleBTree();
+	const crossFieldKeys: CrossFieldKeyTable = newCrossFieldRangeTable();
+	const rootNodes = newRootTable();
+
+	const decodeNode: NodeDecoder = (
+		encodedNode: EncodedNodeChangeset,
+		fieldId: NodeLocation,
+	): NodeId => {
+		const nodeId: NodeId = {
+			revision: context.revision,
+			localId: brand(idAllocator.allocate()),
+		};
+
+		const node = decodeNodeChangesetFromJson(
+			encodedNode,
+			nodeId,
+			crossFieldKeys,
+			rootNodes,
+			context,
+			decodeNode,
+			idAllocator,
+			fieldKinds,
+			fieldChangesetCodecs,
+		);
+
+		nodeChanges.set([nodeId.revision, nodeId.localId], node);
+
+		if (fieldId !== undefined) {
+			nodeToParent.set([nodeId.revision, nodeId.localId], fieldId);
+		}
+
+		return nodeId;
+	};
+
+	const decoded: Mutable<ModularChangeset> = {
+		rebaseVersion: 1,
+		fieldChanges: decodeFieldChangesFromJson(
+			encodedChange.changes,
+			undefined,
+			crossFieldKeys,
+			rootNodes,
+			context,
+			decodeNode,
+			idAllocator,
+			fieldKinds,
+			fieldChangesetCodecs,
+		),
+		nodeChanges,
+		rootNodes,
+		nodeToParent,
+		nodeAliases: newTupleBTree(),
+		crossFieldKeys,
+	};
+
+	if (encodedChange.builds !== undefined) {
+		decoded.builds = decodeDetachedNodes(
+			encodedChange.builds,
+			context,
+			revisionTagCodec,
+			fieldsCodec,
+			chunkCompressionStrategy,
+		);
+	}
+	if (encodedChange.refreshers !== undefined) {
+		decoded.refreshers = decodeDetachedNodes(
+			encodedChange.refreshers,
+			context,
+			revisionTagCodec,
+			fieldsCodec,
+			chunkCompressionStrategy,
+		);
+	}
+
+	if (encodedChange.violations !== undefined) {
+		decoded.constraintViolationCount = encodedChange.violations;
+	}
+
+	const decodedRevInfos = decodeRevisionInfos(
+		encodedChange.revisions,
+		context,
+		revisionTagCodec,
+	);
+	if (decodedRevInfos !== undefined) {
+		decoded.revisions = decodedRevInfos;
+	}
+	if (encodedChange.maxId !== undefined) {
+		decoded.maxId = encodedChange.maxId;
+	}
+
+	// XXX: This is an expensive assert which should be removed before merging.
+	validateChangeset(decoded, fieldKindsFromConfiguration(fieldKinds));
+	return decoded;
+}
+
+function fieldKindsFromConfiguration(
+	configuration: FieldKindConfiguration,
+): ReadonlyMap<FieldKindIdentifier, FlexFieldKind> {
+	const map = new Map();
+	for (const [id, entry] of configuration.entries()) {
+		map.set(id, entry.kind);
+	}
+	return map;
+}