@supernova-studio/client 0.9.1 → 0.9.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@supernova-studio/client",
-  "version": "0.9.1",
+  "version": "0.9.3",
   "description": "Supernova Data Models",
   "source": "src/index.ts",
   "main": "dist/index.js",

package/src/docs-editor/prosemirror-to-blocks.ts

@@ -37,15 +37,15 @@ import {
   PageBlockItemTableNode,
   PageBlockItemTableCell,
   PageBlockTableCellAlignment,
-  DocumentationItemConfiguration,
   PageBlockAppearanceV2,
-  PageBlockEditorModelV2,
   ColorValue,
+  nullishToOptional,
 } from "@supernova-studio/model";
 import { PageBlockEditorModel } from "./model/block";
 import { DocumentationPageEditorModel } from "./model/page";
 import { BlockDefinitionUtils } from "./utils";
 import { yXmlFragmentToProsemirrorJSON } from "y-prosemirror";
+import { ZodAnyDef, ZodSchema, ZodTypeDef, z } from "zod";
 
 export function yDocToPage(yDoc: Y.Doc, definitions: PageBlockDefinition[]): DocumentationPageEditorModel {
   return yXmlFragmentToPage(yDoc.getXmlFragment("default"), definitions);
@@ -68,13 +68,23 @@ export function prosemirrorDocToPage(
   return {
     blocks: (prosemirrorDoc.content ?? [])
       .map(prosemirrorNode => {
-        const definitionId = parseProsemirrorBlockAttribute(prosemirrorNode, "definitionId");
-        if (!definitionId) throw new Error(`Node is missing defintion id`);
-        if (typeof definitionId !== "string")
-          throw new Error(`Definition id is ${typeof definitionId}, has to be string`);
+        const definitionId = getProsemirrorAttribute(prosemirrorNode, "definitionId", z.string());
+        if (!definitionId) {
+          console.warn(
+            `definitionId on ${prosemirrorNode.type} is required to be interpreted as a block, skipping node`
+          );
+
+          return null;
+        }
 
         const definition = definitionsById.get(definitionId);
-        if (!definition) throw new Error(`Definition by id ${definitionId} doesn't exist`);
+        if (!definition) {
+          console.warn(
+            `Block definitionId "${definitionId}" (prosemirror node ${prosemirrorNode.type}) is not among available definitions`
+          );
+          console.warn(prosemirrorNode);
+          return null;
+        }
 
         return prosemirrorNodeToBlock(prosemirrorNode, definition);
       })
@@ -122,13 +132,11 @@ function parseAsRichText(
   definition: PageBlockDefinition,
   property: PageBlockDefinitionProperty
 ): PageBlockEditorModel | null {
-  const id = parseProsemirrorBlockAttribute(prosemirrorNode, "id");
-  if (typeof id !== "string") return null;
-
-  const variantIdRaw = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "variantId");
-  const variantId = typeof variantIdRaw === "string" ? variantIdRaw : undefined;
+  const id = getProsemirrorBlockId(prosemirrorNode);
+  if (!id) return null;
 
-  const calloutType = parseCalloutType(parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "type"));
+  const variantId = getProsemirrorBlockVariantId(prosemirrorNode);
+  const calloutType = parseCalloutType(getProsemirrorAttribute(prosemirrorNode, "type", z.string().optional()));
 
   return {
     // TODO Artem: indent
@@ -181,11 +189,10 @@ function parseAsMultiRichText(
   definition: PageBlockDefinition,
   property: PageBlockDefinitionProperty
 ): PageBlockEditorModel | null {
-  const id = parseProsemirrorBlockAttribute(prosemirrorNode, "id");
-  if (typeof id !== "string") return null;
+  const id = getProsemirrorBlockId(prosemirrorNode);
+  if (!id) return null;
 
-  const variantIdRaw = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "variantId");
-  const variantId = typeof variantIdRaw === "string" ? variantIdRaw : undefined;
+  const variantId = getProsemirrorBlockVariantId(prosemirrorNode);
 
   return {
     // TODO Artem: indent
@@ -253,11 +260,8 @@ function parseRichTextAttribute(mark: ProsemirrorMark): PageBlockTextSpanAttribu
     case "code":
       return { type: "Code" };
     case "link":
-      const itemIdRaw = parseProsemirrorOptionalBlockAttribute(mark, "itemId");
-      const itemId = typeof itemIdRaw === "string" ? itemIdRaw : undefined;
-
-      const hrefRaw = parseProsemirrorOptionalBlockAttribute(mark, "href");
-      const href = typeof hrefRaw === "string" ? hrefRaw : undefined;
+      const itemId = getProsemirrorAttribute(mark, "itemId", z.string().optional());
+      const href = getProsemirrorAttribute(mark, "href", z.string().optional());
 
       return {
         type: "Link",
@@ -279,13 +283,11 @@ function parseAsTable(
   definition: PageBlockDefinition,
   property: PageBlockDefinitionProperty
 ): PageBlockEditorModel | null {
-  const id = parseProsemirrorBlockAttribute(prosemirrorNode, "id");
-  if (typeof id !== "string") return null;
-
-  const variantIdRaw = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "variantId");
-  const variantId = typeof variantIdRaw === "string" ? variantIdRaw : undefined;
+  const id = getProsemirrorBlockId(prosemirrorNode);
+  if (!id) return null;
 
-  const hasBorder = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "hasBorder") !== false;
+  const variantId = getProsemirrorBlockVariantId(prosemirrorNode);
+  const hasBorder = getProsemirrorAttribute(prosemirrorNode, "hasBorder", z.boolean().optional()) !== false;
 
   const tableChild = prosemirrorNode.content?.find(c => c.type === "table");
   if (!tableChild) {
@@ -338,14 +340,14 @@ function parseAsTable(
 }
 
 function parseAsTableCell(prosemirrorNode: ProsemirrorNode): PageBlockItemTableCell | null {
-  const id = parseProsemirrorBlockAttribute(prosemirrorNode, "id");
-  if (typeof id !== "string") return null;
+  const id = getProsemirrorBlockId(prosemirrorNode);
+  if (!id) return null;
 
-  const textAlign = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "textAlign");
+  const textAlign = getProsemirrorAttribute(prosemirrorNode, "textAlign", z.string().optional());
 
   let columnWidth: number | undefined;
-  const columnWidthArray = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "colwidth");
-  if (Array.isArray(columnWidthArray) && typeof columnWidthArray[0] === "number") {
+  const columnWidthArray = getProsemirrorAttribute(prosemirrorNode, "colwidth", z.array(z.number()).optional());
+  if (columnWidthArray) {
     columnWidth = columnWidthArray[0];
   }
 
@@ -384,8 +386,8 @@ function parseAsTableNode(prosemirrorNode: ProsemirrorNode): PageBlockItemTableN
       };
 
     case "image":
-      const url = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "src");
-      if (!url || typeof url !== "string") return null;
+      const url = getProsemirrorAttribute(prosemirrorNode, "src", z.string().optional());
+      if (!url) return null;
 
       return {
         type: "Image",
@@ -458,15 +460,14 @@ function parseAsEmbed(
   definition: PageBlockDefinition,
   property: PageBlockDefinitionProperty
 ): PageBlockEditorModel | null {
-  const id = parseProsemirrorBlockAttribute(prosemirrorNode, "id");
-  if (typeof id !== "string") return null;
+  const id = getProsemirrorBlockId(prosemirrorNode);
+  if (!id) return null;
 
-  const variantIdRaw = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "variantId");
-  const variantId = typeof variantIdRaw === "string" ? variantIdRaw : undefined;
+  const variantId = getProsemirrorBlockVariantId(prosemirrorNode);
 
-  const url = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "url");
-  const caption = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "caption");
-  const height = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "height");
+  const url = getProsemirrorAttribute(prosemirrorNode, "url", z.string().optional());
+  const caption = getProsemirrorAttribute(prosemirrorNode, "caption", z.string().optional());
+  const height = getProsemirrorAttribute(prosemirrorNode, "height", z.number().optional());
 
   return {
     id: id,
@@ -483,7 +484,7 @@ function parseAsEmbed(
             [property.id]: {
               value: url,
               caption: caption,
-              height: height,
+              height: height ?? 200,
             } as PageBlockItemEmbedValue,
           },
         },
@@ -500,11 +501,10 @@ function parseAsDivider(
   prosemirrorNode: ProsemirrorNode,
   definition: PageBlockDefinition
 ): PageBlockEditorModel | null {
-  const id = parseProsemirrorBlockAttribute(prosemirrorNode, "id");
-  if (typeof id !== "string") return null;
+  const id = getProsemirrorBlockId(prosemirrorNode);
+  if (!id) return null;
 
-  const variantIdRaw = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "variantId");
-  const variantId = typeof variantIdRaw === "string" ? variantIdRaw : undefined;
+  const variantId = getProsemirrorBlockVariantId(prosemirrorNode);
 
   return {
     id: id,
@@ -527,11 +527,10 @@ function parseAsCustomBlock(
   prosemirrorNode: ProsemirrorNode,
   definition: PageBlockDefinition
 ): PageBlockEditorModel | null {
-  const id = parseProsemirrorBlockAttribute(prosemirrorNode, "id");
-  if (typeof id !== "string") return null;
+  const id = getProsemirrorBlockId(prosemirrorNode);
+  if (!id) return null;
 
-  const variantIdRaw = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "variantId");
-  const variantId = typeof variantIdRaw === "string" ? variantIdRaw : undefined;
+  const variantId = getProsemirrorBlockVariantId(prosemirrorNode);
 
   const appearance = parseAppearance(prosemirrorNode);
   const parsedItems = parseBlockItems(prosemirrorNode, definition);
@@ -552,8 +551,8 @@ function parseAsCustomBlock(
 }
 
 function parseBlockItems(prosemirrorNode: ProsemirrorNode, definition: PageBlockDefinition): PageBlockItemV2[] | null {
-  const itemsString = parseProsemirrorBlockAttribute(prosemirrorNode, "items");
-  if (typeof itemsString !== "string") return null;
+  const itemsString = getProsemirrorAttribute(prosemirrorNode, "items", z.string());
+  if (!itemsString) return null;
 
   const itemsJson: unknown = JSON.parse(itemsString);
   if (!Array.isArray(itemsJson)) {
@@ -567,13 +566,13 @@ function parseBlockItems(prosemirrorNode: ProsemirrorNode, definition: PageBlock
 function parseAppearance(prosemirrorNode: ProsemirrorNode): PageBlockAppearanceV2 | undefined {
   const appearance: PageBlockAppearanceV2 = {};
 
-  const columns: unknown = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "columns");
-  if (typeof columns === "number") {
+  const columns = getProsemirrorAttribute(prosemirrorNode, "columns", z.number().optional());
+  if (columns) {
     appearance.numberOfColumns = columns;
   }
 
-  const backgroundColor: unknown = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, "backgroundColor");
-  if (typeof backgroundColor === "string") {
+  const backgroundColor = getProsemirrorAttribute(prosemirrorNode, "backgroundColor", z.string().optional());
+  if (backgroundColor) {
     const parsedColor = ColorValue.safeParse(JSON.parse(backgroundColor));
     if (parsedColor.success) {
       appearance.itemBackgroundColor = parsedColor.data;
@@ -673,17 +672,29 @@ function valueSchemaForPropertyType(type: PageBlockDefinitionPropertyType) {
 // Attributes
 //
 
-function parseProsemirrorBlockAttribute(prosemirrorNode: ProsemirrorNode, attributeName: string): unknown {
-  const attributeValue = parseProsemirrorOptionalBlockAttribute(prosemirrorNode, attributeName);
-  if (!attributeValue) {
-    throw new Error(`Attribute ${attributeName} was not defined on node ${prosemirrorNode.type}`);
-  }
+function getProsemirrorBlockId(prosemirrorNode: ProsemirrorNode): string | undefined {
+  const id = getProsemirrorAttribute(prosemirrorNode, "id", z.string());
+  if (!id) console.warn(`Prosemirror attribute "id" on ${prosemirrorNode.type} is required`);
+  return id;
+}
 
-  return attributeValue;
+function getProsemirrorBlockVariantId(prosemirrorNode: ProsemirrorNode): string | undefined {
+  return getProsemirrorAttribute(prosemirrorNode, "variantId", nullishToOptional(z.string()));
 }
 
-function parseProsemirrorOptionalBlockAttribute(prosemirrorNode: ProsemirrorNode, attributeName: string): unknown {
-  return prosemirrorNode.attrs?.[attributeName] ?? undefined;
+function getProsemirrorAttribute<I, O>(
+  prosemirrorNode: ProsemirrorNode,
+  attributeName: string,
+  validationSchema: ZodSchema<O, ZodTypeDef, I>
+): O | undefined {
+  const parsedAttr = validationSchema.safeParse(prosemirrorNode.attrs?.[attributeName]);
+  if (parsedAttr.success) {
+    return parsedAttr.data;
+  } else {
+    console.warn(`Prosemirror attribute ${attributeName} on ${prosemirrorNode.type} is invalid`);
+    console.warn(parsedAttr.error.flatten());
+    return undefined;
+  }
 }
 
 //