chore: skip script rendering after hydration (#422)

Should be what we need for preact support

.gitignore

@@ -11,6 +11,5 @@ yarn-error.log
 /fixtures/test
 /fixtures/my-remix-app
 /fixtures/deno-app
-/fixtures/my-remix-app
 
 .eslintcache

contributors.yml

@@ -12,6 +12,7 @@
 - coryhouse
 - davecalnan
 - derekr
+- developit
 - dunglas
 - dwt47
 - edgesoft

packages/remix-react/components.tsx

@@ -658,6 +658,12 @@ export function Meta() {
   );
 }
 
+/**
+ * Tracks whether Remix has finished hydrating or not, so scripts can be skipped
+ * during client-side updates.
+ */
+let isHydrated = false;
+
 type ScriptProps = Omit<
   React.HTMLProps<HTMLScriptElement>,
   | "children"
@@ -687,6 +693,10 @@ export function Scripts(props: ScriptProps) {
     serverHandoffString
   } = useRemixEntryContext();
 
+  React.useEffect(() => {
+    isHydrated = true;
+  }, []);
+
   let initialScripts = React.useMemo(() => {
     let contextScript = serverHandoffString
       ? `window.__remixContext = ${serverHandoffString};`
@@ -758,7 +768,7 @@ window.__remixRouteModules = {${matches
           crossOrigin={props.crossOrigin}
         />
       ))}
-      {initialScripts}
+      {isHydrated ? null : initialScripts}
     </>
   );
 }