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

package/CHANGELOG.md

@@ -8,6 +8,23 @@ 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.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.11",
   "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,