@fluidframework/tree 2.80.0 → 2.81.0-374083

package/lib/feature-libraries/optional-field/optionalFieldCodecV3.js

@@ -0,0 +1,53 @@
+/*!
+ * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
+ * Licensed under the MIT License.
+ */
+import { makeChangeAtomIdCodec } from "../changeAtomIdCodec.js";
+import { EncodedNodeChangeset, } from "../modular-schema/index.js";
+import { EncodedOptionalChangeset } from "./optionalFieldChangeFormatV3.js";
+export function makeOptionalFieldCodec(revisionTagCodec) {
+    const changeAtomIdCodec = makeChangeAtomIdCodec(revisionTagCodec);
+    return {
+        encode: (change, context) => {
+            const encoded = {};
+            if (change.valueReplace !== undefined) {
+                encoded.r = {
+                    e: change.valueReplace.isEmpty,
+                    d: changeAtomIdCodec.encode(change.valueReplace.dst, context.baseContext),
+                };
+                if (change.valueReplace.src !== undefined) {
+                    encoded.r.s = changeAtomIdCodec.encode(change.valueReplace.src, context.baseContext);
+                }
+            }
+            if (change.nodeDetach !== undefined) {
+                encoded.d = changeAtomIdCodec.encode(change.nodeDetach, context.baseContext);
+            }
+            if (change.childChange !== undefined) {
+                encoded.c = context.encodeNode(change.childChange);
+            }
+            return encoded;
+        },
+        decode: (encoded, context) => {
+            const decoded = {};
+            if (encoded.r !== undefined) {
+                const replace = {
+                    isEmpty: encoded.r.e,
+                    dst: changeAtomIdCodec.decode(encoded.r.d, context.baseContext),
+                };
+                if (encoded.r.s !== undefined) {
+                    replace.src = changeAtomIdCodec.decode(encoded.r.s, context.baseContext);
+                }
+                decoded.valueReplace = replace;
+            }
+            if (encoded.d !== undefined) {
+                decoded.nodeDetach = changeAtomIdCodec.decode(encoded.d, context.baseContext);
+            }
+            if (encoded.c !== undefined) {
+                decoded.childChange = context.decodeNode(encoded.c);
+            }
+            return decoded;
+        },
+        encodedSchema: EncodedOptionalChangeset(EncodedNodeChangeset),
+    };
+}
+//# sourceMappingURL=optionalFieldCodecV3.js.map

