@phis/ui 0.1.0 → 0.2.1

package/helpers/cms-area-config.d.ts

@@ -1,7 +1,7 @@
 import type { PhiBuilderAreaKey } from "../constants/cms-areas";
 import type { PhiRuntimeModuleId } from "../types";
 import type { PhiResolvedCmsAreaPresetTree } from "../types/cms";
-import type { PhiCmsPresetIdentity } from "../types/cms-module-descriptors";
+import type { PhiCmsInstanceId } from "../types/cms-instance-id";
 import { type PhiPageReference } from "../types/references";
 /**
  * The two halves of an Area preset config, expressed as a path rather than as a list of field names.
@@ -84,7 +84,7 @@ export declare function readPhiAreaRootRoute(config: Record<string, unknown> | n
  */
 export type PhiAreaLandingSelection = {
     kind: "preset";
-    identity: PhiCmsPresetIdentity;
+    pageId: PhiCmsInstanceId;
 } | {
     kind: "empty";
 };
@@ -122,6 +122,26 @@ export declare function choosePhiAreaRootApplicant<TApplicant>(applicants: reado
     baseModuleId: PhiRuntimeModuleId | null;
     landingSelection?: PhiAreaLandingSelection | null;
 }): TApplicant | null;
+export type PhiAreaPathClaim = {
+    ownerModuleId: PhiRuntimeModuleId;
+};
+/**
+ * Who answers a path several Modules declare, in the order the answer is taken.
+ *
+ * One rule and two readers, for the reason `choosePhiAreaRootApplicant` above is shared: the route
+ * table decides which Module answers an address, and the Builder's Page list decides which Page an
+ * author is offered for it. Two copies of this would agree until the day a package covers a base
+ * Page, which is the one day it matters -- and the author would be editing a Page nobody is served.
+ *
+ * The rule is the whole of the cover (MODULES.md, "Who owns an address"): an Area's base Module holds
+ * its Pages as a floor rather than as a choice, so it yields to any other Module declaring one of
+ * their paths, and keeps the path when none does. An address answers with its first claim, so
+ * yielding is nothing more than standing last. A `baseModuleId` of `null` is an Area where nothing
+ * yields -- outside Public no package can declare a base path -- and the order is left alone.
+ */
+export declare function orderPhiAreaPathClaimsByCover<TClaim>(claims: readonly TClaim[], readClaim: (claim: TClaim) => PhiAreaPathClaim, { baseModuleId }: {
+    baseModuleId: PhiRuntimeModuleId | null;
+}): readonly TClaim[];
 export declare const PHI_AREA_META_KEY: "meta";
 /**
  * What an Area says about itself in the head of every Page it draws.

package/helpers/cms-area-config.js

@@ -1,6 +1,6 @@
-import { createPhiDefaultAreaRuntimeModuleIds } from "../plugins/runtime-modules/builder/runtime-module-defaults";
+import { createPhiDefaultAreaRuntimeModuleIds } from "../plugins/runtime-modules/area-module-defaults";
 import { readPhiRuntimeModuleIds } from "../plugins/runtime-modules/settings";
-import { readPhiPageReference } from "../types/references";
+import { createPhiPresetCmsPageId, readPhiPageReference, } from "../types/references";
 /**
  * The two halves of an Area preset config, expressed as a path rather than as a list of field names.
  *
@@ -90,21 +90,19 @@ export function resolvePhiAreaLandingSelection(rootRoute) {
     }
     const parsed = rootRoute.target ? readPhiPageReference(rootRoute.target) : null;
     return parsed?.target.kind === "module"
-        ? {
-            kind: "preset",
-            identity: {
-                ownerModuleId: parsed.target.ownerModuleId,
-                presetKey: parsed.target.presetKey,
-            },
-        }
+        ? { kind: "preset", pageId: parsed.target.pageId }
         : { kind: "empty" };
 }
 export function choosePhiAreaRootApplicant(applicants, readApplicant, { baseModuleId, landingSelection, }) {
+    /*
+     * Compared as Page ids rather than as the pair they are derived from. The Builder stores a reference,
+     * and a Module Page's reference names it by that id -- so the applicant is hashed to meet the answer
+     * rather than the answer unpacked to meet the applicant, which a one-way id does not allow.
+     */
     const named = landingSelection?.kind === "preset"
         ? applicants.find((applicant) => {
             const identity = readApplicant(applicant);
-            return identity.ownerModuleId === landingSelection.identity.ownerModuleId &&
-                identity.presetKey === landingSelection.identity.presetKey;
+            return createPhiPresetCmsPageId(identity) === landingSelection.pageId;
         })
         : undefined;
     /*
@@ -122,6 +120,29 @@ export function choosePhiAreaRootApplicant(applicants, readApplicant, { baseModu
         ?? (landingSelection?.kind === "empty" ? null : applicants[0])
         ?? null;
 }
+/**
+ * Who answers a path several Modules declare, in the order the answer is taken.
+ *
+ * One rule and two readers, for the reason `choosePhiAreaRootApplicant` above is shared: the route
+ * table decides which Module answers an address, and the Builder's Page list decides which Page an
+ * author is offered for it. Two copies of this would agree until the day a package covers a base
+ * Page, which is the one day it matters -- and the author would be editing a Page nobody is served.
+ *
+ * The rule is the whole of the cover (MODULES.md, "Who owns an address"): an Area's base Module holds
+ * its Pages as a floor rather than as a choice, so it yields to any other Module declaring one of
+ * their paths, and keeps the path when none does. An address answers with its first claim, so
+ * yielding is nothing more than standing last. A `baseModuleId` of `null` is an Area where nothing
+ * yields -- outside Public no package can declare a base path -- and the order is left alone.
+ */
+export function orderPhiAreaPathClaimsByCover(claims, readClaim, { baseModuleId }) {
+    if (baseModuleId == null) {
+        return claims;
+    }
+    return [
+        ...claims.filter((claim) => readClaim(claim).ownerModuleId !== baseModuleId),
+        ...claims.filter((claim) => readClaim(claim).ownerModuleId === baseModuleId),
+    ];
+}
 export const PHI_AREA_META_KEY = "meta";
 /**
  * What an Area that was never asked means, in the one Area the indexing question is ever asked in.

package/helpers/cms-area-root-route.d.ts

@@ -1,5 +1,4 @@
 import type { PhiCmsAreaKey } from "../constants/cms-areas";
-import type { PhiBlockRuntime } from "../types";
 import type { PhiResolvedCmsPagePayload, PhiResolvedCmsPageTree } from "../types/cms";
 import type { PhiCmsCompiledDescriptorCatalog, PhiRuntimeModuleId } from "../types/cms-module-descriptors";
 import { type PhiPageReference } from "../types/references";
@@ -21,16 +20,18 @@ export type PhiAreaRootRouteDecision = {
 /**
  * A reference to a Page a Module carries, as the Area-relative path it names.
  *
- * Checked against this Area, this Module selection and this viewer, so a reference to a route that is
- * no longer reachable resolves to nothing rather than to a path that answers 404. A Site-authored Page
- * cannot be resolved without asking the server and is handled by the caller.
+ * Checked against this Area and this Module selection, so a reference to a route no Module carries any
+ * more resolves to nothing rather than to a path that answers 404. A Site-authored Page cannot be
+ * resolved without asking the server and is handled by the caller.
+ *
+ * Not against the viewer. This is what an Area's root forwards to, and a front door that moved with the
+ * reader is the thing ACCESS.md forbids.
  */
