@json-to-office/jto-ops 5.3.2 → 6.2.0

package/dist/index.d.ts

@@ -469,7 +469,7 @@ declare function extractPdfFonts(pdfPath: string, options?: {
  * pass over it, memoised per fact, so eight rules cost one search.
  */
 
-type RenderedRuleId = 'rendered/clip' | 'rendered/spill' | 'rendered/overlap' | 'rendered/text-missing' | 'rendered/font-substituted' | 'rendered/empty-page' | 'rendered/heading-stranded' | 'rendered/paragraph-split' | 'rendered/page-underfilled';
+type RenderedRuleId = 'rendered/clip' | 'rendered/spill' | 'rendered/overlap' | 'rendered/text-missing' | 'rendered/font-substituted' | 'rendered/empty-page' | 'rendered/heading-stranded' | 'rendered/paragraph-split' | 'rendered/table-split' | 'rendered/page-underfilled';
 declare const RENDERED_QUALITY_RULES: QualityRulePack;
 
 /**

package/dist/index.js

@@ -2149,6 +2149,14 @@ var RENDERED_QUALITY_RULES = {
       defaultSeverity: "warning",
       formats: ["docx"]
     }),
+    renderedRule({
+      id: "rendered/table-split",
+      code: QUALITY_CODES.RENDERED_TABLE_SPLIT,
+      description: "A table broken across a page badly: its header left alone at the foot of a page (`kind: header-alone`), or one row alone on either side of the break (`kind: orphan-row` or `widow-row`).",
+      category: "composition",
+      defaultSeverity: "info",
+      formats: ["docx"]
+    }),
     renderedRule({
       id: "rendered/paragraph-split",
       code: QUALITY_CODES.RENDERED_PARAGRAPH_SPLIT,
@@ -2975,6 +2983,79 @@ function draftRenderedFindings(input) {
         })
       );
     }
+    const TABLE_CELL = /^(.*)\/props\/(?:slots\/)?columns\/(\d+)\/(?:header|cells\/(\d+))(?:\/.*)?$/;
+    const tables = /* @__PURE__ */ new Map();
+    for (const match of matches) {
+      if (match.entry.role !== "table-header" && match.entry.role !== "table-cell" || match.status !== "mapped")
+        continue;
+      const parsed = TABLE_CELL.exec(match.entry.path);
+      if (!parsed) continue;
+      const [, table, column, row] = parsed;
+      const rowIndex = row === void 0 ? -1 : Number(row);
+      const entry = tables.get(table) ?? {
+        pages: /* @__PURE__ */ new Map(),
+        paths: /* @__PURE__ */ new Map()
+      };
+      for (const occurrence2 of match.occurrences)
+        for (const part of occurrence2.parts) {
+          const rows = entry.pages.get(part.pageIndex) ?? /* @__PURE__ */ new Set();
+          rows.add(rowIndex);
+          entry.pages.set(part.pageIndex, rows);
+        }
+      const held = entry.paths.get(rowIndex);
+      if (!held || Number(column) < held.column)
+        entry.paths.set(rowIndex, {
+          column: Number(column),
+          path: match.entry.path
+        });
+      tables.set(table, entry);
+    }
+    for (const [table, { pages: onPage, paths }] of tables) {
+      const ordered = [...onPage.keys()].sort((a, b) => a - b);
+      if (ordered.length < 2) continue;
+      const bodyRows = (page) => [...onPage.get(page) ?? []].filter((row) => row >= 0);
+      const report = (kind, page, path9, message, suggestion) => findings.push(
+        finding({
+          ruleId: "rendered/table-split",
+          mapping: "mapped",
+          page: page + 1,
+          path: path9,
+          message,
+          suggestion,
+          context: { kind, table }
+        })
+      );
+      const firstPage = ordered[0];
+      const lastPage = ordered[ordered.length - 1];
+      if (onPage.get(firstPage).has(-1) && bodyRows(firstPage).length === 0) {
+        report(
+          "header-alone",
+          firstPage,
+          paths.get(-1).path,
+          `The table's header is the only part of it on page ${firstPage + 1}; its rows start on the next page.`,
+          "Keep the header with its first rows, or move the table so it starts on the next page."
+        );
+        continue;
+      }
+      const opening = bodyRows(firstPage);
+      const closing = bodyRows(lastPage);
+      if (opening.length === 1)
+        report(
+          "orphan-row",
+          firstPage,
+          paths.get(opening[0]).path,
+          `The table leaves one row alone at the foot of page ${firstPage + 1}.`,
+          "Keep the table together on one page, or edit the rows above it so the break falls elsewhere."
+        );
+      else if (closing.length === 1)
+        report(
+          "widow-row",
+          lastPage,
+          paths.get(closing[0]).path,
+          `The table leaves one row alone at the top of page ${lastPage + 1}.`,
+          "Keep the table together on one page, or edit the rows above it so the break falls elsewhere."
+        );
+    }
   }
   const inventory = {
     mapped: 0,