@jsenv/core 27.3.2 → 27.3.3

package/src/plugins/bundling/js_module/bundle_js_module.js

@@ -125,7 +125,6 @@ const rollupPluginJsenv = ({
               usesImport:
                 rollupFileInfo.imports.length > 0 ||
                 rollupFileInfo.dynamicImports.length > 0,
-              usesExport: rollupFileInfo.exports.length > 0,
             },
             contentType: "text/javascript",
             content: rollupFileInfo.code,

package/src/plugins/node_esm_resolution/jsenv_plugin_node_esm_resolution.js

@@ -56,8 +56,7 @@ export const jsenvPluginNodeEsmResolution = ({
   const onFileChange = () => {
     packageScopesCache.clear()
     packageJsonsCache.clear()
-    Object.keys(urlGraph.urlInfos).forEach((url) => {
-      const urlInfo = urlGraph.getUrlInfo(url)
+    urlGraph.urlInfoMap.forEach((urlInfo) => {
       if (urlInfo.dependsOnPackageJson) {
         urlGraph.considerModified(urlInfo)
       }

package/src/plugins/url_analysis/js/js_urls.js

@@ -13,22 +13,13 @@ export const parseAndTransformJsUrls = async (urlInfo, context) => {
   const { rootDirectoryUrl, referenceUtils } = context
   const actions = []
   const magicSource = createMagicSource(urlInfo.content)
-  urlInfo.data.usesImport = false
-  urlInfo.data.usesExport = false
-  urlInfo.data.usesImportAssertion = false
   jsMentions.forEach((jsMention) => {
-    if (jsMention.assert) {
-      urlInfo.data.usesImportAssertion = true
-    }
     if (
       jsMention.subtype === "import_static" ||
       jsMention.subtype === "import_dynamic"
     ) {
       urlInfo.data.usesImport = true
     }
-    if (jsMention.subtype === "export") {
-      urlInfo.data.usesExport = true
-    }
     const [reference] = referenceUtils.found({
       type: jsMention.type,
       subtype: jsMention.subtype,

package/src/test/coverage/report_to_coverage.js

@@ -50,7 +50,7 @@ export const reportToCoverage = async (
         coverageMethodForNodeJs !== "NODE_V8_COVERAGE"
       ) {
         logger.warn(
-          `No "coverageFileUrl" from execution named "${executionName}" of ${file}`,
+          `"${executionName}" execution of ${file} did not properly write coverage into ${executionResult.coverageFileUrl}`,
         )
       }
     },
@@ -152,18 +152,23 @@ const getCoverageFromReport = async ({ signal, report, onMissing }) => {
           const executionResultForFileOnRuntime =
             executionResultForFile[executionName]
           const { coverageFileUrl } = executionResultForFileOnRuntime
-          if (!coverageFileUrl) {
-            onMissing({
-              executionName,
-              file,
-              executionResult: executionResultForFileOnRuntime,
-            })
-            return
+          let executionCoverage
+          try {
+            executionCoverage = JSON.parse(
+              String(readFileSync(new URL(coverageFileUrl))),
+            )
+          } catch (e) {
+            if (e.code === "ENOENT" || e.name === "SyntaxError") {
+              onMissing({
+                executionName,
+                file,
+                executionResult: executionResultForFileOnRuntime,
+              })
+              return
+            }
+            throw e
           }
 
-          const executionCoverage = JSON.parse(
-            String(readFileSync(new URL(coverageFileUrl))),
-          )
           if (isV8Coverage(executionCoverage)) {
             v8Coverage = v8Coverage
               ? composeTwoV8Coverages(v8Coverage, executionCoverage)

package/src/test/execute_plan.js

@@ -143,7 +143,7 @@ export const executePlan = async (
           createDetailedMessage(
             `process.env.NODE_V8_COVERAGE is required to generate coverage for Node.js subprocesses`,
             {
-              "suggestion": `Preprend NODE_V8_COVERAGE=.coverage/node to the command executing this process`,
+              "suggestion": `set process.env.NODE_V8_COVERAGE`,
               "suggestion 2": `use coverageMethodForNodeJs: "Profiler". But it means coverage for child_process and worker_thread cannot be collected`,
             },
           ),
@@ -224,7 +224,8 @@ export const executePlan = async (
               getCustomBabelPlugins: ({ clientRuntimeCompat }) => {
                 if (
                   coverageEnabled &&
-                  Object.keys(clientRuntimeCompat)[0] !== "chrome"
+                  (coverageMethodForBrowsers !== "playwright_api" ||
+                    Object.keys(clientRuntimeCompat)[0] !== "chrome")
                 ) {
                   return {
                     "transform-instrument": [

package/src/test/execute_test_plan.js

@@ -67,7 +67,9 @@ export const executeTestPlan = async ({
   },
   coverageIncludeMissing = true,
   coverageAndExecutionAllowed = false,
-  coverageMethodForNodeJs = "NODE_V8_COVERAGE", // "Profiler" also accepted
+  coverageMethodForNodeJs = process.env.NODE_V8_COVERAGE
+    ? "NODE_V8_COVERAGE"
+    : "Profiler",
   coverageMethodForBrowsers = "playwright_api", // "istanbul" also accepted
   coverageV8ConflictWarning = true,
   coverageTempDirectoryRelativeUrl = "./.coverage/tmp/",