chore: cleanup esm

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

Author: lishaduck

packages/wxt/package.json

@@ -63,8 +63,14 @@
     "web-ext-run": "^0.2.4"
   },
   "peerDependencies": {
+    "eslint": "^8.57.0 || ^9.0.0",
     "vite": "^5.4.19 || ^6.3.4 || ^7.0.0"
   },
+  "peerDependenciesMeta": {
+    "eslint": {
+      "optional": true
+    }
+  },
   "devDependencies": {
     "@aklinker1/check": "^2.1.0",
     "@faker-js/faker": "^10.0.0",
@@ -73,6 +79,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",
@@ -83,7 +90,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/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 { isWsl } from '../utils/wsl';
+import isWsl from 'is-wsl';
+import type { ExtensionRunner } from '../../types';
 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