@jsenv/core 27.3.3 → 27.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "27.3.3",
+  "version": "27.5.0",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -43,7 +43,6 @@
     "eslint": "npx eslint . --ext=.js,.mjs,.cjs,.html",
     "dev": "node --conditions=development ./scripts/dev/dev.mjs",
     "test": "node --conditions=development ./scripts/test/test.mjs",
-    "test:coverage": "cross-env NODE_V8_COVERAGE=.coverage/node npm run test -- --coverage",
     "test:workspace": "npm run test --workspaces --if-present -- --workspace",
     "build": "node --conditions=development ./scripts/build/build.mjs",
     "workspace:versions": "node ./scripts/publish/workspace_versions.mjs",
@@ -66,31 +65,33 @@
     "@c88/v8-coverage": "0.1.1",
     "@financial-times/polyfill-useragent-normaliser": "2.0.1",
     "@jsenv/abort": "4.2.3",
-    "@jsenv/ast": "1.1.2",
+    "@jsenv/ast": "1.1.3",
     "@jsenv/babel-plugins": "1.0.6",
-    "@jsenv/filesystem": "4.1.0",
+    "@jsenv/filesystem": "4.1.1",
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.1",
     "@jsenv/log": "3.1.0",
     "@jsenv/node-esm-resolution": "0.1.0",
-    "@jsenv/server": "12.8.0",
-    "@jsenv/sourcemap": "1.0.1",
+    "@jsenv/server": "13.0.0",
+    "@jsenv/sourcemap": "1.0.2",
     "@jsenv/uneval": "1.6.0",
     "@jsenv/url-meta": "7.0.0",
-    "@jsenv/urls": "1.2.6",
+    "@jsenv/urls": "1.2.7",
     "@jsenv/utils": "2.0.1",
     "acorn-import-assertions": "1.8.0",
+    "construct-style-sheets-polyfill": "3.1.0",
     "cuid": "2.1.8",
     "html-minifier": "4.0.0",
     "istanbul-lib-coverage": "3.2.0",
     "istanbul-lib-instrument": "5.2.0",
     "istanbul-lib-report": "3.0.0",
-    "istanbul-reports": "3.1.4",
+    "istanbul-reports": "3.1.5",
+    "launch-editor": "2.4.0",
     "pidtree": "0.6.0",
-    "rollup": "2.76.0",
+    "rollup": "2.77.0",
     "string-width": "5.1.2",
     "strip-ansi": "7.0.1",
-    "terser": "5.14.1",
+    "terser": "5.14.2",
     "v8-to-istanbul": "9.0.1",
     "wrap-ansi": "8.0.1"
   },
@@ -98,17 +99,16 @@
     "@babel/eslint-parser": "7.18.2",
     "@babel/plugin-syntax-import-assertions": "7.18.6",
     "@jsenv/assert": "2.6.0",
-    "@jsenv/eslint-config": "16.1.0",
+    "@jsenv/eslint-config": "16.2.1",
     "@jsenv/file-size-impact": "13.0.1",
     "@jsenv/https-local": "2.1.0",
     "@jsenv/package-workspace": "0.4.1",
     "@jsenv/performance-impact": "3.0.1",
-    "cross-env": "7.0.3",
-    "eslint": "8.19.0",
-    "eslint-plugin-html": "6.2.0",
+    "eslint": "8.20.0",
+    "eslint-plugin-html": "7.0.0",
     "eslint-plugin-import": "2.26.0",
     "eslint-plugin-react": "7.30.1",
-    "playwright": "1.23.2",
+    "playwright": "1.23.4",
     "prettier": "2.7.1"
   }
 }

package/{README.md → readme.md}

@@ -1,21 +1,32 @@
 # @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.
+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. 
 
-- :exploding_head: Directly execute HTML files as tests
-- :relieved: Consistent and integral developer experience (dev + test + build)
-- :relieved: Same dev experience for source and test files
+- :exploding_head: Execute HTML files as tests
+- :sparkles: A single tool for the whole developer experience
 - :ok_hand: Seamless integration with standard HTML, CSS and JS
 
 # Demo
 
-Run the following command to create a demo of a jsenv project on your machine.
+The following command can be used to create a jsenv project on a machine.
 
 ```console
 npm create jsenv@latest
 ```
 
