feat(cli): Use ESM worker entry file only when import config is pro…

…vided

packages/cli/src/commands/generate/env.ts

@@ -46,6 +46,9 @@ async function importEnvNode(cwd: string): Promise<Environment[]> {
 
   const envs: Environment[] = [];
 
+  // Global `require` is only available in CommonJS environment. We don't use
+  // `createRequire` here because it will create a new instance of `require`,
+  // which doesn't share the same cache with the global `require`.
   if (BUILD_FORMAT === "cjs") {
     // eslint-disable-next-line @typescript-eslint/no-var-requires
     envs.push(require(envPath));

packages/cli/src/commands/generate/worker.ts

@@ -96,8 +96,18 @@ async function runWithLoaders(options: WorkerOptions) {
         ...options.config.loaders.flatMap((loader) => ["--loader", loader]),
         // ESM import
         ...options.config.import.flatMap((imp) => ["--import", imp]),
-        // Entry file. Always use ESM entry file.
-        join(fileURLToPath(import.meta.url), "../worker-bin.node.mjs")
+        // Entry file. Always use ESM entry file when `import` is provided
+        // because of the following error, which seems to happen only when using
+        // `@swc-node/register/esm-register`.
+        //
+        // ReferenceError: require is not defined in ES module scope, you can use import instead
+        //
+        // Otherwise, use CJS entry file because it supports both `require` and
+        // `import`.
+        join(
+          fileURLToPath(import.meta.url),
+          "../worker-bin.node." + (options.config.import.length ? "mjs" : "cjs")
+        )
       ],
       {
         stdio: ["pipe", "inherit", "inherit"],