@beyondwork/docx-react-component 1.0.72 → 1.0.74

package/src/api/v3/ui/scope.ts

@@ -33,9 +33,92 @@
  */
 
 import type { ApiV3FnMetadata } from "../_layer-metadata.ts";
-import type { ScopeBundle } from "../../public-types.ts";
+import type {
+  ScopeBundle,
+  ScopeCardModel,
+  ScopeRailSnapshot,
+  SemanticScope,
+  SemanticScopeKind,
+} from "../../public-types.ts";
 import type { UiApiContext } from "./_context.ts";
 
+export interface UiScopeListFilter {
+  readonly kind?: SemanticScopeKind;
+  readonly limit?: number;
+}
+
+export interface UiScopeRailOptions {
+  /**
+   * Narrow the snapshot to a single page index. Omit (default) to
+   * span every page the runtime's page graph knows about.
+   */
+  readonly pageIndex?: number;
+}
+
+export const cardMetadata: ApiV3FnMetadata = {
+  name: "ui.scope.card",
+  status: "live-with-adapter",
+  sourceLayer: "presentation",
+  liveEvidence: {
+    runnerTest: "test/api/v3/ui/scope-card-rail.test.ts",
+    commit: "refactor-10-ki-008-ui-scope-card-rail",
+  },
+  uxIntent: { uiVisible: false },
+  agentMetadata: {
+    readOrMutate: "read",
+    boundedScope: "scope",
+    auditCategory: "ui-scope-card-read",
+  },
+  // Same classification as other `ui.scope.*` reads — canonical scope
+  // projection (not class-C local preference). KI-008 close.
+  stateClass: "A-canonical",
+  persistsTo: "canonical",
+  rwdReference:
+    "§UI API § ui.scope.card. Wraps handle.compileScopeCardById(scopeId) which projects runtime.workflow.getAllScopeCardModels() filtered to the matching scope. Returns plain-value ScopeCardModel — no React handlers, no DOM refs. Hosts wire event handlers in their own chrome tree.",
+};
+
+export const railMetadata: ApiV3FnMetadata = {
+  name: "ui.scope.rail",
+  status: "live-with-adapter",
+  sourceLayer: "presentation",
+  liveEvidence: {
+    runnerTest: "test/api/v3/ui/scope-card-rail.test.ts",
+    commit: "refactor-10-ki-008-ui-scope-card-rail",
+  },
+  uxIntent: { uiVisible: false },
+  agentMetadata: {
+    readOrMutate: "read",
+    boundedScope: "document",
+    auditCategory: "ui-scope-rail-read",
+  },
+  stateClass: "A-canonical",
+  persistsTo: "canonical",
+  rwdReference:
+    "§UI API § ui.scope.rail. Wraps handle.compileScopeRailSnapshot({pageIndex?}) which projects runtime.workflow.getRailSegments(pageIndex) / getAllRailSegments() into a ScopeRailSnapshot envelope. Plain-value data; no React handlers.",
+};
+
+export const listMetadata: ApiV3FnMetadata = {
+  name: "ui.scope.list",
+  status: "live-with-adapter",
+  sourceLayer: "presentation",
+  liveEvidence: {
+    runnerTest: "test/api/v3/ui/scope-list.test.ts",
+    commit: "refactor-10-ki-008-ui-scope-list",
+  },
+  uxIntent: { uiVisible: false },
+  agentMetadata: {
+    readOrMutate: "read",
+    boundedScope: "document",
+    auditCategory: "ui-scope-list",
+  },
+  // Same classification as `ui.scope.getBundle` — reads canonical
+  // scope state (not class-C local preference). KI-008 closure.
+  stateClass: "A-canonical",
+  persistsTo: "canonical",
+  rwdReference:
+    "§UI API § ui.scope.list. Wraps handle.compileScopeList(filter?) to enumerate compiled SemanticScope records through the L10 seam without importing src/runtime/scopes/**. Filter matches ai.listScopes({kind?, limit?}) for identity-set symmetry.",
+};
+
 export const getBundleMetadata: ApiV3FnMetadata = {
   name: "ui.scope.getBundle",
   status: "live-with-adapter",
@@ -67,5 +150,34 @@ export function createScopeFamily(ctx: UiApiContext) {
       const nowUtc = new Date().toISOString();
       return compile.call(ctx.handle, scopeId, nowUtc);
     },
+    list(filter?: UiScopeListFilter): readonly SemanticScope[] {
+      // KI-008 — enumeration through the handle's batch primitive
+      // (L10 purity blocks importing src/runtime/scopes/**). Returns
+      // a fresh array so callers that sort / splice don't leak into
+      // internal compiler state.
+      const enumerate = ctx.handle.compileScopeList;
+      if (typeof enumerate !== "function") return Object.freeze([]);
+      return enumerate.call(ctx.handle, filter);
+    },
+    card(scopeId: string): ScopeCardModel | null {
+      // KI-008 close — single-scope presentation projection. Returns
+      // a plain-value ScopeCardModel (posture, label, issue, suggestion
+      // groups, review actions, agent-pending flag). Host chrome wires
+      // event handlers around it.
+      const compile = ctx.handle.compileScopeCardById;
+      if (typeof compile !== "function") return null;
+      return compile.call(ctx.handle, scopeId);
+    },
+    rail(options?: UiScopeRailOptions): ScopeRailSnapshot {
+      // KI-008 close — rail snapshot. When `options.pageIndex` is
+      // supplied, narrows to that page; otherwise spans every page
+      // in the runtime's page graph. Envelope shape is stable either
+      // way; callers check `snapshot.pageIndex` to detect narrowing.
+      const compile = ctx.handle.compileScopeRailSnapshot;
+      if (typeof compile !== "function") {
+        return { segments: Object.freeze([]) };
+      }
+      return compile.call(ctx.handle, options);
+    },
   };
 }

