@jsenv/core 28.0.2 → 28.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "28.0.2",
+  "version": "28.1.2",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -66,14 +66,14 @@
     "@c88/v8-coverage": "0.1.1",
     "@financial-times/polyfill-useragent-normaliser": "2.0.1",
     "@jsenv/abort": "4.2.3",
-    "@jsenv/ast": "1.1.3",
+    "@jsenv/ast": "1.2.0",
     "@jsenv/babel-plugins": "1.0.6",
-    "@jsenv/filesystem": "4.1.1",
+    "@jsenv/filesystem": "4.1.2",
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.1",
-    "@jsenv/log": "3.1.0",
+    "@jsenv/log": "3.2.0",
     "@jsenv/node-esm-resolution": "0.1.0",
-    "@jsenv/server": "14.1.0",
+    "@jsenv/server": "14.1.1",
     "@jsenv/sourcemap": "1.0.4",
     "@jsenv/uneval": "1.6.0",
     "@jsenv/url-meta": "7.0.0",
@@ -87,9 +87,9 @@
     "istanbul-lib-instrument": "5.2.0",
     "istanbul-lib-report": "3.0.0",
     "istanbul-reports": "3.1.5",
-    "launch-editor": "2.4.0",
+    "launch-editor": "2.5.0",
     "pidtree": "0.6.0",
-    "rollup": "2.77.0",
+    "rollup": "2.77.2",
     "string-width": "5.1.2",
     "strip-ansi": "7.0.1",
     "terser": "5.14.2",
@@ -97,19 +97,19 @@
     "wrap-ansi": "8.0.1"
   },
   "devDependencies": {
-    "@babel/eslint-parser": "7.18.2",
+    "@babel/eslint-parser": "7.18.9",
     "@babel/plugin-syntax-import-assertions": "7.18.6",
-    "@jsenv/assert": "2.6.0",
+    "@jsenv/assert": "2.7.0",
     "@jsenv/eslint-config": "16.2.1",
     "@jsenv/file-size-impact": "13.0.1",
-    "@jsenv/https-local": "2.1.0",
+    "@jsenv/https-local": "3.0.1",
     "@jsenv/package-workspace": "0.4.1",
     "@jsenv/performance-impact": "3.0.1",
-    "eslint": "8.20.0",
-    "eslint-plugin-html": "7.0.0",
+    "eslint": "8.21.0",
+    "eslint-plugin-html": "7.1.0",
     "eslint-plugin-import": "2.26.0",
     "eslint-plugin-react": "7.30.1",
-    "playwright": "1.24.1",
+    "playwright": "1.24.2",
     "prettier": "2.7.1"
   }
 }

package/readme.md

@@ -1,7 +1,7 @@
 # @jsenv/core [![npm package](https://img.shields.io/npm/v/@jsenv/core.svg?logo=npm&label=package)](https://www.npmjs.com/package/@jsenv/core)
 
 Jsenv was first created to write tests that could be executed in different runtimes.
-It has naturally evolved to cover the core needs of a JavaScript project: developement, testing and building for production. 
+It has naturally evolved to cover the core needs of a JavaScript project: developement, testing and building for production.
 
 - :exploding_head: Execute HTML files as tests
 - :sparkles: A single tool for the whole developer experience

package/src/build/inject_global_version_mappings.js

