@designcrowd/library.brand-page 6.0.12 → 6.0.13

package/lib/src/metadata-builder/builders/section-builders/text-section-builder.js

@@ -3,6 +3,7 @@ import { isTiptapDocument, } from '../../parsers/section-tool-arguments';
 import { applyScalarFields, applyStyleFields, resolveGlobalPresetIntoStyle, } from '../section-field-utils';
 import { BaseSectionBuilder } from './base-section-builder';
 import { validateTextSection } from '../section-rules/text-section.rules';
+import { ensureItemIds, generateItemId, removeItemsByIds, repositionItemByAnchor, } from '../item-id-utils';
 /** Resolve a RichTextField (string or TiptapDocument) to a plain HTML string. */
 const resolveText = (value) => {
     if (isTiptapDocument(value))
@@ -16,55 +17,58 @@ const createTextItem = (item, baseStyle, textStyleOptions) => {
     const textStyle = (item.text_style ? { ...baseStyle, ...item.text_style } : { ...baseStyle });
     const { html, wasTiptap } = resolveText(item.text);
     return {
+        id: item.item_id || generateItemId(),
         text: html,
         textStyle: clearTextStyleTypeIfTiptap(textStyle, wasTiptap, textStyleOptions),
     };
 };
+/**
+ * Full replacement: discard existing items and rebuild from the payload,
+ * assigning a stable id to each new item.
+ */
 const applyTextItems = (content, items, textStyleOptions) => {
     content.textItems = items.map((item) => createTextItem(item, content.textStyle || {}, textStyleOptions));
 };
-const mergeTextItems = (content, items, replaceAll, textStyleOptions) => {
-    const hasIndexedItems = items.some((item) => item.item_index !== undefined);
-    if (hasIndexedItems) {
-        for (const item of items) {
-            const targetIndex = item.item_index;
-            if (targetIndex === undefined ||
-                targetIndex < 0 ||
-                !content.textItems?.[targetIndex])
+/**
+ * Id-based merge (BP-5603): mirrors the section-level id contract one level
+ * deeper.
+ *   - item with `item_id` matching an existing item → update in place and
+ *     reposition if `move_before_item_id` is set; unknown id is skipped.
+ *   - item without `item_id`                        → add a new item,
+ *     positioned by `move_before_item_id`.
+ *   - `removeIds`                                   → delete by id.
+ *
+ * Existing items are id-backfilled first so legacy sections without ids
+ * become referenceable the moment this builder processes them.
+ */
+const mergeTextItems = (content, items, removeIds, textStyleOptions) => {
+    ensureItemIds(content.textItems);
+    content.textItems = removeItemsByIds(content.textItems, removeIds);
+    for (const item of items) {
+        if (item.item_id) {
+            const existing = content.textItems?.find((i) => i.id === item.item_id);
+            // Unknown id — skip rather than guess at a position.
+            if (!existing)
                 continue;
-            const existing = content.textItems[targetIndex];
             if (item.text !== undefined) {
                 const { html, wasTiptap } = resolveText(item.text);
                 existing.text = html;
                 existing.textStyle = clearTextStyleTypeIfTiptap(existing.textStyle || {}, wasTiptap, textStyleOptions);
             }
             if (item.text_style && typeof item.text_style === 'object') {
-                const mergedStyle = (item.text_style
-                    ? { ...(existing.textStyle || {}), ...item.text_style }
-                    : existing.textStyle || {});
-                existing.textStyle = mergedStyle;
+                existing.textStyle = {
+                    ...(existing.textStyle || {}),
+                    ...item.text_style,
+                };
             }
+            repositionItemByAnchor(content.textItems, existing, item.move_before_item_id);
+        }
+        else {
+            const newItem = createTextItem(item, content.textStyle || {}, textStyleOptions);
+            content.textItems = content.textItems ?? [];
+            repositionItemByAnchor(content.textItems, newItem, item.move_before_item_id);
         }
-        return;
-    }
-    const itemsToMerge = items.filter((item) => item.insert_at === undefined);
-    const updatedItems = itemsToMerge.map((item, index) => {
-        const existing = content.textItems?.[index];
-        const baseStyle = existing?.textStyle || content.textStyle || {};
-        const mergedStyle = (item.text_style ? { ...baseStyle, ...item.text_style } : baseStyle);
-        const resolved = item.text !== undefined ? resolveText(item.text) : null;
-        const text = resolved ? resolved.html : (existing?.text ?? '');
-        return {
-            text,
-            textStyle: clearTextStyleTypeIfTiptap(mergedStyle, resolved?.wasTiptap ?? false, textStyleOptions),
-        };
-    });
-    if (!replaceAll &&
-        content.textItems &&
-        content.textItems.length > itemsToMerge.length) {
-        updatedItems.push(...content.textItems.slice(itemsToMerge.length));
     }
-    content.textItems = updatedItems;
 };
 /**
  * Handles the single `text` + `text_style` fields (BrandPage/BrandContact
@@ -83,6 +87,7 @@ const applySingleTextField = (content, args, textStyleOptions) => {
         else {
             content.textItems = [
                 {
+                    id: generateItemId(),
                     text: html,
                     textStyle: clearTextStyleTypeIfTiptap({ ...(content.textStyle || {}) }, wasTiptap, textStyleOptions),
                 },
@@ -121,56 +126,27 @@ export class TextSectionBuilder extends BaseSectionBuilder {
         applyScalarFields(content, this.args, this.sectionType);
         applyStyleFields(content, this.args, this.sectionType, this.getGlobalTextStyles(model));
         const args = this.args;
-        if (args.items && Array.isArray(args.items)) {
-            if (this.mode === 'create') {
+        if (args._replace_items && args.items && Array.isArray(args.items)) {
+            // Full replacement: discard existing items and rebuild from the payload.
+            applyTextItems(content, args.items, textStyleOptions);
+        }
+        else if (this.mode === 'create') {
+            if (args.items && Array.isArray(args.items)) {
                 applyTextItems(content, args.items, textStyleOptions);
             }
             else {
-                mergeTextItems(content, args.items, !!args._replace_items, textStyleOptions);
+                applySingleTextField(content, args, textStyleOptions);
             }
         }
+        else if ((args.items && Array.isArray(args.items)) ||
+            args._remove_item_ids?.length) {
+            // Update: id-based add / edit / move / remove in a single pass.
+            mergeTextItems(content, args.items ?? [], args._remove_item_ids, textStyleOptions);
+        }
         else {
+            // No items array: apply the single text/text_style shorthand fields.
             applySingleTextField(content, args, textStyleOptions);
         }
-        // Phase 2: Item operations (remove, insert, reorder)
-        if (args._remove_item_indices?.length && content.textItems) {
-            const sorted = [...args._remove_item_indices].sort((a, b) => b - a);
-            for (const idx of sorted) {
-                if (idx >= 0 && idx < content.textItems.length) {
-                    content.textItems.splice(idx, 1);
-                }
-            }
-        }
-        if (args.items && Array.isArray(args.items)) {
-            const insertItems = args.items.filter((item) => item.insert_at !== undefined && item.item_index === undefined);
-            const sortedInserts = [...insertItems].sort((a, b) => (b.insert_at ?? 0) - (a.insert_at ?? 0));
-            for (const item of sortedInserts) {
-                const pos = Math.max(0, Math.min(item.insert_at ?? 0, content.textItems?.length ?? 0));
-                const baseStyle = content.textStyle || {};
-                const textStyle = (item.text_style
-                    ? { ...baseStyle, ...item.text_style }
-                    : { ...baseStyle });
-                const { html, wasTiptap } = resolveText(item.text);
-                const newItem = {
-                    text: html,
-                    textStyle: clearTextStyleTypeIfTiptap(textStyle, wasTiptap, textStyleOptions),
-                };
-                content.textItems = content.textItems ?? [];
-                content.textItems.splice(pos, 0, newItem);
-            }
-        }
-        if (args._reorder_items?.length && content.textItems) {
-            const newOrder = args._reorder_items;
-            const unique = new Set(newOrder);
-            if (unique.size === newOrder.length) {
-                const reordered = newOrder
-                    .filter((idx) => idx >= 0 && idx < content.textItems.length)
-                    .map((idx) => content.textItems[idx]);
-                if (reordered.length === content.textItems.length) {
-                    content.textItems = reordered;
-                }
-            }
-        }
         if (args.background_color !== undefined) {
             content.backgroundColor = {
                 ...content.backgroundColor,

package/lib/src/metadata-builder/builders/section-field-maps.js

@@ -12,6 +12,12 @@
 function defineFieldMap(map) {
     return map;
 }
+// BP-5687: vertical padding fields shared by all sections that render
+// contentPaddingStyle (see src/brand-page/utils/content-padding.util.ts).
+const PADDING_FIELDS = {
+    padding_top: 'topPadding',
+    padding_bottom: 'bottomPadding',
+};
 const bannerFields = defineFieldMap({
     title: 'title',
     description: 'description',
@@ -20,12 +26,14 @@ const bannerFields = defineFieldMap({
     background_image_url: 'imageUrl',
     background_shape_url: 'imageUrl',
     display_style: 'displayStyle',
+    ...PADDING_FIELDS,
 });
 const bookingFields = defineFieldMap({
     title: 'title',
     description: 'description',
     button_text: 'buttonText',
     button_url: 'url',
+    ...PADDING_FIELDS,
 });
 const aboutFields = defineFieldMap({
     title: 'title',
@@ -33,11 +41,13 @@ const aboutFields = defineFieldMap({
     about_image_url: 'imageUrl',
     about_shape_url: 'imageUrl',
     display_style: 'displayStyle',
+    ...PADDING_FIELDS,
 });
 const testimonialFields = defineFieldMap({
     title: 'title',
     display_style: 'displayStyle',
     // items handled separately
+    ...PADDING_FIELDS,
 });
 const imagesSectionFields = defineFieldMap({
     title: 'title',
@@ -46,6 +56,7 @@ const imagesSectionFields = defineFieldMap({
     aspect_ratio: 'aspectRatio',
     are_images_full_width: 'areImagesFullWidth',
     // items handled separately
+    ...PADDING_FIELDS,
 });
 const shapesSectionFields = defineFieldMap({
     title: 'title',
@@ -53,10 +64,12 @@ const shapesSectionFields = defineFieldMap({
     display_style: 'displayStyle',
     size: 'size',
     // items handled separately
+    ...PADDING_FIELDS,
 });
 const textFields = defineFieldMap({
     text: 'text',
     column_style: 'columnStyle',
+    ...PADDING_FIELDS,
 });
 const profileFields = defineFieldMap({
     name: 'name',
@@ -73,15 +86,18 @@ const contactFormFields = defineFieldMap({
     header_image_url: 'imageUrl',
     display_style: 'displayStyle',
     layout_style: 'layoutStyle',
+    ...PADDING_FIELDS,
 });
 const mailingListFields = defineFieldMap({
     button_text: 'buttonText',
     accepted_fields: 'acceptedFields',
+    ...PADDING_FIELDS,
 });
 const socialLinksFields = defineFieldMap({
     title: 'title',
     icon_style: 'iconDisplayStyle',
     // links handled separately
+    ...PADDING_FIELDS,
 });
 const dividerFields = defineFieldMap({
     divider_style: 'dividerStyle',
@@ -94,6 +110,7 @@ const imageFields = defineFieldMap({
 const imageGalleryFields = defineFieldMap({
     is_full_width: 'isFullWidth',
     // images handled separately
+    ...PADDING_FIELDS,
 });
 const buttonWithIconFields = defineFieldMap({
     title: 'title',

package/lib/src/metadata-builder/builders/set-page-visibility-builder.d.ts

@@ -0,0 +1,23 @@
+import type { BrandPageModel } from '../../brand-page/models';
+import { BaseMetadataBuilder, type BuilderResult, type MetadataBuilderRequest } from '../models';
+import type { Logger } from '../logger';
+export interface SetPageVisibilityPayload {
+    /** The page whose visibility is toggled (resolved from page_id). */
+    pageId: string;
+    /** true = hide the page from the navigation; false = show it. */
+    isHidden: boolean;
+}
+/**
+ * Toggle a page's navigation visibility (the AI-chat equivalent of the page-settings
+ * "Hide page / Show page" menu). Sets `isHidden` on BOTH the matching `NavRoute`
+ * (in `model.nav.routes`) and the `PageModel` (in `model.pages`) — the two places the
+ * editor keeps in sync. No slug/order recompute (the nav order is unchanged); the
+ * published-site nav already filters hidden pages via `isHidden`.
+ */
+export declare class SetPageVisibilityBuilder extends BaseMetadataBuilder<SetPageVisibilityPayload> {
+    constructor(request: MetadataBuilderRequest<SetPageVisibilityPayload>, logger?: Logger);
+    validateAsync(): Promise<{
+        errors: string[];
+    }>;
+    buildAsync(model: BrandPageModel): Promise<BuilderResult>;
+}

package/lib/src/metadata-builder/builders/set-page-visibility-builder.js

@@ -0,0 +1,83 @@
+import { BaseMetadataBuilder, } from '../models';
+import { syncPagesWithNavOrders } from './nav-utils';
+/**
+ * Toggle a page's navigation visibility (the AI-chat equivalent of the page-settings
+ * "Hide page / Show page" menu). Sets `isHidden` on BOTH the matching `NavRoute`
+ * (in `model.nav.routes`) and the `PageModel` (in `model.pages`) — the two places the
+ * editor keeps in sync. No slug/order recompute (the nav order is unchanged); the
+ * published-site nav already filters hidden pages via `isHidden`.
+ */
+export class SetPageVisibilityBuilder extends BaseMetadataBuilder {
+    constructor(request, logger) {
+        super(request, logger);
+    }
+    async validateAsync() {
+        const errors = [];
+        if (!this.request.payload?.pageId) {
+            errors.push('pageId is required for set_page_visibility');
+        }
+        return { errors };
+    }
+    async buildAsync(model) {
+        this.log('Build called');
+        const validation = await this.validateAsync();
+        if (validation.errors.length) {
+            return { request: this.request, model, errors: validation.errors };
+        }
+        const { pageId, isHidden } = this.request.payload;
+        const targetRoute = model.nav.routes.find((r) => r.pageId === pageId);
+        if (!targetRoute) {
+            return {
+                request: this.request,
+                model,
+                errors: [`Page with ID "${pageId}" not found`],
+            };
+        }
+        // Guards apply only when HIDING; showing a page is always safe.
+        if (isHidden) {
+            // --- GUARD: the home page (orders[0]) cannot be hidden ---
+            const homeRouteId = model.nav.orders[0]?.routeId;
+            if (homeRouteId === targetRoute.id) {
+                return {
+                    request: this.request,
+                    model,
+                    errors: [
+                        `Home page "${pageId}" cannot be hidden from the navigation`,
+                    ],
+                };
+            }
+            // --- GUARD: cannot hide the last remaining visible page ---
+            const visibleCount = model.nav.routes.filter((r) => !r.isHidden).length;
+            if (!targetRoute.isHidden && visibleCount <= 1) {
+                return {
+                    request: this.request,
+                    model,
+                    errors: [
+                        `Cannot hide "${pageId}" — at least one page must stay visible`,
+                    ],
+                };
+            }
+        }
+        // Set on the NavRoute (drives the nav) and the PageModel (drives header page lists).
+        targetRoute.isHidden = isHidden;
+        const targetPage = model.pages.find((p) => p.id === pageId);
+        if (targetPage) {
+            targetPage.isHidden = isHidden;
+        }
+        // Match the editor's page-navigation store (reorderVisibleAndHiddenRoutes): after any
+        // visibility change, float visible pages to the top and sink hidden pages to the bottom
+        // of the top-level nav order, stable within each group. Top-level only — mirrors the
+        // editor, which does not re-partition nested subRoutes. `model.pages` is then re-synced
+        // to the new DFS order (same as MovePageBuilder).
+        const routeById = new Map(model.nav.routes.map((r) => [r.id, r]));
+        const isOrderHidden = (o) => Boolean(routeById.get(o.routeId)?.isHidden);
+        const visibleOrders = model.nav.orders.filter((o) => !isOrderHidden(o));
+        const hiddenOrders = model.nav.orders.filter((o) => isOrderHidden(o));
+        model.nav.orders = [...visibleOrders, ...hiddenOrders];
+        syncPagesWithNavOrders(model);
+        this.log({
+            message: `Set page "${pageId}" visibility — isHidden: ${isHidden}`,
+        });
+        return { request: this.request, model };
+    }
+}

package/lib/src/metadata-builder/metadata-executor.js

@@ -7,6 +7,8 @@ import { BannerSectionBuilder, BookingSectionBuilder, AboutSectionBuilder, Conta
 import { AddPageBuilder } from './builders/add-page-builder';
 import { RemovePageBuilder } from './builders/remove-page-builder';
 import { RenamePageBuilder } from './builders/rename-page-builder';
+import { MovePageBuilder } from './builders/move-page-builder';
+import { SetPageVisibilityBuilder } from './builders/set-page-visibility-builder';
 import { FooterBuilder } from './builders/footer-builder';
 import { GlobalStylesBuilder } from './builders/global-styles-builder';
 import { HeaderBuilder } from './builders/header-builder';
@@ -74,6 +76,8 @@ const getBuilderRegistry = () => ({
     [BUILDER_TYPES.ADD_PAGE]: AddPageBuilder,
     [BUILDER_TYPES.REMOVE_PAGE]: RemovePageBuilder,
     [BUILDER_TYPES.RENAME_PAGE]: RenamePageBuilder,
+    [BUILDER_TYPES.MOVE_PAGE]: MovePageBuilder,
+    [BUILDER_TYPES.SET_PAGE_VISIBILITY]: SetPageVisibilityBuilder,
     // Footer operations
     [BUILDER_TYPES.UPDATE_FOOTER]: FooterBuilder,
     // Global styles operations

package/lib/src/metadata-builder/models.d.ts

@@ -59,6 +59,8 @@ export declare const BUILDER_TYPES: {
     readonly ADD_PAGE: "add-page";
     readonly REMOVE_PAGE: "remove-page";
     readonly RENAME_PAGE: "rename-page";
+    readonly MOVE_PAGE: "move-page";
+    readonly SET_PAGE_VISIBILITY: "set-page-visibility";
     readonly UPDATE_FOOTER: "update-footer";
     readonly UPDATE_GLOBAL_STYLES: "update-global-styles";
     readonly UPDATE_HEADER: "update-header";

package/lib/src/metadata-builder/models.js

@@ -61,6 +61,8 @@ export const BUILDER_TYPES = {
     ADD_PAGE: 'add-page',
     REMOVE_PAGE: 'remove-page',
     RENAME_PAGE: 'rename-page',
+    MOVE_PAGE: 'move-page',
+    SET_PAGE_VISIBILITY: 'set-page-visibility',
     // Footer operations
     UPDATE_FOOTER: 'update-footer',
     // Global styles operations

package/lib/src/metadata-builder/parsers/ai-toolcall-parser.d.ts

@@ -35,6 +35,8 @@ export declare const AI_ASSISTANT_TOOL_NAMES: {
     readonly ADD_PAGE: "add_page";
     readonly REMOVE_PAGE: "remove_page";
     readonly RENAME_PAGE: "rename_page";
+    readonly MOVE_PAGE: "move_page";
+    readonly SET_PAGE_VISIBILITY: "set_page_visibility";
     readonly UPDATE_FOOTER: "update_footer";
     readonly UPDATE_GLOBAL_STYLES: "tool_update_global_styles";
     readonly UPDATE_HEADER: "update_header";
@@ -72,7 +74,11 @@ export interface DeleteSectionArguments {
 }
 export interface MoveSectionArguments {
     content_id: string;
-    position: number;
+    /**
+     * Relative anchor on the destination page: the content_id to insert this section
+     * BEFORE (a page's first content_id moves it to the top). Omit to append to the end.
+     */
+    move_before_content_id?: string;
     target_page_id?: string;
 }
 export interface AddPageArguments {
@@ -86,8 +92,17 @@ export interface RenamePageArguments {
     page_id: string;
     page_name: string;
 }
-export interface ReorderPagesArguments {
-    page_order: string[];
+export interface MovePageArguments {
+    page_id: string;
+    /** Destination parent page_id, or null/omitted for top-level. */
+    parent_page_id?: string | null;
+    /** Sibling anchor page_id to insert before, or "__end__"/omitted to append. */
+    move_before_page_id?: string | null;
+}
+export interface SetPageVisibilityArguments {
+    page_id: string;
+    /** true = hide the page from the navigation; false = show it. */
+    is_hidden: boolean;
 }
 /**
  * Arguments for the `undo` tool call.
@@ -99,8 +114,17 @@ export interface ReorderPagesArguments {
 export interface UndoArguments {
 }
 export interface FooterSocialLinkItem {
-    item_index?: number;
-    insert_at?: number;
+    /**
+     * Stable id of the existing link to update or move. Omit to ADD a new link.
+     * Mirrors the section-level item_id convention (BP-5603).
+     */
+    item_id?: string;
+    /**
+     * Id of an existing link to position this link before (insert for new links,
+     * move for existing ones). Use "__end__" for the end of the list. Omit to
+     * leave an existing link in place / append a new link.
+     */
+    move_before_item_id?: string;
     social_link_icon?: string;
     url?: string;
 }
@@ -122,8 +146,8 @@ export interface UpdateFooterArguments {
     copyright_text_style?: TextStyleArguments;
     social_links?: FooterSocialLinkItem[];
     _replace_social_links?: boolean;
-    _remove_social_link_indices?: number[];
-    _reorder_social_links?: number[];
+    /** Stable ids of social links to remove. Order-independent (replaces index-based _remove_social_link_indices). */
+    _remove_social_link_ids?: string[];
     /** Desktop-only styling (display_style, background_color, icon_color, is_logo_visible, logo sizes). */
     desktop?: ViewportFooterOverrides;
     /** Mobile-only styling (display_style, background_color, icon_color, is_logo_visible, logo sizes). */

package/lib/src/metadata-builder/parsers/ai-toolcall-parser.js

@@ -41,6 +41,8 @@ export const AI_ASSISTANT_TOOL_NAMES = {
     ADD_PAGE: 'add_page',
     REMOVE_PAGE: 'remove_page',
     RENAME_PAGE: 'rename_page',
+    MOVE_PAGE: 'move_page',
+    SET_PAGE_VISIBILITY: 'set_page_visibility',
     // Footer operation tools
     UPDATE_FOOTER: 'update_footer',
     // Global styles operation tools