diff options
author | Bartek IwaĆczuk <biwanczuk@gmail.com> | 2023-12-03 00:40:27 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-12-02 23:40:27 +0000 |
commit | f6b889b43219e3c9be770c8b2758bff3048ddcbd (patch) | |
tree | d379a0a92e301b548ba91697804650fb13eba046 /cli/worker.rs | |
parent | 0f990d9d927a0b25bc0eac32f2e7eee7c0460693 (diff) |
refactor: snapshotting of runtime/ and cli/ (#21430)
This commit removes some of the technical debt related
to snapshotting JS code:
- "cli/ops/mod.rs" and "cli/build.rs" no longer define "cli" extension
which was not required anymore
- Cargo features for "deno_runtime" crate have been unified in
"cli/Cargo.toml"
- "cli/build.rs" uses "deno_runtime::snapshot::create_runtime_snapshot"
API
instead of copy-pasting the code
- "cli/js/99_main.js" was completely removed as it's not necessary
anymore
Towards https://github.com/denoland/deno/issues/21137
Diffstat (limited to 'cli/worker.rs')
-rw-r--r-- | cli/worker.rs | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/cli/worker.rs b/cli/worker.rs index 5be64e117..5e9d27918 100644 --- a/cli/worker.rs +++ b/cli/worker.rs @@ -52,7 +52,6 @@ use crate::args::StorageKeyResolver; use crate::emit::Emitter; use crate::errors; use crate::npm::CliNpmResolver; -use crate::ops; use crate::tools; use crate::tools::coverage::CoverageCollector; use crate::tools::run::hmr::HmrRunner; @@ -459,7 +458,7 @@ impl CliMainWorkerFactory { &self, main_module: ModuleSpecifier, permissions: PermissionsContainer, - mut custom_extensions: Vec<Extension>, + custom_extensions: Vec<Extension>, stdio: deno_runtime::deno_io::Stdio, ) -> Result<CliMainWorker, AnyError> { let shared = &self.shared; @@ -564,9 +563,6 @@ impl CliMainWorkerFactory { .join(checksum::gen(&[key.as_bytes()])) }); - let mut extensions = ops::cli_exts(); - extensions.append(&mut custom_extensions); - // TODO(bartlomieju): this is cruft, update FeatureChecker to spit out // list of enabled features. let feature_checker = shared.feature_checker.clone(); @@ -601,7 +597,7 @@ impl CliMainWorkerFactory { .maybe_binary_npm_command_name .clone(), }, - extensions, + extensions: custom_extensions, startup_snapshot: crate::js::deno_isolate_init(), create_params: None, unsafely_ignore_certificate_errors: shared @@ -753,8 +749,6 @@ fn create_web_worker_callback( let create_web_worker_cb = create_web_worker_callback(shared.clone(), stdio.clone()); - let extensions = ops::cli_exts(); - let maybe_storage_key = shared .storage_key_resolver .resolve_storage_key(&args.main_module); @@ -800,7 +794,7 @@ fn create_web_worker_callback( .maybe_binary_npm_command_name .clone(), }, - extensions, + extensions: vec![], startup_snapshot: crate::js::deno_isolate_init(), unsafely_ignore_certificate_errors: shared .options |