@dexto/tools-filesystem 1.5.2 → 1.5.4

package/dist/path-validator.test.cjs

@@ -16,7 +16,7 @@ const createMockLogger = () => ({
   });
   (0, import_vitest.describe)("validatePath", () => {
     (0, import_vitest.describe)("Empty and Invalid Paths", () => {
-      (0, import_vitest.it)("should reject empty path", () => {
+      (0, import_vitest.it)("should reject empty path", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -29,11 +29,11 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("");
+        const result = await validator.validatePath("");
         (0, import_vitest.expect)(result.isValid).toBe(false);
         (0, import_vitest.expect)(result.error).toBe("Path cannot be empty");
       });
-      (0, import_vitest.it)("should reject whitespace-only path", () => {
+      (0, import_vitest.it)("should reject whitespace-only path", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -46,13 +46,13 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("   ");
+        const result = await validator.validatePath("   ");
         (0, import_vitest.expect)(result.isValid).toBe(false);
         (0, import_vitest.expect)(result.error).toBe("Path cannot be empty");
       });
     });
     (0, import_vitest.describe)("Allowed Paths", () => {
-      (0, import_vitest.it)("should allow paths within allowed directories", () => {
+      (0, import_vitest.it)("should allow paths within allowed directories", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -65,11 +65,11 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/src/file.ts");
+        const result = await validator.validatePath("/home/user/project/src/file.ts");
         (0, import_vitest.expect)(result.isValid).toBe(true);
         (0, import_vitest.expect)(result.normalizedPath).toBeDefined();
       });
-      (0, import_vitest.it)("should allow relative paths within working directory", () => {
+      (0, import_vitest.it)("should allow relative paths within working directory", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -82,10 +82,10 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("src/file.ts");
+        const result = await validator.validatePath("src/file.ts");
         (0, import_vitest.expect)(result.isValid).toBe(true);
       });
-      (0, import_vitest.it)("should reject paths outside allowed directories", () => {
+      (0, import_vitest.it)("should reject paths outside allowed directories", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -98,11 +98,11 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/external/project/file.ts");
+        const result = await validator.validatePath("/external/project/file.ts");
         (0, import_vitest.expect)(result.isValid).toBe(false);
         (0, import_vitest.expect)(result.error).toContain("not within allowed paths");
       });
-      (0, import_vitest.it)("should allow all paths when allowedPaths is empty", () => {
+      (0, import_vitest.it)("should allow all paths when allowedPaths is empty", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: [],
@@ -115,12 +115,12 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/anywhere/file.ts");
+        const result = await validator.validatePath("/anywhere/file.ts");
         (0, import_vitest.expect)(result.isValid).toBe(true);
       });
     });
     (0, import_vitest.describe)("Path Traversal Detection", () => {
-      (0, import_vitest.it)("should reject path traversal attempts", () => {
+      (0, import_vitest.it)("should reject path traversal attempts", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -133,13 +133,15 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/../../../etc/passwd");
+        const result = await validator.validatePath(
+          "/home/user/project/../../../etc/passwd"
+        );
         (0, import_vitest.expect)(result.isValid).toBe(false);
         (0, import_vitest.expect)(result.error).toBe("Path traversal detected");
       });
     });
     (0, import_vitest.describe)("Blocked Paths", () => {
-      (0, import_vitest.it)("should reject paths in blocked directories", () => {
+      (0, import_vitest.it)("should reject paths in blocked directories", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -152,11 +154,11 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/.git/config");
+        const result = await validator.validatePath("/home/user/project/.git/config");
         (0, import_vitest.expect)(result.isValid).toBe(false);
         (0, import_vitest.expect)(result.error).toContain("blocked");
       });
-      (0, import_vitest.it)("should reject paths in node_modules", () => {
+      (0, import_vitest.it)("should reject paths in node_modules", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -169,7 +171,7 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath(
+        const result = await validator.validatePath(
           "/home/user/project/node_modules/lodash/index.js"
         );
         (0, import_vitest.expect)(result.isValid).toBe(false);
@@ -177,7 +179,7 @@ const createMockLogger = () => ({
       });
     });
     (0, import_vitest.describe)("Blocked Extensions", () => {
-      (0, import_vitest.it)("should reject files with blocked extensions", () => {
+      (0, import_vitest.it)("should reject files with blocked extensions", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -190,11 +192,11 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/malware.exe");
+        const result = await validator.validatePath("/home/user/project/malware.exe");
         (0, import_vitest.expect)(result.isValid).toBe(false);
         (0, import_vitest.expect)(result.error).toContain(".exe is not allowed");
       });
-      (0, import_vitest.it)("should handle extensions without leading dot", () => {
+      (0, import_vitest.it)("should handle extensions without leading dot", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -208,10 +210,10 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/file.exe");
+        const result = await validator.validatePath("/home/user/project/file.exe");
         (0, import_vitest.expect)(result.isValid).toBe(false);
       });
-      (0, import_vitest.it)("should be case-insensitive for extensions", () => {
+      (0, import_vitest.it)("should be case-insensitive for extensions", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -224,12 +226,12 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/file.EXE");
+        const result = await validator.validatePath("/home/user/project/file.EXE");
         (0, import_vitest.expect)(result.isValid).toBe(false);
       });
     });
     (0, import_vitest.describe)("Directory Approval Checker Integration", () => {
-      (0, import_vitest.it)("should consult approval checker for external paths", () => {
+      (0, import_vitest.it)("should consult approval checker for external paths", async () => {
         const validator = new import_path_validator.PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -242,16 +244,16 @@ const createMockLogger = () => ({
           },
           mockLogger
         );
-        let result = validator.validatePath("/external/project/file.ts");
+        let result = await validator.validatePath("/external/project/file.ts");
         (0, import_vitest.expect)(result.isValid).toBe(false);
         const approvalChecker = (filePath) => {
           return filePath.startsWith("/external/project");
         };
         validator.setDirectoryApprovalChecker(approvalChecker);
-        result = validator.validatePath("/external/project/file.ts");
+        result = await validator.validatePath("/external/project/file.ts");
         (0, import_vitest.expect)(result.isValid).toBe(true);
       });
-      (0, import_vitest.it)("should not use approval checker for config-allowed paths", () => {
+      (0, import_vitest.it)("should not use approval checker for config-allowed paths", async () => {
         const approvalChecker = import_vitest.vi.fn().mockReturnValue(false);
         const validator = new import_path_validator.PathValidator(
           {
@@ -266,14 +268,14 @@ const createMockLogger = () => ({
           mockLogger
         );
         validator.setDirectoryApprovalChecker(approvalChecker);
-        const result = validator.validatePath("/home/user/project/src/file.ts");
+        const result = await validator.validatePath("/home/user/project/src/file.ts");
         (0, import_vitest.expect)(result.isValid).toBe(true);
         (0, import_vitest.expect)(approvalChecker).not.toHaveBeenCalled();
       });
     });
   });
   (0, import_vitest.describe)("isPathWithinAllowed", () => {
-    (0, import_vitest.it)("should return true for paths within config-allowed directories", () => {
+    (0, import_vitest.it)("should return true for paths within config-allowed directories", async () => {
       const validator = new import_path_validator.PathValidator(
         {
           allowedPaths: ["/home/user/project"],
@@ -286,12 +288,14 @@ const createMockLogger = () => ({
         },
         mockLogger
       );
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/home/user/project/src/file.ts")).toBe(true);
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/home/user/project/deep/nested/file.ts")).toBe(
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/home/user/project/src/file.ts")).toBe(
         true
       );
+      (0, import_vitest.expect)(
+        await validator.isPathWithinAllowed("/home/user/project/deep/nested/file.ts")
+      ).toBe(true);
     });
-    (0, import_vitest.it)("should return false for paths outside config-allowed directories", () => {
+    (0, import_vitest.it)("should return false for paths outside config-allowed directories", async () => {
       const validator = new import_path_validator.PathValidator(
         {
           allowedPaths: ["/home/user/project"],
@@ -304,10 +308,10 @@ const createMockLogger = () => ({
         },
         mockLogger
       );
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/home/user/other/file.ts")).toBe(false);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/home/user/other/file.ts")).toBe(false);
     });
-    (0, import_vitest.it)("should NOT consult approval checker (used for prompting decisions)", () => {
+    (0, import_vitest.it)("should NOT consult approval checker (used for prompting decisions)", async () => {
       const approvalChecker = import_vitest.vi.fn().mockReturnValue(true);
       const validator = new import_path_validator.PathValidator(
         {
@@ -322,10 +326,10 @@ const createMockLogger = () => ({
         mockLogger
       );
       validator.setDirectoryApprovalChecker(approvalChecker);
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
       (0, import_vitest.expect)(approvalChecker).not.toHaveBeenCalled();
     });
-    (0, import_vitest.it)("should return false for empty path", () => {
+    (0, import_vitest.it)("should return false for empty path", async () => {
       const validator = new import_path_validator.PathValidator(
         {
           allowedPaths: ["/home/user/project"],
@@ -338,10 +342,10 @@ const createMockLogger = () => ({
         },
         mockLogger
       );
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("")).toBe(false);
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("   ")).toBe(false);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("")).toBe(false);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("   ")).toBe(false);
     });
-    (0, import_vitest.it)("should return true when allowedPaths is empty (all paths allowed)", () => {
+    (0, import_vitest.it)("should return true when allowedPaths is empty (all paths allowed)", async () => {
       const validator = new import_path_validator.PathValidator(
         {
           allowedPaths: [],
@@ -354,11 +358,11 @@ const createMockLogger = () => ({
         },
         mockLogger
       );
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/anywhere/file.ts")).toBe(true);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/anywhere/file.ts")).toBe(true);
     });
   });
   (0, import_vitest.describe)("Path Containment (Parent Directory Coverage)", () => {
-    (0, import_vitest.it)("should recognize that approving parent covers child paths", () => {
+    (0, import_vitest.it)("should recognize that approving parent covers child paths", async () => {
       const validator = new import_path_validator.PathValidator(
         {
           allowedPaths: ["/external/sub"],
@@ -371,9 +375,11 @@ const createMockLogger = () => ({
         },
         mockLogger
       );
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/external/sub/deep/nested/file.ts")).toBe(true);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/external/sub/deep/nested/file.ts")).toBe(
+        true
+      );
     });
-    (0, import_vitest.it)("should not allow sibling directories", () => {
+    (0, import_vitest.it)("should not allow sibling directories", async () => {
       const validator = new import_path_validator.PathValidator(
         {
           allowedPaths: ["/external/sub"],
@@ -386,9 +392,9 @@ const createMockLogger = () => ({
         },
         mockLogger
       );
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/external/other/file.ts")).toBe(false);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/external/other/file.ts")).toBe(false);
     });
-    (0, import_vitest.it)("should not allow parent directories when child is approved", () => {
+    (0, import_vitest.it)("should not allow parent directories when child is approved", async () => {
       const validator = new import_path_validator.PathValidator(
         {
           allowedPaths: ["/external/sub/deep"],
@@ -401,7 +407,7 @@ const createMockLogger = () => ({
         },
         mockLogger
       );
-      (0, import_vitest.expect)(validator.isPathWithinAllowed("/external/sub/file.ts")).toBe(false);
+      (0, import_vitest.expect)(await validator.isPathWithinAllowed("/external/sub/file.ts")).toBe(false);
     });
   });
   (0, import_vitest.describe)("getAllowedPaths and getBlockedPaths", () => {

package/dist/path-validator.test.js

@@ -15,7 +15,7 @@ describe("PathValidator", () => {
   });
   describe("validatePath", () => {
     describe("Empty and Invalid Paths", () => {
-      it("should reject empty path", () => {
+      it("should reject empty path", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -28,11 +28,11 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("");
+        const result = await validator.validatePath("");
         expect(result.isValid).toBe(false);
         expect(result.error).toBe("Path cannot be empty");
       });
-      it("should reject whitespace-only path", () => {
+      it("should reject whitespace-only path", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -45,13 +45,13 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("   ");
+        const result = await validator.validatePath("   ");
         expect(result.isValid).toBe(false);
         expect(result.error).toBe("Path cannot be empty");
       });
     });
     describe("Allowed Paths", () => {
-      it("should allow paths within allowed directories", () => {
+      it("should allow paths within allowed directories", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -64,11 +64,11 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/src/file.ts");
+        const result = await validator.validatePath("/home/user/project/src/file.ts");
         expect(result.isValid).toBe(true);
         expect(result.normalizedPath).toBeDefined();
       });
-      it("should allow relative paths within working directory", () => {
+      it("should allow relative paths within working directory", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -81,10 +81,10 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("src/file.ts");
+        const result = await validator.validatePath("src/file.ts");
         expect(result.isValid).toBe(true);
       });
-      it("should reject paths outside allowed directories", () => {
+      it("should reject paths outside allowed directories", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -97,11 +97,11 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/external/project/file.ts");
+        const result = await validator.validatePath("/external/project/file.ts");
         expect(result.isValid).toBe(false);
         expect(result.error).toContain("not within allowed paths");
       });
-      it("should allow all paths when allowedPaths is empty", () => {
+      it("should allow all paths when allowedPaths is empty", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: [],
@@ -114,12 +114,12 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/anywhere/file.ts");
+        const result = await validator.validatePath("/anywhere/file.ts");
         expect(result.isValid).toBe(true);
       });
     });
     describe("Path Traversal Detection", () => {
-      it("should reject path traversal attempts", () => {
+      it("should reject path traversal attempts", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -132,13 +132,15 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/../../../etc/passwd");
+        const result = await validator.validatePath(
+          "/home/user/project/../../../etc/passwd"
+        );
         expect(result.isValid).toBe(false);
         expect(result.error).toBe("Path traversal detected");
       });
     });
     describe("Blocked Paths", () => {
-      it("should reject paths in blocked directories", () => {
+      it("should reject paths in blocked directories", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -151,11 +153,11 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/.git/config");
+        const result = await validator.validatePath("/home/user/project/.git/config");
         expect(result.isValid).toBe(false);
         expect(result.error).toContain("blocked");
       });
-      it("should reject paths in node_modules", () => {
+      it("should reject paths in node_modules", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -168,7 +170,7 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath(
+        const result = await validator.validatePath(
           "/home/user/project/node_modules/lodash/index.js"
         );
         expect(result.isValid).toBe(false);
@@ -176,7 +178,7 @@ describe("PathValidator", () => {
       });
     });
     describe("Blocked Extensions", () => {
-      it("should reject files with blocked extensions", () => {
+      it("should reject files with blocked extensions", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -189,11 +191,11 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/malware.exe");
+        const result = await validator.validatePath("/home/user/project/malware.exe");
         expect(result.isValid).toBe(false);
         expect(result.error).toContain(".exe is not allowed");
       });
-      it("should handle extensions without leading dot", () => {
+      it("should handle extensions without leading dot", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -207,10 +209,10 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/file.exe");
+        const result = await validator.validatePath("/home/user/project/file.exe");
         expect(result.isValid).toBe(false);
       });
-      it("should be case-insensitive for extensions", () => {
+      it("should be case-insensitive for extensions", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -223,12 +225,12 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        const result = validator.validatePath("/home/user/project/file.EXE");
+        const result = await validator.validatePath("/home/user/project/file.EXE");
         expect(result.isValid).toBe(false);
       });
     });
     describe("Directory Approval Checker Integration", () => {
-      it("should consult approval checker for external paths", () => {
+      it("should consult approval checker for external paths", async () => {
         const validator = new PathValidator(
           {
             allowedPaths: ["/home/user/project"],
@@ -241,16 +243,16 @@ describe("PathValidator", () => {
           },
           mockLogger
         );
-        let result = validator.validatePath("/external/project/file.ts");
+        let result = await validator.validatePath("/external/project/file.ts");
         expect(result.isValid).toBe(false);
         const approvalChecker = (filePath) => {
           return filePath.startsWith("/external/project");
         };
         validator.setDirectoryApprovalChecker(approvalChecker);
-        result = validator.validatePath("/external/project/file.ts");
+        result = await validator.validatePath("/external/project/file.ts");
         expect(result.isValid).toBe(true);
       });
-      it("should not use approval checker for config-allowed paths", () => {
+      it("should not use approval checker for config-allowed paths", async () => {
         const approvalChecker = vi.fn().mockReturnValue(false);
         const validator = new PathValidator(
           {
@@ -265,14 +267,14 @@ describe("PathValidator", () => {
           mockLogger
         );
         validator.setDirectoryApprovalChecker(approvalChecker);
-        const result = validator.validatePath("/home/user/project/src/file.ts");
+        const result = await validator.validatePath("/home/user/project/src/file.ts");
         expect(result.isValid).toBe(true);
         expect(approvalChecker).not.toHaveBeenCalled();
       });
     });
   });
   describe("isPathWithinAllowed", () => {
-    it("should return true for paths within config-allowed directories", () => {
+    it("should return true for paths within config-allowed directories", async () => {
       const validator = new PathValidator(
         {
           allowedPaths: ["/home/user/project"],
@@ -285,12 +287,14 @@ describe("PathValidator", () => {
         },
         mockLogger
       );
-      expect(validator.isPathWithinAllowed("/home/user/project/src/file.ts")).toBe(true);
-      expect(validator.isPathWithinAllowed("/home/user/project/deep/nested/file.ts")).toBe(
+      expect(await validator.isPathWithinAllowed("/home/user/project/src/file.ts")).toBe(
         true
       );
+      expect(
+        await validator.isPathWithinAllowed("/home/user/project/deep/nested/file.ts")
+      ).toBe(true);
     });
-    it("should return false for paths outside config-allowed directories", () => {
+    it("should return false for paths outside config-allowed directories", async () => {
       const validator = new PathValidator(
         {
           allowedPaths: ["/home/user/project"],
@@ -303,10 +307,10 @@ describe("PathValidator", () => {
         },
         mockLogger
       );
-      expect(validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
-      expect(validator.isPathWithinAllowed("/home/user/other/file.ts")).toBe(false);
+      expect(await validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
+      expect(await validator.isPathWithinAllowed("/home/user/other/file.ts")).toBe(false);
     });
-    it("should NOT consult approval checker (used for prompting decisions)", () => {
+    it("should NOT consult approval checker (used for prompting decisions)", async () => {
       const approvalChecker = vi.fn().mockReturnValue(true);
       const validator = new PathValidator(
         {
@@ -321,10 +325,10 @@ describe("PathValidator", () => {
         mockLogger
       );
       validator.setDirectoryApprovalChecker(approvalChecker);
-      expect(validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
+      expect(await validator.isPathWithinAllowed("/external/project/file.ts")).toBe(false);
       expect(approvalChecker).not.toHaveBeenCalled();
     });
-    it("should return false for empty path", () => {
+    it("should return false for empty path", async () => {
       const validator = new PathValidator(
         {
           allowedPaths: ["/home/user/project"],
@@ -337,10 +341,10 @@ describe("PathValidator", () => {
         },
         mockLogger
       );
-      expect(validator.isPathWithinAllowed("")).toBe(false);
-      expect(validator.isPathWithinAllowed("   ")).toBe(false);
+      expect(await validator.isPathWithinAllowed("")).toBe(false);
+      expect(await validator.isPathWithinAllowed("   ")).toBe(false);
     });
-    it("should return true when allowedPaths is empty (all paths allowed)", () => {
+    it("should return true when allowedPaths is empty (all paths allowed)", async () => {
       const validator = new PathValidator(
         {
           allowedPaths: [],
@@ -353,11 +357,11 @@ describe("PathValidator", () => {
         },
         mockLogger
       );
-      expect(validator.isPathWithinAllowed("/anywhere/file.ts")).toBe(true);
+      expect(await validator.isPathWithinAllowed("/anywhere/file.ts")).toBe(true);
     });
   });
   describe("Path Containment (Parent Directory Coverage)", () => {
-    it("should recognize that approving parent covers child paths", () => {
+    it("should recognize that approving parent covers child paths", async () => {
       const validator = new PathValidator(
         {
           allowedPaths: ["/external/sub"],
@@ -370,9 +374,11 @@ describe("PathValidator", () => {
         },
         mockLogger
       );
-      expect(validator.isPathWithinAllowed("/external/sub/deep/nested/file.ts")).toBe(true);
+      expect(await validator.isPathWithinAllowed("/external/sub/deep/nested/file.ts")).toBe(
+        true
+      );
     });
-    it("should not allow sibling directories", () => {
+    it("should not allow sibling directories", async () => {
       const validator = new PathValidator(
         {
           allowedPaths: ["/external/sub"],
@@ -385,9 +391,9 @@ describe("PathValidator", () => {
         },
         mockLogger
       );
-      expect(validator.isPathWithinAllowed("/external/other/file.ts")).toBe(false);
+      expect(await validator.isPathWithinAllowed("/external/other/file.ts")).toBe(false);
     });
-    it("should not allow parent directories when child is approved", () => {
+    it("should not allow parent directories when child is approved", async () => {
       const validator = new PathValidator(
         {
           allowedPaths: ["/external/sub/deep"],
@@ -400,7 +406,7 @@ describe("PathValidator", () => {
         },
         mockLogger
       );
-      expect(validator.isPathWithinAllowed("/external/sub/file.ts")).toBe(false);
+      expect(await validator.isPathWithinAllowed("/external/sub/file.ts")).toBe(false);
     });
   });
   describe("getAllowedPaths and getBlockedPaths", () => {

package/dist/read-file-tool.cjs

@@ -50,10 +50,10 @@ function createReadFileTool(options) {
      * Check if this read operation needs directory access approval.
      * Returns custom approval request if the file is outside allowed paths.
      */
-    getApprovalOverride: (args) => {
+    getApprovalOverride: async (args) => {
       const { file_path } = args;
       if (!file_path) return null;
-      const isAllowed = fileSystemService.isPathWithinConfigAllowed(file_path);
+      const isAllowed = await fileSystemService.isPathWithinConfigAllowed(file_path);
       if (isAllowed) {
         return null;
       }