@beyondwork/docx-react-component 1.0.18 → 1.0.20

package/src/validation/compatibility-engine.ts

@@ -104,23 +104,13 @@ function collectContentFeatures(
     lists: false,
     hyperlinks: false,
     images: false,
+    tables: false,
+    sections: false,
+    contentControls: false,
   };
 
   for (let index = 0; index < content.children.length; index += 1) {
-    const block = content.children[index];
-    if (block.type !== "paragraph") {
-      continue;
-    }
-
-    flags.paragraphs = true;
-    if (block.styleId?.toLowerCase().startsWith("heading")) {
-      flags.headings = true;
-    }
-    if (block.numbering) {
-      flags.lists = true;
-    }
-
-    measureParagraph(block, flags);
+    measureBlock(content.children[index], flags);
   }
 
   const entries: CompatibilityFeatureEntry[] = [];
@@ -145,16 +135,94 @@ function collectContentFeatures(
   if (flags.images) {
     entries.push(supportedEntry("inline-images", "Inline image placements stay attached to preserved media parts."));
   }
+  if (flags.tables) {
+    entries.push(
+      supportedEntry(
+        "tables",
+        "Structured tables keep cell topology and common visual properties through runtime editing and export.",
+      ),
+    );
+  }
+  if (flags.sections) {
+    entries.push(
+      supportedEntry(
+        "sections",
+        "Section boundaries and page-layout properties remain structured and export-safe.",
+      ),
+    );
+  }
+  if (flags.contentControls) {
+    entries.push(
+      supportedEntry(
+        "content-controls",
+        "Common content controls render through structured SDT state and round-trip without flattening to opaque placeholders.",
+      ),
+    );
+  }
   return entries;
 }
 
