@stll/folio-core 0.54.2 → 0.56.0

package/dist/docx/imageParser.js

@@ -10,7 +10,7 @@ import { RELATIONSHIP_TYPES, resolveRelationshipIdOfType } from "./relsParser.js
 import { isTextBoxDrawing } from "./textBoxParser.js";
 import { percentageSpelling } from "./transitionalSpelling.js";
 import { captureVerbatimXml } from "./verbatimCapture.js";
-import { findByFullName, findChildByNamespaceUri, findChildrenByNamespaceUri, getAttribute, getChildElements, getLocalName, getNamespaceUri, parseNumericAttribute, parseOnOffAttribute, parseOnOffValue } from "./xmlParser.js";
+import { OFFICE_RELATIONSHIP_NAMESPACE_URIS, findAttributeByNamespaceUri, findByFullName, findChildByNamespaceUri, findChildrenByNamespaceUri, getAttribute, getChildElements, getLocalName, getNamespaceUri, parseNumericAttribute, parseOnOffAttribute, parseOnOffValue } from "./xmlParser.js";
 import { relationshipIdOf } from "@stll/docx-core/model";
 //#region src/docx/imageParser.ts
 /**
@@ -129,7 +129,7 @@ const parseDocPropsExtensions = (docPr) => {
 const parseDocPrLink = (docPr, localName) => {
 	const element = findChildByNamespaceUri(docPr, DRAWINGML_NAMESPACE_URIS, localName);
 	if (!element) return;
-	const rId = getAttribute(element, "r", "id");
+	const rId = findAttributeByNamespaceUri(element, OFFICE_RELATIONSHIP_NAMESPACE_URIS, "id")?.value;
 	return {
 		xml: captureVerbatimXml(element),
 		...rId == null ? {} : { rId }
@@ -382,6 +382,13 @@ function resolveImageData(rId, rels, media) {
 		...filename !== void 0 ? { filename } : {}
 	};
 }
+/** Facts the parser obtains from exported drawing markup and image relationships. */
+const imageDerivedFactsOf = ({ id, pictureNames, imageData }) => ({
+	...id ? { id } : {},
+	...Object.keys(pictureNames).length > 0 ? { pictureNames } : {},
+	...imageData.mimeType ? { mimeType: imageData.mimeType } : {},
+	...imageData.filename ? { filename: imageData.filename } : {}
+});
 /**
 * Parse a wp:inline element (inline image)
 *
@@ -411,18 +418,19 @@ function parseInline(inlineEl, rels, media) {
 		size,
 		wrap
 	};
-	if (props.id) image.id = props.id;
+	Object.assign(image, imageDerivedFactsOf({
+		id: props.id,
+		pictureNames,
+		imageData
+	}));
 	if (props.name !== void 0) image.docPrName = props.name;
 	if (props.alt !== void 0) image.alt = props.alt;
 	if (props.title !== void 0) image.title = props.title;
-	if (Object.keys(pictureNames).length > 0) image.pictureNames = pictureNames;
 	if (props.decorative !== void 0) image.decorative = props.decorative;
 	if (props.hidden !== void 0) image.hidden = props.hidden;
 	if (props.docPrExtensions !== void 0) image.docPrExtensions = props.docPrExtensions;
 	const safeSrc = sanitizeImageSrc(imageData.src);
 	if (safeSrc) image.src = safeSrc;
-	if (imageData.mimeType) image.mimeType = imageData.mimeType;
-	if (imageData.filename) image.filename = imageData.filename;
 	if (padding) image.padding = padding;
 	if (transform) image.transform = transform;
 	if (crop) image.crop = crop;
@@ -492,18 +500,19 @@ function parseAnchor(anchorEl, rels, media) {
 		size,
 		wrap
 	};
-	if (props.id) image.id = props.id;
+	Object.assign(image, imageDerivedFactsOf({
+		id: props.id,
+		pictureNames,
+		imageData
+	}));
 	if (props.name !== void 0) image.docPrName = props.name;
 	if (props.alt !== void 0) image.alt = props.alt;
 	if (props.title !== void 0) image.title = props.title;
-	if (Object.keys(pictureNames).length > 0) image.pictureNames = pictureNames;
 	if (props.decorative !== void 0) image.decorative = props.decorative;
 	if (props.hidden !== void 0) image.hidden = props.hidden;
 	if (props.docPrExtensions !== void 0) image.docPrExtensions = props.docPrExtensions;
 	const safeSrc = sanitizeImageSrc(imageData.src);
 	if (safeSrc) image.src = safeSrc;
-	if (imageData.mimeType) image.mimeType = imageData.mimeType;
-	if (imageData.filename) image.filename = imageData.filename;
 	if (position) image.position = position;
 	if (padding) image.padding = padding;
 	if (transform) image.transform = transform;
@@ -628,4 +637,4 @@ function needsTextWrapping(image) {
 	].includes(image.wrap.type);
 }
 //#endregion
-export { DECORATIVE_EXTENSION_URI, DECORATIVE_NAMESPACE, getImageDimensionsPx, getImageHeightPx, getImageWidthPx, getWrapDistancesPx, hasAltText, isBehindText, isDecorativeImage, isFloatingImage, isInFrontOfText, isInlineImage, needsTextWrapping, parseDrawing, parseImage, resolveImageData };
+export { DECORATIVE_EXTENSION_URI, DECORATIVE_NAMESPACE, getImageDimensionsPx, getImageHeightPx, getImageWidthPx, getWrapDistancesPx, hasAltText, imageDerivedFactsOf, isBehindText, isDecorativeImage, isFloatingImage, isInFrontOfText, isInlineImage, needsTextWrapping, parseDrawing, parseImage, resolveImageData };

package/dist/docx/metadataPrivacy.js

@@ -1,6 +1,7 @@
 import { countCentralDirectoryRecords, createInflationBudget, inflateEntryWithinLimits } from "./archiveInflation.js";
 import { elementToXml, getLocalName, parseXmlDocument } from "./xmlParser.js";
 import { TaggedError } from "better-result";
+import { writeZipPart } from "@stll/docx-core/zip";
 import JSZip from "jszip";
 //#region src/docx/metadataPrivacy.ts
 const FOLIO_DOCUMENT_METADATA_PROPERTIES = Object.freeze([
@@ -152,7 +153,11 @@ const rewriteDocxMetadataPrivacy = async (buffer, { transforms }) => {
 			removedMetadataProperties: []
 		}
 	};
-	zip.file(coreProperties.name, rewritten.xml);
+	writeZipPart({
+		zip,
+		path: coreProperties.name,
+		data: rewritten.xml
+	});
 	return {
 		buffer: await zip.generateAsync({
 			type: "arraybuffer",

package/dist/docx/noteIds.d.ts

@@ -1,4 +1,9 @@
 //#region src/docx/noteIds.d.ts
+/**
+ * The next free endnote id among `existingIds`, a document's endnotes. Ids are
+ * unique per document, so the allocator lives for one call: the result
+ * depends only on the document, never on what other documents minted.
+ */
 declare const mintEndnoteId: (existingIds: Iterable<number>) => number;
 //#endregion
 export { mintEndnoteId };