package/lib/feature-libraries/optional-field/optionalFieldCodecV3.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"optionalFieldCodecV3.js","sourceRoot":"","sources":["../../../src/feature-libraries/optional-field/optionalFieldCodecV3.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAWH,OAAO,EAAE,qBAAqB,EAAE,MAAM,yBAAyB,CAAC;AAChE,OAAO,EACN,oBAAoB,GAEpB,MAAM,4BAA4B,CAAC;AAEpC,OAAO,EAAE,wBAAwB,EAAE,MAAM,kCAAkC,CAAC;AAG5E,MAAM,UAAU,sBAAsB,CACrC,gBAKC;IAOD,MAAM,iBAAiB,GAAG,qBAAqB,CAAC,gBAAgB,CAAC,CAAC;IAElE,OAAO;QACN,MAAM,EAAE,CAAC,MAAyB,EAAE,OAAmC,EAAE,EAAE;YAC1E,MAAM,OAAO,GAAyC,EAAE,CAAC;YAEzD,IAAI,MAAM,CAAC,YAAY,KAAK,SAAS,EAAE,CAAC;gBACvC,OAAO,CAAC,CAAC,GAAG;oBACX,CAAC,EAAE,MAAM,CAAC,YAAY,CAAC,OAAO;oBAC9B,CAAC,EAAE,iBAAiB,CAAC,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,WAAW,CAAC;iBACzE,CAAC;gBAEF,IAAI,MAAM,CAAC,YAAY,CAAC,GAAG,KAAK,SAAS,EAAE,CAAC;oBAC3C,OAAO,CAAC,CAAC,CAAC,CAAC,GAAG,iBAAiB,CAAC,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,WAAW,CAAC,CAAC;gBACtF,CAAC;YACF,CAAC;YAED,IAAI,MAAM,CAAC,UAAU,KAAK,SAAS,EAAE,CAAC;gBACrC,OAAO,CAAC,CAAC,GAAG,iBAAiB,CAAC,MAAM,CAAC,MAAM,CAAC,UAAU,EAAE,OAAO,CAAC,WAAW,CAAC,CAAC;YAC9E,CAAC;YAED,IAAI,MAAM,CAAC,WAAW,KAAK,SAAS,EAAE,CAAC;gBACtC,OAAO,CAAC,CAAC,GAAG,OAAO,CAAC,UAAU,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;YACpD,CAAC;YAED,OAAO,OAAO,CAAC;QAChB,CAAC;QAED,MAAM,EAAE,CACP,OAA6C,EAC7C,OAAmC,EAClC,EAAE;YACH,MAAM,OAAO,GAA+B,EAAE,CAAC;YAE/C,IAAI,OAAO,CAAC,CAAC,KAAK,SAAS,EAAE,CAAC;gBAC7B,MAAM,OAAO,GAAqB;oBACjC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC;oBACpB,GAAG,EAAE,iBAAiB,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,WAAW,CAAC;iBAC/D,CAAC;gBACF,IAAI,OAAO,CAAC,CAAC,CAAC,CAAC,KAAK,SAAS,EAAE,CAAC;oBAC/B,OAAO,CAAC,GAAG,GAAG,iBAAiB,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,WAAW,CAAC,CAAC;gBAC1E,CAAC;gBACD,OAAO,CAAC,YAAY,GAAG,OAAO,CAAC;YAChC,CAAC;YAED,IAAI,OAAO,CAAC,CAAC,KAAK,SAAS,EAAE,CAAC;gBAC7B,OAAO,CAAC,UAAU,GAAG,iBAAiB,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,EAAE,OAAO,CAAC,WAAW,CAAC,CAAC;YAC/E,CAAC;YAED,IAAI,OAAO,CAAC,CAAC,KAAK,SAAS,EAAE,CAAC;gBAC7B,OAAO,CAAC,WAAW,GAAG,OAAO,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC;YACrD,CAAC;YAED,OAAO,OAAO,CAAC;QAChB,CAAC;QACD,aAAa,EAAE,wBAAwB,CAAC,oBAAoB,CAAC;KAC7D,CAAC;AACH,CAAC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport type { TAnySchema } from \"@sinclair/typebox\";\n\nimport type { IJsonCodec } from \"../../codec/index.js\";\nimport type {\n\tChangeEncodingContext,\n\tEncodedRevisionTag,\n\tRevisionTag,\n} from \"../../core/index.js\";\nimport type { Mutable } from \"../../util/index.js\";\nimport { makeChangeAtomIdCodec } from \"../changeAtomIdCodec.js\";\nimport {\n\tEncodedNodeChangeset,\n\ttype FieldChangeEncodingContext,\n} from \"../modular-schema/index.js\";\n\nimport { EncodedOptionalChangeset } from \"./optionalFieldChangeFormatV3.js\";\nimport type { OptionalChangeset, Replace } from \"./optionalFieldChangeTypes.js\";\n\nexport function makeOptionalFieldCodec(\n\trevisionTagCodec: IJsonCodec<\n\t\tRevisionTag,\n\t\tEncodedRevisionTag,\n\t\tEncodedRevisionTag,\n\t\tChangeEncodingContext\n\t>,\n): IJsonCodec<\n\tOptionalChangeset,\n\tEncodedOptionalChangeset<TAnySchema>,\n\tEncodedOptionalChangeset<TAnySchema>,\n\tFieldChangeEncodingContext\n> {\n\tconst changeAtomIdCodec = makeChangeAtomIdCodec(revisionTagCodec);\n\n\treturn {\n\t\tencode: (change: OptionalChangeset, context: FieldChangeEncodingContext) => {\n\t\t\tconst encoded: EncodedOptionalChangeset<TAnySchema> = {};\n\n\t\t\tif (change.valueReplace !== undefined) {\n\t\t\t\tencoded.r = {\n\t\t\t\t\te: change.valueReplace.isEmpty,\n\t\t\t\t\td: changeAtomIdCodec.encode(change.valueReplace.dst, context.baseContext),\n\t\t\t\t};\n\n\t\t\t\tif (change.valueReplace.src !== undefined) {\n\t\t\t\t\tencoded.r.s = changeAtomIdCodec.encode(change.valueReplace.src, context.baseContext);\n\t\t\t\t}\n\t\t\t}\n\n\t\t\tif (change.nodeDetach !== undefined) {\n\t\t\t\tencoded.d = changeAtomIdCodec.encode(change.nodeDetach, context.baseContext);\n\t\t\t}\n\n\t\t\tif (change.childChange !== undefined) {\n\t\t\t\tencoded.c = context.encodeNode(change.childChange);\n\t\t\t}\n\n\t\t\treturn encoded;\n\t\t},\n\n\t\tdecode: (\n\t\t\tencoded: EncodedOptionalChangeset<TAnySchema>,\n\t\t\tcontext: FieldChangeEncodingContext,\n\t\t) => {\n\t\t\tconst decoded: Mutable<OptionalChangeset> = {};\n\n\t\t\tif (encoded.r !== undefined) {\n\t\t\t\tconst replace: Mutable<Replace> = {\n\t\t\t\t\tisEmpty: encoded.r.e,\n\t\t\t\t\tdst: changeAtomIdCodec.decode(encoded.r.d, context.baseContext),\n\t\t\t\t};\n\t\t\t\tif (encoded.r.s !== undefined) {\n\t\t\t\t\treplace.src = changeAtomIdCodec.decode(encoded.r.s, context.baseContext);\n\t\t\t\t}\n\t\t\t\tdecoded.valueReplace = replace;\n\t\t\t}\n\n\t\t\tif (encoded.d !== undefined) {\n\t\t\t\tdecoded.nodeDetach = changeAtomIdCodec.decode(encoded.d, context.baseContext);\n\t\t\t}\n\n\t\t\tif (encoded.c !== undefined) {\n\t\t\t\tdecoded.childChange = context.decodeNode(encoded.c);\n\t\t\t}\n\n\t\t\treturn decoded;\n\t\t},\n\t\tencodedSchema: EncodedOptionalChangeset(EncodedNodeChangeset),\n\t};\n}\n"]}

