@adeu/core 1.6.2 → 1.6.4

package/src/engine.atomic.test.ts

@@ -1,58 +1,58 @@
-import { describe, it, expect } from 'vitest';
-import { createTestDocument, addParagraph } from './test-utils.js';
-import { DocumentObject } from './docx/bridge.js';
-import { extractTextFromBuffer } from './ingest.js';
-import { RedlineEngine } from './engine.js';
-import { ModifyText, AcceptChange } from './models.js';
-
-describe('Atomic Batch Pipeline (Node.js Port)', () => {
-  it('prevents cascading misanchor when accepting changes shifts indices', async () => {
-    // 1. Setup initial doc
-    const doc = await createTestDocument();
-    addParagraph(doc, "First paragraph.");
-    addParagraph(doc, "Second paragraph.");
-    addParagraph(doc, "Third paragraph.");
-
-    // 2. Make an initial tracked change (Simulating Round 1)
-    const engine = new RedlineEngine(doc, "Round1");
-    engine.apply_edits([{ type: 'modify', target_text: "First", new_text: "1st" } as ModifyText]);
-
-    const midBuf = await doc.save();
-
-    // Verify intermediate state (Round 1)
-    const midText = await extractTextFromBuffer(midBuf);
-    expect(midText).toContain("{--First--}");
-    expect(midText).toContain("{++1st++}");
-
-    // Extract dynamically generated Change IDs for the Accept action
-    const matches = Array.from(midText.matchAll(/\[Chg:(\d+)(?:\s+\w+)?\]/g));
-    const chgIds = new Set(matches.map(m => m[1]));
-    expect(chgIds.size).toBeGreaterThan(0);
-
-    // 3. Execute the Atomic Batch (Simulating Round 2)
-    const midDoc = await DocumentObject.load(midBuf);
-    const engine2 = new RedlineEngine(midDoc, "Round2");
-
-    const actions = Array.from(chgIds).map(id => ({ type: 'accept', target_id: `Chg:${id}` } as AcceptChange));
-    const edits = [{ type: 'modify', target_text: "Third", new_text: "3rd" } as ModifyText];
-    
-    const changes = [...actions, ...edits];
-    const stats = engine2.process_batch(changes);
-
-    // 4. Assertions on the Tool Execution
-    expect(stats.actions_applied).toBe(actions.length);
-    expect(stats.edits_applied).toBe(1);
-
-    // 5. Assertions on the Final Document State
-    const finalBuf = await midDoc.save();
-    const final_text = await extractTextFromBuffer(finalBuf);
-
-    // The first paragraph should be cleanly accepted
-    expect(final_text).toContain("1st paragraph.");
-    expect(final_text).not.toContain("{--First--}");
-
-    // The third paragraph should have the new tracked change anchored perfectly
-    expect(final_text).toContain("{--Third--}");
-    expect(final_text).toContain("{++3rd++}");
-  });
+import { describe, it, expect } from 'vitest';
+import { createTestDocument, addParagraph } from './test-utils.js';
+import { DocumentObject } from './docx/bridge.js';
+import { extractTextFromBuffer } from './ingest.js';
+import { RedlineEngine } from './engine.js';
+import { ModifyText, AcceptChange } from './models.js';
+
+describe('Atomic Batch Pipeline (Node.js Port)', () => {
+  it('prevents cascading misanchor when accepting changes shifts indices', async () => {
+    // 1. Setup initial doc
+    const doc = await createTestDocument();
+    addParagraph(doc, "First paragraph.");
+    addParagraph(doc, "Second paragraph.");
+    addParagraph(doc, "Third paragraph.");
+
+    // 2. Make an initial tracked change (Simulating Round 1)
+    const engine = new RedlineEngine(doc, "Round1");
+    engine.apply_edits([{ type: 'modify', target_text: "First", new_text: "1st" } as ModifyText]);
+
+    const midBuf = await doc.save();
+
+    // Verify intermediate state (Round 1)
+    const midText = await extractTextFromBuffer(midBuf);
+    expect(midText).toContain("{--First--}");
+    expect(midText).toContain("{++1st++}");
+
+    // Extract dynamically generated Change IDs for the Accept action
+    const matches = Array.from(midText.matchAll(/\[Chg:(\d+)(?:\s+\w+)?\]/g));
+    const chgIds = new Set(matches.map(m => m[1]));
+    expect(chgIds.size).toBeGreaterThan(0);
+
+    // 3. Execute the Atomic Batch (Simulating Round 2)
+    const midDoc = await DocumentObject.load(midBuf);
+    const engine2 = new RedlineEngine(midDoc, "Round2");
+
+    const actions = Array.from(chgIds).map(id => ({ type: 'accept', target_id: `Chg:${id}` } as AcceptChange));
+    const edits = [{ type: 'modify', target_text: "Third", new_text: "3rd" } as ModifyText];
+    
+    const changes = [...actions, ...edits];
+    const stats = engine2.process_batch(changes);
+
+    // 4. Assertions on the Tool Execution
+    expect(stats.actions_applied).toBe(actions.length);
+    expect(stats.edits_applied).toBe(1);
+
+    // 5. Assertions on the Final Document State
+    const finalBuf = await midDoc.save();
+    const final_text = await extractTextFromBuffer(finalBuf);
+
+    // The first paragraph should be cleanly accepted
+    expect(final_text).toContain("1st paragraph.");
+    expect(final_text).not.toContain("{--First--}");
+
+    // The third paragraph should have the new tracked change anchored perfectly
+    expect(final_text).toContain("{--Third--}");
+    expect(final_text).toContain("{++3rd++}");
+  });
 });

package/src/engine.batch.test.ts

@@ -1,93 +1,93 @@
-import { describe, it, expect } from 'vitest';
-import { createTestDocument, addParagraph } from './test-utils.js';
-import { DocumentObject } from './docx/bridge.js';
-import { extractTextFromBuffer } from './ingest.js';
-import { RedlineEngine } from './engine.js';
-import { ModifyText, AcceptChange, RejectChange } from './models.js';
-
-describe('Batch Reliability (Node.js Port)', () => {
-  it('batch accept does not corrupt the document', async () => {
-    const doc = await createTestDocument();
-    addParagraph(doc, "Para 1");
-    addParagraph(doc, "Para 2");
-    addParagraph(doc, "Para 3");
-
-    const engine = new RedlineEngine(doc);
-    const edits: ModifyText[] = [
-      { type: 'modify', target_text: "Para 1", new_text: "Para One" },
-      { type: 'modify', target_text: "Para 2", new_text: "Para Two" },
-      { type: 'modify', target_text: "Para 3", new_text: "Para Three" },
-    ];
-
-    engine.apply_edits(edits);
-    const redlinedBuf = await doc.save();
-
-    const text = await extractTextFromBuffer(redlinedBuf);
-    expect(text).toContain("[Chg:1 ");
-    expect(text).toContain("[Chg:6 ");
-
-    // BATCH ACCEPT ALL
-    const midDoc = await DocumentObject.load(redlinedBuf);
-    const engine2 = new RedlineEngine(midDoc);
-    
-    const actions: AcceptChange[] = [1, 2, 3, 4, 5, 6].map(id => ({ type: 'accept', target_id: `Chg:${id}` }));
-    
-    // Test direct apply_review_actions (Note: method missing in raw TS port right now, needs implementing)
-    const [applied, skipped] = (engine2 as any).apply_review_actions(actions);
-    
-    expect(applied).toBe(6);
-    expect(skipped).toBe(0);
-
-    const finalBuf = await midDoc.save();
-    const final_text = await extractTextFromBuffer(finalBuf);
-
-    expect(final_text).toContain("Para One");
-    expect(final_text).toContain("Para Two");
-    expect(final_text).toContain("Para Three");
-
-    expect(final_text).not.toContain("Para 1");
-    expect(final_text).not.toContain("Para 2");
-    expect(final_text).not.toContain("Para 3");
-
-    expect(final_text).not.toContain("[Chg:");
-    expect(final_text).not.toContain("{++");
-    expect(final_text).not.toContain("{--");
-  });
-
-  it('batch mixed accept and reject maintains integrity', async () => {
-    const doc = await createTestDocument();
-    addParagraph(doc, "Para 1");
-    addParagraph(doc, "Para 2");
-
-    const engine = new RedlineEngine(doc);
-    const edits: ModifyText[] = [
-      { type: 'modify', target_text: "Para 1", new_text: "Para One" },
-      { type: 'modify', target_text: "Para 2", new_text: "Para Two" },
-    ];
-
-    engine.apply_edits(edits);
-    const redlinedBuf = await doc.save();
-
-    const midDoc = await DocumentObject.load(redlinedBuf);
-    const engine2 = new RedlineEngine(midDoc);
-
-    const actions = [
-      { type: 'accept', target_id: "Chg:3" } as AcceptChange,
-      { type: 'accept', target_id: "Chg:4" } as AcceptChange,
-      { type: 'reject', target_id: "Chg:1" } as RejectChange,
-      { type: 'reject', target_id: "Chg:2" } as RejectChange,
-    ];
-
-    const [applied] = (engine2 as any).apply_review_actions(actions);
-    expect(applied).toBe(4);
-
-    const finalBuf = await midDoc.save();
-    const text_final = await extractTextFromBuffer(finalBuf);
-
-    expect(text_final).toContain("Para One");
-    expect(text_final).not.toContain("Para 1");
-
-    expect(text_final).toContain("Para 2");
-    expect(text_final).not.toContain("Para Two");
-  });
+import { describe, it, expect } from 'vitest';
+import { createTestDocument, addParagraph } from './test-utils.js';
+import { DocumentObject } from './docx/bridge.js';
+import { extractTextFromBuffer } from './ingest.js';
+import { RedlineEngine } from './engine.js';
+import { ModifyText, AcceptChange, RejectChange } from './models.js';
+
+describe('Batch Reliability (Node.js Port)', () => {
+  it('batch accept does not corrupt the document', async () => {
+    const doc = await createTestDocument();
+    addParagraph(doc, "Para 1");
+    addParagraph(doc, "Para 2");
+    addParagraph(doc, "Para 3");
+
+    const engine = new RedlineEngine(doc);
+    const edits: ModifyText[] = [
+      { type: 'modify', target_text: "Para 1", new_text: "Para One" },
+      { type: 'modify', target_text: "Para 2", new_text: "Para Two" },
+      { type: 'modify', target_text: "Para 3", new_text: "Para Three" },
+    ];
+
+    engine.apply_edits(edits);
+    const redlinedBuf = await doc.save();
+
+    const text = await extractTextFromBuffer(redlinedBuf);
+    expect(text).toContain("[Chg:1 ");
+    expect(text).toContain("[Chg:6 ");
+
+    // BATCH ACCEPT ALL
+    const midDoc = await DocumentObject.load(redlinedBuf);
+    const engine2 = new RedlineEngine(midDoc);
+    
+    const actions: AcceptChange[] = [1, 2, 3, 4, 5, 6].map(id => ({ type: 'accept', target_id: `Chg:${id}` }));
+    
+    // Test direct apply_review_actions (Note: method missing in raw TS port right now, needs implementing)
+    const [applied, skipped] = (engine2 as any).apply_review_actions(actions);
+    
+    expect(applied).toBe(6);
+    expect(skipped).toBe(0);
+
+    const finalBuf = await midDoc.save();
+    const final_text = await extractTextFromBuffer(finalBuf);
+
+    expect(final_text).toContain("Para One");
+    expect(final_text).toContain("Para Two");
+    expect(final_text).toContain("Para Three");
+
+    expect(final_text).not.toContain("Para 1");
+    expect(final_text).not.toContain("Para 2");
+    expect(final_text).not.toContain("Para 3");
+
+    expect(final_text).not.toContain("[Chg:");
+    expect(final_text).not.toContain("{++");
+    expect(final_text).not.toContain("{--");
+  });
+
+  it('batch mixed accept and reject maintains integrity', async () => {
+    const doc = await createTestDocument();
+    addParagraph(doc, "Para 1");
+    addParagraph(doc, "Para 2");
+
+    const engine = new RedlineEngine(doc);
+    const edits: ModifyText[] = [
+      { type: 'modify', target_text: "Para 1", new_text: "Para One" },
+      { type: 'modify', target_text: "Para 2", new_text: "Para Two" },
+    ];
+
+    engine.apply_edits(edits);
+    const redlinedBuf = await doc.save();
+
+    const midDoc = await DocumentObject.load(redlinedBuf);
+    const engine2 = new RedlineEngine(midDoc);
+
+    const actions = [
+      { type: 'accept', target_id: "Chg:3" } as AcceptChange,
+      { type: 'accept', target_id: "Chg:4" } as AcceptChange,
+      { type: 'reject', target_id: "Chg:1" } as RejectChange,
+      { type: 'reject', target_id: "Chg:2" } as RejectChange,
+    ];
+
+    const [applied] = (engine2 as any).apply_review_actions(actions);
+    expect(applied).toBe(4);
+
+    const finalBuf = await midDoc.save();
+    const text_final = await extractTextFromBuffer(finalBuf);
+
+    expect(text_final).toContain("Para One");
+    expect(text_final).not.toContain("Para 1");
+
+    expect(text_final).toContain("Para 2");
+    expect(text_final).not.toContain("Para Two");
+  });
 });

package/src/engine.safety.test.ts

@@ -1,42 +1,42 @@
-import { describe, it, expect } from 'vitest';
-import { createTestDocument, addParagraph } from './test-utils.js';
-import { RedlineEngine } from './engine.js';
-import { ModifyText } from './models.js';
-
-describe('Safety Engine Constraints (Node.js Port)', () => {
-  it('rejects empty target heuristic to prevent accidental insertions', async () => {
-    const doc = await createTestDocument();
-    addParagraph(doc, "Content");
-
-    const engine = new RedlineEngine(doc);
-    const edit: ModifyText = { type: 'modify', target_text: "", new_text: "Unexpected Header" };
-
-    const [applied, skipped] = engine.apply_edits([edit]);
-
-    expect(applied).toBe(0);
-    expect(skipped).toBe(1);
-
-    const xml = (doc.element as Element).ownerDocument?.documentElement.toString();
-    expect(xml).not.toContain("Unexpected Header");
-  });
-
-  it('applies heuristic edits strictly once for multiple occurrences', async () => {
-    const doc = await createTestDocument();
-    addParagraph(doc, "Repeat");
-    addParagraph(doc, "Repeat");
-
-    const engine = new RedlineEngine(doc);
-    const edit: ModifyText = { type: 'modify', target_text: "Repeat", new_text: "Changed" };
-
-    const [applied, skipped] = engine.apply_edits([edit]);
-
-    expect(applied).toBe(1);
-    expect(skipped).toBe(0);
-
-    const xml = (doc.element as Element).ownerDocument?.documentElement.toString();
-    
-    // We expect one deletion of Repeat and one insertion of Changed
-    expect((xml?.match(/<w:delText[^>]*>Repeat<\/w:delText>/g) || []).length).toBe(1);
-    expect((xml?.match(/<w:t[^>]*>Changed<\/w:t>/g) || []).length).toBe(1);
-  });
+import { describe, it, expect } from 'vitest';
+import { createTestDocument, addParagraph } from './test-utils.js';
+import { RedlineEngine } from './engine.js';
+import { ModifyText } from './models.js';
+
+describe('Safety Engine Constraints (Node.js Port)', () => {
+  it('rejects empty target heuristic to prevent accidental insertions', async () => {
+    const doc = await createTestDocument();
+    addParagraph(doc, "Content");
+
+    const engine = new RedlineEngine(doc);
+    const edit: ModifyText = { type: 'modify', target_text: "", new_text: "Unexpected Header" };
+
+    const [applied, skipped] = engine.apply_edits([edit]);
+
+    expect(applied).toBe(0);
+    expect(skipped).toBe(1);
+
+    const xml = (doc.element as Element).ownerDocument?.documentElement.toString();
+    expect(xml).not.toContain("Unexpected Header");
+  });
+
+  it('applies heuristic edits strictly once for multiple occurrences', async () => {
+    const doc = await createTestDocument();
+    addParagraph(doc, "Repeat");
+    addParagraph(doc, "Repeat");
+
+    const engine = new RedlineEngine(doc);
+    const edit: ModifyText = { type: 'modify', target_text: "Repeat", new_text: "Changed" };
+
+    const [applied, skipped] = engine.apply_edits([edit]);
+
+    expect(applied).toBe(1);
+    expect(skipped).toBe(0);
+
+    const xml = (doc.element as Element).ownerDocument?.documentElement.toString();
+    
+    // We expect one deletion of Repeat and one insertion of Changed
+    expect((xml?.match(/<w:delText[^>]*>Repeat<\/w:delText>/g) || []).length).toBe(1);
+    expect((xml?.match(/<w:t[^>]*>Changed<\/w:t>/g) || []).length).toBe(1);
+  });
 });