@jsenv/core 27.0.1 → 27.0.2

package/dist/main.js

@@ -19850,7 +19850,7 @@ const rollupPluginJsenv = ({
     outputOptions: outputOptions => {
       // const sourcemapFile = buildDirectoryUrl
       Object.assign(outputOptions, {
-        format: "esm",
+        format: jsModuleUrlInfos.some(jsModuleUrlInfo => jsModuleUrlInfo.filename.endsWith(".cjs")) ? "cjs" : "esm",
         dir: fileUrlConverter.asFilePath(buildDirectoryUrl),
         sourcemap: sourcemaps === "file" || sourcemaps === "inline",
         // sourcemapFile,
@@ -25478,6 +25478,7 @@ const executePlan = async (plan, {
     });
 
     if (!keepRunning) {
+      logger.debug("stopAfterAllSignal.notify()");
       await stopAfterAllSignal.notify();
     }
 
@@ -25888,7 +25889,7 @@ const createRuntimeFromPlaywright = ({
 
   runtime.run = async ({
     signal = new AbortController().signal,
-    // logger,
+    logger,
     rootDirectoryUrl,
     fileRelativeUrl,
     server,
@@ -25947,10 +25948,10 @@ const createRuntimeFromPlaywright = ({
         };
 
         browser.on("disconnected", disconnectedCallback);
-      }) : Promise.resolve(); // for some reason without this 100ms timeout
+      }) : Promise.resolve(); // for some reason without this 150ms timeout
       // browser.close() never resolves (playwright does not like something)
 
-      await new Promise(resolve => setTimeout(resolve, 100));
+      await new Promise(resolve => setTimeout(resolve, 150));
 
       try {
         await browser.close();
@@ -26148,6 +26149,7 @@ const createRuntimeFromPlaywright = ({
             stopAfterAllSignal.notify = async () => {
               await notifyPrevious();
               browser.removeListener("disconnected", disconnectedCallback);
+              logger.debug(`stopAfterAllSignal notified -> closing ${browserName}`);
               await closeBrowser();
             };
           }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "27.0.1",
+  "version": "27.0.2",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -43,17 +43,17 @@
     "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-packages": "npm run test --workspaces --if-present -- --workspace",
+    "test:coverage": "npm run test -- --coverage",
     "build": "node --conditions=development ./scripts/build/build.mjs",
-    "start_file_server": "node ./scripts/dev/start_file_server.mjs",
-    "workspace-versions": "node ./scripts/publish/workspace_versions.mjs",
-    "workspace-publish": "node ./scripts/publish/workspace_publish.mjs",
+    "workspace:test": "npm run test --workspaces --if-present -- --workspace",
+    "workspace:versions": "node ./scripts/publish/workspace_versions.mjs",
+    "workspace:publish": "node ./scripts/publish/workspace_publish.mjs",
     "performances": "node --expose-gc ./scripts/performance/generate_performance_report.mjs --log --once",
     "file-size": "node ./scripts/file_size/file_size.mjs --log",
+    "start_file_server": "node ./scripts/dev/start_file_server.mjs",
     "prettier": "prettier --write .",
     "playwright-install": "npx playwright install-deps && npx playwright install",
     "certificate-install": "node ./scripts/dev/install_certificate_authority.mjs",
-    "test-with-coverage": "npm run test -- --coverage",
     "prepublishOnly": "npm run build"
   },
   "optionalDependencies": {

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

@@ -30,7 +30,7 @@ export const createRuntimeFromPlaywright = ({
   let browserAndContextPromise
   runtime.run = async ({
     signal = new AbortController().signal,
-    // logger,
+    logger,
     rootDirectoryUrl,
     fileRelativeUrl,
     server,
@@ -82,9 +82,9 @@ export const createRuntimeFromPlaywright = ({
             browser.on("disconnected", disconnectedCallback)
           })
         : Promise.resolve()
-      // for some reason without this 100ms timeout
+      // for some reason without this 150ms timeout
       // browser.close() never resolves (playwright does not like something)
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 150))
       try {
         await browser.close()
       } catch (e) {
@@ -275,6 +275,9 @@ export const createRuntimeFromPlaywright = ({
               stopAfterAllSignal.notify = async () => {
                 await notifyPrevious()
                 browser.removeListener("disconnected", disconnectedCallback)
+                logger.debug(
+                  `stopAfterAllSignal notified -> closing ${browserName}`,
+                )
                 await closeBrowser()
               }
             }

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

@@ -147,7 +147,11 @@ const rollupPluginJsenv = ({
     outputOptions: (outputOptions) => {
       // const sourcemapFile = buildDirectoryUrl
       Object.assign(outputOptions, {
-        format: "esm",
+        format: jsModuleUrlInfos.some((jsModuleUrlInfo) =>
+          jsModuleUrlInfo.filename.endsWith(".cjs"),
+        )
+          ? "cjs"
+          : "esm",
         dir: fileUrlConverter.asFilePath(buildDirectoryUrl),
         sourcemap: sourcemaps === "file" || sourcemaps === "inline",
         // sourcemapFile,

package/src/test/execute_plan.js

@@ -457,6 +457,7 @@ export const executePlan = async (
       },
     })
     if (!keepRunning) {
+      logger.debug("stopAfterAllSignal.notify()")
       await stopAfterAllSignal.notify()
     }