wxt 0.19.1-alpha3 → 0.19.2

package/dist/core/utils/transform.d.ts

@@ -1,7 +1,8 @@
 /**
  * Removes any code used at runtime related to an entrypoint's main function.
  * 1. Removes or clears out `main` function from returned object
- * 2. Removes unused imports
+ * 2. Removes any unused functions/variables outside the definition that aren't being called/used
+ * 3. Removes unused imports
  * 3. Removes value-less, side-effect only imports (like `import "./styles.css"` or `import "webextension-polyfill"`)
  */
 export declare function removeMainFunctionCode(code: string): {

package/dist/core/utils/transform.mjs

@@ -2,7 +2,15 @@ import { parseModule } from "magicast";
 export function removeMainFunctionCode(code) {
   const mod = parseModule(code);
   emptyMainFunction(mod);
-  removeUnusedImports(mod);
+  let removedCount = 0;
+  let depth = 0;
+  const maxDepth = 10;
+  do {
+    removedCount = 0;
+    removedCount += removeUnusedTopLevelVariables(mod);
+    removedCount += removeUnusedTopLevelFunctions(mod);
+    removedCount += removeUnusedImports(mod);
+  } while (removedCount > 0 && depth++ <= maxDepth);
   removeSideEffectImports(mod);
   return mod.generate();
 }
@@ -17,26 +25,77 @@ function emptyMainFunction(mod) {
     }
   }
 }
+function removeUnusedTopLevelVariables(mod) {
+  const simpleAst = getSimpleAstJson(mod.$ast);
+  const usedMap = findUsedIdentifiers(simpleAst);
+  let deletedCount = 0;
+  const ast = mod.$ast;
+  for (let i = ast.body.length - 1; i >= 0; i--) {
+    if (ast.body[i].type === "VariableDeclaration") {
+      for (let j = ast.body[i].declarations.length - 1; j >= 0; j--) {
+        if (!usedMap.get(ast.body[i].declarations[j].id.name)) {
+          ast.body[i].declarations.splice(j, 1);
+          deletedCount++;
+        }
+      }
+      if (ast.body[i].declarations.length === 0) {
+        ast.body.splice(i, 1);
+      }
+    }
+  }
+  return deletedCount;
+}
+function removeUnusedTopLevelFunctions(mod) {
+  const simpleAst = getSimpleAstJson(mod.$ast);
+  const usedMap = findUsedIdentifiers(simpleAst);
+  let deletedCount = 0;
+  const ast = mod.$ast;
+  for (let i = ast.body.length - 1; i >= 0; i--) {
+    if (ast.body[i].type === "FunctionDeclaration" && !usedMap.get(ast.body[i].id.name)) {
+      ast.body.splice(i, 1);
+      deletedCount++;
+    }
+  }
+  return deletedCount;
+}
 function removeUnusedImports(mod) {
+  const simpleAst = getSimpleAstJson(mod.$ast);
+  const usedMap = findUsedIdentifiers(simpleAst);
   const importSymbols = Object.keys(mod.imports);
-  const usedMap = new Map(importSymbols.map((sym) => [sym, false]));
-  const queue = [getSimpleAstJson(mod.$ast)];
+  let deletedCount = 0;
+  importSymbols.forEach((name) => {
+    if (usedMap.get(name)) return;
+    delete mod.imports[name];
+    deletedCount++;
+  });
+  return deletedCount;
+}
+function findUsedIdentifiers(simpleAst) {
+  const usedMap = /* @__PURE__ */ new Map();
+  const queue = [simpleAst];
   for (const item of queue) {
     if (!item) {
-      continue;
     } else if (Array.isArray(item)) {
       queue.push(...item);
     } else if (item.type === "ImportDeclaration") {
       continue;
     } else if (item.type === "Identifier") {
-      if (usedMap.has(item.name)) usedMap.set(item.name, true);
+      usedMap.set(item.name, true);
     } else if (typeof item === "object") {
-      queue.push(Object.values(item));
+      const filterFns = {
+        // Ignore the function declaration's name
+        FunctionDeclaration: ([key]) => key !== "id",
+        // Ignore object property names
+        ObjectProperty: ([key]) => key !== "key",
+        // Ignore variable declaration's name
+        VariableDeclarator: ([key]) => key !== "id"
+      };
+      queue.push(
+        Object.entries(item).filter(filterFns[item.type] ?? (() => true)).map(([_, value]) => value)
+      );
     }
   }
-  for (const [name, used] of usedMap.entries()) {
-    if (!used) delete mod.imports[name];
-  }
+  return usedMap;
 }
 function deleteImportAst(mod, shouldDelete) {
   const importIndexesToDelete = [];

package/dist/core/wxt.mjs

@@ -5,6 +5,7 @@ import { createViteBuilder } from "./builders/vite/index.mjs";
 import { builtinModules } from "../builtin-modules/index.mjs";
 export let wxt;
 export async function registerWxt(command, inlineConfig = {}, getServer) {
+  process.env.NODE_ENV ??= command === "serve" ? "development" : "production";
   const hooks = createHooks();
   const config = await resolveConfig(inlineConfig, command);
   const server = await getServer?.(config);

package/dist/version.mjs

@@ -1 +1 @@
-export const version = "0.19.1-alpha3";
+export const version = "0.19.2";

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "wxt",
   "type": "module",
-  "version": "0.19.1-alpha3",
+  "version": "0.19.2",
   "description": "Next gen framework for developing web extensions",
   "repository": {
     "type": "git",
@@ -105,20 +105,20 @@
     "picocolors": "^1.0.1",
     "prompts": "^2.4.2",
     "publish-browser-extension": "^2.1.3",
-    "unimport": "^3.9.0",
-    "vite": "^5.3.4",
-    "vite-node": "^2.0.0",
+    "unimport": "^3.9.1",
+    "vite": "^5.3.5",
+    "vite-node": "^2.0.4",
     "web-ext-run": "^0.2.1",
     "webextension-polyfill": "^0.12.0"
   },
   "devDependencies": {
     "@aklinker1/check": "^1.3.1",
     "@faker-js/faker": "^8.4.1",
-    "@types/chrome": "^0.0.268",
+    "@types/chrome": "^0.0.269",
     "@types/fs-extra": "^11.0.4",
     "@types/lodash.merge": "^4.6.9",
     "@types/natural-compare": "^1.4.3",
-    "@types/node": "^20.14.11",
+    "@types/node": "^20.14.12",
     "@types/normalize-path": "^3.0.2",
     "@types/prompts": "^2.4.9",
     "execa": "^9.3.0",
@@ -128,9 +128,9 @@
     "p-map": "^7.0.2",
     "publint": "^0.2.9",
     "tsx": "4.15.7",
-    "typescript": "^5.5.3",
+    "typescript": "^5.5.4",
     "unbuild": "^2.0.0",
-    "vitest": "^2.0.3",
+    "vitest": "^2.0.4",
     "vitest-plugin-random-seed": "^1.1.0"
   },
   "peerDependencies": {
@@ -144,7 +144,7 @@
   "scripts": {
     "wxt": "tsx src/cli/index.ts",
     "build": "buildc -- unbuild",
-    "check": "buildc --deps-only -- run-s -c check:*",
+    "check": "pnpm build && run-s -c check:*",
     "check:default": "check",
     "check:tsc-virtual": "tsc --noEmit -p src/virtual",
     "test": "buildc --deps-only -- vitest",