@beyondwork/docx-react-component 1.0.13 → 1.0.15

package/package.json

@@ -1,9 +1,8 @@
 {
   "name": "@beyondwork/docx-react-component",
   "publisher": "beyondwork",
-  "version": "1.0.13",
+  "version": "1.0.15",
   "description": "Embeddable React Word (docx) editor with review, comments, tracked changes, and round-trip OOXML fidelity.",
-  "packageManager": "pnpm@10.30.3",
   "type": "module",
   "sideEffects": [
     "**/*.css"
@@ -34,10 +33,18 @@
       "types": "./src/api/public-types.ts",
       "import": "./src/api/public-types.ts"
     },
+    "./compare": {
+      "types": "./src/compare/index.ts",
+      "import": "./src/compare/index.ts"
+    },
     "./io/docx-session": {
       "types": "./src/io/docx-session.ts",
       "import": "./src/io/docx-session.ts"
     },
+    "./legal": {
+      "types": "./src/legal/index.ts",
+      "import": "./src/legal/index.ts"
+    },
     "./runtime/document-runtime": {
       "types": "./src/runtime/document-runtime.ts",
       "import": "./src/runtime/document-runtime.ts"
@@ -46,29 +53,6 @@
     "./package.json": "./package.json"
   },
   "types": "./src/index.ts",
-  "scripts": {
-    "build": "tsup",
-    "test": "bash scripts/run-workspace-tests.sh",
-    "test:repo": "node scripts/run-repo-tests.mjs core",
-    "test:repo:all": "node scripts/run-repo-tests.mjs all",
-    "test:repo:optional": "node scripts/run-repo-tests.mjs optional",
-    "test:wcag-audit": "node scripts/run-repo-tests.mjs wcag-audit",
-    "test:harness": "pnpm --filter @docx-react-component/react-word-editor-harness test",
-    "lint": "pnpm run lint:no-authored-js && pnpm run lint:tsgo && pnpm run lint:tsgo:harness",
-    "lint:no-authored-js": "bash scripts/check-no-authored-js.sh",
-    "lint:tsgo": "tsgo --noEmit -p tsconfig.build.json",
-    "lint:tsgo:harness": "pnpm --filter @docx-react-component/react-word-editor-harness lint:tsgo",
-    "context7:api-check": "bash scripts/context7-export-env.sh run bash scripts/context7-api-check.sh",
-    "wave:doctor": "bash scripts/context7-export-env.sh run pnpm exec wave doctor --json",
-    "wave:dry-run": "bash scripts/context7-export-env.sh run pnpm exec wave launch --lane main --dry-run --no-dashboard",
-    "wave:launch": "bash scripts/context7-export-env.sh run pnpm exec wave launch --lane main",
-    "wave:launch:managed": "bash scripts/wave-launch.sh",
-    "wave:status": "bash scripts/wave-status.sh",
-    "wave:watch": "bash scripts/wave-watch.sh --follow",
-    "wave:dashboard:current": "bash scripts/wave-dashboard-attach.sh current",
-    "wave:dashboard:global": "bash scripts/wave-dashboard-attach.sh global",
-    "harness:dev": "pnpm --filter @docx-react-component/react-word-editor-harness dev"
-  },
   "keywords": [
     "docx",
     "word",
@@ -114,7 +98,7 @@
     "tailwindcss": "^4.2.2"
   },
   "devDependencies": {
-    "@chllming/wave-orchestration": "^0.9.13",
+    "@chllming/wave-orchestration": "^0.9.15",
     "@types/react": "19.2.14",
     "@types/react-dom": "19.2.3",
     "@typescript/native-preview": "7.0.0-dev.20260409.1",
@@ -131,14 +115,27 @@
     "tsup": "^8.3.0",
     "tsx": "^4.21.0"
   },
-  "pnpm": {
-    "onlyBuiltDependencies": [
-      "esbuild",
-      "sharp"
-    ],
-    "overrides": {
-      "react": "19.2.4",
-      "react-dom": "19.2.4"
-    }
+  "scripts": {
+    "build": "tsup",
+    "test": "bash scripts/run-workspace-tests.sh",
+    "test:repo": "node scripts/run-repo-tests.mjs core",
+    "test:repo:all": "node scripts/run-repo-tests.mjs all",
+    "test:repo:optional": "node scripts/run-repo-tests.mjs optional",
+    "test:wcag-audit": "node scripts/run-repo-tests.mjs wcag-audit",
+    "test:harness": "pnpm --filter @docx-react-component/react-word-editor-harness test",
+    "lint": "pnpm run lint:no-authored-js && pnpm run lint:tsgo && pnpm run lint:tsgo:harness",
+    "lint:no-authored-js": "bash scripts/check-no-authored-js.sh",
+    "lint:tsgo": "tsgo --noEmit -p tsconfig.build.json",
+    "lint:tsgo:harness": "pnpm --filter @docx-react-component/react-word-editor-harness lint:tsgo",
+    "context7:api-check": "bash scripts/context7-export-env.sh run bash scripts/context7-api-check.sh",
+    "wave:doctor": "bash scripts/context7-export-env.sh run pnpm exec wave doctor --json",
+    "wave:dry-run": "bash scripts/context7-export-env.sh run pnpm exec wave launch --lane main --dry-run --no-dashboard",
+    "wave:launch": "bash scripts/context7-export-env.sh run pnpm exec wave launch --lane main",
+    "wave:launch:managed": "bash scripts/wave-launch.sh",
+    "wave:status": "bash scripts/wave-status.sh",
+    "wave:watch": "bash scripts/wave-watch.sh --follow",
+    "wave:dashboard:current": "bash scripts/wave-dashboard-attach.sh current",
+    "wave:dashboard:global": "bash scripts/wave-dashboard-attach.sh global",
+    "harness:dev": "pnpm --filter @docx-react-component/react-word-editor-harness dev"
   }
 }

package/src/api/public-types.ts

@@ -571,16 +571,21 @@ export interface WordReviewEditorRef {
   reopenComment(commentId: string): void;
   addCommentReply(commentId: string, body: string): void;
   editCommentBody(commentId: string, body: string): void;
+  deleteComment(commentId: string): void;
   acceptChange(changeId: string): void;
   rejectChange(changeId: string): void;
   acceptAllChanges(): void;
   rejectAllChanges(): void;
   exportDocx(options?: ExportDocxOptions): Promise<ExportResult>;
   getSnapshot(): PersistedEditorSnapshot;
+  getRenderSnapshot(): RuntimeRenderSnapshot;
   getCompatibilityReport(): CompatibilityReport;
   getWarnings(): EditorWarning[];
+  getCommentSidebarSnapshot(): CommentSidebarSnapshot;
+  getTrackedChangesSnapshot(): TrackedChangesSnapshot;
   getComments(): CommentSidebarSnapshot;
   getTrackedChanges(): TrackedChangesSnapshot;
+  isDirty(): boolean;
   getFormattingState(): FormattingStateSnapshot;
   toggleBold(): void;
   toggleItalic(): void;
@@ -610,6 +615,7 @@ export interface WordReviewEditorRef {
   setCellBackground(color: string): void;
   search(query: string, options?: SearchOptions): SearchResultSnapshot[];
   clearSearch(): void;
+  setSelection(selection: SelectionSnapshot | null): void;
   scrollToRevision(revisionId: string): void;
   scrollToComment(commentId: string): void;
 }

package/src/compare/diff-engine.ts

@@ -20,6 +20,7 @@ export interface CompareVersionRef {
 export type CompareChangeKind =
   | "paragraph-insertion"
   | "paragraph-deletion"
+  | "paragraph-modification"
   | "structural-insertion"
   | "structural-deletion";
 
@@ -96,6 +97,22 @@ export function compareDocumentSnapshots(
       const insertBlock = nextTargetIndex < match.targetIndex ? targetBlocks[nextTargetIndex] : undefined;
 
       if (deleteBlock && insertBlock) {
+        if (deleteBlock.type === "paragraph" && insertBlock.type === "paragraph") {
+          const modificationId = `change-${nextChangeNumber}`;
+          nextChangeNumber += 1;
+          addParagraphModificationEntry(
+            buildEntries,
+            changes,
+            modificationId,
+            deleteBlock,
+            insertBlock,
+            nextTargetIndex,
+          );
+          nextBaseIndex += 1;
+          nextTargetIndex += 1;
+          continue;
+        }
+
         const deletionId = `change-${nextChangeNumber}`;
         nextChangeNumber += 1;
         addTrackedOrStructuralEntry(
@@ -153,13 +170,34 @@ export function compareDocumentSnapshots(
     }
 
     if (match.baseIndex < baseBlocks.length && match.targetIndex < targetBlocks.length) {
-      buildEntries.push({
-        block: cloneBlock(targetBlocks[match.targetIndex]),
-        tracked: false,
-        blockType: targetBlocks[match.targetIndex].type,
-        beforeText: getBlockDisplayText(baseBlocks[match.baseIndex]),
-        afterText: getBlockDisplayText(targetBlocks[match.targetIndex]),
-      });
+      const baseText = getBlockDisplayText(baseBlocks[match.baseIndex]);
+      const targetText = getBlockDisplayText(targetBlocks[match.targetIndex]);
+      const isModified =
+        baseBlocks[match.baseIndex].type === "paragraph" &&
+        targetBlocks[match.targetIndex].type === "paragraph" &&
+        baseText !== targetText;
+
+      if (isModified) {
+        const modChangeId = `change-${nextChangeNumber}`;
+        nextChangeNumber += 1;
+        addParagraphModificationEntry(
+          buildEntries,
+          changes,
+          modChangeId,
+          baseBlocks[match.baseIndex] as ParagraphNode,
+          targetBlocks[match.targetIndex] as ParagraphNode,
+          match.targetIndex,
+        );
+      } else {
+        buildEntries.push({
+          block: cloneBlock(targetBlocks[match.targetIndex]),
+          tracked: false,
+          blockType: targetBlocks[match.targetIndex].type,
+          beforeText: baseText,
+          afterText: targetText,
+        });
+      }
+
       nextBaseIndex = match.baseIndex + 1;
       nextTargetIndex = match.targetIndex + 1;
     }
@@ -222,6 +260,45 @@ function addTrackedOrStructuralEntry(
   });
 }
 
+function addParagraphModificationEntry(
+  buildEntries: BuildEntry[],
+  changes: CompareChange[],
+  changeId: string,
+  baseParagraph: ParagraphNode,
+  targetParagraph: ParagraphNode,
+  targetIndex: number,
+): void {
+  const beforeText = getBlockDisplayText(baseParagraph);
+  const afterText = getBlockDisplayText(targetParagraph);
+
+  buildEntries.push({
+    changeId: `${changeId}-before`,
+    block: cloneBlock(baseParagraph),
+    trackChange: "deletion",
+    tracked: true,
+    blockType: "paragraph",
+    beforeText,
+  });
+  buildEntries.push({
+    changeId: `${changeId}-after`,
+    block: cloneBlock(targetParagraph),
+    trackChange: "insertion",
+    tracked: true,
+    blockType: "paragraph",
+    afterText,
+  });
+
+  changes.push({
+    changeId,
+    kind: "paragraph-modification",
+    tracked: true,
+    blockType: "paragraph",
+    beforeText,
+    afterText,
+    targetIndex,
+  });
+}
+
 function buildComparedDocument(
   base: CanonicalDocument,
   target: CanonicalDocument,

package/src/compare/index.ts

@@ -0,0 +1,25 @@
+export {
+  compareDocumentSnapshots,
+  type CompareChange,
+  type CompareChangeKind,
+  type CompareDocumentVersionsOptions,
+  type CompareVersionRef,
+  type VersionCompareResult,
+} from "./diff-engine.ts";
+
+export {
+  exportComparedDocumentRedlines,
+  type ExportComparedDocumentOptions,
+  type ExportComparedDocumentResult,
+} from "./export-redlines.ts";
+
+export {
+  buildVersionAuditTrail,
+  createDocumentVersionSnapshot,
+  createDocumentVersionSnapshotId,
+  lockDocumentVersionSnapshot,
+  saveDocumentVersionSnapshot,
+  type CreateDocumentVersionSnapshotOptions,
+  type DocumentVersionSnapshot,
+  type VersionAuditEntry,
+} from "./snapshot.ts";

package/src/compare/snapshot.ts

@@ -53,6 +53,37 @@ export function saveDocumentVersionSnapshot(
   return next.sort(compareDocumentVersionSnapshots);
 }
 
+export interface VersionAuditEntry {
+  versionId: string;
+  name: string;
+  createdAt: string;
+  documentSignature: string;
+  action: "created" | "compared" | "locked";
+}
+
+export function buildVersionAuditTrail(
+  snapshots: readonly DocumentVersionSnapshot[],
+): VersionAuditEntry[] {
+  return snapshots.map((snapshot) => ({
+    versionId: snapshot.versionId,
+    name: snapshot.name,
+    createdAt: snapshot.createdAt,
+    documentSignature: snapshot.documentSignature,
+    action: "created",
+  }));
+}
+
+export function lockDocumentVersionSnapshot(
+  snapshot: DocumentVersionSnapshot,
+): DocumentVersionSnapshot {
+  return {
+    ...snapshot,
+    name: snapshot.name.startsWith("[locked] ")
+      ? snapshot.name
+      : `[locked] ${snapshot.name}`,
+  };
+}
+
 function compareDocumentVersionSnapshots(
   left: DocumentVersionSnapshot,
   right: DocumentVersionSnapshot,

package/src/core/selection/review-anchors.ts

@@ -76,6 +76,16 @@ export function rangeStaysWithinSingleParagraph(
     return true;
   }
 
+  const surfaceBlocks = readSurfaceBlocks(content);
+  if (surfaceBlocks) {
+    return surfaceBlocks.some(
+      (block) =>
+        block.kind === "paragraph" &&
+        normalized.from >= block.from &&
+        normalized.to <= block.to,
+    );
+  }
+
   const story = parseTextStory(content);
   const upperBound = Math.min(normalized.to, story.units.length);
 
@@ -92,3 +102,82 @@ export function rangeStaysWithinSingleParagraph(
 
   return true;
 }
+
+export function canCreateDocxCommentAnchor(
+  content: unknown,
+  anchor: ReviewAnchor,
+): boolean {
+  if (anchor.kind !== "range") {
+    return false;
+  }
+
+  const normalized = normalizeRange(anchor.range);
+  if (normalized.from === normalized.to) {
+    return false;
+  }
+
+  return rangeStaysWithinSingleParagraph(content, normalized);
+}
+
+function readSurfaceBlocks(
+  content: unknown,
+): Array<{ kind: string; from: number; to: number }> | undefined {
+  if (!content || typeof content !== "object" || !("blocks" in content)) {
+    return undefined;
+  }
+
+  const blocks = (content as { blocks?: unknown }).blocks;
+  if (!Array.isArray(blocks)) {
+    return undefined;
+  }
+
+  const normalized = flattenSurfaceBlocks(blocks);
+
+  return normalized.length > 0 ? normalized : undefined;
+}
+
+function flattenSurfaceBlocks(
+  blocks: unknown[],
+): Array<{ kind: string; from: number; to: number }> {
+  const flattened: Array<{ kind: string; from: number; to: number }> = [];
+
+  for (const block of blocks) {
+    if (
+      !block ||
+      typeof block !== "object" ||
+      typeof (block as { kind?: unknown }).kind !== "string" ||
+      typeof (block as { from?: unknown }).from !== "number" ||
+      typeof (block as { to?: unknown }).to !== "number"
+    ) {
+      continue;
+    }
+
+    flattened.push({
+      kind: (block as { kind: string }).kind,
+      from: (block as { from: number }).from,
+      to: (block as { to: number }).to,
+    });
+
+    if (
+      (block as { kind: string }).kind === "table" &&
+      Array.isArray((block as { rows?: unknown }).rows)
+    ) {
+      for (const row of (block as { rows: Array<{ cells?: unknown[] }> }).rows) {
+        for (const cell of row.cells ?? []) {
+          if (cell && typeof cell === "object" && Array.isArray((cell as { content?: unknown[] }).content)) {
+            flattened.push(...flattenSurfaceBlocks((cell as { content: unknown[] }).content));
+          }
+        }
+      }
+    }
+
+    if (
+      (block as { kind: string }).kind === "sdt_block" &&
+      Array.isArray((block as { children?: unknown[] }).children)
+    ) {
+      flattened.push(...flattenSurfaceBlocks((block as { children: unknown[] }).children));
+    }
+  }
+
+  return flattened;
+}

package/src/formats/xlsx/io/serialize-shared-strings.ts

@@ -0,0 +1,72 @@
+import type { CanonicalWorkbook } from "../model/workbook.ts";
+import { listSheets } from "../model/workbook.ts";
+
+const SHARED_STRINGS_NAMESPACE = "http://schemas.openxmlformats.org/spreadsheetml/2006/main";
+
+export interface SharedStringTableSerialization {
+  xml: string;
+  strings: string[];
+  indexByValue: Map<string, number>;
+  totalCount: number;
+}
+
+export function buildSharedStringsTable(
+  workbook: CanonicalWorkbook,
+): SharedStringTableSerialization {
+  const strings: string[] = [];
+  const indexByValue = new Map<string, number>();
+  let totalCount = 0;
+
+  for (const sheet of listSheets(workbook)) {
+    for (const cell of sheet.cells.values()) {
+      if (cell.kind !== "text") {
+        continue;
+      }
+
+      totalCount += 1;
+
+      if (!indexByValue.has(cell.value)) {
+        indexByValue.set(cell.value, strings.length);
+        strings.push(cell.value);
+      }
+    }
+  }
+
+  return {
+    xml: serializeSharedStringsXml(strings, totalCount),
+    strings,
+    indexByValue,
+    totalCount,
+  };
+}
+
+export function serializeSharedStringsXml(
+  strings: readonly string[],
+  totalCount: number = strings.length,
+): string {
+  const items = strings.map(serializeSharedStringItem);
+  const body = items.length > 0 ? `\n${items.join("\n")}\n` : "";
+
+  return [
+    `<?xml version="1.0" encoding="UTF-8" standalone="yes"?>`,
+    `<sst xmlns="${SHARED_STRINGS_NAMESPACE}" count="${totalCount}" uniqueCount="${strings.length}">${body}</sst>`,
+  ].join("\n");
+}
+
+function serializeSharedStringItem(value: string): string {
+  const preserveSpace = requiresPreservedSpace(value) ? ` xml:space="preserve"` : "";
+  return `  <si><t${preserveSpace}>${escapeXml(value)}</t></si>`;
+}
+
+function requiresPreservedSpace(value: string): boolean {
+  return /^\s/.test(value) || /\s$/.test(value);
+}
+
+function escapeXml(value: string): string {
+  return value
+    .replace(/&/g, "&amp;")
+    .replace(/"/g, "&quot;")
+    .replace(/</g, "&lt;")
+    .replace(/>/g, "&gt;")
+    .replace(/'/g, "&apos;");
+}