refactor: move opts into scope, semantic naming for conditionals

Author: thejackshelton

packages/qwik-router/src/buildtime/vite/plugin.ts

@@ -215,7 +215,9 @@ function qwikRouterPlugin(userOpts?: QwikRouterVitePluginOptions): any {
     },
 
     async transform(code, id) {
-      if (id.startsWith('\0')) {
+      const isVirtualId = id.startsWith('\0');
+
+      if (isVirtualId) {
         return;
       }
       const ext = extname(id).toLowerCase();

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

@@ -410,27 +410,29 @@ export function qwikVite(qwikViteOpts: QwikVitePluginOptions = {}): any {
 
     resolveId(id, importer, resolveIdOpts) {
       const shouldResolveFile = fileFilter(id, 'resolveId');
+      const isDevClientModule = isClientDevOnly && id === VITE_CLIENT_MODULE;
 
       if (isVirtualId(id) || !shouldResolveFile) {
         return null;
       }
-      if (isClientDevOnly && id === VITE_CLIENT_MODULE) {
+      if (isDevClientModule) {
         return id;
       }
       return qwikPlugin.resolveId(this, id, importer, resolveIdOpts);
     },
 
     load(id, loadOpts) {
       const shouldLoadFile = fileFilter(id, 'load');
+      const isDevClientModule = isClientDevOnly && id === VITE_CLIENT_MODULE;
 
       if (isVirtualId(id) || !shouldLoadFile) {
         return null;
       }
 
       id = qwikPlugin.normalizePath(id);
-      const opts = qwikPlugin.getOptions();
 
-      if (isClientDevOnly && id === VITE_CLIENT_MODULE) {
+      if (isDevClientModule) {
+        const opts = qwikPlugin.getOptions();
         return getViteDevModule(opts);
       }
       if (viteCommand === 'serve' && id.endsWith(QWIK_CLIENT_MANIFEST_ID)) {