fix

Author: marvinhagemeister

packages/fresh/src/runtime/client/preact_hooks_client.ts

@@ -64,34 +64,29 @@ function WrappedHead(
     }
 
     applyProps(props, matched);
-  }, []);
+  }, [originalType, props, key]);
 
   if (enabled) {
     return null;
+  } else {
+    const inner = h(originalType, props);
+    PATCHED.add(inner);
+    return inner;
   }
-
-  const inner = h(originalType, props);
-  PATCHED.add(inner);
-  return inner;
 }
 
 const oldVNodeHook = options.vnode;
 options.vnode = (vnode) => {
   assetHashingHook(vnode, BUILD_ID);
 
-  if (typeof vnode.type === "string") {
+  const originalType = vnode.type;
+  if (typeof originalType === "string") {
     if (CLIENT_NAV_ATTR in vnode.props) {
       const value = vnode.props[CLIENT_NAV_ATTR];
       if (typeof value === "boolean") {
         vnode.props[CLIENT_NAV_ATTR] = String(value);
       }
-    }
-  }
-
-  const originalType = vnode.type;
-
-  if (typeof originalType === "string") {
-    if (!PATCHED.has(vnode)) {
+    } else if (!PATCHED.has(vnode)) {
       switch (originalType) {
         case "title":
         case "meta":
@@ -100,14 +95,19 @@ options.vnode = (vnode) => {
         case "style":
         case "base":
         case "noscript":
-        case "template":
-          vnode = h(WrappedHead, {
+        case "template": {
+          // deno-lint-ignore no-explicit-any
+          const v = vnode as VNode<any>;
+          const props = vnode.props;
+          const key = vnode.key;
+          v.type = WrappedHead;
+          v.props = {
             originalType,
-            props: vnode.props,
-            key: vnode.key,
-            // deno-lint-ignore no-explicit-any
-          }) as any;
+            props,
+            key,
+          };
           break;
+        }
       }
     }
   }

packages/fresh/tests/head_test.tsx

@@ -4,6 +4,7 @@ import {
   buildProd,
   parseHtml,
   waitFor,
+  waitForText,
   withBrowserApp,
 } from "./test_utils.tsx";
 import { expect } from "@std/expect";
@@ -171,12 +172,7 @@ Deno.test({
       await page.locator(".ready").wait();
       await page.locator("button").click();
 
-      await waitFor(async () => {
-        const title = await page.evaluate(() => document.title);
-        // deno-lint-ignore no-console
-        console.log({ title });
-        return title === "Count: 1";
-      });
+      await waitForText(page, "title", "Count: 1");
     });
   },
   sanitizeOps: false,