perf(core): read client manifest only if and when needed

Author: wmertens

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

@@ -285,27 +285,6 @@ export function createQwikPlugin(optimizerOptions: OptimizerOptions = {}) {
     if (updatedOpts.manifestInput) {
       opts.manifestInput = getValidManifest(updatedOpts.manifestInput) || null;
     }
-    if (
-      !opts.manifestInput &&
-      opts.target === 'ssr' &&
-      opts.buildMode === 'production' &&
-      maybeFs
-    ) {
-      let clientManifestPath = resolvePath(opts.clientOutDir, Q_MANIFEST_FILENAME);
-      if (!(await maybeFs.promises.stat(clientManifestPath).catch(() => false))) {
-        clientManifestPath = resolvePath(opts.rootDir, CLIENT_OUT_DIR, Q_MANIFEST_FILENAME);
-      }
-      try {
-        const clientManifestStr = await maybeFs.promises.readFile(clientManifestPath, 'utf-8');
-        opts.manifestInput = getValidManifest(JSON.parse(clientManifestStr)) || null;
-        // eslint-disable-next-line no-console
-        console.info('Read client manifest from', clientManifestPath);
-      } catch (e) {
-        console.warn(
-          `could not read Qwik client manifest ${clientManifestPath}, ignoring. Make sure you provide it to the SSR renderer. (${e})`
-        );
-      }
-    }
 
     if (typeof updatedOpts.transformedModuleOutput === 'function') {
       opts.transformedModuleOutput = updatedOpts.transformedModuleOutput;
@@ -944,6 +923,29 @@ export const isDev = ${JSON.stringify(isDev)};
   }
 
   async function getQwikServerManifestModule(isServer: boolean) {
+    if (
+      !opts.manifestInput &&
+      opts.target === 'ssr' &&
+      opts.buildMode === 'production' &&
+      maybeFs
+    ) {
+      const path = getPath();
+      let clientManifestPath = path.resolve(opts.clientOutDir, Q_MANIFEST_FILENAME);
+      if (!(await maybeFs.promises.stat(clientManifestPath).catch(() => false))) {
+        clientManifestPath = path.resolve(opts.rootDir, CLIENT_OUT_DIR, Q_MANIFEST_FILENAME);
+      }
+      try {
+        const clientManifestStr = await maybeFs.promises.readFile(clientManifestPath, 'utf-8');
+        opts.manifestInput = getValidManifest(JSON.parse(clientManifestStr)) || null;
+        // eslint-disable-next-line no-console
+        console.info('Read client manifest from', clientManifestPath);
+      } catch (e) {
+        console.warn(
+          `could not read Qwik client manifest ${clientManifestPath}, ignoring. Make sure you provide it to the SSR renderer. (${e})`
+        );
+      }
+    }
+
     const manifest = isServer ? opts.manifestInput : null;
     let serverManifest: ServerQwikManifest | null = null;
     if (manifest?.manifestHash) {