@agent-api/sdk 1.4.4 → 1.4.6

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog — @agent-api/sdk
 
+## 1.4.6
+
+### Fixed
+
+- Made local workdir scans skip inaccessible or broken child entries with structured `scan_warnings` instead of failing an otherwise valid workdir.
+
+## 1.4.5
+
+### Fixed
+
+- Made local workdir summaries honor `maxDepth`, and aligned model-facing summarize options with the direct SDK API.
+
 ## 1.4.4
 
 ### Added

package/README.md

@@ -219,7 +219,7 @@ await workdir.patchLines("src/index.ts", {
   replacement: "console.log('patched');",
 });
 
-const summary = await workdir.summarize();
+const summary = await workdir.summarize({ maxDepth: 3, maxFiles: 500 });
 ```
 
 For project/workdir roots, prefer `local.workdir()` so SDK defaults protect common generated directories such as `.git`, `node_modules`, `dist`, and build caches.

package/dist/local/core.d.ts

@@ -66,6 +66,12 @@ export interface LocalEntry {
 export interface LocalEntryList {
     object: "list";
     entries: LocalEntry[];
+    scan_warnings?: LocalScanWarning[];
+}
+export interface LocalScanWarning {
+    path: string;
+    code?: string;
+    message: string;
 }
 export interface LocalSearchEntriesParams {
     query: string;
@@ -157,6 +163,7 @@ export interface LocalSummarizeParams {
     maxPreviews?: number;
     previewBytes?: number;
     topPaths?: number;
+    maxDepth?: number;
     ignore?: LocalListOptions["ignore"];
 }
 export interface LocalSummaryPreview {
@@ -173,6 +180,7 @@ export interface LocalSummary {
     text_previews: LocalSummaryPreview[];
     generated_at_unix: number;
     scan_truncated: boolean;
+    scan_warnings?: LocalScanWarning[];
 }
 export interface LocalWorkdirOptions {
     name?: string;
@@ -309,6 +317,7 @@ export declare class LocalFileStore {
     grep(params: LocalGrepParams): Promise<LocalGrepResponse>;
     private grepCandidates;
     summarize(params?: LocalSummarizeParams): Promise<LocalSummary>;
+    private listWithWarnings;
     private walk;
 }
 export declare class LocalConfigStore {

package/dist/local/core.js

@@ -139,15 +139,11 @@ export class LocalFileStore {
         return fullPath;
     }
     async list(relativePath = ".", options = {}) {
-        const base = this.resolvePath(relativePath);
-        const maxDepth = options.recursive ? options.maxDepth ?? Number.POSITIVE_INFINITY : 1;
-        const out = [];
-        await this.walk(this.root, base, base, out, maxDepth, options);
-        return out.sort((a, b) => a.path.localeCompare(b.path));
+        return (await this.listWithWarnings(relativePath, options)).stats;
     }
     async listEntries(relativePath = ".", options = {}) {
-        const stats = await this.list(relativePath, { ...options, includeDirectories: options.includeDirectories ?? true });
-        return { object: "list", entries: stats.map(localEntryFromStat) };
+        const { stats, warnings } = await this.listWithWarnings(relativePath, { ...options, includeDirectories: options.includeDirectories ?? true });
+        return withScanWarnings({ object: "list", entries: stats.map(localEntryFromStat) }, warnings);
     }
     async searchEntries(params) {
         const query = params.query.trim().toLowerCase();
@@ -374,7 +370,7 @@ export class LocalFileStore {
         const maxPreviews = positiveInt(params.maxPreviews, 20);
         const previewBytes = positiveInt(params.previewBytes, 4096);
         const topPaths = positiveInt(params.topPaths, 20);
-        const stats = await this.list(params.path ?? ".", { recursive: true, ignore: params.ignore });
+        const { stats, warnings } = await this.listWithWarnings(params.path ?? ".", { recursive: true, maxDepth: params.maxDepth, ignore: params.ignore });
         const files = stats.filter((item) => item.type === "file").slice(0, maxFiles);
         const scanTruncated = stats.filter((item) => item.type === "file").length > files.length;
         const totalBytes = files.reduce((sum, item) => sum + item.size, 0);
@@ -402,7 +398,7 @@ export class LocalFileStore {
                 preview_truncated: truncated || undefined,
             });
         }
-        return {
+        return withScanWarnings({
             summary_path: "",
             file_count: files.length,
             total_bytes: totalBytes,
@@ -410,17 +406,34 @@ export class LocalFileStore {
             text_previews: previews,
             generated_at_unix: Math.floor(Date.now() / 1000),
             scan_truncated: scanTruncated,
-        };
+        }, warnings);
     }
-    async walk(storeRoot, scanRoot, dir, out, maxDepth, options) {
-        const entries = await readdir(dir, { withFileTypes: true });
+    async listWithWarnings(relativePath = ".", options = {}) {
+        const base = this.resolvePath(relativePath);
+        const maxDepth = options.recursive ? options.maxDepth ?? Number.POSITIVE_INFINITY : 1;
+        const out = [];
+        const warnings = [];
+        await this.walk(this.root, base, base, out, maxDepth, options, warnings);
+        return { stats: out.sort((a, b) => a.path.localeCompare(b.path)), warnings };
+    }
+    async walk(storeRoot, scanRoot, dir, out, maxDepth, options, warnings) {
+        let entries;
+        try {
+            entries = await readdir(dir, { withFileTypes: true });
+        }
+        catch (error) {
+            if (dir !== scanRoot && recordScanWarning(warnings, toPortablePath(path.relative(storeRoot, dir)), error)) {
+                return;
+            }
+            throw error;
+        }
         for (const entry of entries) {
             const fullPath = path.join(dir, entry.name);
             const relativePath = toPortablePath(path.relative(storeRoot, fullPath));
             if (ignored(relativePath, options.ignore)) {
                 continue;
             }
-            const info = await lstatOptional(fullPath);
+            const info = await lstatOptional(fullPath, relativePath, warnings);
             if (!info) {
                 continue;
             }
@@ -437,7 +450,7 @@ export class LocalFileStore {
                 }
                 const depth = toPortablePath(path.relative(scanRoot, fullPath)).split("/").filter(Boolean).length;
                 if (depth < maxDepth) {
-                    await this.walk(storeRoot, scanRoot, fullPath, out, maxDepth, options);
+                    await this.walk(storeRoot, scanRoot, fullPath, out, maxDepth, options, warnings);
                 }
                 continue;
             }
@@ -847,11 +860,38 @@ async function fileExists(fullPath) {
         return false;
     }
 }
-async function lstatOptional(fullPath) {
+const maxScanWarnings = 20;
+const ignorableScanErrorCodes = new Set(["ENOENT", "ENOTDIR", "EACCES", "EPERM", "ENOTCONN", "ELOOP", "EINVAL"]);
+function withScanWarnings(value, warnings) {
+    const out = value;
+    if (warnings.length > 0) {
+        out.scan_warnings = warnings;
+    }
+    return out;
+}
+function recordScanWarning(warnings, relativePath, error) {
+    const scanError = error;
+    const code = typeof scanError?.code === "string" ? scanError.code : undefined;
+    if (!code || !ignorableScanErrorCodes.has(code)) {
+        return false;
+    }
+    if (warnings.length < maxScanWarnings) {
+        warnings.push({
+            path: relativePath || ".",
+            code,
+            message: typeof scanError.message === "string" ? scanError.message : code,
+        });
+    }
+    return true;
+}
+async function lstatOptional(fullPath, relativePath, warnings) {
     try {
         return await lstat(fullPath);
     }
     catch (error) {
+        if (warnings && relativePath && recordScanWarning(warnings, relativePath, error)) {
+            return null;
+        }
         if (error?.code === "ENOENT" || error?.code === "ENOTDIR") {
             return null;
         }
@@ -863,7 +903,7 @@ async function readOptionalFile(fullPath) {
         return await readFile(fullPath);
     }
     catch (error) {
-        if (error?.code === "ENOENT" || error?.code === "ENOTDIR" || error?.code === "EISDIR") {
+        if (error?.code === "ENOENT" || error?.code === "ENOTDIR" || error?.code === "EISDIR" || error?.code === "ENOTCONN") {
             return null;
         }
         throw error;

package/dist/local/tools.js

@@ -202,6 +202,9 @@ function summaryArgs(args) {
         path: optionalStringArg(args, "path"),
         maxFiles: optionalNumberArg(args, "maxFiles", "max_files"),
         maxPreviews: optionalNumberArg(args, "maxPreviews", "max_previews"),
+        previewBytes: optionalNumberArg(args, "previewBytes", "preview_bytes"),
+        topPaths: optionalNumberArg(args, "topPaths", "top_paths"),
+        maxDepth: optionalNumberArg(args, "maxDepth", "max_depth"),
     };
 }
 function grepArgs(args) {

package/dist/version.d.ts

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

package/dist/version.js

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

package/dist-cjs/local/core.js

@@ -154,15 +154,11 @@ class LocalFileStore {
         return fullPath;
     }
     async list(relativePath = ".", options = {}) {
-        const base = this.resolvePath(relativePath);
-        const maxDepth = options.recursive ? options.maxDepth ?? Number.POSITIVE_INFINITY : 1;
-        const out = [];
-        await this.walk(this.root, base, base, out, maxDepth, options);
-        return out.sort((a, b) => a.path.localeCompare(b.path));
+        return (await this.listWithWarnings(relativePath, options)).stats;
     }
     async listEntries(relativePath = ".", options = {}) {
-        const stats = await this.list(relativePath, { ...options, includeDirectories: options.includeDirectories ?? true });
-        return { object: "list", entries: stats.map(localEntryFromStat) };
+        const { stats, warnings } = await this.listWithWarnings(relativePath, { ...options, includeDirectories: options.includeDirectories ?? true });
+        return withScanWarnings({ object: "list", entries: stats.map(localEntryFromStat) }, warnings);
     }
     async searchEntries(params) {
         const query = params.query.trim().toLowerCase();
@@ -389,7 +385,7 @@ class LocalFileStore {
         const maxPreviews = positiveInt(params.maxPreviews, 20);
         const previewBytes = positiveInt(params.previewBytes, 4096);
         const topPaths = positiveInt(params.topPaths, 20);
-        const stats = await this.list(params.path ?? ".", { recursive: true, ignore: params.ignore });
+        const { stats, warnings } = await this.listWithWarnings(params.path ?? ".", { recursive: true, maxDepth: params.maxDepth, ignore: params.ignore });
         const files = stats.filter((item) => item.type === "file").slice(0, maxFiles);
         const scanTruncated = stats.filter((item) => item.type === "file").length > files.length;
         const totalBytes = files.reduce((sum, item) => sum + item.size, 0);
@@ -417,7 +413,7 @@ class LocalFileStore {
                 preview_truncated: truncated || undefined,
             });
         }
-        return {
+        return withScanWarnings({
             summary_path: "",
             file_count: files.length,
             total_bytes: totalBytes,
@@ -425,17 +421,34 @@ class LocalFileStore {
             text_previews: previews,
             generated_at_unix: Math.floor(Date.now() / 1000),
             scan_truncated: scanTruncated,
-        };
+        }, warnings);
     }
-    async walk(storeRoot, scanRoot, dir, out, maxDepth, options) {
-        const entries = await (0, promises_1.readdir)(dir, { withFileTypes: true });
+    async listWithWarnings(relativePath = ".", options = {}) {
+        const base = this.resolvePath(relativePath);
+        const maxDepth = options.recursive ? options.maxDepth ?? Number.POSITIVE_INFINITY : 1;
+        const out = [];
+        const warnings = [];
+        await this.walk(this.root, base, base, out, maxDepth, options, warnings);
+        return { stats: out.sort((a, b) => a.path.localeCompare(b.path)), warnings };
+    }
+    async walk(storeRoot, scanRoot, dir, out, maxDepth, options, warnings) {
+        let entries;
+        try {
+            entries = await (0, promises_1.readdir)(dir, { withFileTypes: true });
+        }
+        catch (error) {
+            if (dir !== scanRoot && recordScanWarning(warnings, toPortablePath(node_path_1.default.relative(storeRoot, dir)), error)) {
+                return;
+            }
+            throw error;
+        }
         for (const entry of entries) {
             const fullPath = node_path_1.default.join(dir, entry.name);
             const relativePath = toPortablePath(node_path_1.default.relative(storeRoot, fullPath));
             if (ignored(relativePath, options.ignore)) {
                 continue;
             }
-            const info = await lstatOptional(fullPath);
+            const info = await lstatOptional(fullPath, relativePath, warnings);
             if (!info) {
                 continue;
             }
@@ -452,7 +465,7 @@ class LocalFileStore {
                 }
                 const depth = toPortablePath(node_path_1.default.relative(scanRoot, fullPath)).split("/").filter(Boolean).length;
                 if (depth < maxDepth) {
-                    await this.walk(storeRoot, scanRoot, fullPath, out, maxDepth, options);
+                    await this.walk(storeRoot, scanRoot, fullPath, out, maxDepth, options, warnings);
                 }
                 continue;
             }
@@ -867,11 +880,38 @@ async function fileExists(fullPath) {
         return false;
     }
 }
-async function lstatOptional(fullPath) {
+const maxScanWarnings = 20;
+const ignorableScanErrorCodes = new Set(["ENOENT", "ENOTDIR", "EACCES", "EPERM", "ENOTCONN", "ELOOP", "EINVAL"]);
+function withScanWarnings(value, warnings) {
+    const out = value;
+    if (warnings.length > 0) {
+        out.scan_warnings = warnings;
+    }
+    return out;
+}
+function recordScanWarning(warnings, relativePath, error) {
+    const scanError = error;
+    const code = typeof scanError?.code === "string" ? scanError.code : undefined;
+    if (!code || !ignorableScanErrorCodes.has(code)) {
+        return false;
+    }
+    if (warnings.length < maxScanWarnings) {
+        warnings.push({
+            path: relativePath || ".",
+            code,
+            message: typeof scanError.message === "string" ? scanError.message : code,
+        });
+    }
+    return true;
+}
+async function lstatOptional(fullPath, relativePath, warnings) {
     try {
         return await (0, promises_1.lstat)(fullPath);
     }
     catch (error) {
+        if (warnings && relativePath && recordScanWarning(warnings, relativePath, error)) {
+            return null;
+        }
         if (error?.code === "ENOENT" || error?.code === "ENOTDIR") {
             return null;
         }
@@ -883,7 +923,7 @@ async function readOptionalFile(fullPath) {
         return await (0, promises_1.readFile)(fullPath);
     }
     catch (error) {
-        if (error?.code === "ENOENT" || error?.code === "ENOTDIR" || error?.code === "EISDIR") {
+        if (error?.code === "ENOENT" || error?.code === "ENOTDIR" || error?.code === "EISDIR" || error?.code === "ENOTCONN") {
             return null;
         }
         throw error;

package/dist-cjs/local/tools.js

@@ -209,6 +209,9 @@ function summaryArgs(args) {
         path: optionalStringArg(args, "path"),
         maxFiles: optionalNumberArg(args, "maxFiles", "max_files"),
         maxPreviews: optionalNumberArg(args, "maxPreviews", "max_previews"),
+        previewBytes: optionalNumberArg(args, "previewBytes", "preview_bytes"),
+        topPaths: optionalNumberArg(args, "topPaths", "top_paths"),
+        maxDepth: optionalNumberArg(args, "maxDepth", "max_depth"),
     };
 }
 function grepArgs(args) {

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.4.4";
+exports.VERSION = "1.4.6";
 exports.USER_AGENT = `@agent-api/sdk/${exports.VERSION}`;

package/package.json

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