Merge pull request #3 from ReactiveBayes/claude/benchmarks-rxinfer-rendering-39T8i

Author: bvdmitri

.github/workflows/pages.yml

@@ -41,8 +41,6 @@ jobs:
       - run: npm run typecheck
       - run: npm run test
       - run: npm run build
-        env:
-          GITHUB_PAGES: "true"
       - name: Verify the build is fully static (hard rule)
         run: npm run check-static
       - uses: actions/configure-pages@v5

.github/workflows/test.yml

@@ -107,7 +107,5 @@ jobs:
       - run: npm run typecheck
       - run: npm run test
       - run: npm run build
-        env:
-          GITHUB_PAGES: "true"
       - name: Verify the build is fully static (hard rule)
         run: npm run check-static

Makefile

@@ -93,7 +93,7 @@ frontend-dev: ## Dashboard dev server against local data/ (http://localhost:3000
 
 .PHONY: frontend-build
 frontend-build: ## Static export build into frontend/out/
-	GITHUB_PAGES=true $(NPM) run build
+	$(NPM) run build
 
 .PHONY: frontend-check-static
 frontend-check-static: ## Verify the build is fully static (no dynamic pages)

design/frontend.md

@@ -16,8 +16,12 @@ and contains **no server runtime remnants**. Builds that render dynamic pages ar
 
 ## Deployment & data loading
 
-- GitHub Pages **project page** → `basePath`/`assetPrefix` `/RxInferBenchmarks.jl`, enabled when
-  `GITHUB_PAGES=true` (CI); local dev stays at `/`. `public/.nojekyll` prevents Jekyll mangling.
+- Served from the **custom domain** `https://benchmarks.rxinfer.com`, which maps to the site
+  **root** (`/`) — *not* a GitHub Pages project page. So **no `basePath`/`assetPrefix`**: assets
+  and routes resolve from `/` in both local dev and prod. `public/CNAME` (`benchmarks.rxinfer.com`)
+  pins the domain across deploys and `public/.nojekyll` prevents Jekyll from mangling `_next/`.
+  `npm run check-static` guards against a project-page basePath regression (which would 404 every
+  asset on the custom domain and leave the page unstyled).
 - **Data is not bundled.** The app fetches JSON at runtime:
   - prod: `https://raw.githubusercontent.com/<owner>/RxInferBenchmarks.jl/main/data`
   - dev: `/local-data` — a gitignored symlink `frontend/public/local-data → ../../data`,

frontend/next.config.ts

@@ -1,19 +1,18 @@
 import type { NextConfig } from "next";
 
-// GitHub Pages serves this app as a *project page* under /RxInferBenchmarks.jl.
-// The base path is enabled only for the Pages build (GITHUB_PAGES=true) so that
-// local dev and local static previews stay at /.
-const isGitHubPages = process.env.GITHUB_PAGES === "true";
-const basePath = "/RxInferBenchmarks.jl";
-
+// The dashboard is served from the custom domain https://benchmarks.rxinfer.com,
+// which maps to the site root ("/"). It is therefore NOT a GitHub Pages *project
+// page* (those live under /<repo>/), so no basePath/assetPrefix is needed: assets
+// and routes resolve from "/" in both local dev and the deployed site.
+// `public/CNAME` pins the custom domain across deploys; `public/.nojekyll`
+// prevents Jekyll from mangling the `_next/` asset directory.
 const nextConfig: NextConfig = {
   // Hard rule (CLAUDE.md): the build must be fully static. `output: "export"`
   // makes `next build` fail on any dynamically rendered page; scripts/check-static.mjs
   // additionally verifies the exported artifact.
   output: "export",
   trailingSlash: true,
   images: { unoptimized: true },
-  ...(isGitHubPages ? { basePath, assetPrefix: `${basePath}/` } : {}),
 };
 
 export default nextConfig;

frontend/public/CNAME

@@ -0,0 +1 @@
+benchmarks.rxinfer.com

frontend/scripts/check-static.mjs

@@ -3,8 +3,11 @@
 // this script verifies the exported artifact as a second line of defense:
 //   - out/ exists with the expected entry points
 //   - no server runtime remnants are present in the artifact
+//   - the custom domain (CNAME) is pinned and assets resolve from the site root
+//     — no /RxInferBenchmarks.jl/ project-page prefix, which 404s on the custom
+//     domain and leaves the page unstyled
 // Exits non-zero when the build is not a pure static export.
-import { existsSync, readdirSync, statSync } from "node:fs";
+import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
 import { resolve, dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 
@@ -15,11 +18,22 @@ const failures = [];
 if (!existsSync(out)) {
   fail(`out/ does not exist — run \`npm run build\` first (output: "export" must be set)`);
 } else {
-  for (const required of ["index.html", "404.html", ".nojekyll"]) {
+  for (const required of ["index.html", "404.html", ".nojekyll", "CNAME"]) {
     if (!existsSync(join(out, required))) {
       failures.push(`missing required static file: out/${required}`);
     }
   }
+  // The site is served from the custom domain at the root. A project-page
+  // basePath/assetPrefix would point assets at /RxInferBenchmarks.jl/_next/...,
+  // which 404s on benchmarks.rxinfer.com (page renders without styles/JS).
+  for (const html of walk(out).filter((f) => f.endsWith(".html"))) {
+    if (readFileSync(html, "utf8").includes("/RxInferBenchmarks.jl/_next/")) {
+      failures.push(
+        `asset references a project-page basePath (/RxInferBenchmarks.jl/_next/) in ${html} — ` +
+          `the custom domain serves from "/"; remove basePath/assetPrefix in next.config.ts`,
+      );
+    }
+  }
   // Server runtime remnants must never appear in a static export.
   const forbidden = ["server", "standalone", "cache", "BUILD_ID"].filter((name) =>
     existsSync(join(out, name)),