perf(beasties): move checks after early return

danielroe · danielroe · commit e24b57b6c1dc · 2025-04-07T11:08:28.000-07:00
diff --git a/packages/beasties/src/index.ts b/packages/beasties/src/index.ts
@@ -231,13 +231,6 @@ export default class Beasties {
    */
   async embedLinkedStylesheet(link: ChildNode, document: HTMLDocument) {
     const href = link.getAttribute('href')
-    let media: string | undefined = link.getAttribute('media')
-
-    if (media && !validateMediaQuery(media)) {
-      media = undefined
-    }
-
-    const preloadMode = this.options.preload
 
     // skip filtered resources, or network resources if no filter is provided
     if (!href?.endsWith('.css')) {
@@ -262,6 +255,12 @@ export default class Beasties {
       return
     }
 
+    let media: string | undefined = link.getAttribute('media')
+
+    if (media && !validateMediaQuery(media)) {
+      media = undefined
+    }
+    const preloadMode = this.options.preload
     // CSS loader is only injected for the first sheet, then this becomes an empty string
     let cssLoaderPreamble
       = 'function $loadcss(u,m,l){(l=document.createElement(\'link\')).rel=\'stylesheet\';l.href=u;document.head.appendChild(l)}'
