@adeu/core 1.18.5 → 1.19.0

package/dist/index.d.cts

@@ -196,6 +196,7 @@ declare class RedlineEngine {
      * and the edit succeeds despite it. Mirrors the Python engine.
      */
     private _check_punctuation_warning;
+    private _word_diff_sub_edits;
     /**
      * Best-effort "did you mean" hint for a failed target. The common loop trap
      * (observed in the field) is an anchored regex like `^\( x \)$` against a

package/dist/index.d.ts

@@ -196,6 +196,7 @@ declare class RedlineEngine {
      * and the edit succeeds despite it. Mirrors the Python engine.
      */
     private _check_punctuation_warning;
+    private _word_diff_sub_edits;
     /**
      * Best-effort "did you mean" hint for a failed target. The common loop trap
      * (observed in the field) is an anchored regex like `^\( x \)$` against a

package/dist/index.js

@@ -3030,6 +3030,76 @@ var RedlineEngine = class {
     }
     return null;
   }
+  _word_diff_sub_edits(target_str, new_str, base_offset, parent_comment = null, is_table = false, active_mapper = null) {
+    let raw_sub_edits = [];
+    try {
+      raw_sub_edits = generate_edits_from_text(target_str, new_str);
+      console.log("_word_diff_sub_edits RAW_SUB_EDITS:", JSON.stringify(raw_sub_edits, null, 2));
+    } catch (e) {
+      console.warn("generate_edits_from_text failed, falling back to wholesale edit", e);
+      raw_sub_edits = [];
+    }
+    if (!raw_sub_edits || raw_sub_edits.length === 0) {
+      const fallback_edit = {
+        type: "modify",
+        target_text: target_str,
+        new_text: new_str,
+        comment: parent_comment
+      };
+      fallback_edit._resolved_start_idx = base_offset;
+      fallback_edit._match_start_index = base_offset;
+      fallback_edit._active_mapper_ref = active_mapper;
+      if (is_table) {
+        fallback_edit._is_table_edit = true;
+      }
+      if (target_str === new_str) {
+        fallback_edit._internal_op = "COMMENT_ONLY";
+      } else if (!target_str && new_str) {
+        fallback_edit._internal_op = "INSERTION";
+      } else if (target_str && !new_str) {
+        fallback_edit._internal_op = "DELETION";
+      } else if (target_str && new_str) {
+        fallback_edit._internal_op = "MODIFICATION";
+      } else {
+        fallback_edit._internal_op = "COMMENT_ONLY";
+      }
+      return [fallback_edit];
+    }
+    const sub_edits = [];
+    let comment_assigned = false;
+    for (const raw_edit of raw_sub_edits) {
+      const sub_start = base_offset + (raw_edit._match_start_index || 0);
+      const should_attach_comment = parent_comment !== null && !comment_assigned;
+      if (should_attach_comment) {
+        comment_assigned = true;
+      }
+      const sub_edit = {
+        type: "modify",
+        target_text: raw_edit.target_text,
+        new_text: raw_edit.new_text,
+        comment: should_attach_comment ? parent_comment : null
+      };
+      sub_edit._resolved_start_idx = sub_start;
+      sub_edit._match_start_index = sub_start;
+      sub_edit._active_mapper_ref = active_mapper;
+      if (is_table) {
+        sub_edit._is_table_edit = true;
+      }
+      const t_val = raw_edit.target_text;
+      const n_val = raw_edit.new_text;
+      if (!t_val && n_val) {
+        sub_edit._internal_op = "INSERTION";
+      } else if (t_val && !n_val) {
+        sub_edit._internal_op = "DELETION";
+      } else if (t_val && n_val) {
+        sub_edit._internal_op = "MODIFICATION";
+      } else {
+        sub_edit._internal_op = "COMMENT_ONLY";
+      }
+      sub_edits.push(sub_edit);
+    }
+    return sub_edits;
+  }
   /**
    * Best-effort "did you mean" hint for a failed target. The common loop trap
    * (observed in the field) is an anchored regex like `^\( x \)$` against a
@@ -3959,6 +4029,8 @@ var RedlineEngine = class {
         continue;
       }
       if (!edit.target_text) continue;
+      if (edit._match_start_index !== void 0 && edit._match_start_index !== null || edit._resolved_start_idx !== void 0 && edit._resolved_start_idx !== null)
+        continue;
       const is_regex = edit.regex || false;
       const match_mode = edit.match_mode || "strict";
       let matches = this.mapper.find_all_match_indices(
@@ -4262,16 +4334,31 @@ var RedlineEngine = class {
     let applied_edits = 0;
     let skipped_edits = 0;
     if (edits.length > 0) {
+      const pinned_idx = (e) => {
+        if (e._resolved_start_idx !== void 0 && e._resolved_start_idx !== null)
+          return e._resolved_start_idx;
+        if (e._match_start_index !== void 0 && e._match_start_index !== null)
+          return e._match_start_index;
+        return null;
+      };
+      const ordered_edits = edits.map((edit, i) => ({ edit, i })).sort((a, b) => {
+        const ka = pinned_idx(a.edit);
+        const kb = pinned_idx(b.edit);
+        if (ka === null && kb === null) return a.i - b.i;
+        if (ka === null) return 1;
+        if (kb === null) return -1;
+        return kb - ka || a.i - b.i;
+      });
       if (dry_run_mode) {
-        for (let i = 0; i < edits.length; i++) {
-          const edit = edits[i];
+        const reports_by_input = new Array(edits.length);
+        for (const { edit, i } of ordered_edits) {
           const single_errors = this.validate_edits([edit], i);
           if (single_errors.length > 0) {
             skipped_edits++;
             const warning = this._check_punctuation_warning(
               edit.target_text || ""
             );
-            edits_reports.push({
+            reports_by_input[i] = {
               status: "failed",
               target_text: edit.target_text || "",
               new_text: edit.new_text || "",
@@ -4279,14 +4366,14 @@ var RedlineEngine = class {
               error: single_errors[0],
               critic_markup: null,
               clean_text: null
-            });
+            };
             continue;
           }
           const res = this.apply_edits([edit], page_offsets);
           if (edit._applied_status) {
             applied_edits++;
             const previews = this._build_edit_context_previews(edit);
-            edits_reports.push({
+            reports_by_input[i] = {
               status: "applied",
               target_text: edit.target_text || "",
               new_text: edit.new_text || "",
@@ -4298,7 +4385,7 @@ var RedlineEngine = class {
               heading_path: edit._heading_path || "",
               occurrences_modified: edit._occurrences_modified || 0,
               match_mode: edit.match_mode || "strict"
-            });
+            };
             this.mapper = new DocumentMapper(this.doc);
             this.clean_mapper = null;
           } else {
@@ -4307,7 +4394,7 @@ var RedlineEngine = class {
             const warning = this._check_punctuation_warning(
               edit.target_text || ""
             );
-            edits_reports.push({
+            reports_by_input[i] = {
               status: "failed",
               target_text: edit.target_text || "",
               new_text: edit.new_text || "",
@@ -4315,16 +4402,16 @@ var RedlineEngine = class {
               error: error_msg,
               critic_markup: null,
               clean_text: null
-            });
+            };
           }
         }
+        edits_reports.push(...reports_by_input);
       } else {
         const snapshot = takeSnapshot(this.doc);
         const originalCurrentId = this.current_id;
         try {
           const sequential_errors = [];
-          for (let i = 0; i < edits.length; i++) {
-            const edit = edits[i];
+          for (const { edit, i } of ordered_edits) {
             const single_errors = this.validate_edits([edit], i);
             if (single_errors.length > 0) {
               sequential_errors.push(...single_errors);
@@ -4492,7 +4579,8 @@ var RedlineEngine = class {
       }
       let success = false;
       if (edit.type === "modify") {
-        success = this._apply_single_edit_indexed(edit, orig_new, false);
+        const rebuild = edit._split_group_id !== void 0 && edit._split_group_id !== null;
+        success = this._apply_single_edit_indexed(edit, orig_new, rebuild);
       } else if (edit.type === "insert_row" || edit.type === "delete_row") {
         success = this._apply_table_edit(edit, false);
       }
@@ -4800,15 +4888,87 @@ var RedlineEngine = class {
         });
         continue;
       }
+      let overlaps_virtual_pipe = false;
+      if (active_mapper) {
+        overlaps_virtual_pipe = active_mapper.spans.some(
+          (s) => s.text === " | " && (s.run === null || s.run === void 0) && s.start < start_idx + match_len && s.end > start_idx
+        );
+      }
+      if (overlaps_virtual_pipe) {
+        const actual_cells = actual_doc_text.split("|");
+        const new_cells = current_effective_new_text.split("|");
+        if (actual_cells.length === new_cells.length && actual_cells.length > 1) {
+          const sub_edits2 = [];
+          let search_offset = start_idx;
+          let target_comment_idx = 0;
+          for (let idx = 0; idx < actual_cells.length; idx++) {
+            if (actual_cells[idx].trim() !== new_cells[idx].trim()) {
+              target_comment_idx = idx;
+              break;
+            }
+          }
+          for (let cell_idx = 0; cell_idx < actual_cells.length; cell_idx++) {
+            const a_cell = actual_cells[cell_idx];
+            const n_cell = new_cells[cell_idx];
+            const a_clean = a_cell.trim();
+            const n_clean = n_cell.trim();
+            let actual_start = search_offset;
+            if (a_clean) {
+              actual_start = this.mapper.full_text.indexOf(a_clean, search_offset);
+              if (actual_start === -1 || actual_start > search_offset + 10) {
+                actual_start = search_offset;
+              }
+            }
+            const should_attach_comment = edit.comment !== null && edit.comment !== void 0 && cell_idx === target_comment_idx;
+            if (a_clean !== n_clean || should_attach_comment) {
+              const cell_sub_edits = this._word_diff_sub_edits(
+                a_clean,
+                n_clean,
+                actual_start,
+                should_attach_comment ? edit.comment : null,
+                true,
+                active_mapper
+              );
+              for (const se of cell_sub_edits) {
+                se._original_target_text = edit.target_text;
+                se._split_group_id = start_idx;
+                sub_edits2.push(se);
+              }
+            }
+            if (a_clean) {
+              search_offset = actual_start + a_clean.length;
+            }
+            const next_pipe = this.mapper.full_text.indexOf(" | ", search_offset);
+            if (next_pipe !== -1 && next_pipe <= search_offset + 10) {
+              search_offset = next_pipe + 3;
+            } else {
+              search_offset += a_cell.length + 1;
+            }
+          }
+          for (const sub of sub_edits2) {
+            all_sub_edits.push(sub);
+          }
+          continue;
+        } else {
+          throw new BatchValidationError([
+            `Target text spans ${actual_cells.length} table cells, but replacement provides ${new_cells.length}. To modify text without altering table structure (rows or columns), ensure the replacement contains the exact same number of '|' separators (e.g., replace with 'CellC | ' to empty the second cell).`
+          ]);
+        }
+      }
+      let has_markdown = false;
+      if (edit.target_text && (edit.target_text.includes("**") || edit.target_text.includes("_"))) {
+        has_markdown = true;
+      }
+      if (current_effective_new_text && (current_effective_new_text.includes("**") || current_effective_new_text.includes("_"))) {
+        has_markdown = true;
+      }
       let effective_op = "";
       let final_target = "";
       let final_new = "";
       let effective_start_idx = start_idx;
       if (current_effective_new_text.startsWith(actual_doc_text)) {
         effective_op = "INSERTION";
-        final_new = current_effective_new_text.substring(
-          actual_doc_text.length
-        );
+        final_new = current_effective_new_text.substring(actual_doc_text.length);
         effective_start_idx = start_idx + match_len;
       } else {
         const [prefix_len, suffix_len] = trim_common_context(
@@ -4820,68 +4980,84 @@ var RedlineEngine = class {
         final_target = actual_doc_text.substring(prefix_len, t_end);
         final_new = current_effective_new_text.substring(prefix_len, n_end);
         effective_start_idx = start_idx + prefix_len;
-        const target_segs = actual_doc_text.split("\n\n");
-        const new_segs = current_effective_new_text.split("\n\n");
-        if (actual_doc_text.includes("\n\n") && target_segs.length === new_segs.length) {
-          const split_sub_edits = [];
-          let seg_offset = start_idx;
-          let comment_assigned = false;
-          for (let k = 0; k < target_segs.length; k++) {
-            const t_seg = target_segs[k];
-            const n_seg = new_segs[k];
-            if (t_seg !== n_seg) {
-              const [seg_prefix, seg_suffix] = trim_common_context(
-                t_seg,
-                n_seg
-              );
-              const seg_target = t_seg.substring(
-                seg_prefix,
-                t_seg.length - seg_suffix
-              );
-              const seg_new = n_seg.substring(
-                seg_prefix,
-                n_seg.length - seg_suffix
-              );
-              const seg_start = seg_offset + seg_prefix;
-              let seg_op;
-              if (!seg_target && seg_new) seg_op = "INSERTION";
-              else if (seg_target && !seg_new) seg_op = "DELETION";
-              else if (seg_target && seg_new) seg_op = "MODIFICATION";
-              else seg_op = "COMMENT_ONLY";
-              const seg_comment = edit.comment && !comment_assigned ? edit.comment : null;
-              if (seg_comment) comment_assigned = true;
-              split_sub_edits.push({
-                type: "modify",
-                target_text: seg_target,
-                new_text: seg_new,
-                comment: seg_comment,
-                _match_start_index: seg_start,
-                _internal_op: seg_op,
-                _active_mapper_ref: active_mapper,
-                _split_group_id: start_idx
-              });
+      }
+      if (has_markdown) {
+        if (!final_target && final_new) {
+          effective_op = "INSERTION";
+        } else if (final_target && !final_new) {
+          effective_op = "DELETION";
+        } else if (final_target && final_new) {
+          effective_op = "MODIFICATION";
+        } else {
+          all_sub_edits.push({
+            type: "modify",
+            target_text: final_target,
+            new_text: final_new,
+            comment: edit.comment,
+            _match_start_index: effective_start_idx,
+            _internal_op: "COMMENT_ONLY",
+            _active_mapper_ref: active_mapper
+          });
+          continue;
+        }
+        all_sub_edits.push({
+          type: "modify",
+          target_text: final_target,
+          new_text: final_new,
+          comment: edit.comment,
+          _resolved_start_idx: effective_start_idx,
+          _match_start_index: effective_start_idx,
+          _internal_op: effective_op,
+          _active_mapper_ref: active_mapper
+        });
+        continue;
+      }
+      const target_segs = actual_doc_text.split("\n\n");
+      const new_segs = current_effective_new_text.split("\n\n");
+      if (actual_doc_text.includes("\n\n") && target_segs.length === new_segs.length) {
+        const split_sub_edits = [];
+        let seg_offset = start_idx;
+        let comment_assigned = false;
+        for (let k = 0; k < target_segs.length; k++) {
+          const t_seg = target_segs[k];
+          const n_seg = new_segs[k];
+          if (t_seg !== n_seg) {
+            const seg_comment = edit.comment && !comment_assigned ? edit.comment : null;
+            const seg_sub_edits = this._word_diff_sub_edits(
+              t_seg,
+              n_seg,
+              seg_offset,
+              seg_comment,
+              false,
+              active_mapper
+            );
+            if (seg_sub_edits.some((se) => se.comment !== null && se.comment !== void 0)) {
+              comment_assigned = true;
+            }
+            for (const se of seg_sub_edits) {
+              se._split_group_id = start_idx;
+              split_sub_edits.push(se);
             }
-            seg_offset += t_seg.length + 2;
-          }
-          if (split_sub_edits.length > 0) {
-            for (const sub of split_sub_edits) all_sub_edits.push(sub);
-            continue;
           }
+          seg_offset += t_seg.length + 2;
+        }
+        if (split_sub_edits.length > 0) {
+          for (const sub of split_sub_edits) all_sub_edits.push(sub);
+          continue;
         }
-        if (!final_target && final_new) effective_op = "INSERTION";
-        else if (final_target && !final_new) effective_op = "DELETION";
-        else if (final_target && final_new) effective_op = "MODIFICATION";
-        else effective_op = "COMMENT_ONLY";
       }
-      all_sub_edits.push({
-        type: "modify",
-        target_text: final_target,
-        new_text: final_new,
-        comment: edit.comment,
-        _match_start_index: effective_start_idx,
-        _internal_op: effective_op,
-        _active_mapper_ref: active_mapper
-      });
+      const sub_edits = this._word_diff_sub_edits(
+        actual_doc_text,
+        current_effective_new_text,
+        start_idx,
+        edit.comment,
+        false,
+        active_mapper
+      );
+      for (const se of sub_edits) {
+        se._split_group_id = start_idx;
+        all_sub_edits.push(se);
+      }
     }
     if (all_sub_edits.length === 0) return null;
     if (match_mode === "all" || all_sub_edits.length > 1) return all_sub_edits;
@@ -4895,7 +5071,7 @@ var RedlineEngine = class {
    * our replacement <w:ins> lands as a sibling, so we never nest <w:ins> in
    * <w:ins>.
    */
-  _insert_and_split_ins(parent_ins, split_after, new_elem) {
+  _insert_and_split_ins(parent_ins, anchor, new_elem, split_before = false) {
     const grandparent = parent_ins.parentNode;
     if (!grandparent) return;
     const left = parent_ins.cloneNode(false);
@@ -4903,9 +5079,10 @@ var RedlineEngine = class {
     let toRight = false;
     for (const kid of Array.from(parent_ins.childNodes)) {
       parent_ins.removeChild(kid);
+      if (split_before && kid === anchor) toRight = true;
       if (!toRight) {
         left.appendChild(kid);
-        if (kid === split_after) toRight = true;
+        if (kid === anchor) toRight = true;
       } else {
         right.appendChild(kid);
       }
@@ -4921,6 +5098,15 @@ var RedlineEngine = class {
     const active_mapper = edit._active_mapper_ref || this.mapper;
     const start_idx = edit._resolved_start_idx !== void 0 && edit._resolved_start_idx !== null ? edit._resolved_start_idx : edit._match_start_index || 0;
     const length = edit.target_text ? edit.target_text.length : 0;
+    if (op === void 0 || op === null) {
+      if (!edit.target_text && edit.new_text) {
+        op = "INSERTION";
+      } else if (edit.target_text && !edit.new_text) {
+        op = "DELETION";
+      } else {
+        op = "MODIFICATION";
+      }
+    }
     if (op === "STYLE_ONLY" || op === "STYLE_AND_TEXT") {
       const [anchor_run, anchor_para] = active_mapper.get_insertion_anchor(
         start_idx,
@@ -5124,17 +5310,26 @@ var RedlineEngine = class {
       if (is_inline_first) {
         if (anchor_run) {
           const anchor_parent = anchor_run._element.parentNode;
+          const before_anchor = start_idx === 0;
           if (anchor_parent && anchor_parent.tagName === "w:ins") {
             this._insert_and_split_ins(
               anchor_parent,
               anchor_run._element,
-              result.first_node
+              result.first_node,
+              before_anchor
             );
+          } else if (before_anchor && anchor_parent) {
+            anchor_parent.insertBefore(result.first_node, anchor_run._element);
           } else {
             insertAfter(result.first_node, anchor_run._element);
           }
         } else if (anchor_para) {
-          anchor_para._element.appendChild(result.first_node);
+          const para_el = anchor_para._element;
+          let ref = para_el.firstChild;
+          while (ref && ref.tagName === "w:pPr") {
+            ref = ref.nextSibling;
+          }
+          para_el.insertBefore(result.first_node, ref);
         }
       }
       if (edit.comment) {