From 99daad054102dbca7fad704d5e81ada1357331fd Mon Sep 17 00:00:00 2001 From: David Sherret Date: Mon, 31 Jul 2023 13:46:58 -0400 Subject: refactor: NodeCodeTranslator - optional source to translate_cjs_to_esm (#20000) --- cli/module_loader.rs | 2 +- ext/node/analyze.rs | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/cli/module_loader.rs b/cli/module_loader.rs index 8395016b3..6f1a23761 100644 --- a/cli/module_loader.rs +++ b/cli/module_loader.rs @@ -782,7 +782,7 @@ impl NpmModuleLoader { // translate cjs to esm if it's cjs and inject node globals self.node_code_translator.translate_cjs_to_esm( specifier, - &code, + Some(code.as_str()), permissions, )? } else { diff --git a/ext/node/analyze.rs b/ext/node/analyze.rs index cf292d82f..902fe9f44 100644 --- a/ext/node/analyze.rs +++ b/ext/node/analyze.rs @@ -72,15 +72,13 @@ impl NodeCodeTranslator { pub fn translate_cjs_to_esm( &self, specifier: &ModuleSpecifier, - source: &str, + source: Option<&str>, permissions: &dyn NodePermissions, ) -> Result { let mut temp_var_count = 0; let mut handled_reexports: HashSet = HashSet::default(); - let analysis = self - .cjs_code_analyzer - .analyze_cjs(specifier, Some(source))?; + let analysis = self.cjs_code_analyzer.analyze_cjs(specifier, source)?; let mut source = vec![ r#"import {createRequire as __internalCreateRequire} from "node:module"; -- cgit v1.2.3