@ansonlai/docx-redline-js 0.8.0 → 0.8.1

package/node/index.d.ts

@@ -22,7 +22,7 @@ export class DocxDocument {
   preflight(operations: DocumentOperation[], author?: string, options?: StandaloneRunnerOptions): OperationPreflightResult;
   applyOperations(operations: DocumentOperation[], options?: DocxApplyOptions): Promise<DocxApplyResult>;
   resolveRevisions(action: 'accept' | 'reject', options: { author?: string; allAuthors?: boolean; validate?: boolean }): Promise<DocxApplyResult>;
-  deleteComments(options: { author?: string; allAuthors?: boolean; validate?: boolean }): Promise<DocxApplyResult>;
+  deleteComments(options: { author?: string; allAuthors?: boolean; ids?: Array<string | number>; validate?: boolean }): Promise<DocxApplyResult>;
   toBuffer(): Uint8Array;
 }
 export function computePackageRevisionToken(input: unknown): RevisionToken;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ansonlai/docx-redline-js",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Host-independent OOXML reconciliation engine for .docx manipulation with track changes",
   "license": "MIT",
   "type": "module",
@@ -108,6 +108,7 @@
     "smoke:word": "powershell -File scripts/word-com-smoke.ps1",
     "smoke:word:diff": "powershell -File scripts/word-com-differential.ps1",
     "test:word": "powershell -NoProfile -ExecutionPolicy Bypass -File scripts/word-com-suite.ps1",
+    "test:word:package": "node scripts/generate-word-package-integrity-cases.mjs && powershell -NoProfile -ExecutionPolicy Bypass -File scripts/word-com-package-integrity.ps1",
     "test:word:visual": "powershell -NoProfile -ExecutionPolicy Bypass -File scripts/word-com-visual-suite.ps1",
     "test:corpus:word": "powershell -NoProfile -ExecutionPolicy Bypass -File scripts/word-com-corpus-suite.ps1",
     "test:corpus:word:visual": "powershell -NoProfile -ExecutionPolicy Bypass -File scripts/word-com-corpus-visual-suite.ps1",

package/scripts/generate-word-comment-edge-case-fixture.ps1

@@ -0,0 +1,58 @@
+# Generates tests/fixtures/word-authored/comment-edge-cases.docx using real Word: comments on middle words,
+# across two paragraphs, inside a table cell, on text with another author's tracked insertion, and on a hyperlink.
+$ErrorActionPreference = 'Stop'
+$outDir = [System.IO.Path]::GetFullPath((Join-Path $PSScriptRoot '..\tests\fixtures\word-authored'))
+New-Item -ItemType Directory -Path $outDir -Force | Out-Null
+$path = [string](Join-Path $outDir 'comment-edge-cases.docx')
+if (Test-Path $path) { Remove-Item $path -Force }
+$word = New-Object -ComObject Word.Application
+$word.Visible = $false
+$word.DisplayAlerts = 0
+try {
+    $doc = $word.Documents.Add()
+    $doc.TrackRevisions = $false
+    $doc.Range(0, 0).Text = "Alpha beta gamma delta epsilon.`rThe counterparty proposed tenure.`rSpanning first paragraph.`rSpanning second paragraph.`rLink here: example.com now.`rTable follows."
+
+    # 1: comment on the middle words only
+    $r = $doc.Paragraphs.Item(1).Range
+    $mid = $doc.Range($r.Start + 6, $r.Start + 16)   # "beta gamma"
+    $c = $doc.Comments.Add($mid, "Middle words comment"); $c.Author = "Reviewer"
+
+    # 2: another author's tracked insertion inside a commented paragraph
+    $word.UserName = "Other Author"
+    $doc.TrackRevisions = $true
+    $p2 = $doc.Paragraphs.Item(2).Range
+    $ins = $doc.Range($p2.End - 8, $p2.End - 8)      # before "tenure."
+    $ins.InsertAfter("long-term ")
+    $doc.TrackRevisions = $false
+    $word.UserName = "Reviewer"
+    $c = $doc.Comments.Add($doc.Paragraphs.Item(2).Range, "Comment over a tracked insertion"); $c.Author = "Reviewer"
+
+    # 3: one comment spanning two paragraphs
+    $span = $doc.Range($doc.Paragraphs.Item(3).Range.Start, $doc.Paragraphs.Item(4).Range.End - 1)
+    $c = $doc.Comments.Add($span, "Spans two paragraphs"); $c.Author = "Reviewer"
+
+    # 4: comment on a hyperlink
+    $p5 = $doc.Paragraphs.Item(5).Range
+    $linkRange = $doc.Range($p5.Start + 11, $p5.Start + 22)   # "example.com"
+    $doc.Hyperlinks.Add($linkRange, "https://example.com") | Out-Null
+    $p5 = $doc.Paragraphs.Item(5).Range
+    $c = $doc.Comments.Add($p5, "On a paragraph with a hyperlink"); $c.Author = "Reviewer"
+
+    # 5: a table with a commented cell
+    $end = $doc.Content; $end.InsertParagraphAfter()
+    $tableRange = $doc.Paragraphs.Item($doc.Paragraphs.Count).Range
+    $table = $doc.Tables.Add($tableRange, 2, 2)
+    $table.Cell(1, 1).Range.Text = "Cell one"
+    $table.Cell(1, 2).Range.Text = "Cell two"
+    $table.Cell(2, 1).Range.Text = "Cell three"
+    $table.Cell(2, 2).Range.Text = "Cell four"
+    $cellText = $table.Cell(1, 2).Range
+    $cellRange = $doc.Range($cellText.Start, $cellText.End - 1)
+    $c = $doc.Comments.Add($cellRange, "Comment in a table cell"); $c.Author = "Reviewer"
+
+    [object]$p = $path; [object]$f = 16
+    $doc.SaveAs2([ref]$p, [ref]$f)
+    $doc.Close($false)
+    Write-Host "Saved $path"
+} finally { $word.Quit() }

