feat: Return null on error in Deno.dir() (#3531)

1 files changed, 12 insertions, 2 deletions

diff --git a/cli/js/os.ts b/cli/js/os.ts
index baa62c220..8e91be4af 100644
--- a/cli/js/os.ts
+++ b/cli/js/os.ts
@@ -2,6 +2,7 @@
 import { core } from "./core.ts";
 import * as dispatch from "./dispatch.ts";
 import { sendSync } from "./dispatch_json.ts";
+import { ErrorKind } from "./errors.ts";
 import { assert } from "./util.ts";
 import * as util from "./util.ts";
 import { window } from "./window.ts";
@@ -147,6 +148,8 @@ type DirKind =
 /**
  * Returns the user and platform specific directories.
  * Requires the `--allow-env` flag.
+ * Returns null if there is no applicable directory or if any other error
+ * occurs.
  *
  * Argument values: "home", "cache", "config", "data", "data_local", "audio",
  * "desktop", "document", "download", "font", "picture", "public", "template",
@@ -243,8 +246,15 @@ type DirKind =
  * | macOS   | `$HOME`/Movies      | /Users/Alice/Movies   |
  * | Windows | `{FOLDERID_Videos}` | C:\Users\Alice\Videos |
  */
-export function dir(kind: DirKind): string {
-  return sendSync(dispatch.OP_GET_DIR, { kind });
+export function dir(kind: DirKind): string | null {
+  try {
+    return sendSync(dispatch.OP_GET_DIR, { kind });
+  } catch (error) {
+    if (error.kind == ErrorKind.PermissionDenied) {
+      throw error;
+    }
+    return null;
+  }
 }
 
 /**