-> "web" demos are running tests in headless browsers. If you never have installed playwright on your machine npm install can take a bit of time.
+This command prompts to choose a demo from a list.
+Each demo contains preconfigured scripts such as:
+
+- `npm run dev`: starts a dev server with autoreload.
+- `npm run test`: execute test files on browsers(s) and/or Node.js.
+- `npm run build`: generate files optimized for production.
+
+> **Info**
+> Executing "npm install" in web demos can take time.
+> It is because in these demos tests are runned in headless browsers that needs to be installed first.
+
+See also [packages/create-jsenv](./packages/create-jsenv).
 
 # Installation
 
@@ -23,7 +34,7 @@ npm create jsenv@latest
 npm install --save-dev @jsenv/core
 ```
 
-_@jsenv/core_ is tested on Mac, Windows, Linux on Node.js 18.5.0. Other operating systems and Node.js versions are not tested.
+_@jsenv/core_ is tested on Mac, Windows, Linux with Node.js 18.5.0. Other operating systems and Node.js versions are not tested.
 
 # Documentation
 

package/src/build/build.js

@@ -102,7 +102,7 @@ export const build = async ({
   sourcemaps = false,
   sourcemapsSourcesContent,
   urlAnalysis = {},
-  nodeEsmResolution,
+  nodeEsmResolution = true,
   fileSystemMagicResolution,
   directoryReferenceAllowed,
   transpilation = {},
@@ -228,7 +228,9 @@ build ${entryPointKeys.length} entry points`)
         startLoading: (cookEntryFile) => {
           Object.keys(entryPoints).forEach((key) => {
             const [, entryUrlInfo] = cookEntryFile({
-              trace: `"${key}" in entryPoints parameter`,
+              trace: {
+                message: `"${key}" in entryPoints parameter`,
+              },
               type: "entry_point",
               specifier: key,
             })
@@ -415,8 +417,6 @@ build ${entryPointKeys.length} entry points`)
     })
     const postBuildRedirections = {}
     const finalGraph = createUrlGraph()
-    const optimizeUrlContentHooks =
-      rawGraphKitchen.pluginController.addHook("optimizeUrlContent")
     const finalGraphKitchen = createKitchen({
       logger,
       rootDirectoryUrl,
@@ -692,19 +692,17 @@ build ${entryPointKeys.length} entry points`)
           name: "jsenv:optimize",
           appliesDuring: "build",
           finalizeUrlContent: async (urlInfo, context) => {
-            if (optimizeUrlContentHooks.length) {
-              await rawGraphKitchen.pluginController.callAsyncHooks(
-                "optimizeUrlContent",
-                urlInfo,
-                context,
-                async (optimizeReturnValue) => {
-                  await finalGraphKitchen.urlInfoTransformer.applyFinalTransformations(
-                    urlInfo,
-                    optimizeReturnValue,
-                  )
-                },
-              )
-            }
+            await rawGraphKitchen.pluginController.callAsyncHooks(
+              "optimizeUrlContent",
+              urlInfo,
+              context,
+              async (optimizeReturnValue) => {
+                await finalGraphKitchen.urlInfoTransformer.applyFinalTransformations(
+                  urlInfo,
+                  optimizeReturnValue,
+                )
+              },
+            )
           },
         },
       ],