package/scripts/generate-word-comment-thread-fixture.ps1

@@ -0,0 +1,36 @@
+# Generates tests/fixtures/word-authored/threaded-comments.docx using real Word.
+# The fixture is the oracle for package content types: tests compare our output against
+# what Word itself wrote, so the expected values are not derived from our own constants.
+$ErrorActionPreference = 'Stop'
+$outDir = Join-Path $PSScriptRoot '..\tests\fixtures\word-authored'
+New-Item -ItemType Directory -Path $outDir -Force | Out-Null
+$outPath = [System.IO.Path]::GetFullPath((Join-Path $outDir 'threaded-comments.docx'))
+if (Test-Path $outPath) { Remove-Item $outPath -Force }
+
+$word = New-Object -ComObject Word.Application
+$word.Visible = $false
+$word.DisplayAlerts = 0
+try {
+    $doc = $word.Documents.Add()
+    $doc.TrackRevisions = $false
+    $doc.Range(0, 0).Text = "The Supplier shall deliver the goods within thirty days.`rPayment is due upon receipt of invoice.`rThis agreement is governed by local law."
+
+    # Thread 1: comment with a reply (open)
+    $c1 = $doc.Comments.Add($doc.Paragraphs.Item(1).Range, "Can we shorten the delivery window?")
+    $c1.Author = "Counterparty"
+    $r1 = $c1.Replies.Add($c1.Scope, "We can offer twenty days.")
+    $r1.Author = "Internal"
+
+    # Thread 2: resolved comment
+    $c2 = $doc.Comments.Add($doc.Paragraphs.Item(2).Range, "Confirm payment terms.")
+    $c2.Author = "Counterparty"
+    $c2.Done = $true
+
+    [object]$path = $outPath
+    [object]$fmt = 16
+    $doc.SaveAs2([ref]$path, [ref]$fmt)
+    $doc.Close($false)
+    Write-Host "Saved $outPath"
+} finally {
+    $word.Quit()
+}

package/scripts/generate-word-header-footer-fixture.ps1

@@ -0,0 +1,32 @@
+# Generates tests/fixtures/word-authored/header-footer.docx using real Word: default + first-page
+# header/footer with a PAGE field, and a second section. Ground truth for header/footer part
+# content types, relationship types and sectPr references.
+$ErrorActionPreference = 'Stop'
+$outDir = Join-Path $PSScriptRoot '..\tests\fixtures\word-authored'
+New-Item -ItemType Directory -Path $outDir -Force | Out-Null
+$outPath = [System.IO.Path]::GetFullPath((Join-Path $outDir 'header-footer.docx'))
+if (Test-Path $outPath) { Remove-Item $outPath -Force }
+
+$word = New-Object -ComObject Word.Application
+$word.Visible = $false
+$word.DisplayAlerts = 0
+try {
+    $doc = $word.Documents.Add()
+    $doc.TrackRevisions = $false
+    $doc.Range(0, 0).Text = "Body paragraph one.`rBody paragraph two."
+    $sec = $doc.Sections.Item(1)
+    $sec.PageSetup.DifferentFirstPageHeaderFooter = -1
+    $sec.Headers.Item(1).Range.Text = "CONFIDENTIAL - Acme Master Agreement"      # wdHeaderFooterPrimary
+    $sec.Headers.Item(2).Range.Text = "First page header"                          # wdHeaderFooterFirstPage
+    $sec.Footers.Item(1).Range.Text = "Page "
+    $footerRange = $sec.Footers.Item(1).Range
+    $footerRange.Collapse(0)
+    $doc.Fields.Add($footerRange, 33) | Out-Null                                   # wdFieldPage
+    $sec.Footers.Item(2).Range.Text = "First page footer"
+
+    [object]$path = $outPath
+    [object]$fmt = 16
+    $doc.SaveAs2([ref]$path, [ref]$fmt)
+    $doc.Close($false)
+    Write-Host "Saved $outPath"
+} finally { $word.Quit() }

