@designcrowd/library.brand-page 6.1.1-json-prereq-3 → 6.1.1-json-prereq-4

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@designcrowd/library.brand-page",
   "description": "A collection of Maker/Designer applications",
-  "version": "6.1.1-json-prereq-3",
+  "version": "6.1.1-json-prereq-4",
   "author": "Design.com Eng",
   "license": "UNLICENSED",
   "type": "module",

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

@@ -101,6 +101,31 @@ describe('getRteTextLength', () => {
     expect(getRteTextLength(doc)).toBe(11);
   });
 
+  it('counts adjacent split nodes with no phantom separator (BP-5787)', () => {
+    // Hyperlinking (or bolding) a word splits its text node. Length must match
+    // the editor's textContent — "clickhere" = 9 — NOT a space-joined
+    // "click here" = 10, which would fire the max-length indicator one char
+    // early and flicker as content settles.
+    const doc = {
+      type: 'doc',
+      content: [
+        {
+          type: 'paragraph',
+          content: [
+            { type: 'text', text: 'click' },
+            {
+              type: 'text',
+              text: 'here',
+              marks: [{ type: 'link', attrs: { href: 'https://x.com' } }],
+            },
+          ],
+        },
+      ],
+    };
+
+    expect(getRteTextLength(doc)).toBe(9);
+  });
+
   it('returns 0 for null/undefined and an empty doc', () => {
     expect(getRteTextLength(null)).toBe(0);
     expect(getRteTextLength(undefined)).toBe(0);

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

@@ -316,13 +316,33 @@ export const getPlainTextFromRTE = (content: unknown): string => {
  * concatenated visible text. Without this, `doc.length` is `undefined` (silently
  * disabling the check) and `doc.trim()` throws — the class of bug that survived
  * the `RichTextValue` widening because Vue Options-API `this`/item access is `any`.
+ *
+ * The JSON branch concatenates text nodes with NO separator, matching the
+ * editor's own char count (ProseMirror textContent) — the metric the MaxLength
+ * extension gates on. It deliberately does NOT reuse getRteJsonPlainText, whose
+ * space-join + trim (for readable plain-text extraction) over-counts by one per
+ * inline mark boundary — so hyperlinking or formatting a word made the item's
+ * max-length indicator disagree with the gate and fire at content that was
+ * actually under the cap (BP-5787).
  */
 export const getRteTextLength = (
   value: RichTextValue | null | undefined,
 ): number => {
   if (typeof value === 'string') return value.length;
 
-  return value ? getRteJsonPlainText(value).length : 0;
+  if (!value) return 0;
+
+  let length = 0;
+
+  const walk = (node: JSONContent): void => {
+    if (typeof node.text === 'string') length += node.text.length;
+
+    node.content?.forEach(walk);
+  };
+
+  walk(value);
+
+  return length;
 };
 
 export const replaceTextInRTE = (