From a9606d0a321462c2ab9b3c46063e6283ab092551 Mon Sep 17 00:00:00 2001 From: OJ Kwon <1210596+kwonoj@users.noreply.github.com> Date: Fri, 12 May 2023 17:39:14 -0700 Subject: [PATCH] refactor(turbopack): consolidate jsx fast refresh --- crates/turbopack/src/module_options/mod.rs | 2 +- crates/turbopack/src/module_options/module_options_context.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/turbopack/src/module_options/mod.rs b/crates/turbopack/src/module_options/mod.rs index 5aee1137fc548..de00016fcc97a 100644 --- a/crates/turbopack/src/module_options/mod.rs +++ b/crates/turbopack/src/module_options/mod.rs @@ -139,7 +139,7 @@ impl ModuleOptionsVc { let jsx = enable_jsx.await?; transforms.push(EcmascriptInputTransform::React { - refresh: enable_react_refresh, + refresh: enable_react_refresh || jsx.react_refresh, import_source: OptionStringVc::cell(jsx.import_source.clone()), runtime: OptionStringVc::cell(jsx.runtime.clone()), }); diff --git a/crates/turbopack/src/module_options/module_options_context.rs b/crates/turbopack/src/module_options/module_options_context.rs index c5098d817f69d..ad082adf8bf31 100644 --- a/crates/turbopack/src/module_options/module_options_context.rs +++ b/crates/turbopack/src/module_options/module_options_context.rs @@ -106,10 +106,10 @@ impl WebpackLoadersOptions { } } -// [TODO]: should enabled_react_refresh belong to this options? #[turbo_tasks::value(shared)] #[derive(Default, Clone, Debug)] pub struct JsxTransformOptions { + pub react_refresh: bool, pub import_source: Option, pub runtime: Option, }