@agent-api/sdk 1.2.2 → 1.2.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog — @agent-api/sdk
 
+## 1.2.3
+
+### Fixed
+
+- Hardened local workdir scans so broken symlinks and files that disappear during recursive list, grep, and summarize operations do not abort the scan.
+
 ## 1.2.2
 
 ### Added

package/dist/local/core.js

@@ -1,6 +1,6 @@
 import { createHash, randomUUID } from "node:crypto";
 import { constants as fsConstants, watch as watchFS } from "node:fs";
-import { access, copyFile, mkdir, readdir, readFile, rename, rm, stat, writeFile } from "node:fs/promises";
+import { access, copyFile, lstat, mkdir, readdir, readFile, rename, rm, stat, writeFile } from "node:fs/promises";
 import { homedir, tmpdir } from "node:os";
 import path from "node:path";
 import { localSkillFromDirectory } from "../local-skills.js";
@@ -322,12 +322,15 @@ export class LocalFileStore {
             if (item.type !== "file" || item.size > maxBytesPerFile || !isLikelyTextFile(item.path)) {
                 continue;
             }
-            const raw = await readFile(item.fullPath);
+            const raw = await readOptionalFile(item.fullPath);
+            if (!raw) {
+                continue;
+            }
             if (looksBinary(raw)) {
                 continue;
             }
             filesScanned++;
-            const lines = splitLines(raw.toString("utf8"));
+            const lines = splitLines(Buffer.from(raw).toString("utf8"));
             for (let i = 0; i < lines.length; i++) {
                 if (!lines[i].includes(pattern)) {
                     continue;
@@ -363,7 +366,10 @@ export class LocalFileStore {
             if (!isLikelyTextFile(item.path) || item.size > previewBytes * 4) {
                 continue;
             }
-            const raw = await readFile(item.fullPath);
+            const raw = await readOptionalFile(item.fullPath);
+            if (!raw) {
+                continue;
+            }
             if (looksBinary(raw)) {
                 continue;
             }
@@ -371,7 +377,7 @@ export class LocalFileStore {
             previews.push({
                 path: item.path,
                 size: item.size,
-                preview: raw.subarray(0, previewBytes).toString("utf8"),
+                preview: Buffer.from(raw.subarray(0, previewBytes)).toString("utf8"),
                 preview_truncated: truncated || undefined,
             });
         }
@@ -393,7 +399,10 @@ export class LocalFileStore {
             if (ignored(relativePath, options.ignore)) {
                 continue;
             }
-            const info = await stat(fullPath);
+            const info = await lstatOptional(fullPath);
+            if (!info) {
+                continue;
+            }
             const item = {
                 path: relativePath,
                 fullPath,
@@ -401,7 +410,7 @@ export class LocalFileStore {
                 size: info.size,
                 modifiedAt: info.mtime,
             };
-            if (entry.isDirectory()) {
+            if (info.isDirectory()) {
                 if (options.includeDirectories) {
                     out.push(item);
                 }
@@ -813,6 +822,28 @@ async function fileExists(fullPath) {
         return false;
     }
 }
+async function lstatOptional(fullPath) {
+    try {
+        return await lstat(fullPath);
+    }
+    catch (error) {
+        if (error?.code === "ENOENT" || error?.code === "ENOTDIR") {
+            return null;
+        }
+        throw error;
+    }
+}
+async function readOptionalFile(fullPath) {
+    try {
+        return await readFile(fullPath);
+    }
+    catch (error) {
+        if (error?.code === "ENOENT" || error?.code === "ENOTDIR" || error?.code === "EISDIR") {
+            return null;
+        }
+        throw error;
+    }
+}
 function normalizeAppName(appName) {
     const trimmed = appName.trim();
     if (!trimmed) {

package/dist/version.d.ts

@@ -1,2 +1,2 @@
-export declare const VERSION = "1.2.2";
-export declare const USER_AGENT = "@agent-api/sdk/1.2.2";
+export declare const VERSION = "1.2.3";
+export declare const USER_AGENT = "@agent-api/sdk/1.2.3";

package/dist/version.js

@@ -1,2 +1,2 @@
-export const VERSION = "1.2.2";
+export const VERSION = "1.2.3";
 export const USER_AGENT = `@agent-api/sdk/${VERSION}`;

package/dist-cjs/local/core.js

@@ -337,12 +337,15 @@ class LocalFileStore {
             if (item.type !== "file" || item.size > maxBytesPerFile || !isLikelyTextFile(item.path)) {
                 continue;
             }
-            const raw = await (0, promises_1.readFile)(item.fullPath);
+            const raw = await readOptionalFile(item.fullPath);
+            if (!raw) {
+                continue;
+            }
             if (looksBinary(raw)) {
                 continue;
             }
             filesScanned++;
-            const lines = splitLines(raw.toString("utf8"));
+            const lines = splitLines(Buffer.from(raw).toString("utf8"));
             for (let i = 0; i < lines.length; i++) {
                 if (!lines[i].includes(pattern)) {
                     continue;
@@ -378,7 +381,10 @@ class LocalFileStore {
             if (!isLikelyTextFile(item.path) || item.size > previewBytes * 4) {
                 continue;
             }
-            const raw = await (0, promises_1.readFile)(item.fullPath);
+            const raw = await readOptionalFile(item.fullPath);
+            if (!raw) {
+                continue;
+            }
             if (looksBinary(raw)) {
                 continue;
             }
@@ -386,7 +392,7 @@ class LocalFileStore {
             previews.push({
                 path: item.path,
                 size: item.size,
-                preview: raw.subarray(0, previewBytes).toString("utf8"),
+                preview: Buffer.from(raw.subarray(0, previewBytes)).toString("utf8"),
                 preview_truncated: truncated || undefined,
             });
         }
@@ -408,7 +414,10 @@ class LocalFileStore {
             if (ignored(relativePath, options.ignore)) {
                 continue;
             }
-            const info = await (0, promises_1.stat)(fullPath);
+            const info = await lstatOptional(fullPath);
+            if (!info) {
+                continue;
+            }
             const item = {
                 path: relativePath,
                 fullPath,
@@ -416,7 +425,7 @@ class LocalFileStore {
                 size: info.size,
                 modifiedAt: info.mtime,
             };
-            if (entry.isDirectory()) {
+            if (info.isDirectory()) {
                 if (options.includeDirectories) {
                     out.push(item);
                 }
@@ -833,6 +842,28 @@ async function fileExists(fullPath) {
         return false;
     }
 }
+async function lstatOptional(fullPath) {
+    try {
+        return await (0, promises_1.lstat)(fullPath);
+    }
+    catch (error) {
+        if (error?.code === "ENOENT" || error?.code === "ENOTDIR") {
+            return null;
+        }
+        throw error;
+    }
+}
+async function readOptionalFile(fullPath) {
+    try {
+        return await (0, promises_1.readFile)(fullPath);
+    }
+    catch (error) {
+        if (error?.code === "ENOENT" || error?.code === "ENOTDIR" || error?.code === "EISDIR") {
+            return null;
+        }
+        throw error;
+    }
+}
 function normalizeAppName(appName) {
     const trimmed = appName.trim();
     if (!trimmed) {

package/dist-cjs/version.js

@@ -1,5 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.USER_AGENT = exports.VERSION = void 0;
-exports.VERSION = "1.2.2";
+exports.VERSION = "1.2.3";
 exports.USER_AGENT = `@agent-api/sdk/${exports.VERSION}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-api/sdk",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "Production JavaScript SDK for the Managed Agent API",
   "license": "MIT",
   "repository": {