Merge 4662e23 into 169f943

Author: sokra

.prettierignore

@@ -21,5 +21,6 @@ crates/next-core/js/src/compiled
 crates/turbopack-node/js/src/compiled
 crates/turbopack/bench.json
 crates/turbopack/tests
+crates/turbopack-ecmascript/tests/analyzer/graph
 crates/next-transform-strip-page-exports/tests
 crates/next-transform-dynamic/tests

Cargo.lock

@@ -0,0 +1,61 @@
+function func() {
+  if (false) {
+    require("fail");
+    import("fail");
+  }
+  if (true) {
+    require("./ok");
+  }
+  if (true) {
+    require("./ok");
+  } else {
+    require("fail");
+    import("fail");
+  }
+  if (false) {
+    require("fail");
+    import("fail");
+  } else {
+    require("./ok");
+  }
+}
+
+it("should not follow conditional references", () => {
+  func();
+
+  expect(func.toString()).not.toContain("import(");
+});
+
+it("should allow replacements in IIFEs", () => {
+  (function func() {
+    if (false) {
+      require("fail");
+      import("fail");
+    }
+  })();
+});
+
+it("should evaluate process.turbopack", () => {
+  let ok = false;
+  if (process.turbopack) {
+    ok = true;
+  } else {
+    require("fail");
+    import("fail");
+  }
+  expect(ok).toBe(true);
+});
+
+it("should evaluate !process.turbopack", () => {
+  if (!process.turbopack) {
+    require("fail");
+    import("fail");
+  }
+});
+
+// it("should evaluate NODE_ENV", () => {
+//   if (process.env.NODE_ENV !== "development") {
+//     require("fail");
+//     import("fail");
+//   }
+// });

crates/next-dev-tests/tests/integration/turbopack/basic/comptime/index.js

@@ -20,6 +20,8 @@ lazy_static = "1.4.0"
 next-font = { path = "../next-font" }
 next-transform-dynamic = { path = "../next-transform-dynamic" }
 next-transform-strip-page-exports = { path = "../next-transform-strip-page-exports" }
+num-bigint = "0.4"
+num-traits = "0.2.15"
 once_cell = "1.13.0"
 pin-project-lite = "0.2.9"
 regex = "1.5.4"
@@ -58,9 +60,6 @@ swc_core = { workspace = true, features = [
   "base",
 ] }
 
-  [dependencies.num-bigint]
-  version = "0.4"
-
 [dev-dependencies]
 criterion = { version = "0.3.5", features = ["async_tokio"] }
 rstest = "0.12.0"

crates/next-dev-tests/tests/integration/turbopack/basic/comptime/ok.js

@@ -24,7 +24,7 @@ use turbopack_core::{
 use turbopack_ecmascript::analyzer::{
     graph::{create_graph, EvalContext, VarGraph},
     linker::{link, LinkCache},
-    test_utils::visitor,
+    test_utils::{early_visitor, visitor},
 };
 
 pub fn benchmark(c: &mut Criterion) {
@@ -98,24 +98,21 @@ fn bench_link(b: &mut Bencher, input: &BenchInput) {
         let cache = Mutex::new(LinkCache::new());
         for val in input.var_graph.values.values() {
             VcStorage::with(async {
+                let env = EnvironmentVc::new(
+                    Value::new(ExecutionEnvironment::NodeJsLambda(
+                        NodeJsEnvironment {
+                            compile_target: CompileTargetVc::unknown(),
+                            ..Default::default()
+                        }
+                        .into(),
+                    )),
+                    Value::new(EnvironmentIntention::ServerRendering),
+                );
                 link(
                     &input.var_graph,
                     val.clone(),
-                    &(|val| {
-                        Box::pin(visitor(
-                            val,
-                            EnvironmentVc::new(
-                                Value::new(ExecutionEnvironment::NodeJsLambda(
-                                    NodeJsEnvironment {
-                                        compile_target: CompileTargetVc::unknown(),
-                                        ..Default::default()
-                                    }
-                                    .into(),
-                                )),
-                                Value::new(EnvironmentIntention::ServerRendering),
-                            ),
-                        ))
-                    }),
+                    &(|val| early_visitor(val)),
+                    &(|val| visitor(val, env)),
                     &cache,
                 )
                 .await