@teamscala/service-boot 0.23.376 → 0.23.378

package/package.json

@@ -6,7 +6,7 @@
   },
   "name": "@teamscala/service-boot",
   "type": "module",
-  "version": "0.23.376",
+  "version": "0.23.378",
   "sideEffects": false,
   "imports": {
     "#service-boot/*": "./src/*",

package/src/register-registry-page-route/meta-title.ts

@@ -85,3 +85,45 @@ export function resolveTemplateTitle(
 	});
 	return result || undefined;
 }
+
+/**
+ * THE ONE SUBJECT DERIVATION. What a page is ABOUT — the entity whose name the
+ * browser tab should carry.
+ *
+ * This existed as THREE hand-written `||` chains, one per render branch in
+ * resolve-page-data, and they had already diverged: two ordered form before
+ * proposal, the third ordered proposal first. So which resource named the page
+ * depended on which branch happened to render it, silently.
+ *
+ * The divergence is the cheap half of the cost. The expensive half is that a
+ * correction has to be applied three times to work, and applying it twice looks
+ * exactly like applying it correctly — measured 2026-08-17, when a fix to the
+ * quiz subject was shipped, installed and verified present in the bundle while
+ * the page kept its old title, because the branch that rendered it held the
+ * second copy.
+ *
+ * One chain, one precedence, one place to change. Adding a resource type is an
+ * edit here rather than a fourth branch in three files.
+ */
+export function resolveEntityTitle(resolved: Record<string, unknown>): string | undefined {
+	const pick = (key: string, ...fields: string[]): string | undefined => {
+		const row = resolved[key] as Record<string, unknown> | undefined;
+		if (!row) return undefined;
+		for (const field of fields) {
+			const value = row[field];
+			if (typeof value === "string" && value.trim()) return value.trim();
+		}
+		return undefined;
+	};
+	return (
+		pick("form", "name") ||
+		pick("proposal", "name") ||
+		// A report surface carries its own document name in `report_title`
+		// ("Personalised Authority Blueprint"); `title` names the DIAGNOSTIC
+		// ("The Launcha Authority Index"), which is the wrong subject for the tab
+		// of a document sent to a client. The landing page has no report_title
+		// and correctly falls through to the quiz title.
+		pick("quiz", "report_title", "title") ||
+		pick("agent", "name")
+	);
+}

package/src/register-registry-page-route/resolve-entity-title.test.ts

@@ -0,0 +1,54 @@
+/**
+ * @system ui-service-boot
+ * @status handwritten
+ * @edit edit directly
+ *
+ * Locks the single subject derivation that replaced three divergent copies.
+ */
+
+import { describe, expect, test } from "bun:test";
+import { resolveEntityTitle } from "./meta-title.ts";
+
+describe("resolveEntityTitle", () => {
+	test("a report is named by its own document name, not the diagnostic", () => {
+		// The Launcha Blueprint: `title` names the QUIZ, `report_title` names the
+		// DOCUMENT the client receives. The tab must carry the document.
+		expect(
+			resolveEntityTitle({
+				quiz: {
+					title: "The Launcha Authority Index",
+					report_title: "Personalised Authority Blueprint",
+				},
+			}),
+		).toBe("Personalised Authority Blueprint");
+	});
+
+	test("a quiz with no report name falls through to its title", () => {
+		expect(resolveEntityTitle({ quiz: { title: "The Launcha Authority Index" } })).toBe(
+			"The Launcha Authority Index",
+		);
+	});
+
+	test("precedence is single and fixed across every caller", () => {
+		// The three former copies disagreed: two put form first, one put proposal
+		// first, so the subject depended on which render branch ran. One order now.
+		const all = {
+			form: { name: "Form" },
+			proposal: { name: "Proposal" },
+			quiz: { report_title: "Report" },
+			agent: { name: "Agent" },
+		};
+		expect(resolveEntityTitle(all)).toBe("Form");
+		expect(resolveEntityTitle({ ...all, form: undefined })).toBe("Proposal");
+		expect(resolveEntityTitle({ ...all, form: undefined, proposal: undefined })).toBe("Report");
+	});
+
+	test("blank and whitespace-only names do not win", () => {
+		expect(resolveEntityTitle({ form: { name: "   " }, agent: { name: "Agent" } })).toBe("Agent");
+	});
+
+	test("undefined when nothing names the page", () => {
+		expect(resolveEntityTitle({})).toBeUndefined();
+		expect(resolveEntityTitle({ quiz: {} })).toBeUndefined();
+	});
+});

package/src/register-registry-page-route/resolve-page-data.ts

@@ -10,6 +10,7 @@
  */
 import { resolveThemeMode } from "../resolve-theme-mode.ts";
 import { composePageTitle } from "../compose-page-title.ts";
+import { resolveEntityTitle } from "./meta-title.ts";
 import { getAppLogger } from "@teamscala/logger/app-loggers";
 
 /**
@@ -124,11 +125,7 @@ export function resolvedToPageData(
 		// name in <title>/OG, not the generic "Portal". Falls back to the website
 		// name, then "Portal". Without this, the region-slots branch returned
 		// early with a hardcoded "Portal" title (gpt-test: SSR/DOM/OG title FAIL).
-		const entityTitle =
-			(resolved.form as { name?: string })?.name ||
-			(resolved.quiz as { title?: string })?.title ||
-			(resolved.proposal as { name?: string })?.name ||
-			(resolved.agent as { name?: string })?.name;
+		const entityTitle = resolveEntityTitle(resolved);
 
 		const regions: RegionEntry[] = [];
 		for (const slot of regionSlots) {
@@ -306,11 +303,7 @@ export function resolvedToPageData(
 		const overrideBody = pickRootProp<string>(regions, "bodyFont");
 
 		const pageName = (page.name as string) || null;
-		const entityTitle =
-			(resolved.proposal as { name?: string })?.name ||
-			(resolved.form as { name?: string })?.name ||
-			(resolved.quiz as { title?: string })?.title ||
-			(resolved.agent as { name?: string })?.name;
+		const entityTitle = resolveEntityTitle(resolved);
 		const bodyMetaTitle = regions.find((r) => r.slot === "body")?.metaTitle;
 		const pageRootTitle = pickRootProp<string>(regions, "title");
 		const pageDescription =
@@ -438,11 +431,7 @@ export function resolvedToPageData(
 	const screenMode = pageRoot?.screenMode as string | undefined;
 
 	const pageName = (page?.name as string) || null;
-	const entityTitle =
-		(resolved.form as { name?: string })?.name ||
-		(resolved.quiz as { title?: string })?.title ||
-		(resolved.proposal as { name?: string })?.name ||
-		(resolved.agent as { name?: string })?.name;
+	const entityTitle = resolveEntityTitle(resolved);
 	const entityDescription =
 		((resolved.form as Record<string, unknown>)?.description as string) ||
 		((resolved.quiz as Record<string, unknown>)?.description as string) ||