@jsenv/core 30.4.1 → 31.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "30.4.1",
+  "version": "31.0.0",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -67,47 +67,48 @@
     "@c88/v8-coverage": "0.1.1",
     "@financial-times/polyfill-useragent-normaliser": "1.10.2",
     "@jsenv/abort": "4.2.4",
-    "@jsenv/ast": "3.0.2",
-    "@jsenv/babel-plugins": "1.1.4",
-    "@jsenv/plugin-bundling": "1.2.1",
-    "@jsenv/filesystem": "4.1.9",
+    "@jsenv/ast": "3.0.3",
+    "@jsenv/babel-plugins": "1.1.5",
+    "@jsenv/filesystem": "4.2.0",
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.1",
-    "@jsenv/log": "3.3.3",
+    "@jsenv/log": "3.3.4",
     "@jsenv/node-esm-resolution": "1.0.1",
-    "@jsenv/server": "14.1.15",
-    "@jsenv/sourcemap": "1.0.8",
+    "@jsenv/plugin-bundling": "2.0.0",
+    "@jsenv/server": "15.0.0",
+    "@jsenv/sourcemap": "1.0.9",
     "@jsenv/uneval": "1.6.0",
-    "@jsenv/url-meta": "7.0.2",
+    "@jsenv/url-meta": "7.1.0",
     "@jsenv/urls": "1.2.8",
     "@jsenv/utils": "2.0.1",
-    "@paralleldrive/cuid2": "2.0.1",
+    "@paralleldrive/cuid2": "2.2.0",
     "construct-style-sheets-polyfill": "3.1.0",
     "istanbul-lib-coverage": "3.2.0",
     "istanbul-lib-instrument": "5.2.1",
     "istanbul-lib-report": "3.0.0",
     "istanbul-reports": "3.1.5",
     "launch-editor": "2.6.0",
-    "lightningcss": "1.18.0",
+    "lightningcss": "1.19.0",
     "pidtree": "0.6.0",
     "string-width": "5.1.2",
     "strip-ansi": "7.0.1",
