@designcrowd/library.brand-page 6.0.6-json-prereq-7 → 6.0.6-undo-2

package/lib/src/brand-page-maker/components/seo-settings/SeoPageSettings.mixin.d.ts

@@ -391,7 +391,7 @@ declare const _default: import("vue").DefineComponent<import("vue").ExtractPropT
             content: TContent;
         }>(payload: P): Promise<void>;
         _captureTemplateSwitchIfEnabled(): void;
-        applyHistorySnapshot(snapshot: import("../../../brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string): Promise<void>;
+        applyHistorySnapshot(snapshot: import("../../../brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string, selectedTemplateId?: string | false): Promise<void>;
         _syncLogoSettingsFromSnapshot(snapshot: import("../../../brand-page").BrandPageModel, logoEditsStr?: string): Promise<void>;
         _navigateToActivePage(activePageId?: string): void;
         _fallbackSidebarIfSelectedContentMissing(snapshot: import("../../../brand-page").BrandPageModel): void;

package/lib/src/brand-page-maker/components/seo-settings/UrlSettings.mixin.d.ts

@@ -392,7 +392,7 @@ declare const _default: import("vue").DefineComponent<import("vue").ExtractPropT
             content: TContent;
         }>(payload: P): Promise<void>;
         _captureTemplateSwitchIfEnabled(): void;
-        applyHistorySnapshot(snapshot: import("../../../brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string): Promise<void>;
+        applyHistorySnapshot(snapshot: import("../../../brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string, selectedTemplateId?: string | false): Promise<void>;
         _syncLogoSettingsFromSnapshot(snapshot: import("../../../brand-page").BrandPageModel, logoEditsStr?: string): Promise<void>;
         _navigateToActivePage(activePageId?: string): void;
         _fallbackSidebarIfSelectedContentMissing(snapshot: import("../../../brand-page").BrandPageModel): void;

package/lib/src/brand-page-maker/components/social-share/SocialShare.mixin.d.ts

@@ -123,7 +123,7 @@ declare const _default: import("vue").DefineComponent<import("vue").ExtractPropT
             content: TContent;
         }>(payload: P): Promise<void>;
         _captureTemplateSwitchIfEnabled(): void;
-        applyHistorySnapshot(snapshot: import("../../../brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string): Promise<void>;
+        applyHistorySnapshot(snapshot: import("../../../brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string, selectedTemplateId?: string | false): Promise<void>;
         _syncLogoSettingsFromSnapshot(snapshot: import("../../../brand-page").BrandPageModel, logoEditsStr?: string): Promise<void>;
         _navigateToActivePage(activePageId?: string): void;
         _fallbackSidebarIfSelectedContentMissing(snapshot: import("../../../brand-page").BrandPageModel): void;

package/lib/src/brand-page-maker/components/template-preview/TemplatePreview.mixin.d.ts

@@ -441,7 +441,7 @@ declare const _default: import("vue").DefineComponent<import("vue").ExtractPropT
             content: TContent;
         }>(payload: P): Promise<void>;
         _captureTemplateSwitchIfEnabled(): void;
-        applyHistorySnapshot(snapshot: import("@brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string): Promise<void>;
+        applyHistorySnapshot(snapshot: import("@brand-page").BrandPageModel, activePageId?: string, logoEditsStr?: string, selectedTemplateId?: string | false): Promise<void>;
         _syncLogoSettingsFromSnapshot(snapshot: import("@brand-page").BrandPageModel, logoEditsStr?: string): Promise<void>;
         _navigateToActivePage(activePageId?: string): void;
         _fallbackSidebarIfSelectedContentMissing(snapshot: import("@brand-page").BrandPageModel): void;

package/lib/src/brand-page-maker/composables/rich-text-editor/useRichTextEditor.js

@@ -1,25 +1,72 @@
 import { onBeforeUnmount, computed, watch, } from 'vue';
 import { Editor } from '@tiptap/vue-3';
-import { getRteExtensions } from './getRteExtensions';
+import StarterKit from '@tiptap/starter-kit';
+import Underline from '@tiptap/extension-underline';
+import TextStyle from '@tiptap/extension-text-style';
+import TextAlign from '@tiptap/extension-text-align';
+import FontFamily from '@tiptap/extension-font-family';
+import ColorExtension from '@tiptap/extension-color';
+import { FontSize } from './extensions/FontSize';
+import { LetterSpacing } from './extensions/LetterSpacing';
+import { LineHeight } from './extensions/LineHeight';
+import Highlight from '@tiptap/extension-highlight';
+import Placeholder from '@tiptap/extension-placeholder';
+import Link from '@tiptap/extension-link';
 import { createFormattingActions } from './formatting';
 import { useTextAlign } from './textAlign';
 import { useFont } from './font';
 import { useSpacing } from './spacing';
 import Color from 'color';
-import { buildInitialMarks, emToPx, isStyleChanged, pxToEm, sanitizeRteJson, skipStyleDetection, } from '@brand-page-maker/utils';
+import { buildInitialMarks, emToPx, isStyleChanged, pxToEm, sanitiseClientSideRTE, skipStyleDetection, } from '@brand-page-maker/utils';
 import { HYPERLINK_TYPE, } from '@brand-page';
 import { letterSpacingValueConvertor, colorConvertor, backgroundColorConvertor, } from '@brand-page-maker/convertors';
 import { useUiContentSettingsStore } from '@shared/content-settings/store';
 import { useHistoryStore } from '@brand-page-maker/stores/history/history-store';
 import { AllSelection } from 'prosemirror-state';
-import { useHyperlink } from './hyperlink';
+import { MaxLength } from './extensions/MaxLength';
+import { AutoLink } from './extensions/AutoLink';
+import { isHrefPageId, useHyperlink } from './hyperlink';
 import { useBubbleMenu } from './bubbleMenu';
 import { hasLogoColorVar, isLogoColor, getRTEColor, } from '@brand-page-maker/utils/rich-text-editor.util';
 import { useEventTracker, EVENTS } from '../useEventTracker';
 export const useExternalRichTextEditor = () => {
     const externalEditor = new Editor({
         editable: false,
-        extensions: getRteExtensions({ includeLinkRel: true }),
+        extensions: [
+            StarterKit,
+            Underline,
+            TextStyle.configure({ mergeNestedSpanStyles: true }),
+            TextAlign.configure({
+                types: ['heading', 'paragraph'],
+            }),
+            FontFamily.configure({
+                types: ['textStyle'],
+            }),
+            FontSize,
+            LetterSpacing,
+            LineHeight,
+            ColorExtension.configure({
+                types: ['textStyle'],
+            }),
+            Highlight.configure({
+                multicolor: true,
+            }),
+            Link.configure({
+                openOnClick: false,
+                defaultProtocol: 'https',
+                autolink: true,
+                HTMLAttributes: (attrs) => {
+                    const href = attrs.href;
+                    const isExternal = href && (href.startsWith('http://') || href.startsWith('https://'));
+                    return {
+                        rel: 'noopener noreferrer',
+                        target: isExternal ? '_blank' : null,
+                    };
+                },
+                isAllowedUri: (url, ctx) => ctx.defaultValidate(url) || isHrefPageId(url),
+            }),
+            AutoLink,
+        ],
     });
     return {
         externalEditor,
@@ -33,13 +80,51 @@ export const useRichTextEditor = (model, props, emit, fontSettings, fontStyleDro
     const editor = new Editor({
         content: model.value ?? '',
         editable: !props.readOnly,
-        extensions: getRteExtensions({
-            restrictStarterKit: true,
-            editing: {
+        extensions: [
+            StarterKit.configure({
+                bulletList: false,
+                orderedList: false,
+                listItem: false,
+                horizontalRule: false,
+                codeBlock: false,
+                code: false,
+            }),
+            Underline,
+            TextStyle.configure({ mergeNestedSpanStyles: true }),
+            TextAlign.configure({
+                types: ['heading', 'paragraph'],
+            }),
+            FontFamily.configure({
+                types: ['textStyle'],
+            }),
+            FontSize,
+            LetterSpacing,
+            LineHeight,
+            ColorExtension.configure({
+                types: ['textStyle'],
+            }),
+            Highlight.configure({
+                multicolor: true,
+            }),
+            Placeholder.configure({
                 placeholder: props.placeholder,
-                maxLength: props.maxLength,
-            },
-        }),
+            }),
+            MaxLength.configure({ maxLength: props.maxLength }),
+            Link.configure({
+                openOnClick: false,
+                defaultProtocol: 'https',
+                autolink: true,
+                HTMLAttributes: (attrs) => {
+                    const href = attrs.href;
+                    const isExternal = href && (href.startsWith('http://') || href.startsWith('https://'));
+                    return {
+                        target: isExternal ? '_blank' : null,
+                    };
+                },
+                isAllowedUri: (url, ctx) => ctx.defaultValidate(url) || isHrefPageId(url),
+            }),
+            AutoLink,
+        ],
         editorProps: {
             handleDOMEvents: {
                 drop(view, event) {
@@ -157,11 +242,7 @@ export const useRichTextEditor = (model, props, emit, fontSettings, fontStyleDro
         const appliedSinceLast = historyStore.snapshotAppliedToken !== lastAppliedToken;
         lastAppliedToken = historyStore.snapshotAppliedToken;
         const isExternalMutation = updateRichTextEditor.value || appliedSinceLast;
-        // Compare in the stored (JSON) format so a JSON model value isn't always
-        // seen as "changed" against the editor's HTML (which would re-run
-        // setContent and reset the user's just-applied styles).
-        const hasChanged = JSON.stringify(editor.getJSON()) !== JSON.stringify(newValue);
-        if (isExternalMutation && hasChanged) {
+        if (isExternalMutation && editor.getHTML() !== newValue) {
             editor.commands.setContent(newValue, false);
             updateRichTextEditor.value = false;
             //text style type is not switching when reordering
@@ -182,12 +263,26 @@ export const useRichTextEditor = (model, props, emit, fontSettings, fontStyleDro
             });
         }
     });
-    // Update model when editor content changes. RTE content is stored as Tiptap
-    // JSON (BP-5787). getJSON() is already in hand (the editor holds the doc) and
-    // is sanitised natively — no getHTML(), no DOMPurify, no extra editor — so
-    // the per-keystroke path allocates nothing heavy.
+    // Update model when editor content changes
     editor.on('update', () => {
-        emit('update:modelValue', sanitizeRteJson(editor.getJSON()));
+        const sanitizedData = sanitiseClientSideRTE(editor.getHTML());
+        // An update fired while the editor is unfocused is TipTap rewriting the
+        // model back into its canonical HTML during mount rehydration or a
+        // programmatic setContent — not a user edit. Fold it into the current
+        // head so merely selecting a section doesn't push a phantom undo entry
+        // (BP-5810). The construction-time arm above lapses before this async
+        // write-back fires, so re-arm here. Genuine edits happen while focused and
+        // still push a new entry.
+        //
+        // Known limitation: a programmatic toolbar command run on an editor the
+        // user hasn't focused yet would also merge (losing that undo step). Today
+        // the RTE only mounts inside an already-open settings drawer and the user
+        // focuses it before formatting, so this doesn't occur — revisit if a
+        // consumer ever drives toolbar actions on an unfocused editor.
+        if (!editor.isFocused && !historyStore.isApplyingSnapshot) {
+            historyStore.mergeNextMutation();
+        }
+        emit('update:modelValue', sanitizedData.sanitizedHtml);
     });
     editor.on('selectionUpdate', () => {
         // Skip while a history snapshot is being applied — `setContent` fires

package/lib/src/brand-page-maker/composables/useUndoRedoFlag.d.ts

@@ -1,16 +1,15 @@
 import { type ComputedRef } from 'vue';
 /**
- * Returns whether undo/redo is active for the current maker session. All
- * three conditions must hold:
+ * Returns whether undo/redo is active for the current maker session. Both
+ * conditions must hold:
  *
  *   1. `userStore.features.canUseUndoRedo` is true. The flag is populated by
  *      the consumer (BrandCrowd.Net / web-renderer) via the
  *      `WebsiteDesignUserFeatures.CanUseUndoRedo` boolean on the ViewModel —
- *      same precedent as RichTextEditor / GettyVideos / AITextGenerator.
- *   2. The maker is a brand-site (Website Maker). V1 scope is brand-site
- *      only; brand-page (single-page profile) and brand-contact (vCard) are
- *      out of scope.
- *   3. The session is NOT in admin-maker mode (`userStore.isAdmin`). The
+ *      same precedent as RichTextEditor / GettyVideos / AITextGenerator. The
+ *      one flag governs all three brands (brand-site, brand-page,
+ *      brand-contact) — there is no per-brand gating.
+ *   2. The session is NOT in admin-maker mode (`userStore.isAdmin`). The
  *      admin maker uses the same components but a different editing flow
  *      (template authoring) where undo/redo is out of scope.
  */

package/lib/src/brand-page-maker/composables/useUndoRedoFlag.js

@@ -1,25 +1,20 @@
 import { computed } from 'vue';
 import { useBrandPageMakerUserStore } from '@brand-page-maker/stores/user';
-import { useBrandPageMakerStore } from '@brand-page-maker/stores/brandpage-maker/brandpage-maker-store';
 /**
- * Returns whether undo/redo is active for the current maker session. All
- * three conditions must hold:
+ * Returns whether undo/redo is active for the current maker session. Both
+ * conditions must hold:
  *
  *   1. `userStore.features.canUseUndoRedo` is true. The flag is populated by
  *      the consumer (BrandCrowd.Net / web-renderer) via the
  *      `WebsiteDesignUserFeatures.CanUseUndoRedo` boolean on the ViewModel —
- *      same precedent as RichTextEditor / GettyVideos / AITextGenerator.
- *   2. The maker is a brand-site (Website Maker). V1 scope is brand-site
- *      only; brand-page (single-page profile) and brand-contact (vCard) are
- *      out of scope.
- *   3. The session is NOT in admin-maker mode (`userStore.isAdmin`). The
+ *      same precedent as RichTextEditor / GettyVideos / AITextGenerator. The
+ *      one flag governs all three brands (brand-site, brand-page,
+ *      brand-contact) — there is no per-brand gating.
+ *   2. The session is NOT in admin-maker mode (`userStore.isAdmin`). The
  *      admin maker uses the same components but a different editing flow
  *      (template authoring) where undo/redo is out of scope.
  */
 export function useUndoRedoFlag() {
     const userStore = useBrandPageMakerUserStore();
-    const brandPageMakerStore = useBrandPageMakerStore();
-    return computed(() => userStore.features.canUseUndoRedo &&
-        brandPageMakerStore.isBrandSite &&
-        !userStore.isAdmin);
+    return computed(() => userStore.features.canUseUndoRedo && !userStore.isAdmin);
 }

package/lib/src/brand-page-maker/stores/brandpage-maker/brandpage-maker-store.d.ts

@@ -211,7 +211,7 @@ export declare const useBrandPageMakerStore: import("pinia").StoreDefinition<"br
      * feature flag is on — see `BrandPageMaker.mixin.ts`).
      */
     _captureTemplateSwitchIfEnabled(): void;
-    applyHistorySnapshot(snapshot: BrandPageModel, activePageId?: string, logoEditsStr?: string): Promise<void>;
+    applyHistorySnapshot(snapshot: BrandPageModel, activePageId?: string, logoEditsStr?: string, selectedTemplateId?: string | false): Promise<void>;
     _syncLogoSettingsFromSnapshot(snapshot: BrandPageModel, logoEditsStr?: string): Promise<void>;
     /**
      * Restore the active page after a history apply. The id passed in is

package/lib/src/brand-page-maker/stores/brandpage-maker/brandpage-maker-store.js

@@ -494,7 +494,7 @@ export const useBrandPageMakerStore = defineStore('brandPageMaker', {
                 return;
             historyStore.captureTemplateSwitch();
         },
-        async applyHistorySnapshot(snapshot, activePageId, logoEditsStr) {
+        async applyHistorySnapshot(snapshot, activePageId, logoEditsStr, selectedTemplateId) {
             const styleSettingsStore = useUiStyleSettingsStore();
             const navigationStyleSettingsStore = useUiNavigationStyleSettingsStore();
             const backgroundSettingsStore = useUiBackgroundSettingsStore();
@@ -574,6 +574,15 @@ export const useBrandPageMakerStore = defineStore('brandPageMaker', {
             this.ogDescription = snapshot.ogDescription;
             this.cookieOptions = snapshot.cookieOptions;
             this.saveContactSettings = snapshot.saveContactSettings;
+            // 7b. Restore the selected-template pointer that drives the templates
+            //     sidebar highlight. It's a UI pointer, not part of the persisted
+            //     model, so it rides along as a history-entry sidecar (like
+            //     `activePageId`). Its watchers only reset scroll position (no
+            //     template re-application), so setting it here is safe during apply.
+            //     (BP-5813)
+            if (selectedTemplateId !== undefined) {
+                this.setSelectedTemplateId({ templateId: selectedTemplateId });
+            }
             // 8. Re-sync the logo-settings sidebar's `selectedLogoImage` from the
             //    restored header logo. The preview rehydrates via the page contents,
             //    but `uiLogoSettings` holds its own copy that drives the sidebar
@@ -622,9 +631,7 @@ export const useBrandPageMakerStore = defineStore('brandPageMaker', {
                     console.error('brandpage-maker-store: failed to restore logoEdits', error);
                 }
             }
-            const logoImage = snapshot.brandPageType === BRAND_PAGE_TYPES.default
-                ? snapshot.pages?.[0]?.contents?.find((c) => c.type === 'logo')?.logoImage
-                : snapshot.pageHeaderDesktop?.logoImage;
+            const logoImage = logoUtil.getLogoImageFromModel(snapshot);
             if (!logoImage)
                 return;
             logoSettingsStore.setState({
@@ -848,9 +855,7 @@ export const useBrandPageMakerStore = defineStore('brandPageMaker', {
                 this.setBrandPageType({
                     brandPageType: _brandPageType,
                 });
-                let logoImage = brandPageModel.brandPageType === BRAND_PAGE_TYPES.default
-                    ? brandPageModel.pages?.[0].contents.find((c) => c.type === 'logo')?.logoImage
-                    : brandPageModel.pageHeaderDesktop?.logoImage;
+                let logoImage = logoUtil.getLogoImageFromModel(brandPageModel);
                 if (!logoImage?.logoDraftToken) {
                     logoImage = logoUtil.safeGetLogoImageFromLogoImageUrl(logoVariationImageUrls[0]);
                 }

package/lib/src/brand-page-maker/stores/history/history-store.d.ts

@@ -16,6 +16,12 @@ export interface HistoryEntry {
     activePageId: string;
     /** JSON-stringified `editableLogoStore.logoEdits`. Undefined if logo tab never opened. */
     logoEdits?: string;
+    /**
+     * Selected template id at capture time — drives the templates sidebar
+     * highlight. `false` when no template is selected. Sidecar to
+     * `brandPageData` (not part of the persisted model), like `activePageId`.
+     */
+    selectedTemplateId?: string | false;
 }
 export declare const useHistoryStore: import("pinia").StoreDefinition<"history", Pick<{
     entries: import("vue").ShallowRef<HistoryEntry[], HistoryEntry[]>;

package/lib/src/brand-page-maker/stores/history/history-store.js

@@ -97,6 +97,9 @@ export const useHistoryStore = defineStore('history', () => {
     function getCurrentActivePageId() {
         return usePageNavigationStore().activePageId ?? '';
     }
+    function getCurrentSelectedTemplateId() {
+        return useBrandPageMakerStore().selectedTemplateId;
+    }
     function serializeCurrentLogoEdits() {
         try {
             const editableLogoStore = useUiEditableLogoStore();
@@ -150,6 +153,7 @@ export const useHistoryStore = defineStore('history', () => {
             sizeBytes: args.snapshot.length,
             activePageId: args.activePageId,
             logoEdits: args.logoEdits,
+            selectedTemplateId: args.selectedTemplateId,
         };
         // FR-11: truncate redo tail, then append.
         const next = entries.value.slice(0, currentIndex.value + 1);
@@ -209,6 +213,7 @@ export const useHistoryStore = defineStore('history', () => {
                 return;
             const activePageId = getCurrentActivePageId();
             const logoEdits = serializeCurrentLogoEdits();
+            const selectedTemplateId = getCurrentSelectedTemplateId();
             // Settle window: fold post-init reactive cascades back into the
             // baseline so undo doesn't light up before the user has done
             // anything.
@@ -228,6 +233,7 @@ export const useHistoryStore = defineStore('history', () => {
                         sizeBytes: snapshot.length,
                         activePageId,
                         logoEdits,
+                        selectedTemplateId,
                     },
                 ];
                 return;
@@ -241,6 +247,7 @@ export const useHistoryStore = defineStore('history', () => {
                     sizeBytes: snapshot.length,
                     activePageId,
                     logoEdits,
+                    selectedTemplateId,
                 };
                 entries.value = next;
                 return;
@@ -251,6 +258,7 @@ export const useHistoryStore = defineStore('history', () => {
                 actor: 'user',
                 activePageId,
                 logoEdits,
+                selectedTemplateId,
             });
         };
         // Monotonic — first mutation arms; subsequent ones inside the window
@@ -299,6 +307,7 @@ export const useHistoryStore = defineStore('history', () => {
                 sizeBytes: snapshot.length,
                 activePageId: getCurrentActivePageId(),
                 logoEdits: serializeCurrentLogoEdits(),
+                selectedTemplateId: getCurrentSelectedTemplateId(),
             },
         ];
         currentIndex.value = 0;
@@ -318,6 +327,7 @@ export const useHistoryStore = defineStore('history', () => {
             snapshot,
             activePageId: getCurrentActivePageId(),
             logoEdits: serializeCurrentLogoEdits(),
+            selectedTemplateId: getCurrentSelectedTemplateId(),
             ...args,
         });
     }
@@ -393,7 +403,7 @@ export const useHistoryStore = defineStore('history', () => {
         pendingTimerIsMerge = false;
         cancelPendingAutoCapture();
         try {
-            await brandPageMakerStore.applyHistorySnapshot(snapshot, resolvedActivePageId, targetEntry?.logoEdits);
+            await brandPageMakerStore.applyHistorySnapshot(snapshot, resolvedActivePageId, targetEntry?.logoEdits, targetEntry?.selectedTemplateId);
         }
         finally {
             isApplyingSnapshot.value = false;

package/lib/src/brand-page-maker/utils/index.d.ts

@@ -5,6 +5,4 @@ export * from './url.util';
 export * from './slug.util';
 export * from './text.util';
 export * from './rich-text-editor.util';
-export * from './rte-render.util';
-export * from './rte-sanitize.util';
 export declare const isColorMatched: (color: string | null | undefined) => boolean;

package/lib/src/brand-page-maker/utils/index.js

@@ -6,8 +6,6 @@ export * from './url.util';
 export * from './slug.util';
 export * from './text.util';
 export * from './rich-text-editor.util';
-export * from './rte-render.util';
-export * from './rte-sanitize.util';
 export const isColorMatched = (color) => {
     if (!color)
         return false;

package/lib/src/brand-page-maker/utils/rich-text-editor.util.d.ts

@@ -1,6 +1,6 @@
-import type { ChainedCommands, Editor, JSONContent } from '@tiptap/vue-3';
+import type { ChainedCommands, Editor } from '@tiptap/vue-3';
 import type { EditorState, Transaction, Selection } from 'prosemirror-state';
-import { type Content, type TextStyleDeclaration, type TextStyleType, type RichTextValue } from '@brand-page';
+import { type Content, type TextStyleDeclaration, type TextStyleType } from '@brand-page';
 import type { PageSettingsState } from '@shared/page-settings/store';
 import type { StyleSettingsState } from '@shared/style-settings/store';
 import type { Mark } from 'prosemirror-model';
@@ -9,22 +9,12 @@ export declare const RTE_ALLOWED_SECTION_TYPES: string[];
 export declare const isRTEManagedStyleField: (sectionType: string, fieldName: string) => boolean;
 export declare const isStyleChanged: (tr: Transaction) => boolean;
 export declare const skipStyleDetection: (chain: ChainedCommands, marks?: Mark[]) => ChainedCommands;
-/**
- * BP-5787 — canonical discriminator for the JSON RTE model: a ProseMirror
- * document is an object with `type: 'doc'`.
- */
-export declare const isTiptapDocument: (value: unknown) => value is JSONContent;
-/**
- * Extract visible plain text from a Tiptap JSON document (concatenated text
- * nodes). Dependency-free — no editor instantiation.
- */
-export declare const getRteJsonPlainText: (doc: JSONContent) => string;
-export declare const hasRTEContent: (content: unknown) => boolean;
-export declare const getPlainTextFromRTE: (content: unknown) => string;
+export declare const hasRTEContent: (content: string) => boolean;
+export declare const getPlainTextFromRTE: (content: string) => string;
 export declare const replaceTextInRTE: (existingHtml: string, newText: string) => string;
 export declare const transformToRTE: (uiPageSettingStore: PageSettingsState, externalEditor: Editor, uiStyleSettingsStore?: StyleSettingsState, newContent?: Content, type?: TextStyleType, newStyle?: TextStyleDeclaration) => void;
 export declare const switchAllContentToRTE: (uiPageSettingStore: PageSettingsState, externalEditor: Editor, uiStyleSettingsStore?: StyleSettingsState) => void;
-export declare const getUpdatedHTMLFromEditor: (externalEditor: Editor, contentText: RichTextValue, newContentStyle: TextStyleDeclaration) => string | JSONContent;
+export declare const getUpdatedHTMLFromEditor: (externalEditor: Editor, contentText: string, newContentStyle: TextStyleDeclaration) => string;
 export declare const buildInitialMarks: (state: EditorState, style: TextStyleDeclaration) => any[];
 export declare const retainStyleOnEmptyLine: (editor: Editor, selection: Selection) => void;
 export interface SanitizationResult {
@@ -32,10 +22,6 @@ export interface SanitizationResult {
     removedElements: string[];
 }
 export declare const sanitiseClientSideRTE: (html: string) => SanitizationResult;
-/**
- * Validates if an href value is safe and returns a sanitized version
- */
-export declare const validateAndSanitizeHref: (href: string) => string | null;
 /**
  * Sanitize HTML string using DOMPurify.
  * Only allow tags and attributes that correspond to the enabled Tiptap extensions.

package/lib/src/brand-page-maker/utils/rich-text-editor.util.js

@@ -175,41 +175,14 @@ export const skipStyleDetection = (chain, marks) => chain.command(({ tr }) => {
     }
     return true;
 });
-/**
- * BP-5787 — canonical discriminator for the JSON RTE model: a ProseMirror
- * document is an object with `type: 'doc'`.
- */
-export const isTiptapDocument = (value) => typeof value === 'object' &&
-    value !== null &&
-    value.type === 'doc';
-/**
- * Extract visible plain text from a Tiptap JSON document (concatenated text
- * nodes). Dependency-free — no editor instantiation.
- */
-export const getRteJsonPlainText = (doc) => {
-    const parts = [];
-    const walk = (node) => {
-        if (typeof node.text === 'string')
-            parts.push(node.text);
-        node.content?.forEach(walk);
-    };
-    walk(doc);
-    return parts.join(' ').replace(/\s+/g, ' ').trim();
-};
-// Accepts a legacy inline-HTML string or a Tiptap JSON document (BP-5787).
 export const hasRTEContent = (content) => {
-    if (isTiptapDocument(content))
-        return true;
-    if (typeof content !== 'string' || content.length === 0)
+    if (!content || content.length === 0)
         return false;
     const trimmed = content.trim();
     const tagWrapRegex = /^<([A-Za-z][A-Za-z0-9-]*)\b[^>]*>[\s\S]*<\/\1>$/;
     return tagWrapRegex.test(trimmed);
 };
 export const getPlainTextFromRTE = (content) => {
-    if (isTiptapDocument(content)) {
-        return getRteJsonPlainText(content);
-    }
     if (typeof content !== 'string') {
         return '';
     }
@@ -501,10 +474,7 @@ export const getUpdatedHTMLFromEditor = (externalEditor, contentText, newContent
     if (!contentText) {
         return contentText;
     }
-    // BP-5787: accept a string (legacy HTML) or a Tiptap JSON doc. setContent
-    // handles both; only the newline-escaping shim applies to strings.
-    const content = typeof contentText === 'string' &&
-        contentText.includes(ESCAPED_NEWLINE_CHARACTER)
+    const content = contentText.includes(ESCAPED_NEWLINE_CHARACTER)
         ? getRenderText(contentText)
         : contentText;
     externalEditor.commands.setContent(content);
@@ -548,8 +518,8 @@ export const getUpdatedHTMLFromEditor = (externalEditor, contentText, newContent
             : chain.unsetBold();
     chain = newContentStyle?.fontStyle ? chain.setItalic() : chain.unsetItalic();
     chain.run();
-    // BP-5787: return the JSON form, consistent with how RTE is now stored.
-    return externalEditor.getJSON();
+    const html = externalEditor.getHTML();
+    return html;
 };
 // Helper to build stored marks from a style declaration
 export const buildInitialMarks = (state, style) => {
@@ -628,7 +598,7 @@ export const sanitiseClientSideRTE = (html) => {
 /**
  * Validates if an href value is safe and returns a sanitized version
  */
-export const validateAndSanitizeHref = (href) => {
+const validateAndSanitizeHref = (href) => {
     if (!href || typeof href !== 'string')
         return null;
     // Trim whitespace and decode entities

package/lib/src/brand-page-maker/validators/field.validator.js

@@ -1,5 +1,5 @@
 import { isURL, isEmail, isTel } from '@utils/validator.util';
-import { YOUTUBE_URL_RE, isTiptapDocument } from '@brand-page-maker/utils';
+import { YOUTUBE_URL_RE } from '@brand-page-maker/utils';
 import { validatorTr } from '@src/composables/useTranslate';
 const addressValueValidator = {
     validate: (name, value) => {
@@ -62,15 +62,7 @@ export const dummyValidator = {
 };
 const nonEmptyStringValidator = {
     validate: (name, value) => {
-        // BP-5787/BP-5857: match legacy (pre-JSON) validation semantics.
-        // An RTE field's stored value used to be inline HTML — an empty editor was
-        // "<p></p>", a NON-empty string, so the required check always passed for RTE
-        // fields (it only ever failed for genuinely empty PLAIN inputs). Now RTE
-        // fields are Tiptap JSON docs; treat any doc as "present" so behaviour
-        // stays identical. A plain string is still required to be non-empty.
-        const isValid = isTiptapDocument(value)
-            ? true
-            : typeof value === 'string' && value.length > 0;
+        const isValid = typeof value === 'string' && value.length > 0;
         return {
             isValid,
             message: isValid ? '' : validatorTr(`emptyStringMessage`),

package/lib/src/css/library.brand-page-brandCrowd.min.css

@@ -5703,7 +5703,7 @@ input[data-v-4cd691db]::-webkit-inner-spin-button {
 }
 }
 
-.empty-para[data-v-6ed2f53e] p:empty::after {
+.empty-para[data-v-c90cc6bc] p:empty::after {
   content: '\00A0';
 }
 

package/lib/src/css/library.brand-page-brandPage.min.css

@@ -6314,7 +6314,7 @@ input[data-v-4cd691db]::-webkit-inner-spin-button {
 }
 }
 
-.empty-para[data-v-6ed2f53e] p:empty::after {
+.empty-para[data-v-c90cc6bc] p:empty::after {
   content: '\00A0';
 }
 

package/lib/src/css/library.brand-page-designCom.min.css

@@ -5733,7 +5733,7 @@ input[data-v-4cd691db]::-webkit-inner-spin-button {
 }
 }
 
-.empty-para[data-v-6ed2f53e] p:empty::after {
+.empty-para[data-v-c90cc6bc] p:empty::after {
   content: '\00A0';
 }
 

package/lib/src/css/library.brand-page-preview.min.css

@@ -5703,7 +5703,7 @@ input[data-v-4cd691db]::-webkit-inner-spin-button {
 }
 }
 
-.bp .empty-para[data-v-6ed2f53e] p:empty::after {
+.bp .empty-para[data-v-c90cc6bc] p:empty::after {
   content: '\00A0';
 }
 

package/lib/src/index.d.ts

@@ -28,7 +28,6 @@ export * as brandPageUtils from './brand-page/utils';
 export type { CriticalResourceURLs } from './brand-page/utils/view-model.util';
 export { getOptimalLogoSize, calculateOptimizedLogoDimensions, applyLogoImageBase64UrlToBrandPageTemplateModel, recalculateLogoContentDimensions, recalculateHeaderLogoSizes, } from './brand-page/utils/logo.util';
 export { sanitizeRTEHtml } from './brand-page-maker/utils';
-export { renderRteToHtml, toRteJson, isTiptapDocument, sanitizeRteJson, getRteJsonPlainText, type RichTextValue, } from './brand-page-maker/utils';
 export { getPageSlugFromPageTitle } from './brand-page-maker/utils/slug.util';
 export { executeToolCallsAsync } from './metadata-builder/ai-metadata-executor';
 export type { ParsedToolCall } from './metadata-builder/parsers/ai-toolcall-parser';