@stll/folio-core 0.37.5 → 0.38.0

package/dist/ai-edits/apply.js

@@ -1,3 +1,4 @@
+import { formattingEquals } from "../docx/runConsolidator.js";
 import { expectParagraphAttrs, expectRunPropertyChangeMarkAttrs } from "../prosemirror/attrs/index.js";
 import { hasSerializableParagraphPropertyChange, paragraphPropertiesSnapshot } from "../prosemirror/commands/propertyChangeScope.js";
 import { addedBreakCarrierBefore, finalParagraphsOf, paragraphEndsItsContainer } from "../prosemirror/containerFinalParagraph.js";
@@ -5,12 +6,13 @@ import { requestDeterministicParaIds } from "../prosemirror/extensions/features/
 import { markStructuralChange } from "../prosemirror/extensions/features/ParagraphChangeTrackerExtension.js";
 import { CLEARED_LIST_RENDERING_ATTRS } from "../prosemirror/listMarker.js";
 import { directParagraphAlignment } from "../prosemirror/paragraphAlignment.js";
+import { directParagraphIndentation, paragraphIndentationAttrPatch, paragraphIndentationEqual, paragraphIndentationFromFormatting, withDirectParagraphIndentation } from "../prosemirror/paragraphIndentation.js";
 import { directParagraphSpacing, paragraphSpacingAttrPatch, paragraphSpacingEqual, paragraphSpacingFromFormatting, withDirectParagraphSpacing } from "../prosemirror/paragraphSpacing.js";
 import { getDocumentNumbering } from "../prosemirror/plugins/documentNumbering.js";
 import { getDocumentStyleResolver } from "../prosemirror/plugins/documentStyles.js";
-import { selectRunFormattingCarrierRepresentations } from "../prosemirror/runFormattingInlineCarriers.js";
+import { applyMarksToRunFormattingRepresentation, runFormattingInlineControlNodeName, selectRunFormattingCarrierRepresentations } from "../prosemirror/runFormattingInlineCarriers.js";
 import { readAuthoredRunFormatting, reconcileRunFormattingMarks } from "../prosemirror/runFormattingReconciliation.js";
-import { paragraphRunStyleContextAt } from "../prosemirror/runStyleFormatting.js";
+import { paragraphRunStyleContext, paragraphRunStyleContextAt } from "../prosemirror/runStyleFormatting.js";
 import { listLevelAttrPatch } from "../prosemirror/styles/resolvedStyleAttrs.js";
 import { isZeroWidthAnchor } from "../prosemirror/zeroWidthAnchors.js";
 import { getFolioParaIdFromBlockId } from "../types/block-id.js";
