@ansonlai/docx-redline-js 0.8.3 → 0.9.0

package/pipeline/serialization.js

@@ -4,14 +4,14 @@
  * Converts patched run model back to OOXML with track changes.
  */
 
-import { RunKind, escapeXml, createRevisionMetadata } from '../core/types.js';
-import { getApplicableFormatHints } from './markdown-processor.js';
-import { serializeXml } from '../adapters/xml-adapter.js';
-import { warn } from '../adapters/logger.js';
-import { buildDocumentFragmentPackage } from '../services/package-builder.js';
-import { getDefaultAuthor } from '../adapters/config.js';
-
-const XMLNS_ATTR_REGEX = /\s+xmlns:[^=]+="[^"]*"/g;
+import { RunKind, escapeXml, createRevisionMetadata } from '../core/types.js';
+import { getApplicableFormatHints } from './markdown-processor.js';
+import { serializeXml } from '../adapters/xml-adapter.js';
+import { warn } from '../adapters/logger.js';
+import { buildDocumentFragmentPackage } from '../services/package-builder.js';
+import { getDefaultAuthor } from '../adapters/config.js';
+
+const XMLNS_ATTR_REGEX = /\s+xmlns:[^=]+="[^"]*"/g;
 
 /**
  * Serializes a patched run model to OOXML.
@@ -22,48 +22,48 @@ const XMLNS_ATTR_REGEX = /\s+xmlns:[^=]+="[^"]*"/g;
  * @param {import('../core/types.js').SerializationOptions} [options={}] - Serialization options
  * @returns {string} OOXML paragraph string (WITHOUT namespace - added by wrapper)
  */
