@jsenv/core 36.0.2 → 36.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "36.0.2",
+  "version": "36.1.1",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -69,13 +69,13 @@
     "@jsenv/integrity": "0.0.1",
     "@jsenv/log": "3.3.5",
     "@jsenv/node-esm-resolution": "1.0.1",
+    "@jsenv/plugin-supervisor": "1.0.0",
     "@jsenv/server": "15.0.3",
     "@jsenv/sourcemap": "1.0.10",
     "@jsenv/url-meta": "8.1.0",
     "@jsenv/urls": "2.0.0",
     "@jsenv/utils": "2.0.1",
     "construct-style-sheets-polyfill": "3.1.0",
-    "launch-editor": "2.6.0",
     "lightningcss": "1.20.0"
   },
   "devDependencies": {
@@ -92,11 +92,11 @@
     "@jsenv/plugin-placeholders": "./packages/jsenv-plugin-placeholders/",
     "@jsenv/plugin-as-js-classic": "./packages/jsenv-plugin-as-js-classic/",
     "@jsenv/test": "./packages/test/",
-    "eslint": "8.40.0",
+    "eslint": "8.41.0",
     "eslint-plugin-html": "7.1.0",
     "eslint-plugin-import": "2.27.5",
     "eslint-plugin-react": "7.32.2",
-    "playwright": "1.33.0",
+    "playwright": "1.34.2",
     "prettier": "2.8.8"
   }
 }

package/src/build/build.js

