@jsenv/core 34.3.0 → 35.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "34.3.0",
+  "version": "35.0.0",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -10,7 +10,7 @@
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/jsenv/jsenv-core"
+    "url": "https://github.com/jsenv/core"
   },
   "engines": {
     "node": ">=18.5.0"
@@ -25,7 +25,7 @@
     ".": {
       "import": {
         "development": "./src/main.js",
-        "default": "./dist/jsenv.js"
+        "default": "./dist/jsenv_core.js"
       }
     },
     "./*": "./*"
@@ -45,7 +45,7 @@
     "test": "node --conditions=development ./scripts/test/test.mjs",
     "test:no_snapshot_assertion": "npm run test -- --no-snapshot-assertion",
     "test:dev_errors_snapshots": "node --conditions=development ./tests/dev_server/errors/generate_snapshot_files.mjs",
-    "test:workspace": "npm run test --workspaces --if-present -- --workspace",
+    "test:packages": "npm run test --workspaces --if-present -- --workspace",
     "build": "node --conditions=development ./scripts/build/build.mjs",
     "build:file_size": "node ./scripts/build/build_file_size.mjs --log",
     "workspace:versions": "node ./scripts/publish/workspace_versions.mjs",
@@ -56,14 +56,10 @@
     "certificate:install": "node ./scripts/dev/install_certificate_authority.mjs",
     "prepublishOnly": "npm run build"
   },
