summaryrefslogtreecommitdiff
path: root/ext/node/polyfills/_fs/_fs_opendir.ts
diff options
context:
space:
mode:
authorBartek IwaƄczuk <biwanczuk@gmail.com>2023-03-04 22:31:38 -0400
committerGitHub <noreply@github.com>2023-03-05 02:31:38 +0000
commitb40086fd7da3729d1d59b312c89ee57747fc66a9 (patch)
tree991583010635feab13fae77e7c8a35fef0a09095 /ext/node/polyfills/_fs/_fs_opendir.ts
parent01028fcdf4f379a7285cc15079306e3ac31edcc1 (diff)
refactor(core): include_js_files! 'dir' option doesn't change specifiers (#18019)
This commit changes "include_js_files!" macro from "deno_core" in a way that "dir" option doesn't cause specifiers to be rewritten to include it. Example: ``` include_js_files! { dir "js", "hello.js", } ``` The above definition required embedders to use: `import ... from "internal:<ext_name>/js/hello.js"`. But with this change, the "js" directory in which the files are stored is an implementation detail, which for embedders results in: `import ... from "internal:<ext_name>/hello.js"`. The directory the files are stored in, is an implementation detail and in some cases might result in a significant size difference for the snapshot. As an example, in "deno_node" extension, we store the source code in "polyfills" directory; which resulted in each specifier to look like "internal:deno_node/polyfills/<module_name>", but with this change it's "internal:deno_node/<module_name>". Given that "deno_node" has over 100 files, many of them having several import specifiers to the same extension, this change removes 10 characters from each import specifier.
Diffstat (limited to 'ext/node/polyfills/_fs/_fs_opendir.ts')
-rw-r--r--ext/node/polyfills/_fs/_fs_opendir.ts12
1 files changed, 6 insertions, 6 deletions
diff --git a/ext/node/polyfills/_fs/_fs_opendir.ts b/ext/node/polyfills/_fs/_fs_opendir.ts
index 5ee13f951..a14f12f15 100644
--- a/ext/node/polyfills/_fs/_fs_opendir.ts
+++ b/ext/node/polyfills/_fs/_fs_opendir.ts
@@ -1,17 +1,17 @@
// Copyright 2018-2023 the Deno authors. All rights reserved. MIT license.
-import Dir from "internal:deno_node/polyfills/_fs/_fs_dir.ts";
-import { Buffer } from "internal:deno_node/polyfills/buffer.ts";
+import Dir from "internal:deno_node/_fs/_fs_dir.ts";
+import { Buffer } from "internal:deno_node/buffer.ts";
import {
getOptions,
getValidatedPath,
-} from "internal:deno_node/polyfills/internal/fs/utils.mjs";
-import { denoErrorToNodeError } from "internal:deno_node/polyfills/internal/errors.ts";
+} from "internal:deno_node/internal/fs/utils.mjs";
+import { denoErrorToNodeError } from "internal:deno_node/internal/errors.ts";
import {
validateFunction,
validateInteger,
-} from "internal:deno_node/polyfills/internal/validators.mjs";
-import { promisify } from "internal:deno_node/polyfills/internal/util.mjs";
+} from "internal:deno_node/internal/validators.mjs";
+import { promisify } from "internal:deno_node/internal/util.mjs";
/** These options aren't funcitonally used right now, as `Dir` doesn't yet support them.
* However, these values are still validated.