@@ -130,6 +130,7 @@ export const build = async ({
   buildDirectoryUrl,
   entryPoints = {},
   assetsDirectory = "",
+  ignore,
 
   runtimeCompat = defaultRuntimeCompat,
   base = runtimeCompat.node ? "./" : "/",
@@ -311,6 +312,10 @@ build ${entryPointKeys.length} entry points`);
       signal,
       logLevel,
       rootDirectoryUrl: sourceDirectoryUrl,
+      ignore,
+      // during first pass (craft) we keep "ignore:" when a reference is ignored
+      // so that the second pass (shape) properly ignore those urls
+      ignoreProtocol: "keep",
       urlGraph: rawGraph,
       build: true,
       runtimeCompat,
@@ -332,13 +337,7 @@ build ${entryPointKeys.length} entry points`);
           rootDirectoryUrl: sourceDirectoryUrl,
           urlGraph: rawGraph,
           runtimeCompat,
-
-          referenceAnalysis: {
-            ...referenceAnalysis,
-            // during first pass (craft) we inject "ignore:" when a reference must be ignored
-            // so that the second pass (shape) properly ignore those urls
-            ignoreProtocol: "inject",
-          },
+          referenceAnalysis,
           nodeEsmResolution,
           magicExtensions,
           magicDirectoryIndex,
@@ -383,6 +382,14 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
     const finalGraphKitchen = createKitchen({
       logLevel,
       rootDirectoryUrl: buildDirectoryUrl,
+      // here most plugins are not there
+      // - no external plugin
+      // - no plugin putting reference.mustIgnore on https urls
+      // At this stage it's only about redirecting urls to the build directory
+      // consequently only a subset or urls are supported
+      supportedProtocols: ["file:", "data:", "virtual:", "ignore:"],
+      ignore,
+      ignoreProtocol: versioning ? "keep" : "remove",
       urlGraph: finalGraph,
       build: true,
       runtimeCompat,
@@ -390,14 +397,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
       plugins: [
         jsenvPluginReferenceAnalysis({
           ...referenceAnalysis,
-          // here most plugins are not there
-          // - no external plugin
-          // - no plugin putting reference.mustIgnore on https urls
-          // At this stage it's only about redirecting urls to the build directory
-          // consequently only a subset or urls are supported
-          supportedProtocols: ["file:", "data:", "virtual:", "ignore:"],
           fetchInlineUrls: false,
-          ignoreProtocol: versioning ? "keep" : "remove",
         }),
         ...(lineBreakNormalization
           ? [jsenvPluginLineBreakNormalization()]
@@ -1063,14 +1063,11 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
           const contentVersionMap = new Map();
           const hashCallbacks = [];
           GRAPH.forEach(finalGraph, (urlInfo) => {
-            if (urlInfo.url.startsWith("data:")) {
-              return;
-            }
             if (urlInfo.type === "sourcemap") {
               return;
             }
             // ignore:
-            // - inline files:
+            // - inline files and data files:
             //   they are already taken into account in the file where they appear
             // - ignored files:
             //   we don't know their content
@@ -1082,7 +1079,10 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
             if (urlInfo.isInline) {
               return;
             }
-            if (urlInfo.mustIgnore) {
+            if (urlInfo.url.startsWith("data:")) {
+              return;
+            }
+            if (urlInfo.url.startsWith("ignore:")) {
               return;
             }
             if (urlInfo.dependents.size === 0 && !urlInfo.isEntryPoint) {
@@ -1116,7 +1116,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
                   );
                   if (!dependencyContentVersion) {
                     // no content generated for this dependency
-                    // (inline, data:, sourcemap, mustIgnore is true, ...)
+                    // (inline, data:, ignore:, sourcemap, ...)
                     return null;
                   }
                   if (preferWithoutVersioning(reference)) {
@@ -1182,20 +1182,15 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
           const versioningKitchen = createKitchen({
             logLevel: logger.level,
             rootDirectoryUrl: buildDirectoryUrl,
+            ignore,
+            ignoreProtocol: "remove",
             urlGraph: finalGraph,
             build: true,
             runtimeCompat,
             ...contextSharedDuringBuild,
             plugins: [
-              // here most plugins are not there
-              // - no external plugin
-              // - no plugin putting reference.mustIgnore on https urls
-              // At this stage it's only about versioning urls
-              // consequently only a subset or urls are supported
               jsenvPluginReferenceAnalysis({
                 ...referenceAnalysis,
-                supportedProtocols: ["file:", "data:", "virtual:"],
-                ignoreProtocol: "remove",
                 fetchInlineUrls: false,
                 inlineConvertedScript: true, // to be able to version their urls
                 allowEscapeForVersioning: true,
@@ -1238,10 +1233,13 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
                   return url;
                 },
                 formatReference: (reference) => {
-                  if (reference.mustIgnore) {
+                  if (reference.url.startsWith("ignore:")) {
+                    return null;
+                  }
+                  if (reference.isInline) {
                     return null;
                   }
-                  if (reference.isInline || reference.url.startsWith("data:")) {
+                  if (reference.url.startsWith("data:")) {
                     return null;
                   }
                   if (reference.isResourceHint) {
@@ -1255,9 +1253,6 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
                   if (!canUseVersionedUrl(referencedUrlInfo)) {
                     return reference.specifier;
                   }
-                  if (referencedUrlInfo.mustIgnore) {
-                    return null;
-                  }
                   const versionedUrl = versionedUrlMap.get(reference.url);
                   if (!versionedUrl) {
                     // happens for sourcemap
@@ -1384,9 +1379,6 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
       }
       cleanup_jsenv_attributes_from_html: {
         GRAPH.forEach(finalGraph, (urlInfo) => {
-          if (urlInfo.mustIgnore) {
-            return;
-          }
           if (!urlInfo.url.startsWith("file:")) {
             return;
           }
@@ -1569,10 +1561,10 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
         if (serviceWorkerEntryUrlInfos.length > 0) {
           const serviceWorkerResources = {};
           GRAPH.forEach(finalGraph, (urlInfo) => {
-            if (urlInfo.isInline || urlInfo.mustIgnore) {
+            if (!urlInfo.url.startsWith("file:")) {
               return;
             }
-            if (!urlInfo.url.startsWith("file:")) {
+            if (urlInfo.isInline) {
               return;
             }
             if (!canUseVersionedUrl(urlInfo)) {
@@ -1642,9 +1634,6 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
       return buildRelativeUrl;
     };
     GRAPH.forEach(finalGraph, (urlInfo) => {
-      if (urlInfo.mustIgnore) {
-        return;
-      }
       if (!urlInfo.url.startsWith("file:")) {
         return;
       }

package/src/dev/file_service.js

@@ -22,6 +22,7 @@ export const createFileService = ({
 
   sourceDirectoryUrl,
   sourceMainFilePath,
+  ignore,
   sourceFilesConfig,
   runtimeCompat,
 
@@ -97,6 +98,7 @@ export const createFileService = ({
       logLevel,
       rootDirectoryUrl: sourceDirectoryUrl,
       mainFilePath: sourceMainFilePath,
+      ignore,
       urlGraph,
       dev: true,
       runtimeCompat,
@@ -252,7 +254,11 @@ export const createFileService = ({
       return responseFromPlugin;
     }
     let reference;
-    const parentUrl = inferParentFromRequest(request, sourceDirectoryUrl);
+    const parentUrl = inferParentFromRequest(
+      request,
+      sourceDirectoryUrl,
+      sourceMainFilePath,
+    );
     if (parentUrl) {
       reference = urlGraph.inferReference(request.resource, parentUrl);
     }
@@ -423,7 +429,11 @@ export const createFileService = ({
   };
 };
 
-const inferParentFromRequest = (request, sourceDirectoryUrl) => {
+const inferParentFromRequest = (
+  request,
+  sourceDirectoryUrl,
+  sourceMainFilePath,
+) => {
   const { referer } = request.headers;
   if (!referer) {
     return null;
@@ -431,7 +441,12 @@ const inferParentFromRequest = (request, sourceDirectoryUrl) => {
   const refererUrlObject = new URL(referer);
   refererUrlObject.searchParams.delete("hmr");
   refererUrlObject.searchParams.delete("v");
-  return WEB_URL_CONVERTER.asFileUrl(referer, {
+
+  let refererUrl = refererUrlObject.href;
+  if (refererUrl === `${request.origin}/`) {
+    refererUrl = new URL(sourceMainFilePath, request.origin).href;
+  }
+  return WEB_URL_CONVERTER.asFileUrl(refererUrl, {
     origin: request.origin,
     rootDirectoryUrl: sourceDirectoryUrl,
   });

package/src/dev/start_dev_server.js

@@ -25,6 +25,7 @@ import { createFileService } from "./file_service.js";
 export const startDevServer = async ({
   sourceDirectoryUrl,
   sourceMainFilePath = "./index.html",
+  ignore,
   port = 3456,
   hostname,
   acceptAnyIp,
@@ -186,6 +187,7 @@ export const startDevServer = async ({
 
           sourceDirectoryUrl,
           sourceMainFilePath,
+          ignore,
           sourceFilesConfig,
           runtimeCompat,
 

package/src/kitchen/kitchen.js

@@ -6,6 +6,7 @@ import {
   normalizeUrl,
   setUrlFilename,
 } from "@jsenv/urls";
+import { URL_META } from "@jsenv/url-meta";
 import { writeFileSync, ensureWindowsDriveLetter } from "@jsenv/filesystem";
 import { createLogger, createDetailedMessage, ANSI } from "@jsenv/log";
 import { CONTENT_TYPE } from "@jsenv/utils/src/content_type/content_type.js";
@@ -29,6 +30,9 @@ export const createKitchen = ({
 
   rootDirectoryUrl,
   mainFilePath,
+  ignore,
+  ignoreProtocol = "remove",
+  supportedProtocols = ["file:", "data:", "virtual:", "http:", "https:"],
   urlGraph,
   dev = false,
   build = false,
@@ -72,6 +76,35 @@ export const createKitchen = ({
     pluginController.pushPlugin(pluginEntry);
   });
 
+  const isIgnoredByProtocol = (url) => {
+    const { protocol } = new URL(url);
+    const protocolIsSupported = supportedProtocols.some(
+      (supportedProtocol) => protocol === supportedProtocol,
+    );
+    return !protocolIsSupported;
+  };
+  let isIgnoredByParam = () => false;
+  if (ignore) {
+    const associations = URL_META.resolveAssociations(
+      { ignore },
+      rootDirectoryUrl,
+    );
+    const cache = new Map();
+    isIgnoredByParam = (url) => {
+      const fromCache = cache.get(url);
+      if (fromCache) return fromCache;
+      const { ignore } = URL_META.applyAssociations({
+        url,
+        associations,
+      });
+      cache.set(url, ignore);
+      return ignore;
+    };
+  }
+  const isIgnored = (url) => {
+    return isIgnoredByProtocol(url) || isIgnoredByParam(url);
+  };
+
   /*
    * - "http_request"
    * - "entry_point"
@@ -116,7 +149,6 @@ export const createKitchen = ({
     specifierColumn,
     baseUrl,
     isOriginalPosition,
-    mustIgnore,
     isEntryPoint = false,
     isResourceHint = false,
     isImplicit = false,
@@ -165,7 +197,6 @@ export const createKitchen = ({
       specifierColumn,
       isOriginalPosition,
       baseUrl,
-      mustIgnore,
       isEntryPoint,
       isResourceHint,
       isImplicit,
@@ -216,13 +247,36 @@ export const createKitchen = ({
       url = normalizeUrl(url);
       let referencedUrlObject;
       let searchParams;
-      const onReferenceUrlChange = (referenceUrl) => {
+      const setReferenceUrl = (referenceUrl) => {
+        // ignored urls are prefixed with "ignore:" so that reference are associated
+        // to a dedicated urlInfo that is ignored.
+        // this way it's only once a resource is referenced by reference that is not ignored
+        // that the resource is cooked
+        if (
+          reference.specifier[0] === "#" &&
+          // For Html, css and "#" refer to a resource in the page, reference must be preserved
+          // However for js import specifiers they have a different meaning and we want
+          // to resolve them (https://nodejs.org/api/packages.html#imports for instance)
+          reference.type !== "js_import"
+        ) {
+          referenceUrl = `ignore:${referenceUrl}`;
+        } else if (isIgnored(referenceUrl)) {
+          referenceUrl = `ignore:${referenceUrl}`;
+        }
+
+        if (
+          referenceUrl.startsWith("ignore:") &&
+          !reference.specifier.startsWith("ignore:")
+        ) {
+          reference.specifier = `ignore:${reference.specifier}`;
+        }
+
         referencedUrlObject = new URL(referenceUrl);
         searchParams = referencedUrlObject.searchParams;
         reference.url = referenceUrl;
         reference.searchParams = searchParams;
       };
-      onReferenceUrlChange(url);
+      setReferenceUrl(url);
 
       if (reference.debug) {
         logger.debug(`url resolved by "${
@@ -251,7 +305,7 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           }
           const prevReference = { ...reference };
           updateReference(prevReference, reference);
-          onReferenceUrlChange(normalizedReturnValue);
+          setReferenceUrl(normalizedReturnValue);
         },
       );
       reference.generatedUrl = reference.url;
@@ -276,12 +330,16 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           reference.generatedUrl = normalizeUrl(referencedUrlObject.href);
         },
       );
+
       const returnValue = pluginController.callHooksUntil(
         "formatReference",
         reference,
         referenceContext,
       );
-      if (reference.mustIgnore) {
+      if (reference.url.startsWith("ignore:")) {
+        if (ignoreProtocol === "remove") {
+          reference.specifier = reference.specifier.slice("ignore:".length);
+        }
         reference.generatedSpecifier = reference.specifier;
         reference.generatedSpecifier = urlSpecifierEncoding.encode(reference);
       } else {
@@ -469,7 +527,7 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
       });
     };
 
-    if (!urlInfo.mustIgnore) {
+    if (!urlInfo.url.startsWith("ignore:")) {
       // references
       const references = [];
       context.referenceUtils = {
@@ -854,11 +912,6 @@ const traceFromUrlSite = (urlSite) => {
 };
 
 const applyReferenceEffectsOnUrlInfo = (reference, urlInfo, context) => {
-  if (reference.mustIgnore) {
-    urlInfo.mustIgnore = true;
-  } else {
-    urlInfo.mustIgnore = false;
-  }
   urlInfo.originalUrl = urlInfo.originalUrl || reference.url;
 
   if (reference.isEntryPoint || isWebWorkerEntryPointReference(reference)) {

package/src/kitchen/url_graph/url_graph_report.js

@@ -30,15 +30,19 @@ const createUrlGraphReport = (urlGraph) => {
     total: 0,
   };
   urlGraph.urlInfoMap.forEach((urlInfo) => {
-    if (urlInfo.url.startsWith("data:")) {
-      return;
-    }
     // ignore:
-    // - inline files: they are already taken into account in the file where they appear
     // - ignored files: we don't know their content
-    if (urlInfo.isInline || urlInfo.mustIgnore) {
+    // - inline files and data files: they are already taken into account in the file where they appear
+    if (urlInfo.url.startsWith("ignore:")) {
+      return;
+    }
+    if (urlInfo.isInline) {
       return;
     }
+    if (urlInfo.url.startsWith("data:")) {
+      return;
+    }
+
     // file loaded via import assertion are already inside the graph
     // their js module equivalent are ignored to avoid counting it twice
     // in the build graph the file targeted by import assertion will likely be gone

package/src/kitchen/url_graph.js

@@ -268,7 +268,6 @@ const createUrlInfo = (url) => {
     originalUrl: undefined,
     filename: "",
     isEntryPoint: false,
-    mustIgnore: undefined,
     originalContent: undefined,
     content: undefined,
 

package/src/plugins/plugin_controller.js

@@ -153,7 +153,11 @@ export const createPluginController = (kitchenContext) => {
       info.timing[`${hook.name}-${hook.plugin.name.replace("jsenv:", "")}`] =
         performance.now() - startTimestamp;
     }
-    valueReturned = assertAndNormalizeReturnValue(hook.name, valueReturned);
+    valueReturned = assertAndNormalizeReturnValue(
+      hook.name,
+      valueReturned,
+      info,
+    );
     return valueReturned;
   };
   const callAsyncHook = async (hook, info, context) => {
@@ -176,7 +180,11 @@ export const createPluginController = (kitchenContext) => {
       info.timing[`${hook.name}-${hook.plugin.name.replace("jsenv:", "")}`] =
         performance.now() - startTimestamp;
     }
-    valueReturned = assertAndNormalizeReturnValue(hook.name, valueReturned);
+    valueReturned = assertAndNormalizeReturnValue(
+      hook.name,
+      valueReturned,
+      info,
+    );
     return valueReturned;
   };
 
@@ -277,7 +285,7 @@ const getHookFunction = (
   return hookValue;
 };
 
-const assertAndNormalizeReturnValue = (hookName, returnValue) => {
+const assertAndNormalizeReturnValue = (hookName, returnValue, info) => {
   // all hooks are allowed to return null/undefined as a signal of "I don't do anything"
   if (returnValue === null || returnValue === undefined) {
     return returnValue;
@@ -286,7 +294,7 @@ const assertAndNormalizeReturnValue = (hookName, returnValue) => {
     if (!returnValueAssertion.appliesTo.includes(hookName)) {
       continue;
     }
-    const assertionResult = returnValueAssertion.assertion(returnValue);
+    const assertionResult = returnValueAssertion.assertion(returnValue, info);
     if (assertionResult !== undefined) {
       // normalization
       returnValue = assertionResult;
@@ -320,13 +328,13 @@ const returnValueAssertions = [
       "finalizeUrlContent",
       "optimizeUrlContent",
     ],
-    assertion: (valueReturned) => {
+    assertion: (valueReturned, urlInfo) => {
       if (typeof valueReturned === "string" || Buffer.isBuffer(valueReturned)) {
         return { content: valueReturned };
       }
       if (typeof valueReturned === "object") {
-        const { mustIgnore, content, body } = valueReturned;
-        if (mustIgnore) {
+        const { content, body } = valueReturned;
+        if (urlInfo.url.startsWith("ignore:")) {
           return undefined;
         }
         if (typeof content !== "string" && !Buffer.isBuffer(content) && !body) {

package/src/plugins/plugins.js

@@ -1,3 +1,5 @@
+import { jsenvPluginSupervisor } from "@jsenv/plugin-supervisor";
+
 import { jsenvPluginReferenceAnalysis } from "./reference_analysis/jsenv_plugin_reference_analysis.js";
 import { jsenvPluginImportmap } from "./importmap/jsenv_plugin_importmap.js";
 import { jsenvPluginNodeEsmResolution } from "./resolution_node_esm/jsenv_plugin_node_esm_resolution.js";
@@ -6,7 +8,6 @@ import { jsenvPluginVersionSearchParam } from "./version_search_param/jsenv_plug
 import { jsenvPluginProtocolFile } from "./protocol_file/jsenv_plugin_protocol_file.js";
 import { jsenvPluginProtocolHttp } from "./protocol_http/jsenv_plugin_protocol_http.js";
 import { jsenvPluginInlining } from "./inlining/jsenv_plugin_inlining.js";
-import { jsenvPluginSupervisor } from "./supervisor/jsenv_plugin_supervisor.js";
 import { jsenvPluginCommonJsGlobals } from "./commonjs_globals/jsenv_plugin_commonjs_globals.js";
 import { jsenvPluginImportMetaScenarios } from "./import_meta_scenarios/jsenv_plugin_import_meta_scenarios.js";
 import { jsenvPluginGlobalScenarios } from "./global_scenarios/jsenv_plugin_global_scenarios.js";

package/src/plugins/protocol_file/jsenv_plugin_protocol_file.js

@@ -30,6 +30,23 @@ export const jsenvPluginProtocolFile = ({
         if (reference.isInline) {
           return null;
         }
+        // ignore root file url
+        if (reference.url === "file:///" || reference.url === "file://") {
+          reference.leadsToADirectory = true;
+          return `ignore:file:///`;
+        }
+        // ignore "./" on new URL("./")
+        if (
+          reference.subtype === "new_url_first_arg" &&
+          reference.specifier === "./"
+        ) {
+          return `ignore:${reference.url}`;
+        }
+        // ignore all new URL second arg
+        if (reference.subtype === "new_url_second_arg") {
+          return `ignore:${reference.url}`;
+        }
+
         const urlObject = new URL(reference.url);
         let stat;
         try {
@@ -41,11 +58,13 @@ export const jsenvPluginProtocolFile = ({
             throw e;
           }
         }
+
         const { search, hash } = urlObject;
         let { pathname } = urlObject;
         const pathnameUsesTrailingSlash = pathname.endsWith("/");
         urlObject.search = "";
         urlObject.hash = "";
+
         // force trailing slash on directories
         if (stat && stat.isDirectory() && !pathnameUsesTrailingSlash) {
           urlObject.pathname = `${pathname}/`;
@@ -57,46 +76,32 @@ export const jsenvPluginProtocolFile = ({
         }
 
         let url = urlObject.href;
-        const mustIgnore =
-          stat &&
-          stat.isDirectory() &&
-          // ignore new URL second arg
-          (reference.subtype === "new_url_second_arg" ||
-            // ignore root file url
-            reference.url === "file:///" ||
-            (reference.subtype === "new_url_first_arg" &&
-              reference.specifier === "./"));
-
-        if (mustIgnore) {
-          reference.mustIgnore = true;
-        } else {
-          const shouldApplyDilesystemMagicResolution =
-            reference.type === "js_import";
-          if (shouldApplyDilesystemMagicResolution) {
-            const filesystemResolution = applyFileSystemMagicResolution(url, {
-              fileStat: stat,
-              magicDirectoryIndex,
-              magicExtensions: getExtensionsToTry(
-                magicExtensions,
-                reference.parentUrl,
-              ),
-            });
-            if (filesystemResolution.stat) {
-              stat = filesystemResolution.stat;
-              url = filesystemResolution.url;
-            }
+        const shouldApplyDilesystemMagicResolution =
+          reference.type === "js_import";
+        if (shouldApplyDilesystemMagicResolution) {
+          const filesystemResolution = applyFileSystemMagicResolution(url, {
+            fileStat: stat,
+            magicDirectoryIndex,
+            magicExtensions: getExtensionsToTry(
+              magicExtensions,
+              reference.parentUrl,
+            ),
+          });
+          if (filesystemResolution.stat) {
+            stat = filesystemResolution.stat;
+            url = filesystemResolution.url;
           }
-          if (stat && stat.isDirectory()) {
-            const directoryAllowed =
-              reference.type === "filesystem" ||
-              (typeof directoryReferenceAllowed === "function" &&
-                directoryReferenceAllowed(reference)) ||
-              directoryReferenceAllowed;
-            if (!directoryAllowed) {
-              const error = new Error("Reference leads to a directory");
-              error.code = "DIRECTORY_REFERENCE_NOT_ALLOWED";
-              throw error;
-            }
+        }
+        if (stat && stat.isDirectory()) {
+          const directoryAllowed =
+            reference.type === "filesystem" ||
+            (typeof directoryReferenceAllowed === "function" &&
+              directoryReferenceAllowed(reference)) ||
+            directoryReferenceAllowed;
+          if (!directoryAllowed) {
+            const error = new Error("Reference leads to a directory");
+            error.code = "DIRECTORY_REFERENCE_NOT_ALLOWED";
+            throw error;
           }
         }
         reference.leadsToADirectory = stat && stat.isDirectory();

package/src/plugins/protocol_http/jsenv_plugin_protocol_http.js

@@ -3,14 +3,15 @@ export const jsenvPluginProtocolHttp = () => {
     name: "jsenv:protocol_http",
     appliesDuring: "*",
     redirectReference: (reference) => {
+      // TODO: according to some pattern matching jsenv could be allowed
+      // to fetch and transform http urls
       if (
         reference.url.startsWith("http:") ||
         reference.url.startsWith("https:")
       ) {
-        reference.mustIgnore = true;
+        return `ignore:${reference.url}`;
       }
-      // TODO: according to some pattern matching jsenv could be allowed
-      // to fetch and transform http urls
+      return null;
     },
   };
 };