From 2548ef110fd3ca23859c923d2c61393a732a7239 Mon Sep 17 00:00:00 2001
From: Peyton-Spencer <peyton@omniaura.ai>
Date: Tue, 24 Feb 2026 14:30:15 -0500
Subject: [PATCH 1/2] Skip plugin entirely in production builds via apply:
 "serve"

The plugin was consuming 90% of build time during `vite build` because
Rolldown called into the transform hook for every module, even though
it returned null immediately. Using Vite's `apply: "serve"` option
removes the plugin entirely from the build pipeline in production,
eliminating all hook call overhead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/vite.ts               |  8 ++------
 tests/vite-plugin.test.ts | 18 ++++++++----------
 2 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/vite.ts b/src/vite.ts
index ce83372..77d9eb6 100644
--- a/src/vite.ts
+++ b/src/vite.ts
@@ -184,15 +184,14 @@ export default function solidGrab(
   } = options;
 
   let projectRoot = "";
-  let isDev = false;
 
   return {
     name: "solid-grab",
     enforce: "pre", // Run before vite-plugin-solid
+    apply: "serve", // Only active during dev — completely skipped in production builds
 
     configResolved(config: ResolvedConfig) {
       projectRoot = config.root;
-      isDev = config.command === "serve" || config.mode === "development";
     },
 
     // Virtual module that imports the runtime — resolved by Vite's pipeline
@@ -207,9 +206,6 @@ export default function solidGrab(
     },
 
     transform(code, id) {
-      // Only transform in dev mode
-      if (!isDev) return null;
-
       // Only transform JSX/TSX files in the user's project
       if (!/\.[jt]sx$/.test(id)) return null;
       if (id.includes("node_modules")) return null;
@@ -245,7 +241,7 @@ export default function solidGrab(
 
     // Inject a <script src> that Vite's dev server will resolve
     transformIndexHtml() {
-      if (!autoImport || !isDev) return;
+      if (!autoImport) return;
       return [
         {
           tag: "script",
diff --git a/tests/vite-plugin.test.ts b/tests/vite-plugin.test.ts
index a42b757..cc7adb3 100644
--- a/tests/vite-plugin.test.ts
+++ b/tests/vite-plugin.test.ts
@@ -73,11 +73,10 @@ describe("transform", () => {
     expect(result).toBeNull();
   });
 
-  test("skips in production mode", () => {
-    const plugin = createPlugin({}, "production");
-    const code = `function App() {\n  return <div>hello</div>;\n}`;
-    const result = (plugin as any).transform(code, "/project/src/App.tsx");
-    expect(result).toBeNull();
+  test("skips in production mode (apply: serve)", () => {
+    const plugin = solidGrab();
+    // Vite skips the entire plugin during `vite build` because apply is "serve"
+    expect(plugin.apply).toBe("serve");
   });
 
   test("does not inject into TypeScript generics", () => {
@@ -250,11 +249,10 @@ describe("transformIndexHtml", () => {
     expect(result[0].injectTo).toBe("head");
   });
 
-  test("returns undefined in production mode", () => {
-    const plugin = createPlugin({}, "production");
-    const result = (plugin as any).transformIndexHtml();
-
-    expect(result).toBeUndefined();
+  test("skips in production mode (apply: serve)", () => {
+    const plugin = solidGrab();
+    // Vite skips the entire plugin during `vite build` because apply is "serve"
+    expect(plugin.apply).toBe("serve");
   });
 
   test("returns undefined when autoImport is false", () => {

From ba866860446b28f43a5dbc765b4c657e4a21df8f Mon Sep 17 00:00:00 2001
From: Peyton-Spencer <peyton@omniaura.ai>
Date: Tue, 24 Feb 2026 14:50:42 -0500
Subject: [PATCH 2/2] Clean up test file: consolidate metadata checks, simplify
 helper

- Move apply: "serve" assertion into plugin metadata describe block
- Remove duplicate one-line apply tests from transform/transformIndexHtml
- Remove vestigial mode parameter from createPlugin helper
- Rename "returns tag descriptors in dev mode" since isDev was removed

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 tests/vite-plugin.test.ts | 28 ++++++++--------------------
 1 file changed, 8 insertions(+), 20 deletions(-)

diff --git a/tests/vite-plugin.test.ts b/tests/vite-plugin.test.ts
index cc7adb3..dedd59c 100644
--- a/tests/vite-plugin.test.ts
+++ b/tests/vite-plugin.test.ts
@@ -4,17 +4,12 @@ import solidGrab from "../src/vite.js";
 
 /** Helper: create the plugin and simulate Vite's configResolved hook */
 function createPlugin(
-  options: Parameters<typeof solidGrab>[0] = {},
-  mode: "development" | "production" = "development"
+  options: Parameters<typeof solidGrab>[0] = {}
 ): Plugin {
   const plugin = solidGrab(options);
 
   // Simulate Vite calling configResolved
-  const fakeConfig = {
-    root: "/project",
-    command: mode === "development" ? "serve" : "build",
-    mode,
-  } as ResolvedConfig;
+  const fakeConfig = { root: "/project" } as ResolvedConfig;
 
   (plugin as any).configResolved(fakeConfig);
   return plugin;
@@ -30,6 +25,11 @@ describe("plugin metadata", () => {
     const plugin = solidGrab();
     expect(plugin.enforce).toBe("pre");
   });
+
+  test("only applies during dev serve", () => {
+    const plugin = solidGrab();
+    expect(plugin.apply).toBe("serve");
+  });
 });
 
 describe("transform", () => {
@@ -73,12 +73,6 @@ describe("transform", () => {
     expect(result).toBeNull();
   });
 
-  test("skips in production mode (apply: serve)", () => {
-    const plugin = solidGrab();
-    // Vite skips the entire plugin during `vite build` because apply is "serve"
-    expect(plugin.apply).toBe("serve");
-  });
-
   test("does not inject into TypeScript generics", () => {
     const plugin = createPlugin();
     const code = [
@@ -237,7 +231,7 @@ describe("virtual module (key config)", () => {
 });
 
 describe("transformIndexHtml", () => {
-  test("returns tag descriptors in dev mode", () => {
+  test("returns tag descriptors", () => {
     const plugin = createPlugin();
     const result = (plugin as any).transformIndexHtml();
 
@@ -249,12 +243,6 @@ describe("transformIndexHtml", () => {
     expect(result[0].injectTo).toBe("head");
   });
 
-  test("skips in production mode (apply: serve)", () => {
-    const plugin = solidGrab();
-    // Vite skips the entire plugin during `vite build` because apply is "serve"
-    expect(plugin.apply).toBe("serve");
-  });
-
   test("returns undefined when autoImport is false", () => {
     const plugin = createPlugin({ autoImport: false });
     const result = (plugin as any).transformIndexHtml();