@jsenv/core 27.5.2 → 27.6.1

package/dist/main.js

@@ -16,7 +16,7 @@ import { Readable, Stream, Writable } from "node:stream";
 import { Http2ServerResponse } from "node:http2";
 import { lookup } from "node:dns";
 import { parseHtmlString, stringifyHtmlAst, visitHtmlNodes, getHtmlNodeAttribute, analyzeScriptNode, setHtmlNodeAttributes, parseSrcSet, getHtmlNodePosition, getHtmlNodeAttributePosition, applyPostCss, postCssPluginUrlVisitor, parseJsUrls, findHtmlNode, getHtmlNodeText, removeHtmlNode, setHtmlNodeText, applyBabelPlugins, injectScriptNodeAsEarlyAsPossible, createHtmlNode, removeHtmlNodeText, transpileWithParcel, injectJsImport, minifyWithParcel, analyzeLinkNode } from "@jsenv/ast";
-import { createMagicSource, composeTwoSourcemaps, sourcemapConverter, SOURCEMAP, generateSourcemapFileUrl, generateSourcemapDataUrl } from "@jsenv/sourcemap";
+import { createMagicSource, getOriginalPosition, composeTwoSourcemaps, sourcemapConverter, SOURCEMAP, generateSourcemapFileUrl, generateSourcemapDataUrl } from "@jsenv/sourcemap";
 import { createRequire } from "node:module";
 import babelParser from "@babel/parser";
 import v8, { takeCoverage } from "node:v8";
