@jsenv/plugin-supervisor 1.0.0 → 1.1.0

package/README.md

@@ -0,0 +1,4 @@
+This plugin has its own package because:
+
+- It's a direct dependency of `@jsenv/core` but it is easier to reason about this code when its properly splitted in its own package
+- `@jsenv/test` use most of the logic of this plugin. Having a dedicated NPM package allow to reuse that code without creating a direct dependency between `@jsenv/test` and `@jsenv/core`.

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@jsenv/plugin-supervisor",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "license": "MIT",
   "repository": {
     "type": "git",
     "url": "https://github.com/jsenv/core",
-    "directory": "packages/jsenv-plugin-supervisor"
+    "directory": "packages/internal/plugin-supervisor"
   },
   "publishConfig": {
     "access": "public"
@@ -14,7 +14,7 @@
   "exports": {
     ".": {
       "import": {
-        "development": "./src/jsenv_plugin_supervisor.js",
+        "development": "./src/main.js",
         "default": "./dist/jsenv_plugin_supervisor.js"
       }
     },
@@ -25,7 +25,7 @@
     "/src/"
   ],
   "dependencies": {
-    "@jsenv/ast": "4.0.0",
+    "@jsenv/ast": "4.0.3",
     "@jsenv/urls": "2.0.0",
     "@jsenv/sourcemap": "1.0.10",
     "launch-editor": "2.6.0"
@@ -33,6 +33,7 @@
   "scripts": {
     "eslint": "npx eslint . --ext=.js,.mjs,.cjs,.html",
     "build": "node ./scripts/build.mjs",
+    "test": "node ./scripts/test.mjs",
     "prepublishOnly": "npm run build"
   }
 }

package/src/{html_supervisor_injection.js → html_supervisor.js}

@@ -60,7 +60,7 @@ import {
 } from "@jsenv/ast";
 import { generateInlineContentUrl, urlToRelativeUrl } from "@jsenv/urls";
 
-import { injectSupervisorIntoJs } from "./js_supervisor_injection.js";
+import { injectSupervisorIntoJs } from "./js_supervisor.js";
 
 export const supervisorFileUrl = new URL(
   "./client/supervisor.js",

package/src/{js_supervisor_injection.js → js_supervisor.js}

@@ -52,12 +52,10 @@ export const injectSupervisorIntoJs = async ({
       ? babelPluginJsModuleSupervisor
       : babelPluginJsClassicSupervisor;
   const result = await applyBabelPlugins({
-    urlInfo: {
-      content,
-      originalUrl: url,
-      type,
-    },
     babelPlugins: [[babelPluginJsSupervisor, { inlineSrc }]],
+    input: content,
+    inputIsJsModule: type === "js_module",
+    inputUrl: url,
   });
   let code = result.code;
   let map = result.map;

package/src/jsenv_plugin_supervisor.js

@@ -1,5 +1,6 @@
 /*
- * This plugin provides a way for jsenv to know when js execution is done
+ * This plugin provides a way for jsenv to supervisor js execution:
+ * - Know how many js are executed, when they are done, collect errors, etc...
  */
 
 import { createRequire } from "node:module";
@@ -7,12 +8,10 @@ import { fileURLToPath } from "node:url";
 import { getOriginalPosition } from "@jsenv/sourcemap";
 import { stringifyUrlSite } from "@jsenv/urls";
 
-import { injectSupervisorIntoHTML } from "./html_supervisor_injection.js";
-
-export const supervisorFileUrl = new URL(
-  "./client/supervisor.js",
-  import.meta.url,
-).href;
+import {
+  supervisorFileUrl,
+  injectSupervisorIntoHTML,
+} from "./html_supervisor.js";
 
 export const jsenvPluginSupervisor = ({
   logs = false,

package/src/main.js

@@ -0,0 +1,5 @@
+export { jsenvPluginSupervisor } from "./jsenv_plugin_supervisor.js";
+export {
+  supervisorFileUrl,
+  injectSupervisorIntoHTML,
+} from "./html_supervisor.js";