package/lib/feature-libraries/optional-field/optionalFieldCodecs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"optionalFieldCodecs.d.ts","sourceRoot":"","sources":["../../../src/feature-libraries/optional-field/optionalFieldCodecs.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,KAAK,YAAY,EAAE,KAAK,UAAU,EAAmB,MAAM,sBAAsB,CAAC;AAC3F,OAAO,KAAK,EACX,qBAAqB,EACrB,kBAAkB,EAClB,WAAW,EACX,MAAM,qBAAqB,CAAC;AAC7B,OAAO,KAAK,EAAE,0BAA0B,EAAE,MAAM,4BAA4B,CAAC;AAE7E,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,+BAA+B,CAAC;AAGvE,eAAO,MAAM,4BAA4B,qBACtB,WACjB,WAAW,EACX,kBAAkB,EAClB,kBAAkB,EAClB,qBAAqB,CACrB,KACC,aAAa,iBAAiB,EAAE,0BAA0B,CACP,CAAC"}
+{"version":3,"file":"optionalFieldCodecs.d.ts","sourceRoot":"","sources":["../../../src/feature-libraries/optional-field/optionalFieldCodecs.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,KAAK,YAAY,EAAE,KAAK,UAAU,EAAmB,MAAM,sBAAsB,CAAC;AAC3F,OAAO,KAAK,EACX,qBAAqB,EACrB,kBAAkB,EAClB,WAAW,EACX,MAAM,qBAAqB,CAAC;AAC7B,OAAO,KAAK,EAAE,0BAA0B,EAAE,MAAM,4BAA4B,CAAC;AAE7E,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,+BAA+B,CAAC;AAIvE,eAAO,MAAM,4BAA4B,qBACtB,WACjB,WAAW,EACX,kBAAkB,EAClB,kBAAkB,EAClB,qBAAqB,CACrB,KACC,aAAa,iBAAiB,EAAE,0BAA0B,CAI1D,CAAC"}

package/lib/feature-libraries/optional-field/optionalFieldCodecs.js

@@ -4,5 +4,9 @@
  */
 import { makeCodecFamily } from "../../codec/index.js";
 import { makeOptionalFieldCodec as makeV2Codec } from "./optionalFieldCodecV2.js";
-export const makeOptionalFieldCodecFamily = (revisionTagCodec) => makeCodecFamily([[2, makeV2Codec(revisionTagCodec)]]);
+import { makeOptionalFieldCodec as makeV3Codec } from "./optionalFieldCodecV3.js";
+export const makeOptionalFieldCodecFamily = (revisionTagCodec) => makeCodecFamily([
+    [2, makeV2Codec(revisionTagCodec)],
+    [3, makeV3Codec(revisionTagCodec)],
+]);
 //# sourceMappingURL=optionalFieldCodecs.js.map

package/lib/feature-libraries/optional-field/optionalFieldCodecs.js.map

@@ -1 +1 @@
-{"version":3,"file":"optionalFieldCodecs.js","sourceRoot":"","sources":["../../../src/feature-libraries/optional-field/optionalFieldCodecs.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAsC,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAS3F,OAAO,EAAE,sBAAsB,IAAI,WAAW,EAAE,MAAM,2BAA2B,CAAC;AAElF,MAAM,CAAC,MAAM,4BAA4B,GAAG,CAC3C,gBAKC,EAC6D,EAAE,CAChE,eAAe,CAAC,CAAC,CAAC,CAAC,EAAE,WAAW,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CAAC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport { type ICodecFamily, type IJsonCodec, makeCodecFamily } from \"../../codec/index.js\";\nimport type {\n\tChangeEncodingContext,\n\tEncodedRevisionTag,\n\tRevisionTag,\n} from \"../../core/index.js\";\nimport type { FieldChangeEncodingContext } from \"../modular-schema/index.js\";\n\nimport type { OptionalChangeset } from \"./optionalFieldChangeTypes.js\";\nimport { makeOptionalFieldCodec as makeV2Codec } from \"./optionalFieldCodecV2.js\";\n\nexport const makeOptionalFieldCodecFamily = (\n\trevisionTagCodec: IJsonCodec<\n\t\tRevisionTag,\n\t\tEncodedRevisionTag,\n\t\tEncodedRevisionTag,\n\t\tChangeEncodingContext\n\t>,\n): ICodecFamily<OptionalChangeset, FieldChangeEncodingContext> =>\n\tmakeCodecFamily([[2, makeV2Codec(revisionTagCodec)]]);\n"]}
+{"version":3,"file":"optionalFieldCodecs.js","sourceRoot":"","sources":["../../../src/feature-libraries/optional-field/optionalFieldCodecs.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAsC,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAS3F,OAAO,EAAE,sBAAsB,IAAI,WAAW,EAAE,MAAM,2BAA2B,CAAC;AAClF,OAAO,EAAE,sBAAsB,IAAI,WAAW,EAAE,MAAM,2BAA2B,CAAC;AAElF,MAAM,CAAC,MAAM,4BAA4B,GAAG,CAC3C,gBAKC,EAC6D,EAAE,CAChE,eAAe,CAAC;IACf,CAAC,CAAC,EAAE,WAAW,CAAC,gBAAgB,CAAC,CAAC;IAClC,CAAC,CAAC,EAAE,WAAW,CAAC,gBAAgB,CAAC,CAAC;CAClC,CAAC,CAAC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport { type ICodecFamily, type IJsonCodec, makeCodecFamily } from \"../../codec/index.js\";\nimport type {\n\tChangeEncodingContext,\n\tEncodedRevisionTag,\n\tRevisionTag,\n} from \"../../core/index.js\";\nimport type { FieldChangeEncodingContext } from \"../modular-schema/index.js\";\n\nimport type { OptionalChangeset } from \"./optionalFieldChangeTypes.js\";\nimport { makeOptionalFieldCodec as makeV2Codec } from \"./optionalFieldCodecV2.js\";\nimport { makeOptionalFieldCodec as makeV3Codec } from \"./optionalFieldCodecV3.js\";\n\nexport const makeOptionalFieldCodecFamily = (\n\trevisionTagCodec: IJsonCodec<\n\t\tRevisionTag,\n\t\tEncodedRevisionTag,\n\t\tEncodedRevisionTag,\n\t\tChangeEncodingContext\n\t>,\n): ICodecFamily<OptionalChangeset, FieldChangeEncodingContext> =>\n\tmakeCodecFamily([\n\t\t[2, makeV2Codec(revisionTagCodec)],\n\t\t[3, makeV3Codec(revisionTagCodec)],\n\t]);\n"]}

package/lib/feature-libraries/sequence-field/compose.d.ts

@@ -4,9 +4,8 @@
  */
 import { type RevisionMetadataSource } from "../../core/index.js";
 import type { IdAllocator } from "../../util/index.js";
-import { type CrossFieldManager, type NodeChangeComposer } from "../modular-schema/index.js";
-import { type MoveEffectTable } from "./moveEffectTable.js";
-import { type Changeset, type Mark } from "./types.js";
+import type { ComposeNodeManager, NodeChangeComposer } from "../modular-schema/index.js";
+import type { Changeset, Mark } from "./types.js";
 /**
  * Composes a sequence of changesets into a single changeset.
  * @param changes - The changesets to be applied.
@@ -19,7 +18,7 @@ import { type Changeset, type Mark } from "./types.js";
  * - Support for moves is not implemented.
  * - Support for slices is not implemented.
  */
-export declare function compose(change1: Changeset, change2: Changeset, composeChild: NodeChangeComposer, _genId: IdAllocator, manager: CrossFieldManager, revisionMetadata: RevisionMetadataSource): Changeset;
+export declare function compose(change1: Changeset, change2: Changeset, composeChild: NodeChangeComposer, _genId: IdAllocator, manager: ComposeNodeManager, revisionMetadata: RevisionMetadataSource): Changeset;
 export declare class ComposeQueue {
     private readonly moveEffects;
     private readonly revisionMetadata;
@@ -27,7 +26,7 @@ export declare class ComposeQueue {
     private readonly newMarks;
     private readonly baseMarksCellSources;
     private readonly newMarksCellSources;
-    constructor(baseMarks: Changeset, newMarks: Changeset, moveEffects: MoveEffectTable, revisionMetadata: RevisionMetadataSource);
+    constructor(baseMarks: Changeset, newMarks: Changeset, moveEffects: ComposeNodeManager, revisionMetadata: RevisionMetadataSource);
     isEmpty(): boolean;
     pop(): ComposeMarks;
     private dequeueBase;
@@ -36,8 +35,8 @@ export declare class ComposeQueue {
     private peekMinLength;
 }
 interface ComposeMarks {
-    baseMark?: Mark;
-    newMark?: Mark;
+    baseMark: Mark;
+    newMark: Mark;
 }
 export {};
 //# sourceMappingURL=compose.d.ts.map

package/lib/feature-libraries/sequence-field/compose.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"compose.d.ts","sourceRoot":"","sources":["../../../src/feature-libraries/sequence-field/compose.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAIH,OAAO,EAEN,KAAK,sBAAsB,EAG3B,MAAM,qBAAqB,CAAC;AAC7B,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AACvD,OAAO,EACN,KAAK,iBAAiB,EAEtB,KAAK,kBAAkB,EAEvB,MAAM,4BAA4B,CAAC;AAKpC,OAAO,EAEN,KAAK,eAAe,EAQpB,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EAGN,KAAK,SAAS,EAEd,KAAK,IAAI,EAMT,MAAM,YAAY,CAAC;AA6BpB;;;;;;;;;;;GAWG;AACH,wBAAgB,OAAO,CACtB,OAAO,EAAE,SAAS,EAClB,OAAO,EAAE,SAAS,EAClB,YAAY,EAAE,kBAAkB,EAChC,MAAM,EAAE,WAAW,EACnB,OAAO,EAAE,iBAAiB,EAC1B,gBAAgB,EAAE,sBAAsB,GACtC,SAAS,CAQX;AAuZD,qBAAa,YAAY;IASvB,OAAO,CAAC,QAAQ,CAAC,WAAW;IAC5B,OAAO,CAAC,QAAQ,CAAC,gBAAgB;IATlC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAY;IACtC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAY;IACrC,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAuC;IAC5E,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAuC;gBAG1E,SAAS,EAAE,SAAS,EACpB,QAAQ,EAAE,SAAS,EACF,WAAW,EAAE,eAAe,EAC5B,gBAAgB,EAAE,sBAAsB;IAQnD,OAAO,IAAI,OAAO;IAIlB,GAAG,IAAI,YAAY;IAyD1B,OAAO,CAAC,WAAW;IAWnB,OAAO,CAAC,UAAU;IAUlB,OAAO,CAAC,WAAW;IAiBnB,OAAO,CAAC,aAAa;CAUrB;AAED,UAAU,YAAY;IACrB,QAAQ,CAAC,EAAE,IAAI,CAAC;IAChB,OAAO,CAAC,EAAE,IAAI,CAAC;CACf"}
+{"version":3,"file":"compose.d.ts","sourceRoot":"","sources":["../../../src/feature-libraries/sequence-field/compose.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAIH,OAAO,EAIN,KAAK,sBAAsB,EAE3B,MAAM,qBAAqB,CAAC;AAC7B,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AACvD,OAAO,KAAK,EACX,kBAAkB,EAClB,kBAAkB,EAElB,MAAM,4BAA4B,CAAC;AAKpC,OAAO,KAAK,EAEX,SAAS,EAET,IAAI,EAIJ,MAAM,YAAY,CAAC;AA2BpB;;;;;;;;;;;GAWG;AACH,wBAAgB,OAAO,CACtB,OAAO,EAAE,SAAS,EAClB,OAAO,EAAE,SAAS,EAClB,YAAY,EAAE,kBAAkB,EAChC,MAAM,EAAE,WAAW,EACnB,OAAO,EAAE,kBAAkB,EAC3B,gBAAgB,EAAE,sBAAsB,GACtC,SAAS,CAEX;AAsND,qBAAa,YAAY;IASvB,OAAO,CAAC,QAAQ,CAAC,WAAW;IAC5B,OAAO,CAAC,QAAQ,CAAC,gBAAgB;IATlC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAY;IACtC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAY;IACrC,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAuC;IAC5E,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAuC;gBAG1E,SAAS,EAAE,SAAS,EACpB,QAAQ,EAAE,SAAS,EACF,WAAW,EAAE,kBAAkB,EAC/B,gBAAgB,EAAE,sBAAsB;IAanD,OAAO,IAAI,OAAO;IAIlB,GAAG,IAAI,YAAY;IAyD1B,OAAO,CAAC,WAAW;IAOnB,OAAO,CAAC,UAAU;IAUlB,OAAO,CAAC,WAAW;IAiBnB,OAAO,CAAC,aAAa;CAUrB;AAED,UAAU,YAAY;IACrB,QAAQ,EAAE,IAAI,CAAC;IACf,OAAO,EAAE,IAAI,CAAC;CACd"}

package/lib/feature-libraries/sequence-field/compose.js

@@ -3,13 +3,10 @@
  * Licensed under the MIT License.
  */
 import { assert, unreachableCase, fail } from "@fluidframework/core-utils/internal";
-import { offsetChangeAtomId, } from "../../core/index.js";
-import { CrossFieldTarget, } from "../modular-schema/index.js";
+import { areEqualChangeAtomIdOpts, areEqualChangeAtomIds, } from "../../core/index.js";
 import { MarkListFactory } from "./markListFactory.js";
 import { MarkQueue } from "./markQueue.js";
-import { getCrossFieldTargetFromMove, getMoveEffect, getMoveIn, isMoveIn, isMoveMark, isMoveOut, setMoveEffect, } from "./moveEffectTable.js";
-import { NoopMarkType, } from "./types.js";
-import { CellOrder, areEqualCellIds, areInputCellsEmpty, areOutputCellsEmpty, asAttachAndDetach, cellSourcesFromMarks, compareCellPositionsUsingTombstones, extractMarkEffect, getEndpoint, getInputCellId, getOutputCellId, isAttach, isAttachAndDetachEffect, isDetach, isImpactfulCellRename, isNewAttach, isNoopMark, isRename, markEmptiesCells, markFillsCells, markHasCellEffect, normalizeCellRename, settleMark, withNodeChange, withRevision, } from "./utils.js";
+import { CellOrder, areEqualCellIds, areInputCellsEmpty, areOutputCellsEmpty, cellSourcesFromMarks, compareCellPositionsUsingTombstones, extractMarkEffect, getAttachedRootId, getDetachOutputCellId, getDetachedRootId, getInputCellId, getMovedNodeId, getOutputCellId, isAttach, isDetach, isNewAttach, isNoopMark, isPin, isRename, markEmptiesCells, markHasCellEffect, normalizeCellRename, withNodeChange, } from "./utils.js";
 /**
  * Composes a sequence of changesets into a single changeset.
  * @param changes - The changesets to be applied.
@@ -30,26 +27,8 @@ function composeMarkLists(baseMarkList, newMarkList, composeChild, moveEffects,
     const queue = new ComposeQueue(baseMarkList, newMarkList, moveEffects, revisionMetadata);
     while (!queue.isEmpty()) {
         const { baseMark, newMark } = queue.pop();
-        if (newMark === undefined) {
-            assert(baseMark !== undefined, 0x4db /* Non-empty queue should not return two undefined marks */);
-            factory.push(composeMark(baseMark, moveEffects, (node) => composeChildChanges(node, undefined, composeChild)));
-        }
-        else {
-            // We only compose changesets that will not be further rebased.
-            // It is therefore safe to remove any intentions that have no impact in the context they apply to.
-            const settledNewMark = settleMark(newMark);
-            if (baseMark === undefined) {
-                factory.push(composeMark(settledNewMark, moveEffects, (node) => composeChildChanges(undefined, node, composeChild)));
-            }
-            else {
-                // Past this point, we are guaranteed that `settledNewMark` and `baseMark` have the same length and
-                // start at the same location in the revision after the base changes.
-                // They therefore refer to the same range for that revision.
-                const settledBaseMark = settleMark(baseMark);
-                const composedMark = composeMarks(settledBaseMark, settledNewMark, composeChild, moveEffects);
-                factory.push(composedMark);
-            }
-        }
+        const composedMark = composeMarks(baseMark, newMark, composeChild, moveEffects);
+        factory.push(composedMark);
     }
     return factory.list;
 }