package/scripts/generate-word-multi-section-fixture.ps1

@@ -0,0 +1,27 @@
+# Generates tests/fixtures/word-authored/multi-section-headers.docx using real Word: two sections where the second
+# section has its own header (not linked to previous) but shares the first section's footer.
+$ErrorActionPreference = 'Stop'
+$outDir = [System.IO.Path]::GetFullPath((Join-Path $PSScriptRoot '..\tests\fixtures\word-authored'))
+New-Item -ItemType Directory -Path $outDir -Force | Out-Null
+$path = [string](Join-Path $outDir 'multi-section-headers.docx')
+if (Test-Path $path) { Remove-Item $path -Force }
+$word = New-Object -ComObject Word.Application
+$word.Visible = $false
+$word.DisplayAlerts = 0
+try {
+    $doc = $word.Documents.Add()
+    $doc.Range(0, 0).Text = "Section one body."
+    $doc.Sections.Item(1).Headers.Item(1).Range.Text = "Section one header"
+    $doc.Sections.Item(1).Footers.Item(1).Range.Text = "Shared footer"
+    $end = $doc.Content
+    $end.InsertParagraphAfter()
+    $doc.Paragraphs.Item($doc.Paragraphs.Count).Range.InsertBreak(2)   # wdSectionBreakNextPage
+    $doc.Paragraphs.Item($doc.Paragraphs.Count).Range.InsertBefore("Section two body.")
+    $sec2 = $doc.Sections.Item(2)
+    $sec2.Headers.Item(1).LinkToPrevious = $false
+    $sec2.Headers.Item(1).Range.Text = "Section two header"
+    [object]$p = $path; [object]$f = 16
+    $doc.SaveAs2([ref]$p, [ref]$f)
+    $doc.Close($false)
+    Write-Host "Saved $path"
+} finally { $word.Quit() }

package/scripts/generate-word-package-integrity-cases.mjs

