@jsenv/core 25.1.1 → 25.3.0

package/src/internal/response_validation.js

@@ -10,7 +10,6 @@ export const validateResponse = async (
   } = {},
 ) => {
   const validity = { isValid: true }
-
   if (statusValidation) {
     const statusValidity = await checkStatus(response, {
       originalUrl,
@@ -21,7 +20,6 @@ export const validateResponse = async (
       return validity
     }
   }
-
   if (contentTypeExpected) {
     const contentTypeValidity = await checkContentType(response, {
       originalUrl,
@@ -33,7 +31,6 @@ export const validateResponse = async (
       return validity
     }
   }
-
   return validity
 }
 
@@ -47,36 +44,28 @@ const mergeValidity = (parentValidity, childValidityName, childValidity) => {
 const checkStatus = async (response, { originalUrl, urlTrace }) => {
   const url = originalUrl || response.url
   const { status } = response
-
-  if (status === 500) {
-    if (response.headers["content-type"] === "application/json") {
-      return {
-        isValid: false,
-        message: `error 500 on url`,
-        details: {
-          "response status": status,
-          "response json": JSON.stringify(await response.json(), null, "  "),
-          url,
-          ...formatUrlTrace(urlTrace),
-        },
-      }
-    }
-  }
-
   if (status < 200 || status > 299) {
-    const responseText = await response.text()
     return {
       isValid: false,
       message: `invalid response status on url`,
       details: {
         "response status": status,
-        ...(responseText ? { "response text": responseText } : {}),
         url,
         ...formatUrlTrace(urlTrace),
+        ...(response.headers["content-type"] === "application/json"
+          ? {
+              "response text": JSON.stringify(
+                await response.json(),
+                null,
+                "  ",
+              ),
+            }
+          : {
+              "response text": await response.text(),
+            }),
       },
     }
   }
-
   return { isValid: true }
 }
 
@@ -86,11 +75,9 @@ const checkContentType = async (
 ) => {
   const url = originalUrl || response.url
   const responseContentType = response.headers["content-type"] || ""
-
   const isOk = Array.isArray(contentTypeExpected)
     ? contentTypeExpected.includes(responseContentType)
     : responseContentType === contentTypeExpected
-
   if (!isOk) {
     return {
       isValid: false,

package/src/internal/sourceMappingURLUtils.js

@@ -1,3 +1,13 @@
+export const generateSourcemapUrl = (url) => {
+  // we want to remove eventual search params from url
+  const urlString = String(url)
+  const urlObject = new URL(url)
+  const origin = urlString.startsWith("file://") ? "file://" : urlObject.origin
+  const pathname = urlObject.pathname
+  const sourcemapUrl = `${origin}${pathname}.map`
+  return sourcemapUrl
+}
+
 export const getJavaScriptSourceMappingUrl = (javaScriptSource) => {
   let sourceMappingUrl
   replaceSourceMappingUrl(

package/src/internal/url_conversion.js

@@ -124,6 +124,7 @@ export const createUrlConverter = ({
   }
 
   return {
+    compileServerOriginForRollup,
     asRollupUrl,
     asProjectUrl,
     asServerUrl,