@jsenv/core 39.9.5 → 39.9.7

package/dist/jsenv_core.js

@@ -1485,6 +1485,14 @@ const isValidUrl$1 = (url) => {
   }
 };
 
+const asSpecifierWithoutSearch = (specifier) => {
+  if (isValidUrl$1(specifier)) {
+    return asUrlWithoutSearch(specifier);
+  }
+  const [beforeQuestion] = specifier.split("?");
+  return beforeQuestion;
+};
+
 // normalize url search params:
 // Using URLSearchParams to alter the url search params
 // can result into "file:///file.css?css_module"
@@ -13085,6 +13093,7 @@ const createReference = ({
       );
     }
   }
+
   const reference = {
     id: ++referenceId,
     ownerUrlInfo,
@@ -13107,6 +13116,9 @@ const createReference = ({
     integrity,
     crossorigin,
     specifier,
+    get specifierPathname() {
+      return asSpecifierWithoutSearch(reference.specifier);
+    },
     specifierStart,
     specifierEnd,
     specifierLine,
@@ -14503,11 +14515,15 @@ const createUrlInfoTransformer = ({
     if (!generatedUrl.startsWith("file:")) {
       return;
     }
-    if (
-      urlInfo.type === "directory" ||
-      // happens when type is "html" to list directory content for example
-      urlInfo.firstReference?.fsStat?.isDirectory()
-    ) {
+    if (urlToPathname$1(generatedUrl).endsWith("/")) {
+      // when users explicitely request a directory
+      // we can't write the content returned by the server in ".jsenv" at that url
+      // because it would try to write a directory
+      // ideally we would decide a filename for this
+      // for now we just don't write anything
+      return;
+    }
+    if (urlInfo.type === "directory") {
       // no need to write the directory
       return;
     }
@@ -15598,7 +15614,7 @@ const jsenvPluginDirectoryReferenceEffect = (
         reference.filenameHint = `${
           reference.ownerUrlInfo.filenameHint
         }${urlToFilename$1(reference.url)}/`;
-      } else if (reference.specifier.endsWith("./")) ; else {
+      } else if (reference.specifierPathname.endsWith("./")) ; else {
         reference.filenameHint = `${urlToFilename$1(reference.url)}/`;
       }
       let actionForDirectory;
@@ -18886,7 +18902,7 @@ const createNodeEsmResolver = ({
       return reference.specifier;
     }
     const { ownerUrlInfo } = reference;
-    if (reference.specifier[0] === "/") {
+    if (reference.specifierPathname[0] === "/") {
       const url = new URL(
         reference.specifier.slice(1),
         ownerUrlInfo.context.rootDirectoryUrl,
@@ -19088,7 +19104,7 @@ const jsenvPluginWebResolution = () => {
     appliesDuring: "*",
     resolveReference: (reference) => {
       const { ownerUrlInfo } = reference;
-      if (reference.specifier[0] === "/") {
+      if (reference.specifierPathname[0] === "/") {
         const url = new URL(
           reference.specifier.slice(1),
           ownerUrlInfo.context.rootDirectoryUrl,
@@ -21129,7 +21145,7 @@ const getCorePlugins = ({
       appliesDuring: "*",
       resolveReference: (reference) => {
         const { ownerUrlInfo } = reference;
-        if (reference.specifier === "/") {
+        if (reference.specifierPathname === "/") {
           const { mainFilePath, rootDirectoryUrl } = ownerUrlInfo.context;
           const url = new URL(mainFilePath, rootDirectoryUrl);
           return url;
@@ -21582,7 +21598,7 @@ const createBuildSpecifierManager = ({
         const url = new URL(reference.specifier, ownerRawUrl).href;
         return url;
       }
-      if (reference.specifier[0] === "/") {
+      if (reference.specifierPathname[0] === "/") {
         const url = new URL(reference.specifier.slice(1), sourceDirectoryUrl)
           .href;
         return url;
@@ -21707,6 +21723,7 @@ const createBuildSpecifierManager = ({
           type: reference.type,
           expectedType: reference.expectedType,
           specifier: reference.specifier,
+          specifierPathname: reference.specifierPathname,
           specifierLine: reference.specifierLine,
           specifierColumn: reference.specifierColumn,
           specifierStart: reference.specifierStart,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "39.9.5",
+  "version": "39.9.7",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -69,22 +69,22 @@
   "dependencies": {
     "@financial-times/polyfill-useragent-normaliser": "1.10.2",
     "@jsenv/abort": "4.3.0",
-    "@jsenv/ast": "6.4.3",
-    "@jsenv/filesystem": "4.13.1",
+    "@jsenv/ast": "6.4.4",
+    "@jsenv/filesystem": "4.13.2",
     "@jsenv/humanize": "1.2.8",
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.2",
-    "@jsenv/js-module-fallback": "1.3.55",
+    "@jsenv/js-module-fallback": "1.3.56",
     "@jsenv/node-esm-resolution": "1.0.6",
-    "@jsenv/plugin-bundling": "2.7.23",
+    "@jsenv/plugin-bundling": "2.7.24",
     "@jsenv/plugin-minification": "1.5.13",
-    "@jsenv/plugin-supervisor": "1.6.2",
-    "@jsenv/plugin-transpilation": "1.4.91",
+    "@jsenv/plugin-supervisor": "1.6.3",
+    "@jsenv/plugin-transpilation": "1.4.92",
     "@jsenv/runtime-compat": "1.3.1",
     "@jsenv/server": "15.3.3",
-    "@jsenv/sourcemap": "1.2.29",
+    "@jsenv/sourcemap": "1.2.30",
     "@jsenv/url-meta": "8.5.2",
-    "@jsenv/urls": "2.5.4",
+    "@jsenv/urls": "2.6.0",
     "@jsenv/utils": "2.1.2",
     "string-width": "7.2.0"
   },

package/src/build/build_specifier_manager.js

@@ -170,7 +170,7 @@ export const createBuildSpecifierManager = ({
         const url = new URL(reference.specifier, ownerRawUrl).href;
         return url;
       }
-      if (reference.specifier[0] === "/") {
+      if (reference.specifierPathname[0] === "/") {
         const url = new URL(reference.specifier.slice(1), sourceDirectoryUrl)
           .href;
         return url;
@@ -295,6 +295,7 @@ export const createBuildSpecifierManager = ({
           type: reference.type,
           expectedType: reference.expectedType,
           specifier: reference.specifier,
+          specifierPathname: reference.specifierPathname,
           specifierLine: reference.specifierLine,
           specifierColumn: reference.specifierColumn,
           specifierStart: reference.specifierStart,

package/src/kitchen/url_graph/references.js

@@ -1,6 +1,7 @@
 import { generateUrlForInlineContent } from "@jsenv/ast";
 import { generateContentFrame } from "@jsenv/humanize";
 import {
+  asSpecifierWithoutSearch,
   getCallerPosition,
   stringifyUrlSite,
   urlToBasename,
@@ -315,6 +316,7 @@ const createReference = ({
       );
     }
   }
+
   const reference = {
     id: ++referenceId,
     ownerUrlInfo,
@@ -337,6 +339,9 @@ const createReference = ({
     integrity,
     crossorigin,
     specifier,
+    get specifierPathname() {
+      return asSpecifierWithoutSearch(reference.specifier);
+    },
     specifierStart,
     specifierEnd,
     specifierLine,

package/src/kitchen/url_graph/url_info_transformations.js

@@ -4,7 +4,7 @@ import {
   generateSourcemapDataUrl,
   SOURCEMAP,
 } from "@jsenv/sourcemap";
-import { isFileSystemPath, urlToRelativeUrl } from "@jsenv/urls";
+import { isFileSystemPath, urlToPathname, urlToRelativeUrl } from "@jsenv/urls";
 import { pathToFileURL } from "node:url";
 import {
   defineGettersOnPropertiesDerivedFromContent,
@@ -251,11 +251,15 @@ export const createUrlInfoTransformer = ({
     if (!generatedUrl.startsWith("file:")) {
       return;
     }
-    if (
-      urlInfo.type === "directory" ||
-      // happens when type is "html" to list directory content for example
-      urlInfo.firstReference?.fsStat?.isDirectory()
-    ) {
+    if (urlToPathname(generatedUrl).endsWith("/")) {
+      // when users explicitely request a directory
+      // we can't write the content returned by the server in ".jsenv" at that url
+      // because it would try to write a directory
+      // ideally we would decide a filename for this
+      // for now we just don't write anything
+      return;
+    }
+    if (urlInfo.type === "directory") {
       // no need to write the directory
       return;
     }

package/src/plugins/directory_reference_effect/jsenv_plugin_directory_reference_effect.js

@@ -30,7 +30,7 @@ export const jsenvPluginDirectoryReferenceEffect = (
         reference.filenameHint = `${
           reference.ownerUrlInfo.filenameHint
         }${urlToFilename(reference.url)}/`;
-      } else if (reference.specifier.endsWith("./")) {
+      } else if (reference.specifierPathname.endsWith("./")) {
       } else {
         reference.filenameHint = `${urlToFilename(reference.url)}/`;
       }

package/src/plugins/plugins.js

@@ -86,7 +86,7 @@ export const getCorePlugins = ({
       appliesDuring: "*",
       resolveReference: (reference) => {
         const { ownerUrlInfo } = reference;
-        if (reference.specifier === "/") {
+        if (reference.specifierPathname === "/") {
           const { mainFilePath, rootDirectoryUrl } = ownerUrlInfo.context;
           const url = new URL(mainFilePath, rootDirectoryUrl);
           return url;

package/src/plugins/resolution_node_esm/node_esm_resolver.js

@@ -33,7 +33,7 @@ export const createNodeEsmResolver = ({
       return reference.specifier;
     }
     const { ownerUrlInfo } = reference;
-    if (reference.specifier[0] === "/") {
+    if (reference.specifierPathname[0] === "/") {
       const url = new URL(
         reference.specifier.slice(1),
         ownerUrlInfo.context.rootDirectoryUrl,

package/src/plugins/resolution_web/jsenv_plugin_web_resolution.js

@@ -4,7 +4,7 @@ export const jsenvPluginWebResolution = () => {
     appliesDuring: "*",
     resolveReference: (reference) => {
       const { ownerUrlInfo } = reference;
-      if (reference.specifier[0] === "/") {
+      if (reference.specifierPathname[0] === "/") {
         const url = new URL(
           reference.specifier.slice(1),
           ownerUrlInfo.context.rootDirectoryUrl,