@@ -61,12 +61,12 @@ const injectors = {
     injectScriptNodeAsEarlyAsPossible(
       htmlAst,
       createHtmlNode({
-        "tagName": "script",
-        "textContent": generateClientCodeForVersionMappings(versionMappings, {
+        tagName: "script",
+        textContent: generateClientCodeForVersionMappings(versionMappings, {
           globalName: "window",
         }),
-        "injected-by": "jsenv:versioning",
       }),
+      "jsenv:versioning",
     )
     return {
       content: stringifyHtmlAst(htmlAst),

package/src/dev/start_dev_server.js

@@ -48,7 +48,7 @@ export const startDevServer = async ({
   runtimeCompat = defaultRuntimeCompat,
   plugins = [],
   urlAnalysis = {},
-  htmlSupervisor = true,
+  supervisor = true,
   nodeEsmResolution,
   fileSystemMagicResolution,
   transpilation,
@@ -160,7 +160,7 @@ export const startDevServer = async ({
 
     plugins,
     urlAnalysis,
-    htmlSupervisor,
+    supervisor,
     nodeEsmResolution,
     fileSystemMagicResolution,
     transpilation,

package/src/execute/execute.js

@@ -94,7 +94,7 @@ export const execute = async ({
   ```
   But it feels like a hack.
   */
-      throw result.error
+      throw result.errors[result.errors.length - 1]
     }
     return result
   } finally {

package/src/execute/run.js

@@ -16,7 +16,11 @@ export const run = async ({
   runtime,
   runtimeParams,
 }) => {
-  const result = {}
+  const result = {
+    status: "pending",
+    errors: [],
+    namespace: null,
+  }
   const callbacks = []
 
   const onConsoleRef = { current: () => {} }
@@ -25,6 +29,7 @@ export const run = async ({
 
   const runOperation = Abort.startOperation()
   runOperation.addAbortSignal(signal)
+  let timeoutAbortSource
   if (
     // ideally we would rather log than the timeout is ignored
     // when keepRunning is true
@@ -32,24 +37,8 @@ export const run = async ({
     typeof allocatedMs === "number" &&
     allocatedMs !== Infinity
   ) {
-    const timeoutAbortSource = runOperation.timeout(allocatedMs)
-    callbacks.push(() => {
-      if (
-        result.status === "errored" &&
-        Abort.isAbortError(result.error) &&
-        timeoutAbortSource.signal.aborted
-      ) {
-        result.status = "timedout"
-        delete result.error
-      }
-    })
+    timeoutAbortSource = runOperation.timeout(allocatedMs)
   }
-  callbacks.push(() => {
-    if (result.status === "errored" && Abort.isAbortError(result.error)) {
-      result.status = "aborted"
-      delete result.error
-    }
-  })
   const consoleCalls = []
   onConsoleRef.current = ({ type, text }) => {
     if (mirrorConsole) {
@@ -64,9 +53,7 @@ export const run = async ({
     }
   }
   if (collectConsole) {
-    callbacks.push(() => {
-      result.consoleCalls = consoleCalls
-    })
+    result.consoleCalls = consoleCalls
   }
 
   // we do not keep coverage in memory, it can grow very big
@@ -133,29 +120,30 @@ export const run = async ({
     if (winner.name === "aborted") {
       runOperation.throwIfAborted()
     }
-
-    const { status, namespace, error, performance } = winner.data
+    const { status, namespace, errors, performance } = winner.data
     result.status = status
-    if (status === "errored") {
-      result.error = error
-    } else {
-      result.namespace = namespace
-    }
+    result.errors.push(...errors)
+    result.namespace = namespace
     if (collectPerformance) {
       result.performance = performance
     }
-    callbacks.forEach((callback) => {
-      callback()
-    })
-    return result
   } catch (e) {
-    result.status = "errored"
-    result.error = e
-    callbacks.forEach((callback) => {
-      callback()
-    })
-    return result
+    if (Abort.isAbortError(e)) {
+      if (timeoutAbortSource && timeoutAbortSource.signal.aborted) {
+        result.status = "timedout"
+      } else {
+        result.status = "aborted"
+      }
+    } else {
+      result.status = "errored"
+      result.errors.push(e)
+    }
   } finally {
     await runOperation.end()
   }
+
+  callbacks.forEach((callback) => {
+    callback()
+  })
+  return result
 }

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

@@ -1,4 +1,3 @@
-import { Script } from "node:vm"
 import { writeFileSync } from "node:fs"
 
 import { createDetailedMessage } from "@jsenv/log"
@@ -10,7 +9,6 @@ import {
 } from "@jsenv/abort"
 import { moveUrl } from "@jsenv/urls"
 import { memoize } from "@jsenv/utils/src/memoize/memoize.js"
-import { escapeRegexpSpecialChars } from "@jsenv/utils/src/string/escape_regexp_special_chars.js"
 
 import { filterV8Coverage } from "@jsenv/core/src/test/coverage/v8_coverage.js"
 import { composeTwoFileByFileIstanbulCoverages } from "@jsenv/core/src/test/coverage/istanbul_coverage_composition.js"
@@ -121,7 +119,11 @@ export const createRuntimeFromPlaywright = ({
       }
     }
 
-    const result = {}
+    const result = {
+      status: "pending",
+      namespace: null,
+      errors: [],
+    }
     const callbacks = []
     if (coverageEnabled) {
       if (
@@ -251,18 +253,18 @@ export const createRuntimeFromPlaywright = ({
             })
           },
           // https://github.com/GoogleChrome/puppeteer/blob/v1.4.0/docs/api.md#event-pageerror
-          pageerror: (cb) => {
-            return registerEvent({
-              object: page,
-              eventType: "pageerror",
-              callback: (error) => {
-                if (ignoreErrorHook(error)) {
-                  return
-                }
-                cb(transformErrorHook(error))
-              },
-            })
-          },
+          // pageerror: () => {
+          //   return registerEvent({
+          //     object: page,
+          //     eventType: "pageerror",
+          //     callback: (error) => {
+          //       if (ignoreErrorHook(error)) {
+          //         return
+          //       }
+          //       result.errors.push(transformErrorHook(error))
+          //     },
+          //   })
+          // },
           closed: (cb) => {
             // https://github.com/GoogleChrome/puppeteer/blob/v1.4.0/docs/api.md#event-disconnected
             if (isBrowserDedicatedToExecution) {
@@ -301,32 +303,14 @@ export const createRuntimeFromPlaywright = ({
                 /* eslint-disable no-undef */
                 /* istanbul ignore next */
                 () => {
-                  if (!window.__html_supervisor__) {
-                    throw new Error(`window.__html_supervisor__ not found`)
+                  if (!window.__supervisor__) {
+                    throw new Error(`window.__supervisor__ not found`)
                   }
-                  return window.__html_supervisor__.getScriptExecutionResults()
+                  return window.__supervisor__.getDocumentExecutionResult()
                 },
                 /* eslint-enable no-undef */
               )
-              const { status, scriptExecutionResults } = returnValue
-              if (status === "errored") {
-                const { exceptionSource } = returnValue
-                const error = evalException(exceptionSource, {
-                  rootDirectoryUrl,
-                  devServerOrigin,
-                  transformErrorHook,
-                })
-                cb({
-                  status: "errored",
-                  error,
-                  namespace: scriptExecutionResults,
-                })
-              } else {
-                cb({
-                  status: "completed",
-                  namespace: scriptExecutionResults,
-                })
-              }
+              cb(returnValue)
             } catch (e) {
               reject(e)
             }
@@ -336,39 +320,46 @@ export const createRuntimeFromPlaywright = ({
       )
     })
 
-    const getResult = async () => {
+    const writeResult = async () => {
       const winner = await winnerPromise
       if (winner.name === "aborted") {
-        return {
-          status: "aborted",
-        }
+        result.status = "aborted"
+        return
       }
-      if (winner.name === "error" || winner.name === "pageerror") {
-        const error = winner.data
-        return {
-          status: "errored",
-          error,
-        }
+      if (winner.name === "error") {
+        let error = winner.data
+        result.status = "errored"
+        result.errors.push(error)
+        return
       }
       if (winner.name === "closed") {
-        return {
-          status: "errored",
-          error: isBrowserDedicatedToExecution
+        result.status = "errored"
+        result.errors.push(
+          isBrowserDedicatedToExecution
             ? new Error(`browser disconnected during execution`)
             : new Error(`page closed during execution`),
-        }
+        )
+        return
       }
-      return winner.data
+      // winner.name = 'response'
+      const { executionResults } = winner.data
+      result.status = "completed"
+      result.namespace = executionResults
+      Object.keys(executionResults).forEach((key) => {
+        const executionResult = executionResults[key]
+        if (executionResult.status === "errored") {
+          result.status = "errored"
+          result.errors.push({
+            ...executionResult.exception,
+            stack: executionResult.exception.text,
+          })
+        }
+      })
+      return
     }
 
     try {
-      const { status, error, namespace, performance } = await getResult()
-      result.status = status
-      if (status === "errored") {
-        result.error = error
-      } else {
-        result.namespace = namespace
-      }
+      await writeResult()
       if (collectPerformance) {
         result.performance = performance
       }
@@ -378,7 +369,7 @@ export const createRuntimeFromPlaywright = ({
       }, Promise.resolve())
     } catch (e) {
       result.status = "errored"
-      result.error = e
+      result.errors = [e]
     }
     if (keepRunning) {
       stopSignal.notify = cleanup
@@ -533,20 +524,3 @@ const registerEvent = ({ object, eventType, callback }) => {
     object.removeListener(eventType, callback)
   }
 }
-
-const evalException = (
-  exceptionSource,
-  { rootDirectoryUrl, devServerOrigin, transformErrorHook },
-) => {
-  const script = new Script(exceptionSource, { filename: "" })
-  const error = script.runInThisContext()
-  if (error && error instanceof Error) {
-    const remoteRootRegexp = new RegExp(
-      escapeRegexpSpecialChars(`${devServerOrigin}/`),
-      "g",
-    )
-    error.stack = error.stack.replace(remoteRootRegexp, rootDirectoryUrl)
-    error.message = error.message.replace(remoteRootRegexp, rootDirectoryUrl)
-  }
-  return transformErrorHook(error)
-}

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

@@ -185,7 +185,13 @@ nodeChildProcess.run = async ({
       resolve,
     )
   })
-  const getResult = async () => {
+  const result = {
+    status: "executing",
+    errors: [],
+    namespace: null,
+  }
+
+  const writeResult = async () => {
     actionOperation.throwIfAborted()
     await childProcessReadyPromise
     actionOperation.throwIfAborted()
@@ -207,28 +213,27 @@ nodeChildProcess.run = async ({
     })
     const winner = await winnerPromise
     if (winner.name === "aborted") {
-      return {
-        status: "aborted",
-      }
+      result.status = "aborted"
+      return
     }
     if (winner.name === "error") {
       const error = winner.data
       removeOutputListener()
-      return {
-        status: "errored",
-        error,
-      }
+      result.status = "errored"
+      result.errors.push(error)
+      return
     }
     if (winner.name === "exit") {
       const { code } = winner.data
       await cleanup("process exit")
       if (code === 12) {
-        return {
-          status: "errored",
-          error: new Error(
+        result.status = "errored"
+        result.errors.push(
+          new Error(
             `node process exited with 12 (the forked child process wanted to use a non-available port for debug)`,
           ),
-        }
+        )
+        return
       }
       if (
         code === null ||
@@ -237,41 +242,36 @@ nodeChildProcess.run = async ({
         code === EXIT_CODES.SIGTERM ||
         code === EXIT_CODES.SIGABORT
       ) {
-        return {
-          status: "errored",
-          error: new Error(`node process exited during execution`),
-        }
+        result.status = "errored"
+        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.
-      return {
-        status: "errored",
-        error: new Error(
-          `node process exited with code ${code} during execution`,
-        ),
-      }
+      result.status = "errored"
+      result.errors.push(
+        new Error(`node process exited with code ${code} during execution`),
+      )
+      return
     }
     const { status, value } = winner.data
     if (status === "action-failed") {
-      return {
-        status: "errored",
-        error: value,
-      }
-    }
-    return {
-      status: "completed",
-      ...value,
+      result.status = "errored"
+      result.errors.push(value)
+      return
     }
+    const { namespace, performance, coverage } = value
+    result.status = "completed"
+    result.namespace = namespace
+    result.performance = performance
+    result.coverage = coverage
   }
 
-  let result
   try {
-    result = await getResult()
+    await writeResult()
   } catch (e) {
-    result = {
-      status: "errored",
-      error: e,
-    }
+    result.status = "errored"
+    result.errors.push(e)
   }
   if (keepRunning) {
     stopSignal.notify = stop

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

@@ -137,7 +137,13 @@ nodeWorkerThread.run = async ({
     )
   })
 
-  const getResult = async () => {
+  const result = {
+    status: "executing",
+    errors: [],
+    namespace: null,
+  }
+
+  const writeResult = async () => {
     actionOperation.throwIfAborted()
     await workerThreadReadyPromise
     actionOperation.throwIfAborted()
@@ -159,28 +165,27 @@ nodeWorkerThread.run = async ({
     })
     const winner = await winnerPromise
     if (winner.name === "aborted") {
-      return {
-        status: "aborted",
-      }
+      result.status = "aborted"
+      return
     }
     if (winner.name === "error") {
       const error = winner.data
       removeOutputListener()
-      return {
-        status: "errored",
-        error,
-      }
+      result.status = "errored"
+      result.errors.push(error)
+      return
     }
     if (winner.name === "exit") {
       const { code } = winner.data
       await cleanup("process exit")
       if (code === 12) {
-        return {
-          status: "errored",
-          error: new Error(
+        result.status = "errored"
+        result.errors.push(
+          new Error(
             `node process exited with 12 (the forked child process wanted to use a non-available port for debug)`,
           ),
-        }
+        )
+        return
       }
       if (
         code === null ||
@@ -189,44 +194,39 @@ nodeWorkerThread.run = async ({
         code === EXIT_CODES.SIGTERM ||
         code === EXIT_CODES.SIGABORT
       ) {
-        return {
-          status: "errored",
-          error: new Error(`node worker thread exited during execution`),
-        }
+        result.status = "errored"
+        result.errors.push(
+          new Error(`node worker thread 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.
-      return {
-        status: "errored",
-        error: new Error(
+      result.status = "errored"
+      result.errors.push(
+        new Error(
           `node worker thread exited with code ${code} during execution`,
         ),
-      }
+      )
     }
     const { status, value } = winner.data
     if (status === "action-failed") {
-      return {
-        status: "errored",
-        error: value,
-      }
+      result.status = "errored"
+      result.errors.push(value)
+      return
     }
     const { namespace, performance, coverage } = value
-    return {
-      status: "completed",
-      namespace,
-      performance,
-      coverage,
-    }
+    result.status = "completed"
+    result.namespace = namespace
+    result.performance = performance
+    result.coverage = coverage
   }
 
-  let result
   try {
-    result = await getResult()
+    await writeResult()
   } catch (e) {
-    result = {
-      status: "errored",
-      error: e,
-    }
+    result.status = "errored"
+    result.errors.push(e)
   }
 
   if (keepRunning) {