@designcrowd/library.brand-page 6.2.11-builder-v2-1 → 6.2.12

package/src/brand-page-maker/utils/rte-render.util.ts

@@ -1,179 +0,0 @@
-import { getSchema } from '@tiptap/core';
-import { DOMParser as ProseMirrorDOMParser } from '@tiptap/pm/model';
-import type { JSONContent } from '@tiptap/vue-3';
-import { renderToHTMLString } from '@tiptap/static-renderer/pm/html-string';
-import { getRteExtensions } from '../composables/rich-text-editor/getRteExtensions';
-import { isTiptapDocument, getRteJsonPlainText } from './rich-text-editor.util';
-
-export { isTiptapDocument, getRteJsonPlainText };
-
-/**
- * BP-5787 — the boundary plumbing for storing RTE as Tiptap JSON.
- *
- * An RTE field is either a legacy inline-HTML string (pre-migration, or a
- * not-yet-migrated published site) or a Tiptap JSON document (post-migration).
- * These helpers let every consumer treat the two uniformly:
- *
- *   - isTiptapDocument — the canonical discriminator (object with type 'doc').
- *   - renderRteToHtml  — render either form to final HTML for display.
- *   - toRteJson        — normalise either form to JSON for storage.
- *
- * The schema-driven core of both directions is Tiptap/ProseMirror, fed by the
- * single-source-of-truth extension set (getRteExtensions) — so the mapping lives
- * in the extensions' schema, not a hand-maintained copy that drifts:
- *
- *   - renderRteToHtml uses @tiptap/static-renderer (renderToHTMLString). It is
- *     DOM-less, so it runs on the server (SSR/publish/preview), not just the
- *     browser, and needs no Editor instance.
- *   - toRteJson turns an HTML string into a DOM node with an INJECTED parser and
- *     feeds it to ProseMirror's schema-driven DOMParser. The HTML→DOM step needs
- *     a DOM parser; rather than read the global `window.DOMParser` (as Tiptap's
- *     `generateJSON` does — unusable on a pure-node server and fragile across SSR
- *     execution contexts), the parser is supplied by the caller: the native
- *     `DOMParser` in the browser, a jsdom `DOMParser` *instance* on the server
- *     (see resolveHtmlParser). No global is ever mutated, so the conversion is
- *     fully isomorphic regardless of how the SSR host bundles/executes this code.
- */
-export type RichTextValue = string | JSONContent;
-
-// One extension set shared by both directions (render + parse). Schema knowledge
-// lives here, once — Tiptap/extension upgrades propagate to both automatically.
-const RTE_EXTENSIONS = getRteExtensions();
-
-// ProseMirror schema derived once from the shared extension set. This is the
-// schema-driven heart of HTML→JSON (no drift); only the HTML→DOM step is injected.
-const RTE_SCHEMA = getSchema(RTE_EXTENSIONS);
-
-/**
- * A `DOMParser`-compatible object: turns an HTML string into a Document whose
- * `body` is a DOM node ProseMirror can parse. The browser's native `DOMParser`
- * and jsdom's `DOMParser` both satisfy this, so the DOM dependency is injected
- * rather than read from a global.
- */
-export interface HtmlParser {
-  parseFromString(html: string, type: 'text/html'): Document;
-}
-
-/**
- * Render an RTE value to final HTML. Legacy HTML strings pass through untouched
- * (they are already HTML); Tiptap JSON is serialised by @tiptap/static-renderer
- * from the extension schema — no editor, no DOM — so it works on the server
- * (SSR/publish), not just the browser.
- */
-export const renderRteToHtml = (
-  value: RichTextValue | null | undefined,
-): string => {
-  if (value == null || value === '') return '';
-
-  if (typeof value === 'string') return value;
-
-  try {
-    return renderToHTMLString({ content: value, extensions: RTE_EXTENSIONS });
-  } catch (error) {
-    // renderToHTMLString throws on a node/mark type this renderer's schema
-    // doesn't know. The realistic trigger is version skew — the library ships to
-    // multiple apps, so a maker on a newer schema can persist a node an older
-    // deployed renderer can't render. Degrade to visible text rather than let
-    // one field throw out of its (unguarded) computed and 500 the whole page.
-    console.error(
-      'renderRteToHtml: unrenderable doc, falling back to plain text',
-      error,
-    );
-
-    return getRteJsonPlainText(value);
-  }
-};
-
-// Tiptap's elementFromString strips whitespace-only text nodes (pretty-print
-// indentation between tags) before parsing; replicate it so injected-parser
-// output stays identical to the previous generateJSON-based conversion.
-const stripIndentationTextNodes = (node: Node): Node => {
-  const children = node.childNodes;
-
-  for (let i = children.length - 1; i >= 0; i -= 1) {
-    const child = children[i];
-
-    if (
-      child.nodeType === 3 &&
-      child.nodeValue &&
-      /^(\n\s\s|\n)$/.test(child.nodeValue)
-    ) {
-      node.removeChild(child);
-    } else if (child.nodeType === 1) {
-      stripIndentationTextNodes(child);
-    }
-  }
-
-  return node;
-};
-
-const htmlToRteJson = (html: string, parser: HtmlParser): JSONContent => {
-  const body = parser.parseFromString(`<body>${html}</body>`, 'text/html').body;
-  const dom = stripIndentationTextNodes(body);
-
-  return ProseMirrorDOMParser.fromSchema(RTE_SCHEMA).parse(dom).toJSON();
-};
-
-let serverParserPromise: Promise<HtmlParser> | null = null;
-
-/**
- * Resolve an HtmlParser for the current runtime: the native `DOMParser` in the
- * browser (and jsdom-based test envs), or a jsdom `DOMParser` INSTANCE on a
- * pure-node server — created and returned directly, never assigned to a global,
- * so it is immune to SSR execution-context differences. The jsdom specifier is
- * assembled at runtime so no bundler follows it into a browser build. Memoised
- * on the server (JSDOM construction is relatively expensive).
- */
-export const resolveHtmlParser = async (): Promise<HtmlParser> => {
-  if (typeof DOMParser !== 'undefined') return new DOMParser();
-
-  if (!serverParserPromise) {
-    const jsdomModuleId = ['js', 'dom'].join('');
-
-    serverParserPromise = import(/* @vite-ignore */ jsdomModuleId)
-      .then((mod) => {
-        const { JSDOM } = mod as {
-          JSDOM: new (html?: string) => {
-            window: { DOMParser: new () => HtmlParser };
-          };
-        };
-        const { DOMParser: JsdomDOMParser } = new JSDOM('').window;
-
-        return new JsdomDOMParser();
-      })
-      .catch((error) => {
-        // Don't cache the failure — let a later call retry the import.
-        serverParserPromise = null;
-
-        throw error;
-      });
-  }
-
-  return serverParserPromise;
-};
-
-const getDefaultParser = (): HtmlParser => {
-  if (typeof DOMParser !== 'undefined') return new DOMParser();
-
-  throw new Error(
-    'toRteJson: no global DOMParser in this environment. On the server, pass an ' +
-      'explicit parser as the second argument (see resolveHtmlParser).',
-  );
-};
-
-/**
- * Normalise an RTE value to a Tiptap JSON document. HTML strings are parsed with
- * `parser` (defaults to the browser's native DOMParser); JSON documents pass
- * through unchanged. Returns undefined for empty/blank input so callers can omit
- * empty fields.
- */
-export const toRteJson = (
-  value: RichTextValue | null | undefined,
-  parser?: HtmlParser,
-): JSONContent | undefined => {
-  if (value == null || value === '') return undefined;
-
-  if (isTiptapDocument(value)) return value;
-
-  return htmlToRteJson(value, parser ?? getDefaultParser());
-};

