guard webstorage inspector hook behind hasInspector

Author: ryuhei shima

lib/internal/webstorage.js

@@ -2,21 +2,31 @@
 const {
   ObjectDefineProperties,
 } = primordials;
+const { hasInspector } = internalBinding('config');
 const { getOptionValue } = require('internal/options');
 const { kConstructorKey, Storage } = internalBinding('webstorage');
 const { getValidatedPath } = require('internal/fs/utils');
-const { InspectorLocalStorage, InspectorSessionStorage } = require('internal/inspector/webstorage');
 const kInMemoryPath = ':memory:';
 
 module.exports = { Storage };
 
 let lazyLocalStorage;
 let lazySessionStorage;
+let lazyInspectorStorage;
 let localStorageWarned = false;
 
 // Check at load time if localStorage file is provided to determine enumerability.
 // If not provided, localStorage is non-enumerable to avoid breaking {...globalThis}.
 const localStorageLocation = getOptionValue('--localstorage-file');
+const experimentalStorageInspection =
+  hasInspector && getOptionValue('--experimental-storage-inspection');
+
+function getInspectorStorage() {
+  if (lazyInspectorStorage === undefined) {
+    lazyInspectorStorage = require('internal/inspector/webstorage');
+  }
+  return lazyInspectorStorage;
+}
 
 ObjectDefineProperties(module.exports, {
   __proto__: null,
@@ -37,7 +47,8 @@ ObjectDefineProperties(module.exports, {
           return undefined;
         }
 
-        if (getOptionValue('--experimental-storage-inspection')) {
+        if (experimentalStorageInspection) {
+          const { InspectorLocalStorage } = getInspectorStorage();
           lazyLocalStorage = new InspectorLocalStorage(kConstructorKey, getValidatedPath(localStorageLocation), true);
         } else {
           lazyLocalStorage = new Storage(kConstructorKey, getValidatedPath(localStorageLocation));
@@ -52,7 +63,8 @@ ObjectDefineProperties(module.exports, {
     enumerable: true,
     get() {
       if (lazySessionStorage === undefined) {
-        if (getOptionValue('--experimental-storage-inspection')) {
+        if (experimentalStorageInspection) {
+          const { InspectorSessionStorage } = getInspectorStorage();
           lazySessionStorage = new InspectorSessionStorage(kConstructorKey, kInMemoryPath, false);
         } else {
           lazySessionStorage = new Storage(kConstructorKey, kInMemoryPath);

src/node_builtins.cc

@@ -121,6 +121,7 @@ BuiltinLoader::BuiltinCategories BuiltinLoader::GetBuiltinCategories() const {
         "internal/inspector/network", "internal/inspector/network_http",
         "internal/inspector/network_http2", "internal/inspector/network_undici",
         "internal/inspector_async_hook", "internal/inspector_network_tracking",
+        "internal/inspector/webstorage",
 #endif  // !HAVE_INSPECTOR
 
 #if !NODE_USE_V8_PLATFORM || !defined(NODE_HAVE_I18N_SUPPORT)