@player-ui/markdown-plugin 0.14.1 → 0.15.0-next.0

package/src/utils/markdownParser.ts

@@ -1,8 +1,7 @@
 import type { Node, ParseObjectOptions } from "@player-ui/player";
 import { NodeType } from "@player-ui/player";
-import type { Asset } from "@player-ui/types";
 import { fromMarkdown } from "mdast-util-from-markdown";
-import type { Mappers } from "../types";
+import type { Mappers, MarkdownAsset } from "../types";
 import { transformers } from "./transformers";
 
 /**
@@ -16,7 +15,7 @@ export function parseAssetMarkdownContent({
   /**
    * Asset to be parsed
    */
-  asset: Asset;
+  asset: MarkdownAsset;
   /**
    * Mappers record of AST Node to Player Content
    *
@@ -32,36 +31,45 @@ export function parseAssetMarkdownContent({
     options?: ParseObjectOptions,
   ) => Node.Node | null;
 }): Node.Node | null {
-  const { children } = fromMarkdown(asset.value as string);
-  const isMultiParagraph = children.length > 1;
+  const input = asset.value ?? "";
+  const { children } = fromMarkdown(input);
 
-  if (isMultiParagraph) {
-    const value = children.map((node) => {
-      const transformer = transformers[node.type as keyof typeof transformers];
+  // No markdown content: return an empty text asset
+  if (children.length === 0) {
+    const empty = mappers.text({ originalAsset: asset, value: "" });
+    return parser?.(empty, NodeType.Asset) || null;
+  }
+
+  // Map all children to their transformed content
+  const value = children
+    .map((node) => {
+      const transformer = transformers[node.type];
+      if (!transformer) {
+        if (mappers.null) {
+          return mappers?.null({ originalAsset: asset });
+        } else {
+          return null;
+        }
+      }
       return transformer({
-        astNode: node as any,
+        astNode: node,
         asset,
         mappers,
         transformers,
       });
-    });
-
-    const collection = mappers.collection({
-      originalAsset: asset,
-      value,
-    });
+    })
+    .filter((x) => x !== null);
 
-    return parser?.(collection, NodeType.Asset) || null;
+  // If only one item, return it directly; otherwise wrap in collection
+  if (value.length === 1) {
+    const [first] = value;
+    return parser?.(first!, NodeType.Asset) || null;
   }
 
-  const transformer =
-    transformers[children[0].type as keyof typeof transformers];
-  const content = transformer({
-    astNode: children[0] as any,
-    asset,
-    mappers,
-    transformers,
+  const collection = mappers.collection({
+    originalAsset: asset,
+    value,
   });
 
-  return parser?.(content, NodeType.Asset) || null;
+  return parser?.(collection, NodeType.Asset) || null;
 }

package/types/types.d.ts

@@ -1,4 +1,11 @@
 import type { Asset } from "@player-ui/types";
+/**
+ * A markdown asset authored in flows.
+ * The value may be undefined or a string.
+ */
+export interface MarkdownAsset extends Asset<"markdown"> {
+    value?: string;
+}
 export interface BaseArgs {
     /**
      * Unparsed Asset
@@ -11,6 +18,7 @@ export type LiteralMapper<T extends object = object> = (args: {
      */
     value: string;
 } & BaseArgs & T) => Asset;
+export type NullMapper<T extends object = object> = (args: BaseArgs & T) => Asset;
 export type CompositeMapper<T extends object = object> = (args: {
     /**
      * array of assets resulted from the recursion over the AST node children
@@ -109,6 +117,10 @@ export interface Mappers {
      * list item markdown (e.g. - item)
      */
     listItem?: CompositeMapper;
+    /**
+     * Optional mapper for when nothing comes back from the mapping
+     */
+    null?: NullMapper;
 }
 export type Transformer<T = any> = (args: {
     /**

package/types/utils/markdownParser.d.ts

@@ -1,6 +1,5 @@
 import type { Node, ParseObjectOptions } from "@player-ui/player";
-import type { Asset } from "@player-ui/types";
-import type { Mappers } from "../types";
+import type { Mappers, MarkdownAsset } from "../types";
 /**
  * Parses markdown content using a provided mappers record.
  */
@@ -8,7 +7,7 @@ export declare function parseAssetMarkdownContent({ asset, mappers, parser, }: {
     /**
      * Asset to be parsed
      */
-    asset: Asset;
+    asset: MarkdownAsset;
     /**
      * Mappers record of AST Node to Player Content
      *