+function measureBlock(
+  block: BlockNode,
+  flags: {
+    paragraphs: boolean;
+    runs: boolean;
+    whitespace: boolean;
+    headings: boolean;
+    lists: boolean;
+    hyperlinks: boolean;
+    images: boolean;
+    tables: boolean;
+    sections: boolean;
+    contentControls: boolean;
+  },
+): number {
+  switch (block.type) {
+    case "paragraph":
+      flags.paragraphs = true;
+      if (block.styleId?.toLowerCase().startsWith("heading")) {
+        flags.headings = true;
+      }
+      if (block.numbering) {
+        flags.lists = true;
+      }
+      return measureParagraph(block, flags);
+    case "table":
+      flags.tables = true;
+      return block.rows.reduce(
+        (size, row) =>
+          size +
+          row.cells.reduce(
+            (cellSize, cell) =>
+              cellSize + cell.children.reduce((childSize, child) => childSize + measureBlock(child, flags), 0),
+            0,
+          ),
+        0,
+      );
+    case "sdt":
+      flags.contentControls = true;
+      return block.children.reduce((size, child) => size + measureBlock(child, flags), 0);
+    case "custom_xml":
+      return block.children.reduce((size, child) => size + measureBlock(child, flags), 0);
+    case "section_break":
+      flags.sections = true;
+      return 1;
+    default:
+      return 0;
+  }
+}
+
 function measureParagraph(
   paragraph: ParagraphNode,
   flags: {
+    paragraphs: boolean;
     runs: boolean;
     whitespace: boolean;
     hyperlinks: boolean;
     images: boolean;
+    tables: boolean;
+    sections: boolean;
+    contentControls: boolean;
   },
 ): number {
   let size = 0;
@@ -170,10 +238,14 @@ function measureParagraph(
 function measureInlineNode(
   node: InlineNode,
   flags: {
+    paragraphs: boolean;
     runs: boolean;
     whitespace: boolean;
     hyperlinks: boolean;
     images: boolean;
+    tables: boolean;
+    sections: boolean;
+    contentControls: boolean;
   },
 ): number {
   switch (node.type) {
@@ -310,12 +382,12 @@ function collectSubPartFeatures(
     Object.keys(subParts.footnoteCollection?.footnotes ?? {}).length +
     Object.keys(subParts.footnoteCollection?.endnotes ?? {}).length;
 
-  if (hasHeaderFooterContent) {
+  if (hasHeaderFooterContent && !entries.some((entry) => entry.featureKey === "headers-footers-lossy")) {
     entries.push({
       featureEntryId: "feature:subparts:headers-footers",
       featureKey: "headers-footers",
-      featureClass: "preserve-only",
-      message: "Headers and footers are preserved through structured sub-part ownership.",
+      featureClass: "supported-roundtrip",
+      message: "Headers and footers resolve as structured secondary stories with export-safe ownership.",
       details: {
         headerCount: subParts.headers?.length ?? 0,
         footerCount: subParts.footers?.length ?? 0,
@@ -323,12 +395,12 @@ function collectSubPartFeatures(
     });
   }
 
-  if (noteCount > 0) {
+  if (noteCount > 0 && !entries.some((entry) => entry.featureKey === "notes-lossy")) {
     entries.push({
       featureEntryId: "feature:subparts:notes",
       featureKey: "notes",
-      featureClass: "preserve-only",
-      message: "Footnotes and endnotes are preserved through structured sub-part ownership.",
+      featureClass: "supported-roundtrip",
+      message: "Footnotes and endnotes resolve as structured secondary stories with export-safe ownership.",
       details: {
         footnoteCount: Object.keys(subParts.footnoteCollection?.footnotes ?? {}).length,
         endnoteCount: Object.keys(subParts.footnoteCollection?.endnotes ?? {}).length,
@@ -407,6 +479,15 @@ function collectLossyBlocks(
   const issues: string[] = [];
 
   for (const block of blocks) {
+    if (block.type === "table") {
+      for (const row of block.rows) {
+        for (const cell of row.cells) {
+          issues.push(...collectLossyBlocks(cell.children, `${surface}:table-cell`));
+        }
+      }
+      continue;
+    }
+
     if (block.type !== "paragraph") {
       issues.push(`${surface}:${block.type}`);
       continue;
@@ -414,9 +495,6 @@ function collectLossyBlocks(
 
     if (
       block.numbering !== undefined ||
-      block.spacing !== undefined ||
-      block.indentation !== undefined ||
-      block.tabStops !== undefined ||
       block.keepNext !== undefined ||
       block.keepLines !== undefined ||
       block.outlineLevel !== undefined ||
@@ -445,6 +523,8 @@ function collectLossyInlineContent(
 ): string[] {
   switch (node.type) {
     case "text": {
+      const allowSecondaryStoryColorMarks =
+        surface.startsWith("header:") || surface.startsWith("footer:");
       const unsupportedMarks = (node.marks ?? [])
         .filter(
           (mark) =>
@@ -452,7 +532,14 @@ function collectLossyInlineContent(
             mark.type !== "italic" &&
             mark.type !== "underline" &&
             mark.type !== "strikethrough" &&
-            mark.type !== "doubleStrikethrough",
+            mark.type !== "doubleStrikethrough" &&
+            mark.type !== "fontFamily" &&
+            mark.type !== "fontSize" &&
+            mark.type !== "textColor" &&
+            (!allowSecondaryStoryColorMarks || mark.type !== "backgroundColor") &&
+            (!allowSecondaryStoryColorMarks || mark.type !== "highlight") &&
+            mark.type !== "smallCaps" &&
+            mark.type !== "allCaps",
         )
         .map((mark) => `${surface}:mark:${mark.type}`);
       return unsupportedMarks;
@@ -460,6 +547,14 @@ function collectLossyInlineContent(
     case "tab":
     case "hard_break":
     case "footnote_ref":
+    case "field":
+    case "bookmark_start":
+    case "bookmark_end":
+    case "shape":
+    case "wordart":
+    case "vml_shape":
+    case "chart_preview":
+    case "smartart_preview":
       return [];
     default:
       return [`${surface}:${node.type}`];

package/src/validation/compatibility-report.ts

@@ -44,6 +44,7 @@ export const COMPATIBILITY_FEATURE_KEYS = [
   "unknown-package-parts",
   "package-integrity",
   "relationship-integrity",
+  "workflow-scopes",
 ] as const;
 
 export type CompatibilityFeatureKey =

package/src/validation/diagnostics.ts

@@ -48,6 +48,7 @@ export const EDITOR_ERROR_SOURCES = [
   "runtime",
   "validation",
   "datastore",
+  "host",
   "export",
 ] as const;