@adeu/core 1.18.0 → 1.18.1

package/dist/index.d.cts

@@ -185,6 +185,16 @@ declare class RedlineEngine {
     original_mapper: DocumentMapper | null;
     skipped_details: string[];
     constructor(doc: DocumentObject, author?: string);
+    /**
+     * Return a hint when a short, single-token anchor contains punctuation that
+     * can split awkwardly, else null.
+     *
+     * Surface this ONLY for edits that actually failed to match/apply. On a
+     * successful edit the batch report already carries the redline preview, so
+     * emitting this would be a false positive: the punctuation (dates,
+     * `[_name_]` placeholders, `____` blanks) is frequently the literal target
+     * and the edit succeeds despite it. Mirrors the Python engine.
+     */
     private _check_punctuation_warning;
     /**
      * Best-effort "did you mean" hint for a failed target. The common loop trap

package/dist/index.d.ts

@@ -185,6 +185,16 @@ declare class RedlineEngine {
     original_mapper: DocumentMapper | null;
     skipped_details: string[];
     constructor(doc: DocumentObject, author?: string);
+    /**
+     * Return a hint when a short, single-token anchor contains punctuation that
+     * can split awkwardly, else null.
+     *
+     * Surface this ONLY for edits that actually failed to match/apply. On a
+     * successful edit the batch report already carries the redline preview, so
+     * emitting this would be a false positive: the punctuation (dates,
+     * `[_name_]` placeholders, `____` blanks) is frequently the literal target
+     * and the edit succeeds despite it. Mirrors the Python engine.
+     */
     private _check_punctuation_warning;
     /**
      * Best-effort "did you mean" hint for a failed target. The common loop trap

package/dist/index.js

@@ -3012,8 +3012,19 @@ var RedlineEngine = class {
     this.mapper = new DocumentMapper(this.doc);
     this.comments_manager = new CommentsManager(this.doc);
   }
+  /**
+   * Return a hint when a short, single-token anchor contains punctuation that
+   * can split awkwardly, else null.
+   *
+   * Surface this ONLY for edits that actually failed to match/apply. On a
+   * successful edit the batch report already carries the redline preview, so
+   * emitting this would be a false positive: the punctuation (dates,
+   * `[_name_]` placeholders, `____` blanks) is frequently the literal target
+   * and the edit succeeds despite it. Mirrors the Python engine.
+   */
   _check_punctuation_warning(target_text) {
     if (!target_text) return null;
+    if (target_text.length > 20 || target_text.includes(" ")) return null;
     if (target_text.includes("_") || target_text.includes("-")) {
       return `Warning: target_text '${target_text}' contains tokenization-splitting punctuation ('_' or '-'). This can trigger mid-word splits in the diff engine. Consider using a longer plain-prose anchor.`;
     }
@@ -4230,11 +4241,11 @@ var RedlineEngine = class {
         for (let i = 0; i < edits.length; i++) {
           const edit = edits[i];
           const single_errors = this.validate_edits([edit], i);
-          const warning = this._check_punctuation_warning(
-            edit.target_text || ""
-          );
           if (single_errors.length > 0) {
             skipped_edits++;
+            const warning = this._check_punctuation_warning(
+              edit.target_text || ""
+            );
             edits_reports.push({
               status: "failed",
               target_text: edit.target_text || "",
@@ -4254,7 +4265,7 @@ var RedlineEngine = class {
               status: "applied",
               target_text: edit.target_text || "",
               new_text: edit.new_text || "",
-              warning,
+              warning: null,
               error: null,
               critic_markup: previews[0],
               clean_text: previews[1],
@@ -4268,6 +4279,9 @@ var RedlineEngine = class {
           } else {
             skipped_edits++;
             const error_msg = this.skipped_details.length > 0 ? this.skipped_details[this.skipped_details.length - 1] : "Failed to apply edit";
+            const warning = this._check_punctuation_warning(
+              edit.target_text || ""
+            );
             edits_reports.push({
               status: "failed",
               target_text: edit.target_text || "",
@@ -4313,15 +4327,17 @@ var RedlineEngine = class {
         for (const edit of edits) {
           const success = edit._applied_status || false;
           const error_msg = edit._error_msg || null;
-          const warning = this._check_punctuation_warning(
-            edit.target_text || ""
-          );
           let critic_markup = null;
           let clean_text = null;
+          let warning = null;
           if (success) {
             const previews = this._build_edit_context_previews(edit);
             critic_markup = previews[0];
             clean_text = previews[1];
+          } else {
+            warning = this._check_punctuation_warning(
+              edit.target_text || ""
+            );
           }
           edits_reports.push({
             status: success ? "applied" : "failed",