package/src/brand-page-maker/utils/rte-shape.ts

@@ -1,29 +0,0 @@
-import type { JSONContent } from '@tiptap/vue-3';
-
-/**
- * Is this value RTE content, and in which shape?
- *
- * Pure predicates with NO runtime imports, kept out of `rich-text-editor.util.ts`
- * (which pulls `dompurify` and the render helpers) so a pure consumer — the Color
- * Matches panel, which must tell a plain string from legacy RTE HTML — doesn't drag
- * in the editor toolchain. That module re-exports both, so callers are unchanged.
- */
-
-/** Canonical discriminator for the JSON RTE model: an object with `type: 'doc'`. */
-export const isTiptapDocument = (value: unknown): value is JSONContent =>
-  typeof value === 'object' &&
-  value !== null &&
-  (value as JSONContent).type === 'doc';
-
-/**
- * A legacy inline-HTML RTE string: opens with a tag and closes with one. It does
- * NOT match plain strings (`Price < 10`), CSS values, or text with merely embedded
- * HTML, because genuine RTE always begins with a block tag. Legacy shape only —
- * post-migration use isTiptapDocument, or hasRTEContent for either shape.
- */
-export const isRteHtmlString = (value: unknown): value is string =>
-  typeof value === 'string' &&
-  value.length > 0 &&
-  /^<[A-Za-z][A-Za-z0-9-]*\b[^>]*>[\s\S]*<\/[A-Za-z][A-Za-z0-9-]*>$/.test(
-    value.trim(),
-  );

package/src/clients/ai-assistant-client.test.ts

