@jsenv/plugin-as-js-classic 1.1.3 → 1.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/plugin-as-js-classic",
-  "version": "1.1.3",
+  "version": "1.2.0",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -25,7 +25,7 @@
   },
   "dependencies": {
     "@jsenv/urls": "2.0.0",
-    "@jsenv/js-module-fallback": "1.0.4",
-    "@jsenv/plugin-bundling": "2.1.11"
+    "@jsenv/js-module-fallback": "1.1.0",
+    "@jsenv/plugin-bundling": "2.2.0"
   }
 }

package/src/jsenv_plugin_as_js_classic.js

@@ -1,18 +1,14 @@
 import { urlToFilename } from "@jsenv/urls";
-import { convertJsModuleToJsClassic } from "@jsenv/js-module-fallback";
+import {
+  convertJsModuleToJsClassic,
+  systemJsClientFileUrlDefault,
+} from "@jsenv/js-module-fallback";
 import { bundleJsModules } from "@jsenv/plugin-bundling";
 
 import { createUrlGraphLoader } from "./url_graph_loader.js";
 
-export const jsenvPluginAsJsClassic = ({
-  systemJsInjection = true,
-  systemJsClientFileUrl,
-} = {}) => {
+export const jsenvPluginAsJsClassic = () => {
   const markAsJsClassicProxy = (reference) => {
-    if (reference.searchParams.has("dynamic_import")) {
-    } else {
-      reference.isEntryPoint = true;
-    }
     reference.expectedType = "js_classic";
     reference.filename = generateJsClassicFilename(reference.url);
   };
@@ -75,14 +71,14 @@ export const jsenvPluginAsJsClassic = ({
       } else if (context.build) {
         jsModuleBundledUrlInfo.sourceUrls.forEach((sourceUrl) => {
           const sourceUrlInfo = context.urlGraph.getUrlInfo(sourceUrl);
-          if (sourceUrlInfo && sourceUrlInfo.dependents.size === 0) {
+          if (sourceUrlInfo && !context.urlGraph.hasDependent(sourceUrlInfo)) {
             context.urlGraph.deleteUrlInfo(sourceUrl);
           }
         });
       }
 
       let outputFormat;
-      if (urlInfo.isEntryPoint && !jsModuleUrlInfo.data.usesImport) {
+      if (urlInfo.isEntryPoint && !jsModuleBundledUrlInfo.data.usesImport) {
         // if it's an entry point without dependency (it does not use import)
         // then we can use UMD
         outputFormat = "umd";
@@ -91,14 +87,17 @@ export const jsenvPluginAsJsClassic = ({
         // by an other file (for entry points)
         // or to be able to import when it uses import
         outputFormat = "system";
+        urlInfo.type = "js_classic";
+        context.referenceUtils.foundSideEffectFile({
+          sideEffectFileUrl: systemJsClientFileUrlDefault,
+          expectedType: "js_classic",
+          line: 0,
+          column: 0,
+        });
       }
-      urlInfo.data.jsClassicFormat = outputFormat;
       const { content, sourcemap } = await convertJsModuleToJsClassic({
         rootDirectoryUrl: context.rootDirectoryUrl,
-        systemJsInjection,
-        systemJsClientFileUrl,
         input: jsModuleBundledUrlInfo.content,
-        inputIsEntryPoint: urlInfo.isEntryPoint,
         inputSourcemap: jsModuleBundledUrlInfo.sourcemap,
         inputUrl: jsModuleBundledUrlInfo.url,
         outputUrl: jsModuleBundledUrlInfo.generatedUrl,