chore: Improve patch tool tolerance and reporting

Refactor `patch.tool.js` to be more robust against whitespace differences in patch markers and context lines. Also, enhance the reporting to detail how many segments were applied. Update version to 0.1.13. Added test files.

Author: iovdin

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tune-basic-toolset",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "Basic toolset for tune",
   "main": "src/index.js",
   "files": [

src/patch.tool.js

@@ -1,20 +1,21 @@
 const fs = require('fs').promises;
 
 // Patch tool to apply custom diffs marked with <<<<<<< ORIGINAL and >>>>>>> UPDATED
-// Handles patches with context and applies only the segments between markers.
-
+// More tolerant to whitespace differences on each line and reports per-block success.
 module.exports = async function patch({ text, filename }, ctx) {
-  // Regex to match each patch block
-  // Be lenient about the number of conflict marker characters because some
-  // environments may trim one or more > or < characters.
-  const patchRegex = /<{6,}\s*ORIGINAL[^\n]*\n([\s\S]*?)=+\n([\s\S]*?)>{6,}\s*UPDATED[^\n]*(?:\n|$)/g;
-  const patches = [];
-  let match;
+  if (!text || !filename) {
+    return "No patch text or filename provided";
+  }
 
-  // Extract all old/new segments
-  while ((match = patchRegex.exec(text)) !== null) {
-    const oldPart = match[1].replace(/^\n+|\n+$/g, "");
-    const newPart = match[2].replace(/^\n+|\n+$/g, "");
+  // Match: <<<<<<< ORIGINAL ... ======= ... >>>>>>> UPDATED
+  // Be tolerant to CRLF/LF and optional trailing text/spaces on the markers.
+  const patchRegex = /<{6,}\s*ORIGINAL[^\n]*\r?\n([\s\S]*?)=+[^\n]*\r?\n([\s\S]*?)>{6,}\s*UPDATED[^\n]*(?:\r?\n|$)/g;
+
+  const patches = [];
+  let m;
+  while ((m = patchRegex.exec(text)) !== null) {
+    const oldPart = String(m[1]).replace(/^\s*\r?\n+|\r?\n+\s*$/g, "");
+    const newPart = String(m[2]).replace(/^\s*\r?\n+|\r?\n+\s*$/g, "");
     patches.push({ oldPart, newPart });
   }
 
@@ -24,18 +25,41 @@ module.exports = async function patch({ text, filename }, ctx) {
 
   let fileContent = await ctx.read(filename);
 
-  for (const { oldPart, newPart } of patches) {
-    // Escape regex special chars in oldPart.
-    // Do NOT relax all whitespace to \s+; that can swallow preceding newlines.
-    // Only normalize line endings so CRLF in patches can match LF in files.
-    let escaped = oldPart.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  function buildPattern(oldStr) {
+    // Escape special regex chars
+    let escaped = oldStr.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+    // Normalize line endings to \r?\n so CRLF/LF both match
     escaped = escaped.replace(/\r?\n/g, "\\r?\\n");
-    const oldRegex = new RegExp(escaped, "g");
+    // Tolerate indentation/space differences (spaces or tabs), zero-or-more
+    // Keep newlines strict so structure must still match.
+    escaped = escaped.replace(/[ \t]+/g, "[ \\t]*");
+    return new RegExp(escaped, "g");
+  }
+
+  const totalSegments = patches.length;
+  let appliedSegments = 0;
+  let totalReplacements = 0;
 
-    // Perform replacement using a function to avoid replacement string ambiguities
-    fileContent = fileContent.replace(oldRegex, () => newPart);
+  for (const { oldPart, newPart } of patches) {
+    const re = buildPattern(oldPart);
+    let matches = 0;
+    fileContent = fileContent.replace(re, () => {
+      matches += 1;
+      return newPart;
+    });
+    if (matches > 0) {
+      appliedSegments += 1;
+      totalReplacements += matches;
+    }
   }
 
   await ctx.write(filename, fileContent);
-  return "patched";
+
+  if (appliedSegments === 0) {
+    return `no matches applied (0/${totalSegments})`;
+  }
+  if (appliedSegments < totalSegments) {
+    return `patched partially (${appliedSegments}/${totalSegments}), replacements: ${totalReplacements}`;
+  }
+  return `patched (${appliedSegments}/${totalSegments}), replacements: ${totalReplacements}`;
 };

test/p14.txt

@@ -0,0 +1,13 @@
+a()
+  b()
+-------------------------------
+<<<<<<< ORIGINAL
+a()
+	b()
+=======
+A
+B
+>>>>>>> UPDATED
+-------------------------------
+A
+B

test/p15.txt

@@ -0,0 +1,13 @@
+foo(
+)
+-------------------------------
+<<<<<<< ORIGINAL
+foo(    
+)
+=======
+X
+Y
+>>>>>>> UPDATED
+-------------------------------
+X
+Y

test/p16.txt

@@ -0,0 +1,18 @@
+aaa
+bbb
+ccc
+-------------------------------
+<<<<<<< ORIGINAL
+bbb
+=======
+B
+>>>>>>> UPDATED
+<<<<<<< ORIGINAL
+xxx
+=======
+X
+>>>>>>> UPDATED
+-------------------------------
+aaa
+B
+ccc