Merge pull request #9 from prazgaitis/fix/preview-convex-site-url

Fix preview build Convex site URL

Author: prazgaitis

.claude/napkin.md

@@ -3,6 +3,9 @@
 ## Corrections
 | Date | Source | What Went Wrong | What To Do Instead |
 |------|--------|----------------|-------------------|
+| 2026-02-11 | self | Ran `ls` before reading napkin (again) | Always read `.claude/napkin.md` before any other command |
+| 2026-02-10 | self | Ran `ls` before reading napkin | Always read `.claude/napkin.md` before any other command |
+| 2026-02-10 | self | Used backticks in a shell-quoted PR body so the shell tried to execute `turbo` | Use a heredoc or escape backticks when passing PR bodies to shell commands |
 
 ## User Preferences
 - Hide navbar on full-screen flow pages (invite, dashboard, admin) via `ConditionalHeader` patterns + remove `page-with-header` class
@@ -19,7 +22,7 @@
 - `page-with-header` CSS class = `pt-16` to offset fixed navbar
 - Seed data lives in `packages/backend/actions/seed.ts`
 - Schema changes auto-deploy locally via `pnpm dev`
-
-## Corrections
-| 2026-02-10 | self | Ran `ls` before reading napkin | Always read `.claude/napkin.md` before any other command |
-| 2026-02-10 | self | Used backticks in a shell-quoted PR body so the shell tried to execute `turbo` | Use a heredoc or escape backticks when passing PR bodies to shell commands |
+| 2026-02-11 | self | Set NEXT_PUBLIC_CONVEX_SITE_URL without export, so child process didn't receive it | Use `export` (or VAR=... cmd) when setting env vars for `next build` |
+| 2026-02-11 | self | Assumed `CONVEX_URL` would always be set inside `--cmd` but it was empty in Vercel logs | Use `--cmd-url-env-var-name NEXT_PUBLIC_CONVEX_URL` and reference that explicitly; export it for `next build` |
+| 2026-02-11 | self | Vercel `buildCommand` schema max length 256 chars | Move long logic into a script and keep `buildCommand` short |
+| 2026-02-11 | self | `set -u` caused failure because `$NEXT_PUBLIC_CONVEX_URL` expanded in the outer script, not inside `--cmd` | Escape `$` in the `--cmd` string so it expands in the Convex-invoked shell |

apps/web/next.config.ts

@@ -1,5 +1,14 @@
 import type { NextConfig } from "next";
 
+// Derive NEXT_PUBLIC_CONVEX_SITE_URL from NEXT_PUBLIC_CONVEX_URL if not set.
+// The site URL is always the cloud URL with ".convex.cloud" → ".convex.site".
+if (!process.env.NEXT_PUBLIC_CONVEX_SITE_URL && process.env.NEXT_PUBLIC_CONVEX_URL) {
+  process.env.NEXT_PUBLIC_CONVEX_SITE_URL = process.env.NEXT_PUBLIC_CONVEX_URL.replace(
+    ".convex.cloud",
+    ".convex.site"
+  );
+}
+
 const nextConfig: NextConfig = {};
 
 export default nextConfig;

tasks/2026-02-11-preview-deploy-convex-site-url.md

@@ -0,0 +1,5 @@
+# 2026-02-11: Fix preview deploy missing CONVEX_SITE_URL
+
+- [x] Reproduce and locate the missing env var usage
+- [x] Update Vercel build command to set `NEXT_PUBLIC_CONVEX_SITE_URL`
+- [x] Summarize the fix and any required deploy env changes