-export function serializeToOoxml(patchedModel, pPr, formatHints = [], options = {}) {
-    const serializationOptions = normalizeSerializationOptions(options);
-    const { generateRedlines } = serializationOptions;
-    const paragraphs = [];
-    let currentPPrXml = '';
-    let currentPPrElement = null;
-    let currentRuns = [];
+export function serializeToOoxml(patchedModel, pPr, formatHints = [], options = {}) {
+    const serializationOptions = normalizeSerializationOptions(options);
+    const { generateRedlines } = serializationOptions;
+    const paragraphs = [];
+    let currentPPrXml = '';
+    let currentPPrElement = null;
+    let currentRuns = [];
 
     // Helper to flush accumulated runs into a paragraph
     function flushParagraph() {
         if (currentRuns.length > 0 || paragraphs.length === 0) {
             // Build paragraph properties - handle both string and DOM element
-            let pPrContent = '';
-            if (currentPPrXml) {
-                pPrContent = stripNamespaceDeclarations(currentPPrXml);
-            } else if (currentPPrElement) {
-                currentPPrXml = stripNamespaceDeclarations(serializeXml(currentPPrElement));
-                pPrContent = currentPPrXml;
-            } else if (pPr) {
-                // Fallback to legacy pPr if no PARAGRAPH_START was seen
-                if (typeof pPr === 'string') {
-                    pPrContent = pPr;
-                } else {
-                    pPrContent = serializeXml(pPr);
-                }
-                pPrContent = stripNamespaceDeclarations(pPrContent);
-            }
-            paragraphs.push(`<w:p>${pPrContent}${currentRuns.join('')}</w:p>`);
-            currentRuns = [];
-        }
+            let pPrContent = '';
+            if (currentPPrXml) {
+                pPrContent = stripNamespaceDeclarations(currentPPrXml);
+            } else if (currentPPrElement) {
+                currentPPrXml = stripNamespaceDeclarations(serializeXml(currentPPrElement));
+                pPrContent = currentPPrXml;
+            } else if (pPr) {
+                // Fallback to legacy pPr if no PARAGRAPH_START was seen
+                if (typeof pPr === 'string') {
+                    pPrContent = pPr;
+                } else {
+                    pPrContent = serializeXml(pPr);
+                }
+                pPrContent = stripNamespaceDeclarations(pPrContent);
+            }
+            paragraphs.push(`<w:p>${pPrContent}${currentRuns.join('')}</w:p>`);
+            currentRuns = [];
+        }
     }
 
     for (const item of patchedModel) {
         switch (item.kind) {
             case RunKind.PARAGRAPH_START:
                 // Flush previous paragraph before starting a new one
-                if (currentRuns.length > 0 || paragraphs.length > 0) {
-                    flushParagraph();
-                }
-                currentPPrXml = item.pPrXml || '';
-                currentPPrElement = item.pPrElement || null;
-                break;
+                if (currentRuns.length > 0 || paragraphs.length > 0) {
+                    flushParagraph();
+                }
+                currentPPrXml = item.pPrXml || '';
+                currentPPrElement = item.pPrElement || null;
+                break;
 
             case RunKind.TEXT:
                 currentRuns.push(buildRunXmlWithHints(item, formatHints, serializationOptions));
@@ -87,11 +87,11 @@ export function serializeToOoxml(patchedModel, pPr, formatHints = [], options =
 
             case RunKind.BOOKMARK:
             case RunKind.HYPERLINK:
-                // Pass through original XML - but strip any namespace declarations
-                if (item.nodeXml) {
-                    currentRuns.push(stripNamespaceDeclarations(item.nodeXml));
-                }
-                break;
+                // Pass through original XML - but strip any namespace declarations
+                if (item.nodeXml) {
+                    currentRuns.push(stripNamespaceDeclarations(item.nodeXml));
+                }
+                break;
 
             case RunKind.CONTAINER_START:
                 if (item.containerKind === 'sdt') {
@@ -134,29 +134,29 @@ export function serializeToOoxml(patchedModel, pPr, formatHints = [], options =
  * @param {import('../core/types.js').SerializationOptions|string|undefined|null} options - Raw options
  * @returns {import('../core/types.js').SerializationOptions}
  */
-function normalizeSerializationOptions(options) {
-    // Backward compatibility: accept legacy `font` string signature.
-    if (typeof options === 'string') {
-        return {
-            author: getDefaultAuthor(),
-            generateRedlines: true,
-            font: options,
-            revisionIdAllocator: null
-        };
-    }
-
-    const normalized = options && typeof options === 'object' ? options : {};
-    const resolvedAuthor = typeof normalized.author === 'string' && normalized.author.trim()
-        ? normalized.author.trim()
-        : getDefaultAuthor();
-
-    return {
-        author: resolvedAuthor,
-        generateRedlines: normalized.generateRedlines ?? true,
-        font: normalized.font ?? null,
-        revisionIdAllocator: normalized.revisionIdAllocator ?? null
-    };
-}
+function normalizeSerializationOptions(options) {
+    // Backward compatibility: accept legacy `font` string signature.
+    if (typeof options === 'string') {
+        return {
+            author: getDefaultAuthor(),
+            generateRedlines: true,
+            font: options,
+            revisionIdAllocator: null
+        };
+    }
+
+    const normalized = options && typeof options === 'object' ? options : {};
+    const resolvedAuthor = typeof normalized.author === 'string' && normalized.author.trim()
+        ? normalized.author.trim()
+        : getDefaultAuthor();
+
+    return {
+        author: resolvedAuthor,
+        generateRedlines: normalized.generateRedlines ?? true,
+        font: normalized.font ?? null,
+        revisionIdAllocator: normalized.revisionIdAllocator ?? null
+    };
+}
 
 /**
  * Builds a run XML element, applying format hints if applicable.
@@ -166,10 +166,10 @@ function normalizeSerializationOptions(options) {
  * @param {import('../core/types.js').SerializationOptions} options - Serialization options
  * @returns {string}
  */
-function buildRunXmlWithHints(item, formatHints, options = {}) {
-    const applicableHints = getApplicableFormatHints(formatHints, item.startOffset, item.endOffset);
-    const font = options?.font ?? null;
-    let cleanRPr = item.rPrXml ? stripNamespaceDeclarations(item.rPrXml) : '';
+function buildRunXmlWithHints(item, formatHints, options = {}) {
+    const applicableHints = getApplicableFormatHints(formatHints, item.startOffset, item.endOffset);
+    const font = options?.font ?? null;
+    let cleanRPr = item.rPrXml ? stripNamespaceDeclarations(item.rPrXml) : '';
 
     if (font) {
         cleanRPr = applyFont(cleanRPr, font);
@@ -219,7 +219,24 @@ function buildRunXmlWithHints(item, formatHints, options = {}) {
 function buildSimpleRun(text, rPrXml) {
     if (!text) return '';
     const rPr = rPrXml || '';
-    return `<w:r>${rPr}<w:t xml:space="preserve">${escapeXml(text)}</w:t></w:r>`;
+    return `<w:r>${rPr}${buildRunContent(text, 'w:t')}</w:r>`;
+}
+
+/**
+ * Builds run content, restoring in-run line breaks and tabs (ingested from
+ * w:br/w:cr/w:tab as "\n"/"\t") as elements; raw control characters inside
+ * w:t or w:delText render as spaces in Word.
+ *
+ * @param {string} text - Run text
+ * @param {'w:t'|'w:delText'} textTag - Text element name
+ * @returns {string}
+ */
+function buildRunContent(text, textTag) {
+    return String(text).split(/(\r\n|[\n\r\v\t])/).map(part => {
+        if (part === '\t') return '<w:tab/>';
+        if (part === '\n' || part === '\r' || part === '\r\n' || part === '\v') return '<w:br/>';
+        return part ? `<${textTag} xml:space="preserve">${escapeXml(part)}</${textTag}>` : '';
+    }).join('');
 }
 
 /**
@@ -229,21 +246,21 @@ function buildSimpleRun(text, rPrXml) {
  * @param {import('../core/types.js').SerializationOptions} options - Serialization options
  * @returns {string}
  */
-function buildDeletionXml(item, options = {}) {
-    const metadata = createRevisionMetadata(
-        options.author ?? getDefaultAuthor(),
-        options.revisionIdAllocator,
-        'del'
-    );
-    const font = options.font ?? null;
-    let rPr = item.rPrXml ? stripNamespaceDeclarations(item.rPrXml) : '';
+function buildDeletionXml(item, options = {}) {
+    const metadata = createRevisionMetadata(
+        options.author ?? getDefaultAuthor(),
+        options.revisionIdAllocator,
+        'del'
+    );
+    const font = options.font ?? null;
+    let rPr = item.rPrXml ? stripNamespaceDeclarations(item.rPrXml) : '';
 
     if (font) {
         rPr = applyFont(rPr, font);
     }
 
     return `<w:del w:id="${metadata.id}" w:author="${escapeXml(metadata.author)}" w:date="${metadata.date}">` +
-        `<w:r>${rPr}<w:delText xml:space="preserve">${escapeXml(item.text)}</w:delText></w:r>` +
+        `<w:r>${rPr}${buildRunContent(item.text, 'w:delText')}</w:r>` +
         `</w:del>`;
 }
 
@@ -255,18 +272,18 @@ function buildDeletionXml(item, options = {}) {
  * @param {import('../core/types.js').SerializationOptions} options - Serialization options
  * @returns {string}
  */
-function buildInsertionXml(item, formatHints, options = {}) {
-    const metadata = createRevisionMetadata(
-        options.author ?? getDefaultAuthor(),
-        options.revisionIdAllocator,
-        'ins'
-    );
+function buildInsertionXml(item, formatHints, options = {}) {
+    const metadata = createRevisionMetadata(
+        options.author ?? getDefaultAuthor(),
+        options.revisionIdAllocator,
+        'ins'
+    );
     const font = options.font ?? null;
 
     // Build the inner run content with format hints
     const applicableHints = getApplicableFormatHints(formatHints, item.startOffset, item.endOffset);
     let innerContent = '';
-    let cleanRPr = item.rPrXml ? stripNamespaceDeclarations(item.rPrXml) : '';
+    let cleanRPr = item.rPrXml ? stripNamespaceDeclarations(item.rPrXml) : '';
 
     if (font) {
         cleanRPr = applyFont(cleanRPr, font);
@@ -383,7 +400,7 @@ export function wrapInDocumentFragment(paragraphXml, options = {}) {
  * @param {import('../core/types.js').DocumentFragmentOptions|boolean|undefined|null} options - Raw options
  * @returns {import('../core/types.js').DocumentFragmentOptions}
  */
-function normalizeFragmentOptions(options) {
+function normalizeFragmentOptions(options) {
     if (typeof options === 'boolean') {
         return { includeNumbering: options };
     }
@@ -397,9 +414,9 @@ function normalizeFragmentOptions(options) {
         numberingXml: options.numberingXml ?? null,
         appendTrailingParagraph: options.appendTrailingParagraph ?? true
     };
-}
-
-function stripNamespaceDeclarations(xml) {
-    return xml ? xml.replace(XMLNS_ATTR_REGEX, '') : '';
-}
+}
+
+function stripNamespaceDeclarations(xml) {
+    return xml ? xml.replace(XMLNS_ATTR_REGEX, '') : '';
+}
 

package/scripts/export-validation-fixtures.mjs

@@ -16,6 +16,7 @@ import {
   rejectTrackedChangesInOoxml
 } from '../services/revision-comment-management.js';
 import { WORD_TASK_CASES } from '../tests/fixtures/word-task-cases.mjs';
+import { mergeNumberingParts } from '../services/numbering-allocation.js';
 
 const { DOMParser, XMLSerializer } = await import('@xmldom/xmldom');
 configureXmlProvider({ DOMParser, XMLSerializer });
@@ -56,7 +57,8 @@ for (const testCase of cases) {
       sourceDocumentXml,
       testCase.batchOperations,
       'Validation',
-      null,
+      // A numbering binding resolves against the source numbering part, as it does in a real package.
+      testCase.sourceNumbering ? { numberingXml: testCase.packageParts?.numberingXml } : null,
       operationOptions
     )
     : await applyOperationToDocumentXml(
@@ -118,7 +120,7 @@ for (const testCase of cases) {
   }
 
   if (testCase.requiredNumberingFormats) {
-    const numberingXml = testCase.packageParts?.numberingXml || result.numberingXml || '';
+    const numberingXml = [testCase.packageParts?.numberingXml, result.numberingXml, ...(result.numberingXmlParts || [])].filter(Boolean).join('');
     for (const format of testCase.requiredNumberingFormats) {
       if (!numberingXml.includes(`<w:numFmt w:val="${format}"`)) {
         console.error(`FAIL ${testCase.name}: required numbering format ${format} is missing`);
@@ -162,14 +164,20 @@ for (const testCase of cases) {
     continue;
   }
 
+  // A batch returns numbering payloads as parts; merge them into the source part through the one allocator merge.
+  const batchNumberingXml = Array.isArray(result.numberingXmlParts) && result.numberingXmlParts.length > 0
+    ? mergeNumberingParts(testCase.packageParts?.numberingXml || '', result.numberingXmlParts)
+    : null;
+  const resultNumberingXml = result.numberingXml || batchNumberingXml || null;
   writeFileSync(join(outputDir, `${testCase.name}.document.xml`), result.documentXml, 'utf8');
-  if (result.numberingXml) {
-    writeFileSync(join(outputDir, `${testCase.name}.numbering.xml`), result.numberingXml, 'utf8');
+  if (resultNumberingXml) {
+    writeFileSync(join(outputDir, `${testCase.name}.numbering.xml`), resultNumberingXml, 'utf8');
   }
 
   const packageParts = {
-    numberingXml: result.numberingXml || null,
-    ...(testCase.packageParts || {})
+    numberingXml: resultNumberingXml,
+    ...(testCase.packageParts || {}),
+    ...(batchNumberingXml ? { numberingXml: batchNumberingXml } : {})
   };
   let packageEntries;
   let docx;
@@ -228,6 +236,11 @@ for (const testCase of cases) {
     sourceText: testCase.sourceText || testCase.original,
     modifiedText: preprocessMarkdown(testCase.modified).cleanText,
     requiredNumberingFormats: testCase.requiredNumberingFormats || [],
+    ...(testCase.expectedAcceptedListStrings ? {
+      expectedAcceptedListStrings: testCase.expectedAcceptedListStrings,
+      expectedRejectedListStrings: testCase.expectedRejectedListStrings
+    } : {}),
+    ...(testCase.expectedAcceptedListGroupSizes ? { expectedAcceptedListGroupSizes: testCase.expectedAcceptedListGroupSizes } : {}),
     untouchedPartSha256
   };
   writeFileSync(join(outputDir, `${testCase.name}.expected.json`), `${JSON.stringify(expected, null, 2)}\n`, 'utf8');

package/scripts/generate-word-revision-semantics-matrix.mjs

@@ -0,0 +1,134 @@
+/**
+ * Builds the Word revision-semantics fixture matrix (WP-2 COM backstop).
+ *
+ *   node scripts/generate-word-revision-semantics-matrix.mjs
+ *   powershell -File scripts/word-revision-semantics-matrix.ps1
+ *
+ * Writes tests/fixtures/word-revision-semantics/cases.json and one minimal .docx
+ * per case; the PowerShell script drives Word and writes expectations.json.
+ */
+import { mkdirSync, writeFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { buildZip } from './lib/minimal-zip.mjs';
+
+const OUT = join(dirname(fileURLToPath(import.meta.url)), '..', 'tests', 'fixtures', 'word-revision-semantics');
+const W = 'http://schemas.openxmlformats.org/wordprocessingml/2006/main';
+const R = 'http://schemas.openxmlformats.org/officeDocument/2006/relationships';
+const PKG = 'http://schemas.openxmlformats.org/package/2006/relationships';
+const DATE = '2026-01-01T00:00:00Z';
+
+let id = 100;
+const rev = (tag, author = 'A') => `<w:${tag} w:id="${id++}" w:author="${author}" w:date="${DATE}"/>`;
+const run = t => `<w:r><w:t xml:space="preserve">${t}</w:t></w:r>`;
+const insRun = (t, a = 'A') => `<w:ins w:id="${id++}" w:author="${a}" w:date="${DATE}"><w:r><w:t xml:space="preserve">${t}</w:t></w:r></w:ins>`;
+const delRun = (t, a = 'A') => `<w:del w:id="${id++}" w:author="${a}" w:date="${DATE}"><w:r><w:delText xml:space="preserve">${t}</w:delText></w:r></w:del>`;
+const NUM = '<w:numPr><w:ilvl w:val="0"/><w:numId w:val="1"/></w:numPr>';
+const LIST = '<w:pStyle w:val="ListParagraph"/>' + NUM;
+const mark = (kind, a = 'A') => `<w:rPr>${rev(kind, a)}</w:rPr>`;
+// pPr child order: style/numPr..., rPr (mark), sectPr, pPrChange
+const para = (content, { ppr = '', mark: m = '', extra = '' } = {}) =>
+    `<w:p>${ppr || m || extra ? `<w:pPr>${ppr}${m}${extra}</w:pPr>` : ''}${content}</w:p>`;
+const cell = inner => `<w:tc><w:tcPr><w:tcW w:w="4000" w:type="dxa"/></w:tcPr>${inner}</w:tc>`;
+const table = rows => `<w:tbl><w:tblPr><w:tblW w:w="0" w:type="auto"/></w:tblPr><w:tblGrid><w:gridCol w:w="4000"/></w:tblGrid>${rows}</w:tbl>`;
+const row = (inner, trPr = '') => `<w:tr>${trPr ? `<w:trPr>${trPr}</w:trPr>` : ''}${inner}</w:tr>`;
+const SECT = '<w:sectPr><w:pgSz w:w="12240" w:h="15840"/><w:pgMar w:top="1440" w:right="1440" w:bottom="1440" w:left="1440" w:header="720" w:footer="720" w:gutter="0"/></w:sectPr>';
+
+const defs = [];
+function add(name, description, build) { id = 100; defs.push({ name, description, bodyXml: build() + SECT }); }
+const P = t => para(run(t));
+const pprChange = old => `<w:pPrChange w:id="${id++}" w:author="A" w:date="${DATE}"><w:pPr>${old}</w:pPr></w:pPrChange>`;
+const BOLD_MARK = '<w:rPr><w:b/></w:rPr>';
+
+add('ins-mark-mid-plain', 'Mid-body paragraph with inserted content and inserted mark',
+    () => P('Alpha.') + para(insRun('Inserted.'), { mark: mark('ins') }) + P('Omega.'));
+add('ins-mark-mid-untracked-content', 'Mid-body paragraph, inserted mark, untracked content',
+    () => P('Alpha.') + para(run('Untracked.'), { mark: mark('ins') }) + P('Omega.'));
+add('ins-mark-list-then-plain', 'List paragraph with inserted mark followed by plain paragraph',
+    () => P('Alpha.') + para(insRun('Item.'), { ppr: LIST, mark: mark('ins') }) + P('Plain after.'));
+add('ins-mark-last-prev-deleted', 'Last body paragraph inserted mark; preceding paragraph mark deleted (replacement shape)',
+    () => P('First.') + para(delRun('Second paragraph.'), { mark: mark('del') })
+        + para(insRun('Item one.'), { ppr: LIST, mark: mark('ins') })
+        + para(insRun('Item two.'), { ppr: LIST, mark: mark('ins') }));
+add('ins-mark-last-prev-plain', 'Last body paragraph inserted mark; preceding paragraph plain',
+    () => P('First.') + P('Second paragraph.') + para(insRun('Added last.'), { ppr: LIST, mark: mark('ins') }));
+add('ins-mark-last-plain-style', 'Last body paragraph inserted mark, no list props',
+    () => P('First.') + para(insRun('Added last.'), { mark: mark('ins') }));
+add('ins-mark-cell-sole', 'Sole paragraph in a table cell has inserted mark',
+    () => P('Intro.') + table(row(cell(para(insRun('Cell ins.'), { mark: mark('ins') })))) + P('After table.'));
+add('ins-mark-cell-last-of-two', 'Last of two paragraphs in a cell has inserted mark',
+    () => P('Intro.') + table(row(cell(P('Cell one.') + para(insRun('Cell two.'), { mark: mark('ins') })))) + P('After table.'));
+add('ins-mark-before-table', 'Paragraph with inserted mark directly before a table',
+    () => P('Intro.') + para(insRun('Inserted.'), { mark: mark('ins') }) + table(row(cell(P('Cell.')))) + P('After table.'));
+add('ins-mark-bookmark-between', 'Inserted mark paragraph with bookmark between paragraphs',
+    () => P('Alpha.') + para('<w:bookmarkStart w:id="900" w:name="bm1"/>' + insRun('Inserted.') + '<w:bookmarkEnd w:id="900"/>', { mark: mark('ins') }) + P('Omega.'));
+
+add('del-mark-mid-plain', 'Mid-body paragraph with deleted content and deleted mark',
+    () => P('Alpha.') + para(delRun('Deleted.'), { mark: mark('del') }) + P('Omega.'));
+add('del-mark-mid-untracked-content', 'Mid-body paragraph, deleted mark, untracked content',
+    () => P('Alpha.') + para(run('Untracked.'), { mark: mark('del') }) + P('Omega.'));
+add('del-mark-list-then-plain', 'List paragraph with deleted mark followed by plain paragraph',
+    () => P('Alpha.') + para(delRun('Item.'), { ppr: LIST, mark: mark('del') }) + P('Plain after.'));
+add('del-mark-last', 'Last body paragraph with deleted mark',
+    () => P('First.') + para(delRun('Gone last.'), { mark: mark('del') }));
+add('del-mark-last-list-prev-plain', 'Last body list paragraph with deleted mark; preceding plain',
+    () => P('First.') + P('Second.') + para(delRun('Gone last.'), { ppr: LIST, mark: mark('del') }));
+add('del-mark-cell-sole', 'Sole cell paragraph with deleted mark',
+    () => P('Intro.') + table(row(cell(para(delRun('Cell del.'), { mark: mark('del') })))) + P('After table.'));
+add('del-mark-cell-last-of-two', 'Last of two cell paragraphs with deleted mark',
+    () => P('Intro.') + table(row(cell(P('Cell one.') + para(delRun('Cell two.'), { mark: mark('del') })))) + P('After table.'));
+add('del-mark-before-table', 'Paragraph with deleted mark directly before a table',
+    () => P('Intro.') + para(delRun('Deleted.'), { mark: mark('del') }) + table(row(cell(P('Cell.')))) + P('After table.'));
+add('del-mark-bookmark-between', 'Deleted mark paragraph with bookmark between paragraphs',
+    () => P('Alpha.') + para('<w:bookmarkStart w:id="900" w:name="bm1"/>' + delRun('Deleted.') + '<w:bookmarkEnd w:id="900"/>', { mark: mark('del') }) + P('Omega.'));
+
+add('pprchange-restores-list', 'pPrChange: current Heading1, previous list item; mark rPr has bold',
+    () => P('Alpha.') + para(run('Was a list item.'), { ppr: '<w:pStyle w:val="Heading1"/>', mark: BOLD_MARK, extra: pprChange(LIST) }) + P('Omega.'));
+add('pprchange-removes-list', 'pPrChange: current list item, previous plain; mark rPr has bold',
+    () => P('Alpha.') + para(run('Became a list item.'), { ppr: LIST, mark: BOLD_MARK, extra: pprChange('') }) + P('Omega.'));
+add('del-mark-with-sectpr', 'Deleted-mark paragraph carrying a section break',
+    () => para(delRun('Section one end.'), { mark: mark('del'), extra: '<w:sectPr><w:pgSz w:w="12240" w:h="15840"/></w:sectPr>' }) + P('Section two.'));
+add('adjacent-ins-then-del-marks', 'Inserted-mark paragraph followed by deleted-mark paragraph',
+    () => P('Alpha.') + para(insRun('New.'), { mark: mark('ins') }) + para(delRun('Old.'), { mark: mark('del') }) + P('Keep.'));
+add('adjacent-del-then-ins-marks', 'Deleted-mark paragraph followed by inserted-mark paragraph',
+    () => P('Alpha.') + para(delRun('Old.'), { mark: mark('del') }) + para(insRun('New.'), { mark: mark('ins') }) + P('Keep.'));
+add('mixed-authors-marks', 'Inserted marks by two authors plus a deleted mark by a third',
+    () => P('Alpha.') + para(insRun('By A.', 'A'), { mark: mark('ins', 'A') })
+        + para(insRun('By B.', 'B'), { mark: mark('ins', 'B') })
+        + para(delRun('By C.', 'C'), { mark: mark('del', 'C') }) + P('Omega.'));
+add('ins-row', 'Table with an inserted row',
+    () => P('Intro.') + table(
+        row(cell(P('Existing row.')))
+        + row(cell(para(insRun('Inserted row.'), { mark: mark('ins') })), rev('ins'))) + P('After table.'));
+add('del-row', 'Table with a deleted row',
+    () => P('Intro.') + table(
+        row(cell(P('Existing row.')))
+        + row(cell(para(delRun('Deleted row.'), { mark: mark('del') })), rev('del'))) + P('After table.'));
+
+// R5: end-of-document table append. insRow = row-level ins with inserted cell marks (library shape).
+const insRow = (text, cols = 2) => row(Array.from({ length: cols }, (_, i) => cell(para(insRun(`${text}${i + 1}`), { mark: mark('ins') }))).join(''), rev('ins'));
+add('table-append-end-trailing-ins-mark', 'Table appended after the final paragraph: inserted rows, then an empty trailing paragraph with an inserted mark (library shape)',
+    () => P('Alpha.') + P('Final paragraph.') + table(insRow('R1C') + insRow('R2C')) + para('', { mark: mark('ins') }));
+add('table-append-end-word-native', 'Table appended after the final paragraph as Word writes it: final paragraph mark inserted, inserted rows, untracked empty trailing paragraph',
+    () => P('Alpha.') + para(run('Final paragraph.'), { mark: mark('ins') }) + table(insRow('R1C') + insRow('R2C')) + P(''));
+add('table-append-end-untracked-trailing', 'Table appended after the final paragraph: inserted rows followed by an untracked empty trailing paragraph',
+    () => P('Alpha.') + P('Final paragraph.') + table(insRow('R1C') + insRow('R2C')) + P(''));
+add('table-append-end-no-trailing-paragraph', 'Table appended at the very end with no paragraph after it: Word supplies an untracked empty paragraph (the pretty-printed-body defect shape)',
+    () => P('Alpha.') + P('Final paragraph.') + table(insRow('R1C') + insRow('R2C')));
+
+const wrap = body => `<?xml version="1.0" encoding="UTF-8" standalone="yes"?><w:document xmlns:w="${W}" xmlns:r="${R}"><w:body>${body}</w:body></w:document>`;
+const styles = `<?xml version="1.0" encoding="UTF-8" standalone="yes"?><w:styles xmlns:w="${W}"><w:style w:type="paragraph" w:default="1" w:styleId="Normal"><w:name w:val="Normal"/></w:style><w:style w:type="paragraph" w:styleId="Heading1"><w:name w:val="heading 1"/><w:basedOn w:val="Normal"/><w:pPr><w:outlineLvl w:val="0"/></w:pPr><w:rPr><w:b/></w:rPr></w:style><w:style w:type="paragraph" w:styleId="ListParagraph"><w:name w:val="List Paragraph"/><w:basedOn w:val="Normal"/><w:pPr><w:ind w:left="720"/></w:pPr></w:style></w:styles>`;
+const numbering = `<?xml version="1.0" encoding="UTF-8" standalone="yes"?><w:numbering xmlns:w="${W}"><w:abstractNum w:abstractNumId="0"><w:multiLevelType w:val="hybridMultilevel"/><w:lvl w:ilvl="0"><w:start w:val="1"/><w:numFmt w:val="bullet"/><w:lvlText w:val="-"/><w:lvlJc w:val="left"/><w:pPr><w:ind w:left="720" w:hanging="360"/></w:pPr></w:lvl></w:abstractNum><w:num w:numId="1"><w:abstractNumId w:val="0"/></w:num></w:numbering>`;
+const entries = body => [
+    { name: '[Content_Types].xml', data: `<?xml version="1.0" encoding="UTF-8" standalone="yes"?><Types xmlns="http://schemas.openxmlformats.org/package/2006/content-types"><Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml"/><Default Extension="xml" ContentType="application/xml"/><Override PartName="/word/document.xml" ContentType="application/vnd.openxmlformats-officedocument.wordprocessingml.document.main+xml"/><Override PartName="/word/styles.xml" ContentType="application/vnd.openxmlformats-officedocument.wordprocessingml.styles+xml"/><Override PartName="/word/numbering.xml" ContentType="application/vnd.openxmlformats-officedocument.wordprocessingml.numbering+xml"/></Types>` },
+    { name: '_rels/.rels', data: `<?xml version="1.0" encoding="UTF-8" standalone="yes"?><Relationships xmlns="${PKG}"><Relationship Id="rId1" Type="${R}/officeDocument" Target="word/document.xml"/></Relationships>` },
+    { name: 'word/document.xml', data: wrap(body) },
+    { name: 'word/_rels/document.xml.rels', data: `<?xml version="1.0" encoding="UTF-8" standalone="yes"?><Relationships xmlns="${PKG}"><Relationship Id="rId1" Type="${R}/styles" Target="styles.xml"/><Relationship Id="rId2" Type="${R}/numbering" Target="numbering.xml"/></Relationships>` },
+    { name: 'word/styles.xml', data: styles },
+    { name: 'word/numbering.xml', data: numbering }
+];
+
+mkdirSync(OUT, { recursive: true });
+for (const def of defs) writeFileSync(join(OUT, `${def.name}.docx`), buildZip(entries(def.bodyXml)));
+writeFileSync(join(OUT, 'cases.json'), JSON.stringify(defs, null, 2) + '\n');
+console.log(`Wrote ${defs.length} cases to ${OUT}`);

package/scripts/lib/word-coverage-catalogue.mjs

@@ -50,6 +50,7 @@ function assertSyntheticClaim(testCase, structure) {
             || testCase.original.includes('\n') || testCase.modified.includes('\n'),
         'formatted-runs': () => /<(?:w:)?(?:b|i|u)(?:\s|\/|>)/.test(xml) || /(?:\*\*|\*|\+\+)/.test(testCase.modified),
         list: () => elementCount(xml, 'numPr') > 0
+            || (testCase.batchOperations || []).some(operation => operation?.type === 'list-format')
             || (testCase.operationOptions?.structuredContent === true
                 && /^\s*(?:[-+*]|\d+[.)])\s+/m.test(testCase.modified)),
         table: () => elementCount(xml, 'tbl') > 0
@@ -74,8 +75,12 @@ function assertSyntheticClaim(testCase, structure) {
 }
 
 function syntheticTask(testCase) {
-    if (testCase.operation?.type === 'list-change') return 'list-change';
-    if (testCase.operation?.type === 'table-reconciliation') return 'table-reconciliation';
+    const operationTypes = [testCase.operation, ...(testCase.batchOperations || [])]
+        .map(operation => operation?.type)
+        .filter(Boolean);
+    if (operationTypes.includes('list-change') || operationTypes.includes('list-format')) return 'list-change';
+    if (operationTypes.includes('table-reconciliation')) return 'table-reconciliation';
+    if (operationTypes.includes('paragraph-format') || operationTypes.includes('character-format')) return 'format';
     if (testCase.expectAtomicRollback) return 'mixed-batch';
     if (testCase.expectNoOp) return 'accept-reject';
     if (testCase.task.startsWith('apply-')) return 'format';

package/scripts/render-multilevel-bullet-visual.ps1

@@ -16,8 +16,8 @@ try {
         $pdfPath = Join-Path $resolvedOut "multilevel-bullet--$view.pdf"
         $doc = $word.Documents.OpenNoRepairDialog($resolvedDocx)
         try {
-            if ($view -eq 'acceptAll') { $doc.AcceptAllRevisions(); $exportItem = 0 }
-            elseif ($view -eq 'rejectAll') { $doc.RejectAllRevisions(); $exportItem = 0 }
+            if ($view -eq 'acceptAll') { $doc.Revisions.AcceptAll(); $exportItem = 0 }
+            elseif ($view -eq 'rejectAll') { $doc.Revisions.RejectAll(); $exportItem = 0 }
             else { $doc.ShowRevisions = $true; $doc.PrintRevisions = $true; $exportItem = 7 }
             $pages = $doc.ComputeStatistics(2)
             $doc.ExportAsFixedFormat($pdfPath, 17, $false, 0, 0, 1, 1, $exportItem, $true, $true, 0, $true, $true, $false)

package/scripts/word-com-corpus-visual-suite.ps1

@@ -60,8 +60,8 @@ function Export-View($word, [string]$docxPath, [string]$pdfPath, [string]$viewNa
     $document = $null
     try {
         $document = $word.Documents.OpenNoRepairDialog($docxPath)
-        if ($viewName -eq 'acceptAll') { $document.AcceptAllRevisions(); $exportItem = 0 }
-        elseif ($viewName -eq 'rejectAll') { $document.RejectAllRevisions(); $exportItem = 0 }
+        if ($viewName -eq 'acceptAll') { $document.Revisions.AcceptAll(); $exportItem = 0 }
+        elseif ($viewName -eq 'rejectAll') { $document.Revisions.RejectAll(); $exportItem = 0 }
         else { $document.ShowRevisions = $true; $document.PrintRevisions = $true; $exportItem = 7 }
         $pages = $document.ComputeStatistics(2)
         $document.ExportAsFixedFormat($pdfPath, 17, $false, 0, 0, 1, 1, $exportItem, $true, $true, 0, $true, $true, $false)