@createiq/htmldiff 1.2.0-beta.1 → 1.2.0-beta.10

package/test/HtmlDiff.threeWay.spec.ts

@@ -1,6 +1,6 @@
-import { describe, expect, it } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
-import HtmlDiff from '../src/HtmlDiff'
+import HtmlDiff, { WORD_ALIGNED_OPTIONS } from '../src/HtmlDiff'
 
 /**
  * Three-way diff tests under the genesis-spine model.
@@ -43,9 +43,13 @@ describe('HtmlDiff.executeThreeWay (genesis-spine)', () => {
 
     it('CP and Me each change the same word differently', () => {
       // Genesis: "Hello world". CP made "Hello cruel world", Me made "Hello brave world".
-      // Disagreement — show both authors' insertions.
+      // Both inserted at the same boundary. Under the intent-reading
+      // model, the reviewer sees CP's proposal relative to Me's
+      // current content: "CP wants `cruel` where Me has `brave`" —
+      // ins-cp `cruel` + del-cp `brave`. Reads as a substitution
+      // intent, which is what a legal reviewer needs to act on.
       expect(HtmlDiff.executeThreeWay('Hello world', 'Hello cruel world', 'Hello brave world')).toBe(
-        "Hello <ins class='diffins cp' data-author='cp'>cruel </ins><ins class='diffins me' data-author='me'>brave </ins>world"
+        "Hello <ins class='diffins cp' data-author='cp'>cruel</ins><del class='diffdel cp' data-author='cp'>brave</del> world"
       )
     })
 
@@ -71,6 +75,77 @@ describe('HtmlDiff.executeThreeWay (genesis-spine)', () => {
       )
     })
 
+    it("CP accepted Me's addition with a word removed — del-cp on the removed word", () => {
+      // Real flow on the live preview:
+      //   - Me appends "And I add more things here"
+      //   - CP "accepts" Me's addition but deletes the word "things"
+      // Intent reading: CP wants "things" gone. del-cp on the me-only
+      // word; the rest of the shared addition is settled.
+      expect(
+        HtmlDiff.executeThreeWay('baseline.', 'baseline. And I add more here', 'baseline. And I add more things here')
+      ).toBe("baseline. And I add more<del class='diffdel cp' data-author='cp'>&nbsp;things</del> here")
+    })
+
+    it("CP rewrote one of Me's added words — del-cp on the replaced word + ins-cp on the replacement", () => {
+      // Non-subset variant: CP didn't just delete, they substituted.
+      //   Me: "And I add more things here"
+      //   CP: "And I add more anything here"
+      // The single word differs — the intent reading is "CP wants
+      // things replaced with anything".
+      expect(
+        HtmlDiff.executeThreeWay(
+          'baseline.',
+          'baseline. And I add more anything here',
+          'baseline. And I add more things here'
+        )
+      ).toBe(
+        "baseline. And I add more <ins class='diffins cp' data-author='cp'>anything</ins><del class='diffdel cp' data-author='cp'>things</del> here"
+      )
+    })
+
+    it("CP extended Me's addition with extra words — ins-cp on the additions, no del", () => {
+      // CP added beyond me — cp-extras stay as ins-cp, no del-cp
+      // since CP didn't remove anything.
+      //   Me: "And I add more things here"
+      //   CP: "And I add more things and other stuff here"
+      expect(
+        HtmlDiff.executeThreeWay(
+          'baseline.',
+          'baseline. And I add more things and other stuff here',
+          'baseline. And I add more things here'
+        )
+      ).toBe("baseline. And I add more things<ins class='diffins cp' data-author='cp'>&nbsp;and other stuff</ins> here")
+    })
+
+    it("Me added text that CP didn't engage with — stays as ins-me, NOT del-cp", () => {
+      // The critical inverse: Me appends a new paragraph that CP
+      // doesn't have anything for at that boundary. This is a
+      // genuine Me-side insertion — NOT a "CP removed" event. The
+      // emitBoundary's single-side branch (!hasCp) preserves
+      // ins-me attribution. Without this carve-out, every Me
+      // insertion would be mis-attributed as "CP wants this gone",
+      // even when CP never engaged with the content.
+      expect(HtmlDiff.executeThreeWay('baseline.', 'baseline.', 'baseline. New paragraph Me added.')).toBe(
+        "baseline.<ins class='diffins me' data-author='me'>&nbsp;New paragraph Me added.</ins>"
+      )
+    })
+
+    it('CP and Me each added different content at different boundaries — each side keeps their own attribution', () => {
+      // Me added at the end of one sentence; CP added at the end of
+      // another. Different genesis boundaries — each goes through
+      // the !hasCp / !hasMe single-side branch. Neither side's
+      // addition surfaces as the other's deletion.
+      expect(
+        HtmlDiff.executeThreeWay(
+          'First sentence. Second sentence.',
+          'First sentence with cp-addition. Second sentence.',
+          'First sentence. Second sentence with me-addition.'
+        )
+      ).toBe(
+        "First sentence<ins class='diffins cp' data-author='cp'>&nbsp;with cp-addition</ins>. Second sentence<ins class='diffins me' data-author='me'>&nbsp;with me-addition</ins>."
+      )
+    })
+
     it('Stable across no-change rounds — V5 produces same output as V3 when V5==V3', () => {
       // The user's V3/V5 invariant: when neither party changes their position
       // in a subsequent turn, the diff should look identical to the previous
@@ -120,11 +195,19 @@ describe('HtmlDiff.executeThreeWay (genesis-spine)', () => {
     })
 
     it('cp matches genesis (only Me changed)', () => {
-      expect(HtmlDiff.executeThreeWay('Hello world', 'Hello world', 'Hello brave world')).toContain("data-author='me'")
+      // Negative assertion is load-bearing: without `not.toContain`
+      // a cp↔me swap inside the genesis-spine merge would still
+      // emit `data-author='cp'` somewhere in the output and the
+      // positive assertion would silently pass.
+      const out = HtmlDiff.executeThreeWay('Hello world', 'Hello world', 'Hello brave world')
+      expect(out).toContain("data-author='me'")
+      expect(out).not.toContain("data-author='cp'")
     })
 
     it('me matches genesis (only CP changed)', () => {
-      expect(HtmlDiff.executeThreeWay('Hello world', 'Hello cruel world', 'Hello world')).toContain("data-author='cp'")
+      const out = HtmlDiff.executeThreeWay('Hello world', 'Hello cruel world', 'Hello world')
+      expect(out).toContain("data-author='cp'")
+      expect(out).not.toContain("data-author='me'")
     })
   })
 
@@ -144,6 +227,24 @@ describe('HtmlDiff.executeThreeWay (genesis-spine)', () => {
       expect(out).toMatch(/<p>First paragraph.*data-author='cp'.*<\/p>/)
       expect(out).toMatch(/<p>Second paragraph.*data-author='me'.*<\/p>/)
     })
+
+    it('overlapping formatting wraps from each author do not unbalance the emission stack', () => {
+      // Genesis: plain "three". CP wrapped it in <strong>, Me in <u>. The
+      // mod-strong (cp) and mod-u (me) wraps cross: <strong> opens before
+      // <u>, but </strong> arrives before </u>. The emitter must split
+      // the inner wrap so the output stays well-formed instead of
+      // throwing an unbalanced-stack error.
+      //
+      // Under the intent-reading model, Me's `<u>` wrap is a
+      // formatting choice CP didn't make — surfaces as a CP-attributed
+      // mod wrap (since CP's view doesn't include the underline). The
+      // load-bearing assertion here is that the emission stays
+      // balanced — the exact mod-author labelling reflects the
+      // asymmetric reading and matches the rest of the suite.
+      expect(HtmlDiff.executeThreeWay('three', '<strong>three</strong>', '<u>three</u>')).toBe(
+        "<strong><ins class='mod strong cp' data-author='cp'><ins class='mod u cp' data-author='cp'>three</ins></ins></strong><ins class='mod u cp' data-author='cp'></ins>"
+      )
+    })
   })
 
   describe('options pass-through', () => {
@@ -152,10 +253,135 @@ describe('HtmlDiff.executeThreeWay (genesis-spine)', () => {
       const without = HtmlDiff.executeThreeWay('a  b', 'a  b', 'a b')
       const withFlag = HtmlDiff.executeThreeWay('a  b', 'a  b', 'a b', { ignoreWhitespaceDifferences: true })
       expect(without).toContain("data-author='me'")
+      // CP matches genesis — any cp attribution would be a mis-merge.
+      expect(without).not.toContain("data-author='cp'")
       expect(withFlag).not.toContain('data-author=')
     })
   })
 
+  describe('stack-balance defence', () => {
+    // The emission walks segments built by `buildSegments`: ins/del
+    // segments go through `insertTag` (which manages the formatting-
+    // tag stack), but equal segments push raw words straight to the
+    // content buffer. When a formatting opener is in a del segment
+    // and its matching closer falls in an equal segment, the stack
+    // entry never gets popped — the emitter used to throw "emission
+    // left 1 unclosed formatting tag(s) on the stack" and crash the
+    // caller. Now it closes the leftover wraps defensively with
+    // `</ins>` so the output stays renderable.
+
+    it('CP inserted a <strong> opener whose closer is matched as equal — does not throw', () => {
+      // Genesis has an orphan closer (`X</strong>`); CP wrapped X in
+      // a fresh `<strong>`. The opener is ins-cp (no genesis match)
+      // but the closer is shared by all three and emits as equal.
+      // The mod-`<ins>` opened on the strong push needs to be closed
+      // somehow; the defensive path emits a trailing `</ins>`.
+      expect(() => HtmlDiff.executeThreeWay('X</strong>', '<strong>X</strong>', 'X</strong>')).not.toThrow()
+    })
+
+    it('CP deleted only the <strong> opener — does not throw', () => {
+      // Symmetric: genesis had `<strong>X</strong>`, CP dropped the
+      // opener but kept the closer. The opener-delete pushes onto
+      // the stack and the closer arrives via an equal segment.
+      expect(() => HtmlDiff.executeThreeWay('<strong>X</strong>', 'X</strong>', '<strong>X</strong>')).not.toThrow()
+    })
+
+    it('emits the defensive </ins> close and logs a warning when the stack is unbalanced', async () => {
+      const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
+      try {
+        const out = HtmlDiff.executeThreeWay('X</strong>', '<strong>X</strong>', 'X</strong>')
+        // The content survives.
+        expect(out).toContain('X')
+        // The defensive close path actually ran — output contains
+        // at least one `</ins>` that wasn't paired by `insertTag`
+        // (the only way the defensive branch can add one).
+        expect(out).toMatch(/<\/ins>/)
+        // And the warn was emitted. Without this assertion the path
+        // could silently stop firing in a future refactor and the
+        // test would still pass on the (incidentally-present) content.
+        expect(warn).toHaveBeenCalledWith(expect.stringContaining('unclosed formatting wrap'))
+      } finally {
+        warn.mockRestore()
+      }
+    })
+  })
+
+  describe('WORD_ALIGNED_OPTIONS — opinionated consumer defaults', () => {
+    // The library default (`orphanMatchThreshold = 0`) keeps every LCS
+    // match, however small — which fragments long sentence rewrites
+    // into many tiny ins/del pairs around stray word matches. Word's
+    // track-changes collapses those into a single coarse del+ins,
+    // which is markedly more readable for legal text. The exported
+    // `WORD_ALIGNED_OPTIONS` lets consumers opt into that without
+    // re-tuning the magic number themselves.
+    const longGenesis =
+      '"Specified Indebtedness" will have the meaning specified in Section 14 and shall include, with respect to Party B, any obligation (whether present or future, contingent or otherwise) for the payment or repayment of money.'
+    const longCp =
+      '"Specified Indebtedness" will have the meaning specified in Section 14 of the Agreement except that such term shall not include obligations.'
+
+    it('exports a 0.25 orphan threshold tuned for Word-aligned output', () => {
+      expect(WORD_ALIGNED_OPTIONS).toEqual({ orphanMatchThreshold: 0.25 })
+    })
+
+    it('plumbs through HtmlDiff.execute and reduces fragmentation versus the bare default', () => {
+      const bare = HtmlDiff.execute(longGenesis, longCp)
+      const aligned = HtmlDiff.execute(longGenesis, longCp, WORD_ALIGNED_OPTIONS)
+      const count = (s: string, re: RegExp) => (s.match(re) ?? []).length
+      // The bare default keeps every tiny match — Word-aligned produces
+      // strictly fewer ins/del wrappers for the same input.
+      expect(count(aligned, /<ins/g)).toBeLessThan(count(bare, /<ins/g))
+      expect(count(aligned, /<del/g)).toBeLessThan(count(bare, /<del/g))
+    })
+
+    it('plumbs through HtmlDiff.executeThreeWay too', () => {
+      const bare = HtmlDiff.executeThreeWay(longGenesis, longCp, longGenesis)
+      const aligned = HtmlDiff.executeThreeWay(longGenesis, longCp, longGenesis, WORD_ALIGNED_OPTIONS)
+      const count = (s: string, re: RegExp) => (s.match(re) ?? []).length
+      expect(count(aligned, /<ins/g)).toBeLessThan(count(bare, /<ins/g))
+      expect(count(aligned, /<del/g)).toBeLessThan(count(bare, /<del/g))
+    })
+  })
+
+  describe('orphan-match guard for structural tags', () => {
+    // Real regression from the live preview (Additional Condition
+    // Precedent in the 2002 ISDA Schedule): when CP deletes a section
+    // whose answer renders as an empty formatting shell —
+    //   <p data-html="x"><em><strong></strong></em></p>
+    // — the `</strong>` and `</em>` matches sit between two content
+    // deletions ("Heading. " before, body after). At
+    // WORD_ALIGNED_OPTIONS.orphanMatchThreshold=0.25 those structural
+    // matches were rejected as orphans, swallowed into the deletion
+    // span, and the browser auto-closed the openers AT THE END of
+    // the deletion — visually rendering the entire deletion as
+    // bold-italic. The orphan filter now exempts tag-only matches
+    // so structural boundaries always survive.
+
+    it('CP deletes section with em+strong heading + plain body — closers stay between heading and body', () => {
+      const genesis =
+        '<p data-html="x"><em><strong>Additional Condition Precedent. </strong></em>For the purposes of Section 2(a)(iii).</p>'
+      const cp = '<p data-html="x"><em><strong></strong></em></p>'
+      const me = genesis
+
+      const out = HtmlDiff.executeThreeWay(genesis, cp, me, WORD_ALIGNED_OPTIONS)
+
+      // </strong> appears BEFORE the body deletion — meaning the
+      // body sits outside the bold-italic wrap, not inside it.
+      const closeStrongIdx = out.indexOf('</strong>')
+      const bodyDelIdx = out.indexOf('For the purposes')
+      expect(closeStrongIdx).toBeGreaterThan(0)
+      expect(bodyDelIdx).toBeGreaterThan(closeStrongIdx)
+      // No `<strong>…<del>body` substring exists — confirm by exact
+      // shape too. Heading wraps in strong+em, body is a plain del.
+      expect(out).toBe(
+        '<p data-html="x"><em><strong>' +
+          "<del class='diffdel cp' data-author='cp'>Additional Condition Precedent. </del>" +
+          '</strong></em>' +
+          "<del class='diffdel cp' data-author='cp'>For the purposes of Section 2(a)(iii).</del>" +
+          '</p>'
+      )
+    })
+  })
+
   describe('first-turn fallback', () => {
     it('cp == genesis means CP made no changes — Me-only attribution', () => {
       // Common case: this is the first turn where the counterparty hasn't

package/test/HtmlDiff.threeWay.tables.spec.ts

@@ -44,6 +44,14 @@ describe('HtmlDiff.executeThreeWay (tables, genesis-spine)', () => {
     })
 
     it('Disagreement — different changes at the same place', () => {
+      // Genesis cell "five"; CP extended to "five and a half"; Me
+      // replaced with "seven". Intent reading from a reviewer's
+      // perspective:
+      //   - del-me "five": Me already removed the genesis word from
+      //     their cell (genesis tracking).
+      //   - ins-cp " and a half": CP wants this appended.
+      //   - del-cp "seven": CP wants Me's "seven" removed (Me has it,
+      //     CP doesn't).
       expect(
         HtmlDiff.executeThreeWay(
           '<table><tr><td>five</td></tr></table>',
@@ -51,7 +59,7 @@ describe('HtmlDiff.executeThreeWay (tables, genesis-spine)', () => {
           '<table><tr><td>seven</td></tr></table>'
         )
       ).toBe(
-        "<table><tr><td><del class='diffdel me' data-author='me'>five</del><ins class='diffins cp' data-author='cp'>&nbsp;and a half</ins><ins class='diffins me' data-author='me'>seven</ins></td></tr></table>"
+        "<table><tr><td><del class='diffdel me' data-author='me'>five</del><ins class='diffins cp' data-author='cp'>&nbsp;and a half</ins><del class='diffdel cp' data-author='cp'>seven</del></td></tr></table>"
       )
     })
 
@@ -259,6 +267,75 @@ describe('HtmlDiff.executeThreeWay (tables, genesis-spine)', () => {
       const html = `<table>${rows}</table>`
       expect(HtmlDiff.executeThreeWay(html, html, html)).toBe(html)
     })
+
+    it('cell-count mismatch: CP added a column — CP row content is visible (not silently dropped)', () => {
+      // Regression: the previous fallback in emitPreservedRow emitted
+      // only `del me` + `ins me` for any cell-count mismatch, which
+      // silently destroyed CP's row content whenever CP changed the
+      // cell count. A reader in cp-only mode would see no trace of
+      // CP's added column — a content-loss bug that violates the
+      // "CP's changes always visible" invariant.
+      const out = HtmlDiff.executeThreeWay(
+        '<table><tr><td>a</td><td>b</td></tr></table>',
+        '<table><tr><td>a</td><td>X</td><td>b</td></tr></table>',
+        '<table><tr><td>a</td><td>b</td></tr></table>'
+      )
+      expect(out).toBe(
+        "<table><tr class='diffdel cp' data-author='cp'><td class='diffdel cp' data-author='cp'><del class='diffdel cp' data-author='cp'>a</del></td><td class='diffdel cp' data-author='cp'><del class='diffdel cp' data-author='cp'>b</del></td></tr><tr class='diffins cp' data-author='cp'><td class='diffins cp' data-author='cp'><ins class='diffins cp' data-author='cp'>a</ins></td><td class='diffins cp' data-author='cp'><ins class='diffins cp' data-author='cp'>X</ins></td><td class='diffins cp' data-author='cp'><ins class='diffins cp' data-author='cp'>b</ins></td></tr></table>"
+      )
+    })
+
+    it('cell-count mismatch: Me removed a column — symmetric to the CP case', () => {
+      const out = HtmlDiff.executeThreeWay(
+        '<table><tr><td>a</td><td>b</td></tr></table>',
+        '<table><tr><td>a</td><td>b</td></tr></table>',
+        '<table><tr><td>a</td></tr></table>'
+      )
+      expect(out).toBe(
+        "<table><tr class='diffdel me' data-author='me'><td class='diffdel me' data-author='me'><del class='diffdel me' data-author='me'>a</del></td><td class='diffdel me' data-author='me'><del class='diffdel me' data-author='me'>b</del></td></tr><tr class='diffins me' data-author='me'><td class='diffins me' data-author='me'><ins class='diffins me' data-author='me'>a</ins></td></tr></table>"
+      )
+    })
+
+    it('CP edited one cell in a row (same shape) — fuzzy-pairs and emits a cell-level diff, not whole-row del+ins', () => {
+      // Regression: the 3-way row aligner only did exact lcsAlign over
+      // rowKey, so a row where CP edited a single cell's text produced
+      // no key match and the algorithm split the row into a whole-row
+      // delete + whole-row insert. The 2-way path has always run a
+      // fuzzy-pairing pass after lcsAlign; bringing the 3-way path in
+      // step removes the asymmetry where cp-only / all-changes views
+      // looked materially worse than 2-way for ordinary cell edits.
+      //
+      // Same-shape genesis/cp/me; CP edited the middle cell's text.
+      // Me === genesis. Expect a paired row with cell-level cp-ins
+      // markup, NOT two distinct whole-row entries.
+      const out = HtmlDiff.executeThreeWay(
+        '<table><tr><td>Party A</td><td>old details</td><td>kept</td></tr></table>',
+        '<table><tr><td>Party A</td><td>new details</td><td>kept</td></tr></table>',
+        '<table><tr><td>Party A</td><td>old details</td><td>kept</td></tr></table>'
+      )
+      // CP's edit lives inside the row, not as a parallel whole-row
+      // delete-then-insert. Whole-row markers would carry `class='diffdel ...'`
+      // or `class='diffins ...'` on the `<tr>` itself.
+      expect(out).not.toMatch(/<tr [^>]*class=['"]diffdel/)
+      expect(out).not.toMatch(/<tr [^>]*class=['"]diffins/)
+      expect(out).toContain('Party A')
+      expect(out).toContain("data-author='cp'")
+      // Me === genesis so any me attribution would indicate a swap.
+      expect(out).not.toContain("data-author='me'")
+    })
+
+    it('cell-count mismatch: both sides restructured differently — both ins rows attributed', () => {
+      // Genesis 2 cells, CP 3 cells, Me 4 cells. Neither side keeps
+      // the genesis shape, so both restructures must be visible.
+      const out = HtmlDiff.executeThreeWay(
+        '<table><tr><td>a</td><td>b</td></tr></table>',
+        '<table><tr><td>a</td><td>X</td><td>b</td></tr></table>',
+        '<table><tr><td>a</td><td>b</td><td>Y</td><td>Z</td></tr></table>'
+      )
+      expect(out).toBe(
+        "<table><tr class='diffins cp' data-author='cp'><td class='diffins cp' data-author='cp'><ins class='diffins cp' data-author='cp'>a</ins></td><td class='diffins cp' data-author='cp'><ins class='diffins cp' data-author='cp'>X</ins></td><td class='diffins cp' data-author='cp'><ins class='diffins cp' data-author='cp'>b</ins></td></tr><tr class='diffins me' data-author='me'><td class='diffins me' data-author='me'><ins class='diffins me' data-author='me'>a</ins></td><td class='diffins me' data-author='me'><ins class='diffins me' data-author='me'>b</ins></td><td class='diffins me' data-author='me'><ins class='diffins me' data-author='me'>Y</ins></td><td class='diffins me' data-author='me'><ins class='diffins me' data-author='me'>Z</ins></td></tr></table>"
+      )
+    })
   })
 
   describe('nested tables', () => {
@@ -270,6 +347,9 @@ describe('HtmlDiff.executeThreeWay (tables, genesis-spine)', () => {
       )
       expect(out).toMatch(/<del[^>]*data-author='cp'[^>]*>inner<\/del>/)
       expect(out).toMatch(/<ins[^>]*data-author='cp'[^>]*>INNER<\/ins>/)
+      // me == genesis here, so any me attribution would indicate a
+      // cp↔me swap inside the table-cell merge.
+      expect(out).not.toContain("data-author='me'")
       expect(out.startsWith('<table><tr><td><table>')).toBe(true)
       expect(out.endsWith('</table></td></tr></table>')).toBe(true)
     })
@@ -298,4 +378,34 @@ describe('HtmlDiff.executeThreeWay (tables, genesis-spine)', () => {
       expect(HtmlDiff.executeThreeWay('<p>a</p>', '<p>a</p>', '<p>a</p>')).toBe('<p>a</p>')
     })
   })
+
+  describe('positional pairing under moderate dissimilarity', () => {
+    it('column rename + value rewrite still routes through cell-level diff (not whole-table del+ins)', () => {
+      // Real-world regression: cp renamed a column ("Form/Document/Certificate"
+      // → "Extra column") and replaced the values in that column with short
+      // tokens. Word-level Jaccard between the genesis table and cp's edited
+      // table drops to ~0.38 — under the 0.5 threshold the three-way path
+      // used to take, which kicked the diff into multi-table content-LCS
+      // and produced whole-table del+ins (the cp's CP-bubble showed the
+      // entire old table struck through and the entire new table inserted).
+      // 2-way had no such guard and produced a cell-level diff for the same
+      // inputs; lowering the 3-way threshold brings the two paths in step.
+      const genesis =
+        '<table><tr><td>A</td><td>Form/Document/Certificate</td><td>Date</td></tr><tr><td>Party A</td><td>IRS W-8</td><td>On execution</td></tr></table>'
+      const cp =
+        '<table><tr><td>A</td><td>Extra column</td><td>Date</td></tr><tr><td>Party A</td><td>Yes</td><td>On execution</td></tr></table>'
+      const me = genesis
+      const out = HtmlDiff.executeThreeWay(genesis, cp, me)
+      // Expect cell-level cp attribution INSIDE the table cells, NOT a
+      // whole-table del+ins wrapping the entire <table>.
+      expect(out).not.toMatch(/<del[^>]*><table/)
+      expect(out).toMatch(/data-author='cp'/)
+      // me === genesis, so any me-attribution markers would mean the
+      // diff swapped CP's edits onto Me. Negative assertion locks the
+      // attribution direction.
+      expect(out).not.toContain("data-author='me'")
+      expect(out).toContain('Extra column')
+      expect(out).toContain('Form/Document/Certificate')
+    })
+  })
 })

package/test/Utils.spec.ts

@@ -138,10 +138,10 @@ describe('Utils', () => {
     it('combines extraClasses and dataAttrs in one call', () => {
       expect(
         Utils.wrapText('hello', 'del', 'diffdel', {
-          extraClasses: 'me rejects-cp',
-          dataAttrs: { author: 'me', rejects: 'cp' },
+          extraClasses: 'me',
+          dataAttrs: { author: 'me', source: 'edit' },
         })
-      ).toBe("<del class='diffdel me rejects-cp' data-author='me' data-rejects='cp'>hello</del>")
+      ).toBe("<del class='diffdel me' data-author='me' data-source='edit'>hello</del>")
     })
 
     it('skips the metadata path entirely when neither extraClasses nor dataAttrs is set', () => {