@proxysoul/soulforge 1.3.8 → 1.4.0

package/dist/index.js

@@ -57814,7 +57814,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@proxysoul/soulforge",
-    version: "1.3.8",
+    version: "1.4.0",
     description: "Graph-powered code intelligence \u2014 multi-agent coding with codebase-aware AI",
     repository: {
       type: "git",
@@ -62243,7 +62243,6 @@ function markToolWrite(filePath) {
 async function readBufferContent(filePath) {
   const toolWriteTime = recentToolWrites.get(filePath);
   if (toolWriteTime && Date.now() - toolWriteTime < TOOL_WRITE_FRESHNESS_MS) {
-    recentToolWrites.delete(filePath);
     return readFile3(filePath, "utf-8");
   }
   inflight++;
@@ -317553,7 +317552,7 @@ function buildRichEditError(content, oldStr, lineHint) {
   const escapeHint = backslashDensity > 0.05 ? `
 [Escape-heavy content detected \u2014 use lineStart for line-based replacement, or use editor(action: edit, startLine, endLine, replacement)]` : "";
   return {
-    output: `old_string not found in file (re-read performed \u2014 content below is current):
+    output: `old_string not found in file. Current content at that region:
 ${snippet}${escapeHint}`
   };
 }
@@ -317713,7 +317712,7 @@ var init_edit_file = __esm(() => {
   init_file_events();
   editFileTool = {
     name: "edit_file",
-    description: "[TIER-1] Edit a file by replacing content. Read first, then provide path, oldString, newString. " + "Always provide lineStart (1-indexed from read_file output) \u2014 the range is derived from oldString line count. " + "Empty oldString creates a new file. Use multi_edit for multiple changes to the same file. Edits are applied immediately.",
+    description: "[TIER-1] Edit a file by replacing content. Read first, then provide path, oldString, newString. " + "Provide lineStart (1-indexed from read_file output) for reliable line-anchored matching \u2014 " + "the range is derived from oldString line count. Without lineStart, falls back to string matching (fails if ambiguous). Empty oldString creates a new file. Use multi_edit for multiple changes to the same file. Edits are applied immediately.",
     execute: async (args2) => {
       try {
         const filePath = resolve13(args2.path);
@@ -334637,7 +334636,7 @@ var init_multi_edit = __esm(() => {
   init_file_events();
   multiEditTool = {
     name: "multi_edit",
-    description: "Apply multiple edits to a single file atomically. All-or-nothing validation. " + "Each edit's oldString and lineStart reference the ORIGINAL file \u2014 the tool handles offset tracking internally. " + "Always provide lineStart (1-indexed). The range is derived from oldString line count.",
+    description: "Apply multiple edits to a single file atomically. All-or-nothing: if any edit fails, ZERO edits are applied. " + "lineStart values reference the ORIGINAL file (pre-edit) \u2014 the tool tracks cumulative line offsets internally. " + "Provide lineStart (1-indexed) for reliable line-anchored matching. Without it, falls back to string matching against evolved content. " + "The range is derived from oldString line count.",
     execute: async (args2) => {
       try {
         const filePath = resolve19(args2.path);
@@ -334681,7 +334680,6 @@ var init_multi_edit = __esm(() => {
           return a.lineStart - b.lineStart;
         });
         let lineOffset = 0;
-        const warnings = [];
         for (let i2 = 0;i2 < sortedEdits.length; i2++) {
           const edit = sortedEdits[i2];
           if (!edit)
@@ -334728,21 +334726,21 @@ var init_multi_edit = __esm(() => {
 `);
               return {
                 success: false,
-                output: `Edit ${String(i2 + 1)}: oldString does not match lines ${String(edit.lineStart)}-${String((edit.lineStart ?? 0) + oldLineCount - 1)}. Actual content:
+                output: `Edit ${String(i2 + 1)}/${String(args2.edits.length)} failed: oldString does not match lines ${String(edit.lineStart)}-${String((edit.lineStart ?? 0) + oldLineCount - 1)}. NO edits were applied (atomic rollback). Actual content at that range:
 ${rangeSnippet}
-Re-read the file and retry with the correct content.`,
-                error: `edit ${String(i2 + 1)}: oldString mismatch at line range`
+Re-read the file and retry ALL edits.`,
+                error: `edit ${String(i2 + 1)}: oldString mismatch at line range (0 edits applied)`
               };
             }
           }
           if (content.includes(edit.oldString)) {
             const occurrences = content.split(edit.oldString).length - 1;
             if (occurrences > 1) {
-              const msg = `${label}: found ${String(occurrences)} matches. Provide lineStart to disambiguate.`;
+              const msg = `${label}: found ${String(occurrences)} matches. Provide lineStart to disambiguate. NO edits were applied (atomic rollback).`;
               return {
                 success: false,
                 output: msg,
-                error: msg
+                error: `${label}: ambiguous match (0 edits applied)`
               };
             }
             const idx = content.indexOf(edit.oldString);
@@ -334767,8 +334765,9 @@ Re-read the file and retry with the correct content.`,
           const err2 = buildRichEditError(content, edit.oldString, adjustedLineStart);
           return {
             success: false,
-            output: `${label} failed: ${err2.output}`,
-            error: `edit ${String(i2 + 1)} failed`
+            output: `${label} failed: ${err2.output}
+NO edits were applied (atomic rollback). Re-read the file and retry ALL edits.`,
+            error: `edit ${String(i2 + 1)} failed (0 edits applied)`
           };
         }
         const beforeMetrics = analyzeFile(originalContent);
@@ -334794,11 +334793,11 @@ Re-read the file and retry with the correct content.`,
         }).catch(() => null);
         const currentOnDisk = await readFile13(filePath, "utf-8");
         if (currentOnDisk !== originalContent) {
-          const msg = "File was modified concurrently since last read. Re-read and retry.";
+          const msg = "File was modified concurrently since last read. NO edits were applied (atomic rollback). Re-read and retry ALL edits.";
           return {
             success: false,
             output: msg,
-            error: "concurrent modification"
+            error: "concurrent modification (0 edits applied)"
           };
         }
         pushEdit(filePath, originalContent, content, args2.tabId);
@@ -334818,10 +334817,6 @@ Re-read the file and retry with the correct content.`,
           deltas.push(`imports: ${String(beforeMetrics.importCount)}\u2192${String(afterMetrics.importCount)} (${sign}${String(importDelta)})`);
         }
         let output = `Applied ${String(args2.edits.length)} edits to ${args2.path}`;
-        if (warnings.length > 0)
-          output += `
-\u26A0 ${warnings.join(`
-\u26A0 `)}`;
         if (deltas.length > 0)
           output += ` (${deltas.join(", ")})`;
         const formatted = await autoFormatAfterEdit(filePath);
@@ -474859,8 +474854,6 @@ var init_SkillSearch = __esm(async () => {
   SkillSearch = import_react115.memo(function SkillSearch2({
     visible,
     contextManager,
-    agentSkillsEnabled,
-    onToggleAgentSkills,
     onClose,
     onSystemMessage
   }) {
@@ -475040,10 +475033,6 @@ var init_SkillSearch = __esm(async () => {
         resetScroll();
         return;
       }
-      if (evt.name === "a" && !evt.ctrl && !evt.meta && !query2) {
-        onToggleAgentSkills();
-        return;
-      }
       if (evt.name === "space") {
         setQuery((prev_3) => `${prev_3} `);
         resetScroll();
@@ -475469,32 +475458,6 @@ var init_SkillSearch = __esm(async () => {
               children: ""
             }, undefined, false, undefined, this)
           }, undefined, false, undefined, this),
-          /* @__PURE__ */ import_jsx_dev_runtime2.jsxDEV(PopupRow, {
-            w: innerW,
-            children: [
-              /* @__PURE__ */ import_jsx_dev_runtime2.jsxDEV("text", {
-                fg: t2.textMuted,
-                bg: POPUP_BG,
-                children: [
-                  "Agent access:",
-                  " "
-                ]
-              }, undefined, true, undefined, this),
-              /* @__PURE__ */ import_jsx_dev_runtime2.jsxDEV("text", {
-                fg: agentSkillsEnabled ? t2.success : t2.error,
-                bg: POPUP_BG,
-                children: agentSkillsEnabled ? "on" : "off"
-              }, undefined, false, undefined, this),
-              /* @__PURE__ */ import_jsx_dev_runtime2.jsxDEV("text", {
-                fg: t2.textDim,
-                bg: POPUP_BG,
-                children: [
-                  " ",
-                  "(a to toggle)"
-                ]
-              }, undefined, true, undefined, this)
-            ]
-          }, undefined, true, undefined, this),
           /* @__PURE__ */ import_jsx_dev_runtime2.jsxDEV(PopupRow, {
             w: innerW,
             children: /* @__PURE__ */ import_jsx_dev_runtime2.jsxDEV("text", {
@@ -477061,11 +477024,6 @@ function App({
       /* @__PURE__ */ import_jsx_dev_runtime2.jsxDEV(SkillSearch, {
         visible: modalSkillSearch,
         contextManager: tabMgr.getActiveChat()?.contextManager ?? contextManager,
-        agentSkillsEnabled: !toolsState.disabledTools.has("skills"),
-        onToggleAgentSkills: () => {
-          toolsState.toggleTool("skills");
-          addSystemMessage(`Agent skills ${toolsState.disabledTools.has("skills") ? "enabled" : "disabled"}`);
-        },
         onClose: getCloser2("skillSearch"),
         onSystemMessage: addSystemMessage
       }, undefined, false, undefined, this),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@proxysoul/soulforge",
-  "version": "1.3.8",
+  "version": "1.4.0",
   "description": "Graph-powered code intelligence — multi-agent coding with codebase-aware AI",
   "repository": {
     "type": "git",