@@ -722,7 +720,7 @@ build ${entryPointKeys.length} entry points`)
         startLoading: (cookEntryFile) => {
           entryUrls.forEach((entryUrl) => {
             const [, postBuildEntryUrlInfo] = cookEntryFile({
-              trace: `entryPoint`,
+              trace: { message: `entryPoint` },
               type: "entry_point",
               specifier: entryUrl,
             })

package/src/build/start_build_server.js

@@ -17,12 +17,9 @@ import { parentPort } from "node:worker_threads"
 import {
   jsenvAccessControlAllowedHeaders,
   startServer,
-  pluginServerTiming,
-  pluginRequestWaitingCheck,
-  pluginCORS,
   fetchFileSystem,
-  composeServices,
-  findFreePort,
+  jsenvServiceCORS,
+  jsenvServiceErrorHandler,
 } from "@jsenv/server"
 import {
   assertAndNormalizeDirectoryUrl,
@@ -43,10 +40,10 @@ export const startBuildServer = async ({
   http2,
   certificate,
   privateKey,
-  listenAnyIp,
-  ip,
+  acceptAnyIp,
+  host,
   port = 9779,
-  services = {},
+  services = [],
   keepProcessAlive = true,
 
   rootDirectoryUrl,
@@ -94,9 +91,6 @@ export const startBuildServer = async ({
       )
     })
   }
-  if (port === 0) {
-    port = await findFreePort(port, { signal: operation.signal })
-  }
 
   let reloadableWorker
   if (buildServerAutoreload) {
@@ -137,12 +131,12 @@ export const startBuildServer = async ({
       const messagePromise = new Promise((resolve) => {
         worker.once("message", resolve)
       })
-      await messagePromise
+      const origin = await messagePromise
       // if (!keepProcessAlive) {
       //   worker.unref()
       // }
       return {
-        origin: `${protocol}://127.0.0.1:${port}`,
+        origin,
         stop: () => {
           stopWatchingBuildServerFiles()
           reloadableWorker.terminate()
@@ -168,30 +162,32 @@ export const startBuildServer = async ({
     http2,
     certificate,
     privateKey,
-    listenAnyIp,
-    ip,
+    acceptAnyIp,
+    host,
     port,
-    plugins: {
-      ...pluginCORS({
+    serverTiming: true,
+    requestWaitingMs: 60_000,
+    services: [
+      jsenvServiceCORS({
         accessControlAllowRequestOrigin: true,
         accessControlAllowRequestMethod: true,
         accessControlAllowRequestHeaders: true,
         accessControlAllowedRequestHeaders: jsenvAccessControlAllowedHeaders,
         accessControlAllowCredentials: true,
+        timingAllowOrigin: true,
       }),
-      ...pluginServerTiming(),
-      ...pluginRequestWaitingCheck({
-        requestWaitingMs: 60 * 1000,
-      }),
-    },
-    sendErrorDetails: true,
-    requestToResponse: composeServices({
       ...services,
-      build_files_service: createBuildFilesService({
-        buildDirectoryUrl,
-        buildIndexPath,
+      {
+        name: "jsenv:build_files_service",
+        handleRequest: createBuildFilesService({
+          buildDirectoryUrl,
+          buildIndexPath,
+        }),
+      },
+      jsenvServiceErrorHandler({
+        sendErrorDetails: true,
       }),
-    }),
+    ],
   })
   startBuildServerTask.done()
   logger.info(``)

package/src/dev/start_dev_server.js

@@ -1,5 +1,4 @@
 import { parentPort } from "node:worker_threads"
-import { findFreePort } from "@jsenv/server"
 import {
   assertAndNormalizeDirectoryUrl,
   registerDirectoryLifecycle,
@@ -9,14 +8,8 @@ import { createLogger, loggerToLevels, createTaskLog } from "@jsenv/log"
 import { getCallerPosition } from "@jsenv/urls"
 
 import { createReloadableWorker } from "@jsenv/core/src/helpers/worker_reload.js"
-import { getCorePlugins } from "@jsenv/core/src/plugins/plugins.js"
-import { createUrlGraph } from "@jsenv/core/src/omega/url_graph.js"
-import { createKitchen } from "@jsenv/core/src/omega/kitchen.js"
 import { startOmegaServer } from "@jsenv/core/src/omega/omega_server.js"
 
-import { jsenvPluginExplorer } from "./plugins/explorer/jsenv_plugin_explorer.js"
-// import { jsenvPluginToolbar } from "./plugins/toolbar/jsenv_plugin_toolbar.js"
-
 export const startDevServer = async ({
   signal = new AbortController().signal,
   handleSIGINT = true,
@@ -24,14 +17,14 @@ export const startDevServer = async ({
   omegaServerLogLevel = "warn",
   port = 3456,
   protocol = "http",
-  listenAnyIp,
+  acceptAnyIp,
   // 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,
   keepProcessAlive = true,
-  serverPlugins,
+  services,
 
   rootDirectoryUrl,
   clientFiles = {
@@ -47,7 +40,6 @@ export const startDevServer = async ({
   devServerMainFile = getCallerPosition().url,
   cooldownBetweenFileEvents,
 
-  sourcemaps = "inline",
   // default runtimeCompat assume dev server will be request by recent browsers
   // Used by "jsenv_plugin_node_runtime.js" to deactivate itself
   // If dev server can be requested by Node.js to exec files
@@ -64,16 +56,19 @@ export const startDevServer = async ({
   nodeEsmResolution,
   fileSystemMagicResolution,
   transpilation,
-  explorerGroups = {
-    source: {
-      "./*.html": true,
-      "./src/**/*.html": true,
-    },
-    test: {
-      "./tests/**/*.test.html": true,
+  explorer = {
+    groups: {
+      src: {
+        "./src/**/*.html": true,
+      },
+      tests: {
+        "./tests/**/*.test.html": true,
+      },
     },
   },
   // toolbar = false,
+
+  sourcemaps = "inline",
   writeGeneratedFiles = true,
 }) => {
   const logger = createLogger({ logLevel })
@@ -90,9 +85,6 @@ export const startDevServer = async ({
       )
     })
   }
-  if (port === 0) {
-    port = await findFreePort(port, { signal: operation.signal })
-  }
 
   let reloadableWorker
   if (devServerAutoreload) {
@@ -134,12 +126,12 @@ export const startDevServer = async ({
       const messagePromise = new Promise((resolve) => {
         worker.once("message", resolve)
       })
-      await messagePromise
+      const origin = await messagePromise
       // if (!keepProcessAlive) {
       //   worker.unref()
       // }
       return {
-        origin: `${protocol}://127.0.0.1:${port}`,
+        origin,
         stop: () => {
           stopWatchingDevServerFiles()
           reloadableWorker.terminate()
@@ -152,82 +144,32 @@ export const startDevServer = async ({
     disabled: !loggerToLevels(logger).info,
   })
 
-  const clientFileChangeCallbackList = []
-  const clientFilesPruneCallbackList = []
-  const clientFileChangeCallback = ({ relativeUrl, event }) => {
-    const url = new URL(relativeUrl, rootDirectoryUrl).href
-    clientFileChangeCallbackList.forEach((callback) => {
-      callback({ url, event })
-    })
-  }
-  const stopWatchingClientFiles = registerDirectoryLifecycle(rootDirectoryUrl, {
-    watchPatterns: {
-      ...clientFiles,
-      ".jsenv/": false,
-    },
-    cooldownBetweenFileEvents,
-    keepProcessAlive: false,
-    recursive: true,
-    added: ({ relativeUrl }) => {
-      clientFileChangeCallback({ event: "added", relativeUrl })
-    },
-    updated: ({ relativeUrl }) => {
-      clientFileChangeCallback({ event: "modified", relativeUrl })
-    },
-    removed: ({ relativeUrl }) => {
-      clientFileChangeCallback({ event: "removed", relativeUrl })
-    },
-  })
-  const urlGraph = createUrlGraph({
-    clientFileChangeCallbackList,
-    clientFilesPruneCallbackList,
-  })
-  const kitchen = createKitchen({
-    signal,
-    logger,
-    rootDirectoryUrl,
-    urlGraph,
-    scenario: "dev",
-    runtimeCompat,
-    sourcemaps,
-    writeGeneratedFiles,
-    plugins: [
-      ...plugins,
-      ...getCorePlugins({
-        rootDirectoryUrl,
-        urlGraph,
-        scenario: "dev",
-        runtimeCompat,
-
-        urlAnalysis,
-        htmlSupervisor,
-        nodeEsmResolution,
-        fileSystemMagicResolution,
-        transpilation,
-        clientAutoreload,
-        clientFileChangeCallbackList,
-        clientFilesPruneCallbackList,
-      }),
-      jsenvPluginExplorer({
-        groups: explorerGroups,
-      }),
-      // ...(toolbar ? [jsenvPluginToolbar(toolbar)] : []),
-    ],
-  })
   const server = await startOmegaServer({
     logLevel: omegaServerLogLevel,
     keepProcessAlive,
-    listenAnyIp,
+    acceptAnyIp,
     port,
     protocol,
     http2,
     certificate,
     privateKey,
+    services,
+
     rootDirectoryUrl,
-    urlGraph,
-    kitchen,
     scenario: "dev",
-    serverPlugins,
+    runtimeCompat,
+    plugins,
+    urlAnalysis,
+    htmlSupervisor,
+    nodeEsmResolution,
+    fileSystemMagicResolution,
+    transpilation,
+    clientFiles,
+    clientAutoreload,
+    cooldownBetweenFileEvents,
+    explorer,
+    sourcemaps,
+    writeGeneratedFiles,
   })
   startDevServerTask.done()
   logger.info(``)
@@ -235,18 +177,12 @@ export const startDevServer = async ({
     logger.info(`- ${server.origins[key]}`)
   })
   logger.info(``)
-  server.addEffect(() => {
-    return () => {
-      kitchen.pluginController.callHooks("destroy", {})
-    }
-  })
   if (reloadableWorker && reloadableWorker.isWorker) {
     parentPort.postMessage(server.origin)
   }
   return {
     origin: server.origin,
     stop: () => {
-      stopWatchingClientFiles()
       server.stop()
     },
   }

package/src/execute/execute.js

@@ -2,11 +2,8 @@ import { Abort, raceProcessTeardownEvents } from "@jsenv/abort"
 
 import { assertAndNormalizeDirectoryUrl } from "@jsenv/filesystem"
 import { createLogger } from "@jsenv/log"
-import { getCorePlugins } from "@jsenv/core/src/plugins/plugins.js"
-import { createUrlGraph } from "@jsenv/core/src/omega/url_graph.js"
-import { createKitchen } from "@jsenv/core/src/omega/kitchen.js"
-import { startOmegaServer } from "@jsenv/core/src/omega/omega_server.js"
 
+import { startOmegaServer } from "@jsenv/core/src/omega/omega_server.js"
 import { run } from "./run.js"
 
 export const execute = async ({
@@ -19,6 +16,7 @@ export const execute = async ({
   allocatedMs,
   mirrorConsole = true,
   keepRunning = false,
+  services,
   collectConsole,
   collectCoverage,
   coverageTempDirectoryUrl,
@@ -27,12 +25,12 @@ export const execute = async ({
   runtimeParams,
 
   scenario = "dev",
-  sourcemaps = "inline",
   plugins = [],
   nodeEsmResolution,
   fileSystemMagicResolution,
   transpilation,
   htmlSupervisor = true,
+  sourcemaps = "inline",
   writeGeneratedFiles = false,
 
   port,
@@ -65,45 +63,29 @@ export const execute = async ({
     ...runtimeParams,
   }
   if (runtime.needsServer) {
-    const urlGraph = createUrlGraph()
-    const runtimeCompat = { [runtime.name]: runtime.version }
-    const kitchen = createKitchen({
-      signal,
-      logger,
-      rootDirectoryUrl,
-      urlGraph,
-      scenario,
-      sourcemaps,
-      runtimeCompat,
-      writeGeneratedFiles,
-      plugins: [
-        ...plugins,
-        ...getCorePlugins({
-          rootDirectoryUrl,
-          urlGraph,
-          scenario,
-          runtimeCompat,
-
-          htmlSupervisor,
-          nodeEsmResolution,
-          fileSystemMagicResolution,
-          transpilation,
-        }),
-      ],
-    })
     const server = await startOmegaServer({
       signal: executeOperation.signal,
       logLevel: "warn",
-      rootDirectoryUrl,
-      urlGraph,
-      kitchen,
-      scenario,
       keepProcessAlive: false,
+      services,
       port,
       protocol,
       http2,
       certificate,
       privateKey,
+
+      rootDirectoryUrl,
+      scenario,
+      runtimeCompat: { [runtime.name]: runtime.version },
+
+      plugins,
+
+      htmlSupervisor,
+      nodeEsmResolution,
+      fileSystemMagicResolution,
+      transpilation,
+      sourcemaps,
+      writeGeneratedFiles,
     })
     executeOperation.addEndCallback(async () => {
       await server.stop("execution done")

package/src/execute/run.js

@@ -40,12 +40,14 @@ export const run = async ({
         timeoutAbortSource.signal.aborted
       ) {
         result.status = "timedout"
+        delete result.error
       }
     })
   }
   callbacks.push(() => {
     if (result.status === "errored" && Abort.isAbortError(result.error)) {
       result.status = "aborted"
+      delete result.error
     }
   })
   const consoleCalls = []