Properly track isFile, isSymlink, isDirectory (#4541)

* Properly track isFile, isSymlink, isDirectory

These don't exhaust all the possibilities, so none of them should be
defined as "neither of the others".

* empty

2 files changed, 4 insertions, 1 deletions

diff --git a/cli/js/file_info.ts b/cli/js/file_info.ts
index c5b884f75..27df0bbb7 100644
--- a/cli/js/file_info.ts
+++ b/cli/js/file_info.ts
@@ -25,6 +25,7 @@ export interface FileInfo {
 // @internal
 export class FileInfoImpl implements FileInfo {
   readonly #isFile: boolean;
+  readonly #isDirectory: boolean;
   readonly #isSymlink: boolean;
   size: number;
   modified: number | null;
@@ -53,6 +54,7 @@ export class FileInfoImpl implements FileInfo {
     const { dev, ino, mode, nlink, uid, gid, rdev, blksize, blocks } = res;
 
     this.#isFile = res.isFile;
+    this.#isDirectory = res.isDirectory;
     this.#isSymlink = res.isSymlink;
     this.size = res.size;
     this.modified = modified ? modified : null;
@@ -76,7 +78,7 @@ export class FileInfoImpl implements FileInfo {
   }
 
   isDirectory(): boolean {
-    return !this.#isFile && !this.#isSymlink;
+    return this.#isDirectory;
   }
 
   isSymlink(): boolean {
diff --git a/cli/js/ops/fs/stat.ts b/cli/js/ops/fs/stat.ts
index f455484c1..5d8860939 100644
--- a/cli/js/ops/fs/stat.ts
+++ b/cli/js/ops/fs/stat.ts
@@ -4,6 +4,7 @@ import { FileInfo, FileInfoImpl } from "../../file_info.ts";
 
 export interface StatResponse {
   isFile: boolean;
+  isDirectory: boolean;
   isSymlink: boolean;
   size: number;
   modified: number;