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

package/CHANGELOG.md

@@ -8,6 +8,28 @@ 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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bygghammar/cms-platform",
-  "version": "0.90.0-m7.11",
+  "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/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;
 }