@robota-sdk/agent-tools 3.0.0-beta.56 → 3.0.0-beta.57

package/README.md

@@ -57,6 +57,10 @@ const agent = new Robota({
 | `webFetchTool`  | WebFetch  | Fetch URL content (HTML-to-text conversion)      |
 | `webSearchTool` | WebSearch | Web search via Brave Search API                  |
 
+## Edit and Write Safety
+
+Recent file tool updates keep write/edit behavior atomic and make Edit tool results easier for higher layers to display. The Edit tool returns line metadata for changed regions, allowing the CLI to render concise context hunks instead of dumping full files or opaque summaries.
+
 ## Tool Infrastructure
 
 | Export                  | Description                                            |

package/dist/node/index.cjs

@@ -1031,9 +1031,33 @@ var readTool = createZodFunctionTool(
 );
 
 // src/builtins/write-tool.ts
+var import_zod3 = require("zod");
+
+// src/builtins/atomic-file-write.ts
+var import_node_crypto = require("crypto");
 var import_promises2 = require("fs/promises");
 var import_node_path = require("path");
-var import_zod3 = require("zod");
+var TEMP_RANDOM_BYTES = 6;
+function createTempFilePath(filePath) {
+  const dir = (0, import_node_path.dirname)(filePath);
+  const name = (0, import_node_path.basename)(filePath);
+  const suffix = (0, import_node_crypto.randomBytes)(TEMP_RANDOM_BYTES).toString("hex");
+  return (0, import_node_path.join)(dir, `.${name}.robota-tmp-${process.pid}-${Date.now()}-${suffix}`);
+}
+async function atomicWriteUtf8File(filePath, content) {
+  const dir = (0, import_node_path.dirname)(filePath);
+  await (0, import_promises2.mkdir)(dir, { recursive: true });
+  const tempFilePath = createTempFilePath(filePath);
+  try {
+    await (0, import_promises2.writeFile)(tempFilePath, content, "utf8");
+    await (0, import_promises2.rename)(tempFilePath, filePath);
+  } catch (error) {
+    await (0, import_promises2.rm)(tempFilePath, { force: true }).catch(() => void 0);
+    throw error;
+  }
+}
+
+// src/builtins/write-tool.ts
 var WriteSchema = import_zod3.z.object({
   filePath: import_zod3.z.string().describe("The absolute path to the file to write"),
   content: import_zod3.z.string().describe("The content to write to the file")
@@ -1041,8 +1065,7 @@ var WriteSchema = import_zod3.z.object({
 async function writeFileTool(args) {
   const { filePath, content } = args;
   try {
-    await (0, import_promises2.mkdir)((0, import_node_path.dirname)(filePath), { recursive: true });
-    await (0, import_promises2.writeFile)(filePath, content, "utf8");
+    await atomicWriteUtf8File(filePath, content);
     const result = {
       success: true,
       output: `Written ${Buffer.byteLength(content, "utf8")} bytes to ${filePath}`
@@ -1113,7 +1136,7 @@ async function editFileTool(args) {
   }
   const updated = replaceAll ? content.split(oldString).join(newString) : content.slice(0, content.indexOf(oldString)) + newString + content.slice(content.indexOf(oldString) + oldString.length);
   try {
-    await (0, import_promises3.writeFile)(filePath, updated, "utf8");
+    await atomicWriteUtf8File(filePath, updated);
   } catch (err) {
     const result2 = {
       success: false,

package/dist/node/index.js

@@ -981,9 +981,33 @@ var readTool = createZodFunctionTool(
 );
 
 // src/builtins/write-tool.ts
-import { writeFile, mkdir } from "fs/promises";
-import { dirname } from "path";
 import { z as z3 } from "zod";
+
+// src/builtins/atomic-file-write.ts
+import { randomBytes } from "crypto";
+import { mkdir, rename, rm, writeFile } from "fs/promises";
+import { basename, dirname, join } from "path";
+var TEMP_RANDOM_BYTES = 6;
+function createTempFilePath(filePath) {
+  const dir = dirname(filePath);
+  const name = basename(filePath);
+  const suffix = randomBytes(TEMP_RANDOM_BYTES).toString("hex");
+  return join(dir, `.${name}.robota-tmp-${process.pid}-${Date.now()}-${suffix}`);
+}
+async function atomicWriteUtf8File(filePath, content) {
+  const dir = dirname(filePath);
+  await mkdir(dir, { recursive: true });
+  const tempFilePath = createTempFilePath(filePath);
+  try {
+    await writeFile(tempFilePath, content, "utf8");
+    await rename(tempFilePath, filePath);
+  } catch (error) {
+    await rm(tempFilePath, { force: true }).catch(() => void 0);
+    throw error;
+  }
+}
+
+// src/builtins/write-tool.ts
 var WriteSchema = z3.object({
   filePath: z3.string().describe("The absolute path to the file to write"),
   content: z3.string().describe("The content to write to the file")
@@ -991,8 +1015,7 @@ var WriteSchema = z3.object({
 async function writeFileTool(args) {
   const { filePath, content } = args;
   try {
-    await mkdir(dirname(filePath), { recursive: true });
-    await writeFile(filePath, content, "utf8");
+    await atomicWriteUtf8File(filePath, content);
     const result = {
       success: true,
       output: `Written ${Buffer.byteLength(content, "utf8")} bytes to ${filePath}`
@@ -1017,7 +1040,7 @@ var writeTool = createZodFunctionTool(
 );
 
 // src/builtins/edit-tool.ts
-import { readFile as readFile2, writeFile as writeFile2 } from "fs/promises";
+import { readFile as readFile2 } from "fs/promises";
 import { z as z4 } from "zod";
 var EditSchema = z4.object({
   filePath: z4.string().describe("The absolute path to the file to modify"),
@@ -1063,7 +1086,7 @@ async function editFileTool(args) {
   }
   const updated = replaceAll ? content.split(oldString).join(newString) : content.slice(0, content.indexOf(oldString)) + newString + content.slice(content.indexOf(oldString) + oldString.length);
   try {
-    await writeFile2(filePath, updated, "utf8");
+    await atomicWriteUtf8File(filePath, updated);
   } catch (err) {
     const result2 = {
       success: false,
@@ -1165,7 +1188,7 @@ var globTool = createZodFunctionTool(
 
 // src/builtins/grep-tool.ts
 import { readFile as readFile3, readdir, stat as stat3 } from "fs/promises";
-import { join, resolve as resolve2 } from "path";
+import { join as join2, resolve as resolve2 } from "path";
 import { z as z6 } from "zod";
 var GrepSchema = z6.object({
   pattern: z6.string().describe("The regular expression pattern to search for in file contents"),
@@ -1199,7 +1222,7 @@ async function collectFiles(dirPath, glob) {
     }
     for (const name of entryNames) {
       if (name === "node_modules" || name === ".git") continue;
-      const fullPath = join(current, name);
+      const fullPath = join2(current, name);
       let fileStat;
       try {
         fileStat = await stat3(fullPath);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@robota-sdk/agent-tools",
-  "version": "3.0.0-beta.56",
+  "version": "3.0.0-beta.57",
   "description": "Tool registry and implementations for Robota SDK",
   "type": "module",
   "main": "dist/node/index.js",
@@ -34,7 +34,7 @@
     "zod": "^3.24.0"
   },
   "peerDependencies": {
-    "@robota-sdk/agent-core": "3.0.0-beta.56"
+    "@robota-sdk/agent-core": "3.0.0-beta.57"
   },
   "devDependencies": {
     "openapi-types": "^12.1.3",
@@ -42,7 +42,7 @@
     "tsup": "^8.0.1",
     "typescript": "^5.3.3",
     "vitest": "^1.6.1",
-    "@robota-sdk/agent-core": "3.0.0-beta.56"
+    "@robota-sdk/agent-core": "3.0.0-beta.57"
   },
   "license": "MIT",
   "publishConfig": {
@@ -53,6 +53,7 @@
     "build:js": "tsup src/index.ts --format esm,cjs --out-dir dist/node --clean",
     "build:types": "tsup src/index.ts --format esm,cjs --dts-only --out-dir dist/node",
     "test": "vitest run --passWithNoTests",
+    "test:coverage": "vitest run --coverage --passWithNoTests",
     "typecheck": "tsc --noEmit",
     "lint": "eslint src/ --ext .ts,.tsx",
     "lint:fix": "eslint src/ --ext .ts,.tsx --fix",