@jsenv/core 41.0.4 → 41.0.5

package/dist/build/build.js

@@ -3860,7 +3860,11 @@ const jsenvPluginDirectoryReferenceEffect = (
           reference.ownerUrlInfo.filenameHint
         }${urlToFilename(reference.url)}/`;
       } else if (reference.specifierPathname.endsWith("./")) ; else {
-        reference.filenameHint = `${urlToFilename(reference.url)}/`;
+        const directoryRelativeUrl = urlToRelativeUrl(
+          reference.url,
+          reference.ownerUrlInfo.originalUrl,
+        );
+        reference.filenameHint = directoryRelativeUrl;
       }
       let actionForDirectory;
       if (reference.type === "a_href") {
@@ -7497,6 +7501,14 @@ const jsenvPluginImportMetaScenarios = () => {
     appliesDuring: "*",
     transformUrlContent: {
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.dev/import.meta.build
+        // - node modules won't have this in their code
+        // - ;or should use other an other technic as this one won't be available
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (
           !urlInfo.content.includes("import.meta.dev") &&
           !urlInfo.content.includes("import.meta.test") &&
@@ -7593,6 +7605,14 @@ const babelPluginMetadataImportMetaScenarios = () => {
 
 const jsenvPluginGlobalScenarios = () => {
   const transformIfNeeded = (urlInfo) => {
+    // Do not scan node modules for __DEV__/__BUILD__
+    // - node modules won't have this in their code
+    // - ;or should use other an other technic as this one won't be available
+    // They would be discarded by content.includes detection
+    // but it's cheaper to detect by URL than to scan potentially large files
+    if (urlInfo.url.includes("/node_modules/")) {
+      return null;
+    }
     return {
       contentInjections: {
         __DEV__: INJECTIONS.optional(urlInfo.context.dev),
@@ -7662,6 +7682,14 @@ const jsenvPluginImportMetaCss = () => {
     appliesDuring: "*",
     transformUrlContent: {
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.css
+        // - unlikely to be there
+        // - we don't watch node modules (too expensive)
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (!urlInfo.content.includes("import.meta.css")) {
           return null;
         }
@@ -8120,6 +8148,14 @@ const jsenvPluginImportMetaHot = () => {
         cssUrlInfo.data.hotAcceptDependencies = [];
       },
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.hot
+        // - unlikely to be there
+        // - we don't watch node modules (too expensive)
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (!urlInfo.content.includes("import.meta.hot")) {
           return null;
         }

package/dist/start_dev_server/start_dev_server.js

@@ -3724,7 +3724,11 @@ const jsenvPluginDirectoryReferenceEffect = (
           reference.ownerUrlInfo.filenameHint
         }${urlToFilename(reference.url)}/`;
       } else if (reference.specifierPathname.endsWith("./")) ; else {
-        reference.filenameHint = `${urlToFilename(reference.url)}/`;
+        const directoryRelativeUrl = urlToRelativeUrl(
+          reference.url,
+          reference.ownerUrlInfo.originalUrl,
+        );
+        reference.filenameHint = directoryRelativeUrl;
       }
       let actionForDirectory;
       if (reference.type === "a_href") {
@@ -4450,6 +4454,14 @@ const jsenvPluginImportMetaScenarios = () => {
     appliesDuring: "*",
     transformUrlContent: {
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.dev/import.meta.build
+        // - node modules won't have this in their code
+        // - ;or should use other an other technic as this one won't be available
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (
           !urlInfo.content.includes("import.meta.dev") &&
           !urlInfo.content.includes("import.meta.test") &&
@@ -4546,6 +4558,14 @@ const babelPluginMetadataImportMetaScenarios = () => {
 
 const jsenvPluginGlobalScenarios = () => {
   const transformIfNeeded = (urlInfo) => {
+    // Do not scan node modules for __DEV__/__BUILD__
+    // - node modules won't have this in their code
+    // - ;or should use other an other technic as this one won't be available
+    // They would be discarded by content.includes detection
+    // but it's cheaper to detect by URL than to scan potentially large files
+    if (urlInfo.url.includes("/node_modules/")) {
+      return null;
+    }
     return {
       contentInjections: {
         __DEV__: INJECTIONS.optional(urlInfo.context.dev),
@@ -4615,6 +4635,14 @@ const jsenvPluginImportMetaCss = () => {
     appliesDuring: "*",
     transformUrlContent: {
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.css
+        // - unlikely to be there
+        // - we don't watch node modules (too expensive)
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (!urlInfo.content.includes("import.meta.css")) {
           return null;
         }
@@ -5073,6 +5101,14 @@ const jsenvPluginImportMetaHot = () => {
         cssUrlInfo.data.hotAcceptDependencies = [];
       },
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.hot
+        // - unlikely to be there
+        // - we don't watch node modules (too expensive)
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (!urlInfo.content.includes("import.meta.hot")) {
           return null;
         }
@@ -10190,6 +10226,8 @@ const startDevServer = async ({
     port,
     requestWaitingMs: 60_000,
     plugins: finalServerPlugins,
+    // will allow to open file, provide more context on each route
+    canExposeSensitiveData: true,
   });
   server.stoppedPromise.then((reason) => {
     onStop();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "41.0.4",
+  "version": "41.0.5",
   "description": "Tool to develop, test and build js projects",
   "repository": {
     "type": "git",
@@ -81,7 +81,7 @@
     "@jsenv/plugin-minification": "1.7.3",
     "@jsenv/plugin-supervisor": "1.7.15",
     "@jsenv/plugin-transpilation": "1.5.70",
-    "@jsenv/server": "17.0.3",
+    "@jsenv/server": "17.1.0",
     "@jsenv/sourcemap": "1.3.17",
     "react-table": "7.8.0"
   },
@@ -94,6 +94,7 @@
     "@jsenv/assert": "workspace:*",
     "@jsenv/cli": "workspace:*",
     "@jsenv/core": "./",
+    "@jsenv/database-manager": "workspace:*",
     "@jsenv/eslint-config-relax": "workspace:*",
     "@jsenv/file-size-impact": "workspace:*",
     "@jsenv/filesystem": "workspace:*",
@@ -108,7 +109,6 @@
     "@jsenv/os-metrics": "workspace:*",
     "@jsenv/performance-impact": "workspace:*",
     "@jsenv/plugin-as-js-classic": "workspace:*",
-    "@jsenv/plugin-database-manager": "workspace:*",
     "@jsenv/runtime-compat": "workspace:*",
     "@jsenv/snapshot": "workspace:*",
     "@jsenv/terminal-table": "workspace:*",

package/src/dev/start_dev_server.js

@@ -271,6 +271,8 @@ export const startDevServer = async ({
     port,
     requestWaitingMs: 60_000,
     plugins: finalServerPlugins,
+    // will allow to open file, provide more context on each route
+    canExposeSensitiveData: true,
   });
   server.stoppedPromise.then((reason) => {
     onStop();

package/src/plugins/directory_reference_effect/jsenv_plugin_directory_reference_effect.js

@@ -1,5 +1,6 @@
 import { URL_META } from "@jsenv/url-meta";
-import { urlToFilename } from "@jsenv/urls";
+import { urlToFilename, urlToRelativeUrl } from "@jsenv/urls";
+
 import { defineNonEnumerableProperties } from "../../kitchen/errors.js";
 
 export const jsenvPluginDirectoryReferenceEffect = (
@@ -51,7 +52,11 @@ export const jsenvPluginDirectoryReferenceEffect = (
         }${urlToFilename(reference.url)}/`;
       } else if (reference.specifierPathname.endsWith("./")) {
       } else {
-        reference.filenameHint = `${urlToFilename(reference.url)}/`;
+        const directoryRelativeUrl = urlToRelativeUrl(
+          reference.url,
+          reference.ownerUrlInfo.originalUrl,
+        );
+        reference.filenameHint = directoryRelativeUrl;
       }
       let actionForDirectory;
       if (reference.type === "a_href") {

package/src/plugins/global_scenarios/jsenv_plugin_global_scenarios.js

@@ -9,6 +9,14 @@ import { INJECTIONS } from "../../kitchen/url_graph/url_info_injections.js";
 
 export const jsenvPluginGlobalScenarios = () => {
   const transformIfNeeded = (urlInfo) => {
+    // Do not scan node modules for __DEV__/__BUILD__
+    // - node modules won't have this in their code
+    // - ;or should use other an other technic as this one won't be available
+    // They would be discarded by content.includes detection
+    // but it's cheaper to detect by URL than to scan potentially large files
+    if (urlInfo.url.includes("/node_modules/")) {
+      return null;
+    }
     return {
       contentInjections: {
         __DEV__: INJECTIONS.optional(urlInfo.context.dev),

package/src/plugins/import_meta_css/jsenv_plugin_import_meta_css.js

@@ -36,6 +36,14 @@ export const jsenvPluginImportMetaCss = () => {
     appliesDuring: "*",
     transformUrlContent: {
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.css
+        // - unlikely to be there
+        // - we don't watch node modules (too expensive)
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (!urlInfo.content.includes("import.meta.css")) {
           return null;
         }

package/src/plugins/import_meta_hot/jsenv_plugin_import_meta_hot.js

@@ -53,6 +53,14 @@ export const jsenvPluginImportMetaHot = () => {
         cssUrlInfo.data.hotAcceptDependencies = [];
       },
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.hot
+        // - unlikely to be there
+        // - we don't watch node modules (too expensive)
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (!urlInfo.content.includes("import.meta.hot")) {
           return null;
         }

package/src/plugins/import_meta_scenarios/jsenv_plugin_import_meta_scenarios.js

@@ -19,6 +19,14 @@ export const jsenvPluginImportMetaScenarios = () => {
     appliesDuring: "*",
     transformUrlContent: {
       js_module: async (urlInfo) => {
+        // Do not scan node modules for import.meta.dev/import.meta.build
+        // - node modules won't have this in their code
+        // - ;or should use other an other technic as this one won't be available
+        // They would be discarded by content.includes detection
+        // but it's cheaper to detect by URL than to scan potentially large files
+        if (urlInfo.url.includes("/node_modules/")) {
+          return null;
+        }
         if (
           !urlInfo.content.includes("import.meta.dev") &&
           !urlInfo.content.includes("import.meta.test") &&