chore: cleanup esm

As far as I could tell, the original reasons are no longer reproducible.

Author: lishaduck

packages/wxt/package.json

@@ -62,6 +62,14 @@
     "vite-node": "^2.1.4 || ^3.1.2",
     "web-ext-run": "^0.2.4"
   },
+  "peerDependencies": {
+    "eslint": "^8.57.0 || ^9.0.0"
+  },
+  "peerDependenciesMeta": {
+    "eslint": {
+      "optional": true
+    }
+  },
   "devDependencies": {
     "@aklinker1/check": "^2.1.0",
     "@faker-js/faker": "^10.0.0",
@@ -70,6 +78,7 @@
     "@types/node": "^20.17.6",
     "@types/normalize-path": "^3.0.2",
     "@types/prompts": "^2.4.9",
+    "eslint": "^9.38.0",
     "extract-zip": "^2.0.1",
     "happy-dom": "^18.0.1",
     "lodash.merge": "^4.6.2",
@@ -80,7 +89,6 @@
     "vitest": "^3.2.4",
     "vitest-plugin-random-seed": "^1.1.1"
   },
-  "peerDependenciesMeta": {},
   "repository": {
     "type": "git",
     "url": "git+https://github.com/wxt-dev/wxt.git"

packages/wxt/src/core/builders/vite/plugins/removeEntrypointMainFunction.ts

@@ -1,5 +1,5 @@
 import { ResolvedConfig } from '../../../../types';
-import * as vite from 'vite';
+import type * as vite from 'vite';
 import { normalizePath } from '../../../utils/paths';
 import { removeMainFunctionCode } from '../../../utils/transform';
 import { resolve } from 'node:path';

packages/wxt/src/core/resolve-config.ts

@@ -536,17 +536,10 @@ async function getUnimportEslintOptions(
  * Returns the path to `node_modules/wxt`.
  */
 function resolveWxtModuleDir() {
-  // TODO: Drop the __filename expression once we're fully running in ESM
-  // (see https://github.com/wxt-dev/wxt/issues/277)
-  const importer =
-    typeof __filename === 'string'
-      ? pathToFileURL(__filename).href
-      : import.meta.url;
-
   // TODO: Switch to import.meta.resolve() once the parent argument is unflagged
   // (e.g. --experimental-import-meta-resolve) and all Node.js versions we support
   // have it.
-  const url = esmResolve('wxt', importer);
+  const url = esmResolve('wxt', import.meta.url);
 
   // esmResolve() returns the "wxt/dist/index.mjs" file, not the package's root
   // directory, which we want to return from this function.

packages/wxt/src/core/runners/index.ts

@@ -1,15 +1,15 @@
-import { ExtensionRunner } from '../../types';
-import { createWslRunner } from './wsl';
-import { createWebExtRunner } from './web-ext';
-import { createSafariRunner } from './safari';
-import { createManualRunner } from './manual';
+import type { ExtensionRunner } from '../../types';
 import { isWsl } from '../utils/wsl';
 import { wxt } from '../wxt';
+import { createManualRunner } from './manual';
+import { createSafariRunner } from './safari';
+import { createWebExtRunner } from './web-ext';
+import { createWslRunner } from './wsl';
 
 export async function createExtensionRunner(): Promise<ExtensionRunner> {
   if (wxt.config.browser === 'safari') return createSafariRunner();
 
-  if (await isWsl()) return createWslRunner();
+  if (isWsl()) return createWslRunner();
   if (wxt.config.runnerConfig.config?.disabled) return createManualRunner();
 
   return createWebExtRunner();

packages/wxt/src/core/utils/eslint.ts

@@ -1,9 +1,6 @@
 export async function getEslintVersion(): Promise<string[]> {
   try {
-    const require = (await import('node:module')).default.createRequire(
-      import.meta.url,
-    );
-    const { ESLint } = require('eslint');
+    const { ESLint } = await import('eslint');
     return ESLint.version?.split('.') ?? [];
   } catch {
     // Return an empty version when there's an error importing ESLint

packages/wxt/src/core/utils/wsl.ts

@@ -1,7 +1,9 @@
+// TODO: Someone smarter than me should just mock this module instead.
+import isWsl_ from 'is-wsl';
+
 /**
  * Returns true when running on WSL or WSL2.
  */
-export async function isWsl(): Promise<boolean> {
-  const { default: isWsl } = await import('is-wsl'); // ESM only, requires dynamic import
-  return isWsl;
+export function isWsl(): boolean {
+  return isWsl_;
 }

packages/wxt/src/modules.ts

@@ -171,7 +171,7 @@ export function addImportPreset(
     if (!wxt.config.imports) return;
 
     wxt.config.imports.presets ??= [];
-    // De-dupelicate built-in named presets
+    // De-duplicate built-in named presets
     if (wxt.config.imports.presets.includes(preset)) return;
 
     wxt.config.imports.presets.push(preset);