@designcrowd/library.brand-page 6.0.11 → 6.0.13

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

@@ -0,0 +1,122 @@
+import type { BrandPageModel, NavOrder } from '../../brand-page/models';
+import {
+  BaseMetadataBuilder,
+  type BuilderResult,
+  type MetadataBuilderRequest,
+} from '../models';
+import type { Logger } from '../logger';
+import { syncPagesWithNavOrders } from './nav-utils';
+
+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 class SetPageVisibilityBuilder extends BaseMetadataBuilder<SetPageVisibilityPayload> {
+  constructor(
+    request: MetadataBuilderRequest<SetPageVisibilityPayload>,
+    logger?: Logger,
+  ) {
+    super(request, logger);
+  }
+
+  async validateAsync(): Promise<{ errors: string[] }> {
+    const errors: string[] = [];
+
+    if (!this.request.payload?.pageId) {
+      errors.push('pageId is required for set_page_visibility');
+    }
+
+    return { errors };
+  }
+
+  async buildAsync(model: BrandPageModel): Promise<BuilderResult> {
+    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: NavOrder) =>
+      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/src/metadata-builder/metadata-executor.ts

@@ -31,6 +31,8 @@ import {
 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';
@@ -111,6 +113,8 @@ const getBuilderRegistry = (): Record<BuilderType, BuilderConstructor> => ({
   [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,

package/src/metadata-builder/models.ts

@@ -68,6 +68,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',

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

@@ -52,6 +52,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',
@@ -169,7 +171,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;
 }
 
@@ -187,8 +193,18 @@ export interface RenamePageArguments {
   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;
 }
 
 /**
@@ -202,8 +218,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;
 }
@@ -227,8 +252,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/src/metadata-builder/parsers/section-tool-arguments.ts

@@ -142,10 +142,15 @@ export interface ProfileSectionArguments {
 }
 
 export interface TextSectionItem {
-  /** 0-based index of existing item to update in-place */
-  item_index?: number;
-  /** 0-based position to insert a new item at */
-  insert_at?: number;
+  /** Stable id of the existing item to update or move. Omit to ADD a new item. */
+  item_id?: string;
+  /**
+   * Id of an existing item to position this item before (insert for new items,
+   * move for existing ones). Use "__end__" for the end of the list. Omit to
+   * leave an existing item in place / append a new item. Mirrors the
+   * section-level move_before_content_id.
+   */
+  move_before_item_id?: string;
   text?: RichTextField;
   text_style?: TextStyleArguments;
 }
@@ -159,11 +164,11 @@ export interface TextSectionArguments {
   /** Multiple text items (BrandSite) */
   items?: TextSectionItem[];
   _replace_items?: boolean;
-  /** 0-based indices of items to remove */
-  _remove_item_indices?: number[];
-  /** New order as array of current 0-based indices */
-  _reorder_items?: number[];
+  /** Stable ids of items to remove. Order-independent (no index shifting). */
+  _remove_item_ids?: string[];
   background_color?: string | null;
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface ImageSectionArguments {
@@ -172,10 +177,15 @@ export interface ImageSectionArguments {
 }
 
 export interface ImageGalleryItem {
-  /** 0-based index of existing item to update in-place */
-  item_index?: number;
-  /** 0-based position to insert a new item at */
-  insert_at?: number;
+  /** Stable id of the existing item to update or move. Omit to ADD a new item. */
+  item_id?: string;
+  /**
+   * Id of an existing item to position this item before (insert for new items,
+   * move for existing ones). Use "__end__" for the end of the list. Omit to
+   * leave an existing item in place / append a new item. Mirrors the
+   * section-level move_before_content_id.
+   */
+  move_before_item_id?: string;
   image_url?: string;
   caption?: string;
 }
@@ -185,10 +195,10 @@ export interface ImageGallerySectionArguments {
   gallery_type?: 'grid' | 'carousel' | 'masonry';
   is_full_width?: boolean;
   _replace_items?: boolean;
-  /** 0-based indices of items to remove */
-  _remove_item_indices?: number[];
-  /** New order as array of current 0-based indices */
-  _reorder_items?: number[];
+  /** Stable ids of items to remove. Order-independent (no index shifting). */
+  _remove_item_ids?: string[];
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface GeneralLinkSectionArguments {
@@ -256,10 +266,15 @@ export interface TiktokSectionArguments {
 }
 
 export interface SocialLink {
-  /** 0-based index of existing item to update in-place */
-  item_index?: number;
-  /** 0-based position to insert a new item at */
-  insert_at?: number;
+  /** Stable id of the existing item to update or move. Omit to ADD a new item. */
+  item_id?: string;
+  /**
+   * Id of an existing item to position this item before (insert for new items,
+   * move for existing ones). Use "__end__" for the end of the list. Omit to
+   * leave an existing item in place / append a new item. Mirrors the
+   * section-level move_before_content_id.
+   */
+  move_before_item_id?: string;
   platform?:
     | 'facebook'
     | 'x'
@@ -280,10 +295,10 @@ export interface SocialLinksSectionArguments {
   /** Style for the icon container background (used when icon_style is 'square' or 'circle'). */
   icon_background_style?: TextStyleArguments;
   _replace_items?: boolean;
-  /** 0-based indices of items to remove */
-  _remove_item_indices?: number[];
-  /** New order as array of current 0-based indices */
-  _reorder_items?: number[];
+  /** Stable ids of items to remove. Order-independent (no index shifting). */
+  _remove_item_ids?: string[];
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface MailingListSectionArguments {
@@ -292,6 +307,8 @@ export interface MailingListSectionArguments {
   accepted_fields?: 'emailOnly' | 'nameAndEmail';
   title_style?: TextStyleArguments;
   button_text_style?: ButtonTextStyleArguments;
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface ContactFormFieldToggle {
@@ -324,6 +341,8 @@ export interface ContactFormSectionArguments {
   fields?: ContactFormFieldToggle[];
   background_color?: string | null;
   card_style?: CardStyleArguments;
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface BannerSectionArguments {
@@ -350,6 +369,8 @@ export interface BannerSectionArguments {
   button_text_style?: ButtonTextStyleArguments;
   background_color?: string | null;
   card_style?: CardStyleArguments;
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface AboutSectionArguments {
@@ -368,13 +389,20 @@ export interface AboutSectionArguments {
   title_style?: TextStyleArguments;
   paragraph_style?: TextStyleArguments;
   background_color?: string | null;
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface ImagesSectionItem {
-  /** 0-based index of existing item to update in-place */
-  item_index?: number;
-  /** 0-based position to insert a new item at */
-  insert_at?: number;
+  /** Stable id of the existing item to update or move. Omit to ADD a new item. */
+  item_id?: string;
+  /**
+   * Id of an existing item to position this item before (insert for new items,
+   * move for existing ones). Use "__end__" for the end of the list. Omit to
+   * leave an existing item in place / append a new item. Mirrors the
+   * section-level move_before_content_id.
+   */
+  move_before_item_id?: string;
   image_url?: string;
   heading?: string;
   text?: string;
@@ -393,10 +421,8 @@ export interface ImagesSectionArguments {
     | 'center-card-aligned-image'
     | 'alternate-image';
   _replace_items?: boolean;
-  /** 0-based indices of items to remove */
-  _remove_item_indices?: number[];
-  /** New order as array of current 0-based indices */
-  _reorder_items?: number[];
+  /** Stable ids of items to remove. Order-independent (no index shifting). */
+  _remove_item_ids?: string[];
   title_style?: TextStyleArguments;
   description_style?: TextStyleArguments;
   heading_style?: TextStyleArguments;
@@ -409,10 +435,15 @@ export interface ImagesSectionArguments {
 }
 
 export interface TestimonialItem {
-  /** 0-based index of existing item to update in-place */
-  item_index?: number;
-  /** 0-based position to insert a new item at */
-  insert_at?: number;
+  /** Stable id of the existing item to update or move. Omit to ADD a new item. */
+  item_id?: string;
+  /**
+   * Id of an existing item to position this item before (insert for new items,
+   * move for existing ones). Use "__end__" for the end of the list. Omit to
+   * leave an existing item in place / append a new item. Mirrors the
+   * section-level move_before_content_id.
+   */
+  move_before_item_id?: string;
   testimonial?: string;
   name?: string;
   position?: string;
@@ -426,23 +457,28 @@ export interface TestimonialSectionArguments {
   items?: TestimonialItem[];
   display_style?: 'centered' | 'big-image' | 'columns';
   _replace_items?: boolean;
-  /** 0-based indices of items to remove */
-  _remove_item_indices?: number[];
-  /** New order as array of current 0-based indices */
-  _reorder_items?: number[];
+  /** Stable ids of items to remove. Order-independent (no index shifting). */
+  _remove_item_ids?: string[];
   title_style?: TextStyleArguments;
   testimonial_style?: TextStyleArguments;
   name_style?: TextStyleArguments;
   position_style?: TextStyleArguments;
   background_style?: BackgroundStyleArguments;
   card_style?: CardStyleArguments;
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface ShapesSectionItem {
-  /** 0-based index of existing item to update in-place */
-  item_index?: number;
-  /** 0-based position to insert a new item at */
-  insert_at?: number;
+  /** Stable id of the existing item to update or move. Omit to ADD a new item. */
+  item_id?: string;
+  /**
+   * Id of an existing item to position this item before (insert for new items,
+   * move for existing ones). Use "__end__" for the end of the list. Omit to
+   * leave an existing item in place / append a new item. Mirrors the
+   * section-level move_before_content_id.
+   */
+  move_before_item_id?: string;
   icon_image_url?: string;
   shape_url?: string;
   color_theme?: string;
@@ -462,10 +498,8 @@ export interface ShapesSectionArguments {
     | 'center-card-aligned-image'
     | 'alternate-image';
   _replace_items?: boolean;
-  /** 0-based indices of items to remove */
-  _remove_item_indices?: number[];
-  /** New order as array of current 0-based indices */
-  _reorder_items?: number[];
+  /** Stable ids of items to remove. Order-independent (no index shifting). */
+  _remove_item_ids?: string[];
   title_style?: TextStyleArguments;
   description_style?: TextStyleArguments;
   heading_style?: TextStyleArguments;
@@ -485,6 +519,8 @@ export interface BookingSectionArguments {
   description_style?: TextStyleArguments;
   button_text_style?: ButtonTextStyleArguments;
   background_color?: string | null;
+  padding_top?: number;
+  padding_bottom?: number;
 }
 
 export interface DividerSectionArguments {

package/src/shared/content-settings/components/links-field/LinksField.mixin.ts

@@ -11,6 +11,7 @@ import type {
   UpdateItemValuePayload,
 } from '../../models';
 import cloneDeep from 'lodash.clonedeep';
+import * as uuid from 'uuid';
 
 const ACCEPTED_MIME_TYPES = 'image/png, image/jpeg, image/svg+xml';
 
@@ -88,6 +89,7 @@ export default defineComponent({
     },
     addNewItem(): void {
       const newItem: LinksItem = {
+        id: uuid.v4(),
         height: 0,
         imageTitle: '',
         width: 0,

package/src/tracker.ts

@@ -0,0 +1,28 @@
+// Lightweight subpath entry: `@designcrowd/library.brand-page/tracker`.
+// Re-exports ONLY the analytics surface (useEventTracker + event constants),
+// which depends solely on `vue`. This lets consumers import the tracker without
+// pulling the full library graph (maker, TipTap, renderer). See
+// docs/plans/multi-entrypoint-tree-shaking.md and the /tracker proposal.
+//
+// IMPORTANT: useEventTracker holds module-scoped pub/sub state (LISTENERS,
+// GLOBAL_EVENT_PROPS). It MUST resolve to a single shared instance across this
+// entry and the root `.` entry, or subscribers registered here won't receive
+// events published via the root. The multi-entry build hoists it into a shared
+// chunk (both `index` and `tracker` import it) — verified at build time.
+export {
+  useEventTracker,
+  EVENTS,
+  SELECT_LOCATIONS,
+  UPGRADE_CTA_LOCATIONS,
+  AI_CTA_LOCATIONS,
+  TRIGGER_TYPE,
+  PUBLISH_MODAL_CONTINUATION_EVENTS,
+  GENERATE_TEXT_CONTINUATION_EVENTS,
+  PUBLISH_OPTION_MODAL_CONTINUATION_EVENTS,
+} from './brand-page-maker/composables/useEventTracker';
+export type {
+  EventTracker,
+  BpEventTrackerEvent,
+  BpGlobalEventProps,
+  BpUpsellEventProps,
+} from './brand-page-maker/composables/useEventTracker';

package/src/utils/minify-brand-page-metadata.ts

@@ -19,7 +19,12 @@ import {
   resolveLogoData,
 } from './build-asset-manifest';
 
-type MinifiedFooterSocialLink = { imageUrl: string; url: string };
+type MinifiedFooterSocialLink = {
+  position: number;
+  id?: string;
+  imageUrl: string;
+  url: string;
+};
 
 type MinifiedFooter = {
   displayStyle?: string;
@@ -284,6 +289,7 @@ const minifyContent = (
         ...(Array.isArray(c.items) && {
           items: c.items.map((item: any, index: number) => ({
             position: index + 1,
+            ...(item.id && { id: item.id }),
             ...st('name', item.name, editor),
             ...st('jobTitle', item.position, editor),
             ...st('testimonials', item.testimonials, editor),
@@ -312,6 +318,7 @@ const minifyContent = (
         ...(Array.isArray(c.items) && {
           items: c.items.map((item: any, index: number) => ({
             position: index + 1,
+            ...(item.id && { id: item.id }),
             ...includeImageWithKey(item.imageUrl),
             ...st('title', item.title, editor),
             ...st('paragraph', item.paragraph, editor),
@@ -342,6 +349,7 @@ const minifyContent = (
         ...(Array.isArray(c.textItems) && {
           textItems: c.textItems.map((item: any, index: number) => ({
             position: index + 1,
+            ...(item.id && { id: item.id }),
             ...st('text', item.text, editor),
             ...(includeStyle(item.textStyle) && { textStyle: item.textStyle }),
           })),
@@ -431,6 +439,7 @@ const minifyContent = (
         ...(Array.isArray(c.links) && {
           links: c.links.map((link: any, index: number) => ({
             position: index + 1,
+            ...(link.id && { id: link.id }),
             ...(includeString(link.url) && { url: link.url }),
             ...includeImageWithKey(link.imageUrl),
           })),
@@ -474,6 +483,7 @@ const minifyContent = (
         ...(Array.isArray(c.images) && {
           images: c.images.map((img: any, index: number) => ({
             position: index + 1,
+            ...(img.id && { id: img.id }),
             ...includeImageWithKey(img.imageUrl),
             ...st('imageTitle', img.imageTitle, editor),
           })),
@@ -523,7 +533,9 @@ const minifyFooter = (
     logoImageWidth: footer.logoImageWidth || undefined,
     ...(logoAssetKey && { logoAssetKey }),
     socialLinks: footer.socialLinks?.length
-      ? footer.socialLinks.map((l) => ({
+      ? footer.socialLinks.map((l, index) => ({
+          position: index + 1,
+          ...(l.id && { id: l.id }),
           imageUrl: l.imageUrl || '',
           url: l.url || '',
         }))

package/lib/src/metadata-builder/builders/reorder-pages-builder.d.ts

@@ -1,13 +0,0 @@
-import type { BrandPageModel } from '../../brand-page/models';
-import { BaseMetadataBuilder, type BuilderResult, type MetadataBuilderRequest } from '../models';
-import type { Logger } from '../logger';
-export interface ReorderPagesPayload {
-    pageOrder: string[];
-}
-export declare class ReorderPagesBuilder extends BaseMetadataBuilder<ReorderPagesPayload> {
-    constructor(request: MetadataBuilderRequest<ReorderPagesPayload>, logger?: Logger);
-    validateAsync(): Promise<{
-        errors: string[];
-    }>;
-    buildAsync(model: BrandPageModel): Promise<BuilderResult>;
-}

package/lib/src/metadata-builder/builders/reorder-pages-builder.js

@@ -1,49 +0,0 @@
-import { BaseMetadataBuilder, } from '../models';
-export class ReorderPagesBuilder extends BaseMetadataBuilder {
-    constructor(request, logger) {
-        super(request, logger);
-    }
-    async validateAsync() {
-        const errors = [];
-        const payload = this.request.payload;
-        if (!payload?.pageOrder?.length) {
-            errors.push('page_order is required for reorder_pages');
-        }
-        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 { pageOrder } = this.request.payload;
-        // Build pageId → routeId lookup from nav.routes
-        const pageIdToRouteId = new Map();
-        for (const route of model.nav.routes) {
-            pageIdToRouteId.set(route.pageId, route.id);
-        }
-        // Build new nav.orders from the flat page_order (array of pageIds)
-        const newOrders = [];
-        const seenRouteIds = new Set();
-        for (const pageId of pageOrder) {
-            const routeId = pageIdToRouteId.get(pageId);
-            if (!routeId)
-                continue;
-            newOrders.push({ routeId, subRoutes: [] });
-            seenRouteIds.add(routeId);
-        }
-        // Safety: append any routes not mentioned (don't lose pages)
-        for (const route of model.nav.routes) {
-            const routeId = pageIdToRouteId.get(route.pageId);
-            if (routeId && !seenRouteIds.has(routeId)) {
-                newOrders.push({ routeId, subRoutes: [] });
-            }
-        }
-        model.nav.orders = newOrders;
-        this.log({
-            message: `Reordered pages: ${pageOrder.join(', ')}`,
-        });
-        return { request: this.request, model };
-    }
-}