@@ -152,7 +152,7 @@ const generateInlineContentUrl = ({
   lineEnd,
   columnEnd
 }) => {
-  const generatedName = line === lineEnd ? `L${line}C${column}-L${lineEnd}C${columnEnd}` : `L${line}-L${lineEnd}`;
+  const generatedName = `L${line}C${column}-L${lineEnd}C${columnEnd}`;
   const filenameRaw = urlToFilename$1(url);
   const filename = `${filenameRaw}@${generatedName}${extension}`; // ideally we should keep query params from url
   // maybe we could use a custom scheme like "inline:"
@@ -12046,31 +12046,14 @@ const jsenvPluginHtmlSupervisor = ({
       dev: true,
       test: true
     },
-    serve: (request, context) => {
-      if (request.ressource.startsWith("/__open_in_editor__/")) {
-        const file = request.ressource.slice("/__open_in_editor__/".length);
-
-        if (!file) {
-          return {
-            status: 400,
-            body: "Missing file in url"
-          };
-        }
-
-        const launch = requireFromJsenv("launch-editor");
-        launch(fileURLToPath(file), () => {// ignore error for now
-        });
-        return {
-          status: 200,
-          headers: {
-            "cache-control": "no-store"
-          }
-        };
-      }
-
+    serve: async (request, context) => {
       if (request.ressource.startsWith("/__get_code_frame__/")) {
-        const url = request.ressource.slice("/__get_code_frame__/".length);
-        const match = url.match(/:([0-9]+):([0-9]+)$/);
+        const {
+          pathname,
+          searchParams
+        } = new URL(request.url);
+        const urlWithLineAndColumn = pathname.slice("/__get_code_frame__/".length);
+        const match = urlWithLineAndColumn.match(/:([0-9]+):([0-9]+)$/);
 
         if (!match) {
           return {
@@ -12079,9 +12062,9 @@ const jsenvPluginHtmlSupervisor = ({
           };
         }
 
-        const file = url.slice(0, match.index);
-        const line = parseInt(match[1]);
-        const column = parseInt(match[2]);
+        const file = urlWithLineAndColumn.slice(0, match.index);
+        let line = parseInt(match[1]);
+        let column = parseInt(match[2]);
         const urlInfo = context.urlGraph.getUrlInfo(file);
 
         if (!urlInfo) {
@@ -12090,6 +12073,23 @@ const jsenvPluginHtmlSupervisor = ({
           };
         }
 
+        const remap = searchParams.has("remap");
+
+        if (remap) {
+          const sourcemap = urlInfo.sourcemap;
+
+          if (sourcemap) {
+            const original = await getOriginalPosition({
+              sourcemap,
+              url: file,
+              line,
+              column
+            });
+            line = original.line;
+            column = original.column;
+          }
+        }
+
         const codeFrame = stringifyUrlSite({
           url: file,
           line,
@@ -12106,6 +12106,87 @@ const jsenvPluginHtmlSupervisor = ({
         };
       }
 
+      if (request.ressource.startsWith("/__get_error_cause__/")) {
+        const file = request.ressource.slice("/__get_error_cause__/".length);
+
+        if (!file) {
+          return {
+            status: 400,
+            body: "Missing file in url"
+          };
+        }
+
+        const getErrorCauseInfo = () => {
+          const urlInfo = context.urlGraph.getUrlInfo(file);
+
+          if (!urlInfo) {
+            return null;
+          }
+
+          const {
+            error
+          } = urlInfo;
+
+          if (error) {
+            return error;
+          } // search in direct dependencies (404 or 500)
+
+
+          const {
+            dependencies
+          } = urlInfo;
+
+          for (const dependencyUrl of dependencies) {
+            const dependencyUrlInfo = context.urlGraph.getUrlInfo(dependencyUrl);
+
+            if (dependencyUrlInfo.error) {
+              return dependencyUrlInfo.error;
+            }
+          }
+
+          return null;
+        };
+
+        const causeInfo = getErrorCauseInfo();
+        const body = JSON.stringify(causeInfo ? {
+          code: causeInfo.code,
+          message: causeInfo.message,
+          reason: causeInfo.reason,
+          stack: causeInfo.stack,
+          codeFrame: causeInfo.traceMessage
+        } : null, null, "  ");
+        return {
+          status: 200,
+          headers: {
+            "cache-control": "no-cache",
+            "content-type": "application/json",
+            "content-length": Buffer.byteLength(body)
+          },
+          body
+        };
+      }
+
+      if (request.ressource.startsWith("/__open_in_editor__/")) {
+        const file = request.ressource.slice("/__open_in_editor__/".length);
+
+        if (!file) {
+          return {
+            status: 400,
+            body: "Missing file in url"
+          };
+        }
+
+        const launch = requireFromJsenv("launch-editor");
+        launch(fileURLToPath(file), () => {// ignore error for now
+        });
+        return {
+          status: 200,
+          headers: {
+            "cache-control": "no-store"
+          }
+        };
+      }
+
       return null;
     },
     transformUrlContent: {
@@ -21030,6 +21111,7 @@ const createUrlGraph = ({
 
 const createUrlInfo = url => {
   return {
+    error: null,
     modifiedTimestamp: 0,
     contentEtag: null,
     dependsOnPackageJson: false,
@@ -21435,12 +21517,30 @@ const returnValueAssertions = [{
 const createUrlInfoTransformer = ({
   logger,
   sourcemaps,
+  sourcemapsSourcesProtocol,
   sourcemapsSourcesContent,
   sourcemapsRelativeSources,
   urlGraph,
+  clientRuntimeCompat,
   injectSourcemapPlaceholder,
   foundSourcemap
 }) => {
+  const runtimeNames = Object.keys(clientRuntimeCompat);
+  const chromeAsSingleRuntime = runtimeNames.length === 1 && runtimeNames[0] === "chrome";
+
+  if (sourcemapsSourcesProtocol === undefined) {
+    sourcemapsSourcesProtocol = "file:///";
+  }
+
+  if (sourcemapsSourcesContent === undefined) {
+    if (chromeAsSingleRuntime && sourcemapsSourcesProtocol === "file:///") {
+      // chrome is able to fetch source when referenced with "file:"
+      sourcemapsSourcesContent = false;
+    } else {
+      sourcemapsSourcesContent = true;
+    }
+  }
+
   const sourcemapsEnabled = sourcemaps === "inline" || sourcemaps === "file" || sourcemaps === "programmatic";
 
   const normalizeSourcemap = (urlInfo, sourcemap) => {
@@ -21614,6 +21714,16 @@ const createUrlInfoTransformer = ({
         });
       }
 
+      if (sourcemapsSourcesProtocol !== "file:///") {
+        sourcemap.sources = sourcemap.sources.map(source => {
+          if (source.startsWith("file:///")) {
+            return `${sourcemapsSourcesProtocol}${source.slice("file:///".length)}`;
+          }
+
+          return source;
+        });
+      }
+
       sourcemapUrlInfo.content = JSON.stringify(sourcemap, null, "  ");
 
       if (!urlInfo.sourcemapIsWrong) {
@@ -22013,6 +22123,9 @@ const createKitchen = ({
   rootDirectoryUrl,
   scenario,
   runtimeCompat,
+  // during dev/test clientRuntimeCompat is a single runtime
+  // during build clientRuntimeCompat is runtimeCompat
+  clientRuntimeCompat = runtimeCompat,
   urlGraph,
   plugins,
   sourcemaps = {
@@ -22021,13 +22134,8 @@ const createKitchen = ({
     test: "inline",
     build: "none"
   }[scenario],
-  sourcemapsSourcesContent = {
-    // during dev/test, chrome is able to find the sourcemap sources
-    // as long as they use file:// protocol in the sourcemap files
-    dev: false,
-    test: false,
-    build: true
-  }[scenario],
+  sourcemapsSourcesProtocol,
+  sourcemapsSourcesContent,
   sourcemapsRelativeSources,
   writeGeneratedFiles
 }) => {
@@ -22046,6 +22154,10 @@ const createKitchen = ({
     urlGraph,
     scenario,
     runtimeCompat,
+    clientRuntimeCompat,
+    isSupportedOnCurrentClients: feature => {
+      return RUNTIME_COMPAT.isSupported(clientRuntimeCompat, feature);
+    },
     isSupportedOnFutureClients: feature => {
       return RUNTIME_COMPAT.isSupported(runtimeCompat, feature);
     },
@@ -22194,8 +22306,10 @@ const createKitchen = ({
     logger,
     urlGraph,
     sourcemaps,
+    sourcemapsSourcesProtocol,
     sourcemapsSourcesContent,
     sourcemapsRelativeSources,
+    clientRuntimeCompat,
     injectSourcemapPlaceholder: ({
       urlInfo,
       specifier
@@ -22334,19 +22448,8 @@ const createKitchen = ({
   };
 
   const _cook = async (urlInfo, dishContext) => {
-    // during dev/test clientRuntimeCompat is a single runtime
-    // during build clientRuntimeCompat is runtimeCompat
-    const {
-      clientRuntimeCompat = runtimeCompat
-    } = dishContext;
-
-    kitchenContext.isSupportedOnCurrentClients = feature => {
-      return RUNTIME_COMPAT.isSupported(clientRuntimeCompat, feature);
-    };
-
     const context = { ...kitchenContext,
-      ...dishContext,
-      clientRuntimeCompat
+      ...dishContext
     };
     const {
       cookDuringCook = cook
@@ -22754,7 +22857,7 @@ const applyReferenceEffectsOnUrlInfo = (reference, urlInfo, context) => {
       column: reference.specifierColumn
     };
     urlInfo.contentType = reference.contentType;
-    urlInfo.originalContent = context === "build" ? urlInfo.originalContent === undefined ? reference.content : urlInfo.originalContent : reference.content;
+    urlInfo.originalContent = context.scenario === "build" ? urlInfo.originalContent === undefined ? reference.content : urlInfo.originalContent : reference.content;
     urlInfo.content = reference.content;
   }
 };
@@ -23117,17 +23220,11 @@ const createFileService = ({
   cooldownBetweenFileEvents,
   explorer,
   sourcemaps,
+  sourcemapsSourcesProtocol,
+  sourcemapsSourcesContent,
   writeGeneratedFiles
 }) => {
   const jsenvDirectoryUrl = new URL(".jsenv/", rootDirectoryUrl).href;
-  const onErrorWhileServingFileCallbacks = [];
-
-  const onErrorWhileServingFile = data => {
-    onErrorWhileServingFileCallbacks.forEach(onErrorWhileServingFileCallback => {
-      onErrorWhileServingFileCallback(data);
-    });
-  };
-
   const clientFileChangeCallbackList = [];
   const clientFilesPruneCallbackList = [];
 
@@ -23220,6 +23317,9 @@ const createFileService = ({
       rootDirectoryUrl,
       scenario,
       runtimeCompat,
+      clientRuntimeCompat: {
+        [runtimeName]: runtimeVersion
+      },
       urlGraph,
       plugins: [...plugins, ...getCorePlugins({
         rootDirectoryUrl,
@@ -23236,6 +23336,8 @@ const createFileService = ({
         explorer
       })],
       sourcemaps,
+      sourcemapsSourcesProtocol,
+      sourcemapsSourcesContent,
       writeGeneratedFiles
     });
     serverStopCallbacks.push(() => {
@@ -23276,20 +23378,6 @@ const createFileService = ({
               });
             }
           });
-        });
-        onErrorWhileServingFileCallbacks.push(data => {
-          serverEventsDispatcher.dispatchToRoomsMatching({
-            type: "error_while_serving_file",
-            data
-          }, room => {
-            // send only to page depending on this file
-            const errorFileUrl = data.url;
-            const roomEntryPointUrl = new URL(room.request.ressource.slice(1), rootDirectoryUrl).href;
-            const isErrorRelatedToEntryPoint = Boolean(urlGraph.findDependent(errorFileUrl, dependentUrlInfo => {
-              return dependentUrlInfo.url === roomEntryPointUrl;
-            }));
-            return isErrorRelatedToEntryPoint;
-          });
         }); // "unshift" because serve must come first to catch event source client request
 
         kitchen.pluginController.unshiftPlugin({
@@ -23387,6 +23475,7 @@ const createFileService = ({
     try {
       // urlInfo objects are reused, they must be "reset" before cooking them again
       if (urlInfo.contentEtag && !urlInfo.isInline && urlInfo.type !== "sourcemap") {
+        urlInfo.error = null;
         urlInfo.sourcemap = null;
         urlInfo.sourcemapReference = null;
         urlInfo.content = null;
@@ -23400,9 +23489,6 @@ const createFileService = ({
       await kitchen.cook(urlInfo, {
         request,
         reference,
-        clientRuntimeCompat: {
-          [runtimeName]: runtimeVersion
-        },
         outDirectoryUrl: scenario === "dev" ? `${rootDirectoryUrl}.jsenv/${runtimeName}@${runtimeVersion}/` : `${rootDirectoryUrl}.jsenv/${scenario}/${runtimeName}@${runtimeVersion}/`
       });
       let {
@@ -23434,19 +23520,10 @@ const createFileService = ({
       });
       return response;
     } catch (e) {
+      urlInfo.error = e;
       const code = e.code;
 
       if (code === "PARSE_ERROR") {
-        onErrorWhileServingFile({
-          requestedRessource: request.ressource,
-          code: "PARSE_ERROR",
-          message: e.reason,
-          url: e.url,
-          traceUrl: e.traceUrl,
-          traceLine: e.traceLine,
-          traceColumn: e.traceColumn,
-          traceMessage: e.traceMessage
-        });
         return {
           url: reference.url,
           status: 200,
@@ -23481,17 +23558,6 @@ const createFileService = ({
       }
 
       if (code === "NOT_FOUND") {
-        onErrorWhileServingFile({
-          requestedRessource: request.ressource,
-          isFaviconAutoRequest: request.ressource === "/favicon.ico" && reference.type === "http_request",
-          code: "NOT_FOUND",
-          message: e.reason,
-          url: e.url,
-          traceUrl: e.traceUrl,
-          traceLine: e.traceLine,
-          traceColumn: e.traceColumn,
-          traceMessage: e.traceMessage
-        });
         return {
           url: reference.url,
           status: 404,
@@ -23500,16 +23566,6 @@ const createFileService = ({
         };
       }
 
-      onErrorWhileServingFile({
-        requestedRessource: request.ressource,
-        code: "UNEXPECTED",
-        stack: e.stack,
-        url: e.url,
-        traceUrl: e.traceUrl,
-        traceLine: e.traceLine,
-        traceColumn: e.traceColumn,
-        traceMessage: e.traceMessage
-      });
       return {
         url: reference.url,
         status: 500,
@@ -23578,6 +23634,8 @@ const startOmegaServer = async ({
   cooldownBetweenFileEvents,
   explorer,
   sourcemaps,
+  sourcemapsSourcesProtocol,
+  sourcemapsSourcesContent,
   writeGeneratedFiles
 }) => {
   const serverStopCallbacks = [];
@@ -23624,6 +23682,8 @@ const startOmegaServer = async ({
         cooldownBetweenFileEvents,
         explorer,
         sourcemaps,
+        sourcemapsSourcesProtocol,
+        sourcemapsSourcesContent,
         writeGeneratedFiles
       })
     }, {
@@ -23736,6 +23796,8 @@ const startDevServer = async ({
   },
   // toolbar = false,
   sourcemaps = "inline",
+  sourcemapsSourcesProtocol,
+  sourcemapsSourcesContent,
   writeGeneratedFiles = true
 }) => {
   const logger = createLogger({
@@ -23851,6 +23913,8 @@ const startDevServer = async ({
     cooldownBetweenFileEvents,
     explorer,
     sourcemaps,
+    sourcemapsSourcesProtocol,
+    sourcemapsSourcesContent,
     writeGeneratedFiles
   });
   startDevServerTask.done();
@@ -25715,7 +25779,7 @@ const executeTestPlan = async ({
   keepRunning = false,
   cooldownBetweenExecutions = 0,
   gcBetweenExecutions = logMemoryHeapUsage,
-  coverageEnabled = process.argv.includes("--cover") || process.argv.includes("--coverage"),
+  coverageEnabled = process.argv.includes("--coverage"),
   coverageConfig = {
     "./src/": true
   },
@@ -27466,8 +27530,7 @@ const loadUrlGraph = async ({
   kitchen,
   startLoading,
   writeGeneratedFiles,
-  outDirectoryUrl,
-  clientRuntimeCompat
+  outDirectoryUrl
 }) => {
   if (writeGeneratedFiles && outDirectoryUrl) {
     await ensureEmptyDirectory(outDirectoryUrl);
@@ -27482,7 +27545,6 @@ const loadUrlGraph = async ({
 
     const promise = _cook(urlInfo, {
       outDirectoryUrl,
-      clientRuntimeCompat,
       ...context
     });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "27.5.2",
+  "version": "27.6.1",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -74,7 +74,7 @@
     "@jsenv/log": "3.1.0",
     "@jsenv/node-esm-resolution": "0.1.0",
     "@jsenv/server": "13.0.0",
-    "@jsenv/sourcemap": "1.0.2",
+    "@jsenv/sourcemap": "1.0.4",
     "@jsenv/uneval": "1.6.0",
     "@jsenv/url-meta": "7.0.0",
     "@jsenv/urls": "1.2.7",

package/src/dev/start_dev_server.js

@@ -70,6 +70,8 @@ export const startDevServer = async ({
   // toolbar = false,
 
   sourcemaps = "inline",
+  sourcemapsSourcesProtocol,
+  sourcemapsSourcesContent,
   writeGeneratedFiles = true,
 }) => {
   const logger = createLogger({ logLevel })
@@ -171,6 +173,8 @@ export const startDevServer = async ({
     cooldownBetweenFileEvents,
     explorer,
     sourcemaps,
+    sourcemapsSourcesProtocol,
+    sourcemapsSourcesContent,
     writeGeneratedFiles,
   })
   startDevServerTask.done()

package/src/omega/kitchen.js

@@ -30,6 +30,9 @@ export const createKitchen = ({
   rootDirectoryUrl,
   scenario,
   runtimeCompat,
+  // during dev/test clientRuntimeCompat is a single runtime
+  // during build clientRuntimeCompat is runtimeCompat
+  clientRuntimeCompat = runtimeCompat,
   urlGraph,
   plugins,
   sourcemaps = {
@@ -37,13 +40,8 @@ export const createKitchen = ({
     test: "inline",
     build: "none",
   }[scenario],
-  sourcemapsSourcesContent = {
-    // during dev/test, chrome is able to find the sourcemap sources
-    // as long as they use file:// protocol in the sourcemap files
-    dev: false,
-    test: false,
-    build: true,
-  }[scenario],
+  sourcemapsSourcesProtocol,
+  sourcemapsSourcesContent,
   sourcemapsRelativeSources,
   writeGeneratedFiles,
 }) => {
@@ -60,6 +58,10 @@ export const createKitchen = ({
     urlGraph,
     scenario,
     runtimeCompat,
+    clientRuntimeCompat,
+    isSupportedOnCurrentClients: (feature) => {
+      return RUNTIME_COMPAT.isSupported(clientRuntimeCompat, feature)
+    },
     isSupportedOnFutureClients: (feature) => {
       return RUNTIME_COMPAT.isSupported(runtimeCompat, feature)
     },
@@ -214,12 +216,15 @@ export const createKitchen = ({
     }
   }
   kitchenContext.resolveReference = resolveReference
+
   const urlInfoTransformer = createUrlInfoTransformer({
     logger,
     urlGraph,
     sourcemaps,
+    sourcemapsSourcesProtocol,
     sourcemapsSourcesContent,
     sourcemapsRelativeSources,
+    clientRuntimeCompat,
     injectSourcemapPlaceholder: ({ urlInfo, specifier }) => {
       const sourcemapReference = createReference({
         trace: {
@@ -356,16 +361,9 @@ export const createKitchen = ({
   }
 
   const _cook = async (urlInfo, dishContext) => {
-    // during dev/test clientRuntimeCompat is a single runtime
-    // during build clientRuntimeCompat is runtimeCompat
-    const { clientRuntimeCompat = runtimeCompat } = dishContext
-    kitchenContext.isSupportedOnCurrentClients = (feature) => {
-      return RUNTIME_COMPAT.isSupported(clientRuntimeCompat, feature)
-    }
     const context = {
       ...kitchenContext,
       ...dishContext,
-      clientRuntimeCompat,
     }
     const { cookDuringCook = cook } = dishContext
     context.cook = (urlInfo, nestedDishContext) => {
@@ -772,7 +770,7 @@ const applyReferenceEffectsOnUrlInfo = (reference, urlInfo, context) => {
     }
     urlInfo.contentType = reference.contentType
     urlInfo.originalContent =
-      context === "build"
+      context.scenario === "build"
         ? urlInfo.originalContent === undefined
           ? reference.content
           : urlInfo.originalContent

package/src/omega/omega_server.js

@@ -38,6 +38,8 @@ export const startOmegaServer = async ({
   cooldownBetweenFileEvents,
   explorer,
   sourcemaps,
+  sourcemapsSourcesProtocol,
+  sourcemapsSourcesContent,
   writeGeneratedFiles,
 }) => {
   const serverStopCallbacks = []
@@ -93,6 +95,8 @@ export const startOmegaServer = async ({
           cooldownBetweenFileEvents,
           explorer,
           sourcemaps,
+          sourcemapsSourcesProtocol,
+          sourcemapsSourcesContent,
           writeGeneratedFiles,
         }),
       },