@bygghammar/cms-platform 0.90.0-m7.10 → 0.90.0-m7.12

package/CHANGELOG.md

@@ -8,6 +8,45 @@ matches `package.json` exactly. See
 Versions 0.1.0 through 0.78.0 predate this repository and were reconstructed
 from the published npm tarballs.
 
+## 0.90.0-m7.12
+
+- The page SEO panel no longer prints a bare page path under the Canonical
+  heading. For a locale with no configured public domain there is no canonical
+  URL, and a path shown there reads as an address on whatever hostname the
+  backoffice happens to be open at.
+- That case now says so in words — *No public URL — no domain configured for
+  this language* — instead of a relative path, the current or backoffice
+  hostname, another locale's domain, or a fabricated absolute URL.
+- The Route row is unchanged and still shows the page path alongside the
+  current registry route, so nothing that was accurate is lost.
+- A mapped locale is unaffected: its canonical preview stays absolute on that
+  locale's own domain. So is single-origin mode, where the one hostname serving
+  every language answers for each of them.
+- `pageSeoPreviewCanonical` keeps its signature and its return value, path
+  fallback included, so existing callers are unaffected; what an empty origin
+  means is decided by `PageSeoPanelView`, which owns the display. The helper's
+  documentation now records that the fallback is not a URL.
+- Regression coverage renders the panel and asserts the Canonical row itself
+  for the mapped, single-origin and unmapped cases.
+- No migration and no database change.
+
+## 0.90.0-m7.11
+
+- The page builder no longer invents a public URL for a locale with no
+  configured public domain. An empty `publicHost(locale)` means no public URL
+  exists, and is no longer read as a relative one.
+- For an unmapped locale the page path is shown as plain, non-clickable
+  information, saying that no domain is configured for that language. The
+  public-link anchor and the Copy URL action are omitted rather than pointed
+  somewhere else: no relative href, no backoffice or request hostname, no other
+  locale's domain.
+- Draft Preview keeps working for an unmapped locale. Previewing is a
+  backoffice operation and has never needed a public hostname.
+- The `publicHost`, `publicPath` and `usePageUrl` host contracts document this,
+  so a host knows what the empty string means before it implements one.
+- No migration and no database change. Nothing in this release concerns a
+  host's own integration code.
+
 ## 0.90.0-m7.10
 
 - Code & tracking drafts survive a refetch. A background reload replaced the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bygghammar/cms-platform",