@@ -1,135 +0,0 @@
-import axios from 'axios';
-import { createPinia, setActivePinia } from 'pinia';
-import { beforeEach, describe, expect, test, vi } from 'vitest';
-import {
-  getInstance,
-  getWebsiteCreateThreadEndpoint,
-  getWebsiteSendMessageEndpoint,
-  getWebsiteToolSubmissionEndpoint,
-  type WebsiteCreateThreadRequest,
-  type WebsiteSendMessageRequest,
-  type WebsiteToolSubmissionRequest,
-  type WebsiteCreateThreadBuildRequest,
-  type WebsiteSendMessageBuildRequest,
-} from './ai-assistant-client';
-
-vi.mock('axios');
-
-const THREAD_ID = 'thread-1';
-
-beforeEach(() => {
-  setActivePinia(createPinia());
-  vi.clearAllMocks();
-  vi.mocked(axios.post).mockResolvedValue({
-    data: { threadId: THREAD_ID, buildId: 'build-1' },
-    headers: {},
-  } as any);
-});
-
-// BP-6193: the client, not the caller, names the agent — tool results sent to a
-// different agent than the one running the thread fail the build
-// (ai-v2-tool-submission-routing.md).
-describe('AiAssistantClient — every website call names the V2 builder', () => {
-  test('the three build methods post agentType: V2', async () => {
-    const client = getInstance();
-
-    await client.createWebsiteThreadBuildAsync({
-      message: 'hello',
-      connectionId: 'conn-1',
-    } as WebsiteCreateThreadBuildRequest);
-    await client.sendWebsiteMessageBuildAsync(THREAD_ID, {
-      message: 'hello',
-      connectionId: 'conn-1',
-    } as WebsiteSendMessageBuildRequest);
-    await client.submitToolResultsBuildAsync(THREAD_ID, {
-      toolCalls: [],
-    } as WebsiteToolSubmissionRequest);
-
-    vi.mocked(axios.post).mock.calls.forEach(([, body]) => {
-      expect(body).toMatchObject({ agentType: 'website_builder_v2' });
-    });
-  });
-
-  test('the three unused V1-era methods also post agentType: V2', async () => {
-    const client = getInstance();
-
-    await client.createWebsiteThreadAndSendMessageAsync({
-      message: 'hello',
-    } as WebsiteCreateThreadRequest);
-    await client.sendWebsiteMessageAsync(THREAD_ID, {
-      message: 'hello',
-    } as WebsiteSendMessageRequest);
-    await client.submitWebsiteToolCallOutputAsync(THREAD_ID, {
-      toolCalls: [],
-    } as WebsiteToolSubmissionRequest);
-
-    vi.mocked(axios.post).mock.calls.forEach(([, body]) => {
-      expect(body).toMatchObject({ agentType: 'website_builder_v2' });
-    });
-  });
-
-  test('a caller-supplied agentType cannot override V2', async () => {
-    const client = getInstance();
-
-    await client.createWebsiteThreadBuildAsync({
-      message: 'hello',
-      connectionId: 'conn-1',
-      agentType: 'website_builder',
-    } as never);
-
-    const [, body] = vi.mocked(axios.post).mock.calls[0];
-
-    expect(body).toMatchObject({ agentType: 'website_builder_v2' });
-  });
-
-  test('each method posts to its own endpoint and keeps the caller fields', async () => {
-    const client = getInstance();
-
-    await client.createWebsiteThreadBuildAsync({
-      message: 'hello',
-      connectionId: 'conn-1',
-      userPlan: 'premium',
-      locale: 'en-US',
-    } as WebsiteCreateThreadBuildRequest);
-    await client.sendWebsiteMessageBuildAsync(THREAD_ID, {
-      message: 'follow-up',
-      connectionId: 'conn-1',
-      userPlan: 'free',
-      locale: 'en-US',
-    } as WebsiteSendMessageBuildRequest);
-    await client.submitToolResultsBuildAsync(THREAD_ID, {
-      toolCalls: [{ toolId: 't1', success: 'success' }],
-      connectionId: 'conn-1',
-      retryCount: 1,
-      userPlan: 'free',
-      locale: 'en-US',
-    } as WebsiteToolSubmissionRequest);
-
-    const calls = vi.mocked(axios.post).mock.calls;
-
-    expect(calls[0][0]).toBe(getWebsiteCreateThreadEndpoint());
-    expect(calls[0][1]).toMatchObject({
-      message: 'hello',
-      connectionId: 'conn-1',
-      userPlan: 'premium',
-      locale: 'en-US',
-    });
-
-    expect(calls[1][0]).toBe(getWebsiteSendMessageEndpoint(THREAD_ID));
-    expect(calls[1][1]).toMatchObject({
-      message: 'follow-up',
-      connectionId: 'conn-1',
-      userPlan: 'free',
-      locale: 'en-US',
-    });
-
-    expect(calls[2][0]).toBe(getWebsiteToolSubmissionEndpoint(THREAD_ID));
-    expect(calls[2][1]).toMatchObject({
-      toolCalls: [{ toolId: 't1', success: 'success' }],
-      connectionId: 'conn-1',
-      retryCount: 1,
-      userPlan: 'free',
-      locale: 'en-US',
-    });
-  });
-});