1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-12-25 00:29:09 -05:00

refactor(core): remove RuntimeOptions::extensions_with_js (#18099)

This commit removes "deno_core::RuntimeOptions::extensions_with_js".

Now it's embedders' responsibility to properly register extensions
that will not contains JavaScript sources when running from an existing
snapshot.

Prerequisite for https://github.com/denoland/deno/pull/18080
This commit is contained in:
Bartek Iwańczuk 2023-03-09 20:22:27 -04:00 committed by Yoshiya Hinosawa
parent 159f444d16
commit c7528240cb
14 changed files with 100 additions and 159 deletions

View file

@ -9,7 +9,7 @@ use crate::profiling::is_profiling;
pub fn create_js_runtime(setup: impl FnOnce() -> Vec<Extension>) -> JsRuntime { pub fn create_js_runtime(setup: impl FnOnce() -> Vec<Extension>) -> JsRuntime {
JsRuntime::new(RuntimeOptions { JsRuntime::new(RuntimeOptions {
extensions_with_js: setup(), extensions: setup(),
module_loader: Some( module_loader: Some(
std::rc::Rc::new(deno_core::ExtModuleLoader::default()), std::rc::Rc::new(deno_core::ExtModuleLoader::default()),
), ),

View file

@ -276,8 +276,7 @@ mod ts {
cargo_manifest_dir: env!("CARGO_MANIFEST_DIR"), cargo_manifest_dir: env!("CARGO_MANIFEST_DIR"),
snapshot_path, snapshot_path,
startup_snapshot: None, startup_snapshot: None,
extensions: vec![], extensions: vec![tsc_extension],
extensions_with_js: vec![tsc_extension],
// NOTE(bartlomieju): Compressing the TSC snapshot in debug build took // NOTE(bartlomieju): Compressing the TSC snapshot in debug build took
// ~45s on M1 MacBook Pro; without compression it took ~1s. // ~45s on M1 MacBook Pro; without compression it took ~1s.
@ -321,7 +320,7 @@ mod ts {
} }
fn create_cli_snapshot(snapshot_path: PathBuf) { fn create_cli_snapshot(snapshot_path: PathBuf) {
let extensions: Vec<Extension> = vec![ let mut extensions: Vec<Extension> = vec![
deno_webidl::init(), deno_webidl::init(),
deno_console::init(), deno_console::init(),
deno_url::init_ops(), deno_url::init_ops(),
@ -364,20 +363,21 @@ fn create_cli_snapshot(snapshot_path: PathBuf) {
std::path::PathBuf::from(deno_runtime::js::PATH_FOR_99_MAIN_JS), std::path::PathBuf::from(deno_runtime::js::PATH_FOR_99_MAIN_JS),
), ),
}); });
let extensions_with_js = vec![Extension::builder("cli") extensions.push(
Extension::builder("cli")
// FIXME(bartlomieju): information about which extensions were // FIXME(bartlomieju): information about which extensions were
// already snapshotted is not preserved in the snapshot. This should be // already snapshotted is not preserved in the snapshot. This should be
// fixed, so we can reliably depend on that information. // fixed, so we can reliably depend on that information.
// .dependencies(vec!["runtime"]) // .dependencies(vec!["runtime"])
.esm(esm_files) .esm(esm_files)
.build()]; .build(),
);
create_snapshot(CreateSnapshotOptions { create_snapshot(CreateSnapshotOptions {
cargo_manifest_dir: env!("CARGO_MANIFEST_DIR"), cargo_manifest_dir: env!("CARGO_MANIFEST_DIR"),
snapshot_path, snapshot_path,
startup_snapshot: Some(deno_runtime::js::deno_isolate_init()), startup_snapshot: Some(deno_runtime::js::deno_isolate_init()),
extensions, extensions,
extensions_with_js,
compression_cb: Some(Box::new(|vec, snapshot_slice| { compression_cb: Some(Box::new(|vec, snapshot_slice| {
lzzzz::lz4_hc::compress_to_vec( lzzzz::lz4_hc::compress_to_vec(
snapshot_slice, snapshot_slice,

View file

@ -278,7 +278,6 @@ pub async fn run(
inspect: ps.options.is_inspecting(), inspect: ps.options.is_inspecting(),
}, },
extensions: ops::cli_exts(ps), extensions: ops::cli_exts(ps),
extensions_with_js: vec![],
startup_snapshot: Some(crate::js::deno_isolate_init()), startup_snapshot: Some(crate::js::deno_isolate_init()),
unsafely_ignore_certificate_errors: metadata unsafely_ignore_certificate_errors: metadata
.unsafely_ignore_certificate_errors, .unsafely_ignore_certificate_errors,

View file

@ -524,7 +524,6 @@ async fn create_main_worker_internal(
inspect: ps.options.is_inspecting(), inspect: ps.options.is_inspecting(),
}, },
extensions, extensions,
extensions_with_js: vec![],
startup_snapshot: Some(crate::js::deno_isolate_init()), startup_snapshot: Some(crate::js::deno_isolate_init()),
unsafely_ignore_certificate_errors: ps unsafely_ignore_certificate_errors: ps
.options .options
@ -758,7 +757,6 @@ mod tests {
inspect: false, inspect: false,
}, },
extensions: vec![], extensions: vec![],
extensions_with_js: vec![],
startup_snapshot: Some(crate::js::deno_isolate_init()), startup_snapshot: Some(crate::js::deno_isolate_init()),
unsafely_ignore_certificate_errors: None, unsafely_ignore_certificate_errors: None,
root_cert_store: None, root_cert_store: None,

View file

@ -104,7 +104,7 @@ impl Extension {
/// Check if dependencies have been loaded, and errors if either: /// Check if dependencies have been loaded, and errors if either:
/// - The extension is depending on itself or an extension with the same name. /// - The extension is depending on itself or an extension with the same name.
/// - A dependency hasn't been loaded yet. /// - A dependency hasn't been loaded yet.
pub fn check_dependencies(&self, previous_exts: &[&mut Extension]) { pub fn check_dependencies(&self, previous_exts: &[Extension]) {
if let Some(deps) = self.deps { if let Some(deps) = self.deps {
'dep_loop: for dep in deps { 'dep_loop: for dep in deps {
if dep == &self.name { if dep == &self.name {
@ -124,18 +124,12 @@ impl Extension {
/// returns JS source code to be loaded into the isolate (either at snapshotting, /// returns JS source code to be loaded into the isolate (either at snapshotting,
/// or at startup). as a vector of a tuple of the file name, and the source code. /// or at startup). as a vector of a tuple of the file name, and the source code.
pub fn get_js_sources(&self) -> &[ExtensionFileSource] { pub fn get_js_sources(&self) -> Option<&Vec<ExtensionFileSource>> {
match &self.js_files { self.js_files.as_ref()
Some(files) => files,
None => &[],
}
} }
pub fn get_esm_sources(&self) -> &[ExtensionFileSource] { pub fn get_esm_sources(&self) -> Option<&Vec<ExtensionFileSource>> {
match &self.esm_files { self.esm_files.as_ref()
Some(files) => files,
None => &[],
}
} }
pub fn get_esm_entry_point(&self) -> Option<&'static str> { pub fn get_esm_entry_point(&self) -> Option<&'static str> {

View file

@ -1,3 +1,4 @@
use crate::ExtensionBuilder;
// Copyright 2018-2023 the Deno authors. All rights reserved. MIT license. // Copyright 2018-2023 the Deno authors. All rights reserved. MIT license.
use crate::error::format_file_name; use crate::error::format_file_name;
use crate::error::type_error; use crate::error::type_error;
@ -18,14 +19,12 @@ use std::io::stdout;
use std::io::Write; use std::io::Write;
use std::rc::Rc; use std::rc::Rc;
pub(crate) fn init_builtins() -> Extension { fn ext() -> ExtensionBuilder {
Extension::builder("core") Extension::builder("core")
.js(include_js_files!( }
"00_primordials.js",
"01_core.js", fn ops(ext: &mut ExtensionBuilder) -> &mut ExtensionBuilder {
"02_error.js", let mut ops = vec![
))
.ops(vec![
op_close::decl(), op_close::decl(),
op_try_close::decl(), op_try_close::decl(),
op_print::decl(), op_print::decl(),
@ -45,11 +44,25 @@ pub(crate) fn init_builtins() -> Extension {
op_format_file_name::decl(), op_format_file_name::decl(),
op_is_proxy::decl(), op_is_proxy::decl(),
op_str_byte_length::decl(), op_str_byte_length::decl(),
]) ];
.ops(crate::ops_builtin_v8::init_builtins_v8()) ops.extend(crate::ops_builtin_v8::init_builtins_v8());
ext.ops(ops)
}
pub(crate) fn init_builtin_ops_and_esm() -> Extension {
ops(&mut ext())
.js(include_js_files!(
"00_primordials.js",
"01_core.js",
"02_error.js",
))
.build() .build()
} }
pub(crate) fn init_builtin_ops() -> Extension {
ops(&mut ext()).build()
}
/// Return map of resources with id as key /// Return map of resources with id as key
/// and string representation as value. /// and string representation as value.
#[op] #[op]

View file

@ -89,7 +89,6 @@ pub struct JsRuntime {
snapshot_options: SnapshotOptions, snapshot_options: SnapshotOptions,
allocations: IsolateAllocations, allocations: IsolateAllocations,
extensions: Vec<Extension>, extensions: Vec<Extension>,
extensions_with_js: Vec<Extension>,
event_loop_middlewares: Vec<Box<OpEventLoopFn>>, event_loop_middlewares: Vec<Box<OpEventLoopFn>>,
// Marks if this is considered the top-level runtime. Used only be inspector. // Marks if this is considered the top-level runtime. Used only be inspector.
is_main: bool, is_main: bool,
@ -253,20 +252,12 @@ pub struct RuntimeOptions {
/// JsRuntime extensions, not to be confused with ES modules. /// JsRuntime extensions, not to be confused with ES modules.
/// Only ops registered by extensions will be initialized. If you need /// Only ops registered by extensions will be initialized. If you need
/// to execute JS code from extensions, use `extensions_with_js` options /// to execute JS code from extensions, pass source files in `js` or `esm`
/// instead. /// option on `ExtensionBuilder`.
pub extensions: Vec<Extension>,
/// JsRuntime extensions, not to be confused with ES modules.
/// Ops registered by extensions will be initialized and JS code will be
/// executed. If you don't need to execute JS code from extensions, use
/// `extensions` option instead.
/// ///
/// This is useful when creating snapshots, in such case you would pass /// If you are creating a runtime from a snapshot take care not to include
/// extensions using `extensions_with_js`, later when creating a runtime /// JavaScript sources in the extensions.
/// from the snapshot, you would pass these extensions using `extensions` pub extensions: Vec<Extension>,
/// option.
pub extensions_with_js: Vec<Extension>,
/// V8 snapshot that should be loaded on startup. /// V8 snapshot that should be loaded on startup.
pub startup_snapshot: Option<Snapshot>, pub startup_snapshot: Option<Snapshot>,
@ -373,18 +364,15 @@ impl JsRuntime {
let has_startup_snapshot = options.startup_snapshot.is_some(); let has_startup_snapshot = options.startup_snapshot.is_some();
if !has_startup_snapshot { if !has_startup_snapshot {
options options
.extensions_with_js .extensions
.insert(0, crate::ops_builtin::init_builtins()); .insert(0, crate::ops_builtin::init_builtin_ops_and_esm());
} else { } else {
options options
.extensions .extensions
.insert(0, crate::ops_builtin::init_builtins()); .insert(0, crate::ops_builtin::init_builtin_ops());
} }
let ops = Self::collect_ops( let ops = Self::collect_ops(&mut options.extensions);
&mut options.extensions,
&mut options.extensions_with_js,
);
let mut op_state = OpState::new(ops.len()); let mut op_state = OpState::new(ops.len());
if let Some(get_error_class_fn) = options.get_error_class_fn { if let Some(get_error_class_fn) = options.get_error_class_fn {
@ -624,9 +612,12 @@ impl JsRuntime {
let loader = if snapshot_options != SnapshotOptions::Load { let loader = if snapshot_options != SnapshotOptions::Load {
let esm_sources = options let esm_sources = options
.extensions_with_js .extensions
.iter() .iter()
.flat_map(|ext| ext.get_esm_sources().to_owned()) .flat_map(|ext| match ext.get_esm_sources() {
Some(s) => s.to_owned(),
None => vec![],
})
.collect::<Vec<ExtensionFileSource>>(); .collect::<Vec<ExtensionFileSource>>();
#[cfg(feature = "include_js_files_for_snapshotting")] #[cfg(feature = "include_js_files_for_snapshotting")]
@ -689,7 +680,6 @@ impl JsRuntime {
allocations: IsolateAllocations::default(), allocations: IsolateAllocations::default(),
event_loop_middlewares: Vec::with_capacity(options.extensions.len()), event_loop_middlewares: Vec::with_capacity(options.extensions.len()),
extensions: options.extensions, extensions: options.extensions,
extensions_with_js: options.extensions_with_js,
state: state_rc, state: state_rc,
module_map: Some(module_map_rc), module_map: Some(module_map_rc),
is_main: options.is_main, is_main: options.is_main,
@ -752,7 +742,7 @@ impl JsRuntime {
/// Creates a new realm (V8 context) in this JS execution context, /// Creates a new realm (V8 context) in this JS execution context,
/// pre-initialized with all of the extensions that were passed in /// pre-initialized with all of the extensions that were passed in
/// [`RuntimeOptions::extensions_with_js`] when the [`JsRuntime`] was /// [`RuntimeOptions::extensions`] when the [`JsRuntime`] was
/// constructed. /// constructed.
pub fn create_realm(&mut self) -> Result<JsRealm, Error> { pub fn create_realm(&mut self) -> Result<JsRealm, Error> {
let realm = { let realm = {
@ -868,10 +858,10 @@ impl JsRuntime {
} }
// Take extensions to avoid double-borrow // Take extensions to avoid double-borrow
let extensions = std::mem::take(&mut self.extensions_with_js); let extensions = std::mem::take(&mut self.extensions);
for ext in &extensions { for ext in &extensions {
{ {
let esm_files = ext.get_esm_sources(); if let Some(esm_files) = ext.get_esm_sources() {
if let Some(entry_point) = ext.get_esm_entry_point() { if let Some(entry_point) = ext.get_esm_entry_point() {
let file_source = esm_files let file_source = esm_files
.iter() .iter()
@ -884,9 +874,10 @@ impl JsRuntime {
} }
} }
} }
}
{ {
let js_files = ext.get_js_sources(); if let Some(js_files) = ext.get_js_sources() {
for file_source in js_files { for file_source in js_files {
// TODO(@AaronO): use JsRuntime::execute_static() here to move src off heap // TODO(@AaronO): use JsRuntime::execute_static() here to move src off heap
realm.execute_script( realm.execute_script(
@ -897,21 +888,15 @@ impl JsRuntime {
} }
} }
} }
}
// Restore extensions // Restore extensions
self.extensions_with_js = extensions; self.extensions = extensions;
Ok(()) Ok(())
} }
/// Collects ops from extensions & applies middleware /// Collects ops from extensions & applies middleware
fn collect_ops( fn collect_ops(exts: &mut [Extension]) -> Vec<OpDecl> {
extensions: &mut [Extension],
extensions_with_js: &mut [Extension],
) -> Vec<OpDecl> {
let mut exts = vec![];
exts.extend(extensions);
exts.extend(extensions_with_js);
for (ext, previous_exts) in for (ext, previous_exts) in
exts.iter().enumerate().map(|(i, ext)| (ext, &exts[..i])) exts.iter().enumerate().map(|(i, ext)| (ext, &exts[..i]))
{ {
@ -970,24 +955,6 @@ impl JsRuntime {
// Restore extensions // Restore extensions
self.extensions = extensions; self.extensions = extensions;
} }
{
let mut extensions: Vec<Extension> =
std::mem::take(&mut self.extensions_with_js);
// Setup state
for e in extensions.iter_mut() {
// ops are already registered during in bindings::initialize_context();
e.init_state(&mut op_state.borrow_mut());
// Setup event-loop middleware
if let Some(middleware) = e.init_event_loop_middleware() {
self.event_loop_middlewares.push(middleware);
}
}
// Restore extensions
self.extensions_with_js = extensions;
}
Ok(()) Ok(())
} }

View file

@ -17,7 +17,6 @@ pub struct CreateSnapshotOptions {
pub snapshot_path: PathBuf, pub snapshot_path: PathBuf,
pub startup_snapshot: Option<Snapshot>, pub startup_snapshot: Option<Snapshot>,
pub extensions: Vec<Extension>, pub extensions: Vec<Extension>,
pub extensions_with_js: Vec<Extension>,
pub compression_cb: Option<Box<CompressionCb>>, pub compression_cb: Option<Box<CompressionCb>>,
pub snapshot_module_load_cb: Option<ExtModuleLoaderCb>, pub snapshot_module_load_cb: Option<ExtModuleLoaderCb>,
} }
@ -29,7 +28,6 @@ pub fn create_snapshot(create_snapshot_options: CreateSnapshotOptions) {
will_snapshot: true, will_snapshot: true,
startup_snapshot: create_snapshot_options.startup_snapshot, startup_snapshot: create_snapshot_options.startup_snapshot,
extensions: create_snapshot_options.extensions, extensions: create_snapshot_options.extensions,
extensions_with_js: create_snapshot_options.extensions_with_js,
snapshot_module_load_cb: create_snapshot_options.snapshot_module_load_cb, snapshot_module_load_cb: create_snapshot_options.snapshot_module_load_cb,
..Default::default() ..Default::default()
}); });

View file

@ -149,17 +149,7 @@ pub fn init_ops<FP>(options: Options) -> Extension
where where
FP: FetchPermissions + 'static, FP: FetchPermissions + 'static,
{ {
ops::<FP>(&mut ext(), options) ops::<FP>(&mut ext(), options).build()
.esm(include_js_files!(
"20_headers.js",
"21_formdata.js",
"22_body.js",
"22_http_client.js",
"23_request.js",
"23_response.js",
"26_fetch.js",
))
.build()
} }
pub type CancelableResponseFuture = pub type CancelableResponseFuture =

View file

@ -169,7 +169,5 @@ pub fn init_ops_and_esm<P: FfiPermissions + 'static>(
} }
pub fn init_ops<P: FfiPermissions + 'static>(unstable: bool) -> Extension { pub fn init_ops<P: FfiPermissions + 'static>(unstable: bool) -> Extension {
ops::<P>(&mut ext(), unstable) ops::<P>(&mut ext(), unstable).build()
.esm(include_js_files!("00_ffi.js",))
.build()
} }

View file

@ -202,9 +202,7 @@ pub fn init_ops_and_esm<P: FsPermissions + 'static>(
} }
pub fn init_ops<P: FsPermissions + 'static>(unstable: bool) -> Extension { pub fn init_ops<P: FsPermissions + 'static>(unstable: bool) -> Extension {
ops::<P>(&mut ext(), unstable) ops::<P>(&mut ext(), unstable).build()
.esm(include_js_files!("30_fs.js",))
.build()
} }
fn default_err_mapper(err: Error, desc: String) -> Error { fn default_err_mapper(err: Error, desc: String) -> Error {

View file

@ -250,7 +250,7 @@ mod startup_snapshot {
)) ))
.build(); .build();
let mut extensions_with_js: Vec<Extension> = vec![ let mut extensions: Vec<Extension> = vec![
deno_webidl::init_esm(), deno_webidl::init_esm(),
deno_console::init_esm(), deno_console::init_esm(),
deno_url::init_ops_and_esm(), deno_url::init_ops_and_esm(),
@ -291,15 +291,14 @@ mod startup_snapshot {
]; ];
if let Some(additional_extension) = maybe_additional_extension { if let Some(additional_extension) = maybe_additional_extension {
extensions_with_js.push(additional_extension); extensions.push(additional_extension);
} }
create_snapshot(CreateSnapshotOptions { create_snapshot(CreateSnapshotOptions {
cargo_manifest_dir: env!("CARGO_MANIFEST_DIR"), cargo_manifest_dir: env!("CARGO_MANIFEST_DIR"),
snapshot_path, snapshot_path,
startup_snapshot: None, startup_snapshot: None,
extensions: vec![], extensions,
extensions_with_js,
compression_cb: Some(Box::new(|vec, snapshot_slice| { compression_cb: Some(Box::new(|vec, snapshot_slice| {
lzzzz::lz4_hc::compress_to_vec( lzzzz::lz4_hc::compress_to_vec(
snapshot_slice, snapshot_slice,

View file

@ -43,7 +43,6 @@ async fn main() -> Result<(), AnyError> {
inspect: false, inspect: false,
}, },
extensions: vec![], extensions: vec![],
extensions_with_js: vec![],
startup_snapshot: None, startup_snapshot: None,
unsafely_ignore_certificate_errors: None, unsafely_ignore_certificate_errors: None,
root_cert_store: None, root_cert_store: None,

View file

@ -72,21 +72,11 @@ pub struct WorkerOptions {
pub bootstrap: BootstrapOptions, pub bootstrap: BootstrapOptions,
/// JsRuntime extensions, not to be confused with ES modules. /// JsRuntime extensions, not to be confused with ES modules.
/// Only ops registered by extensions will be initialized. If you need
/// to execute JS code from extensions, use `extensions_with_js` options
/// instead.
pub extensions: Vec<Extension>,
/// JsRuntime extensions, not to be confused with ES modules.
/// Ops registered by extensions will be initialized and JS code will be
/// executed. If you don't need to execute JS code from extensions, use
/// `extensions` option instead.
/// ///
/// This is useful when creating snapshots, in such case you would pass /// Extensions register "ops" and JavaScript sources provided in `js` or `esm`
/// extensions using `extensions_with_js`, later when creating a runtime /// configuration. If you are using a snapshot, then extensions shouldn't
/// from the snapshot, you would pass these extensions using `extensions` /// provide JavaScript sources that were already snapshotted.
/// option. pub extensions: Vec<Extension>,
pub extensions_with_js: Vec<Extension>,
/// V8 snapshot that should be loaded on startup. /// V8 snapshot that should be loaded on startup.
pub startup_snapshot: Option<Snapshot>, pub startup_snapshot: Option<Snapshot>,
@ -174,7 +164,6 @@ impl Default for WorkerOptions {
npm_resolver: Default::default(), npm_resolver: Default::default(),
blob_store: Default::default(), blob_store: Default::default(),
extensions: Default::default(), extensions: Default::default(),
extensions_with_js: Default::default(),
startup_snapshot: Default::default(), startup_snapshot: Default::default(),
bootstrap: Default::default(), bootstrap: Default::default(),
stdio: Default::default(), stdio: Default::default(),
@ -299,7 +288,6 @@ impl MainWorker {
shared_array_buffer_store: options.shared_array_buffer_store.clone(), shared_array_buffer_store: options.shared_array_buffer_store.clone(),
compiled_wasm_module_store: options.compiled_wasm_module_store.clone(), compiled_wasm_module_store: options.compiled_wasm_module_store.clone(),
extensions, extensions,
extensions_with_js: options.extensions_with_js,
inspector: options.maybe_inspector_server.is_some(), inspector: options.maybe_inspector_server.is_some(),
is_main: true, is_main: true,
leak_isolate: options.leak_isolate, leak_isolate: options.leak_isolate,