@jsenv/core 27.0.0-alpha.49 → 27.0.0-alpha.51

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "27.0.0-alpha.49",
+  "version": "27.0.0-alpha.51",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "repository": {
@@ -68,7 +68,7 @@
     "@jsenv/node-esm-resolution": "0.0.6",
     "@jsenv/server": "12.6.2",
     "@jsenv/uneval": "1.6.0",
-    "@jsenv/utils": "1.7.0",
+    "@jsenv/utils": "1.7.1",
     "construct-style-sheets-polyfill": "3.1.0",
     "cssnano": "5.1.7",
     "cssnano-preset-default": "5.2.7",

package/src/build/build_urls_generator.js

@@ -1,6 +1,6 @@
 import { urlToFilename } from "@jsenv/filesystem"
 
-import { memoizeByUrl } from "@jsenv/utils/memoize/memoize_by_url.js"
+import { memoizeByFirstArgument } from "@jsenv/utils/memoize/memoize_by_first_argument.js"
 
 export const createBuilUrlsGenerator = ({ buildDirectoryUrl }) => {
   const cache = {}
@@ -15,7 +15,7 @@ export const createBuilUrlsGenerator = ({ buildDirectoryUrl }) => {
     return urlToFilename(url)
   }
 
-  const generate = memoizeByUrl((url, { urlInfo, parentUrlInfo }) => {
+  const generate = memoizeByFirstArgument((url, { urlInfo, parentUrlInfo }) => {
     const directoryPath = determineDirectoryPath({
       urlInfo,
       parentUrlInfo,

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

@@ -262,6 +262,9 @@ export const createRuntimeFromPlaywright = ({
                 cb({ reason: "page closed" })
               })
               cleanupCallbackList.add(closePage)
+              cleanupCallbackList.add(() => {
+                browser.removeListener("disconnected", disconnectedCallback)
+              })
               const notifyPrevious = stopAfterAllSignal.notify
               stopAfterAllSignal.notify = async () => {
                 await notifyPrevious()

package/src/omega/server/file_service.js

@@ -39,9 +39,6 @@ export const createFileService = ({
     if (responseFromPlugin) {
       return responseFromPlugin
     }
-    const { runtimeName, runtimeVersion } = parseUserAgentHeader(
-      request.headers["user-agent"],
-    )
     const [reference, urlInfo] = kitchen.prepareEntryPoint({
       parentUrl: inferParentFromRequest(request, rootDirectoryUrl),
       type: "entry_point",
@@ -65,6 +62,9 @@ export const createFileService = ({
       if (!urlInfo.isInline && !urlInfo.type === "sourcemap") {
         urlGraph.resetUrlInfo(urlInfo)
       }
+      const { runtimeName, runtimeVersion } = parseUserAgentHeader(
+        request.headers["user-agent"],
+      )
       await kitchen.cook({
         reference: referenceFromGraph || reference,
         urlInfo,

package/src/omega/server/user_agent.js

@@ -1,8 +1,10 @@
 import { createRequire } from "node:module"
 
+import { memoizeByFirstArgument } from "@jsenv/utils/memoize/memoize_by_first_argument.js"
+
 const require = createRequire(import.meta.url)
 
-export const parseUserAgentHeader = (userAgent) => {
+export const parseUserAgentHeader = memoizeByFirstArgument((userAgent) => {
   if (userAgent.includes("node-fetch/")) {
     // it's not really node and conceptually we can't assume the node version
     // but good enough for now
@@ -19,4 +21,4 @@ export const parseUserAgentHeader = (userAgent) => {
     runtimeVersion:
       family === "Other" ? "unknown" : `${major}.${minor}${patch}`,
   }
-}
+})

package/src/plugins/html_supervisor/jsenv_plugin_html_supervisor.js

@@ -147,9 +147,6 @@ export const jsenvPluginHtmlSupervisor = ({
             return
           }
         })
-        if (scriptsToSupervise.length === 0) {
-          return null
-        }
         const [htmlSupervisorInstallerFileReference] = referenceUtils.inject({
           type: "js_import_export",
           expectedType: "js_module",

package/src/plugins/plugin_controller.js

@@ -113,6 +113,9 @@ export const createPluginController = ({
   }
   const callAsyncHooksUntil = (hookName, info, context) => {
     const hooks = hookGroups[hookName]
+    if (hooks.length === 0) {
+      return null
+    }
     return new Promise((resolve, reject) => {
       const visit = (index) => {
         if (index >= hooks.length) {

package/src/test/logs_file_execution.js

@@ -60,7 +60,7 @@ export const formatExecutionResult = (
       file: fileRelativeUrl,
       runtime: `${runtimeName}/${runtimeVersion}`,
       duration: msAsDuration(duration),
-      ...(error ? { error: error.stack } : {}),
+      ...(error ? { error: error.stack || error.message || error } : {}),
     },
     consoleOutput,
   })