@jsenv/core 23.4.1 → 23.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "23.4.1",
+  "version": "23.4.2",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "repository": {

package/src/internal/compiling/createCompiledFileService.js

@@ -71,7 +71,10 @@ export const createCompiledFileService = ({
 
   return (request) => {
     const { origin, ressource } = request
-    const requestUrl = `${origin}${ressource}`
+    // we use "ressourceToPathname" to remove eventual query param from the url
+    // Without this a pattern like "**/*.js" would not match "file.js?t=1"
+    // This would result in file not being compiled when they should
+    const requestUrl = `${origin}${ressourceToPathname(ressource)}`
 
     const requestCompileInfo = serverUrlToCompileInfo(requestUrl, {
       outDirectoryRelativeUrl,
@@ -276,3 +279,12 @@ const babelPluginMapFromCompileId = (
 
   return babelPluginMapForGroup
 }
+
+const ressourceToPathname = (ressource) => {
+  const searchSeparatorIndex = ressource.indexOf("?")
+  const pathname =
+    searchSeparatorIndex === -1
+      ? ressource
+      : ressource.slice(0, searchSeparatorIndex)
+  return pathname
+}

package/src/internal/executing/coverage_utils/v8_coverage_from_directory.js

@@ -29,6 +29,7 @@ export const visitNodeV8Directory = async ({
     const tryReadJsonFile = async () => {
       const fileContent = await readFile(dirEntryUrl, { as: "string" })
       if (fileContent === "") {
+        console.warn(`Coverage JSON file is empty at ${dirEntryUrl}`)
         return null
       }