fix(plugin): don't import node:fs in browser

Author: wmertens

packages/qwik/src/optimizer/src/plugins/plugin.ts

@@ -136,15 +136,20 @@ export function createQwikPlugin(optimizerOptions: OptimizerOptions = {}) {
     if (!internalOptimizer) {
       internalOptimizer = await createOptimizer(optimizerOptions);
       lazyNormalizePath = makeNormalizePath(internalOptimizer.sys);
-      try {
-        // only try once, don't spam the console
-        if (maybeFs === undefined) {
-          maybeFs = await internalOptimizer.sys.dynamicImport('node:fs');
+      if (
+        internalOptimizer.sys.env !== 'browsermain' &&
+        internalOptimizer.sys.env !== 'webworker'
+      ) {
+        try {
+          // only try once, don't spam the console
+          if (maybeFs === undefined) {
+            maybeFs = await internalOptimizer.sys.dynamicImport('node:fs');
+          }
+        } catch {
+          // eslint-disable-next-line no-console
+          console.log('node:fs not available, disabling automatic manifest reading');
+          maybeFs = null;
         }
-      } catch {
-        // eslint-disable-next-line no-console
-        console.log('node:fs not available, disabling automatic manifest reading');
-        maybeFs = null;
       }
     }
   };

packages/qwik/src/optimizer/src/plugins/utils.ts

@@ -1,6 +1,9 @@
 import type { OptimizerSystem } from '../types';
 
 export async function findDepPkgJsonPath(sys: OptimizerSystem, dep: string, parent: string) {
+  if (sys.env === 'browsermain' || sys.env === 'webworker') {
+    return undefined;
+  }
   const fs: typeof import('fs') = await sys.dynamicImport('node:fs');
   let root = parent;
   while (root) {