restore JSX renderAsset, fix beta.15 crash with NoHydration wrapper

The raw-HTML-string workaround existed for a babel-plugin-jsx-dom-expressions
spread crash that's fixed in 0.50.0-next.14. The remaining blocker was
beta.15's owner-based hydration keys: useAssets thunks run at shell-injection
time with no owner, so ssrElement's getNextContextId call threw and killed the
server process. Wrapping the asset JSX in <NoHydration> provides an owner with
hydration-key generation disabled — correct for head tags, which never hydrate.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: brenelz

packages/start/src/server/assets/render.tsx

@@ -1,63 +1,39 @@
 // @refresh skip
-//
-// NOTE: This file intentionally avoids JSX to work around a crash in
-// babel-plugin-jsx-dom-expressions@0.41.0-next.9 when spread attributes are
-// used on native HTML elements during SSR transformation.  The previous
-// implementation used JSX (<style {...props.attrs}> etc.) which triggered the
-// bug.  Instead we build the HTML strings directly — these assets are only
-// ever rendered during SSR so reactive JSX is unnecessary.
-
+import { NoHydration } from "@solidjs/web";
 import type { JSX } from "@solidjs/web";
 
-/** Escape HTML special characters for safe attribute / text insertion. */
-function esc(s: string): string {
-  return s.replace(/&/g, "&amp;").replace(/"/g, "&quot;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
-}
-
-function renderAttrs(obj: Record<string, any>): string {
-  let out = "";
-  for (const [k, v] of Object.entries(obj)) {
-    if (v == null || v === false || v === undefined) continue;
-    if (v === true) {
-      out += ` ${esc(k)}`;
-    } else {
-      out += ` ${esc(k)}="${esc(String(v))}"`;
-    }
-  }
-  return out;
-}
+const assetMap = {
+  style: (props: { attrs: JSX.StyleHTMLAttributes<HTMLStyleElement>; children?: JSX.Element }) => (
+    <style {...props.attrs}>{props.children}</style>
+  ),
+  link: (props: { attrs: JSX.LinkHTMLAttributes<HTMLLinkElement> }) => <link {...props.attrs} />,
+  script: (props: {
+    attrs: JSX.ScriptHTMLAttributes<HTMLScriptElement>;
+    key: string | undefined;
+  }) => {
+    return props.attrs.src ? (
+      <script {...props.attrs} id={props.key}>
+        {" "}
+      </script>
+    ) : null;
+  },
+  noscript: (props: { attrs: JSX.HTMLAttributes<HTMLElement>; children: JSX.Element }) => (
+    <noscript {...props.attrs}>{props.children}</noscript>
+  ),
+};
 
 export function renderAsset(asset: Asset, nonce?: string) {
   let {
     tag,
-    attrs: { key, ...rest } = { key: undefined },
+    attrs: { key, ...attrs } = { key: undefined },
     children,
   } = asset as any;
-
-  const allAttrs = { ...rest, nonce } as Record<string, any>;
-
-  let html: string;
-  switch (tag as string) {
-    case "style":
-      html = `<style${renderAttrs(allAttrs)}>${children ?? ""}</style>`;
-      break;
-    case "link":
-      html = `<link${renderAttrs(allAttrs)} />`;
-      break;
-    case "script":
-      if (!allAttrs.src) return { t: "" };
-      html = `<script${renderAttrs(allAttrs)}${key ? ` id="${esc(key)}"` : ""}> </script>`;
-      break;
-    case "noscript":
-      html = `<noscript${renderAttrs(allAttrs)}>${children ?? ""}</noscript>`;
-      break;
-    default:
-      html = "";
-  }
-
-  // Return as an SSR node object ({ t: string }) so that @solidjs/web's
-  // escape() passes it through unmodified and resolveSSRNode() uses it directly.
-  return { t: html };
+  // Asset thunks run at shell-injection time with no reactive owner; NoHydration
+  // provides one (with hydration-key generation disabled) so ssrElement doesn't
+  // throw. Head asset tags never hydrate, so they shouldn't carry _hk anyway.
+  return (
+    <NoHydration>{(assetMap as any)[tag]({ attrs: { ...attrs, nonce }, key, children })}</NoHydration>
+  );
 }
 
 export type Asset =