@adeu/core 1.26.0 → 1.27.0

package/dist/index.js

@@ -2259,7 +2259,7 @@ ${header}`;
     let dom_modified = false;
     const first_real_span = real_spans[0];
     let start_split_adjustment = 0;
-    const local_start = start_idx - first_real_span.start + (first_real_span.run_offset || 0);
+    const local_start = Math.max(start_idx, first_real_span.start) - first_real_span.start + (first_real_span.run_offset || 0);
     if (local_start > 0) {
       const split_source = working_runs[0];
       const [, right_run] = this._split_run_at_index(
@@ -4809,7 +4809,7 @@ var RedlineEngine = class _RedlineEngine {
    *
    * Does NOT attach comments; callers handle that.
    */
-  _track_insert_multiline(text, anchor_run, anchor_paragraph, reuse_id, positional_anchor_el = null, suppress_emphasis = false) {
+  _track_insert_multiline(text, anchor_run, anchor_paragraph, reuse_id, positional_anchor_el = null, suppress_emphasis = false, insert_before = false) {
     if (!text) {
       return {
         first_node: null,
@@ -4831,7 +4831,11 @@ var RedlineEngine = class _RedlineEngine {
       current_p = walker;
     }
     const suffix_nodes = [];
-    const pos_source = positional_anchor_el && positional_anchor_el.parentNode ? positional_anchor_el : anchor_run !== null && anchor_run._element.parentNode ? anchor_run._element : null;
+    const relocatable = /* @__PURE__ */ new Set(["w:r", "w:ins", "w:del"]);
+    const pos_from_positional = positional_anchor_el && positional_anchor_el.parentNode ? positional_anchor_el : null;
+    const pos_from_anchor_run = anchor_run !== null && anchor_run._element.parentNode ? anchor_run._element : null;
+    const pos_source = pos_from_positional ?? pos_from_anchor_run;
+    const suffix_includes_anchor = insert_before && pos_from_positional === null && pos_from_anchor_run !== null;
     if (current_p !== null && pos_source !== null) {
       let pos_anchor = pos_source;
       while (pos_anchor && pos_anchor.parentNode !== current_p) {
@@ -4842,8 +4846,7 @@ var RedlineEngine = class _RedlineEngine {
         }
       }
       if (pos_anchor) {
-        const relocatable = /* @__PURE__ */ new Set(["w:r", "w:ins", "w:del"]);
-        let nxt = pos_anchor.nextSibling;
+        let nxt = suffix_includes_anchor ? pos_anchor : pos_anchor.nextSibling;
         while (nxt) {
           if (nxt.nodeType === 1 && relocatable.has(nxt.tagName)) {
             suffix_nodes.push(nxt);
@@ -4851,6 +4854,14 @@ var RedlineEngine = class _RedlineEngine {
           nxt = nxt.nextSibling;
         }
       }
+    } else if (current_p !== null && insert_before) {
+      let child = current_p.firstChild;
+      while (child) {
+        if (child.nodeType === 1 && relocatable.has(child.tagName)) {
+          suffix_nodes.push(child);
+        }
+        child = child.nextSibling;
+      }
     }
     while (lines.length > 1 && lines[lines.length - 1] === "" && suffix_nodes.length === 0) {
       lines.pop();
@@ -5589,6 +5600,45 @@ var RedlineEngine = class _RedlineEngine {
   }
   validate_review_actions(actions) {
     const errors = [];
+    const seen_resolutions = /* @__PURE__ */ new Map();
+    const seen_replies = /* @__PURE__ */ new Set();
+    for (let i = 0; i < actions.length; i++) {
+      const action = actions[i];
+      const type = action.type;
+      const target_id = action.target_id ?? "";
+      if (type === "reply") {
+        if (!String(action.text ?? "").trim()) {
+          errors.push(
+            `- Action ${i + 1} Failed: reply text for ${target_id} is empty or whitespace-only. Word would show a blank comment bubble \u2014 provide the reply content in 'text'.`
+          );
+          continue;
+        }
+        const reply_key = `${target_id}\0${String(action.text).trim()}`;
+        if (seen_replies.has(reply_key)) {
+          errors.push(
+            `- Action ${i + 1} Failed: duplicate reply \u2014 this batch already replies to ${target_id} with the same text. Remove the duplicate action.`
+          );
+        }
+        seen_replies.add(reply_key);
+      } else if (type === "accept" || type === "reject") {
+        const prior = seen_resolutions.get(target_id);
+        if (prior !== void 0) {
+          const [first_idx, first_type] = prior;
+          if (first_type === type) {
+            errors.push(
+              `- Action ${i + 1} Failed: duplicate action \u2014 Action ${first_idx + 1} in this batch already applies '${type}' to ${target_id}. A change can only be resolved once; remove the duplicate action.`
+            );
+          } else {
+            errors.push(
+              `- Action ${i + 1} Failed: conflicting actions \u2014 Action ${first_idx + 1} in this batch applies '${first_type}' to ${target_id}, but this action applies '${type}'. Decide the outcome and keep exactly one of them.`
+            );
+          }
+        } else {
+          seen_resolutions.set(target_id, [i, type]);
+        }
+      }
+    }
+    if (errors.length > 0) return errors;
     for (let i = 0; i < actions.length; i++) {
       const action = actions[i];
       const type = action.type;
@@ -6836,7 +6886,10 @@ var RedlineEngine = class _RedlineEngine {
         anchor_para,
         ins_id,
         null,
-        suppress_emphasis
+        suppress_emphasis,
+        // Paragraph-start insertions attach BEFORE the anchor (see
+        // before_anchor below): the suffix relocation must know.
+        start_idx === 0
       );
       if (!result.first_node) return false;
       const is_inline_first = result.first_node.tagName === "w:ins";