Greptile Summary

This PR fixes a critical build failure that occurs in CI environments where the NEXT_PUBLIC_APP_URL environment variable is not available at build time. The changes are made to apps/sim/lib/branding/metadata.ts and specifically address Next.js metadata generation requirements.

The core issue was that the code previously assumed NEXT_PUBLIC_APP_URL would always be available when generating metadata for SEO and social media sharing. However, in CI environments, this variable may not be set during the build phase, causing Next.js to fail when trying to create URL objects for the metadataBase and openGraph.url properties.

The fix introduces proper null checks and fallback values:

• metadataBase now falls back to new URL('https://sim.ai') when NEXT_PUBLIC_APP_URL is undefined
• openGraph.url defaults to 'https://sim.ai' using the logical OR operator

This approach aligns with the application's architecture that supports runtime environment variable injection via Docker (as seen in the codebase's use of next-runtime-env). The fallback to 'https://sim.ai' is logical since it matches the application's primary domain and ensures consistent metadata generation across different deployment scenarios.

Confidence score: 4/5

• This PR is safe to merge with minimal risk as it only adds defensive null checks and sensible fallbacks
• Score reflects a straightforward bug fix that addresses a specific build failure without introducing complex logic changes
• No files require special attention - the changes are isolated to metadata configuration with well-understood fallback behavior

_{1 file reviewed, 1 comment}

_{Edit Code Review Bot Settings | Greptile}