package/dist/docx/noteIds.js

@@ -1,8 +1,12 @@
 import { createNumericIdAllocator } from "./numericIdAllocator.js";
 //#region src/docx/noteIds.ts
-let endnoteIds;
+/**
+* The next free endnote id among `existingIds`, a document's endnotes. Ids are
+* unique per document, so the allocator lives for one call: the result
+* depends only on the document, never on what other documents minted.
+*/
 const mintEndnoteId = (existingIds) => {
-	const allocator = endnoteIds ??= createNumericIdAllocator({
+	const allocator = createNumericIdAllocator({
 		space: "endnote",
 		firstId: 1
 	});

package/dist/docx/noteReferenceMark.d.ts

@@ -3,6 +3,8 @@ import { document_d_exports } from "../types/document.js";
 type NoteKind = "footnote" | "endnote";
 /** The run carrying a note's in-note reference mark, in the note reference style. */
 declare const noteReferenceMarkRun: (kind: NoteKind) => document_d_exports.Run;
+/** Whether a note already owns its typed marker, including inside tracked content. */
+declare const hasNoteReferenceMark: (kind: NoteKind, content: readonly document_d_exports.BlockContent[]) => boolean;
 /**
  * A note's content with its reference mark run at the start of its first
  * paragraph, added when the note has none.
@@ -12,4 +14,4 @@ declare const withNoteReferenceMark: (kind: NoteKind, content: readonly document
 declare function createNote(kind: "footnote", id: number, content: readonly document_d_exports.BlockContent[]): document_d_exports.Footnote;
 declare function createNote(kind: "endnote", id: number, content: readonly document_d_exports.BlockContent[]): document_d_exports.Endnote;
 //#endregion
-export { createNote, noteReferenceMarkRun, withNoteReferenceMark };
+export { createNote, hasNoteReferenceMark, noteReferenceMarkRun, withNoteReferenceMark };

package/dist/docx/noteReferenceMark.js

@@ -1,37 +1,38 @@
 import { cloneParagraphWithPropertySource } from "./paragraphPropertySource.js";
+import { visitDocxParagraphs, visitParagraphRuns } from "./paragraphTraversal.js";
 //#region src/docx/noteReferenceMark.ts
-const MARK_ELEMENT = /<(?:[\w.-]+:)?(?:footnote|endnote)Ref[\s/>]/u;
 /** The run carrying a note's in-note reference mark, in the note reference style. */
 const noteReferenceMarkRun = (kind) => ({
 	type: "run",
 	formatting: { styleId: kind === "footnote" ? "FootnoteReference" : "EndnoteReference" },
 	content: [{
-		type: "preservedXml",
-		xml: `<w:${kind}Ref/>`,
-		text: ""
+		type: "noteMarker",
+		kind
 	}]
 });
-const holdsReferenceMark = (value) => {
-	if (Array.isArray(value)) return value.some(holdsReferenceMark);
-	if (typeof value !== "object" || value === null) return false;
-	const record = value;
-	if (record["type"] === "preservedXml" && typeof record["xml"] === "string") return MARK_ELEMENT.test(record["xml"]);
-	return Object.values(record).some(holdsReferenceMark);
+/** Whether a note already owns its typed marker, including inside tracked content. */
+const hasNoteReferenceMark = (kind, content) => {
+	let found = false;
+	visitDocxParagraphs({ documentBody: { content: [...content] } }, (paragraph) => {
+		visitParagraphRuns(paragraph, (run) => {
+			if (run.content.some((child) => child.type === "noteMarker" && child.kind === kind)) found = true;
+		});
+	});
+	return found;
 };
 /**
 * A note's content with its reference mark run at the start of its first
 * paragraph, added when the note has none.
 */
 const withNoteReferenceMark = (kind, content) => {
-	if (holdsReferenceMark(content)) return [...content];
+	if (hasNoteReferenceMark(kind, content)) return [...content];
 	const index = content.findIndex((block) => block.type === "paragraph");
 	const mark = noteReferenceMarkRun(kind);
 	if (index === -1) return [{
 		type: "paragraph",
 		content: [mark]
 	}, ...content];
-	const paragraph = content[index];
-	return content.map((block, position) => position === index ? cloneParagraphWithPropertySource(paragraph, { content: [mark, ...paragraph.content] }) : block);
+	return content.map((block, position) => position === index && block.type === "paragraph" ? cloneParagraphWithPropertySource(block, { content: [mark, ...block.content] }) : block);
 };
 function createNote(kind, id, content) {
 	return kind === "footnote" ? {
@@ -45,4 +46,4 @@ function createNote(kind, id, content) {
 	};
 }
 //#endregion
-export { createNote, noteReferenceMarkRun, withNoteReferenceMark };
+export { createNote, hasNoteReferenceMark, noteReferenceMarkRun, withNoteReferenceMark };

package/dist/docx/numberingParser.d.ts

@@ -81,7 +81,9 @@ declare function getBulletCharacter(level: document_d_exports.ListLevel): string
  * Check if a list level is a bullet (not numbered)
  */
 declare function isBulletLevel(level: document_d_exports.ListLevel): boolean;
+/** Whether a numbering definition uses a bullet marker rather than a numeric or hidden one. */
+declare const numberingLevelUsesBulletMarker: (level: Pick<document_d_exports.ListLevel, "numFmt"> | null | undefined) => boolean;
 /** Whether a numbering level reserves horizontal space for a visible marker. */
 declare const numberingLevelHasMarkerSlot: (level: Pick<document_d_exports.ListLevel, "numFmt">) => boolean;
 //#endregion
-export { NumberingMap, computeListRendering, counterFormatOf, createNumberingMap, customNumberFormatPadWidth, formatOoxmlCounter as formatNumber, getBulletCharacter, getCachedNumberingMap, isBulletLevel, markerAlignmentForLevel, markerFormattingFromLevel, numberingLevelHasMarkerSlot, padDecimal, parseNumbering, renderListMarker };
+export { NumberingMap, computeListRendering, counterFormatOf, createNumberingMap, customNumberFormatPadWidth, formatOoxmlCounter as formatNumber, getBulletCharacter, getCachedNumberingMap, isBulletLevel, markerAlignmentForLevel, markerFormattingFromLevel, numberingLevelHasMarkerSlot, numberingLevelUsesBulletMarker, padDecimal, parseNumbering, renderListMarker };

package/dist/docx/numberingParser.js

@@ -6,7 +6,20 @@ import { parseParagraphProperties } from "./paragraphProperties.js";
 import { LevelSuffixSchema, NumberFormatSchema, ParagraphAlignmentSchema, narrowEnum } from "./parserEnums.js";
 import { RUN_PROPERTY_OWNERS, parseRunProperties } from "./runParser.js";
 import { findChild, getAttribute, parseBooleanElement, parseNumericAttribute, parseOnOffAttribute, parseXmlDocument } from "./xmlParser.js";
+import { isNumberingLevel } from "@stll/docx-core/model";
 //#region src/docx/numberingParser.ts
+/**
+* Numbering/List Parser for DOCX
+*
+* Parses numbering.xml to extract:
+* - Abstract numbering definitions (templates with levels)
+* - Numbering instances (concrete references with optional overrides)
+*
+* OOXML Structure:
+* - w:abstractNum - Template definitions with 9 levels (0-8)
+* - w:num - Instances that reference abstractNum and can override levels
+* - w:lvl - Level definition with start, format, text pattern, etc.
+*/
 /** `w:abstractNum` reads its id; everything else it carries is the remainder's. */
 const MODELLED_ABSTRACT_NUM_ATTRIBUTES = /* @__PURE__ */ new Set(["abstractNumId"]);
 /** `w:lvl` reads the level it defines and Word's two template markers. */
@@ -117,7 +130,6 @@ function parseAbstractNumbering(element) {
 	return abstractNum;
 }
 /** ECMA-376 §17.9.1: a `w:abstractNum` defines levels 0 through 8 and no more. */
-const MAX_LIST_LEVEL = 8;
 const MULTI_LEVEL_TYPES = [
 	"hybridMultilevel",
 	"multilevel",
@@ -392,7 +404,7 @@ function createNumberingMap(definitions) {
 	return {
 		definitions,
 		getLevel(numId, ilvl) {
-			if (ilvl < 0 || ilvl > MAX_LIST_LEVEL) return null;
+			if (!isNumberingLevel(ilvl)) return null;
 			const num = numMap.get(numId);
 			if (!num) return null;
 			if (num.levelOverrides) {
@@ -524,7 +536,9 @@ function getBulletCharacter(level) {
 function isBulletLevel(level) {
 	return level.numFmt === "bullet" || level.numFmt === "none";
 }
+/** Whether a numbering definition uses a bullet marker rather than a numeric or hidden one. */
+const numberingLevelUsesBulletMarker = (level) => level?.numFmt === "bullet";
 /** Whether a numbering level reserves horizontal space for a visible marker. */
 const numberingLevelHasMarkerSlot = (level) => level.numFmt !== "none";
 //#endregion
-export { computeListRendering, counterFormatOf, createNumberingMap, customNumberFormatPadWidth, formatOoxmlCounter as formatNumber, getBulletCharacter, getCachedNumberingMap, isBulletLevel, markerAlignmentForLevel, markerFormattingFromLevel, numberingLevelHasMarkerSlot, padDecimal, parseNumbering, renderListMarker };
+export { computeListRendering, counterFormatOf, createNumberingMap, customNumberFormatPadWidth, formatOoxmlCounter as formatNumber, getBulletCharacter, getCachedNumberingMap, isBulletLevel, markerAlignmentForLevel, markerFormattingFromLevel, numberingLevelHasMarkerSlot, numberingLevelUsesBulletMarker, padDecimal, parseNumbering, renderListMarker };

package/dist/docx/packageParts.d.ts

@@ -1,5 +1,7 @@
 import JSZip from "jszip";
 //#region src/docx/packageParts.d.ts
+/** Resolve an internal URI reference once, before comparing it with ZIP entries. */
+declare const resolvePackageRelationshipTarget: (target: string, relsPath: string) => string | undefined;
 /**
  * Whether an entry path would escape the package when a host writes the archive
  * to disk: absolute, backslash-separated, or climbing out through `..`. This is
@@ -76,4 +78,4 @@ type PackageIntegrityReport = {
  */
 declare const checkPackageIntegrity: (zip: JSZip) => Promise<PackageIntegrityReport>;
 //#endregion
-export { PackageIntegrityReport, PackageReferenceRepair, checkPackageIntegrity, isUnsafePackagePath, reconcilePackageReferences, removeUnsafeEntries };
+export { PackageIntegrityReport, PackageReferenceRepair, checkPackageIntegrity, isUnsafePackagePath, reconcilePackageReferences, removeUnsafeEntries, resolvePackageRelationshipTarget };

package/dist/docx/packageParts.js

@@ -1,5 +1,31 @@
-import { resolveRelativePath } from "./relsParser.js";
+import { Result } from "better-result";
+import { writeZipPart } from "@stll/docx-core/zip";
 //#region src/docx/packageParts.ts
+/**
+* What a repack carries out of the source package, and the guarantee that
+* nothing in the output points at a part the output does not hold.
+*
+* A `.docx` is an OPC package: `[Content_Types].xml` declares every part in it
+* and the `.rels` parts wire them together. Folio models a handful of those
+* parts (the document body, headers, footers, notes, numbering, styles,
+* comments) and rewrites them on save. Everything else — embedded objects,
+* media, custom XML, fonts, charts, macro projects, parts folio has never heard
+* of — belongs to the author, and a repack hands it back byte for byte.
+*
+* Folio never executes document content and never removes it. The document is
+* the user's; the only thing a save refuses is a package path that would let
+* the archive escape its own directory when a host writes it out, which is a
+* property of the package structure rather than of the content. A macro-enabled
+* document therefore stays macro-enabled, main-part content type included:
+* silently handing back a `.docm` stripped of its project is data loss dressed
+* up as safety.
+*
+* A refused path loses its relationships and its content-type override with it.
+* That is the invariant this module exists for: Word and LibreOffice both
+* refuse a package whose `document.xml.rels` points at nothing, so a repack
+* that removes a part without removing its references produces a file that
+* does not open.
+*/
 const CONTENT_TYPE_ELEMENT = /<(?:Default|Override)\b[^>]*?\/?>/giu;
 const PART_NAME_ATTRIBUTE = /\bPartName\s*=\s*(?<quote>["'])(?<value>[^"']*)\k<quote>/u;
 const RELATIONSHIP_ELEMENT = /<Relationship\b[^>]*?(?:\/>|>\s*<\/Relationship>)/giu;
@@ -24,12 +50,38 @@ const owningPartPath = (relsPath) => {
 };
 /** Normalize a package path for comparison: lower case, no leading slash. */
 const normalizePartPath = (path) => (path.startsWith("/") ? path.slice(1) : path).toLowerCase();
-/**
-* Decode `%XX` escapes in a part name. `decodeURI` throws on a malformed
-* escape, and a malformed target is exactly the input this has to survive, so
-* the replacement is done by hand.
-*/
-const decodePartName = (name) => name.replaceAll(/%[0-9A-Fa-f]{2}/gu, (escape) => String.fromCharCode(Number.parseInt(escape.slice(1), 16)));
+/** Decode URI escapes as UTF-8; malformed references cannot name a part. */
+const decodePartName = (name) => {
+	const decoded = Result.try({
+		try: () => decodeURIComponent(name),
+		catch: () => void 0
+	});
+	return decoded.isOk() ? decoded.value : void 0;
+};
+/** Resolve an internal URI reference once, before comparing it with ZIP entries. */
+const resolvePackageRelationshipTarget = (target, relsPath) => {
+	const fragment = target.indexOf("#");
+	if (fragment === 0) {
+		const owner = owningPartPath(relsPath);
+		return owner === void 0 ? void 0 : normalizePartPath(owner);
+	}
+	const reference = fragment > 0 ? target.slice(0, fragment) : target;
+	if (!reference || reference.startsWith("//") || /[\\?:]/u.test(reference)) return void 0;
+	const owner = owningPartPath(relsPath);
+	const segments = reference.startsWith("/") || owner === void 0 ? [] : owner.split("/").slice(0, -1);
+	for (const encoded of reference.split("/")) {
+		const segment = decodePartName(encoded);
+		if (segment === void 0 || /[/\\]/u.test(segment) || segment.includes(String.fromCharCode(0))) return void 0;
+		if (segment === "" || segment === ".") continue;
+		if (segment === "..") {
+			if (segments.pop() === void 0) return void 0;
+			continue;
+		}
+		segments.push(segment);
+	}
+	const path = normalizePartPath(segments.join("/"));
+	return isUnsafePackagePath(path) ? void 0 : path;
+};
 /**
 * Whether an entry path would escape the package when a host writes the archive
 * to disk: absolute, backslash-separated, or climbing out through `..`. This is
@@ -53,15 +105,13 @@ const presentPartPaths = (zip) => {
 	for (const [path, file] of Object.entries(zip.files)) if (!file.dir) present.add(normalizePartPath(path));
 	return present;
 };
-const isPresent = (present, partPath) => present.has(normalizePartPath(partPath)) || present.has(normalizePartPath(decodePartName(partPath)));
+const isPresent = (present, partPath) => present.has(normalizePartPath(partPath)) || present.has(normalizePartPath(decodePartName(partPath) ?? ""));
 /** The part a relationship names, or null when it points outside the package. */
 const relationshipTarget = (element, relsPath) => {
 	if (TARGET_MODE_ATTRIBUTE.exec(element)?.groups?.["value"] === "External") return null;
 	const target = TARGET_ATTRIBUTE.exec(element)?.groups?.["value"];
 	if (target === void 0) return null;
-	const fragment = target.indexOf("#");
-	if (fragment === 0) return owningPartPath(relsPath) ?? "";
-	return resolveRelativePath(relsPath, fragment > 0 ? target.slice(0, fragment) : target);
+	return resolvePackageRelationshipTarget(target, relsPath) ?? "";
 };
 /**
 * Every `.rels` part with its text. Read one at a time rather than through
@@ -80,9 +130,14 @@ const readRelationshipParts = async (zip) => {
 	return parts;
 };
 const writePart = ({ zip, path, xml, compressionLevel }) => {
-	zip.file(path, xml, {
-		compression: "DEFLATE",
-		compressionOptions: { level: compressionLevel }
+	writeZipPart({
+		zip,
+		path,
+		data: xml,
+		options: {
+			compression: "DEFLATE",
+			compressionOptions: { level: compressionLevel }
+		}
 	});
 };
 /**
@@ -107,8 +162,8 @@ const reconcilePackageReferences = async (zip, compressionLevel) => {
 		let dropped = false;
 		const pruned = xml.replaceAll(RELATIONSHIP_ELEMENT, (element) => {
 			const target = relationshipTarget(element, path);
-			if (target === null || isPresent(present, target)) return element;
-			repair.danglingRelationships.push(target);
+			if (target === null || present.has(target)) return element;
+			repair.danglingRelationships.push(target || TARGET_ATTRIBUTE.exec(element)?.groups?.["value"] || "");
 			dropped = true;
 			const id = RELATIONSHIP_ID_ATTRIBUTE.exec(element)?.groups?.["value"];
 			const owningPart = owningPartPath(path);
@@ -154,7 +209,7 @@ const reconcilePackageReferences = async (zip, compressionLevel) => {
 	const prunedContentTypes = (await contentTypes.async("text")).replaceAll(CONTENT_TYPE_ELEMENT, (element) => {
 		const partName = PART_NAME_ATTRIBUTE.exec(element)?.groups?.["value"];
 		if (partName === void 0 || isPresent(present, partName)) return element;
-		repair.danglingOverrides.push(normalizePartPath(decodePartName(partName)));
+		repair.danglingOverrides.push(normalizePartPath(decodePartName(partName) ?? partName));
 		droppedOverride = true;
 		return "";
 	});
@@ -195,7 +250,7 @@ const checkPackageIntegrity = async (zip) => {
 			const id = RELATIONSHIP_ID_ATTRIBUTE.exec(element)?.groups?.["value"];
 			if (id !== void 0) ids.add(id);
 			const target = relationshipTarget(element, path);
-			if (target !== null && !isPresent(present, target)) report.danglingRelationshipTargets.push(`${path}|${target}`);
+			if (target !== null && !present.has(target)) report.danglingRelationshipTargets.push(`${path}|${target || TARGET_ATTRIBUTE.exec(element)?.groups?.["value"] || ""}`);
 		}
 		if (owningPart !== void 0) idsByOwningPart.set(owningPart, ids);
 	}
@@ -211,4 +266,4 @@ const checkPackageIntegrity = async (zip) => {
 	return report;
 };
 //#endregion
-export { checkPackageIntegrity, isUnsafePackagePath, reconcilePackageReferences, removeUnsafeEntries };
+export { checkPackageIntegrity, isUnsafePackagePath, reconcilePackageReferences, removeUnsafeEntries, resolvePackageRelationshipTarget };

package/dist/docx/paragraphMarkPropertyPresence.d.ts

@@ -0,0 +1,9 @@
+import { document_d_exports } from "../types/document.js";
+import { XmlElement } from "./xmlParser.js";
+//#region src/docx/paragraphMarkPropertyPresence.d.ts
+declare const EMPTY_MARK_PROPERTIES_ATTRIBUTE = "emptyMarkProperties";
+/** A paragraph-owned rPr carrier cannot distinguish absent and explicitly empty formatting. */
+declare const serializeEmptyMarkProperties: (paragraph: document_d_exports.Paragraph) => string | undefined;
+declare const restoreEmptyMarkProperties: (paragraph: document_d_exports.Paragraph, element: XmlElement) => void;
+//#endregion
+export { EMPTY_MARK_PROPERTIES_ATTRIBUTE, restoreEmptyMarkProperties, serializeEmptyMarkProperties };

package/dist/docx/paragraphMarkPropertyPresence.js

@@ -0,0 +1,21 @@
+import { OOXML_NAMESPACES } from "./serializer/partNamespaces.js";
+import { getAttributeByNamespaceUri } from "./xmlParser.js";
+import { EMPTY_PROPERTY_SETS } from "@stll/docx-core/ops";
+//#region src/docx/paragraphMarkPropertyPresence.ts
+const EMPTY_MARK_PROPERTIES_ATTRIBUTE = "emptyMarkProperties";
+const REVIEW_HISTORY_NAMESPACES = /* @__PURE__ */ new Set([OOXML_NAMESPACES.folio.uri]);
+/** A paragraph-owned rPr carrier cannot distinguish absent and explicitly empty formatting. */
+const serializeEmptyMarkProperties = (paragraph) => {
+	const properties = paragraph.formatting?.runProperties;
+	if (!(paragraph.pPrMark !== void 0 || paragraph.formatting?.runInWithNext !== void 0) || properties === void 0 || Object.keys(properties).length !== 0) return void 0;
+	return `folio:${EMPTY_MARK_PROPERTIES_ATTRIBUTE}="${EMPTY_PROPERTY_SETS.KEEP}"`;
+};
+const restoreEmptyMarkProperties = (paragraph, element) => {
+	if (paragraph.pPrMark === void 0 && paragraph.formatting?.runInWithNext === void 0 || getAttributeByNamespaceUri(element, REVIEW_HISTORY_NAMESPACES, "emptyMarkProperties") !== EMPTY_PROPERTY_SETS.KEEP || paragraph.formatting?.runProperties !== void 0) return;
+	paragraph.formatting = {
+		...paragraph.formatting,
+		runProperties: {}
+	};
+};
+//#endregion
+export { EMPTY_MARK_PROPERTIES_ATTRIBUTE, restoreEmptyMarkProperties, serializeEmptyMarkProperties };