@@ -0,0 +1,141 @@
+/**
+ * Builds .docx outputs for the Word open-check (scripts/word-com-package-integrity.ps1).
+ *
+ * Starting from a document Word itself saved (tests/fixtures/word-authored/threaded-comments.docx),
+ * runs each write path of the package and writes the result to tmp/word-package-integrity/.
+ * `expect-open` cases must open in Word with repair disabled. `expect-fail-*` cases are controls
+ * that reproduce the original bug and prove the Word check can actually detect it.
+ */
+import { mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { openDocx } from '../node/index.js';
+import { unzipDocx, zipDocx } from '../document/zip-archive.js';
+
+const root = join(dirname(fileURLToPath(import.meta.url)), '..');
+const outDir = join(root, 'tmp', 'word-package-integrity');
+const fixture = readFileSync(join(root, 'tests', 'fixtures', 'word-authored', 'threaded-comments.docx'));
+const WORD_TYPE = 'application/vnd.openxmlformats-officedocument.wordprocessingml.commentsExtended+xml';
+const BAD_TYPE = 'application/vnd.ms-word.commentsExtended+xml';
+
+rmSync(outDir, { recursive: true, force: true });
+mkdirSync(outDir, { recursive: true });
+
+const write = (name, bytes) => writeFileSync(join(outDir, name), Buffer.from(bytes));
+
+function withContentType(bytes, from, to) {
+    const entries = unzipDocx(bytes);
+    const xml = new TextDecoder().decode(entries.get('[Content_Types].xml'));
+    if (!xml.includes(from)) throw new Error(`fixture does not contain ${from}`);
+    entries.set('[Content_Types].xml', new TextEncoder().encode(xml.replace(from, to)));
+    return zipDocx(entries);
+}
+
+// `expectDone`: the resolved state Word must report for each comment (document order). Checked through Word's
+// own object model, so it proves Word reads our threading parts the way we wrote them.
+// `expect` is either an array of Done states or { commentsDone?, ancestors?, texts? } where `ancestors[i]` is the
+// 1-based index of comment i's thread parent (0 = top level), read from Word's Comment.Ancestor.
+async function run(name, source, fn, expect) {
+    const doc = openDocx(source);
+    const result = await fn(doc);
+    if (result.status !== 'ok' || !result.written) throw new Error(`${name}: ${result.error?.message || 'not written'}`);
+    write(`expect-open-${name}.docx`, result.toBuffer());
+    if (expect) writeFileSync(join(outDir, `expect-open-${name}.expect.json`), JSON.stringify(Array.isArray(expect) ? { commentsDone: expect } : expect));
+}
+const load = name => readFileSync(join(root, 'tests', 'fixtures', 'word-authored', name));
+
+const edit = doc => doc.applyOperations([{ type: 'replace', target: { exactText: 'This agreement is governed by local law.' }, modified: 'This agreement is governed by New York law.' }], { author: 'Agent', atomic: true });
+const reply = doc => doc.applyOperations([{ type: 'comment_reply', parentCommentId: 0, commentContent: 'Agreed, twenty days works.', author: 'Agent' }], { author: 'Agent', atomic: true });
+
+write('expect-open-00-word-original.docx', fixture);
+await run('01-edit-unrelated-to-comments', fixture, edit);
+await run('02-reply', fixture, reply, [false, false, false, true]);
+await run('03-add-comment', fixture, doc => doc.applyOperations([{ type: 'comment', target: { exactText: 'This agreement is governed by local law.' }, commentContent: 'Check governing law.', author: 'Agent' }], { author: 'Agent', atomic: true }));
+await run('04-edit-then-accept', fixture, async doc => { await edit(doc); return doc.resolveRevisions('accept', { allAuthors: true }); });
+await run('05-delete-comments', fixture, doc => doc.deleteComments({ author: 'Counterparty' }));
+
+await run('07-resolve-thread-via-reply', fixture, doc => doc.resolveComment(1), [true, true, true]);
+await run('08-reopen-word-resolved-thread', load('resolved-threads.docx'), doc => doc.resolveComment(3, { resolved: false }), [true, true, true, false, false]);
+await run('09-reply-to-multi-paragraph-comment', load('multi-paragraph-thread.docx'), doc => doc.applyOperations([{ type: 'comment_reply', parentCommentId: 0, commentContent: 'Another reply', author: 'Agent' }], { author: 'Agent', atomic: true }), [false, false, false]);
+await run('10-resolve-multi-paragraph-thread', load('multi-paragraph-thread.docx'), doc => doc.resolveComment(0), [true, true]);
+
+// --- additional Word probes: scenarios likelier to expose bugs than the happy paths above ---
+const stripThreadingParts = bytes => {
+    const entries = unzipDocx(bytes);
+    for (const name of ['word/commentsExtended.xml', 'word/commentsIds.xml', 'word/commentsExtensible.xml']) entries.delete(name);
+    const dec = new TextDecoder(); const enc = new TextEncoder();
+    entries.set('[Content_Types].xml', enc.encode(dec.decode(entries.get('[Content_Types].xml')).replace(/<Override PartName="\/word\/comments(Extended|Ids|Extensible)\.xml"[^>]*\/>/g, '')));
+    entries.set('word/_rels/document.xml.rels', enc.encode(dec.decode(entries.get('word/_rels/document.xml.rels')).replace(/<Relationship [^>]*Target="comments(Extended|Ids|Extensible)\.xml"[^>]*\/>/g, '')));
+    return zipDocx(entries);
+};
+const replyTo = (parent, text) => ({ type: 'comment_reply', parentCommentId: parent, commentContent: text, author: 'Agent' });
+const apply = (doc, ops) => doc.applyOperations(ops, { author: 'Agent', atomic: true });
+
+await run('11-reply-to-a-reply', fixture, doc => apply(doc, [replyTo(1, 'Reply to the reply')]), {ancestors: [0, 1, 1, 0], commentsDone: [false, false, false, true]});
+await run('12-two-replies-same-root', fixture, doc => apply(doc, [replyTo(0, 'First new'), replyTo(0, 'Second new')]), {ancestors: [0, 1, 1, 1, 0], commentsDone: [false, false, false, false, true]});
+await run('13-comment-then-reply-to-own-comment', fixture, async doc => {
+    const added = await apply(doc, [{ type: 'comment', target: { exactText: 'This agreement is governed by local law.' }, commentContent: 'Governing law?', author: 'Agent' }]);
+    if (added.status !== 'ok') return added;
+    return apply(doc, [replyTo(3, 'Reply to my own comment')]);
+}, {ancestors: [0, 1, 0, 0, 4], commentsDone: [false, false, true, false, false]});
+await run('14-edit-inside-commented-paragraph', fixture, doc => apply(doc, [{ type: 'replace', target: { exactText: 'The Supplier shall deliver the goods within thirty days.' }, modified: 'The Supplier shall deliver the goods within twenty days.' }]), {ancestors: [0, 1, 0], commentsDone: [false, false, true]});
+await run('15-edit-and-reply-in-one-batch', fixture, doc => apply(doc, [replyTo(0, 'Batch reply'), { type: 'replace', target: { exactText: 'Payment is due upon receipt of invoice.' }, modified: 'Payment is due within thirty days of invoice.' }]), {ancestors: [0, 1, 1, 0], commentsDone: [false, false, false, true]});
+await run('16-reply-accept-resolve', fixture, async doc => {
+    await apply(doc, [replyTo(0, 'Then accept')]);
+    await doc.resolveRevisions('accept', { allAuthors: true });
+    return doc.resolveComment(0);
+}, {ancestors: [0, 1, 1, 0], commentsDone: [true, true, true, true]});
+await run('17-legacy-no-threading-parts-reply', stripThreadingParts(fixture), doc => apply(doc, [replyTo(0, 'Reply on legacy file')]), {ancestors: [0, 1, 0, 0], commentsDone: [false, false, false, false]});
+await run('18-legacy-no-threading-parts-resolve', stripThreadingParts(fixture), doc => doc.resolveComment(0), {ancestors: [0, 0, 0], commentsDone: [true, false, false]});
+await run('19-edit-commented-text-multi-paragraph', load('multi-paragraph-thread.docx'), doc => apply(doc, [{ type: 'replace', target: { exactText: 'Alpha paragraph.' }, modified: 'Alpha paragraph, revised.' }]), {ancestors: [0, 1], commentsDone: [false, false]});
+await run('20-delete-reply-only', fixture, doc => doc.deleteComments({ author: 'Internal' }), {ancestors: [0, 0], commentsDone: [false, true]});
+
+// Batch comment_resolve and delete-by-id (fixture: 0 root + 1 reply open, 2 resolved).
+const resolveOp = (commentId, resolved = true) => ({ type: 'comment_resolve', commentId, resolved });
+await run('40-batch-resolve-thread', fixture, doc => apply(doc, [resolveOp(0)]), { ancestors: [0, 1, 0], commentsDone: [true, true, true] });
+await run('41-batch-reply-then-resolve', fixture, doc => apply(doc, [replyTo(0, 'Closing this out'), resolveOp(0)]), { ancestors: [0, 1, 1, 0], commentsDone: [true, true, true, true] });
+await run('42-reply-to-resolved-thread', fixture, doc => apply(doc, [replyTo(2, 'Late reply')]), { ancestors: [0, 1, 0, 3], commentsDone: [false, false, true, true] });
+await run('43-batch-reopen', fixture, doc => apply(doc, [resolveOp(2, false)]), { ancestors: [0, 1, 0], commentsDone: [false, false, false] });
+await run('44-delete-root-by-id', fixture, doc => doc.deleteComments({ ids: [0] }), { ancestors: [0], commentsDone: [true] });
+await run('45-delete-reply-by-id', fixture, doc => doc.deleteComments({ ids: [1] }), { ancestors: [0, 0], commentsDone: [false, true] });
+
+// Comment shapes Word wrote (comment-edge-cases.docx): every edit must leave Word seeing the same comments.
+const edgeCases = load('comment-edge-cases.docx');
+const edgeText = doc => doc.inspect().paragraphs.map(p => p.text);
+const edgeReplace = (index, to) => doc => apply(doc, [{ type: 'replace', target: { exactText: edgeText(doc)[index - 1] }, modified: to }]);
+const five = { ancestors: [0, 0, 0, 0, 0], commentsDone: [false, false, false, false, false] };
+await run('21-edge-edit-middle-words', edgeCases, edgeReplace(1, 'Alpha BETA GAMMA delta epsilon.'), five);
+await run('22-edge-edit-over-tracked-insertion', edgeCases, doc => doc.applyOperations([{ type: 'replace', target: { exactText: edgeText(doc)[1] }, modified: 'The counterparty proposed short-term tenure.' }], { author: 'Agent', atomic: true, existingRevisions: 'slice-cross-author' }), five);
+await run('23-edge-edit-spanning-comment-paragraph', edgeCases, edgeReplace(4, 'Spanning SECOND paragraph.'), five);
+await run('24-edge-edit-hyperlink-paragraph', edgeCases, edgeReplace(5, 'Link here: example.org now.'), five);
+await run('25-edge-edit-table-cell', edgeCases, edgeReplace(8, 'Cell TWO'), five);
+await run('26-edge-reply-to-spanning-comment', edgeCases, doc => apply(doc, [replyTo(3, 'Reply on a two-paragraph range')]), { ancestors: [0, 0, 0, 3, 0, 0], commentsDone: [false, false, false, false, false, false] });
+await run('27-edge-reply-in-table-cell', edgeCases, doc => apply(doc, [replyTo(5, 'Reply in a cell')]), { ancestors: [0, 0, 0, 0, 0, 5], commentsDone: [false, false, false, false, false, false] });
+
+// --- Headers and footers (header-footer.docx was saved by Word; see scripts/generate-word-header-footer-fixture.ps1) ---
+const headerFooter = load('header-footer.docx');
+const HDR = { kind: 'header', type: 'default' };
+const HDR_TEXT = 'CONFIDENTIAL - Acme Master Agreement';
+const hdrEdit = { type: 'replace', part: HDR, target: { exactText: HDR_TEXT }, modified: 'STRICTLY CONFIDENTIAL - Acme Master Agreement' };
+await run('30-header-edit', headerFooter, doc => apply(doc, [hdrEdit]), { headersFooters: [{ section: 1, kind: 'header', slot: 'primary', revisions: 1 }], bodyRevisions: 0 });
+await run('31-footer-edit-around-page-field', headerFooter, doc => apply(doc, [{ type: 'replace', part: { kind: 'footer' }, target: { exactText: 'Page 1' }, modified: 'Sheet 1' }]), { headersFooters: [{ section: 1, kind: 'footer', slot: 'primary', revisions: 2, fields: 1 }] });
+await run('32-body-header-and-first-page-in-one-batch', headerFooter, doc => apply(doc, [
+    { type: 'replace', target: { exactText: 'Body paragraph one.' }, modified: 'Body paragraph ONE.' },
+    hdrEdit,
+    { type: 'replace', part: 'word/header3.xml', target: { exactText: 'First page header' }, modified: 'First page header (rev)' }
+]), { headersFooters: [{ section: 1, kind: 'header', slot: 'primary', revisions: 1 }, { section: 1, kind: 'header', slot: 'first', revisions: 1 }], bodyRevisions: 2 });
+await run('33-header-edit-then-accept', headerFooter, async doc => { await apply(doc, [hdrEdit]); return doc.resolveRevisions('accept', { allAuthors: true }); }, { headersFooters: [{ section: 1, kind: 'header', slot: 'primary', revisions: 0, text: 'STRICTLY CONFIDENTIAL - Acme Master Agreement' }, { section: 1, kind: 'footer', slot: 'primary', fields: 1 }] });
+await run('34-header-edit-then-reject', headerFooter, async doc => { await apply(doc, [hdrEdit]); return doc.resolveRevisions('reject', { allAuthors: true }); }, { headersFooters: [{ section: 1, kind: 'header', slot: 'primary', revisions: 0, text: HDR_TEXT }] });
+
+const multiSection = load('multi-section-headers.docx');
+await run('35-multi-section-second-header', multiSection, doc => apply(doc, [{ type: 'replace', part: { kind: 'header', section: 1 }, target: { exactText: 'Section two header' }, modified: 'Section two header (rev)' }]), { headersFooters: [{ section: 1, kind: 'header', slot: 'primary', revisions: 0 }, { section: 2, kind: 'header', slot: 'primary', revisions: 1 }] });
+await run('36-multi-section-inherited-footer', multiSection, doc => apply(doc, [{ type: 'replace', part: { kind: 'footer', section: 1 }, target: { exactText: 'Shared footer' }, modified: 'Shared footer (rev)' }]), { headersFooters: [{ section: 1, kind: 'footer', slot: 'primary', revisions: 1 }, { section: 2, kind: 'footer', slot: 'primary', revisions: 1 }] });
+
+// Files damaged by earlier versions of this package: the next save must repair them.
+const damaged = withContentType(fixture, WORD_TYPE, BAD_TYPE);
+await run('06-repair-damaged-on-edit', damaged, edit);
+
+// Controls: reproduce the original bug. Word must refuse these, or this check proves nothing.
+write('expect-fail-control-bad-extended-type.docx', damaged);
+
+console.log(`Wrote cases to ${outDir}`);

package/scripts/generate-word-thread-state-fixtures.ps1

@@ -0,0 +1,50 @@
+# Generates real-Word fixtures for comment thread behavior (tests/fixtures/word-authored/):
+#   multi-paragraph-thread.docx  a three-paragraph comment with one reply. Word keys commentsExtended /
+#                                commentsIds on the LAST paragraph of the comment.
+#   resolved-threads.docx        thread 1: root + two replies, resolved by setting Done on the ROOT.
+#                                thread 2: root + reply, resolved by setting Done on the REPLY.
+#                                Word marks the WHOLE thread done in both cases.
+#   reply-to-resolved.docx       a reply added to an already-resolved thread. Word writes the reply done too.
+$ErrorActionPreference = 'Stop'
+$outDir = [System.IO.Path]::GetFullPath((Join-Path $PSScriptRoot '..\tests\fixtures\word-authored'))
+New-Item -ItemType Directory -Path $outDir -Force | Out-Null
+$word = New-Object -ComObject Word.Application
+$word.Visible = $false
+$word.DisplayAlerts = 0
+
+function Save-Doc($doc, $name) {
+    $path = [string](Join-Path $outDir $name)
+    if (Test-Path $path) { Remove-Item $path -Force }
+    [object]$p = $path; [object]$f = 16
+    $doc.SaveAs2([ref]$p, [ref]$f)
+    $doc.Close($false)
+    Write-Host "Saved $path"
+}
+
+try {
+    $doc = $word.Documents.Add()
+    $doc.Range(0, 0).Text = "Alpha paragraph."
+    $c = $doc.Comments.Add($doc.Paragraphs.Item(1).Range, "first para of comment`rsecond para of comment`rthird para")
+    $c.Author = "Counterparty"
+    $r = $c.Replies.Add($c.Scope, "a reply")
+    $r.Author = "Internal"
+    Save-Doc $doc 'multi-paragraph-thread.docx'
+
+    $doc = $word.Documents.Add()
+    $doc.Range(0, 0).Text = "Alpha paragraph.`rBeta paragraph."
+    $c1 = $doc.Comments.Add($doc.Paragraphs.Item(1).Range, "root one"); $c1.Author = "A"
+    $r1 = $c1.Replies.Add($c1.Scope, "reply one"); $r1.Author = "B"
+    $r2 = $c1.Replies.Add($c1.Scope, "reply two"); $r2.Author = "C"
+    $c1.Done = $true
+    $c2 = $doc.Comments.Add($doc.Paragraphs.Item(2).Range, "root two"); $c2.Author = "A"
+    $rr = $c2.Replies.Add($c2.Scope, "reply to two"); $rr.Author = "B"
+    $rr.Done = $true
+    Save-Doc $doc 'resolved-threads.docx'
+
+    $doc = $word.Documents.Add()
+    $doc.Range(0, 0).Text = "Alpha paragraph."
+    $c = $doc.Comments.Add($doc.Paragraphs.Item(1).Range, "root"); $c.Author = "A"
+    $c.Done = $true
+    $r = $c.Replies.Add($c.Scope, "late reply"); $r.Author = "B"
+    Save-Doc $doc 'reply-to-resolved.docx'
+} finally { $word.Quit() }

package/scripts/word-com-package-integrity.ps1

@@ -0,0 +1,98 @@
+<#
+Opens every .docx in tmp/word-package-integrity (built by generate-word-package-integrity-cases.mjs)
+in real Word with OpenAndRepair disabled. Word throws on a package it considers unreadable, which is
+exactly what the "unreadable content" repair prompt reports.
+
+  expect-open-*  must open cleanly.
+  expect-fail-*  are controls that reproduce a known-bad package and must be REFUSED.
+                 If Word opens a control, this check cannot detect the bug and fails.
+#>
+param([string]$Dir = (Join-Path $PSScriptRoot '..\tmp\word-package-integrity'))
+$ErrorActionPreference = 'Stop'
+$Dir = [System.IO.Path]::GetFullPath($Dir)
+$files = Get-ChildItem -LiteralPath $Dir -Filter *.docx | Sort-Object Name
+if (-not $files) { Write-Error "No cases in $Dir. Run: node scripts/generate-word-package-integrity-cases.mjs"; exit 2 }
+
+$word = New-Object -ComObject Word.Application
+$word.Visible = $false
+$word.DisplayAlerts = 0
+$missing = [Type]::Missing
+$failures = 0
+try {
+    foreach ($file in $files) {
+        $opened = $false; $detail = ''
+        $doc = $null
+        try {
+            # FileName, ConfirmConversions, ReadOnly, AddToRecentFiles, PasswordDocument, PasswordTemplate, Revert,
+            # WritePasswordDocument, WritePasswordTemplate, Format, Encoding, Visible, OpenAndRepair
+            $doc = $word.Documents.Open([string]$file.FullName, $false, $true, $false, $missing, $missing, $false, $missing, $missing, $missing, $missing, $false, $false)
+            $opened = $true
+            $detail = "comments=$($doc.Comments.Count)"
+            $expectFile = [System.IO.Path]::ChangeExtension($file.FullName, '.expect.json')
+            if (Test-Path -LiteralPath $expectFile) {
+                $expect = Get-Content -LiteralPath $expectFile -Raw | ConvertFrom-Json
+                $n = $doc.Comments.Count
+                $checks = @{}
+                if ($null -ne $expect.commentsDone) {
+                    $actual = @(); for ($i = 1; $i -le $n; $i++) { $actual += [bool]$doc.Comments.Item($i).Done }
+                    $checks['done'] = @(($actual -join ','), (@($expect.commentsDone) -join ','))
+                }
+                if ($null -ne $expect.ancestors) {
+                    $actual = @()
+                    for ($i = 1; $i -le $n; $i++) {
+                        $anc = $doc.Comments.Item($i).Ancestor
+                        $idx = 0
+                        if ($anc) { for ($j = 1; $j -le $n; $j++) { $c = $doc.Comments.Item($j); if ($c.Author -eq $anc.Author -and $c.Range.Text -eq $anc.Range.Text) { $idx = $j; break } } }
+                        $actual += $idx
+                    }
+                    $checks['ancestors'] = @(($actual -join ','), (@($expect.ancestors) -join ','))
+                }
+                if ($null -ne $expect.texts) {
+                    $actual = @(); for ($i = 1; $i -le $n; $i++) { $actual += $doc.Comments.Item($i).Range.Text }
+                    $checks['texts'] = @(($actual -join '|'), (@($expect.texts) -join '|'))
+                }
+                if ($expect.probe) {
+                    $rows = @()
+                    for ($i = 1; $i -le $n; $i++) {
+                        $c = $doc.Comments.Item($i); $anc = $c.Ancestor; $idx = 0
+                        if ($anc) { for ($j = 1; $j -le $n; $j++) { $o = $doc.Comments.Item($j); if ($o.Author -eq $anc.Author -and $o.Range.Text -eq $anc.Range.Text) { $idx = $j; break } } }
+                        $rows += "#$i[$($c.Author): $($c.Range.Text) | anc=$idx done=$([bool]$c.Done) scope='$($c.Scope.Text)']"
+                    }
+                    $detail += " PROBE " + ($rows -join ' ; ')
+                }
+                # headersFooters: [{ section, kind: header|footer, slot: primary|first|even, revisions?, fields?, text? }]
+                # read from Word's own Headers/Footers objects; text is compared only when given.
+                if ($null -ne $expect.headersFooters) {
+                    $slots = @{ primary = 1; first = 2; even = 3 }
+                    $actualParts = @(); $expectedParts = @()
+                    foreach ($want in @($expect.headersFooters)) {
+                        $section = $doc.Sections.Item([int]$want.section)
+                        $hf = if ($want.kind -eq 'header') { $section.Headers.Item($slots[[string]$want.slot]) } else { $section.Footers.Item($slots[[string]$want.slot]) }
+                        $range = $hf.Range
+                        $label = "$($want.kind)/$($want.slot)"
+                        $got = @(); $exp = @()
+                        if ($null -ne $want.revisions) { $got += "rev=$($range.Revisions.Count)"; $exp += "rev=$($want.revisions)" }
+                        if ($null -ne $want.fields) { $got += "fields=$($range.Fields.Count)"; $exp += "fields=$($want.fields)" }
+                        if ($null -ne $want.text) { $got += "text=$(($range.Text -replace '[\r\a]+$', ''))"; $exp += "text=$($want.text)" }
+                        $actualParts += "$label{$($got -join ',')}"; $expectedParts += "$label{$($exp -join ',')}"
+                    }
+                    $checks['headersFooters'] = @(($actualParts -join ' '), ($expectedParts -join ' '))
+                }
+                if ($null -ne $expect.bodyRevisions) {
+                    $checks['bodyRevisions'] = @([string]$doc.Revisions.Count, [string]$expect.bodyRevisions)
+                }
+                foreach ($key in $checks.Keys) {
+                    $detail += " $key=[$($checks[$key][0])]"
+                    if ($checks[$key][0] -ne $checks[$key][1]) { $opened = $false; $detail += " EXPECTED $key=[$($checks[$key][1])]" }
+                }
+            }
+        } catch { $detail = $_.Exception.Message.Trim() }
+        finally { if ($doc) { $doc.Close($false) | Out-Null } }
+
+        $expectOpen = $file.Name.StartsWith('expect-open-')
+        if ($opened -eq $expectOpen) { Write-Host "PASS  $($file.Name)  ($(if ($opened) { 'opened' } else { 'refused as expected' }); $detail)" }
+        else { Write-Host "FAIL  $($file.Name)  ($(if ($opened) { 'opened but should be refused' } else { 'refused but should open' }); $detail)"; $failures++ }
+    }
+} finally { $word.Quit() | Out-Null }
+if ($failures) { Write-Host "$failures case(s) failed"; exit 1 }
+Write-Host "All Word package-integrity cases passed"