@jsenv/core 39.1.2 → 39.2.0

package/dist/jsenv_core.js

@@ -11558,6 +11558,7 @@ const createDependencies = (ownerUrlInfo) => {
     isOriginalPosition,
     specifierLine,
     specifierColumn,
+    content,
     ...rest
   }) => {
     const parentUrl = isOriginalPosition
@@ -11578,6 +11579,7 @@ const createDependencies = (ownerUrlInfo) => {
       specifierLine,
       specifierColumn,
       isInline: true,
+      content,
       ...rest,
     });
     return reference;
@@ -11905,6 +11907,12 @@ const createReference = ({
     ownerUrlInfo = reference.ownerUrlInfo,
     ...props
   }) => {
+    const content =
+      ownerUrlInfo === undefined
+        ? isOriginalPosition
+          ? reference.ownerUrlInfo.originalContent
+          : reference.ownerUrlInfo.content
+        : ownerUrlInfo.content;
     const trace = traceFromUrlSite({
       url:
         ownerUrlInfo === undefined
@@ -11912,12 +11920,7 @@ const createReference = ({
             ? reference.ownerUrlInfo.url
             : reference.ownerUrlInfo.generatedUrl
           : reference.ownerUrlInfo.url,
-      content:
-        ownerUrlInfo === undefined
-          ? isOriginalPosition
-            ? reference.ownerUrlInfo.originalContent
-            : reference.ownerUrlInfo.content
-          : ownerUrlInfo.content,
+      content,
       line,
       column,
     });
@@ -12114,14 +12117,15 @@ const applyDependencyRemovalEffects = (reference) => {
 };
 
 const traceFromUrlSite = (urlSite) => {
+  const codeFrame = urlSite.content
+    ? generateContentFrame({
+        content: urlSite.content,
+        line: urlSite.line,
+        column: urlSite.column,
+      })
+    : "";
   return {
-    codeFrame: urlSite.content
-      ? generateContentFrame({
-          content: urlSite.content,
-          line: urlSite.line,
-          column: urlSite.column,
-        })
-      : "",
+    codeFrame,
     message: stringifyUrlSite(urlSite),
     url: urlSite.url,
     line: urlSite.line,
@@ -12131,7 +12135,7 @@ const traceFromUrlSite = (urlSite) => {
 
 const adjustUrlSite = (urlInfo, { url, line, column }) => {
   const isOriginal = url === urlInfo.url;
-  const adjust = (urlSite, urlInfo) => {
+  const adjust = (urlInfo, urlSite) => {
     if (!urlSite.isOriginal) {
       return urlSite;
     }
@@ -12140,33 +12144,35 @@ const adjustUrlSite = (urlInfo, { url, line, column }) => {
       return urlSite;
     }
     const parentUrlInfo = urlInfo.graph.getUrlInfo(inlineUrlSite.url);
-    return adjust(
-      {
-        isOriginal: true,
-        url: inlineUrlSite.url,
-        content: inlineUrlSite.content,
-        line:
-          inlineUrlSite.line === undefined
-            ? urlSite.line
-            : inlineUrlSite.line + urlSite.line,
-        column:
-          inlineUrlSite.column === undefined
-            ? urlSite.column
-            : inlineUrlSite.column + urlSite.column,
-      },
-      parentUrlInfo,
-    );
-  };
-  return adjust(
-    {
-      isOriginal,
-      url,
-      content: isOriginal ? urlInfo.originalContent : urlInfo.content,
+    line =
+      inlineUrlSite.line === undefined
+        ? urlSite.line
+        : inlineUrlSite.line + urlSite.line;
+    // we remove 1 to the line because imagine the following html:
+    // <style>body { color: red; }</style>
+    // -> content starts same line as <style> (same for <script>)
+    if (urlInfo.content[0] === "\n") {
+      line = line - 1;
+    }
+    column =
+      inlineUrlSite.column === undefined
+        ? urlSite.column
+        : inlineUrlSite.column + urlSite.column;
+    return adjust(parentUrlInfo, {
+      isOriginal: true,
+      url: inlineUrlSite.url,
+      content: inlineUrlSite.content,
       line,
       column,
-    },
-    urlInfo,
-  );
+    });
+  };
+  return adjust(urlInfo, {
+    isOriginal,
+    url,
+    content: isOriginal ? urlInfo.originalContent : urlInfo.content,
+    line,
+    column,
+  });
 };
 
 const getRedirectedReferenceProps = (reference, url) => {
@@ -13686,13 +13692,16 @@ const createResolveUrlError = ({
     ...details
   }) => {
     const resolveError = new Error(
-      createDetailedMessage$1(`Failed to resolve url reference`, {
-        reason,
-        ...details,
-        "specifier": `"${reference.specifier}"`,
-        "specifier trace": reference.trace.message,
-        ...detailsFromPluginController(pluginController),
-      }),
+      createDetailedMessage$1(
+        `Failed to resolve url reference
+${reference.trace.message}
+${reason}`,
+        {
+          ...detailsFromFirstReference(reference),
+          ...details,
+          ...detailsFromPluginController(pluginController),
+        },
+      ),
     );
     resolveError.name = "RESOLVE_URL_ERROR";
     resolveError.code = code;
@@ -13727,14 +13736,18 @@ const createFetchUrlContentError = ({
     reason,
     ...details
   }) => {
+    const reference = urlInfo.firstReference;
     const fetchError = new Error(
-      createDetailedMessage$1(`Failed to fetch url content`, {
-        reason,
-        ...details,
-        "url": urlInfo.url,
-        "url reference trace": urlInfo.firstReference.trace.message,
-        ...detailsFromPluginController(pluginController),
-      }),
+      createDetailedMessage$1(
+        `Failed to fetch url content
+${reference.trace.message}
+${reason}`,
+        {
+          ...detailsFromFirstReference(reference),
+          ...details,
+          ...detailsFromPluginController(pluginController),
+        },
+      ),
     );
     fetchError.name = "FETCH_URL_CONTENT_ERROR";
     fetchError.code = code;
@@ -13743,7 +13756,7 @@ const createFetchUrlContentError = ({
     if (code === "PARSE_ERROR") {
       fetchError.trace = error.trace;
     } else {
-      fetchError.trace = urlInfo.firstReference.trace;
+      fetchError.trace = reference.trace;
     }
     fetchError.asResponse = error.asResponse;
     return fetchError;
@@ -13792,73 +13805,101 @@ const createTransformUrlContentError = ({
   urlInfo,
   error,
 }) => {
+  if (error.code === "MODULE_NOT_FOUND") {
+    return error;
+  }
   if (error.code === "DIRECTORY_REFERENCE_NOT_ALLOWED") {
     return error;
   }
+  if (error.code === "PARSE_ERROR") {
+    const reference = urlInfo.firstReference;
+    let trace = reference.trace;
+    let line = error.line;
+    let column = error.column;
+    if (urlInfo.isInline) {
+      line = trace.line + line;
+      line = line - 1;
+      trace = {
+        ...trace,
+        line,
+        column,
+        codeFrame: generateContentFrame({
+          line,
+          column,
+          content: urlInfo.inlineUrlSite.content,
+        }),
+        message: stringifyUrlSite({
+          url: urlInfo.inlineUrlSite.url,
+          line,
+          column,
+          content: urlInfo.inlineUrlSite.content,
+        }),
+      };
+    } else {
+      trace = {
+        url: urlInfo.url,
+        line,
+        column: error.column,
+        codeFrame: generateContentFrame({
+          line,
+          column: error.column,
+          content: urlInfo.content,
+        }),
+        message: stringifyUrlSite({
+          url: urlInfo.url,
+          line,
+          column: error.column,
+          content: urlInfo.content,
+        }),
+      };
+    }
+    const transformError = new Error(
+      createDetailedMessage$1(
+        `parse error on "${urlInfo.type}"
+${trace.message}
+${error.message}`,
+        {
+          "first reference": `${reference.trace.url}:${reference.trace.line}:${reference.trace.column}`,
+          ...detailsFromFirstReference(reference),
+          ...detailsFromPluginController(pluginController),
+        },
+      ),
+    );
+    transformError.cause = error;
+    transformError.name = "TRANSFORM_URL_CONTENT_ERROR";
+    transformError.code = "PARSE_ERROR";
+    transformError.stack = error.stack;
+    transformError.reason = error.message;
+    transformError.trace = trace;
+    transformError.asResponse = error.asResponse;
+    return transformError;
+  }
   const createFailedToTransformError = ({
     code = error.code || "TRANSFORM_URL_CONTENT_ERROR",
     reason,
     ...details
   }) => {
+    const reference = urlInfo.firstReference;
+    let trace = reference.trace;
     const transformError = new Error(
       createDetailedMessage$1(
-        `"transformUrlContent" error on "${urlInfo.type}"`,
+        `"transformUrlContent" error on "${urlInfo.type}"
+${trace.message}
+${reason}`,
         {
-          reason,
+          ...detailsFromFirstReference(reference),
           ...details,
-          "url": urlInfo.url,
-          "url reference trace": urlInfo.firstReference.trace.message,
           ...detailsFromPluginController(pluginController),
         },
       ),
     );
+    transformError.cause = error;
     transformError.name = "TRANSFORM_URL_CONTENT_ERROR";
     transformError.code = code;
     transformError.reason = reason;
     transformError.stack = error.stack;
     transformError.url = urlInfo.url;
-    transformError.trace = urlInfo.firstReference.trace;
-    if (code === "PARSE_ERROR") {
-      transformError.reason = `parse error on ${urlInfo.type}`;
-      transformError.cause = error;
-      let line = error.line;
-      if (urlInfo.type === "js_module") {
-        line = line - 1;
-      }
-      if (urlInfo.isInline) {
-        transformError.trace.line = urlInfo.firstReference.trace.line + line;
-        transformError.trace.column =
-          urlInfo.firstReference.trace.column + error.column;
-        transformError.trace.codeFrame = generateContentFrame({
-          line: transformError.trace.line,
-          column: transformError.trace.column,
-          content: urlInfo.inlineUrlSite.content,
-        });
-        transformError.trace.message = stringifyUrlSite({
-          url: urlInfo.inlineUrlSite.url,
-          line: transformError.trace.line,
-          column: transformError.trace.column,
-          content: urlInfo.inlineUrlSite.content,
-        });
-      } else {
-        transformError.trace = {
-          url: urlInfo.url,
-          line,
-          column: error.column,
-          codeFrame: generateContentFrame({
-            line,
-            column: error.column,
-            content: urlInfo.content,
-          }),
-          message: stringifyUrlSite({
-            url: urlInfo.url,
-            line,
-            column: error.column,
-            content: urlInfo.content,
-          }),
-        };
-      }
-    }
+    transformError.trace = trace;
     transformError.asResponse = error.asResponse;
     return transformError;
   };
@@ -13870,17 +13911,20 @@ const createTransformUrlContentError = ({
 
 const createFinalizeUrlContentError = ({
   pluginController,
-
   urlInfo,
   error,
 }) => {
+  const reference = urlInfo.firstReference;
   const finalizeError = new Error(
-    createDetailedMessage$1(`"finalizeUrlContent" error on "${urlInfo.type}"`, {
-      ...detailsFromValueThrown(error),
-      "url": urlInfo.url,
-      "url reference trace": urlInfo.firstReference.trace.message,
-      ...detailsFromPluginController(pluginController),
-    }),
+    createDetailedMessage$1(
+      `"finalizeUrlContent" error on "${urlInfo.type}"
+${reference.trace.message}`,
+      {
+        ...detailsFromFirstReference(reference),
+        ...detailsFromValueThrown(error),
+        ...detailsFromPluginController(pluginController),
+      },
+    ),
   );
   if (error && error instanceof Error) {
     finalizeError.cause = error;
@@ -13891,6 +13935,23 @@ const createFinalizeUrlContentError = ({
   return finalizeError;
 };
 
+const detailsFromFirstReference = (reference) => {
+  const referenceInProject = getFirstReferenceInProject(reference);
+  if (referenceInProject === reference) {
+    return {};
+  }
+  return {
+    "first reference in project": `${referenceInProject.trace.url}:${referenceInProject.trace.line}:${referenceInProject.trace.column}`,
+  };
+};
+const getFirstReferenceInProject = (reference) => {
+  const ownerUrlInfo = reference.ownerUrlInfo;
+  if (!ownerUrlInfo.url.includes("/node_modules/")) {
+    return reference;
+  }
+  return getFirstReferenceInProject(ownerUrlInfo.firstReference);
+};
+
 const detailsFromPluginController = (pluginController) => {
   const currentPlugin = pluginController.getCurrentPlugin();
   if (!currentPlugin) {
@@ -13901,6 +13962,18 @@ const detailsFromPluginController = (pluginController) => {
 
 const detailsFromValueThrown = (valueThrownByPlugin) => {
   if (valueThrownByPlugin && valueThrownByPlugin instanceof Error) {
+    if (
+      valueThrownByPlugin.code === "PARSE_ERROR" ||
+      valueThrownByPlugin.code === "MODULE_NOT_FOUND" ||
+      valueThrownByPlugin.name === "RESOLVE_URL_ERROR" ||
+      valueThrownByPlugin.name === "FETCH_URL_CONTENT_ERROR" ||
+      valueThrownByPlugin.name === "TRANSFORM_URL_CONTENT_ERROR" ||
+      valueThrownByPlugin.name === "FINALIZE_URL_CONTENT_ERROR"
+    ) {
+      return {
+        "error message": valueThrownByPlugin.message,
+      };
+    }
     return {
       "error stack": valueThrownByPlugin.stack,
     };
@@ -14520,16 +14593,21 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           e.code !== "DIRECTORY_REFERENCE_NOT_ALLOWED" &&
           errorOnInlineContentCanSkipThrow(urlInfo)
         ) {
-          const errorInfo =
-            e.code === "PARSE_ERROR" && e.cause
-              ? `${e.cause.reasonCode}\n${e.trace?.message}`
-              : e.stack;
           // When something like <style> or <script> contains syntax error
           // the HTML in itself it still valid
           // keep the syntax error and continue with the HTML
+          if (e.code === "PARSE_ERROR") {
+            logger.error(`parse error on "${urlInfo.type}"
+${e.trace?.message}
+${e.reason}
+--- declared in ---
+${urlInfo.firstReference.trace.message}`);
+            return;
+          }
           logger.error(
-            `Error while cooking ${urlInfo.type} declared in ${urlInfo.firstReference.trace.message}:
-${errorInfo}`,
+            `Error while cooking ${urlInfo.type}
+${urlInfo.firstReference.trace.message}
+${e.stack}`,
           );
           return;
         }
@@ -16101,10 +16179,7 @@ const jsenvPluginHtmlReferenceAnalysis = ({
               type,
               expectedType,
               isOriginalPosition: isOriginal,
-              // we remove 1 to the line because imagine the following html:
-              // <style>body { color: red; }</style>
-              // -> content starts same line as <style> (same for <script>)
-              specifierLine: line - 1,
+              specifierLine: line,
               specifierColumn: column,
               specifier: inlineContentUrl,
               contentType,
@@ -16223,7 +16298,7 @@ const jsenvPluginHtmlReferenceAnalysis = ({
                     },
                   );
                   const importmapReferenceInlined = importmapReference.inline({
-                    line: line - 1,
+                    line,
                     column,
                     isOriginal,
                     specifier: importmapInlineUrl,
@@ -18777,7 +18852,7 @@ const jsenvPluginInliningIntoHtml = () => {
             url: linkReference.url,
           });
           const linkReferenceInlined = linkReference.inline({
-            line: line - 1,
+            line,
             column,
             isOriginal,
             specifier: linkInlineUrl,
@@ -18829,7 +18904,7 @@ const jsenvPluginInliningIntoHtml = () => {
             url: scriptReference.url,
           });
           const scriptReferenceInlined = scriptReference.inline({
-            line: line - 1,
+            line,
             column,
             isOriginal,
             specifier: scriptInlineUrl,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "39.1.2",
+  "version": "39.2.0",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -44,21 +44,21 @@
   ],
   "scripts": {
     "eslint": "npx eslint . --ext=.js,.mjs,.cjs,.html",
-    "dev": "node --conditions=development ./scripts/dev/dev.mjs",
     "test": "node --conditions=development ./scripts/test/test.mjs",
-    "test:workspace": "npm run test --workspaces --if-present -- --workspace",
-    "test:ci": "CI=1 npm run test",
-    "test:workspace:ci": "CI=1 npm run test:workspace",
-    "test:only_dev_server_errors": "node --conditions=development ./tests/dev_server/errors/dev_errors_snapshots.test.mjs",
     "build": "node --conditions=development ./scripts/build/build.mjs",
-    "build:file_size": "node ./scripts/build/build_file_size.mjs --log",
-    "build:workspace": "npm run build --workspaces --if-present --conditions=developement",
-    "monorepo:upgrade_versions": "node ./scripts/monorepo/upgrade_external_versions.mjs",
+    "workspace:test": "npm run test --workspaces --if-present -- --workspace",
+    "workspace:build": "npm run build --workspaces --if-present --conditions=developement",
     "monorepo:sync_packages_versions": "node ./scripts/monorepo/sync_packages_versions.mjs",
     "monorepo:publish": "node ./scripts/monorepo/publish_packages.mjs",
+    "monorepo:upgrade_versions": "node ./scripts/monorepo/upgrade_external_versions.mjs",
     "monorepo:node_modules_clear": "npx @jsenv/snapshot clear **/node_modules/",
+    "build:file_size": "node ./scripts/build/build_file_size.mjs --log",
     "performances": "node --expose-gc ./scripts/performance/generate_performance_report.mjs --log --once",
     "prettier": "prettier --write .",
+    "test:ci": "CI=1 npm run test",
+    "test:only_dev_server_errors": "node --conditions=development ./tests/dev_server/errors/dev_errors_snapshots.test.mjs",
+    "workspace:test:ci": "CI=1 npm run test:workspace",
+    "dev": "node --conditions=development ./scripts/dev/dev.mjs",
     "certificate:install": "node ./scripts/dev/install_certificate_authority.mjs",
     "prepublishOnly": "npm run build"
   },
@@ -73,8 +73,8 @@
     "@jsenv/js-module-fallback": "1.3.23",
     "@jsenv/node-esm-resolution": "1.0.2",
     "@jsenv/plugin-bundling": "2.6.16",
-    "@jsenv/plugin-minification": "1.5.4",
-    "@jsenv/plugin-supervisor": "1.5.2",
+    "@jsenv/plugin-minification": "1.5.5",
+    "@jsenv/plugin-supervisor": "1.5.3",
     "@jsenv/plugin-transpilation": "1.4.6",
     "@jsenv/runtime-compat": "1.3.0",
     "@jsenv/server": "15.2.13",
@@ -98,18 +98,18 @@
     "@jsenv/plugin-as-js-classic": "./packages/related/plugin-as-js-classic/",
     "@jsenv/snapshot": "./packages/independent/snapshot/",
     "@jsenv/test": "./packages/related/test/",
-    "@playwright/browser-chromium": "1.45.1",
-    "@playwright/browser-firefox": "1.45.1",
-    "@playwright/browser-webkit": "1.45.1",
+    "@playwright/browser-chromium": "1.45.2",
+    "@playwright/browser-firefox": "1.45.2",
+    "@playwright/browser-webkit": "1.45.2",
     "babel-plugin-transform-async-to-promises": "0.8.18",
     "eslint": "8.56.0",
     "eslint-plugin-html": "8.1.1",
     "eslint-plugin-import": "2.29.1",
-    "eslint-plugin-react": "7.34.3",
+    "eslint-plugin-react": "7.34.4",
     "eslint-plugin-regexp": "2.6.0",
     "marked": "13.0.2",
     "open": "10.1.0",
-    "playwright": "1.45.1",
-    "prettier": "3.3.2"
+    "playwright": "1.45.2",
+    "prettier": "3.3.3"
   }
 }

package/src/kitchen/errors.js

@@ -13,13 +13,16 @@ export const createResolveUrlError = ({
     ...details
   }) => {
     const resolveError = new Error(
-      createDetailedMessage(`Failed to resolve url reference`, {
-        reason,
-        ...details,
-        "specifier": `"${reference.specifier}"`,
-        "specifier trace": reference.trace.message,
-        ...detailsFromPluginController(pluginController),
-      }),
+      createDetailedMessage(
+        `Failed to resolve url reference
+${reference.trace.message}
+${reason}`,
+        {
+          ...detailsFromFirstReference(reference),
+          ...details,
+          ...detailsFromPluginController(pluginController),
+        },
+      ),
     );
     resolveError.name = "RESOLVE_URL_ERROR";
     resolveError.code = code;
@@ -54,14 +57,18 @@ export const createFetchUrlContentError = ({
     reason,
     ...details
   }) => {
+    const reference = urlInfo.firstReference;
     const fetchError = new Error(
-      createDetailedMessage(`Failed to fetch url content`, {
-        reason,
-        ...details,
-        "url": urlInfo.url,
-        "url reference trace": urlInfo.firstReference.trace.message,
-        ...detailsFromPluginController(pluginController),
-      }),
+      createDetailedMessage(
+        `Failed to fetch url content
+${reference.trace.message}
+${reason}`,
+        {
+          ...detailsFromFirstReference(reference),
+          ...details,
+          ...detailsFromPluginController(pluginController),
+        },
+      ),
     );
     fetchError.name = "FETCH_URL_CONTENT_ERROR";
     fetchError.code = code;
@@ -70,7 +77,7 @@ export const createFetchUrlContentError = ({
     if (code === "PARSE_ERROR") {
       fetchError.trace = error.trace;
     } else {
-      fetchError.trace = urlInfo.firstReference.trace;
+      fetchError.trace = reference.trace;
     }
     fetchError.asResponse = error.asResponse;
     return fetchError;
@@ -119,73 +126,101 @@ export const createTransformUrlContentError = ({
   urlInfo,
   error,
 }) => {
+  if (error.code === "MODULE_NOT_FOUND") {
+    return error;
+  }
   if (error.code === "DIRECTORY_REFERENCE_NOT_ALLOWED") {
     return error;
   }
+  if (error.code === "PARSE_ERROR") {
+    const reference = urlInfo.firstReference;
+    let trace = reference.trace;
+    let line = error.line;
+    let column = error.column;
+    if (urlInfo.isInline) {
+      line = trace.line + line;
+      line = line - 1;
+      trace = {
+        ...trace,
+        line,
+        column,
+        codeFrame: generateContentFrame({
+          line,
+          column,
+          content: urlInfo.inlineUrlSite.content,
+        }),
+        message: stringifyUrlSite({
+          url: urlInfo.inlineUrlSite.url,
+          line,
+          column,
+          content: urlInfo.inlineUrlSite.content,
+        }),
+      };
+    } else {
+      trace = {
+        url: urlInfo.url,
+        line,
+        column: error.column,
+        codeFrame: generateContentFrame({
+          line,
+          column: error.column,
+          content: urlInfo.content,
+        }),
+        message: stringifyUrlSite({
+          url: urlInfo.url,
+          line,
+          column: error.column,
+          content: urlInfo.content,
+        }),
+      };
+    }
+    const transformError = new Error(
+      createDetailedMessage(
+        `parse error on "${urlInfo.type}"
+${trace.message}
+${error.message}`,
+        {
+          "first reference": `${reference.trace.url}:${reference.trace.line}:${reference.trace.column}`,
+          ...detailsFromFirstReference(reference),
+          ...detailsFromPluginController(pluginController),
+        },
+      ),
+    );
+    transformError.cause = error;
+    transformError.name = "TRANSFORM_URL_CONTENT_ERROR";
+    transformError.code = "PARSE_ERROR";
+    transformError.stack = error.stack;
+    transformError.reason = error.message;
+    transformError.trace = trace;
+    transformError.asResponse = error.asResponse;
+    return transformError;
+  }
   const createFailedToTransformError = ({
     code = error.code || "TRANSFORM_URL_CONTENT_ERROR",
     reason,
     ...details
   }) => {
+    const reference = urlInfo.firstReference;
+    let trace = reference.trace;
     const transformError = new Error(
       createDetailedMessage(
-        `"transformUrlContent" error on "${urlInfo.type}"`,
+        `"transformUrlContent" error on "${urlInfo.type}"
+${trace.message}
+${reason}`,
         {
-          reason,
+          ...detailsFromFirstReference(reference),
           ...details,
-          "url": urlInfo.url,
-          "url reference trace": urlInfo.firstReference.trace.message,
           ...detailsFromPluginController(pluginController),
         },
       ),
     );
+    transformError.cause = error;
     transformError.name = "TRANSFORM_URL_CONTENT_ERROR";
     transformError.code = code;
     transformError.reason = reason;
     transformError.stack = error.stack;
     transformError.url = urlInfo.url;
-    transformError.trace = urlInfo.firstReference.trace;
-    if (code === "PARSE_ERROR") {
-      transformError.reason = `parse error on ${urlInfo.type}`;
-      transformError.cause = error;
-      let line = error.line;
-      if (urlInfo.type === "js_module") {
-        line = line - 1;
-      }
-      if (urlInfo.isInline) {
-        transformError.trace.line = urlInfo.firstReference.trace.line + line;
-        transformError.trace.column =
-          urlInfo.firstReference.trace.column + error.column;
-        transformError.trace.codeFrame = generateContentFrame({
-          line: transformError.trace.line,
-          column: transformError.trace.column,
-          content: urlInfo.inlineUrlSite.content,
-        });
-        transformError.trace.message = stringifyUrlSite({
-          url: urlInfo.inlineUrlSite.url,
-          line: transformError.trace.line,
-          column: transformError.trace.column,
-          content: urlInfo.inlineUrlSite.content,
-        });
-      } else {
-        transformError.trace = {
-          url: urlInfo.url,
-          line,
-          column: error.column,
-          codeFrame: generateContentFrame({
-            line,
-            column: error.column,
-            content: urlInfo.content,
-          }),
-          message: stringifyUrlSite({
-            url: urlInfo.url,
-            line,
-            column: error.column,
-            content: urlInfo.content,
-          }),
-        };
-      }
-    }
+    transformError.trace = trace;
     transformError.asResponse = error.asResponse;
     return transformError;
   };
@@ -197,17 +232,20 @@ export const createTransformUrlContentError = ({
 
 export const createFinalizeUrlContentError = ({
   pluginController,
-
   urlInfo,
   error,
 }) => {
+  const reference = urlInfo.firstReference;
   const finalizeError = new Error(
-    createDetailedMessage(`"finalizeUrlContent" error on "${urlInfo.type}"`, {
-      ...detailsFromValueThrown(error),
-      "url": urlInfo.url,
-      "url reference trace": urlInfo.firstReference.trace.message,
-      ...detailsFromPluginController(pluginController),
-    }),
+    createDetailedMessage(
+      `"finalizeUrlContent" error on "${urlInfo.type}"
+${reference.trace.message}`,
+      {
+        ...detailsFromFirstReference(reference),
+        ...detailsFromValueThrown(error),
+        ...detailsFromPluginController(pluginController),
+      },
+    ),
   );
   if (error && error instanceof Error) {
     finalizeError.cause = error;
@@ -218,6 +256,23 @@ export const createFinalizeUrlContentError = ({
   return finalizeError;
 };
 
+const detailsFromFirstReference = (reference) => {
+  const referenceInProject = getFirstReferenceInProject(reference);
+  if (referenceInProject === reference) {
+    return {};
+  }
+  return {
+    "first reference in project": `${referenceInProject.trace.url}:${referenceInProject.trace.line}:${referenceInProject.trace.column}`,
+  };
+};
+const getFirstReferenceInProject = (reference) => {
+  const ownerUrlInfo = reference.ownerUrlInfo;
+  if (!ownerUrlInfo.url.includes("/node_modules/")) {
+    return reference;
+  }
+  return getFirstReferenceInProject(ownerUrlInfo.firstReference);
+};
+
 const detailsFromPluginController = (pluginController) => {
   const currentPlugin = pluginController.getCurrentPlugin();
   if (!currentPlugin) {
@@ -228,6 +283,18 @@ const detailsFromPluginController = (pluginController) => {
 
 const detailsFromValueThrown = (valueThrownByPlugin) => {
   if (valueThrownByPlugin && valueThrownByPlugin instanceof Error) {
+    if (
+      valueThrownByPlugin.code === "PARSE_ERROR" ||
+      valueThrownByPlugin.code === "MODULE_NOT_FOUND" ||
+      valueThrownByPlugin.name === "RESOLVE_URL_ERROR" ||
+      valueThrownByPlugin.name === "FETCH_URL_CONTENT_ERROR" ||
+      valueThrownByPlugin.name === "TRANSFORM_URL_CONTENT_ERROR" ||
+      valueThrownByPlugin.name === "FINALIZE_URL_CONTENT_ERROR"
+    ) {
+      return {
+        "error message": valueThrownByPlugin.message,
+      };
+    }
     return {
       "error stack": valueThrownByPlugin.stack,
     };

package/src/kitchen/kitchen.js

@@ -487,16 +487,21 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           e.code !== "DIRECTORY_REFERENCE_NOT_ALLOWED" &&
           errorOnInlineContentCanSkipThrow(urlInfo)
         ) {
-          const errorInfo =
-            e.code === "PARSE_ERROR" && e.cause
-              ? `${e.cause.reasonCode}\n${e.trace?.message}`
-              : e.stack;
           // When something like <style> or <script> contains syntax error
           // the HTML in itself it still valid
           // keep the syntax error and continue with the HTML
+          if (e.code === "PARSE_ERROR") {
+            logger.error(`parse error on "${urlInfo.type}"
+${e.trace?.message}
+${e.reason}
+--- declared in ---
+${urlInfo.firstReference.trace.message}`);
+            return;
+          }
           logger.error(
-            `Error while cooking ${urlInfo.type} declared in ${urlInfo.firstReference.trace.message}:
-${errorInfo}`,
+            `Error while cooking ${urlInfo.type}
+${urlInfo.firstReference.trace.message}
+${e.stack}`,
           );
           return;
         }

package/src/kitchen/url_graph/references.js

@@ -64,6 +64,7 @@ export const createDependencies = (ownerUrlInfo) => {
     isOriginalPosition,
     specifierLine,
     specifierColumn,
+    content,
     ...rest
   }) => {
     const parentUrl = isOriginalPosition
@@ -84,6 +85,7 @@ export const createDependencies = (ownerUrlInfo) => {
       specifierLine,
       specifierColumn,
       isInline: true,
+      content,
       ...rest,
     });
     return reference;
@@ -411,6 +413,12 @@ const createReference = ({
     ownerUrlInfo = reference.ownerUrlInfo,
     ...props
   }) => {
+    const content =
+      ownerUrlInfo === undefined
+        ? isOriginalPosition
+          ? reference.ownerUrlInfo.originalContent
+          : reference.ownerUrlInfo.content
+        : ownerUrlInfo.content;
     const trace = traceFromUrlSite({
       url:
         ownerUrlInfo === undefined
@@ -418,12 +426,7 @@ const createReference = ({
             ? reference.ownerUrlInfo.url
             : reference.ownerUrlInfo.generatedUrl
           : reference.ownerUrlInfo.url,
-      content:
-        ownerUrlInfo === undefined
-          ? isOriginalPosition
-            ? reference.ownerUrlInfo.originalContent
-            : reference.ownerUrlInfo.content
-          : ownerUrlInfo.content,
+      content,
       line,
       column,
     });
@@ -620,14 +623,15 @@ const applyDependencyRemovalEffects = (reference) => {
 };
 
 const traceFromUrlSite = (urlSite) => {
+  const codeFrame = urlSite.content
+    ? generateContentFrame({
+        content: urlSite.content,
+        line: urlSite.line,
+        column: urlSite.column,
+      })
+    : "";
   return {
-    codeFrame: urlSite.content
-      ? generateContentFrame({
-          content: urlSite.content,
-          line: urlSite.line,
-          column: urlSite.column,
-        })
-      : "",
+    codeFrame,
     message: stringifyUrlSite(urlSite),
     url: urlSite.url,
     line: urlSite.line,
@@ -637,7 +641,7 @@ const traceFromUrlSite = (urlSite) => {
 
 const adjustUrlSite = (urlInfo, { url, line, column }) => {
   const isOriginal = url === urlInfo.url;
-  const adjust = (urlSite, urlInfo) => {
+  const adjust = (urlInfo, urlSite) => {
     if (!urlSite.isOriginal) {
       return urlSite;
     }
@@ -646,33 +650,35 @@ const adjustUrlSite = (urlInfo, { url, line, column }) => {
       return urlSite;
     }
     const parentUrlInfo = urlInfo.graph.getUrlInfo(inlineUrlSite.url);
-    return adjust(
-      {
-        isOriginal: true,
-        url: inlineUrlSite.url,
-        content: inlineUrlSite.content,
-        line:
-          inlineUrlSite.line === undefined
-            ? urlSite.line
-            : inlineUrlSite.line + urlSite.line,
-        column:
-          inlineUrlSite.column === undefined
-            ? urlSite.column
-            : inlineUrlSite.column + urlSite.column,
-      },
-      parentUrlInfo,
-    );
-  };
-  return adjust(
-    {
-      isOriginal,
-      url,
-      content: isOriginal ? urlInfo.originalContent : urlInfo.content,
+    line =
+      inlineUrlSite.line === undefined
+        ? urlSite.line
+        : inlineUrlSite.line + urlSite.line;
+    // we remove 1 to the line because imagine the following html:
+    // <style>body { color: red; }</style>
+    // -> content starts same line as <style> (same for <script>)
+    if (urlInfo.content[0] === "\n") {
+      line = line - 1;
+    }
+    column =
+      inlineUrlSite.column === undefined
+        ? urlSite.column
+        : inlineUrlSite.column + urlSite.column;
+    return adjust(parentUrlInfo, {
+      isOriginal: true,
+      url: inlineUrlSite.url,
+      content: inlineUrlSite.content,
       line,
       column,
-    },
-    urlInfo,
-  );
+    });
+  };
+  return adjust(urlInfo, {
+    isOriginal,
+    url,
+    content: isOriginal ? urlInfo.originalContent : urlInfo.content,
+    line,
+    column,
+  });
 };
 
 const getRedirectedReferenceProps = (reference, url) => {

package/src/plugins/inlining/jsenv_plugin_inlining_into_html.js

@@ -47,7 +47,7 @@ export const jsenvPluginInliningIntoHtml = () => {
             url: linkReference.url,
           });
           const linkReferenceInlined = linkReference.inline({
-            line: line - 1,
+            line,
             column,
             isOriginal,
             specifier: linkInlineUrl,
@@ -99,7 +99,7 @@ export const jsenvPluginInliningIntoHtml = () => {
             url: scriptReference.url,
           });
           const scriptReferenceInlined = scriptReference.inline({
-            line: line - 1,
+            line,
             column,
             isOriginal,
             specifier: scriptInlineUrl,

package/src/plugins/reference_analysis/html/jsenv_plugin_html_reference_analysis.js

@@ -264,10 +264,7 @@ export const jsenvPluginHtmlReferenceAnalysis = ({
               type,
               expectedType,
               isOriginalPosition: isOriginal,
-              // we remove 1 to the line because imagine the following html:
-              // <style>body { color: red; }</style>
-              // -> content starts same line as <style> (same for <script>)
-              specifierLine: line - 1,
+              specifierLine: line,
               specifierColumn: column,
               specifier: inlineContentUrl,
               contentType,
@@ -386,7 +383,7 @@ export const jsenvPluginHtmlReferenceAnalysis = ({
                     },
                   );
                   const importmapReferenceInlined = importmapReference.inline({
-                    line: line - 1,
+                    line,
                     column,
                     isOriginal,
                     specifier: importmapInlineUrl,