package/src/api/v3/ui/viewport.ts

@@ -112,7 +112,7 @@ export const scrollToPageMetadata: ApiV3FnMetadata = {
   stateClass: "C-local",
   persistsTo: "none",
   rwdReference:
-    "§UI API § ui.viewport.scrollToPage. Resolves pageNumber → scrollY via handle.geometry.getPage(pageIndex); dispatches through controller.dispatchScroll({ kind:'page', value, behavior }); returns the settled {actualPage, scrollY}. 1-based page numbers; clamps to [1, pageCount]. First-class API for visual-fidelity harness + 'Go to page N' UX — replaces DOM-scrape fallback (coord-10 §γ). When L07 ships runtime.viewport.getPageAnchor / getPageGeometry (coord-07 §2.9), this wrapper may be simplified to delegate to those primitives; the public shape stays stable.",
+    "§UI API § ui.viewport.scrollToPage. Resolves pageNumber → scrollY via handle.geometry.getPage(pageIndex); dispatches through controller.dispatchScroll({ kind:'page', value, behavior }); returns the settled {actualPage, scrollY}. 1-based page numbers; clamps to [1, pageCount]. First-class API for visual-fidelity harness + 'Go to page N' UX — replaces DOM-scrape fallback (coord-10 §γ). Parity note: reads the same `handle.geometry.getPage(i).frame.topPx` source as `runtime.viewport.getPageAnchor` (L07 coord-07 §2.9, shipped 2026-04-24 in `src/api/v3/runtime/viewport.ts`), so `actualPage + scrollY` here and `{scrollY, pageRect}` on the runtime side stay consistent by construction. No direct delegation today because `scripts/ci-check-ui-api-layer-purity.mjs` restricts `src/api/v3/ui/**` from importing `src/api/v3/runtime/**`; both surfaces are thin wrappers over the shared geometry facet.",
 };
 
 // ----- X5 markup-mode metadata (state-classes cross-cutting Slice X5) -----