-  "optionalDependencies": {
-    "playwright": "1.x"
-  },
   "dependencies": {
     "@babel/plugin-proposal-dynamic-import": "7.18.6",
     "@babel/plugin-transform-modules-systemjs": "7.20.11",
     "@babel/plugin-transform-modules-umd": "7.18.6",
-    "@c88/v8-coverage": "0.1.1",
     "@financial-times/polyfill-useragent-normaliser": "1.10.2",
     "@jsenv/abort": "4.2.4",
     "@jsenv/ast": "3.0.6",
@@ -75,24 +71,12 @@
     "@jsenv/node-esm-resolution": "1.0.1",
     "@jsenv/server": "15.0.2",
     "@jsenv/sourcemap": "1.0.10",
-    "@jsenv/uneval": "1.6.0",
     "@jsenv/url-meta": "8.1.0",
     "@jsenv/urls": "2.0.0",
     "@jsenv/utils": "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.19.0",
-    "pidtree": "0.6.0",
-    "string-width": "5.1.2",
-    "strip-ansi": "7.0.1",
-    "v8-to-istanbul": "9.1.0",
-    "webidl2": "24.2.2",
-    "wrap-ansi": "8.1.0"
+    "lightningcss": "1.19.0"
   },
   "devDependencies": {
     "@babel/eslint-parser": "7.21.3",
@@ -107,6 +91,7 @@
     "@jsenv/plugin-minification": "./packages/jsenv-plugin-minification/",
     "@jsenv/plugin-placeholders": "./packages/jsenv-plugin-placeholders/",
     "@jsenv/plugin-as-js-classic": "./packages/jsenv-plugin-as-js-classic/",
+    "@jsenv/test": "./packages/test/",
     "eslint": "8.38.0",
     "eslint-plugin-html": "7.1.0",
     "eslint-plugin-import": "2.27.5",

package/src/build/build.js

@@ -53,8 +53,8 @@ import {
   findHtmlNode,
 } from "@jsenv/ast"
 
-import { lookupPackageDirectory } from "../lookup_package_directory.js"
-import { watchSourceFiles } from "../watch_source_files.js"
+import { lookupPackageDirectory } from "../helpers/lookup_package_directory.js"
+import { watchSourceFiles } from "../helpers/watch_source_files.js"
 import { createUrlGraph } from "../kitchen/url_graph.js"
 import { createKitchen } from "../kitchen/kitchen.js"
 import { RUNTIME_COMPAT } from "../kitchen/compat/runtime_compat.js"

package/src/dev/file_service.js

@@ -4,14 +4,14 @@ import { bufferToEtag } from "@jsenv/filesystem"
 import { asUrlWithoutSearch } from "@jsenv/urls"
 import { URL_META } from "@jsenv/url-meta"
 
-import { WEB_URL_CONVERTER } from "../web_url_converter.js"
-import { watchSourceFiles } from "../watch_source_files.js"
-import { explorerHtmlFileUrl } from "@jsenv/core/src/plugins/explorer/jsenv_plugin_explorer.js"
-import { createUrlGraph } from "@jsenv/core/src/kitchen/url_graph.js"
-import { createKitchen } from "@jsenv/core/src/kitchen/kitchen.js"
-import { RUNTIME_COMPAT } from "@jsenv/core/src/kitchen/compat/runtime_compat.js"
-import { getCorePlugins } from "@jsenv/core/src/plugins/plugins.js"
-import { jsenvPluginServerEventsClientInjection } from "@jsenv/core/src/plugins/server_events/jsenv_plugin_server_events_client_injection.js"
+import { WEB_URL_CONVERTER } from "../helpers/web_url_converter.js"
+import { watchSourceFiles } from "../helpers/watch_source_files.js"
+import { createUrlGraph } from "../kitchen/url_graph.js"
+import { createKitchen } from "../kitchen/kitchen.js"
+import { RUNTIME_COMPAT } from "../kitchen/compat/runtime_compat.js"
+import { getCorePlugins } from "../plugins/plugins.js"
+import { explorerHtmlFileUrl } from "../plugins/explorer/jsenv_plugin_explorer.js"
+import { jsenvPluginServerEventsClientInjection } from "../plugins/server_events/jsenv_plugin_server_events_client_injection.js"
 import { parseUserAgentHeader } from "./user_agent.js"
 
 export const createFileService = ({

package/src/dev/start_dev_server.js

@@ -9,9 +9,9 @@ import {
 } from "@jsenv/server"
 import { convertFileSystemErrorToResponseProperties } from "@jsenv/server/src/internal/convertFileSystemErrorToResponseProperties.js"
 
-import { lookupPackageDirectory } from "../lookup_package_directory.js"
-import { createServerEventsDispatcher } from "@jsenv/core/src/plugins/server_events/server_events_dispatcher.js"
-import { defaultRuntimeCompat } from "@jsenv/core/src/build/build.js"
+import { lookupPackageDirectory } from "../helpers/lookup_package_directory.js"
+import { createServerEventsDispatcher } from "../plugins/server_events/server_events_dispatcher.js"
+import { defaultRuntimeCompat } from "../build/build.js"
 import { createFileService } from "./file_service.js"
 
 /**

package/src/dev/user_agent.js

@@ -1,6 +1,6 @@
 import { memoizeByFirstArgument } from "@jsenv/utils/src/memoize/memoize_by_first_argument.js"
 
-import { requireFromJsenv } from "@jsenv/core/src/require_from_jsenv.js"
+import { requireFromJsenv } from "@jsenv/core/src/helpers/require_from_jsenv.js"
 
 export const parseUserAgentHeader = memoizeByFirstArgument((userAgent) => {
   if (userAgent.includes("node-fetch/")) {

package/src/main.js

@@ -1,28 +1,5 @@
 // dev
 export { startDevServer } from "./dev/start_dev_server.js"
-// test
-export { executeTestPlan } from "./test/execute_test_plan.js"
-// runtimes (used to execute tests)
-export {
-  chromium,
-  chromiumIsolatedTab,
-} from "./execute/runtimes/browsers/chromium.js"
-export {
-  firefox,
-  firefoxIsolatedTab,
-} from "./execute/runtimes/browsers/firefox.js"
-export {
-  webkit,
-  webkitIsolatedTab,
-} from "./execute/runtimes/browsers/webkit.js"
-export { nodeChildProcess } from "./execute/runtimes/node/node_child_process.js"
-export { nodeWorkerThread } from "./execute/runtimes/node/node_worker_thread.js"
 // build
 export { build } from "./build/build.js"
 export { startBuildServer } from "./build/start_build_server.js"
-
-// helpers
-export { pingServer } from "./ping_server.js"
-
-// advanced
-export { execute } from "./execute/execute.js"

package/src/plugins/supervisor/jsenv_plugin_supervisor.js

@@ -7,7 +7,7 @@ import { getOriginalPosition } from "@jsenv/sourcemap"
 import { stringifyUrlSite } from "@jsenv/urls"
 
 import { injectSupervisorIntoHTML } from "./html_supervisor_injection.js"
-import { requireFromJsenv } from "@jsenv/core/src/require_from_jsenv.js"
+import { requireFromJsenv } from "@jsenv/core/src/helpers/require_from_jsenv.js"
 
 export const supervisorFileUrl = new URL(
   "./client/supervisor.js",

package/src/plugins/transpilation/babel/require_babel_plugin.js

@@ -1,6 +1,6 @@
 import { createRequire } from "node:module"
 import { pathToFileURL } from "node:url"
-import { requireFromJsenv } from "@jsenv/core/src/require_from_jsenv.js"
+import { requireFromJsenv } from "@jsenv/core/src/helpers/require_from_jsenv.js"
 
 const babelPluginPackagePath = requireFromJsenv.resolve("@jsenv/babel-plugins")
 const babelPluginPackageUrl = pathToFileURL(babelPluginPackagePath)

package/src/plugins/transpilation/js_module_fallback/convert_js_module_to_js_classic.js

@@ -7,7 +7,7 @@ import {
 } from "@jsenv/sourcemap"
 import { applyBabelPlugins } from "@jsenv/ast"
 
-import { requireFromJsenv } from "@jsenv/core/src/require_from_jsenv.js"
+import { requireFromJsenv } from "@jsenv/core/src/helpers/require_from_jsenv.js"
 import { requireBabelPlugin } from "../babel/require_babel_plugin.js"
 import { babelPluginTransformImportMetaUrl } from "./helpers/babel_plugin_transform_import_meta_url.js"
 import { babelPluginTransformImportMetaResolve } from "./helpers/babel_plugin_transform_import_meta_resolve.js"

package/dist/controllable_child_process.mjs

@@ -1,129 +0,0 @@
-import { uneval, executeUsingDynamicImport } from "./js/execute_using_dynamic_import.js";
-import "node:fs";
-import "node:inspector";
-import "node:perf_hooks";
-
-const ACTIONS_AVAILABLE = {
-  "execute-using-dynamic-import": executeUsingDynamicImport,
-  "execute-using-require": async ({
-    fileUrl
-  }) => {
-    const {
-      createRequire
-    } = await import("node:module");
-    const {
-      fileURLToPath
-    } = await import("node:url");
-    const filePath = fileURLToPath(fileUrl);
-    const require = createRequire(fileUrl);
-    // eslint-disable-next-line import/no-dynamic-require
-    const namespace = require(filePath);
-    const namespaceResolved = {};
-    await Promise.all(Object.keys(namespace).map(async key => {
-      const value = await namespace[key];
-      namespaceResolved[key] = value;
-    }));
-    return namespaceResolved;
-  }
-};
-const ACTION_REQUEST_EVENT_NAME = "action";
-const ACTION_RESPONSE_EVENT_NAME = "action-result";
-const ACTION_RESPONSE_STATUS_FAILED = "action-failed";
-const ACTION_RESPONSE_STATUS_COMPLETED = "action-completed";
-const sendActionFailed = error => {
-  if (error.hasOwnProperty("toString")) {
-    delete error.toString;
-  }
-  sendToParent(ACTION_RESPONSE_EVENT_NAME,
-  // process.send algorithm does not send non enumerable values
-  // so use @jsenv/uneval
-  uneval({
-    status: ACTION_RESPONSE_STATUS_FAILED,
-    value: error
-  }, {
-    ignoreSymbols: true
-  }));
-};
-const sendActionCompleted = value => {
-  sendToParent(ACTION_RESPONSE_EVENT_NAME,
-  // here we use JSON.stringify because we should not
-  // have non enumerable value (unlike there is on Error objects)
-  // otherwise uneval is quite slow to turn a giant object
-  // into a string (and value can be giant when using coverage)
-  JSON.stringify({
-    status: ACTION_RESPONSE_STATUS_COMPLETED,
-    value
-  }));
-};
-const sendToParent = (type, data) => {
-  // https://nodejs.org/api/process.html#process_process_connected
-  // not connected anymore, cannot communicate with parent
-  if (!process.connected) {
-    return;
-  }
-  // this can keep process alive longer than expected
-  // when source is a long string.
-  // It means node process may stay alive longer than expected
-  // the time to send the data to the parent.
-  process.send({
-    jsenv: true,
-    type,
-    data
-  });
-};
-const onceParentMessage = (type, callback) => {
-  const listener = message => {
-    if (message && message.jsenv && message.type === type) {
-      removeListener(); // commenting this line keep this process alive
-      callback(message.data);
-    }
-  };
-  const removeListener = () => {
-    process.removeListener("message", listener);
-  };
-  process.on("message", listener);
-  return removeListener;
-};
-const removeActionRequestListener = onceParentMessage(ACTION_REQUEST_EVENT_NAME, async ({
-  actionType,
-  actionParams
-}) => {
-  const action = ACTIONS_AVAILABLE[actionType];
-  if (!action) {
-    sendActionFailed(new Error(`unknown action ${actionType}`));
-    return;
-  }
-  let value;
-  let failed = false;
-  try {
-    value = await action(actionParams);
-  } catch (e) {
-    failed = true;
-    value = e;
-  }
-
-  // setTimeout(() => {}, 100)
-
-  if (failed) {
-    sendActionFailed(value);
-  } else {
-    sendActionCompleted(value);
-  }
-
-  // removeActionRequestListener()
-  if (actionParams.exitAfterAction) {
-    removeActionRequestListener();
-    // for some reason this fixes v8 coverage directory sometimes empty on Ubuntu
-    // process.exit()
-  }
-});
-
-// remove listener to process.on('message')
-// which is sufficient to let child process die
-// assuming nothing else keeps it alive
-// process.once("SIGTERM", removeActionRequestListener)
-// process.once("SIGINT", removeActionRequestListener)
-
-setTimeout(() => {
-  sendToParent("ready");
-});

package/dist/controllable_worker_thread.mjs

@@ -1,91 +0,0 @@
-import { parentPort } from "node:worker_threads";
-import { uneval, executeUsingDynamicImport } from "./js/execute_using_dynamic_import.js";
-import "node:fs";
-import "node:inspector";
-import "node:perf_hooks";
-
-const ACTIONS_AVAILABLE = {
-  "execute-using-dynamic-import": executeUsingDynamicImport
-};
-const ACTION_REQUEST_EVENT_NAME = "action";
-const ACTION_RESPONSE_EVENT_NAME = "action-result";
-const ACTION_RESPONSE_STATUS_FAILED = "action-failed";
-const ACTION_RESPONSE_STATUS_COMPLETED = "action-completed";
-const sendActionFailed = error => {
-  if (error.hasOwnProperty("toString")) {
-    delete error.toString;
-  }
-  sendToParent(ACTION_RESPONSE_EVENT_NAME,
-  // process.send algorithm does not send non enumerable values
-  // so use @jsenv/uneval
-  uneval({
-    status: ACTION_RESPONSE_STATUS_FAILED,
-    value: error
-  }, {
-    ignoreSymbols: true
-  }));
-};
-const sendActionCompleted = value => {
-  sendToParent(ACTION_RESPONSE_EVENT_NAME,
-  // here we use JSON.stringify because we should not
-  // have non enumerable value (unlike there is on Error objects)
-  // otherwise uneval is quite slow to turn a giant object
-  // into a string (and value can be giant when using coverage)
-  JSON.stringify({
-    status: ACTION_RESPONSE_STATUS_COMPLETED,
-    value
-  }));
-};
-const sendToParent = (type, data) => {
-  // this can keep process alive longer than expected
-  // when source is a long string.
-  // It means node process may stay alive longer than expected
-  // the time to send the data to the parent.
-  parentPort.postMessage({
-    jsenv: true,
-    type,
-    data
-  });
-};
-const onceParentMessage = (type, callback) => {
-  const listener = message => {
-    if (message && message.jsenv && message.type === type) {
-      removeListener(); // commenting this line keep this worker alive
-      callback(message.data);
-    }
-  };
-  const removeListener = () => {
-    parentPort.removeListener("message", listener);
-  };
-  parentPort.on("message", listener);
-  return removeListener;
-};
-const removeActionRequestListener = onceParentMessage(ACTION_REQUEST_EVENT_NAME, async ({
-  actionType,
-  actionParams
-}) => {
-  const action = ACTIONS_AVAILABLE[actionType];
-  if (!action) {
-    sendActionFailed(new Error(`unknown action ${actionType}`));
-    return;
-  }
-  let value;
-  let failed = false;
-  try {
-    value = await action(actionParams);
-  } catch (e) {
-    failed = true;
-    value = e;
-  }
-  if (failed) {
-    sendActionFailed(value);
-  } else {
-    sendActionCompleted(value);
-  }
-  if (actionParams.exitAfterAction) {
-    removeActionRequestListener();
-  }
-});
-setTimeout(() => {
-  sendToParent("ready");
-});

package/dist/importmap_node_loader.mjs

@@ -1,49 +0,0 @@
-import { readFileSync } from "node:fs";
-import { pathToFileURL } from "node:url";
-import { normalizeImportMap, resolveImport } from "./js/resolveImport.js";
-
-let importMap;
-const cwdUrl = `${String(pathToFileURL(process.cwd()))}/`;
-if (process.env.IMPORT_MAP) {
-  importMap = JSON.parse(process.env.IMPORT_MAP);
-} else if (process.env.IMPORT_MAP_PATH) {
-  const importmapFileUrl = pathToFileURL(process.env.IMPORT_MAP_PATH);
-  const importmapFileContentAsString = readFileSync(importmapFileUrl, "utf8");
-  importMap = JSON.parse(importmapFileContentAsString);
-} else {
-  const importmapFileUrl = new URL("./import_map.json", cwdUrl);
-  const importmapFileContentAsString = readFileSync(importmapFileUrl, "utf8");
-  importMap = JSON.parse(importmapFileContentAsString);
-}
-const importMapBaseUrl = process.env.IMPORT_MAP_BASE_URL || cwdUrl;
-importMap = normalizeImportMap(importMap, importMapBaseUrl);
-const resolve = (specifier, context, nextResolve) => {
-  try {
-    let mapped;
-    const importer = context.parentURL ? String(context.parentURL) : undefined;
-    const resolved = resolveImport({
-      specifier,
-      importer,
-      importMap,
-      defaultExtension: ".js",
-      onImportMapping: () => {
-        mapped = true;
-      }
-    });
-    if (mapped) {
-      return {
-        shortCircuit: true,
-        url: resolved
-      };
-    }
-  } catch (e) {
-    if (e.message.includes("bare specifier")) {
-      return nextResolve(specifier, context);
-    }
-    console.error(e);
-    return nextResolve(specifier, context);
-  }
-  return nextResolve(specifier, context);
-};
-
-export { resolve };