-export declare function resolvePhiAreaModulePageReferencePath({ reference, area, catalog, activeModuleIds, viewer, }: {
+export declare function resolvePhiAreaModulePageReferencePath({ reference, area, catalog, activeModuleIds, }: {
     reference: PhiPageReference;
     area: PhiCmsAreaKey;
     catalog: PhiCmsCompiledDescriptorCatalog;
     activeModuleIds: ReadonlySet<PhiRuntimeModuleId>;
-    viewer: PhiBlockRuntime["viewer"];
 }): string | null;
 /**
  * The decision, applied to whatever the ordinary resolution produced.
@@ -41,7 +42,7 @@ export declare function resolvePhiAreaModulePageReferencePath({ reference, area,
  * one, because a Builder who chose a landing page chose that the root does not forward.
  */
 /**
- * The answer to a folder address (phis-server TODOS.md, "Folder addresses"): a path no Page answers, whose
+ * The answer to a folder address (phis-server REFERENCES.md, "Folder addresses"): a path no Page answers, whose
  * Navigation container leads to a Page. There is no Page to take bookkeeping from, so this is a bare
  * Redirect page -- no regions, no content -- that the Layout answers with a 307 before the shell flushes,
  * as it does a forwarding Area root. Its access is `anyone` because the Area's own access was checked

package/helpers/cms-area-root-route.js

@@ -1,24 +1,23 @@
 import { PhiCmsPageType, PhiCmsStatus } from "../constants/phi-cms";
-import { resolvePhiCmsRoutePresetByIdentity } from "../plugins/runtime-modules/descriptor-compiler";
-import { canPhiViewerAccess } from "../types/access";
+import { resolvePhiCmsRouteDescriptorByPageId } from "../plugins/runtime-modules/descriptor-compiler";
 import { readPhiPageReference } from "../types/references";
 /**
  * A reference to a Page a Module carries, as the Area-relative path it names.
  *
- * Checked against this Area, this Module selection and this viewer, so a reference to a route that is
- * no longer reachable resolves to nothing rather than to a path that answers 404. A Site-authored Page
- * cannot be resolved without asking the server and is handled by the caller.
+ * Checked against this Area and this Module selection, so a reference to a route no Module carries any
+ * more resolves to nothing rather than to a path that answers 404. A Site-authored Page cannot be
+ * resolved without asking the server and is handled by the caller.
+ *
+ * Not against the viewer. This is what an Area's root forwards to, and a front door that moved with the
+ * reader is the thing ACCESS.md forbids.
  */
-export function resolvePhiAreaModulePageReferencePath({ reference, area, catalog, activeModuleIds, viewer, }) {
+export function resolvePhiAreaModulePageReferencePath({ reference, area, catalog, activeModuleIds, }) {
     const parsed = readPhiPageReference(reference);
     if (parsed?.target.kind !== "module") {
         return null;
     }
-    const route = resolvePhiCmsRoutePresetByIdentity(catalog, parsed.target.ownerModuleId, parsed.target.presetKey);
-    return route &&
-        route.area === area &&
-        activeModuleIds.has(route.ownerModuleId) &&
-        canPhiViewerAccess(viewer, route.accessPolicy)
+    const route = resolvePhiCmsRouteDescriptorByPageId(catalog, parsed.target.pageId);
+    return route && route.area === area && activeModuleIds.has(route.ownerModuleId)
         ? route.path
         : null;
 }
@@ -31,7 +30,7 @@ export function resolvePhiAreaModulePageReferencePath({ reference, area, catalog
  * one, because a Builder who chose a landing page chose that the root does not forward.
  */
 /**
- * The answer to a folder address (phis-server TODOS.md, "Folder addresses"): a path no Page answers, whose
+ * The answer to a folder address (phis-server REFERENCES.md, "Folder addresses"): a path no Page answers, whose
  * Navigation container leads to a Page. There is no Page to take bookkeeping from, so this is a bare
  * Redirect page -- no regions, no content -- that the Layout answers with a 307 before the shell flushes,
  * as it does a forwarding Area root. Its access is `anyone` because the Area's own access was checked

package/helpers/cms-config-field-validation.js

@@ -41,6 +41,15 @@ export function readPhiCmsConfigFieldViolation(field, value) {
             }
             return null;
         }
+        case "number-list": {
+            if (!Array.isArray(value)) {
+                return `${field.key} must be a list of numbers, not ${describe(value)}.`;
+            }
+            const stray = value.find((item) => typeof item !== "number" || !Number.isInteger(item) || (field.min !== undefined && item < field.min));
+            return stray === undefined
+                ? null
+                : `${field.key} must list whole numbers${field.min !== undefined ? ` of at least ${field.min}` : ""}, not ${describe(stray)}.`;
+        }
         case "choice": {
             /*
              * Only a closed list can be checked. A field whose options are fetched, or which accepts what

package/helpers/cms-config-serialization.d.ts

@@ -1,71 +1,2 @@
-import type { CSSProperties } from "react";
 import type { PhiRenderableBlockBase } from "../types";
-export declare function serializePhiCmsRenderableBlockConfig(value: Partial<PhiRenderableBlockBase> | null | undefined): {
-    [x: string]: unknown;
-};
-export declare function deserializePhiCmsRenderableBlockConfig(value: unknown): import("../types").PhiRenderableBlock;
-export declare function stripPhiCmsRenderableBlockConfigDefaults(value: Partial<PhiRenderableBlockBase> | null | undefined): Partial<PhiRenderableBlockBase>;
 export declare function mergePhiCmsRenderableBlockConfigDefaults(value: Partial<PhiRenderableBlockBase> | null | undefined): PhiRenderableBlockBase;
-export declare function serializePhiCmsLayerConfig(value: unknown): {
-    [x: string]: unknown;
-};
-export declare function deserializePhiCmsLayerConfig(value: unknown): {
-    padding: number | string | undefined;
-    background: CSSProperties["background"] | undefined;
-    border: CSSProperties["border"] | undefined;
-    borderRadius: number | string | undefined;
-    renderMode?: import("../types").PhiRenderableBlockRenderMode;
-    visibility?: import("../types").PhiRenderableBlockVisibility;
-    accessPolicy?: import("@phis/contracts/access").PhiViewerAccessPolicy;
-    viewportFlags?: import("../types").PhiViewportFlags;
-    enabled?: boolean;
-    debugMode?: boolean;
-    anchor?: import("../types").PhiRenderableBlockAnchor;
-    zIndex?: number;
-    opacity?: number;
-    effect?: import("../types").PhiLayoutEffectId;
-    shadow?: import("../types").PhiShadow;
-    className?: string;
-    size?: import("../types").PhiRenderableBlockSize;
-    minSize?: import("../types").PhiRenderableBlockSize;
-    maxSize?: import("../types").PhiRenderableBlockSize;
-    collapsedSizeHint?: import("../types").PhiRenderableBlockSize;
-    effects?: import("../types").PhiRenderableBlockEffects;
-    capabilities?: import("../types").PhiRenderableBlockCapabilities;
-    runtime?: import("../types").PhiRenderableBlockRuntime;
-};
-export declare function serializePhiCmsDirectionalLayoutConfig(value: unknown): {
-    [x: string]: unknown;
-};
-export declare function deserializePhiCmsDirectionalLayoutConfig(value: unknown): {
-    gap: number | string | undefined;
-    align: CSSProperties["alignItems"] | undefined;
-    justify: CSSProperties["justifyContent"] | undefined;
-    wrap: boolean | CSSProperties["flexWrap"] | undefined;
-    padding: number | string | undefined;
-    background: CSSProperties["background"] | undefined;
-    border: CSSProperties["border"] | undefined;
-    borderRadius: number | string | undefined;
-    renderMode?: import("../types").PhiRenderableBlockRenderMode;
-    visibility?: import("../types").PhiRenderableBlockVisibility;
-    accessPolicy?: import("@phis/contracts/access").PhiViewerAccessPolicy;
-    viewportFlags?: import("../types").PhiViewportFlags;
-    enabled?: boolean;
-    debugMode?: boolean;
-    anchor?: import("../types").PhiRenderableBlockAnchor;
-    zIndex?: number;
-    opacity?: number;
-    effect?: import("../types").PhiLayoutEffectId;
-    shadow?: import("../types").PhiShadow;
-    className?: string;
-    size?: import("../types").PhiRenderableBlockSize;
-    minSize?: import("../types").PhiRenderableBlockSize;
-    maxSize?: import("../types").PhiRenderableBlockSize;
-    collapsedSizeHint?: import("../types").PhiRenderableBlockSize;
-    effects?: import("../types").PhiRenderableBlockEffects;
-    capabilities?: import("../types").PhiRenderableBlockCapabilities;
-    runtime?: import("../types").PhiRenderableBlockRuntime;
-};
-export declare function serializePhiCmsWidgetConfig(value: unknown): {
-    [x: string]: unknown;
-};

package/helpers/cms-config-serialization.js

@@ -1,63 +1,4 @@
-import { deserializeRenderableBlock, mergeRenderableBlockDefaults, serializeRenderableBlock, stripRenderableBlockDefaults, } from "./renderable-block-serialization";
-function isRecord(value) {
-    return Boolean(value) && typeof value === "object" && !Array.isArray(value);
-}
-function copyIfPresent(target, key, value) {
-    if (value === undefined || value === null || value === "") {
-        return;
-    }
-    target[key] = value;
-}
-export function serializePhiCmsRenderableBlockConfig(value) {
-    return serializeRenderableBlock(value);
-}
-export function deserializePhiCmsRenderableBlockConfig(value) {
-    return deserializeRenderableBlock(value);
-}
-export function stripPhiCmsRenderableBlockConfigDefaults(value) {
-    return stripRenderableBlockDefaults(value);
-}
+import { mergeRenderableBlockDefaults } from "./renderable-block-serialization";
 export function mergePhiCmsRenderableBlockConfigDefaults(value) {
     return mergeRenderableBlockDefaults(value);
 }
-export function serializePhiCmsLayerConfig(value) {
-    const normalized = isRecord(value) ? value : {};
-    const next = serializeRenderableBlock(normalized);
-    copyIfPresent(next, "padding", normalized.padding);
-    copyIfPresent(next, "background", normalized.background);
-    copyIfPresent(next, "border", normalized.border);
-    copyIfPresent(next, "borderRadius", normalized.borderRadius);
-    return next;
-}
-export function deserializePhiCmsLayerConfig(value) {
-    const normalized = isRecord(value) ? value : {};
-    return {
-        ...deserializeRenderableBlock(normalized),
-        padding: normalized.padding,
-        background: normalized.background,
-        border: normalized.border,
-        borderRadius: normalized.borderRadius,
-    };
-}
-export function serializePhiCmsDirectionalLayoutConfig(value) {
-    const normalized = isRecord(value) ? value : {};
-    const next = serializePhiCmsLayerConfig(normalized);
-    copyIfPresent(next, "gap", normalized.gap);
-    copyIfPresent(next, "align", normalized.align);
-    copyIfPresent(next, "justify", normalized.justify);
-    copyIfPresent(next, "wrap", normalized.wrap);
-    return next;
-}
-export function deserializePhiCmsDirectionalLayoutConfig(value) {
-    const normalized = isRecord(value) ? value : {};
-    return {
-        ...deserializePhiCmsLayerConfig(normalized),
-        gap: normalized.gap,
-        align: normalized.align,
-        justify: normalized.justify,
-        wrap: normalized.wrap,
-    };
-}
-export function serializePhiCmsWidgetConfig(value) {
-    return serializeRenderableBlock(value);
-}

package/helpers/cms-draft-delete.d.ts

@@ -0,0 +1,8 @@
+/**
+ * Deletes a stored CMS draft through the Site's CMS routes.
+ *
+ * Any Module that keeps drafts discards them the same way -- the Builder its page and Module drafts, the
+ * Revisions page an Area's own shell -- so the call is the Foundation's, and the path says which draft.
+ * Non-empty string values of `payload` become the query; the route's error is thrown as it came back.
+ */
+export declare function deleteCmsDraft(path: string, payload: Record<string, unknown>): Promise<void>;

package/helpers/cms-draft-delete.js

@@ -0,0 +1,23 @@
+/**
+ * Deletes a stored CMS draft through the Site's CMS routes.
+ *
+ * Any Module that keeps drafts discards them the same way -- the Builder its page and Module drafts, the
+ * Revisions page an Area's own shell -- so the call is the Foundation's, and the path says which draft.
+ * Non-empty string values of `payload` become the query; the route's error is thrown as it came back.
+ */
+export async function deleteCmsDraft(path, payload) {
+    const url = new URL(path, typeof window !== "undefined" ? window.location.origin : "http://localhost");
+    for (const [key, value] of Object.entries(payload)) {
+        if (typeof value === "string" && value.trim().length > 0) {
+            url.searchParams.set(key, value);
+        }
+    }
+    const response = await fetch(url.toString(), {
+        method: "DELETE",
+    });
+    if (!response.ok) {
+        const body = (await response.json().catch(() => null));
+        const detail = body?.details?.length ? ` ${body.details.join(" ")}` : "";
+        throw new Error(`${body?.error ?? "CMS draft delete failed."}${detail}`);
+    }
+}

package/helpers/cms-layout-defaults.js

@@ -1,13 +1,10 @@
-import { PHI_COLOR, PHI_MARGIN, PHI_RADIUS, PHI_SHADOW, PHI_SPACE } from "../theme/antd-css-var-contract";
+import { PHI_COLOR, PHI_MARGIN, PHI_SHADOW, PHI_SPACE } from "../theme/antd-css-var-contract";
 import { PHI_RENDERABLE_BLOCK_DEFAULT_ANCHOR } from "./renderable-block-defaults";
-const PHI_LAYOUT_CHROME_PRESET_DEFAULTS = {
-    borderRadius: PHI_RADIUS.base,
-};
+import { isPhiRecord } from "./is-record";
 export const PHI_CONTENT_LAYOUT_DEFAULTS = {
     margin: 0,
 };
 const PHI_CONTENT_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     margin: 0,
     padding: PHI_SPACE.base,
 };
@@ -22,7 +19,6 @@ export const PHI_FLEX_LAYOUT_DEFAULTS = {
     wrap: false,
 };
 const PHI_FLEX_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     distribution: "anchor",
     gap: PHI_MARGIN.base,
     verticalSeparators: false,
@@ -54,7 +50,6 @@ export const PHI_FLEX_VERTICAL_LAYOUT_DEFAULTS = {
     gap: PHI_SPACE.sm,
 };
 const PHI_FLEX_VERTICAL_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     gap: PHI_SPACE.base,
     padding: PHI_SPACE.base,
 };
@@ -64,7 +59,6 @@ const PHI_FLEX_VERTICAL_LAYOUT_PANEL_PRESET = {
  * touching this config -- the preset tree contract rejects a deviating copy.
  */
 const PHI_FLEX_VERTICAL_LAYOUT_PAGE_BASE_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     gap: PHI_SPACE.base,
     padding: PHI_SPACE.base,
     width: "100%",
@@ -78,14 +72,13 @@ const PHI_FLEX_VERTICAL_LAYOUT_PAGE_BASE_PRESET = {
 };
 export const PHI_GRID_LAYOUT_DEFAULTS = {
     gap: 0,
-    columnGap: 0,
     align: undefined,
     justify: undefined,
     wrap: false,
 };
 const PHI_GRID_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     gap: PHI_MARGIN.base,
+    // Rows apart, columns flush: stated, because an absent column gap follows the gap now.
     columnGap: 0,
     align: undefined,
     justify: undefined,
@@ -97,16 +90,23 @@ export const PHI_MASONRY_LAYOUT_DEFAULTS = {
     gap: PHI_MARGIN.md,
 };
 const PHI_MASONRY_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     columns: 3,
     gap: PHI_MARGIN.base,
     padding: PHI_SPACE.base,
 };
 export const PHI_STACK_LAYOUT_DEFAULTS = {
+    // One slot at a time is what a Stack is until somebody says otherwise; the pile is the deliberate
+    // choice, not the standing one.
+    slotDisplay: "single",
     slotTransition: "none",
+    /*
+     * Stated here although the parser resolves the same word on its own, because these defaults are
+     * what the Inspector shows as the standing answer. Left absent, the select stood empty over a
+     * setting that was never in doubt.
+     */
+    mountPolicy: "remount",
 };
 const PHI_STACK_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     padding: PHI_SPACE.base,
 };
 export const PHI_CAROUSEL_LAYOUT_DEFAULTS = {
@@ -128,7 +128,6 @@ export const PHI_CAROUSEL_LAYOUT_DEFAULTS = {
     loop: false,
 };
 const PHI_CAROUSEL_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     padding: PHI_SPACE.base,
 };
 export const PHI_COLLAPSIBLE_LAYOUT_DEFAULTS = {
@@ -138,26 +137,23 @@ export const PHI_COLLAPSIBLE_LAYOUT_DEFAULTS = {
     translateSlotTitles: true,
     defaultOpenSlotKeys: ["slot_0"],
     collapsible: "header",
-    bordered: false,
     ghost: true,
     expandIconPlacement: "start",
-    collapseSize: "middle",
+    collapseSize: "medium",
     titleStrong: true,
     headerPadding: PHI_SPACE.sm,
     innerPadding: PHI_SPACE.base,
 };
 const PHI_COLLAPSIBLE_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     anchor: PHI_RENDERABLE_BLOCK_DEFAULT_ANCHOR,
     accordion: false,
     slotTitles: [],
     translateSlotTitles: true,
     defaultOpenSlotKeys: ["slot_0"],
     collapsible: "header",
-    bordered: true,
     ghost: false,
     expandIconPlacement: "start",
-    collapseSize: "middle",
+    collapseSize: "medium",
     titleStrong: true,
     headerPadding: PHI_SPACE.sm,
     innerPadding: PHI_SPACE.base,
@@ -170,7 +166,6 @@ export const PHI_SPLIT_CARD_LAYOUT_DEFAULTS = {
     gap: 0,
 };
 const PHI_SPLIT_CARD_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     anchor: PHI_SPLIT_CARD_LAYOUT_DEFAULT_ANCHOR,
     gap: PHI_SPACE.base,
     padding: "1.3125rem",
@@ -211,7 +206,6 @@ export const PHI_THREE_COLUMN_LAYOUT_DEFAULTS = {
     wrap: false,
 };
 const PHI_THREE_COLUMN_LAYOUT_PANEL_PRESET = {
-    ...PHI_LAYOUT_CHROME_PRESET_DEFAULTS,
     balancedSides: true,
     gap: PHI_MARGIN.base,
     wrap: false,
@@ -232,14 +226,11 @@ const PHI_LAYOUT_DEFAULTS_BY_KIND = {
     verticalflex: PHI_FLEX_VERTICAL_LAYOUT_DEFAULTS,
     collapsible: PHI_COLLAPSIBLE_LAYOUT_DEFAULTS,
 };
-function isPlainObject(value) {
-    return Boolean(value) && typeof value === "object" && !Array.isArray(value);
-}
 function isSameValue(left, right) {
     if (Object.is(left, right)) {
         return true;
     }
-    if (!isPlainObject(left) || !isPlainObject(right)) {
+    if (!isPhiRecord(left) || !isPhiRecord(right)) {
         return false;
     }
     return JSON.stringify(left) === JSON.stringify(right);
@@ -275,7 +266,7 @@ export function resolvePhiLayoutCreationPreset(layoutKind, preset) {
     return { ...PHI_LAYOUT_PANEL_PRESETS_BY_KIND[layoutKind] };
 }
 export function applyPhiLayoutDefaults(value, defaults) {
-    if (!isPlainObject(value)) {
+    if (!isPhiRecord(value)) {
         return { ...defaults };
     }
     const next = { ...defaults };

package/helpers/cms-layout-depth.d.ts

@@ -4,8 +4,6 @@ export type PhiCmsLayoutDepthNode = {
     id: PhiCmsInstanceId;
     parentLayoutNodeId: PhiCmsInstanceId | null;
 };
-export declare function resolvePhiCmsFlatLayoutNodeDepth(layoutNodes: readonly PhiCmsLayoutDepthNode[], targetNodeId: PhiCmsInstanceId): number | null;
-export declare function collectPhiCmsLayoutDepthValidationErrors(layoutNodes: readonly PhiCmsLayoutDepthNode[], maxDepth?: number): string[];
 export type PhiCmsLayoutRenderDepthNode = PhiCmsLayoutDepthNode & {
     childLayouts?: readonly PhiCmsLayoutRenderDepthNode[];
 };

package/helpers/cms-layout-depth.js

@@ -1,39 +1,4 @@
 export const PHI_CMS_MAX_LAYOUT_SUBLAYOUT_DEPTH = 3;
-export function resolvePhiCmsFlatLayoutNodeDepth(layoutNodes, targetNodeId) {
-    const nodesById = new Map(layoutNodes.map((node) => [node.id, node]));
-    const visiting = new Set();
-    function resolveDepth(nodeId) {
-        const node = nodesById.get(nodeId);
-        if (!node) {
-            return null;
-        }
-        if (node.parentLayoutNodeId == null) {
-            return 0;
-        }
-        if (visiting.has(nodeId)) {
-            return null;
-        }
-        visiting.add(nodeId);
-        const parentDepth = resolveDepth(node.parentLayoutNodeId);
-        visiting.delete(nodeId);
-        return parentDepth == null ? null : parentDepth + 1;
-    }
-    return resolveDepth(targetNodeId);
-}
-export function collectPhiCmsLayoutDepthValidationErrors(layoutNodes, maxDepth = PHI_CMS_MAX_LAYOUT_SUBLAYOUT_DEPTH) {
-    const errors = [];
-    for (const node of layoutNodes) {
-        const depth = resolvePhiCmsFlatLayoutNodeDepth(layoutNodes, node.id);
-        if (depth == null) {
-            errors.push(`Layout node ${node.id} has a circular or unresolved parent chain.`);
-            continue;
-        }
-        if (depth > maxDepth) {
-            errors.push(`Layout node ${node.id} exceeds maximum layout depth ${maxDepth}.`);
-        }
-    }
-    return errors;
-}
 export function resolvePhiCmsRenderLayoutNodeDepth(layoutNodes, targetNodeId, depth = 1) {
     for (const node of layoutNodes) {
         if (node.id === targetNodeId) {

package/helpers/cms-navigation-catalog.d.ts

@@ -1,4 +1,4 @@
-import { type PhiCmsAreaKey } from "../constants/cms-areas";
+import { type PhiBuilderAreaKey, type PhiCmsAreaKey } from "../constants/cms-areas";
 import type { PhiCmsPresetIdentity, PhiCmsNavigationFolder, PhiCmsNavigationOverlay, PhiCmsResolvedNavigationSurface, PhiRuntimeModuleId } from "../types/cms-module-descriptors";
 import type { PhiCmsInstanceId } from "../types/cms-instance-id";
 import type { PhiPresetPageNode } from "./cms-page-catalog";
@@ -10,6 +10,8 @@ export type PhiBuilderNavigationItem = {
     label: string;
     href: string | null;
     targetReference?: string | null;
+    /** The Area the reference resolves in, where it is not this Navigation's own. */
+    targetArea?: PhiCmsAreaKey | null;
     targetDeleted?: boolean;
     icon?: string | null;
     external?: boolean;
@@ -41,10 +43,29 @@ export declare function parsePhiBuilderNavigationScopeKey(value: string | null |
     key: string;
 } | null;
 export declare function requirePhiBuilderNavigationScopeKey(value: string | null | undefined): string;
-export declare function countPhiBuilderNavigationItems(items: PhiBuilderNavigationItem[]): number;
 export declare function getPhiBuilderNavigationDefaultScopeKey(area: PhiCmsAreaKey, family?: PhiBuilderNavigationPresetFamily | null): string;
 export declare function createPhiBuilderCustomNavigationSurface(navKey: string, label?: string | null): PhiCmsResolvedNavigationSurface;
 export declare function materializePhiBuilderNavigationSurface(descriptorSurface: PhiCmsResolvedNavigationSurface, overlay: PhiCmsNavigationOverlay | null): PhiBuilderNavigationTree;
 export declare function findPhiBuilderNavigationSurface(surfaces: readonly PhiCmsResolvedNavigationSurface[], navKey: string): PhiCmsResolvedNavigationSurface | null;
 export declare function resolvePhiBuilderNavigationSurface(surfaces: readonly PhiCmsResolvedNavigationSurface[], navKey: string): PhiCmsResolvedNavigationSurface;
-export declare function resolvePhiBuilderNavigationPageTargets(area: PhiCmsAreaKey, items: readonly PhiBuilderNavigationItem[], pages: readonly PhiPresetPageNode[]): PhiBuilderNavigationItem[];
+/**
+ * Each link's address, worked out in the Area its target actually lives in.
+ *
+ * An item may name another Area, and then that Area's catalog is the only one holding its Page and the
+ * only one that can prefix its path. Resolving everything against the edited Area is what made a
+ * cross-Area link read as unresolvable the moment it was dropped: the reference was right, the catalog
+ * looked in was the wrong one, and a Page that is simply absent looks exactly like a Page that is gone.
+ *
+ * `pagesForArea` rather than one list, because the caller is the only one that can reach another Area's
+ * catalog -- and nearly every call asks for the same Area twice, so the cost is a cached lookup.
+ */
+/**
+ * A Page catalog lookup by CMS Area, for a caller that has one keyed by Builder Area.
+ *
+ * The two lists carry the same names and are still two questions -- the Areas a Site has, and the Areas
+ * the Builder can open -- so the translation is stated rather than assumed. An Area the Builder does not
+ * open has no catalog to show, and an empty list is the honest answer: a link into it resolves to
+ * nothing, which is what the reader sees.
+ */
+export declare function createPhiBuilderNavigationPageLookup(pageCatalogForArea: (area: PhiBuilderAreaKey) => readonly PhiPresetPageNode[]): (area: PhiCmsAreaKey) => readonly PhiPresetPageNode[];
+export declare function resolvePhiBuilderNavigationPageTargets(area: PhiCmsAreaKey, items: readonly PhiBuilderNavigationItem[], pagesForArea: ((area: PhiCmsAreaKey) => readonly PhiPresetPageNode[]) | readonly PhiPresetPageNode[]): PhiBuilderNavigationItem[];