package/src/compare/diff-engine.ts

@@ -410,8 +410,7 @@ function createParagraphRevisionRecords(
       changeId: revision.changeId,
       kind: revision.kind,
       anchor: {
-        kind: "range",
-        range: { from: position, to: position },
+        kind: "range", from: position, to: position,
         assoc: { start: -1, end: 1 },
       },
       authorId,

package/src/core/commands/index.ts

@@ -1525,9 +1525,9 @@ export function remapSelection(
 
   if (activeRange.kind === "range") {
     return {
-      anchor: activeRange.range.from,
-      head: activeRange.range.to,
-      isCollapsed: activeRange.range.from === activeRange.range.to,
+      anchor: activeRange.from,
+      head: activeRange.to,
+      isCollapsed: activeRange.from === activeRange.to,
       activeRange,
     };
   }
@@ -1664,8 +1664,8 @@ function normalizeSelection(selection: SelectionSnapshot): SelectionSnapshot {
   );
 
   if (activeRange.kind === "range") {
-    const rangeFrom = activeRange.range.from;
-    const rangeTo = activeRange.range.to;
+    const rangeFrom = activeRange.from;
+    const rangeTo = activeRange.to;
     const collapsed = rangeFrom === rangeTo;
     const anchorWithinRange = selection.anchor >= rangeFrom && selection.anchor <= rangeTo;
     const headWithinRange = selection.head >= rangeFrom && selection.head <= rangeTo;
@@ -2256,7 +2256,8 @@ function createAuthoredRevision(
     kind,
     anchor: {
       kind: "range",
-      range: { from, to },
+      from,
+      to,
       assoc: { start: 1, end: -1 },
     },
     authorId,
@@ -2396,8 +2397,7 @@ function applySuggestingInsert(
       const updatedRevision: CanonicalRevisionRecord = {
         ...existingInsertion,
         anchor: {
-          kind: "range",
-          range: { from: existingInsertion.anchor.range.from, to: insertedTo },
+          kind: "range", from: existingInsertion.anchor.from, to: insertedTo,
           assoc: { start: 1, end: -1 },
         },
       };
@@ -2635,19 +2635,18 @@ function applySuggestingDelete(
   if (existingDeletion) {
     // Extend the existing deletion revision to cover the new range
     const extendedFrom = Math.min(
-      existingDeletion.anchor.kind === "range" ? existingDeletion.anchor.range.from : deleteFrom,
+      existingDeletion.anchor.kind === "range" ? existingDeletion.anchor.from : deleteFrom,
       deleteFrom,
     );
     const extendedTo = Math.max(
-      existingDeletion.anchor.kind === "range" ? existingDeletion.anchor.range.to : deleteTo,
+      existingDeletion.anchor.kind === "range" ? existingDeletion.anchor.to : deleteTo,
       deleteTo,
     );
 
     const updatedRevision: CanonicalRevisionRecord = {
       ...existingDeletion,
       anchor: {
-        kind: "range",
-        range: { from: extendedFrom, to: extendedTo },
+        kind: "range", from: extendedFrom, to: extendedTo,
         assoc: { start: 1, end: -1 },
       },
     };
@@ -2933,7 +2932,7 @@ function findAdjacentInsertionRevision(
       revision.status === "open" &&
       revision.metadata?.source === "runtime" &&
       revision.anchor.kind === "range" &&
-      revision.anchor.range.to === cursorPos &&
+      revision.anchor.to === cursorPos &&
       revision.authorId === authorId
     ) {
       return revision;
@@ -2954,8 +2953,8 @@ function findOverlappingAuthoredDeletion(
       revision.metadata?.source === "runtime" &&
       revision.anchor.kind === "range"
     ) {
-      const revFrom = revision.anchor.range.from;
-      const revTo = revision.anchor.range.to;
+      const revFrom = revision.anchor.from;
+      const revTo = revision.anchor.to;
       // Adjacent or overlapping: the deletion is directly next to or overlapping the range
       if (
         (from >= revFrom && from <= revTo) ||

package/src/core/selection/anchor-conversion.ts

@@ -79,8 +79,8 @@ export function toPublicAnchorProjection(
     case "range":
       return {
         kind: "range",
-        from: anchor.range.from,
-        to: anchor.range.to,
+        from: anchor.from,
+        to: anchor.to,
         assoc: anchor.assoc,
       };
     case "node":

package/src/core/selection/mapping.ts

@@ -66,9 +66,11 @@ export function createRangeAnchor(
   to = from,
   assoc: BoundaryAssoc = DEFAULT_BOUNDARY_ASSOC,
 ): RangeAnchor {
+  const normalized = normalizeRange({ from, to });
   return {
     kind: "range",
-    range: normalizeRange({ from, to }),
+    from: normalized.from,
+    to: normalized.to,
     assoc,
   };
 }
@@ -89,7 +91,7 @@ export function createEmptyMapping(): TransactionMapping {
 
 export function getEffectiveRange(anchor: EditorAnchorProjection): DocRange {
   if (anchor.kind === "range") {
-    return anchor.range;
+    return { from: anchor.from, to: anchor.to };
   }
   if (anchor.kind === "node") {
     return { from: anchor.at, to: anchor.at };
@@ -140,13 +142,13 @@ export function mapAnchor(
     return createNodeAnchor(mapped.position, anchor.assoc);
   }
 
-  const from = mapPosition(anchor.range.from, anchor.assoc.start, mapping);
-  const to = mapPosition(anchor.range.to, anchor.assoc.end, mapping);
+  const from = mapPosition(anchor.from, anchor.assoc.start, mapping);
+  const to = mapPosition(anchor.to, anchor.assoc.end, mapping);
   const mappedRange = normalizeRange({ from: from.position, to: to.position });
 
   if (from.deleted && to.deleted && mappedRange.from === mappedRange.to) {
     return createDetachedAnchor(
-      anchor.range,
+      { from: anchor.from, to: anchor.to },
       mapping.metadata?.invalidatesStructures
         ? "invalidatedByStructureChange"
         : "deleted",
@@ -187,8 +189,8 @@ export function areAnchorsEqual(
 
   if (left.kind === "range" && right.kind === "range") {
     return (
-      left.range.from === right.range.from &&
-      left.range.to === right.range.to &&
+      left.from === right.from &&
+      left.to === right.to &&
       left.assoc.start === right.assoc.start &&
       left.assoc.end === right.assoc.end
     );
@@ -213,7 +215,7 @@ export function anchorUnaffectedByMapping(
 ): boolean {
   if (anchor.kind === "detached") return true;
   if (mapping.steps.length === 0) return true;
-  const end = anchor.kind === "range" ? anchor.range.to : anchor.at;
+  const end = anchor.kind === "range" ? anchor.to : anchor.at;
   return mapping.steps.every((s) => s.from > end);
 }
 

package/src/core/selection/review-anchors.ts

@@ -40,7 +40,7 @@ export function mapReviewAnchor(
 
 export function getAnchorRange(anchor: ReviewAnchor): DocRange {
   if (anchor.kind === "range") {
-    return normalizeRange(anchor.range);
+    return normalizeRange({ from: anchor.from, to: anchor.to });
   }
 
   if (anchor.kind === "node") {
@@ -136,7 +136,7 @@ export function commentAnchorRejectionReason(
     return "invalid_comment_anchor";
   }
 
-  const normalized = normalizeRange(anchor.range);
+  const normalized = normalizeRange({ from: anchor.from, to: anchor.to });
   if (normalized.from === normalized.to) {
     return "invalid_comment_anchor";
   }
@@ -164,7 +164,7 @@ export function snapCommentAnchorAwayFromTable(
 ): ReviewAnchor | null {
   if (anchor.kind !== "range") return null;
 
-  const normalized = normalizeRange(anchor.range);
+  const normalized = normalizeRange({ from: anchor.from, to: anchor.to });
   if (normalized.from === normalized.to) return null;
 
   // If the anchor is already flagged invalid for other reasons (crosses an

package/src/core/state/editor-state.ts

@@ -582,14 +582,57 @@ export function createPersistedEditorSnapshot(
 }
 
 function estimateParagraphCount(content: unknown): number {
-  if (Array.isArray(content)) {
-    return content.length;
-  }
+  // Canonical shape: `{type:"doc", children: BlockNode[]}`. Older
+  // shapes (array / `.blocks`) handled for persistence-snapshot
+  // fallback. KI-P4 (2026-04-23): pre-fix the array + .blocks
+  // branches never matched the current envelope, so the fallback
+  // returned 1 on any non-empty document regardless of paragraph
+  // count. Fix counts ParagraphNode entries recursively, descending
+  // into table cells + SDT / customXml blocks so nested paragraphs
+  // contribute to the total.
+  let count = 0;
+  const walk = (node: unknown): void => {
+    if (!node || typeof node !== "object") return;
+    const typed = node as { type?: unknown };
+    if (typed.type === "paragraph") {
+      count += 1;
+      return;
+    }
+    if (typed.type === "table") {
+      const rows = (node as { rows?: unknown[] }).rows;
+      if (Array.isArray(rows)) {
+        for (const row of rows) {
+          const cells = (row as { cells?: unknown[] }).cells;
+          if (Array.isArray(cells)) {
+            for (const cell of cells) {
+              const children = (cell as { children?: unknown[] }).children;
+              if (Array.isArray(children)) children.forEach(walk);
+            }
+          }
+        }
+      }
+      return;
+    }
+    const children = (node as { children?: unknown[] }).children;
+    if (Array.isArray(children)) children.forEach(walk);
+  };
 
-  if (content && typeof content === "object" && Array.isArray((content as { blocks?: unknown[] }).blocks)) {
-    return ((content as { blocks: unknown[] }).blocks).length;
+  if (content && typeof content === "object") {
+    const children = (content as { children?: unknown[] }).children;
+    if (Array.isArray(children)) {
+      children.forEach(walk);
+      return count;
+    }
+    const blocks = (content as { blocks?: unknown[] }).blocks;
+    if (Array.isArray(blocks)) {
+      blocks.forEach(walk);
+      return count;
+    }
+  }
+  if (Array.isArray(content)) {
+    content.forEach(walk);
+    return count;
   }
-
   return extractText(content).length > 0 ? 1 : 0;
 }
 

package/src/io/export/export-session.ts

@@ -119,6 +119,7 @@ export class ExportSession {
       this.packageRelationships,
       this.ownedPaths,
     );
+    ensurePicNamespaceOnStoryParts(this.workingParts);
     const parts = pruneReservedManifestParts(this.workingParts);
     const manifest = buildManifest(parts, this.packageRelationships, this.sourcePackage.manifest.contentTypes);
     return {
@@ -140,6 +141,58 @@ export function createExportSession(
   return new ExportSession(sourcePackage, ownedOutputPaths);
 }
 
+/**
+ * LibreOffice strict-parser safety net. Word tolerates `pic:` prefix
+ * usage (from `<pic:pic>` / `<pic:spPr>` inside drawings) without
+ * declaring `xmlns:pic` at the part root, because it inherits scope
+ * from `<wp:inline xmlns:pic=…>` wrappers or from `mc:AlternateContent`
+ * fallbacks. LibreOffice refuses the file ("Namespace prefix pic on
+ * spPr is not defined"). Our own serializers already inject
+ * `xmlns:pic` on freshly serialized parts — but parts that pass
+ * through unmodified from the source package keep the original
+ * declaration (which may be missing it). This sweep runs at toPackage
+ * time over the main story + every header / footer part, and injects
+ * `xmlns:pic` on the root element when the part uses the `pic:` prefix
+ * but doesn't declare it. Byte-stable for parts already correct.
+ *
+ * See commit a3e05a8e (serializer-side fix) + this (pass-through fix).
+ */
+const PIC_NAMESPACE_URI = "http://schemas.openxmlformats.org/drawingml/2006/picture";
+const PIC_NAMESPACE_DECL = ` xmlns:pic="${PIC_NAMESPACE_URI}"`;
+
+function ensurePicNamespaceOnStoryParts(
+  parts: Map<string, OpcPackagePart>,
+): void {
+  for (const [path, part] of parts.entries()) {
+    if (!isPicSensitiveStoryPath(path)) continue;
+    const patched = ensurePicNamespaceOnRoot(part.bytes);
+    if (patched !== part.bytes) {
+      parts.set(path, { ...part, bytes: patched, crc32: 0 });
+    }
+  }
+}
+
+function isPicSensitiveStoryPath(path: string): boolean {
+  // Normalize for leading slash variation.
+  const normalized = path.startsWith("/") ? path.slice(1) : path;
+  if (normalized === "word/document.xml") return true;
+  if (/^word\/(?:header|footer|footnotes|endnotes)[^/]*\.xml$/.test(normalized)) return true;
+  return false;
+}
+
+function ensurePicNamespaceOnRoot(bytes: Uint8Array): Uint8Array {
+  const xml = new TextDecoder().decode(bytes);
+  // Match the first element (skipping the XML prolog + whitespace).
+  const rootMatch = xml.match(/^([\s\S]*?)<([A-Za-z][\w.:-]*)([^>]*)>/);
+  if (!rootMatch) return bytes;
+  const [fullMatch, prefix, tagName, rootAttrs] = rootMatch;
+  if (rootAttrs.includes("xmlns:pic=")) return bytes; // already declared
+  if (!/\bpic:[A-Za-z]/.test(xml)) return bytes; // doesn't use pic: prefix
+  const patchedRoot = `<${tagName}${rootAttrs}${PIC_NAMESPACE_DECL}>`;
+  const patched = `${prefix}${patchedRoot}${xml.slice(fullMatch.length)}`;
+  return new TextEncoder().encode(patched);
+}
+
 function clonePartsForExport(
   parts: Map<string, OpcPackagePart>,
   ownedPaths: ReadonlySet<string>,

package/src/io/export/serialize-comments.ts

@@ -269,16 +269,16 @@ export function serializeCommentAnchorsIntoDocumentXml(
       continue;
     }
 
-    const startParagraph = findParagraphForEndpoint(paragraphs, anchor.range.from, "start");
-    const endParagraph = findParagraphForEndpoint(paragraphs, anchor.range.to, "end");
+    const startParagraph = findParagraphForEndpoint(paragraphs, anchor.from, "start");
+    const endParagraph = findParagraphForEndpoint(paragraphs, anchor.to, "end");
 
     if (!startParagraph || !endParagraph) {
       skippedCommentIds.push(thread.commentId);
       continue;
     }
 
-    const startIndex = startParagraph.boundaries.get(anchor.range.from);
-    const endIndex = endParagraph.boundaries.get(anchor.range.to);
+    const startIndex = startParagraph.boundaries.get(anchor.from);
+    const endIndex = endParagraph.boundaries.get(anchor.to);
 
     if (startIndex === undefined || endIndex === undefined) {
       skippedCommentIds.push(thread.commentId);

package/src/io/export/serialize-footnotes.ts

@@ -18,6 +18,7 @@ import {
 } from "./table-properties-xml.ts";
 import { twip } from "./twip.ts";
 import { escapeXmlAttribute } from "./escape-xml-attribute.ts";
+import { buildFrameXml } from "./serialize-paragraph-formatting.ts";
 
 export const WORD_FOOTNOTES_CONTENT_TYPE =
   "application/vnd.openxmlformats-officedocument.wordprocessingml.footnotes+xml";
@@ -222,6 +223,11 @@ function buildParagraphPropertiesXml(paragraph: ParagraphNode): string {
   if (paragraph.styleId) {
     parts.push(`<w:pStyle w:val="${escapeXmlAttribute(paragraph.styleId)}"/>`);
   }
+  // Coord-04 §1.19.d — direct-paragraph framePr (footnotes path).
+  {
+    const frameXml = buildFrameXml(paragraph.frameProperties);
+    if (frameXml) parts.push(frameXml);
+  }
   if (paragraph.alignment) {
     parts.push(`<w:jc w:val="${escapeXmlAttribute(paragraph.alignment)}"/>`);
   }

package/src/io/export/serialize-headers-footers.ts

@@ -18,6 +18,7 @@ import {
 } from "./table-properties-xml.ts";
 import { twip } from "./twip.ts";
 import { escapeXmlAttribute } from "./escape-xml-attribute.ts";
+import { buildFrameXml } from "./serialize-paragraph-formatting.ts";
 
 export const WORD_HEADER_CONTENT_TYPE =
   "application/vnd.openxmlformats-officedocument.wordprocessingml.header+xml";
@@ -186,6 +187,11 @@ function buildParagraphPropertiesXml(paragraph: ParagraphNode): string {
   if (paragraph.styleId) {
     parts.push(`<w:pStyle w:val="${escapeXmlAttribute(paragraph.styleId)}"/>`);
   }
+  // Coord-04 §1.19.d — direct-paragraph framePr (headers/footers path).
+  {
+    const frameXml = buildFrameXml(paragraph.frameProperties);
+    if (frameXml) parts.push(frameXml);
+  }
   if (paragraph.spacing) {
     const s = paragraph.spacing;
     const attrs: string[] = [];

package/src/io/export/serialize-main-document.ts

@@ -22,6 +22,7 @@ import { SCOPE_MARKER_BOOKMARK_PREFIX } from "../ooxml/parse-scope-markers.ts";
 import { getOpaqueFragment } from "../../preservation/store.ts";
 import { retainRelationshipsForFragment } from "../../preservation/relationship-retention.ts";
 import { serializeParagraphNumberingProperties } from "./serialize-numbering.ts";
+import { buildFrameXml } from "./serialize-paragraph-formatting.ts";
 import {
   serializeTableCellPropertiesXml,
   serializeTablePropertiesXml,
@@ -716,6 +717,12 @@ function buildParagraphPropertiesXml(paragraph: ParagraphNode): string {
   pushOnOffParagraphProperty(children, "keepNext", paragraph.keepNext);
   pushOnOffParagraphProperty(children, "keepLines", paragraph.keepLines);
   pushOnOffParagraphProperty(children, "pageBreakBefore", paragraph.pageBreakBefore);
+  // ECMA-376 §17.3.1 canonical slot for framePr: between pageBreakBefore
+  // and pBdr. Coord-04 §1.19.d — direct-paragraph path.
+  {
+    const frameXml = buildFrameXml(paragraph.frameProperties);
+    if (frameXml) children.push(frameXml);
+  }
   pushOnOffParagraphProperty(children, "widowControl", paragraph.widowControl);
   if (paragraph.outlineLevel !== undefined) {
     children.push(`<w:outlineLvl w:val="${paragraph.outlineLevel}"/>`);

package/src/io/export/serialize-paragraph-formatting.ts

@@ -93,7 +93,7 @@ function buildSpacingXml(s: ParagraphSpacing | undefined): string {
   return attrs.length > 0 ? `<w:spacing ${attrs.join(" ")}/>` : "";
 }
 
-function buildFrameXml(f: FrameProperties | undefined): string {
+export function buildFrameXml(f: FrameProperties | undefined): string {
   if (!f) return "";
   // Prefer parsed rawXml when available — preserves extension attributes
   // (`w14:*`, `w15:*`, `mc:Ignorable`) that the typed field set doesn't

package/src/io/export/serialize-runtime-revisions.ts

@@ -135,8 +135,8 @@ function createParagraphPropertyChangeReplacement(
   }
   const paragraphBoundary = findParagraphBoundaryForRange(
     boundaries,
-    revision.anchor.range.from,
-    revision.anchor.range.to,
+    revision.anchor.from,
+    revision.anchor.to,
   );
   if (!paragraphBoundary) {
     return undefined;
@@ -168,14 +168,14 @@ function createRunPropertyChangeReplacement(
   }
   const paragraphBoundary = findParagraphBoundaryForRange(
     boundaries,
-    revision.anchor.range.from,
-    revision.anchor.range.to,
+    revision.anchor.from,
+    revision.anchor.to,
   );
   if (!paragraphBoundary) {
     return undefined;
   }
-  const startIndex = paragraphBoundary.boundaries.get(revision.anchor.range.from);
-  const endIndex = paragraphBoundary.boundaries.get(revision.anchor.range.to);
+  const startIndex = paragraphBoundary.boundaries.get(revision.anchor.from);
+  const endIndex = paragraphBoundary.boundaries.get(revision.anchor.to);
   if (startIndex === undefined || endIndex === undefined || endIndex < startIndex) {
     return undefined;
   }
@@ -245,13 +245,13 @@ function createRangeRevisionReplacement(
   if (anchor.kind !== "range") {
     return undefined;
   }
-  const paragraphBoundary = findParagraphBoundaryForRange(boundaries, anchor.range.from, anchor.range.to);
+  const paragraphBoundary = findParagraphBoundaryForRange(boundaries, anchor.from, anchor.to);
   if (!paragraphBoundary) {
     return undefined;
   }
 
-  const startIndex = paragraphBoundary.boundaries.get(anchor.range.from);
-  const endIndex = paragraphBoundary.boundaries.get(anchor.range.to);
+  const startIndex = paragraphBoundary.boundaries.get(anchor.from);
+  const endIndex = paragraphBoundary.boundaries.get(anchor.to);
   if (startIndex === undefined || endIndex === undefined || endIndex < startIndex) {
     return undefined;
   }
@@ -340,7 +340,7 @@ function findParagraphBoundaryForAnchor(
   boundaries: readonly RevisionParagraphBoundary[],
   revision: RevisionRecord,
 ): RevisionParagraphBoundary | undefined {
-  const anchor = revision.anchor.kind === "range" ? revision.anchor.range.from : undefined;
+  const anchor = revision.anchor.kind === "range" ? revision.anchor.from : undefined;
   if (anchor === undefined) {
     return undefined;
   }

package/src/io/export/split-review-boundaries.ts

@@ -35,8 +35,8 @@ function collectSplitPositions(
       continue;
     }
 
-    positions.add(thread.anchor.range.from);
-    positions.add(thread.anchor.range.to);
+    positions.add(thread.anchor.from);
+    positions.add(thread.anchor.to);
   }
 
   for (const revision of revisions) {
@@ -51,8 +51,8 @@ function collectSplitPositions(
       continue;
     }
 
-    positions.add(revision.anchor.range.from);
-    positions.add(revision.anchor.range.to);
+    positions.add(revision.anchor.from);
+    positions.add(revision.anchor.to);
   }
 
   return positions;

package/src/io/export/split-story-blocks-for-runtime-revisions.ts

@@ -200,8 +200,8 @@ function collectSplitPoints(
 ): number[] {
   const points = new Set<number>();
   for (const revision of revisions) {
-    const from = revision.anchor.range.from;
-    const to = revision.anchor.range.to;
+    const from = revision.anchor.from;
+    const to = revision.anchor.to;
     if (from > start && from < end) {
       points.add(from);
     }