@agent-api/sdk 1.4.7 → 1.4.8

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog — @agent-api/sdk
 
+## 1.4.8
+
+### Changed
+
+- Restored `LocalFileStore.list()` to strict complete-or-fail scan semantics while keeping `listWithWarnings()` as the explicit tolerant partial-scan API.
+- Made local context packages leave recursive scan depth uncapped unless callers provide `maxDepth`.
+- Moved local skill discovery onto the same warning-aware filesystem scan primitive used by workdir summaries and context packages.
+
 ## 1.4.7
 
 ### Added

package/dist/local/context.js

@@ -6,7 +6,7 @@ export async function createLocalContextPackage(workdir, params = {}) {
     const maxFiles = positiveInt(params.maxFiles, 80);
     const maxBytes = positiveInt(params.maxBytes, 256 * 1024);
     const maxBytesPerFile = positiveInt(params.maxBytesPerFile, 32 * 1024);
-    const maxDepth = positiveInt(params.maxDepth, 3);
+    const maxDepth = optionalPositiveInt(params.maxDepth);
     const previewBytes = positiveInt(params.previewBytes, maxBytesPerFile);
     const includeContent = params.includeContent ?? true;
     const includeHashes = params.includeHashes ?? true;
@@ -151,6 +151,10 @@ function positiveInt(value, fallback) {
     const parsed = Number(value);
     return Number.isFinite(parsed) && parsed > 0 ? Math.trunc(parsed) : fallback;
 }
+function optionalPositiveInt(value) {
+    const parsed = Number(value);
+    return Number.isFinite(parsed) && parsed > 0 ? Math.trunc(parsed) : undefined;
+}
 async function sha256File(workdir, relativePath) {
     const fullPath = workdir.files.resolvePath(relativePath);
     return createHash("sha256").update(await readFile(fullPath)).digest("hex");

package/dist/local/core.js

@@ -139,7 +139,11 @@ export class LocalFileStore {
         return fullPath;
     }
     async list(relativePath = ".", options = {}) {
-        return (await this.listWithWarnings(relativePath, options)).stats;
+        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));
     }
     async listWithWarnings(relativePath = ".", options = {}) {
         const base = this.resolvePath(relativePath);
@@ -422,7 +426,7 @@ export class LocalFileStore {
             entries = await readdir(dir, { withFileTypes: true });
         }
         catch (error) {
-            if (dir !== scanRoot && recordScanWarning(warnings, toPortablePath(path.relative(storeRoot, dir)), error)) {
+            if (warnings && dir !== scanRoot && recordScanWarning(warnings, toPortablePath(path.relative(storeRoot, dir)), error)) {
                 return;
             }
             throw error;
@@ -820,8 +824,9 @@ function defaultDirs(platform, env, home, authorSegment, appSegment, baseDir) {
     };
 }
 async function discoverSkillDirectories(root, options) {
+    const rootStore = new LocalFileStore(root);
     try {
-        const info = await stat(root);
+        const info = await stat(rootStore.root);
         if (!info.isDirectory()) {
             return [];
         }
@@ -832,27 +837,20 @@ async function discoverSkillDirectories(root, options) {
         }
         throw error;
     }
+    const { stats } = await rootStore.listWithWarnings(".", {
+        recursive: options.recursive ?? false,
+        includeDirectories: true,
+        maxDepth: options.maxDepth,
+        ignore: [".git", "node_modules", "__pycache__"],
+    });
+    const dirs = [rootStore.root, ...stats.filter((item) => item.type === "directory").map((item) => item.fullPath)];
     const out = [];
-    const maxDepth = options.recursive ? options.maxDepth ?? Number.POSITIVE_INFINITY : 1;
-    await walkSkillDirectories(root, root, out, maxDepth);
-    return out;
-}
-async function walkSkillDirectories(root, dir, out, maxDepth) {
-    if (await fileExists(path.join(dir, "SKILL.md"))) {
-        out.push(dir);
-    }
-    const relative = path.relative(root, dir);
-    const depth = relative ? relative.split(path.sep).length : 0;
-    if (depth >= maxDepth) {
-        return;
-    }
-    const entries = await readdir(dir, { withFileTypes: true });
-    for (const entry of entries) {
-        if (!entry.isDirectory() || ignoredDirectoryName(entry.name)) {
-            continue;
+    for (const dir of dirs) {
+        if (await fileExists(path.join(dir, "SKILL.md"))) {
+            out.push(dir);
         }
-        await walkSkillDirectories(root, path.join(dir, entry.name), out, maxDepth);
     }
+    return out;
 }
 async function fileExists(fullPath) {
     try {

package/dist/local/scan.d.ts

@@ -0,0 +1,13 @@
+export interface LocalScanWarning {
+    path: string;
+    code?: string;
+    message: string;
+}
+export interface LocalScanWarningList<T> {
+    value: T;
+    warnings: LocalScanWarning[];
+}
+export declare function withScanWarnings<T>(value: T, warnings: LocalScanWarning[]): T & {
+    scan_warnings?: LocalScanWarning[];
+};
+export declare function recordScanWarning(warnings: LocalScanWarning[], relativePath: string, error: unknown): boolean;

package/dist/local/scan.js

@@ -0,0 +1,24 @@
+const maxScanWarnings = 20;
+const ignorableScanErrorCodes = new Set(["ENOENT", "ENOTDIR", "EACCES", "EPERM", "ENOTCONN", "ELOOP", "EINVAL"]);
+export function withScanWarnings(value, warnings) {
+    const out = value;
+    if (warnings.length > 0) {
+        out.scan_warnings = warnings;
+    }
+    return out;
+}
+export 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;
+}

package/dist/version.d.ts

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

package/dist/version.js

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

package/dist-cjs/local/context.js

@@ -10,7 +10,7 @@ async function createLocalContextPackage(workdir, params = {}) {
     const maxFiles = positiveInt(params.maxFiles, 80);
     const maxBytes = positiveInt(params.maxBytes, 256 * 1024);
     const maxBytesPerFile = positiveInt(params.maxBytesPerFile, 32 * 1024);
-    const maxDepth = positiveInt(params.maxDepth, 3);
+    const maxDepth = optionalPositiveInt(params.maxDepth);
     const previewBytes = positiveInt(params.previewBytes, maxBytesPerFile);
     const includeContent = params.includeContent ?? true;
     const includeHashes = params.includeHashes ?? true;
@@ -155,6 +155,10 @@ function positiveInt(value, fallback) {
     const parsed = Number(value);
     return Number.isFinite(parsed) && parsed > 0 ? Math.trunc(parsed) : fallback;
 }
+function optionalPositiveInt(value) {
+    const parsed = Number(value);
+    return Number.isFinite(parsed) && parsed > 0 ? Math.trunc(parsed) : undefined;
+}
 async function sha256File(workdir, relativePath) {
     const fullPath = workdir.files.resolvePath(relativePath);
     return (0, node_crypto_1.createHash)("sha256").update(await (0, promises_1.readFile)(fullPath)).digest("hex");

package/dist-cjs/local/core.js

@@ -154,7 +154,11 @@ class LocalFileStore {
         return fullPath;
     }
     async list(relativePath = ".", options = {}) {
-        return (await this.listWithWarnings(relativePath, options)).stats;
+        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));
     }
     async listWithWarnings(relativePath = ".", options = {}) {
         const base = this.resolvePath(relativePath);
@@ -437,7 +441,7 @@ class LocalFileStore {
             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)) {
+            if (warnings && dir !== scanRoot && recordScanWarning(warnings, toPortablePath(node_path_1.default.relative(storeRoot, dir)), error)) {
                 return;
             }
             throw error;
@@ -840,8 +844,9 @@ function defaultDirs(platform, env, home, authorSegment, appSegment, baseDir) {
     };
 }
 async function discoverSkillDirectories(root, options) {
+    const rootStore = new LocalFileStore(root);
     try {
-        const info = await (0, promises_1.stat)(root);
+        const info = await (0, promises_1.stat)(rootStore.root);
         if (!info.isDirectory()) {
             return [];
         }
@@ -852,27 +857,20 @@ async function discoverSkillDirectories(root, options) {
         }
         throw error;
     }
+    const { stats } = await rootStore.listWithWarnings(".", {
+        recursive: options.recursive ?? false,
+        includeDirectories: true,
+        maxDepth: options.maxDepth,
+        ignore: [".git", "node_modules", "__pycache__"],
+    });
+    const dirs = [rootStore.root, ...stats.filter((item) => item.type === "directory").map((item) => item.fullPath)];
     const out = [];
-    const maxDepth = options.recursive ? options.maxDepth ?? Number.POSITIVE_INFINITY : 1;
-    await walkSkillDirectories(root, root, out, maxDepth);
-    return out;
-}
-async function walkSkillDirectories(root, dir, out, maxDepth) {
-    if (await fileExists(node_path_1.default.join(dir, "SKILL.md"))) {
-        out.push(dir);
-    }
-    const relative = node_path_1.default.relative(root, dir);
-    const depth = relative ? relative.split(node_path_1.default.sep).length : 0;
-    if (depth >= maxDepth) {
-        return;
-    }
-    const entries = await (0, promises_1.readdir)(dir, { withFileTypes: true });
-    for (const entry of entries) {
-        if (!entry.isDirectory() || ignoredDirectoryName(entry.name)) {
-            continue;
+    for (const dir of dirs) {
+        if (await fileExists(node_path_1.default.join(dir, "SKILL.md"))) {
+            out.push(dir);
         }
-        await walkSkillDirectories(root, node_path_1.default.join(dir, entry.name), out, maxDepth);
     }
+    return out;
 }
 async function fileExists(fullPath) {
     try {

package/dist-cjs/local/scan.js

@@ -0,0 +1,28 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.withScanWarnings = withScanWarnings;
+exports.recordScanWarning = recordScanWarning;
+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;
+}

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

package/package.json

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