bsmnt 0.14.0 → 0.15.0

package/src/templates/next-pagebuilder/lib/utils/base-url.ts

@@ -3,15 +3,30 @@
  *
  * Priority:
  * 1. NEXT_PUBLIC_BASE_URL (explicit production URL)
- * 2. VERCEL_PROJECT_PRODUCTION_URL (Vercel production domain)
- * 3. VERCEL_URL (Vercel preview/deployment URL)
+ * 2. VERCEL_PROJECT_PRODUCTION_URL, on production deployments only
+ * 3. VERCEL_URL (the deployment's own URL, so previews never emit production URLs)
  * 4. http://localhost:3000 (local development)
  */
 export const getBaseUrl = () =>
   process.env.NEXT_PUBLIC_BASE_URL ||
-  (process.env.VERCEL_PROJECT_PRODUCTION_URL
+  (process.env.VERCEL_ENV === "production" &&
+  process.env.VERCEL_PROJECT_PRODUCTION_URL
     ? `https://${process.env.VERCEL_PROJECT_PRODUCTION_URL}`
     : undefined) ||
   (process.env.VERCEL_URL
     ? `https://${process.env.VERCEL_URL}`
     : "http://localhost:3000")
+
+// The production build also answers on *.vercel.app; only the launch domain is indexed.
+export const getIndexableHost = () => {
+  const baseUrl = process.env.NEXT_PUBLIC_BASE_URL
+  // Off Vercel VERCEL_ENV is unset, and the configured host is the launch domain.
+  const vercelEnv = process.env.VERCEL_ENV
+  if (!baseUrl || (vercelEnv && vercelEnv !== "production")) return null
+  return URL.canParse(baseUrl) ? new URL(baseUrl).host : null
+}
+
+export const isIndexableHost = (host: string | null | undefined) => {
+  const indexable = getIndexableHost()
+  return indexable !== null && host?.toLowerCase() === indexable
+}

package/src/templates/next-pagebuilder/lib/utils/breadcrumbs.ts

@@ -0,0 +1,70 @@
+import type { BREADCRUMB_PAGES_QUERY_RESULT } from "@/lib/integrations/sanity/sanity.types"
+
+type BreadcrumbPage = BREADCRUMB_PAGES_QUERY_RESULT[number]
+
+type Breadcrumb = {
+  name: string
+  url: string
+}
+
+const HOME_NAME = "Home"
+
+const trimSlashes = (path: string) => path.replace(/^\/+|\/+$/g, "")
+
+const firstNonEmpty = (...values: (string | null | undefined)[]) =>
+  values.find((value) => value?.trim())?.trim()
+
+const urlFor = (path: string) => (path ? `/${path}` : "/")
+
+const labelFor = (page: BreadcrumbPage | undefined) =>
+  page && firstNonEmpty(page.shortTitle, page.folderTitle, page.title)
+
+// Every parent path, not the path itself: "a/b/c" → ["a", "a/b"].
+const ancestorPaths = (path: string) => {
+  const segments = path.split("/").filter(Boolean)
+  return segments
+    .slice(0, -1)
+    .map((_, index) => segments.slice(0, index + 1).join("/"))
+}
+
+// The paths a page's crumbs are named from, besides the homepage: its
+// ancestors and itself.
+export const breadcrumbPaths = (path: string | null) => {
+  const current = trimSlashes(path ?? "")
+  return current ? [...ancestorPaths(current), current] : []
+}
+
+// Crumbs are named from the pages at each level, the current one included:
+// Short Title, then the folder name for a folder's index page, then Title. A
+// level with no page is skipped, so a crumb never links to a 404.
+export const buildBreadcrumbs = ({
+  path,
+  name,
+  pages,
+}: {
+  path: string | null
+  name: string
+  pages: readonly BreadcrumbPage[]
+}): Breadcrumb[] => {
+  const pageAt = (target: string) =>
+    pages.find((page) => trimSlashes(page.path ?? "") === target)
+
+  const home = pageAt("")
+  const crumbs: Breadcrumb[] = [
+    {
+      name: firstNonEmpty(home?.shortTitle, home?.title) ?? HOME_NAME,
+      url: "/",
+    },
+  ]
+
+  const current = trimSlashes(path ?? "")
+  if (!current) return crumbs
+
+  for (const ancestor of ancestorPaths(current)) {
+    const ancestorName = labelFor(pageAt(ancestor))
+    if (ancestorName) crumbs.push({ name: ancestorName, url: urlFor(ancestor) })
+  }
+
+  crumbs.push({ name: labelFor(pageAt(current)) ?? name, url: urlFor(current) })
+  return crumbs
+}

package/src/templates/next-pagebuilder/next.config.ts

@@ -1,5 +1,9 @@
 import withBundleAnalyzer from "@next/bundle-analyzer"
 import type { NextConfig } from "next"
+import { getIndexableHost } from "./lib/utils/base-url"
+
+// Only the launch domain is indexed; every other host is sent noindex.
+const indexableHost = getIndexableHost()
 
 const nextConfig: NextConfig = {
   reactStrictMode: true,
@@ -92,6 +96,18 @@ const nextConfig: NextConfig = {
     imageSizes: [16, 32, 48, 64, 96, 128, 256, 384],
   },
   headers: async () => [
+    {
+      source: "/:path*",
+      ...(indexableHost && {
+        missing: [
+          {
+            type: "host" as const,
+            value: indexableHost.replaceAll(".", "\\."),
+          },
+        ],
+      }),
+      headers: [{ key: "X-Robots-Tag", value: "noindex" }],
+    },
     {
       // Excludes /studio* — must stay mutually exclusive with the rule below.
       source: "/((?!studio(?:/|$)).*)",

package/src/templates/next-pagebuilder/package.json

@@ -17,7 +17,7 @@
     "knip": "knip",
     "knip:fix": "knip --fix",
     "lighthouse": "bunx @unlighthouse/cli --site http://localhost:3000",
-    "lint": "biome lint --max-diagnostics=200",
+    "lint": "biome ci --error-on-warnings --max-diagnostics=200 .",
     "lint:fix": "biome check --write --unsafe --max-diagnostics=200",
     "prepare": "lefthook install",
     "start": "next start",

package/src/templates/next-pagebuilder-cache/.env.example

@@ -26,7 +26,9 @@
 
 # Base URL
 # Inlined at build time — scope this to the Production environment only in
-# Vercel so Preview deployments fall through to VERCEL_PROJECT_PRODUCTION_URL.
+# Vercel so Preview deployments fall through to their own VERCEL_URL. On a
+# production deployment it is also the only host that gets indexed: every other
+# host is sent `X-Robots-Tag: noindex` and a disallow-all robots.txt.
 NEXT_PUBLIC_BASE_URL=
 
 # Draft Mode

package/src/templates/next-pagebuilder-cache/.github/workflows/ci.yml

@@ -8,6 +8,13 @@ on:
     branches:
       - main
 
+permissions:
+  contents: read
+
+concurrency:
+  group: ci-${{ github.ref }}
+  cancel-in-progress: ${{ github.event_name == 'pull_request' }}
+
 jobs:
   ci:
     runs-on: ubuntu-latest
@@ -43,17 +50,33 @@ jobs:
       - name: Type check
         run: bun run typecheck
 
-      - name: Lint
+      - name: Lint and format
         run: bun run lint
 
+      # Runs your tests if any exist; a fresh project with none passes cleanly.
+      - name: Test
+        run: |
+          if git ls-files '*.test.*' '*.spec.*' '*_test.*' '*_spec.*' | grep -q .; then
+            bun test
+          else
+            echo "No test files found — skipping bun test."
+          fi
+
       # bsmnt:if dotenvx
       # dotenvx needs the private key to decrypt secrets at build time. It's
       # pulled from 1Password via an org-wide service-account token + the op-ref
       # committed in .env. Optional: builds pass without it (only plaintext /
       # NEXT_PUBLIC_* is read), and a per-repo DOTENV_PRIVATE_KEY secret still
       # works as a fallback.
+      # The `secrets` context isn't available in step `if:`, so the check runs as its own step.
+      - name: Detect 1Password token
+        id: op
+        env:
+          OP_SERVICE_ACCOUNT_TOKEN: ${{ secrets.OP_SERVICE_ACCOUNT_TOKEN }}
+        run: '[ -n "$OP_SERVICE_ACCOUNT_TOKEN" ] && echo "enabled=true" >> "$GITHUB_OUTPUT" || true'
+
       - name: Install 1Password CLI
-        if: ${{ secrets.OP_SERVICE_ACCOUNT_TOKEN != '' }}
+        if: steps.op.outputs.enabled == 'true'
         uses: 1password/install-cli-action@v1
 
       - name: Build
@@ -92,13 +115,14 @@ jobs:
         run: bun run build
 
       # bsmnt:endif
-      # Runs your tests if any exist; a fresh project with none passes cleanly.
-      - name: Test
+      # `prebuild` regenerated these; any change (or untracked file) means they were committed stale.
+      - name: Generated Sanity files are committed
         run: |
-          if git ls-files '*.test.*' '*.spec.*' '*_test.*' '*_spec.*' | grep -q .; then
-            bun test
-          else
-            echo "No test files found — skipping bun test."
+          cd lib/integrations/sanity
+          if [ -n "$(git status --porcelain -- sanity.types.ts schema.json)" ]; then
+            git status --short -- sanity.types.ts schema.json
+            echo "::error::Run \`bun sanity:typegen\` and commit the result."
+            exit 1
           fi
 
       - name: Knip

package/src/templates/next-pagebuilder-cache/AGENTS.md

@@ -146,3 +146,6 @@ the "Page-builder blocks" section of the README.
 
   (`bun run build`, not `bun build` — the bare form is Bun's bundler.) Say what
   you ran and what it printed; don't report success you haven't seen.
+- **Before a commit or PR, or when told to ship, check the `launch-ready`
+  skill's rules against the diff.** Run all of them when asked whether the site
+  is ready to launch.

package/src/templates/next-pagebuilder-cache/app/(content)/[[...slug]]/page.tsx

@@ -1,6 +1,9 @@
 import { cacheLife, cacheTag } from "next/cache"
 import { notFound } from "next/navigation"
-import { PageDocument } from "@/components/page-document"
+import { toPlainText } from "next-sanity"
+import { findBlogBlock, PageDocument } from "@/components/page-document"
+import { getBreadcrumbPages } from "@/lib/integrations/sanity/fetchers/breadcrumbs"
+import { getLayoutData } from "@/lib/integrations/sanity/fetchers/layout"
 import {
   type DynamicFetchOptions,
   getDynamicFetchOptions,
@@ -80,6 +83,23 @@ export const generateStaticParams = async () => {
   ]
 }
 
+// Blog posts fall back to their body and thumbnail when the SEO fields are
+// empty, and are tagged as articles.
+const blogPostMetadata = (page: NonNullable<PAGE_QUERY_RESULT>) => {
+  const post = findBlogBlock(page)
+  if (!post) return {}
+
+  return {
+    fallbackDescription: post.body ? toPlainText(post.body) : undefined,
+    fallbackImage: post.thumbnail,
+    article: {
+      publishedTime: post.date,
+      modifiedTime: page._updatedAt,
+      authors: [post.author?.name],
+    },
+  }
+}
+
 export const generateMetadata = async ({ params }: Props) => {
   const [{ slug }, fetchOptions] = await Promise.all([
     params,
@@ -91,11 +111,20 @@ export const generateMetadata = async ({ params }: Props) => {
 
   // stega off: it would corrupt a <title> or og:image URL, and metadata renders no
   // DOM for <VisualEditing> anyway. In published mode this is the body's key.
-  const page = await getCachedPage(path, { ...fetchOptions, stega: false })
+  const metadataFetchOptions = { ...fetchOptions, stega: false }
+  const [page, { structuredData }] = await Promise.all([
+    getCachedPage(path, metadataFetchOptions),
+    getLayoutData(metadataFetchOptions),
+  ])
 
-  return page
-    ? generateSanityMetadata({ document: page, url: path ? `/${path}` : "/" })
-    : {}
+  if (!page) return {}
+
+  return generateSanityMetadata({
+    document: page,
+    url: path ? `/${path}` : "/",
+    site: structuredData,
+    ...blogPostMetadata(page),
+  })
 }
 
 // No Suspense wrapper needed: `getDynamicFetchOptions` is prerender-safe and
@@ -112,7 +141,10 @@ export default async function CatchAllPage({
 
   if (slug?.length && path === null) notFound()
 
-  const page = await getCachedPage(path, fetchOptions)
+  const [page, breadcrumbPages] = await Promise.all([
+    getCachedPage(path, fetchOptions),
+    getBreadcrumbPages(path, fetchOptions),
+  ])
 
   if (!page) {
     // Fall back to a CMS redirect before 404ing.
@@ -120,5 +152,12 @@ export default async function CatchAllPage({
     notFound()
   }
 
-  return <PageDocument page={page} path={path} searchParams={searchParams} />
+  return (
+    <PageDocument
+      page={page}
+      path={path}
+      searchParams={searchParams}
+      breadcrumbPages={breadcrumbPages}
+    />
+  )
 }

package/src/templates/next-pagebuilder-cache/app/layout.tsx

@@ -4,17 +4,11 @@ import type { Metadata } from "next"
 import { Geist } from "next/font/google"
 import type { PropsWithChildren } from "react"
 import { cn } from "@/lib/styles/cn"
+import { getBaseUrl } from "@/lib/utils/base-url"
 import AppData from "@/package.json"
 
 const APP_NAME = AppData.name
-const APP_DEFAULT_TITLE = "My Site"
-const APP_TITLE_TEMPLATE = "%s"
-const APP_DESCRIPTION = AppData.description
-// Build-time-stable so static routes (Studio, /_not-found) stay prerenderable
-// under Cache Components. NEXT_PUBLIC_* is inlined at build; reading runtime env
-// (VERCEL_URL) here would taint every route's metadata as request-time data.
-const APP_BASE_URL =
-  process.env.NEXT_PUBLIC_BASE_URL ?? "http://localhost:3000"
+const APP_BASE_URL = getBaseUrl()
 
 const geist = Geist({
   subsets: ["latin"],
@@ -22,55 +16,20 @@ const geist = Geist({
 })
 
 export const metadata: Metadata = {
-  alternates: {
-    canonical: "/",
-    languages: {
-      "en-US": "/en-US",
-    },
-  },
+  metadataBase: new URL(APP_BASE_URL),
+  applicationName: APP_NAME,
   appleWebApp: {
     capable: true,
     statusBarStyle: "default",
-    title: APP_DEFAULT_TITLE,
   },
-  applicationName: APP_NAME,
-  authors: [{ name: "basement.studio", url: "https://basement.studio" }],
-  description: APP_DESCRIPTION,
   formatDetection: { telephone: false },
-  metadataBase: new URL(APP_BASE_URL),
   openGraph: {
-    description: APP_DESCRIPTION,
-    images: [
-      {
-        alt: APP_DEFAULT_TITLE,
-        height: 630,
-        url: "/opengraph-image.jpg",
-        width: 1200,
-      },
-    ],
-    locale: "en_US",
     siteName: APP_NAME,
-    title: {
-      default: APP_DEFAULT_TITLE,
-      template: APP_TITLE_TEMPLATE,
-    },
+    locale: "en_US",
     type: "website",
-    url: APP_BASE_URL,
-  },
-  other: {
-    "fb:app_id": process.env.NEXT_PUBLIC_FACEBOOK_APP_ID || "",
-  },
-  title: {
-    default: APP_DEFAULT_TITLE,
-    template: APP_TITLE_TEMPLATE,
   },
   twitter: {
     card: "summary_large_image",
-    description: APP_DESCRIPTION,
-    title: {
-      default: APP_DEFAULT_TITLE,
-      template: APP_TITLE_TEMPLATE,
-    },
   },
 }
 

package/src/templates/next-pagebuilder-cache/app/robots.ts

@@ -1,12 +1,18 @@
 import type { MetadataRoute } from "next"
-import { getBaseUrl } from "@/lib/utils/base-url"
+import { headers } from "next/headers"
+import { getBaseUrl, isIndexableHost } from "@/lib/utils/base-url"
+
+export default async function robots(): Promise<MetadataRoute.Robots> {
+  const host = (await headers()).get("host")
+  if (!isIndexableHost(host)) {
+    return { rules: { userAgent: "*", disallow: "/" } }
+  }
 
-export default function robots(): MetadataRoute.Robots {
   return {
     rules: {
       userAgent: "*",
       allow: "/",
-      disallow: [],
+      disallow: ["/api/", "/studio"],
     },
     sitemap: `${getBaseUrl()}/sitemap.xml`,
   }

package/src/templates/next-pagebuilder-cache/biome.json

@@ -36,7 +36,8 @@
       "!**/*.mdx",
       "!**/tailwind.css",
       "!**/*.grit",
-      "!**/sanity.types.ts"
+      "!**/sanity.types.ts",
+      "!next-env.d.ts"
     ]
   },
   "formatter": {

package/src/templates/next-pagebuilder-cache/components/layout/footer/index.tsx

@@ -60,7 +60,7 @@ async function CachedFooter(fetchOptions: DynamicFetchOptions) {
           <div className="grid grid-cols-5 gap-x-8 gap-y-6 md:grid-cols-3 lg:grid-cols-5">
             {links.map((linkGroup: FooterLinkGroup) => (
               <div key={linkGroup._key} className="flex flex-col gap-2">
-                <h3>{linkGroup.title}</h3>
+                <h2>{linkGroup.title}</h2>
                 {linkGroup.items && linkGroup.items.length > 0 ? (
                   <ul className="space-y-1">
                     {linkGroup.items

package/src/templates/next-pagebuilder-cache/components/page-document/index.tsx

@@ -1,9 +1,13 @@
 import { stegaClean } from "@sanity/client/stega"
 import type { ContentCollectionSearchParams } from "@/components/page-builder/components/content-collection/types"
 import { Renderer } from "@/components/page-builder/renderer"
-import type { PAGE_QUERY_RESULT } from "@/lib/integrations/sanity/sanity.types"
+import type {
+  BREADCRUMB_PAGES_QUERY_RESULT,
+  PAGE_QUERY_RESULT,
+} from "@/lib/integrations/sanity/sanity.types"
 import { urlForImage } from "@/lib/integrations/sanity/utils/image"
 import { buildPageJsonLd } from "@/lib/integrations/sanity/utils/page-json-ld"
+import { buildBreadcrumbs } from "@/lib/utils/breadcrumbs"
 import {
   generateBreadcrumbJsonLd,
   generateSanityArticleJsonLd,
@@ -15,6 +19,7 @@ type PageDocumentProps = {
   page: NonNullable<PAGE_QUERY_RESULT>
   path: string | null
   searchParams: ContentCollectionSearchParams
+  breadcrumbPages: BREADCRUMB_PAGES_QUERY_RESULT
 }
 
 type PageBuilderBlock = NonNullable<
@@ -22,6 +27,13 @@ type PageBuilderBlock = NonNullable<
 >[number]
 type BlogContentBlock = Extract<PageBuilderBlock, { _type: "blogContent" }>
 
+// Blog posts are pages with a Blog Posts block; it carries the post's date,
+// author and thumbnail.
+export const findBlogBlock = (page: NonNullable<PAGE_QUERY_RESULT>) =>
+  page.pageBuilder?.find(
+    (block): block is BlogContentBlock => block?._type === "blogContent"
+  )
+
 export const getPageSections = (
   page: NonNullable<PAGE_QUERY_RESULT>,
   searchParams: ContentCollectionSearchParams
@@ -40,42 +52,20 @@ export const getPageSections = (
       ]
     : []
 
-const buildBreadcrumbs = (
-  slugPath: string | null,
-  pageTitle: string | null
-): Array<{ name: string; url: string }> => {
-  const items: Array<{ name: string; url: string }> = [
-    { name: "Home", url: "/" },
-  ]
-
-  if (!slugPath) return items
-
-  const segments = slugPath.split("/").filter(Boolean)
-  for (let i = 0; i < segments.length; i++) {
-    const isLast = i === segments.length - 1
-    const url = `/${segments.slice(0, i + 1).join("/")}`
-
-    //TODO: we might change this in the future if slug title and folder diverge too much :)
-
-    const name =
-      isLast && pageTitle
-        ? pageTitle
-        : (segments[i]
-            ?.replace(/-/g, " ")
-            .replace(/\b\w/g, (c) => c.toUpperCase()) ?? "")
-    items.push({ name, url })
-  }
-
-  return items
-}
-
 export const PageDocument = ({
   page,
   path,
   searchParams,
+  breadcrumbPages,
 }: PageDocumentProps) => {
   const title = stegaClean(page.title) || page.title
   const sections = getPageSections(page, searchParams)
+  // A Heading block renders the page's H1, so the title only stands in without
+  // one. Hidden blocks are already filtered out of the query, and an empty one
+  // renders nothing (see the registry's `isEmpty`).
+  const hasHeadingBlock = page.pageBuilder?.some(
+    (block) => block?._type === "hero" && Boolean(block.headline)
+  )
 
   const cleanTitle = stegaClean(page.title) || "Basement"
   const cleanDescription = stegaClean(
@@ -87,7 +77,11 @@ export const PageDocument = ({
     ? urlForImage(page.metadata.image).width(1200).height(630).url()
     : undefined
 
-  const breadcrumbs = buildBreadcrumbs(path, cleanTitle)
+  const breadcrumbs = buildBreadcrumbs({
+    path,
+    name: cleanTitle,
+    pages: breadcrumbPages,
+  })
   const hasBreadcrumbs = breadcrumbs.length > 1
 
   // Editor-selected page-level structured data (FAQ, Event, custom, etc.).
@@ -96,9 +90,7 @@ export const PageDocument = ({
   // Blog posts (pages with a Blog Posts block) emit Article JSON-LD with
   // fields derived from the post — no Custom block needed. Skipped when the
   // editor already supplied an Article (via a Custom block) to avoid duplicates.
-  const blogBlock = page.pageBuilder?.find(
-    (block): block is BlogContentBlock => block?._type === "blogContent"
-  )
+  const blogBlock = findBlogBlock(page)
   const hasManualArticle = pageJsonLd.some(
     ({ data }) => (data as { "@type"?: unknown })["@type"] === "Article"
   )
@@ -136,7 +128,7 @@ export const PageDocument = ({
         <JsonLd key={key} data={data} />
       ))}
       <article className="mx-auto flex w-full max-w-6xl flex-1 flex-col px-4 py-14 md:px-6 md:py-20">
-        <h1>{title}</h1>
+        {hasHeadingBlock ? null : <h1>{title}</h1>}
         {sections.length ? (
           sections
         ) : (

package/src/templates/next-pagebuilder-cache/lefthook.yml

@@ -7,7 +7,7 @@ pre-commit:
   commands:
     biome:
       glob: '*.{js,mjs,ts,jsx,tsx,css,scss}'
-      run: bun biome check --max-diagnostics=200 --write --unsafe {staged_files}
+      run: bun biome check --max-diagnostics=200 --write {staged_files}
       stage_fixed: true
     typecheck:
       glob: '*.{ts,tsx}'