theseus/addon-build/docx-editor/test/roundtrip.mjs

96 lines
4.9 KiB
JavaScript
Raw Normal View History

feat(docx-editor): edit Word documents without quietly eating what Word put in them A .docx editor is easy to write badly: read the file into HTML, let someone edit it, write a fresh document back, and hand them a file that lost its headers, its page size and half its formatting without ever saying so. Three things keep this one honest. The reader doesn't use mammoth's HTML. mammoth's converter is deliberately semantic, and HTML has nowhere to put a run's colour or a paragraph's line spacing, so it drops them — and those are controls this editor puts in the ribbon. Taking its parsed document model instead means what the ribbon offers is what the file can actually carry. Six properties mammoth's model didn't keep are added by build-time patches, each asserting its anchor so an upgrade that moves the code fails the build rather than shipping a lossy reader. The writer rebuilds the body but carries the rest of the package across: headers, footers, footnotes, endnotes, the document's own style catalogue, its theme and its page setup, with relationship ids and content types re-wired. Word features the editor can't model are still lost, so they are detected when the file opens and named in a banner before anyone edits. Tracked changes get their own gate. mammoth renders insertions as ordinary text and drops deletions, so saving would accept every pending revision without Word ever asking. Such a document opens read-only until the user says that is what they want. Verified over 66 real documents: 65 round-trip with an identical model and a structurally valid package, the one exception being a 7 MB WMF picture, which no browser can display and the writer cannot emit. Also driven end to end through a real Theseus over CDP — sidebar, ribbon, typing, save, reopen.
2026-09-20 20:46:29 +02:00
// The round-trip test: fixture.docx -> editor model -> .docx -> editor model,
// then diff the two models. Anything that shows up in the diff is something a
// user would lose by opening a document and pressing Save.
//
// node test/roundtrip.mjs [some-other.docx]
import { writeFileSync, readFileSync, existsSync } from "node:fs";
import { fileURLToPath } from "node:url";
import path from "node:path";
import { loadAddonLibs, summarise, diff } from "./harness.mjs";
import { buildFixture } from "./fixture.mjs";
const here = path.dirname(fileURLToPath(import.meta.url));
const DocxEditor = loadAddonLibs();
const schema = DocxEditor.schema.build();
const input = process.argv[2];
const original = input && existsSync(input)
? new Uint8Array(readFileSync(input))
: new Uint8Array(await buildFixture());
console.log(`input: ${input || "generated fixture"} (${(original.length / 1024).toFixed(1)} KB)\n`);
// --- pass 1: read ---------------------------------------------------------
const first = await DocxEditor.read.docxToDoc(original, schema);
console.log(`read: ${first.doc.childCount} top-level blocks`);
if (first.report.features.length) {
for (const f of first.report.features) {
console.log(` [${f.level}] ${f.label}${f.note ? " — " + f.note : ""}`);
}
}
if (first.warnings.length) console.log(` warnings: ${first.warnings.join(", ")}`);
// --- pass 2: write --------------------------------------------------------
const saved = await DocxEditor.write.docToDocx(first.doc, {
originalBytes: original,
setup: first.setup,
meta: first.meta,
});
console.log(`\nwrote: ${(saved.bytes.length / 1024).toFixed(1)} KB`);
if (saved.carried.length) console.log(` carried over: ${saved.carried.join(", ")}`);
if (saved.warnings.length) console.log(` warnings: ${saved.warnings.join(", ")}`);
const outPath = path.join(here, "roundtrip-out.docx");
writeFileSync(outPath, Buffer.from(saved.bytes));
// --- pass 3: read back ----------------------------------------------------
const second = await DocxEditor.read.docxToDoc(saved.bytes, schema);
console.log(`re-read: ${second.doc.childCount} top-level blocks`);
// --- package sanity -------------------------------------------------------
const zip = await DocxEditor.pkg.loadZip(saved.bytes);
const names = Object.keys(zip.files).sort();
const required = ["[Content_Types].xml", "_rels/.rels", "word/document.xml",
"word/_rels/document.xml.rels", "word/styles.xml"];
const missing = required.filter((r) => !names.includes(r));
let xmlErrors = [];
for (const name of names) {
if (!name.endsWith(".xml") && !name.endsWith(".rels")) continue;
try { DocxEditor.pkg.parseXml(await zip.file(name).async("string")); }
catch (e) { xmlErrors.push(`${name}: ${e.message}`); }
}
// Every r:id the document references must exist in its rels part.
const relsDoc = DocxEditor.pkg.parseXml(await zip.file("word/_rels/document.xml.rels").async("string"));
const declared = new Set(Array.from(relsDoc.getElementsByTagName("*"))
.filter((e) => e.localName === "Relationship").map((e) => e.getAttribute("Id")));
const docXml = await zip.file("word/document.xml").async("string");
const referenced = [...docXml.matchAll(/r:(?:id|embed|link)="([^"]+)"/g)].map((m) => m[1]);
const danglingRels = [...new Set(referenced)].filter((id) => !declared.has(id));
// Every part declared in the rels must actually be in the package.
const danglingParts = Array.from(relsDoc.getElementsByTagName("*"))
.filter((e) => e.localName === "Relationship" && e.getAttribute("TargetMode") !== "External")
.map((e) => "word/" + (e.getAttribute("Target") || "").replace(/^\.\//, ""))
.filter((p) => !p.includes("://") && !names.includes(p));
console.log("\npackage sanity");
console.log(` parts: ${names.length}`);
console.log(` missing required: ${missing.length ? missing.join(", ") : "none"}`);
console.log(` malformed xml: ${xmlErrors.length ? xmlErrors.join("; ") : "none"}`);
console.log(` dangling r:ids: ${danglingRels.length ? danglingRels.join(", ") : "none"}`);
console.log(` rels pointing at missing parts: ${danglingParts.length ? danglingParts.join(", ") : "none"}`);
for (const want of ["word/header1.xml", "word/footer1.xml", "word/footnotes.xml"]) {
if (names.includes(want)) console.log(` kept ${want}`);
}
// --- the diff -------------------------------------------------------------
const a = summarise(first.doc);
const b = summarise(second.doc);
const d = diff(a, b);
console.log(`\nround-trip diff: ${d.length ? d.length + " difference(s)" : "clean"}`);
for (const line of d.slice(0, 60)) console.log(" " + line);
if (d.length > 60) console.log(` … and ${d.length - 60} more`);
writeFileSync(path.join(here, "roundtrip-a.json"), JSON.stringify(a, null, 1));
writeFileSync(path.join(here, "roundtrip-b.json"), JSON.stringify(b, null, 1));
const fatal = missing.length || xmlErrors.length || danglingRels.length || danglingParts.length;
process.exit(fatal ? 2 : d.length ? 1 : 0);