-    "v8-to-istanbul": "9.0.1",
+    "v8-to-istanbul": "9.1.0",
+    "webidl2": "24.2.2",
     "wrap-ansi": "8.1.0"
   },
   "devDependencies": {
-    "@babel/eslint-parser": "7.19.1",
+    "@babel/eslint-parser": "7.21.3",
     "@babel/plugin-syntax-import-assertions": "7.20.0",
     "@jsenv/assert": "./packages/assert/",
     "@jsenv/eslint-config": "./packages/eslint-config/",
     "@jsenv/file-size-impact": "14.0.0",
     "@jsenv/https-local": "3.0.6",
     "@jsenv/package-workspace": "0.5.1",
-    "@jsenv/plugin-minification": "./packages/jsenv-plugin-minification/",
+    "@jsenv/performance-impact": "4.1.0",
     "@jsenv/plugin-globals": "./packages/jsenv-plugin-globals/",
+    "@jsenv/plugin-minification": "./packages/jsenv-plugin-minification/",
     "@jsenv/plugin-placeholders": "./packages/jsenv-plugin-placeholders/",
-    "@jsenv/performance-impact": "4.1.0",
-    "eslint": "8.35.0",
+    "eslint": "8.36.0",
     "eslint-plugin-html": "7.1.0",
     "eslint-plugin-import": "2.27.5",
     "eslint-plugin-react": "7.32.2",

package/src/build/build.js

@@ -28,7 +28,7 @@ import {
   urlToRelativeUrl,
 } from "@jsenv/urls"
 import {
-  assertAndNormalizeDirectoryUrl,
+  validateDirectoryUrl,
   ensureEmptyDirectory,
   writeFileSync,
   registerDirectoryLifecycle,
@@ -155,7 +155,32 @@ export const build = async ({
   writeGeneratedFiles = false,
   assetManifest = versioningMethod === "filename",
   assetManifestFileRelativeUrl = "asset-manifest.json",
+  ...rest
 }) => {
+  // param validation
+  {
+    const unexpectedParamNames = Object.keys(rest)
+    if (unexpectedParamNames.length > 0) {
+      throw new TypeError(
+        `${unexpectedParamNames.join(",")}: there is no such param`,
+      )
+    }
+    const rootDirectoryUrlValidation = validateDirectoryUrl(rootDirectoryUrl)
+    if (!rootDirectoryUrlValidation.valid) {
+      throw new TypeError(
+        `rootDirectoryUrl ${rootDirectoryUrlValidation.message}, got ${rootDirectoryUrl}`,
+      )
+    }
+    rootDirectoryUrl = rootDirectoryUrlValidation.value
+    const buildDirectoryUrlValidation = validateDirectoryUrl(buildDirectoryUrl)
+    if (!buildDirectoryUrlValidation.valid) {
+      throw new TypeError(
+        `buildDirectoryUrl ${buildDirectoryUrlValidation.message}, got ${buildDirectoryUrlValidation}`,
+      )
+    }
+    buildDirectoryUrl = buildDirectoryUrlValidation.value
+  }
+
   const operation = Abort.startOperation()
   operation.addAbortSignal(signal)
   if (handleSIGINT) {
@@ -169,8 +194,6 @@ export const build = async ({
     })
   }
 
-  rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
-  buildDirectoryUrl = assertAndNormalizeDirectoryUrl(buildDirectoryUrl)
   assertEntryPoints({ entryPoints })
   if (!["filename", "search_param"].includes(versioningMethod)) {
     throw new Error(

package/src/build/start_build_server.js

@@ -44,10 +44,8 @@ export const startBuildServer = async ({
   handleSIGINT = true,
   logLevel,
   serverLogLevel = "warn",
-  protocol = "http",
+  https,
   http2,
-  certificate,
-  privateKey,
   acceptAnyIp,
   hostname,
   port = 9779,
@@ -169,10 +167,8 @@ export const startBuildServer = async ({
     logLevel: serverLogLevel,
     startLog: false,
 
-    protocol,
+    https,
     http2,
-    certificate,
-    privateKey,
     acceptAnyIp,
     hostname,
     port,

package/src/dev/start_dev_server.js

@@ -1,6 +1,6 @@
 import { parentPort } from "node:worker_threads"
 import {
-  assertAndNormalizeDirectoryUrl,
+  validateDirectoryUrl,
   registerDirectoryLifecycle,
 } from "@jsenv/filesystem"
 import { Abort, raceProcessTeardownEvents } from "@jsenv/abort"
@@ -32,12 +32,10 @@ export const startDevServer = async ({
   handleSIGINT = true,
   logLevel = "info",
   serverLogLevel = "warn",
-  protocol = "http",
+  https,
   // it's better to use http1 by default because it allows to get statusText in devtools
   // which gives valuable information when there is errors
   http2 = false,
-  certificate,
-  privateKey,
   hostname,
   port = 3456,
   acceptAnyIp,
@@ -82,9 +80,26 @@ export const startDevServer = async ({
   // no real need to write files during github workflow
   // and mitigates https://github.com/actions/runner-images/issues/3885
   writeGeneratedFiles = !process.env.CI,
+  ...rest
 }) => {
+  // params type checking
+  {
+    const unexpectedParamNames = Object.keys(rest)
+    if (unexpectedParamNames.length > 0) {
+      throw new TypeError(
+        `${unexpectedParamNames.join(",")}: there is no such param`,
+      )
+    }
+    const rootDirectoryUrlValidation = validateDirectoryUrl(rootDirectoryUrl)
+    if (!rootDirectoryUrlValidation.valid) {
+      throw new TypeError(
+        `rootDirectoryUrl ${rootDirectoryUrlValidation.message}, got ${rootDirectoryUrl}`,
+      )
+    }
+    rootDirectoryUrl = rootDirectoryUrlValidation.value
+  }
+
   const logger = createLogger({ logLevel })
-  rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
   const operation = Abort.startOperation()
   operation.addAbortSignal(signal)
   if (handleSIGINT) {
@@ -167,10 +182,8 @@ export const startDevServer = async ({
     logLevel: serverLogLevel,
     startLog: false,
 
-    protocol,
+    https,
     http2,
-    certificate,
-    privateKey,
     acceptAnyIp,
     hostname,
     port,
@@ -265,13 +278,13 @@ export const startDevServer = async ({
         sendErrorDetails: true,
       }),
     ],
-    onStop: (reason) => {
-      onStop()
-      serverStopCallbacks.forEach((serverStopCallback) => {
-        serverStopCallback(reason)
-      })
-      serverStopCallbacks.length = 0
-    },
+  })
+  server.stoppedPromise.then((reason) => {
+    onStop()
+    serverStopCallbacks.forEach((serverStopCallback) => {
+      serverStopCallback(reason)
+    })
+    serverStopCallbacks.length = 0
   })
   startDevServerTask.done()
   if (hostname) {

package/src/execute/execute.js

@@ -90,7 +90,7 @@ export const execute = async ({
   result = resultTransformer(result)
 
   try {
-    if (result.status === "errored") {
+    if (result.status === "failed") {
       if (ignoreError) {
         return result
       }

package/src/execute/run.js

@@ -119,7 +119,7 @@ export const run = async ({
               cb(runResult)
             } catch (e) {
               cb({
-                status: "errored",
+                status: "failed",
                 errors: [e],
               })
             }
@@ -147,7 +147,7 @@ export const run = async ({
         result.status = "aborted"
       }
     } else {
-      result.status = "errored"
+      result.status = "failed"
       result.errors.push(e)
     }
   } finally {

package/src/execute/runtimes/browsers/from_playwright.js

@@ -331,12 +331,12 @@ export const createRuntimeFromPlaywright = ({
       }
       if (winner.name === "error") {
         let error = winner.data
-        result.status = "errored"
+        result.status = "failed"
         result.errors.push(error)
         return
       }
       if (winner.name === "closed") {
-        result.status = "errored"
+        result.status = "failed"
         result.errors.push(
           isBrowserDedicatedToExecution
             ? new Error(`browser disconnected during execution`)
@@ -350,8 +350,8 @@ export const createRuntimeFromPlaywright = ({
       result.namespace = executionResults
       Object.keys(executionResults).forEach((key) => {
         const executionResult = executionResults[key]
-        if (executionResult.status === "errored") {
-          result.status = "errored"
+        if (executionResult.status === "failed") {
+          result.status = "failed"
           result.errors.push({
             ...executionResult.exception,
             stack: executionResult.exception.text,
@@ -370,7 +370,7 @@ export const createRuntimeFromPlaywright = ({
         await callback()
       }, Promise.resolve())
     } catch (e) {
-      result.status = "errored"
+      result.status = "failed"
       result.errors = [e]
     }
     if (keepRunning) {

package/src/execute/runtimes/node/node_child_process.js

@@ -219,7 +219,7 @@ nodeChildProcess.run = async ({
     if (winner.name === "error") {
       const error = winner.data
       removeOutputListener()
-      result.status = "errored"
+      result.status = "failed"
       result.errors.push(error)
       return
     }
@@ -227,7 +227,7 @@ nodeChildProcess.run = async ({
       const { code } = winner.data
       await cleanup("process exit")
       if (code === 12) {
-        result.status = "errored"
+        result.status = "failed"
         result.errors.push(
           new Error(
             `node process exited with 12 (the forked child process wanted to use a non-available port for debug)`,
@@ -242,13 +242,13 @@ nodeChildProcess.run = async ({
         code === EXIT_CODES.SIGTERM ||
         code === EXIT_CODES.SIGABORT
       ) {
-        result.status = "errored"
+        result.status = "failed"
         result.errors.push(new Error(`node process exited during execution`))
         return
       }
       // process.exit(1) in child process or process.exitCode = 1 + process.exit()
       // means there was an error even if we don't know exactly what.
-      result.status = "errored"
+      result.status = "failed"
       result.errors.push(
         new Error(`node process exited with code ${code} during execution`),
       )
@@ -256,7 +256,7 @@ nodeChildProcess.run = async ({
     }
     const { status, value } = winner.data
     if (status === "action-failed") {
-      result.status = "errored"
+      result.status = "failed"
       result.errors.push(value)
       return
     }
@@ -270,7 +270,7 @@ nodeChildProcess.run = async ({
   try {
     await writeResult()
   } catch (e) {
-    result.status = "errored"
+    result.status = "failed"
     result.errors.push(e)
   }
   if (keepRunning) {

package/src/execute/runtimes/node/node_worker_thread.js

@@ -171,7 +171,7 @@ nodeWorkerThread.run = async ({
     if (winner.name === "error") {
       const error = winner.data
       removeOutputListener()
-      result.status = "errored"
+      result.status = "failed"
       result.errors.push(error)
       return
     }
@@ -179,7 +179,7 @@ nodeWorkerThread.run = async ({
       const { code } = winner.data
       await cleanup("process exit")
       if (code === 12) {
-        result.status = "errored"
+        result.status = "failed"
         result.errors.push(
           new Error(
             `node process exited with 12 (the forked child process wanted to use a non-available port for debug)`,
@@ -194,7 +194,7 @@ nodeWorkerThread.run = async ({
         code === EXIT_CODES.SIGTERM ||
         code === EXIT_CODES.SIGABORT
       ) {
-        result.status = "errored"
+        result.status = "failed"
         result.errors.push(
           new Error(`node worker thread exited during execution`),
         )
@@ -202,7 +202,7 @@ nodeWorkerThread.run = async ({
       }
       // process.exit(1) in child process or process.exitCode = 1 + process.exit()
       // means there was an error even if we don't know exactly what.
-      result.status = "errored"
+      result.status = "failed"
       result.errors.push(
         new Error(
           `node worker thread exited with code ${code} during execution`,
@@ -211,7 +211,7 @@ nodeWorkerThread.run = async ({
     }
     const { status, value } = winner.data
     if (status === "action-failed") {
-      result.status = "errored"
+      result.status = "failed"
       result.errors.push(value)
       return
     }
@@ -225,7 +225,7 @@ nodeWorkerThread.run = async ({
   try {
     await writeResult()
   } catch (e) {
-    result.status = "errored"
+    result.status = "failed"
     result.errors.push(e)
   }
 

package/src/plugins/supervisor/client/supervisor.js

@@ -814,7 +814,7 @@ window.__supervisor__ = (() => {
       }
 
       const onError = (e) => {
-        executionResult.status = "errored"
+        executionResult.status = "failed"
         const exception = supervisor.createException({ reason: e })
         if (exception.needsReport) {
           supervisor.reportException(exception)

package/src/plugins/transpilation/as_js_classic/jsenv_plugin_as_js_classic_library.js

@@ -51,10 +51,7 @@ export const jsenvPluginAsJsClassicLibrary = ({
           ...context,
           buildDirectoryUrl: context.outDirectoryUrl,
         },
-        options: {
-          babelHelpersChunk: false,
-          preserveDynamicImport: true,
-        },
+        preserveDynamicImport: true,
       })
       const jsModuleBundledUrlInfo = bundleUrlInfos[jsModuleUrlInfo.url]
       if (context.dev) {

package/src/test/execute_plan.js

@@ -225,7 +225,7 @@ export const executePlan = async (
       total: executionSteps.length,
       aborted: 0,
       timedout: 0,
-      errored: 0,
+      failed: 0,
       completed: 0,
       done: 0,
     }
@@ -306,7 +306,7 @@ export const executePlan = async (
           })
         } else {
           executionResult = {
-            status: "errored",
+            status: "failed",
             errors: [
               new Error(
                 `No file at ${fileRelativeUrl} for execution "${executionName}"`,
@@ -336,8 +336,8 @@ export const executePlan = async (
           counters.aborted++
         } else if (executionResult.status === "timedout") {
           counters.timedout++
-        } else if (executionResult.status === "errored") {
-          counters.errored++
+        } else if (executionResult.status === "failed") {
+          counters.failed++
         } else if (executionResult.status === "completed") {
           counters.completed++
         }

package/src/test/execute_test_plan.js

@@ -5,10 +5,7 @@ import {
   urlIsInsideOf,
   urlToRelativeUrl,
 } from "@jsenv/urls"
-import {
-  ensureEmptyDirectory,
-  assertAndNormalizeDirectoryUrl,
-} from "@jsenv/filesystem"
+import { ensureEmptyDirectory, validateDirectoryUrl } from "@jsenv/filesystem"
 import { createLogger, createDetailedMessage } from "@jsenv/log"
 
 import { generateCoverageJsonFile } from "./coverage/coverage_reporter_json_file.js"
@@ -82,51 +79,72 @@ export const executeTestPlan = async ({
   coverageReportTextLog = true,
   coverageReportJsonFile = process.env.CI ? null : "./.coverage/coverage.json",
   coverageReportHtmlDirectory = process.env.CI ? "./.coverage/" : null,
+  ...rest
 }) => {
-  const logger = createLogger({ logLevel })
-  rootDirectoryUrl = assertAndNormalizeDirectoryUrl(rootDirectoryUrl)
-  if (typeof testPlan !== "object") {
-    throw new Error(`testPlan must be an object, got ${testPlan}`)
-  }
-  if (coverageEnabled) {
-    if (typeof coverageConfig !== "object") {
+  // param validation
+  {
+    const unexpectedParamNames = Object.keys(rest)
+    if (unexpectedParamNames.length > 0) {
       throw new TypeError(
-        `coverageConfig must be an object, got ${coverageConfig}`,
+        `${unexpectedParamNames.join(",")}: there is no such param`,
       )
     }
-    if (Object.keys(coverageConfig).length === 0) {
-      logger.warn(
-        `coverageConfig is an empty object. Nothing will be instrumented for coverage so your coverage will be empty`,
+    const rootDirectoryUrlValidation = validateDirectoryUrl(rootDirectoryUrl)
+    if (!rootDirectoryUrlValidation.valid) {
+      throw new TypeError(
+        `rootDirectoryUrl ${rootDirectoryUrlValidation.message}, got ${rootDirectoryUrl}`,
       )
     }
-    if (!coverageAndExecutionAllowed) {
-      const associationsForExecute = URL_META.resolveAssociations(
-        { execute: testPlan },
-        "file:///",
-      )
-      const associationsForCover = URL_META.resolveAssociations(
-        { cover: coverageConfig },
-        "file:///",
-      )
-      const patternsMatchingCoverAndExecute = Object.keys(
-        associationsForExecute.execute,
-      ).filter((testPlanPattern) => {
-        const { cover } = URL_META.applyAssociations({
-          url: testPlanPattern,
-          associations: associationsForCover,
-        })
-        return cover
-      })
-      if (patternsMatchingCoverAndExecute.length) {
-        // It would be strange, for a given file to be both covered and executed
-        throw new Error(
-          createDetailedMessage(`some file will be both covered and executed`, {
-            patterns: patternsMatchingCoverAndExecute,
-          }),
+    rootDirectoryUrl = rootDirectoryUrlValidation.value
+    if (typeof testPlan !== "object") {
+      throw new Error(`testPlan must be an object, got ${testPlan}`)
+    }
+    if (coverageEnabled) {
+      if (typeof coverageConfig !== "object") {
+        throw new TypeError(
+          `coverageConfig must be an object, got ${coverageConfig}`,
         )
       }
+      if (!coverageAndExecutionAllowed) {
+        const associationsForExecute = URL_META.resolveAssociations(
+          { execute: testPlan },
+          "file:///",
+        )
+        const associationsForCover = URL_META.resolveAssociations(
+          { cover: coverageConfig },
+          "file:///",
+        )
+        const patternsMatchingCoverAndExecute = Object.keys(
+          associationsForExecute.execute,
+        ).filter((testPlanPattern) => {
+          const { cover } = URL_META.applyAssociations({
+            url: testPlanPattern,
+            associations: associationsForCover,
+          })
+          return cover
+        })
+        if (patternsMatchingCoverAndExecute.length) {
+          // It would be strange, for a given file to be both covered and executed
+          throw new Error(
+            createDetailedMessage(
+              `some file will be both covered and executed`,
+              {
+                patterns: patternsMatchingCoverAndExecute,
+              },
+            ),
+          )
+        }
+      }
     }
   }
+
+  const logger = createLogger({ logLevel })
+  if (Object.keys(coverageConfig).length === 0) {
+    logger.warn(
+      `coverageConfig is an empty object. Nothing will be instrumented for coverage so your coverage will be empty`,
+    )
+  }
+
   const result = await executePlan(testPlan, {
     signal,
     handleSIGINT,

package/src/test/execution_colors.js

@@ -4,7 +4,7 @@ export const EXECUTION_COLORS = {
   executing: ANSI.BLUE,
   aborted: ANSI.MAGENTA,
   timedout: ANSI.MAGENTA,
-  errored: ANSI.RED,
+  failed: ANSI.RED,
   completed: ANSI.GREEN,
   cancelled: ANSI.GREY,
 }

package/src/test/logs_file_execution.js

@@ -147,8 +147,8 @@ const createStatusSummary = ({ counters }) => {
   if (counters.timedout === counters.total) {
     return `all ${ANSI.color(`timed out`, EXECUTION_COLORS.timedout)}`
   }
-  if (counters.errored === counters.total) {
-    return `all ${ANSI.color(`errored`, EXECUTION_COLORS.errored)}`
+  if (counters.failed === counters.total) {
+    return `all ${ANSI.color(`failed`, EXECUTION_COLORS.failed)}`
   }
   if (counters.completed === counters.total) {
     return `all ${ANSI.color(`completed`, EXECUTION_COLORS.completed)}`
@@ -171,9 +171,9 @@ const createMixedDetails = ({ counters }) => {
       )}`,
     )
   }
-  if (counters.errored) {
+  if (counters.failed) {
     parts.push(
-      `${counters.errored} ${ANSI.color(`errored`, EXECUTION_COLORS.errored)}`,
+      `${counters.failed} ${ANSI.color(`failed`, EXECUTION_COLORS.failed)}`,
     )
   }
   if (counters.completed) {
@@ -221,10 +221,10 @@ const descriptionFormatters = {
       EXECUTION_COLORS.timedout,
     )
   },
-  errored: ({ index, total }) => {
+  efailedrrored: ({ index, total }) => {
     return ANSI.color(
-      `${UNICODE.FAILURE_RAW} execution ${index + 1} of ${total} errored`,
-      EXECUTION_COLORS.errored,
+      `${UNICODE.FAILURE_RAW} execution ${index + 1} of ${total} failed`,
+      EXECUTION_COLORS.failed,
     )
   },
   completed: ({ index, total }) => {