@adeu/core 1.18.2 → 1.18.4

package/dist/index.js

@@ -4119,17 +4119,18 @@ var RedlineEngine = class {
             }
           }
         }
-        if (insAuthors.size > 0 || commentAuthors.size > 0) {
-          const fullyWithinForeignIns = insAuthors.size > 0 && !hasNonForeignRealText && commentAuthors.size === 0;
-          if ((match_mode === "strict" || match_mode === "first") && fullyWithinForeignIns) {
+        if (insAuthors.size > 0) {
+          const fullyWithinForeignIns = !hasNonForeignRealText;
+          if (!((match_mode === "strict" || match_mode === "first") && fullyWithinForeignIns)) {
+            errors.push(
+              `- Edit ${i + 1 + index_offset} Failed: Modification targets an active insertion from another author (${Array.from(insAuthors).join(", ")}). Accept that change first or scope your edit outside of it.`
+            );
             continue;
           }
-          const nestedAuthors = /* @__PURE__ */ new Set([
-            ...insAuthors,
-            ...commentAuthors
-          ]);
+        }
+        if (commentAuthors.size > 0 && match_mode === "all") {
           errors.push(
-            `- Edit ${i + 1 + index_offset} Failed: Modification targets an active insertion from another author (${Array.from(nestedAuthors).join(", ")}). Accept that change first or scope your edit outside of it.`
+            `- Edit ${i + 1 + index_offset} Failed: match_mode="all" would sweep through a comment range from another author (${Array.from(commentAuthors).join(", ")}). Target the commented text deliberately with match_mode "strict" or "first", or scope your edit outside of it.`
           );
         }
       }