src: clean up experimental flag variables

aduh95 · aduh95 · commit dafed453b2bf · 2026-05-07T18:14:57.000+02:00
- `fetch` is no longer disable by a CLI flag - `node:sqlite` requires SQLite, obviously Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com> PR-URL: #62759 Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Xuguang Mei <meixuguang@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
diff --git a/src/api/environment.cc b/src/api/environment.cc
@@ -269,13 +269,9 @@ void SetIsolateMiscHandlers(v8::Isolate* isolate, const IsolateSettings& s) {
   isolate->SetModifyCodeGenerationFromStringsCallback(
       modify_code_generation_from_strings_callback);
 
-  Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
-  if (per_process::cli_options->get_per_isolate_options()
-          ->get_per_env_options()
-          ->experimental_fetch) {
-    isolate->SetWasmStreamingCallback(wasm_web_api::StartStreamingCompilation);
-  }
+  isolate->SetWasmStreamingCallback(wasm_web_api::StartStreamingCompilation);
 
+  Mutex::ScopedLock lock(node::per_process::cli_options_mutex);
   if (per_process::cli_options->get_per_isolate_options()
           ->experimental_shadow_realm) {
     isolate->SetHostCreateShadowRealmContextCallback(
diff --git a/src/node_options.cc b/src/node_options.cc
@@ -582,7 +582,7 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "experimental node:sqlite module",
             &EnvironmentOptions::experimental_sqlite,
             kAllowedInEnvvar,
-            true);
+            HAVE_SQLITE);
   AddOption("--experimental-quic",
 #ifndef OPENSSL_NO_QUIC
             "experimental QUIC support",
diff --git a/src/node_options.h b/src/node_options.h
@@ -124,9 +124,8 @@ class EnvironmentOptions : public Options {
   bool enable_source_maps = false;
   bool experimental_addon_modules = false;
   bool experimental_eventsource = false;
-  bool experimental_fetch = true;
   bool experimental_websocket = true;
-  bool experimental_sqlite = true;
+  bool experimental_sqlite = HAVE_SQLITE;
   bool experimental_webstorage = false;
 #ifndef OPENSSL_NO_QUIC
   bool experimental_quic = false;
diff --git a/test/parallel/test-require-resolve.js b/test/parallel/test-require-resolve.js
@@ -62,6 +62,7 @@ require(fixtures.path('resolve-paths', 'default', 'verify-paths.js'));
   builtinModules.forEach((mod) => {
     // TODO(@jasnell): Remove once node:quic is no longer flagged
     if (mod === 'node:quic') return;
+    if (mod === 'node:sqlite' && !common.hasSQLite) return;
     assert.strictEqual(require.resolve.paths(mod), null);
     if (!mod.startsWith('node:')) {
       assert.strictEqual(require.resolve.paths(`node:${mod}`), null);
