another attempt at URL stuff

Author: ianhi

netlify.toml

@@ -1,6 +1,7 @@
 [build]
   publish = ".next"
-  command = "npm run build"
+  # Use DEPLOY_PRIME_URL for previews, fallback to production URL
+  command = "NEXT_PUBLIC_SITE_URL=${DEPLOY_PRIME_URL:-https://xarray.dev} npm run build"
 
 [build.environment]
   NODE_VERSION = "20"
@@ -17,11 +18,8 @@
   to = "/404.html"
   status = 404
 
-# Environment variables for different deploy contexts
-# Note: Netlify automatically provides URL and DEPLOY_PRIME_URL
-# We don't need to set NEXT_PUBLIC_SITE_URL for previews since our code falls back to process.env.URL
-[context.production.environment]
-  NEXT_PUBLIC_SITE_URL = "https://xarray.dev"
+# Environment variables set via build command above
+# DEPLOY_PRIME_URL is automatically provided by Netlify
 
 # Headers for security
 [[headers]]

src/components/layout.js

@@ -39,10 +39,8 @@ export const Layout = ({
   //   </Link>
   //)
 
-  // Determine the base URL based on the environment
-  const baseUrl = process.env.NEXT_PUBLIC_SITE_URL
-    ? process.env.NEXT_PUBLIC_SITE_URL
-    : process.env.URL || 'http://localhost:3000'
+  // Base URL is set via build command (DEPLOY_PRIME_URL for previews, production URL otherwise)
+  const baseUrl = process.env.NEXT_PUBLIC_SITE_URL || 'http://localhost:3000'
 
   // Canonical URL always points to production for SEO
   const canonicalBaseUrl = 'https://xarray.dev'
@@ -52,7 +50,7 @@ export const Layout = ({
 
   // Construct the full card URL
   const fullCardUrl = card.startsWith('http') ? card : `${baseUrl}${card}`
-  // Construct the full URL for og:url (can use preview URL)
+  // Construct the full URL for og:url (uses preview URL in previews, production in prod)
   const fullUrl = url.startsWith('http') ? url : `${baseUrl}${url}`
 
   return (