-  "version": "0.90.0-m7.10",
+  "version": "0.90.0-m7.12",
   "description": "Shared CMS platform: blocks, block types, i18n core, UI primitives and design-token contract used by every Bygghammar/Voyara site.",
   "type": "module",
   "sideEffects": [

package/src/backoffice/PageBuilderView.tsx

@@ -64,9 +64,23 @@ export type PageBuilderDeps<TBlock> = {
   locales: readonly string[];
   localeLabels: Record<string, string>;
   defaultLocale: string;
-  /** Public path for this page in the given locale. */
+  /**
+   * Site-relative path for this page in the given locale.
+   *
+   * A path on its own is not a public address. It only becomes one when
+   * `publicHost` supplies the hostname it belongs to.
+   */
   publicPath: (locale: string) => string;
-  /** Host name to build the absolute URL with, for the given locale. */
+  /**
+   * The public hostname for this locale, or `""` when there is none.
+   *
+   * Since m7.10 a deployment's locale-to-domain table is explicit: a locale
+   * that is not in it has no public site. `""` is that answer — NO PUBLIC URL
+   * EXISTS — and it must never be read as "use a relative URL" or "fall back
+   * to the current host". Doing so hands the browser a path it resolves
+   * against whatever hostname the backoffice happens to be served from,
+   * inventing a public address for a locale the deployment does not publish.
+   */
   publicHost: (locale: string) => string;
 
   versions: BuilderVersion[] | undefined;
@@ -156,7 +170,11 @@ export function PageBuilderView<TBlock>({ deps }: { deps: PageBuilderDeps<TBlock
 
   const slugPath = deps.publicPath(locale);
   const host = deps.publicHost(locale);
-  const absoluteUrl = host ? `https://${host}${slugPath}` : slugPath;
+  // No hostname, no public URL. A relative href here would be resolved against
+  // the backoffice hostname, which is not where this page is published — and
+  // for an unmapped locale it is not published anywhere.
+  const publicUrl = host ? `https://${host}${slugPath}` : null;
+  const localeName = localeLabels[locale] ?? locale;
 
   const togglePanel = (panel: "history" | "schedule") => {
     if (panel === "history") {
@@ -171,8 +189,9 @@ export function PageBuilderView<TBlock>({ deps }: { deps: PageBuilderDeps<TBlock
   };
 
   const onCopyUrl = async () => {
+    if (!publicUrl) return;
     try {
-      await navigator.clipboard.writeText(absoluteUrl);
+      await navigator.clipboard.writeText(publicUrl);
       setCopied(true);
       setTimeout(() => setCopied(false), 1500);
     } catch {
@@ -329,24 +348,40 @@ export function PageBuilderView<TBlock>({ deps }: { deps: PageBuilderDeps<TBlock
             </div>
           )}
           <div className="mt-2 flex flex-wrap items-center gap-2">
-            <a
-              href={absoluteUrl}
-              target="_blank"
-              rel="noreferrer"
-              title={`Open ${absoluteUrl}`}
-              className="inline-flex items-center gap-1.5 border border-ink/15 bg-white px-2.5 py-1 font-mono text-xs text-ink hover:border-sea"
-            >
-              {slugPath}
-              <ExternalLink className="size-3 text-ink/40" />
-            </a>
-            <button
-              onClick={onCopyUrl}
-              title="Copy full URL"
-              className="inline-flex items-center gap-1.5 border border-ink/15 px-2 py-1 text-xs text-ink/60 hover:border-sea hover:text-ink"
-            >
-              {copied ? <Check className="size-3" /> : <Copy className="size-3" />}
-              {copied ? "Copied" : "Copy URL"}
-            </button>
+            {publicUrl ? (
+              <>
+                <a
+                  href={publicUrl}
+                  target="_blank"
+                  rel="noreferrer"
+                  title={`Open ${publicUrl}`}
+                  className="inline-flex items-center gap-1.5 border border-ink/15 bg-white px-2.5 py-1 font-mono text-xs text-ink hover:border-sea"
+                >
+                  {slugPath}
+                  <ExternalLink className="size-3 text-ink/40" />
+                </a>
+                <button
+                  onClick={onCopyUrl}
+                  title="Copy full URL"
+                  className="inline-flex items-center gap-1.5 border border-ink/15 px-2 py-1 text-xs text-ink/60 hover:border-sea hover:text-ink"
+                >
+                  {copied ? <Check className="size-3" /> : <Copy className="size-3" />}
+                  {copied ? "Copied" : "Copy URL"}
+                </button>
+              </>
+            ) : (
+              /* Not a link and not a copy target: there is no address to open
+                 or to paste into a message. The path is still shown, because
+                 it is the one true thing here — where this page would live if
+                 the locale were given a domain. */
+              <span
+                className="inline-flex items-center gap-1.5 border border-dashed border-ink/20 px-2.5 py-1 font-mono text-xs text-ink/50"
+                title={`${localeName} has no public domain, so this page has no public address.`}
+              >
+                {slugPath}
+                <span className="font-sans">· No domain configured for {localeName}</span>
+              </span>
+            )}
             {locales.length > 1 && (
               <div className="flex items-center gap-1">
                 {locales.map((loc) => (

package/src/backoffice/routes/host.ts

@@ -239,6 +239,14 @@ export type BackofficeHost = {
     /**
      * React hook returning a resolver for a page's public URL. Implemented as a
      * hook so a site can read its slug overrides and locale domains.
+     *
+     * `path` is site-relative. `host` is the public hostname for that locale,
+     * or `""` when the locale has no public hostname — since m7.10 the
+     * locale-to-domain table is explicit, so a locale missing from it has no
+     * public site at all. An empty `host` therefore means NO PUBLIC URL
+     * EXISTS; it is not a signal to use `path` on its own, and a host must
+     * not answer it by borrowing the current hostname or another locale's
+     * domain. Screens are expected to offer no public link in that case.
      */
     usePageUrl: () => (
       pageKey: string,

package/src/backoffice/seo/PageSeoPanelView.tsx

@@ -38,7 +38,21 @@ export type PageSeoPanelViewProps = {
   pageKey?: string | null;
   /** Current registry route for a locale, shown in the preview panel. */
   registryRoute: (locale: string) => string;
-  /** Public origin used to build the canonical preview URL. */
+  /**
+   * The public hostname this locale is published on, or `""` when there is
+   * none.
+   *
+   * Since m7.10 a deployment's locale-to-domain table is explicit: a locale
+   * absent from it has no public site. `""` is that answer — NO PUBLIC URL
+   * EXISTS — and it must never be read as "use a relative URL", "use whatever
+   * hostname the backoffice is open at", or "borrow another locale's domain".
+   * Each of those states an address the page is not served at, under the one
+   * heading a search engine is asked to believe.
+   *
+   * A host in single-origin mode — no mappings at all, every language under a
+   * locale prefix on one hostname — answers with that hostname, so the
+   * canonical stays absolute there.
+   */
   originForLocale: (locale: string) => string;
   /** Images found on this page, for the "use hero / pick from page" buttons. */
   pageImages: readonly PageSeoImageCandidate[];
@@ -171,10 +185,13 @@ export function PageSeoPanelView({
     locale: activeLocale,
     defaultLocale,
   });
-  const previewCanonical = pageSeoPreviewCanonical({
-    path: previewPath,
-    origin: originForLocale(activeLocale),
-  });
+  const previewOrigin = originForLocale(activeLocale);
+  // No hostname, no canonical. The path alone is not a fallback: printed here
+  // it reads as a URL on the hostname this screen is being served from, and an
+  // unmapped locale has no public hostname at all.
+  const previewCanonical = previewOrigin
+    ? pageSeoPreviewCanonical({ path: previewPath, origin: previewOrigin })
+    : null;
 
   return (
     <div>
@@ -344,7 +361,15 @@ export function PageSeoPanelView({
           <div className="text-[0.65rem] uppercase tracking-[0.22em] text-ink/55">Preview</div>
           <dl className="mt-4 grid grid-cols-1 md:grid-cols-[140px_1fr] gap-x-6 gap-y-2 text-sm">
             <dt className="text-ink/50">Canonical</dt>
-            <dd className="font-mono text-ink/80 break-all">{previewCanonical}</dd>
+            <dd className="text-ink/80 break-all">
+              {previewCanonical ? (
+                <span className="font-mono">{previewCanonical}</span>
+              ) : (
+                <span className="text-ink/55">
+                  No public URL — no domain configured for this language
+                </span>
+              )}
+            </dd>
             <dt className="text-ink/50">Route</dt>
             <dd className="font-mono text-ink/80">
               {previewPath} (current registry: {registryRoute(activeLocale)})

package/src/backoffice/seo/page-seo-form.ts

@@ -210,7 +210,17 @@ export function pageSeoPreviewPath(args: {
   return slug === "" ? `/${args.locale}` : `/${args.locale}/${slug}`;
 }
 
-/** Absolute canonical preview URL; falls back to the path when no domain is known. */
+/**
+ * Absolute canonical preview URL for a page path on a known public origin.
+ *
+ * With no origin there is no URL to build, and the bare path returned instead
+ * is not one either: shown under a heading like "Canonical" it reads as an
+ * address on whatever hostname is being browsed — which, for a locale with no
+ * configured domain, is the backoffice rather than a public site. Deciding what
+ * an empty origin means belongs to the caller; `PageSeoPanelView` asks first
+ * and says so in words rather than printing this. The fallback is kept for
+ * callers that already depend on it.
+ */
 export function pageSeoPreviewCanonical(args: { path: string; origin?: string | null }): string {
   return args.origin ? `https://${args.origin}${args.path}` : args.path;
 }