fix: Move redirect logic to useEffect to prevent SSR issues

The redirect logic was previously running directly in the component body,
which could cause issues with React's rendering and Next.js SSR/hydration.
Moving it to useEffect ensures it only runs on the client side after mount.

This should fix the Vercel deployment failure.

Author: snomiao

components/nodes/NodeDetails.tsx

@@ -189,27 +189,29 @@ const NodeDetails = () => {
 
   // redirect to correct node ID if the responded node.id doesn't match the URL nodeId
   // this handles cases where the node ID has changed or been normalized
-  const isNodeIdMismatchedBetweenURLandNode =
-    node && node.id && node.id !== nodeId
-  if (isNodeIdMismatchedBetweenURLandNode) {
-    // if we're on /publishers/[publisherId]/nodes/[nodeId], redirect to the correct publisher and node ID
-    if (_publisherId) {
-      router.replace(`/publishers/${publisherId}/nodes/${node.id}`)
-    } else {
-      // if we're on /nodes/[nodeId], redirect to the correct node ID
-      router.replace(`/nodes/${node.id}`)
+  React.useEffect(() => {
+    const isNodeIdMismatchedBetweenURLandNode =
+      node && node.id && node.id !== nodeId
+    if (isNodeIdMismatchedBetweenURLandNode) {
+      // if we're on /publishers/[publisherId]/nodes/[nodeId], redirect to the correct publisher and node ID
+      if (_publisherId) {
+        router.replace(`/publishers/${publisherId}/nodes/${node.id}`)
+      } else {
+        // if we're on /nodes/[nodeId], redirect to the correct node ID
+        router.replace(`/nodes/${node.id}`)
+      }
     }
-    return null // prevent rendering the component while redirecting
-  }
+  }, [node, nodeId, _publisherId, publisherId, router])
 
   // redirect to correct /publishers/[publisherId]/nodes/[nodeId] if publisherId in query is different from the one in node
   // usually this happens when publisher changes, e.g. when user claims a node
-  const isPublisherIdMismatchedBetweenURLandNode =
-    node && _publisherId && publisherId !== _publisherId
-  if (isPublisherIdMismatchedBetweenURLandNode) {
-    router.replace(`/publishers/${publisherId}/nodes/${nodeId}`)
-    return null // prevent rendering the component while redirecting
-  }
+  React.useEffect(() => {
+    const isPublisherIdMismatchedBetweenURLandNode =
+      node && _publisherId && publisherId !== _publisherId
+    if (isPublisherIdMismatchedBetweenURLandNode) {
+      router.replace(`/publishers/${publisherId}/nodes/${nodeId}`)
+    }
+  }, [node, _publisherId, publisherId, nodeId, router])
 
   const shouldShowLoading = isLoading || !router.isReady || !_nodeId
   if (shouldShowLoading) {