@@ -64,128 +43,64 @@ function composeMarkLists(baseMarkList, newMarkList, composeChild, moveEffects,
  */
 function composeMarks(baseMark, newMark, composeChild, moveEffects) {
     const nodeChange = handleNodeChanges(baseMark, newMark, composeChild, moveEffects);
-    return withUpdatedEndpoint(withNodeChange(composeMarksIgnoreChild(baseMark, newMark, moveEffects), nodeChange), baseMark.count, moveEffects);
+    return withNodeChange(composeMarksIgnoreChild(baseMark, newMark, moveEffects), nodeChange);
 }
 function composeMarksIgnoreChild(baseMark, newMark, moveEffects) {
     if (isNoopMark(baseMark)) {
         return newMark;
     }
     else if (isNoopMark(newMark)) {
-        return baseMark;
+        return updateBaseMarkId(moveEffects, baseMark);
     }
     if (isRename(baseMark) && isRename(newMark)) {
+        if (areEqualChangeAtomIdOpts(baseMark.cellId, newMark.idOverride)) {
+            return createNoopMark(baseMark.count, undefined, baseMark.cellId);
+        }
         return { ...baseMark, idOverride: newMark.idOverride };
     }
     else if (isRename(baseMark)) {
-        assert(isAttach(newMark) || isAttachAndDetachEffect(newMark), 0x9f1 /* Unexpected mark type */);
+        assert(isAttach(newMark), 0x9f1 /* Unexpected mark type */);
+        assert(baseMark.cellId !== undefined && newMark.cellId !== undefined, "Expected marks to target an empty cell");
         return { ...newMark, cellId: baseMark.cellId };
     }
     else if (isRename(newMark)) {
-        assert(isDetach(baseMark) || isAttachAndDetachEffect(baseMark), 0x9f2 /* Unexpected mark type */);
-        return isDetach(baseMark)
-            ? { ...baseMark, idOverride: newMark.idOverride }
-            : { ...baseMark, detach: { ...baseMark.detach, idOverride: newMark.idOverride } };
-    }
-    if (isImpactfulCellRename(newMark)) {
-        const newAttachAndDetach = asAttachAndDetach(newMark);
-        assert(newAttachAndDetach.cellId !== undefined, 0x9f3 /* Impactful cell rename must target empty cell */);
-        const newDetachRevision = newAttachAndDetach.detach.revision;
-        if (markEmptiesCells(baseMark)) {
-            // baseMark is a detach which cancels with the attach portion of the AttachAndDetach,
-            // so we are just left with the detach portion of the AttachAndDetach.
-            const newDetach = {
-                ...newAttachAndDetach.detach,
-                count: baseMark.count,
-            };
-            if (isMoveIn(newAttachAndDetach.attach) && isMoveOut(newAttachAndDetach.detach)) {
-                assert(isMoveOut(baseMark), 0x808 /* Unexpected mark type */);
-                // The base changeset and new changeset both move these nodes.
-                // Call the original position of the nodes A, the position after the base changeset is applied B,
-                // and the position after the new changeset is applied C.
-                // The new changeset moves the nodes from B, temporarily returns them to A, and then moves them to C.
-                // The composition of the base and new changesets will be a move directly from A to C,
-                // since the move from A to B cancels out with the return from B to A.
-                // This if-block is handling marks at A.
-                // When we compose the marks at B we will link the start of the base move (A to B)
-                // with the end of the new move (B to C).
-                // Because we are replacing the mark representing the start of the move with the new changeset's
-                // move-out from A, we update the base move-in at B to consider that its start point.
-                const newDetachId = {
-                    revision: newDetachRevision,
-                    localId: newAttachAndDetach.detach.id,
-                };
-                setTruncatedEndpointForInner(moveEffects, CrossFieldTarget.Destination, getEndpoint(baseMark), baseMark.count, newDetachId);
-                const newEndpoint = getComposedEndpoint(moveEffects, CrossFieldTarget.Source, baseMark.revision, baseMark.id, baseMark.count);
-                if (newEndpoint !== undefined) {
-                    changeFinalEndpoint(newDetach, newEndpoint);
-                    setTruncatedEndpoint(moveEffects, CrossFieldTarget.Destination, newEndpoint, baseMark.count, newDetachId);
-                }
-            }
-            return newDetach;
-        }
-        if (isImpactfulCellRename(baseMark)) {
-            assert(baseMark.cellId !== undefined, 0x9f4 /* Impactful cell rename must target empty cell */);
-            const baseAttachAndDetach = asAttachAndDetach(baseMark);
-            const newOutputId = getOutputCellId(newAttachAndDetach);
-            const originalAttach = { ...baseAttachAndDetach.attach };
-            const finalDetach = { ...newAttachAndDetach.detach };
-            handleMovePivot(baseMark.count, originalAttach, finalDetach, moveEffects);
-            if (areEqualCellIds(newOutputId, baseAttachAndDetach.cellId)) {
-                return { count: baseAttachAndDetach.count, cellId: baseAttachAndDetach.cellId };
+        assert(isDetach(baseMark), 0x9f2 /* Unexpected mark type */);
+        return updateBaseMarkId(moveEffects, { ...baseMark, cellRename: newMark.idOverride });
+    }
+    if (!markHasCellEffect(baseMark)) {
+        assert(baseMark.type === "Insert", "Expected baseMark to be a pin");
+        // `newMark` can be either a remove or another pin.
+        // A pin is treated as a detach and attach, so we call `composeAttachDetach` in either case.
+        moveEffects.composeAttachDetach(getAttachedRootId(baseMark), {
+            revision: newMark.revision,
+            localId: newMark.id,
+        }, baseMark.count);
+        const pinId = getAttachedRootId(baseMark);
+        return newMark.type === "Remove"
+            ? {
+                ...newMark,
+                detachCellId: baseMark.detachCellId ?? pinId,
+                cellRename: getDetachOutputCellId(newMark),
             }
-            // `newMark`'s attach portion cancels with `baseMark`'s detach portion.
-            const detachRevision = finalDetach.revision;
-            if (detachRevision !== undefined) {
-                finalDetach.revision = detachRevision;
-            }
-            return normalizeCellRename(baseMark.cellId, baseMark.count, originalAttach, finalDetach);
-        }
-        return normalizeCellRename(newAttachAndDetach.cellId, newAttachAndDetach.count, newAttachAndDetach.attach, newAttachAndDetach.detach);
-    }
-    if (isImpactfulCellRename(baseMark)) {
-        const baseAttachAndDetach = asAttachAndDetach(baseMark);
-        if (markFillsCells(newMark)) {
-            const originalAttach = withRevision({
-                ...baseAttachAndDetach.attach,
-                cellId: baseAttachAndDetach.cellId,
-                count: baseAttachAndDetach.count,
-            }, baseAttachAndDetach.attach.revision);
-            if (isMoveIn(baseAttachAndDetach.attach) && isMoveOut(baseAttachAndDetach.detach)) {
-                assert(isMoveIn(newMark), 0x809 /* Unexpected mark type */);
-                const originalAttachId = {
-                    revision: baseAttachAndDetach.attach.revision,
-                    localId: baseAttachAndDetach.attach.id,
-                };
-                setTruncatedEndpointForInner(moveEffects, CrossFieldTarget.Source, getEndpoint(newMark), baseAttachAndDetach.count, originalAttachId);
-                const newEndpoint = getComposedEndpoint(moveEffects, CrossFieldTarget.Destination, newMark.revision, newMark.id, newMark.count);
-                if (newEndpoint !== undefined) {
-                    changeFinalEndpoint(originalAttach, newEndpoint);
-                    setTruncatedEndpoint(moveEffects, CrossFieldTarget.Source, newEndpoint, baseMark.count, originalAttachId);
-                }
-            }
-            return originalAttach;
-        }
-        else {
-            // Other mark types have been handled by previous conditional branches.
-            assert(newMark.type === NoopMarkType, 0x80a /* Unexpected mark type */);
-            return baseMark;
-        }
-    }
-    if (!markHasCellEffect(baseMark) && !markHasCellEffect(newMark)) {
-        return createNoopMark(newMark.count, undefined, getInputCellId(baseMark));
-    }
-    else if (!markHasCellEffect(baseMark)) {
-        return newMark;
+            : newMark;
     }
     else if (!markHasCellEffect(newMark)) {
-        return baseMark;
+        if (isAttach(newMark) && isAttach(baseMark)) {
+            // When composing two inserts, the second insert (which is a pin) should take precedence.
+            // We treat the pin as a detach and reattach.
+            moveEffects.composeAttachDetach(getAttachedRootId(baseMark), getAttachedRootId(newMark), baseMark.count);
+            const composed = { cellId: baseMark.cellId, ...newMark };
+            delete composed.detachCellId;
+            return composed;
+        }
+        return updateBaseMarkId(moveEffects, baseMark);
     }
     else if (areInputCellsEmpty(baseMark)) {
         assert(isDetach(newMark), 0x71c /* Unexpected mark type */);
         assert(isAttach(baseMark), 0x71d /* Expected generative mark */);
         const attach = extractMarkEffect(baseMark);
         const detach = extractMarkEffect(newMark);
-        handleMovePivot(baseMark.count, attach, detach, moveEffects);
+        moveEffects.composeAttachDetach(getAttachedRootId(baseMark), getDetachedRootId(newMark), baseMark.count);
         if (areEqualCellIds(getOutputCellId(newMark), baseMark.cellId)) {
             // The output and input cell IDs are the same, so this mark has no effect.
             return { count: baseMark.count, cellId: baseMark.cellId };
@@ -193,37 +108,38 @@ function composeMarksIgnoreChild(baseMark, newMark, moveEffects) {
         return normalizeCellRename(baseMark.cellId, baseMark.count, attach, detach);
     }
     else {
-        const length = baseMark.count;
-        return createNoopMark(length, undefined);
+        assert(baseMark.type === "Remove", "Unexpected mark type");
+        assert(newMark.type === "Insert", "Unexpected mark type");
+        const detachId = getDetachedRootId(baseMark);
+        const attachId = getAttachedRootId(newMark);
+        // Note that we cannot assert that this returns true,
+        // as it may not be until a second pass that MCF can tell that this is a reattach of the same node.
+        moveEffects.composeDetachAttach(detachId, attachId, baseMark.count, true);
+        // The composition has no net effect but we preserve the second change's intention to pin the nodes here.
+        const composedMark = { ...newMark };
+        delete composedMark.cellId;
+        const baseDetachCellId = baseMark.detachCellId ?? detachId;
+        if (!areEqualChangeAtomIds(baseDetachCellId, attachId)) {
+            composedMark.detachCellId = baseDetachCellId;
+        }
+        return composedMark;
     }
 }
-/**
- * Checks if `baseAttach` and `newDetach` are both moves, and if so updates their move endpoints as appropriate,
- * and removes their `finalEndpoint` endpoint fields. Note that can mutate `baseAttach` and `newDetach`.
- * If the effects are not both moves this function does nothing.
- * @param count - The number of cells targeted
- * @param baseAttach - The base attach effect at this location
- * @param newDetach - The new detach effect at this location
- */
-function handleMovePivot(count, baseAttach, newDetach, moveEffects) {
-    if (isMoveIn(baseAttach) && isMoveOut(newDetach)) {
-        const finalSource = getEndpoint(baseAttach);
-        const finalDest = getEndpoint(newDetach);
-        setEndpoint(moveEffects, CrossFieldTarget.Source, finalSource, count, finalDest);
-        const truncatedEndpoint1 = getTruncatedEndpointForInner(moveEffects, CrossFieldTarget.Destination, baseAttach.revision, baseAttach.id, count);
-        if (truncatedEndpoint1 !== undefined) {
-            setTruncatedEndpoint(moveEffects, CrossFieldTarget.Destination, finalDest, count, truncatedEndpoint1);
-        }
-        setEndpoint(moveEffects, CrossFieldTarget.Destination, finalDest, count, finalSource);
-        const truncatedEndpoint2 = getTruncatedEndpointForInner(moveEffects, CrossFieldTarget.Source, newDetach.revision, newDetach.id, count);
-        if (truncatedEndpoint2 !== undefined) {
-            setTruncatedEndpoint(moveEffects, CrossFieldTarget.Source, finalSource, count, truncatedEndpoint2);
+function updateBaseMarkId(moveEffects, baseMark) {
+    if (isDetach(baseMark)) {
+        const baseDetachId = getDetachedRootId(baseMark);
+        const updatedDetachId = getUpdatedDetachId(moveEffects, baseMark);
+        if (updatedDetachId !== undefined &&
+            !areEqualChangeAtomIds(updatedDetachId, baseDetachId)) {
+            return {
+                ...baseMark,
+                revision: updatedDetachId.revision,
+                id: updatedDetachId.localId,
+                detachCellId: baseMark.detachCellId ?? baseDetachId,
+            };
         }
-        // The `finalEndpoint` field of AttachAndDetach move effect pairs is not used,
-        // so we remove it as a normalization.
-        delete baseAttach.finalEndpoint;
-        delete newDetach.finalEndpoint;
     }
+    return baseMark;
 }
 function createNoopMark(length, nodeChange, cellId) {
     const mark = { count: length };
@@ -237,14 +153,13 @@ function createNoopMark(length, nodeChange, cellId) {
     return mark;
 }
 function handleNodeChanges(baseMark, newMark, composeChild, moveEffects) {
-    if (newMark.changes !== undefined) {
-        const baseSource = getMoveIn(baseMark);
-        // TODO: Make sure composeChild is not called twice on the node changes.
-        if (baseSource !== undefined) {
-            setModifyAfter(moveEffects, getEndpoint(baseSource), newMark.changes);
-            return undefined;
-        }
+    if (newMark.changes !== undefined &&
+        baseMark.type === "Insert" &&
+        baseMark.cellId !== undefined) {
+        moveEffects.sendNewChangesToBaseSourceLocation(getAttachedRootId(baseMark), newMark.changes);
+        return undefined;
     }
+    // TODO: Make sure composeChild is not called twice on the node changes.
     return composeChildChanges(baseMark.changes, newMark.changes, composeChild);
 }
 function composeChildChanges(baseChange, newChange, composeChild) {
@@ -253,17 +168,15 @@ function composeChildChanges(baseChange, newChange, composeChild) {
     }
     return composeChild(baseChange, newChange);
 }
-function composeMark(mark, moveEffects, composeChild) {
-    const nodeChanges = mark.changes === undefined ? undefined : composeChild(mark.changes);
-    const updatedMark = withUpdatedEndpoint(mark, mark.count, moveEffects);
-    return withNodeChange(updatedMark, nodeChanges);
-}
 export class ComposeQueue {
     constructor(baseMarks, newMarks, moveEffects, revisionMetadata) {
         this.moveEffects = moveEffects;
         this.revisionMetadata = revisionMetadata;
-        this.baseMarks = new MarkQueue(baseMarks, moveEffects);
-        this.newMarks = new MarkQueue(newMarks, moveEffects);
+        const queryFunc = (mark) => isDetach(mark) || isPin(mark)
+            ? moveEffects.getNewChangesForBaseDetach(getMovedNodeId(mark), mark.count).length
+            : mark.count;
+        this.baseMarks = new MarkQueue(baseMarks, queryFunc);
+        this.newMarks = new MarkQueue(newMarks, (mark) => mark.count);
         this.baseMarksCellSources = cellSourcesFromMarks(baseMarks, getOutputCellId);
         this.newMarksCellSources = cellSourcesFromMarks(newMarks, getInputCellId);
     }
@@ -274,7 +187,7 @@ export class ComposeQueue {
         const baseMark = this.baseMarks.peek();
         const newMark = this.newMarks.peek();
         if (baseMark === undefined && newMark === undefined) {
-            return {};
+            fail("Should not pop when queue is empty");
         }
         else if (baseMark === undefined) {
             return this.dequeueNew();
@@ -323,9 +236,6 @@ export class ComposeQueue {
     dequeueBase(length = Number.POSITIVE_INFINITY) {
         const baseMark = this.baseMarks.dequeueUpTo(length);
         const movedChanges = getMovedChangesFromMark(this.moveEffects, baseMark);
-        if (movedChanges !== undefined) {
-            this.moveEffects.onMoveIn(movedChanges);
-        }
         const newMark = createNoopMark(baseMark.count, movedChanges, getOutputCellId(baseMark));
         return { baseMark, newMark };
     }
@@ -359,100 +269,14 @@ export class ComposeQueue {
     }
 }
 function getMovedChangesFromMark(moveEffects, markEffect) {
-    if (isAttachAndDetachEffect(markEffect)) {
-        return getMovedChangesFromMark(moveEffects, markEffect.detach);
-    }
-    if (!isMoveOut(markEffect)) {
+    if (!isDetach(markEffect)) {
         return undefined;
     }
-    return getModifyAfter(moveEffects, markEffect.revision, markEffect.id);
-}
-// It is expected that the range from `id` to `id + count - 1` has the same move effect.
-// The call sites to this function are making queries about a mark which has already been split by a `MarkQueue`
-// to match the ranges in `moveEffects`.
-// TODO: Reduce the duplication between this and other MoveEffect helpers
-function getModifyAfter(moveEffects, revision, id) {
-    const target = CrossFieldTarget.Source;
-    const effect = getMoveEffect(moveEffects, target, revision, id, 1);
-    if (effect.value?.modifyAfter !== undefined) {
-        return effect.value.modifyAfter;
-    }
-    return undefined;
-}
-// TODO: Reduce the duplication between this and other MoveEffect helpers
-function setModifyAfter(moveEffects, { revision, localId: id }, modifyAfter) {
-    const target = CrossFieldTarget.Source;
-    const count = 1;
-    const effect = getMoveEffect(moveEffects, target, revision, id, count, false);
-    const newEffect = effect.value === undefined ? { modifyAfter } : { ...effect.value, modifyAfter };
-    setMoveEffect(moveEffects, target, revision, id, count, newEffect);
-}
-function setEndpoint(moveEffects, target, id, count, endpoint) {
-    const effect = getMoveEffect(moveEffects, target, id.revision, id.localId, count, false);
-    const newEffect = effect.value === undefined ? { endpoint } : { ...effect.value, endpoint };
-    setMoveEffect(moveEffects, target, id.revision, id.localId, effect.length, newEffect);
-    const remainingCount = count - effect.length;
-    if (remainingCount > 0) {
-        setEndpoint(moveEffects, target, offsetChangeAtomId(id, effect.length), remainingCount, offsetChangeAtomId(endpoint, effect.length));
-    }
-}
-function setTruncatedEndpoint(moveEffects, target, id, count, truncatedEndpoint) {
-    const effect = getMoveEffect(moveEffects, target, id.revision, id.localId, count);
-    const newEffect = effect.value === undefined
-        ? { truncatedEndpoint }
-        : { ...effect.value, truncatedEndpoint };
-    setMoveEffect(moveEffects, target, id.revision, id.localId, effect.length, newEffect);
-    const remainingCount = count - effect.length;
-    if (remainingCount > 0) {
-        setTruncatedEndpoint(moveEffects, target, offsetChangeAtomId(id, effect.length), remainingCount, offsetChangeAtomId(truncatedEndpoint, effect.length));
-    }
-}
-function setTruncatedEndpointForInner(moveEffects, target, id, count, truncatedEndpointForInner) {
-    const effect = getMoveEffect(moveEffects, target, id.revision, id.localId, count);
-    const newEffect = effect.value === undefined
-        ? { truncatedEndpointForInner }
-        : { ...effect.value, truncatedEndpointForInner };
-    setMoveEffect(moveEffects, target, id.revision, id.localId, effect.length, newEffect);
-    const remainingCount = count - effect.length;
-    if (remainingCount > 0) {
-        setTruncatedEndpointForInner(moveEffects, target, offsetChangeAtomId(id, effect.length), remainingCount, offsetChangeAtomId(truncatedEndpointForInner, effect.length));
-    }
-}
-function withUpdatedEndpoint(mark, count, effects) {
-    if (isAttachAndDetachEffect(mark)) {
-        return {
-            ...mark,
-            attach: withUpdatedEndpoint(mark.attach, count, effects),
-            detach: withUpdatedEndpoint(mark.detach, count, effects),
-        };
-    }
-    if (!isMoveMark(mark)) {
-        return mark;
-    }
-    const finalDest = getComposedEndpoint(effects, getCrossFieldTargetFromMove(mark), mark.revision, mark.id, count);
-    if (finalDest === undefined) {
-        return mark;
-    }
-    const output = { ...mark };
-    changeFinalEndpoint(output, finalDest);
-    return output;
-}
-function changeFinalEndpoint(mark, endpoint) {
-    if (areEqualCellIds(endpoint, { revision: mark.revision, localId: mark.id })) {
-        delete mark.finalEndpoint;
-    }
-    else {
-        mark.finalEndpoint = endpoint;
-    }
-}
-function getComposedEndpoint(moveEffects, target, revision, id, count) {
-    const effect = getMoveEffect(moveEffects, target, revision, id, count);
-    assert(effect.length === count, 0x815 /* Expected effect to cover entire mark */);
-    return effect.value?.truncatedEndpoint ?? effect.value?.endpoint;
+    return moveEffects.getNewChangesForBaseDetach(getDetachedRootId(markEffect), 1).value
+        ?.nodeChange;
 }
-function getTruncatedEndpointForInner(moveEffects, target, revision, id, count) {
-    const effect = getMoveEffect(moveEffects, target, revision, id, count);
-    assert(effect.length === count, 0x934 /* Expected effect to cover entire mark */);
-    return effect.value?.truncatedEndpointForInner;
+function getUpdatedDetachId(manager, mark) {
+    return manager.getNewChangesForBaseDetach(getDetachedRootId(mark), mark.count).value
+        ?.detachId;
 }
 //# sourceMappingURL=compose.js.map