@aexol/spectral 0.9.127 → 0.9.129

package/dist/sdk/coding-agent/core/tools/write.d.ts

@@ -6,18 +6,15 @@ declare const writeSchema: Type.TObject<{
     content: Type.TString;
 }>;
 export type WriteToolInput = Static<typeof writeSchema>;
-/**
- * Pluggable operations for the write tool.
- * Override these to delegate file writing to remote systems (for example SSH).
- */
 export interface WriteOperations {
-    /** Write content to a file */
-    writeFile: (absolutePath: string, content: string) => Promise<void>;
-    /** Create directory recursively */
+    /**
+     * Optional atomic write override (temp file + rename). When omitted,
+     * the default {@link atomicWriteFile} helper is used.
+     */
+    atomicWrite?: (absolutePath: string, content: string) => Promise<void>;
     mkdir: (dir: string) => Promise<void>;
 }
 export interface WriteToolOptions {
-    /** Custom operations for file writing. Default: local filesystem */
     operations?: WriteOperations;
 }
 export declare function createWriteToolDefinition(cwd: string, options?: WriteToolOptions): ToolDefinition<typeof writeSchema, undefined>;

package/dist/sdk/coding-agent/core/tools/write.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"write.d.ts","sourceRoot":"","sources":["../../../../../src/sdk/coding-agent/core/tools/write.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAC;AAG9D,OAAO,EAAE,KAAK,MAAM,EAAE,IAAI,EAAE,MAAM,SAAS,CAAC;AAE5C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAM7D,QAAA,MAAM,WAAW;;;EAGf,CAAC;AAEH,MAAM,MAAM,cAAc,GAAG,MAAM,CAAC,OAAO,WAAW,CAAC,CAAC;AAExD;;;GAGG;AACH,MAAM,WAAW,eAAe;IAC/B,8BAA8B;IAC9B,SAAS,EAAE,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IACpE,mCAAmC;IACnC,KAAK,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;CACtC;AAOD,MAAM,WAAW,gBAAgB;IAChC,oEAAoE;IACpE,UAAU,CAAC,EAAE,eAAe,CAAC;CAC7B;AA2DD,wBAAgB,yBAAyB,CACxC,GAAG,EAAE,MAAM,EACX,OAAO,CAAC,EAAE,gBAAgB,GACxB,cAAc,CAAC,OAAO,WAAW,EAAE,SAAS,CAAC,CA2D/C;AAED,wBAAgB,eAAe,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,gBAAgB,GAAG,SAAS,CAAC,OAAO,WAAW,CAAC,CAEtG"}
+{"version":3,"file":"write.d.ts","sourceRoot":"","sources":["../../../../../src/sdk/coding-agent/core/tools/write.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAC;AAG9D,OAAO,EAAE,KAAK,MAAM,EAAE,IAAI,EAAE,MAAM,SAAS,CAAC;AAE5C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAe7D,QAAA,MAAM,WAAW;;;EAKf,CAAC;AACH,MAAM,MAAM,cAAc,GAAG,MAAM,CAAC,OAAO,WAAW,CAAC,CAAC;AACxD,MAAM,WAAW,eAAe;IAC9B;;;OAGG;IACH,WAAW,CAAC,EAAE,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IACvE,KAAK,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;CACvC;AAID,MAAM,WAAW,gBAAgB;IAC/B,UAAU,CAAC,EAAE,eAAe,CAAC;CAC9B;AA6DD,wBAAgB,yBAAyB,CACvC,GAAG,EAAE,MAAM,EACX,OAAO,CAAC,EAAE,gBAAgB,GACzB,cAAc,CAAC,OAAO,WAAW,EAAE,SAAS,CAAC,CA+E/C;AACD,wBAAgB,eAAe,CAC7B,GAAG,EAAE,MAAM,EACX,OAAO,CAAC,EAAE,gBAAgB,GACzB,SAAS,CAAC,OAAO,WAAW,CAAC,CAE/B"}

package/dist/sdk/coding-agent/core/tools/write.js

@@ -1,17 +1,19 @@
-import { mkdir as fsMkdir, writeFile as fsWriteFile } from "fs/promises";
+import { mkdir as fsMkdir } from "fs/promises";
 import { dirname } from "path";
 import { Type } from "typebox";
 import { getLanguageFromPath, highlightCode } from "../theme.js";
+import { atomicWriteFile, defaultAtomicWriteOperations, } from "./atomic-write.js";
 import { withFileMutationQueue } from "./file-mutation-queue.js";
 import { resolveToCwd } from "./path-utils.js";
-import { normalizeDisplayText, replaceTabs, shortenPath, str } from "./render-utils.js";
+import { normalizeDisplayText, replaceTabs, shortenPath, str, } from "./render-utils.js";
 import { wrapToolDefinition } from "./tool-definition-wrapper.js";
 const writeSchema = Type.Object({
-    path: Type.String({ description: "Path to the file to write (relative or absolute)" }),
+    path: Type.String({
+        description: "Path to the file to write (relative or absolute)",
+    }),
     content: Type.String({ description: "Content to write to the file" }),
 });
 const defaultWriteOperations = {
-    writeFile: (path, content) => fsWriteFile(path, content, "utf-8"),
     mkdir: (dir) => fsMkdir(dir, { recursive: true }).then(() => { }),
 };
 function trimTrailingEmptyLines(lines) {
@@ -62,6 +64,7 @@ function formatWriteResult(result, _theme) {
 }
 export function createWriteToolDefinition(cwd, options) {
     const ops = options?.operations ?? defaultWriteOperations;
+    const atomicOps = defaultAtomicWriteOperations;
     return {
         name: "write",
         label: "write",
@@ -73,30 +76,45 @@ export function createWriteToolDefinition(cwd, options) {
             const absolutePath = resolveToCwd(path, cwd);
             const dir = dirname(absolutePath);
             return withFileMutationQueue(absolutePath, () => new Promise((resolve, reject) => {
+                // Abort BEFORE the write begins -> reject immediately.
                 if (signal?.aborted) {
                     reject(new Error("Operation aborted"));
                     return;
                 }
                 let aborted = false;
+                let writeStarted = false;
                 const onAbort = () => {
                     aborted = true;
-                    reject(new Error("Operation aborted"));
+                    // W2: only reject when aborted BEFORE the write begins. An abort
+                    // that fires during/after the write must not swallow the
+                    // successful resolve once the write completes.
+                    if (!writeStarted) {
+                        reject(new Error("Operation aborted"));
+                    }
                 };
                 signal?.addEventListener("abort", onAbort, { once: true });
                 (async () => {
                     try {
-                        // Create parent directories if needed.
                         await ops.mkdir(dir);
+                        // Reject if aborted before the write begins.
                         if (aborted)
                             return;
-                        // Write the file contents.
-                        await ops.writeFile(absolutePath, content);
-                        if (aborted)
-                            return;
+                        writeStarted = true;
+                        // W1 + W5: atomic write (temp + rename). On failure the
+                        // temp is cleaned up and the target is left untouched.
+                        const writeFn = ops.atomicWrite ??
+                            ((p, c) => atomicWriteFile(p, c, "utf-8", atomicOps));
+                        await writeFn(absolutePath, content);
+                        // W2: once the write has completed successfully, resolve
+                        // with the success content even if the abort signal
+                        // fired mid-write. Do NOT swallow the resolve.
                         signal?.removeEventListener("abort", onAbort);
                         resolve({
                             content: [
-                                { type: "text", text: `Successfully wrote ${content.length} bytes to ${path}` },
+                                {
+                                    type: "text",
+                                    text: `Successfully wrote ${Buffer.byteLength(content, "utf-8")} bytes to ${path}`,
+                                },
                             ],
                             details: undefined,
                         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aexol/spectral",
-  "version": "0.9.127",
+  "version": "0.9.129",
   "description": "AI coding agent for Aexol with relay-based browser access.",
   "type": "module",
   "private": false,