@@ -25,7 +27,7 @@ import { findEnclosingTableCell, findEnclosingTableRow, findOutermostTableBounda
 import { tableFromTemplate } from "./table-template.js";
 import { wordDiffSessionFromOptions } from "./word-diff.js";
 import { panic } from "better-result";
-import { Mark } from "prosemirror-model";
+import { Fragment } from "prosemirror-model";
 import { TableMap } from "prosemirror-tables";
 import { canJoin, canSplit } from "prosemirror-transform";
 //#region src/ai-edits/apply.ts
@@ -149,7 +151,9 @@ const resolveReplaceBlockImpact = ({ changesText, changesStyle }) => {
 const paragraphPropertiesPatch = ({ node, properties, resolvedFormattingFromStyle, numbering = null }) => {
 	const attrs = expectParagraphAttrs(node);
 	const currentDirectAlignment = directParagraphAlignment(attrs);
+	const currentDirectIndentation = directParagraphIndentation(attrs);
 	const currentDirectSpacing = directParagraphSpacing(attrs);
+	const resolvedIndentationFromStyle = paragraphIndentationFromFormatting(resolvedFormattingFromStyle);
 	const resolvedSpacingFromStyle = paragraphSpacingFromFormatting(resolvedFormattingFromStyle);
 	const patch = {};
 	let originalFormatting = attrs._originalFormatting === void 0 || attrs._originalFormatting === null ? void 0 : { ...attrs._originalFormatting };
@@ -164,13 +168,30 @@ const paragraphPropertiesPatch = ({ node, properties, resolvedFormattingFromStyl
 			direct: currentDirectSpacing,
 			inherited: resolvedSpacingFromStyle
 		}));
+		if (properties.indentation === void 0) Object.assign(patch, paragraphIndentationAttrPatch({
+			direct: currentDirectIndentation,
+			inherited: resolvedIndentationFromStyle
+		}));
 		originalFormatting ??= {};
 		if (nextStyleId === null) Reflect.deleteProperty(originalFormatting, "styleId");
 		else originalFormatting.styleId = nextStyleId;
 		if (currentDirectAlignment !== void 0) originalFormatting.alignment = currentDirectAlignment;
 		originalFormattingChanged = true;
 	}
-	if (properties.listLevel !== void 0) {
+	if (properties.numbering !== void 0) if (properties.numbering === null) {
+		patch["numPr"] = null;
+		Object.assign(patch, CLEARED_LIST_RENDERING_ATTRS);
+	} else {
+		Object.assign(patch, listLevelAttrPatch(attrs, {
+			numId: properties.numbering.numId,
+			ilvl: properties.numbering.level
+		}, numbering));
+		if (properties.listLevel === null) {
+			patch["numPr"] = { numId: properties.numbering.numId };
+			patch["numPrFromStyle"] = null;
+		}
+	}
+	else if (properties.listLevel !== void 0) {
 		const numPr = node.attrs["numPr"];
 		if (properties.listLevel === null) {
 			patch["numPr"] = null;
@@ -195,6 +216,15 @@ const paragraphPropertiesPatch = ({ node, properties, resolvedFormattingFromStyl
 		patch["alignment"] = properties.alignment ?? alignmentFromStyle ?? null;
 		originalFormattingChanged = true;
 	}
+	if (properties.indentation !== void 0 && (styleChanged || !paragraphIndentationEqual(currentDirectIndentation, properties.indentation))) {
+		const directIndentation = properties.indentation ?? void 0;
+		originalFormatting = withDirectParagraphIndentation(originalFormatting, directIndentation);
+		Object.assign(patch, paragraphIndentationAttrPatch({
+			direct: directIndentation,
+			inherited: resolvedIndentationFromStyle
+		}));
+		originalFormattingChanged = true;
+	}
 	if (properties.spacing !== void 0 && (styleChanged || !paragraphSpacingEqual(currentDirectSpacing, properties.spacing))) {
 		const directSpacing = properties.spacing ?? void 0;
 		originalFormatting = withDirectParagraphSpacing(originalFormatting, directSpacing);
@@ -271,6 +301,63 @@ const resolveFormattingFromStyle = ({ attrs, styleId, styleResolver }) => {
 	if (attrs.alignmentFromStyle !== void 0) fallback.alignment = attrs.alignmentFromStyle;
 	return Object.keys(fallback).length > 0 ? fallback : void 0;
 };
+const preserveRunFormattingAcrossTrackedStyleChange = ({ allocateRevisionInfo, nextParagraphAttrs, paragraphPosition, styleResolver, tr }) => {
+	const propertyChangeType = tr.doc.type.schema.marks["runPropertyChange"];
+	if (!propertyChangeType) return {
+		tr,
+		revisionIds: []
+	};
+	const paragraph = tr.doc.nodeAt(paragraphPosition);
+	if (!paragraph || paragraph.type.name !== "paragraph") return {
+		tr,
+		revisionIds: []
+	};
+	const previousContext = paragraphRunStyleContext(paragraph, styleResolver);
+	const nextContext = paragraphRunStyleContext(paragraph.type.create(nextParagraphAttrs, paragraph.content, paragraph.marks), styleResolver);
+	const representations = selectRunFormattingCarrierRepresentations({
+		doc: tr.doc,
+		from: paragraphPosition + 1,
+		to: paragraphPosition + paragraph.nodeSize - 1
+	});
+	const revisionIds = [];
+	for (const representation of representations) {
+		if (!representation.node.marks.some(({ type }) => type.name === "deletion")) continue;
+		const existingChange = representation.node.marks.find((mark) => mark.type === propertyChangeType);
+		if (existingChange && expectRunPropertyChangeMarkAttrs(existingChange).changes.length > 0) continue;
+		const previousFormatting = readAuthoredRunFormatting({
+			context: previousContext,
+			marks: representation.node.marks,
+			styleResolver
+		});
+		if (formattingEquals(previousFormatting, readAuthoredRunFormatting({
+			context: nextContext,
+			marks: representation.node.marks,
+			styleResolver
+		}))) continue;
+		const info = allocateRevisionInfo();
+		const suggestionAttrs = info.provenance === "suggested" && info.suggestionId !== void 0 && info.suggestionId !== null ? {
+			provenance: "suggested",
+			suggestionId: info.suggestionId
+		} : {};
+		applyMarksToRunFormattingRepresentation({
+			tr,
+			representation,
+			marks: propertyChangeType.create({
+				changes: [{
+					type: "runPropertyChange",
+					info,
+					...Object.keys(previousFormatting).length > 0 && { previousFormatting }
+				}],
+				...suggestionAttrs
+			}).addToSet(representation.node.marks)
+		});
+		revisionIds.push(info.id);
+	}
+	return {
+		tr,
+		revisionIds
+	};
+};
 /**
 * Apply one paragraph-property patch to a live node. A tracked patch stores
 * the complete previous pPr: a partial snapshot cannot distinguish a property
@@ -296,7 +383,8 @@ const applyBlockParagraphProperties = ({ tr, position, node, properties, styleRe
 	if (patch === null) return {
 		tr,
 		changed: false,
-		revisionId: null
+		revisionId: null,
+		revisionIds: []
 	};
 	const changeInfo = revisionInfo?.();
 	const change = changeInfo ? {
@@ -304,27 +392,42 @@ const applyBlockParagraphProperties = ({ tr, position, node, properties, styleRe
 		info: changeInfo,
 		previousFormatting: paragraphPropertiesSnapshot(node)
 	} : null;
+	const preserveRunFormatting = change !== null && properties.styleId !== void 0 && (attrs.styleId ?? null) !== properties.styleId;
 	const existing = attrs._propertyChanges;
+	const nextAttrs = {
+		...node.attrs,
+		...patch,
+		...change ? { _propertyChanges: [...Array.isArray(existing) ? existing : [], change] } : {}
+	};
+	const bridgeResult = preserveRunFormatting && revisionInfo ? preserveRunFormattingAcrossTrackedStyleChange({
+		allocateRevisionInfo: revisionInfo,
+		nextParagraphAttrs: nextAttrs,
+		paragraphPosition: position,
+		styleResolver,
+		tr
+	}) : {
+		tr,
+		revisionIds: []
+	};
 	return {
-		tr: tr.setNodeMarkup(position, void 0, {
-			...node.attrs,
-			...patch,
-			...change ? { _propertyChanges: [...Array.isArray(existing) ? existing : [], change] } : {}
-		}),
+		tr: bridgeResult.tr.setNodeMarkup(position, void 0, nextAttrs),
 		changed: true,
-		revisionId: change?.info.id ?? null
+		revisionId: change?.info.id ?? null,
+		revisionIds: change ? [change.info.id, ...bridgeResult.revisionIds] : []
 	};
 };
 const applyReplaceBlockStyleId = ({ item, tr, styleResolver, revisionInfo }) => {
 	if (item.operation.type !== "replaceBlock" || item.operation.styleId === void 0) return {
 		tr,
-		revisionId: null
+		revisionId: null,
+		revisionIds: []
 	};
 	const blockPosition = tr.mapping.map(item.blockFrom, -1);
 	const block = tr.doc.nodeAt(blockPosition);
 	if (!block) return {
 		tr,
-		revisionId: null
+		revisionId: null,
+		revisionIds: []
 	};
 	const attrs = expectParagraphAttrs(block);
 	const resolvedFormattingFromStyle = resolveFormattingFromStyle({
@@ -339,21 +442,36 @@ const applyReplaceBlockStyleId = ({ item, tr, styleResolver, revisionInfo }) =>
 	});
 	if (patch === null) return {
 		tr,
-		revisionId: null
+		revisionId: null,
+		revisionIds: []
 	};
 	const existing = attrs._propertyChanges;
-	const change = revisionInfo ? {
+	const changeInfo = revisionInfo?.();
+	const change = changeInfo ? {
 		type: "paragraphPropertyChange",
-		info: revisionInfo,
+		info: changeInfo,
 		previousFormatting: paragraphPropertiesSnapshot(block)
 	} : null;
+	const preserveRunFormatting = change !== null && (attrs.styleId ?? null) !== item.operation.styleId;
+	const nextAttrs = {
+		...block.attrs,
+		...patch,
+		...change ? { _propertyChanges: [...Array.isArray(existing) ? existing : [], change] } : {}
+	};
+	const bridgeResult = preserveRunFormatting && revisionInfo ? preserveRunFormattingAcrossTrackedStyleChange({
+		allocateRevisionInfo: revisionInfo,
+		nextParagraphAttrs: nextAttrs,
+		paragraphPosition: blockPosition,
+		styleResolver,
+		tr
+	}) : {
+		tr,
+		revisionIds: []
+	};
 	return {
-		tr: tr.setNodeMarkup(blockPosition, void 0, {
-			...block.attrs,
-			...patch,
-			...change ? { _propertyChanges: [...Array.isArray(existing) ? existing : [], change] } : {}
-		}),
-		revisionId: change?.info.id ?? null
+		tr: bridgeResult.tr.setNodeMarkup(blockPosition, void 0, nextAttrs),
+		revisionId: change?.info.id ?? null,
+		revisionIds: change ? [change.info.id, ...bridgeResult.revisionIds] : []
 	};
 };
 const REPLACEMENT_BACKGROUND_CLEAR_FORMATTING = {
@@ -406,34 +524,23 @@ const applyInlineFormatting = ({ tr, from, to, formatting, includedProperties, s
 			marks: representation.node.marks,
 			...styleResolver !== void 0 ? { styleResolver } : {}
 		});
-		const marks = reconcileRunFormattingMarks({
-			authoredFormatting: patchAuthoredRunFormatting({
-				authoredFormatting,
-				formatting,
-				...includedProperties !== void 0 ? { includedProperties } : {}
-			}),
-			context: styleContext,
-			node: representation.node,
-			...styleResolver !== void 0 ? { styleResolver } : {}
-		});
 		applyMarksToRunFormattingRepresentation({
 			tr,
 			representation,
-			marks
+			marks: reconcileRunFormattingMarks({
+				authoredFormatting: patchAuthoredRunFormatting({
+					authoredFormatting,
+					formatting,
+					...includedProperties !== void 0 ? { includedProperties } : {}
+				}),
+				context: styleContext,
+				node: representation.node,
+				...styleResolver !== void 0 ? { styleResolver } : {}
+			})
 		});
 	}
 	return tr;
 };
-const applyMarksToRunFormattingRepresentation = ({ tr, representation, marks }) => {
-	const { node, position, from, to } = representation;
-	if (Mark.sameSet(node.marks, marks)) return;
-	if (!node.isText) {
-		tr.setNodeMarkup(position, void 0, node.attrs, marks);
-		return;
-	}
-	for (const current of node.marks) if (!marks.some((candidate) => candidate.eq(current))) tr.removeMark(from, to, current.type);
-	for (const next of marks) if (!node.marks.some((candidate) => candidate.eq(next))) tr.addMark(from, to, next);
-};
 const sameDefinedFormattingValue = (left, right) => {
 	if (left === right) return true;
 	if (typeof left !== "object" || left === null || typeof right !== "object" || right === null) return false;
@@ -750,7 +857,11 @@ const buildTableNode = ({ schema, rows, revision }) => {
 		...revision.provenance != null && { provenance: revision.provenance },
 		...revision.suggestionId != null && { suggestionId: revision.suggestionId }
 	})] : void 0;
-	const rowNodes = rows.map((cells) => rowType.create(revision ? { trIns: revision } : null, cells.map((text) => cellType.create(null, splitCellParagraphTexts(text).map((line) => paragraphType.create(null, line.length > 0 ? schema.text(line, marks) : null))))));
+	const rowNodes = rows.map((cells) => rowType.create(revision ? { trIns: revision } : null, cells.map((text) => cellType.create(null, splitCellParagraphTexts(text).map((line) => paragraphType.create(null, line.length > 0 ? cleanTextInlineNodes({
+		schema,
+		text: line,
+		marks
+	}) : null))))));
 	return tableType.create(null, rowNodes);
 };
 /**
@@ -1005,34 +1116,61 @@ const buildInsertedParagraphs = ({ item, schema, formattingFromStyle, mode, auth
 			revisionIds.push(paragraphRevisionId);
 		}
 		if (commentMark) marks.push(commentMark);
-		const content = text.length > 0 ? buildEmphasisInlineContent(schema, text, marks) : null;
+		const hardPageBreak = isFirstParagraph ? operation.hardPageBreak : void 0;
+		let content = null;
+		if (hardPageBreak !== void 0) {
+			const pageBreakRun = schema.nodes["pageBreakRun"];
+			if (pageBreakRun === void 0) panic("The schema has no page-break run node", { operationId: operation.id });
+			content = [pageBreakRun.create(hardPageBreak, null, marks)];
+		} else if (text.length > 0) content = hasCleanTextControls(text) ? cleanTextInlineNodes({
+			schema,
+			text,
+			marks
+		}) : buildEmphasisInlineContent(schema, text, marks);
 		const attrs = isFirstParagraph ? { ...baseAttrs } : {};
 		if (isFirstParagraph && operation.pageBreakBefore === true) attrs["pageBreakBefore"] = true;
+		const explicitNumbering = operation.numbering;
 		const listLevel = operation.listLevel;
-		if (isFirstParagraph && listLevel === null) {
-			attrs["numPr"] = null;
-			Object.assign(attrs, CLEARED_LIST_RENDERING_ATTRS);
-		} else if (isFirstParagraph && typeof listLevel === "number") {
-			const anchorNumPr = Reflect.get(baseAttrs, "numPr");
-			const numId = typeof anchorNumPr === "object" && anchorNumPr !== null && "numId" in anchorNumPr ? anchorNumPr.numId : void 0;
-			if (typeof numId === "number") Object.assign(attrs, listLevelAttrPatch(operation.inheritFormatting === false ? {} : expectParagraphAttrs(item.blockNode), {
-				numId,
-				ilvl: listLevel
-			}, numbering));
-			else {
-				attrs["numPr"] = { ilvl: listLevel };
+		if (isFirstParagraph) {
+			if (explicitNumbering === null) {
+				attrs["numPr"] = null;
 				Object.assign(attrs, CLEARED_LIST_RENDERING_ATTRS);
+			} else if (explicitNumbering !== void 0) {
+				Object.assign(attrs, listLevelAttrPatch(operation.inheritFormatting === false ? {} : expectParagraphAttrs(item.blockNode), {
+					numId: explicitNumbering.numId,
+					ilvl: explicitNumbering.level
+				}, numbering));
+				if (listLevel === null) {
+					attrs["numPr"] = { numId: explicitNumbering.numId };
+					attrs["numPrFromStyle"] = null;
+				}
+			} else if (listLevel === null) {
+				attrs["numPr"] = null;
+				Object.assign(attrs, CLEARED_LIST_RENDERING_ATTRS);
+			} else if (typeof listLevel === "number") {
+				const anchorNumPr = Reflect.get(baseAttrs, "numPr");
+				const numId = typeof anchorNumPr === "object" && anchorNumPr !== null && "numId" in anchorNumPr ? anchorNumPr.numId : void 0;
+				if (typeof numId === "number") Object.assign(attrs, listLevelAttrPatch(operation.inheritFormatting === false ? {} : expectParagraphAttrs(item.blockNode), {
+					numId,
+					ilvl: listLevel
+				}, numbering));
+				else {
+					attrs["numPr"] = { ilvl: listLevel };
+					Object.assign(attrs, CLEARED_LIST_RENDERING_ATTRS);
+				}
 			}
 		}
 		if (isFirstParagraph && operation.styleId !== void 0) {
 			attrs["styleId"] = operation.styleId;
 			if (operation.inheritFormatting !== false && operation.styleId !== null) Object.assign(attrs, CLEARED_LIST_RENDERING_ATTRS);
 		}
-		if (isFirstParagraph && (operation.styleId !== void 0 || operation.alignment !== void 0 || operation.spacing !== void 0)) {
+		if (isFirstParagraph && (operation.styleId !== void 0 || operation.alignment !== void 0 || operation.spacing !== void 0 || operation.indentation !== void 0)) {
 			const inheritedDirectAlignment = operation.inheritFormatting === false ? void 0 : directParagraphAlignment(expectParagraphAttrs(item.blockNode));
 			const inheritedDirectSpacing = operation.inheritFormatting === false ? void 0 : directParagraphSpacing(expectParagraphAttrs(item.blockNode));
+			const inheritedDirectIndentation = operation.inheritFormatting === false ? void 0 : directParagraphIndentation(expectParagraphAttrs(item.blockNode));
 			const directAlignment = operation.alignment === void 0 ? inheritedDirectAlignment : operation.alignment ?? void 0;
 			const directSpacing = operation.spacing === void 0 ? inheritedDirectSpacing : operation.spacing ?? void 0;
+			const directIndentation = operation.indentation === void 0 ? inheritedDirectIndentation : operation.indentation ?? void 0;
 			attrs["alignmentFromStyle"] = formattingFromStyle?.alignment;
 			attrs["alignment"] = directAlignment ?? formattingFromStyle?.alignment ?? null;
 			const sourceFormatting = attrs["_originalFormatting"];
@@ -1048,10 +1186,15 @@ const buildInsertedParagraphs = ({ item, schema, formattingFromStyle, mode, auth
 				else originalFormatting.alignment = directAlignment;
 			}
 			if (operation.spacing !== void 0) originalFormatting = withDirectParagraphSpacing(originalFormatting, directSpacing);
+			if (operation.indentation !== void 0) originalFormatting = withDirectParagraphIndentation(originalFormatting, directIndentation);
 			if (operation.styleId !== void 0 || operation.spacing !== void 0) Object.assign(attrs, paragraphSpacingAttrPatch({
 				direct: directSpacing,
 				inherited: paragraphSpacingFromFormatting(formattingFromStyle)
 			}));
+			if (operation.styleId !== void 0 || operation.indentation !== void 0) Object.assign(attrs, paragraphIndentationAttrPatch({
+				direct: directIndentation,
+				inherited: paragraphIndentationFromFormatting(formattingFromStyle)
+			}));
 			attrs["_originalFormatting"] = originalFormatting && Object.keys(originalFormatting).length > 0 ? originalFormatting : null;
 		}
 		if (isSuggested && suggestionId !== null && paragraphRevisionId !== null) attrs["_suggestedInsert"] = {
@@ -1198,7 +1341,7 @@ const applyFolioAIEditOperationsInternal = ({ view, snapshot, operations, mode =
 			}
 			claimedTableColumns.add(columnKey);
 		}
-		if ((operation.type === "insertAfterBlock" || operation.type === "insertBeforeBlock") && LINE_BREAK_PATTERN.test(operation.text)) normalizations.push({
+		if ((operation.type === "insertAfterBlock" || operation.type === "insertBeforeBlock") && operation.lineBreakMode !== "inline" && LINE_BREAK_PATTERN.test(operation.text)) normalizations.push({
 			id: operation.id,
 			code: "splitMultilineText",
 			paragraphCount: resolution.operation.insertTexts?.length ?? 1
@@ -1525,38 +1668,29 @@ const applyFolioAIEditOperationsInternal = ({ view, snapshot, operations, mode =
 						diffText
 					});
 				}
-				const revisionIdParagraph = producesTrackedChanges && changesStyle ? operationRevisionSeed++ : null;
+				const allocateStyleRevisionInfo = () => ({
+					id: operationRevisionSeed++,
+					author,
+					date,
+					...trackedRevisionExtras
+				});
 				const styleResult = applyReplaceBlockStyleId({
 					item,
 					tr,
 					styleResolver,
-					...revisionIdParagraph !== null ? { revisionInfo: {
-						id: revisionIdParagraph,
-						author,
-						date,
-						...trackedRevisionExtras
-					} } : {}
+					...producesTrackedChanges && changesStyle ? { revisionInfo: allocateStyleRevisionInfo } : {}
 				});
 				tr = styleResult.tr;
 				if (!changesText && commentMark) tr = tr.addMark(item.from, item.to, commentMark);
 				if (producesTrackedChanges) appliedRevisionIds = [
 					...changesText ? [revisionIdDelete, revisionIdInsert] : [],
 					...clearedBackground ? backgroundRevisionIds : [],
-					...styleResult.revisionId === null ? [] : [styleResult.revisionId]
+					...styleResult.revisionIds
 				];
 				break;
 			}
 			case "insertAfterBlock":
 			case "insertBeforeBlock": {
-				const insertTexts = item.insertTexts ?? [""];
-				const isEmptyInsert = insertTexts.length === 1 && insertTexts[0]?.length === 0;
-				if (mode === "tracked-changes" && item.operation.pageBreakBefore === true && isEmptyInsert) {
-					skipped.push({
-						id: item.operation.id,
-						reason: "unsupportedMode"
-					});
-					continue;
-				}
 				const built = buildInsertedParagraphs({
 					item,
 					schema: view.state.schema,
@@ -1971,7 +2105,7 @@ const applyFolioAIEditOperationsInternal = ({ view, snapshot, operations, mode =
 					continue;
 				}
 				tr = appliedProperties.tr;
-				if (appliedProperties.revisionId !== null) appliedRevisionIds = [appliedProperties.revisionId];
+				if (appliedProperties.revisionIds.length > 0) appliedRevisionIds = [...appliedProperties.revisionIds];
 				break;
 			}
 			case "splitBlock": {
@@ -2030,7 +2164,7 @@ const applyFolioAIEditOperationsInternal = ({ view, snapshot, operations, mode =
 						...mode === "direct" ? {} : { revisionInfo: allocateSplitParagraphPropertyRevisionInfo }
 					});
 					tr = appliedProperties.tr;
-					if (appliedProperties.revisionId !== null) appliedRevisionIds.push(appliedProperties.revisionId);
+					if (appliedProperties.revisionIds.length > 0) appliedRevisionIds.push(...appliedProperties.revisionIds);
 				}
 				break;
 			}
@@ -2082,7 +2216,7 @@ const applyFolioAIEditOperationsInternal = ({ view, snapshot, operations, mode =
 						}) }
 					});
 					tr = appliedProperties.tr;
-					if (appliedProperties.revisionId !== null) appliedRevisionIds.push(appliedProperties.revisionId);
+					if (appliedProperties.revisionIds.length > 0) appliedRevisionIds.push(...appliedProperties.revisionIds);
 				}
 				break;
 			}
@@ -2185,6 +2319,41 @@ const previewFolioAIEditOperations = (options) => {
 		nextRevisionId: options.revisionStamp?.idSeed ?? revisionIdCursor
 	};
 };
+const CLEAN_TEXT_CONTROL_PATTERN = /[\t\n]/gu;
+const hasCleanTextControls = (text) => text.includes("	") || text.includes("\n");
+const cleanTextInlineNodes = ({ schema, text, marks = [] }) => {
+	const nodes = [];
+	let offset = 0;
+	for (const match of text.matchAll(CLEAN_TEXT_CONTROL_PATTERN)) {
+		const index = match.index;
+		const preceding = text.slice(offset, index);
+		if (preceding.length > 0) nodes.push(schema.text(preceding, marks));
+		const control = match[0];
+		const nodeName = runFormattingInlineControlNodeName(control);
+		if (nodeName === null) panic("A clean-text control pattern matched an unrepresentable character", { control });
+		const type = schema.nodes[nodeName];
+		if (!type) panic("The editor schema cannot represent a clean-text control character", { control });
+		nodes.push(type.create(null, null, marks));
+		offset = index + control.length;
+	}
+	const trailing = text.slice(offset);
+	if (trailing.length > 0) nodes.push(schema.text(trailing, marks));
+	return nodes;
+};
+const insertCleanText = ({ tr, from, to, text }) => {
+	if (!hasCleanTextControls(text)) return {
+		transaction: tr.insertText(text, from, to),
+		end: from + text.length
+	};
+	const content = Fragment.fromArray(cleanTextInlineNodes({
+		schema: tr.doc.type.schema,
+		text
+	}));
+	return {
+		transaction: tr.replaceWith(from, to, content),
+		end: from + content.size
+	};
+};
 const applyTextReplacement = ({ tr, item, mode, author, date, revisionIdDelete, revisionIdInsert, commentMark, suggestionId = null, initials, diffText }) => {
 	let nextTr = tr;
 	const replacement = stripInlineEmphasisMarkers((() => {
@@ -2197,8 +2366,14 @@ const applyTextReplacement = ({ tr, item, mode, author, date, revisionIdDelete,
 			const content = buildEmphasisInlineContent(nextTr.doc.type.schema, item.operation.replace, commentMark ? [commentMark] : []);
 			return nextTr.replaceWith(item.from, item.to, content);
 		}
-		nextTr = nextTr.insertText(replacement, item.from, item.to);
-		if (commentMark && replacement.length > 0) nextTr = nextTr.addMark(item.from, item.from + replacement.length, commentMark);
+		const inserted = insertCleanText({
+			tr: nextTr,
+			from: item.from,
+			to: item.to,
+			text: replacement
+		});
+		nextTr = inserted.transaction;
+		if (commentMark && replacement.length > 0) nextTr = nextTr.addMark(item.from, inserted.end, commentMark);
 		return nextTr;
 	}
 	const insertionType = nextTr.doc.type.schema.marks["insertion"];
@@ -2284,18 +2459,30 @@ const applyTextReplacement = ({ tr, item, mode, author, date, revisionIdDelete,
 					continue;
 				}
 				if (step.kind === "ins" && insertionType) {
-					nextTr = nextTr.insertText(step.text, step.at, step.at);
-					nextTr = nextTr.addMark(step.at, step.at + step.text.length, insertionType.create(insAttrs));
-					if (commentMark) nextTr = nextTr.addMark(step.at, step.at + step.text.length, commentMark);
+					const inserted = insertCleanText({
+						tr: nextTr,
+						from: step.at,
+						to: step.at,
+						text: step.text
+					});
+					nextTr = inserted.transaction;
+					nextTr = nextTr.addMark(step.at, inserted.end, insertionType.create(insAttrs));
+					if (commentMark) nextTr = nextTr.addMark(step.at, inserted.end, commentMark);
 				}
 			}
 			return nextTr;
 		}
 	}
 	if (replacement.length > 0 && insertionType) {
-		nextTr = nextTr.insertText(replacement, item.to, item.to);
-		nextTr = nextTr.addMark(item.to, item.to + replacement.length, insertionType.create(insAttrs));
-		if (commentMark) nextTr = nextTr.addMark(item.to, item.to + replacement.length, commentMark);
+		const inserted = insertCleanText({
+			tr: nextTr,
+			from: item.to,
+			to: item.to,
+			text: replacement
+		});
+		nextTr = inserted.transaction;
+		nextTr = nextTr.addMark(item.to, inserted.end, insertionType.create(insAttrs));
+		if (commentMark) nextTr = nextTr.addMark(item.to, inserted.end, commentMark);
 	}
 	if (item.to > item.from && deletionType) {
 		nextTr = nextTr.addMark(item.from, item.to, deletionType.create(delAttrs));
@@ -2403,7 +2590,7 @@ const resolveOperation = ({ snapshot, operation, liveBlocks, liveBlocksByParaId,
 		let insertFrom;
 		if (tableBoundary) insertFrom = isInsertAfter ? tableBoundary.after : tableBoundary.before;
 		else insertFrom = isInsertAfter ? blockTo : blockFrom;
-		const insertTexts = LINE_BREAK_PATTERN.test(operation.text) ? splitInsertParagraphTexts(operation.text) : [operation.text];
+		const insertTexts = operation.lineBreakMode !== "inline" && LINE_BREAK_PATTERN.test(operation.text) ? splitInsertParagraphTexts(operation.text) : [operation.text];
 		return {
 			type: "resolved",
 			operation: {

package/dist/ai-edits/clean-text.js

@@ -1,4 +1,5 @@
 import { expectPageBreakRunAttrs } from "../prosemirror/attrs/index.js";
+import { runFormattingInlineControlCharacter } from "../prosemirror/runFormattingInlineCarriers.js";
 //#region src/ai-edits/clean-text.ts
 const EMPTY_CLEAN_TEXT_STRUCTURAL_BOUNDARIES = Object.freeze([]);
 /**
@@ -38,6 +39,7 @@ const DELETION_MARK = "deletion";
 const INSERTION_MARK = "insertion";
 const COMMENT_MARK = "comment";
 const HIDDEN_MARK = "hidden";
+const isOmittedFromCleanView = (node) => node.marks.some((mark) => mark.type.name === DELETION_MARK || mark.type.name === HIDDEN_MARK);
 const buildCleanBlockText = (blockNode, blockFrom) => {
 	let text = "";
 	const offsets = [];
@@ -57,8 +59,17 @@ const buildCleanBlockText = (blockNode, blockFrom) => {
 			});
 			return false;
 		}
+		const controlCharacter = runFormattingInlineControlCharacter(node);
+		if (controlCharacter !== null) {
+			if (isOmittedFromCleanView(node)) return false;
+			const startPos = blockFrom + 1 + pos;
+			offsets.push(startPos);
+			text += controlCharacter;
+			lastEnd = startPos + node.nodeSize;
+			return false;
+		}
 		if (!node.isText || node.text === void 0) return true;
-		if (node.marks.some((mark) => mark.type.name === DELETION_MARK || mark.type.name === HIDDEN_MARK)) return false;
+		if (isOmittedFromCleanView(node)) return false;
 		const startPos = blockFrom + 1 + pos;
 		for (let i = 0; i < node.text.length; i++) offsets.push(startPos + i);
 		text += node.text;