@player-ui/reference-assets-plugin 0.15.5--canary.888.37918 → 0.15.5

package/src/plugins/chat-ui-demo-plugin.ts

@@ -4,25 +4,10 @@ import {
   ExtendedPlayerPlugin,
   NodeType,
   Player,
-  Node,
 } from "@player-ui/player";
 import { ExpressionPlugin } from "@player-ui/expression-plugin";
 import { send } from "./send";
 
-const isInChatDemo = (node: Node.Node) => {
-  if (
-    node.parent?.parent?.type === NodeType.View &&
-    node.parent.parent.value.id === "chat-view"
-  ) {
-    return true;
-  }
-
-  return (
-    node.parent?.parent?.type === NodeType.Asset &&
-    node.parent.parent.value.id.startsWith("collection-async-chat-demo")
-  );
-};
-
 const createContentFromMessage = (message: string, id: string): any => ({
   asset: {
     type: "chat-message",
@@ -37,27 +22,6 @@ const createContentFromMessage = (message: string, id: string): any => ({
   },
 });
 
-/** This content will fail to display its label since it isn't a valid asset */
-const createBrokenRenderContent = (id: string): any => ({
-  asset: {
-    id,
-    type: "input",
-    binding: "binding",
-    label: 100,
-  },
-});
-
-/** this content will fail to fetch from the data model since the binding is an object */
-const createBrokenTransformContent = (id: string): any => ({
-  asset: {
-    id,
-    type: "input",
-    binding: {
-      prop: "value",
-    },
-  },
-});
-
 export class ChatUiDemoPlugin implements ExtendedPlayerPlugin<[], [], [send]> {
   public readonly name = "chat-ui-demo-plugin";
 
@@ -77,10 +41,10 @@ export class ChatUiDemoPlugin implements ExtendedPlayerPlugin<[], [], [send]> {
     let allPromiseKeys: string[] = [];
     let counter = 0;
 
-    const sendMessage = (
+    const sendMessage: send = (
       context: ExpressionContext,
+      message: string,
       nodeId?: string,
-      getContent?: () => any,
     ): void => {
       if (nodeId && !(nodeId in deferredPromises)) {
         context.logger?.warn(
@@ -98,8 +62,12 @@ export class ChatUiDemoPlugin implements ExtendedPlayerPlugin<[], [], [send]> {
       const keys = nodeId ? [nodeId] : allPromiseKeys;
 
       for (const id of keys) {
+        const content = createContentFromMessage(
+          message,
+          `chat-demo-${counter++}`,
+        );
         const resolveFunction = deferredPromises[id];
-        resolveFunction?.(getContent?.());
+        resolveFunction?.(content);
         delete deferredPromises[id];
       }
 
@@ -113,7 +81,11 @@ export class ChatUiDemoPlugin implements ExtendedPlayerPlugin<[], [], [send]> {
 
     asyncNodePlugin.hooks.onAsyncNode.tap(this.name, (node) => {
       // Ensure this is only used on the chat-ui.tsx mock to prevent the promise from setting up during tests.
-      if (!isInChatDemo(node)) {
+      if (
+        (node.parent?.parent?.type !== NodeType.Asset &&
+          node.parent?.parent?.type !== NodeType.View) ||
+        !node.parent.parent.value.id.startsWith("collection-async-chat-demo")
+      ) {
         return Promise.resolve(undefined);
       }
 
@@ -123,37 +95,6 @@ export class ChatUiDemoPlugin implements ExtendedPlayerPlugin<[], [], [send]> {
       });
     });
 
-    const sendRealMessage: send = (
-      context: ExpressionContext,
-      message: string,
-      nodeId?: string,
-    ) => {
-      return sendMessage(context, nodeId, () =>
-        createContentFromMessage(message, `chat-demo-${counter++}`),
-      );
-    };
-
-    /** These expressions are used as examples in the storybook to allow broken content through and show the error recovery fallback pattern. */
-    const sendBrokenMessage: send = (
-      context: ExpressionContext,
-      _: string,
-      nodeId?: string,
-    ) => {
-      return sendMessage(context, nodeId, () =>
-        createBrokenRenderContent(`chat-demo-${counter++}`),
-      );
-    };
-
-    const sendBrokenTransformMessage: send = (
-      context: ExpressionContext,
-      _: string,
-      nodeId?: string,
-    ) => {
-      return sendMessage(context, nodeId, () =>
-        createBrokenTransformContent(`chat-demo-${counter++}`),
-      );
-    };
-
     // Reset at the start of a new view.
     player.hooks.view.tap(this.name, (_) => {
       deferredPromises = {};
@@ -163,11 +104,7 @@ export class ChatUiDemoPlugin implements ExtendedPlayerPlugin<[], [], [send]> {
 
     // Register 'send' expression
     const expressionPlugin = new ExpressionPlugin(
-      new Map([
-        ["send", sendRealMessage],
-        ["sendBroken", sendBrokenMessage],
-        ["sendBrokenTransform", sendBrokenTransformMessage],
-      ]),
+      new Map([["send", sendMessage]]),
     );
     player.registerPlugin(expressionPlugin);
   }

package/src/plugins/reference-assets-transform-plugin.ts

@@ -19,21 +19,18 @@ import type {
   TextAsset,
 } from "../assets";
 
-export class ReferenceAssetsTransformPlugin
-  implements
-    ExtendedPlayerPlugin<
-      [
-        ActionAsset,
-        InputAsset,
-        ImageAsset,
-        TextAsset,
-        CollectionAsset,
-        ChoiceAsset,
-        ChatMessageAsset,
-      ],
-      [InfoAsset]
-    >
-{
+export class ReferenceAssetsTransformPlugin implements ExtendedPlayerPlugin<
+  [
+    ActionAsset,
+    InputAsset,
+    ImageAsset,
+    TextAsset,
+    CollectionAsset,
+    ChoiceAsset,
+    ChatMessageAsset,
+  ],
+  [InfoAsset]
+> {
   name = "reference-assets-transforms";
 
   apply(player: Player): void {

package/src/plugins/error-recovery-plugin.ts

@@ -1,37 +0,0 @@
-import { AsyncNodePlugin } from "@player-ui/async-node-plugin";
-import { Player, PlayerPlugin } from "@player-ui/player";
-
-export class ErrorRecoveryPlugin implements PlayerPlugin {
-  readonly name = "ErrorRecoveryPlugin";
-  /** */
-  apply(player: Player): void {
-    player.applyTo<AsyncNodePlugin>(AsyncNodePlugin.Symbol, (plugin) => {
-      plugin.hooks.onAsyncNodeError.tap(this.name, (err, node) => {
-        const playerState = player.getState();
-        if (playerState.status !== "in-progress") {
-          return;
-        }
-
-        // Limit error recovery to chat-ui view example to avoid breaking tests.
-        const viewId = playerState.controllers.view.currentView?.initialView.id;
-        if (viewId !== "chat-view") {
-          return;
-        }
-
-        return {
-          asset: {
-            type: "chat-message",
-            id: `${node.id}-recovery`,
-            value: {
-              asset: {
-                id: `${node.id}-recovery-text`,
-                type: "text",
-                value: "Something went wrong, please try again.",
-              },
-            },
-          },
-        };
-      });
-    });
-  }
-}

package/types/plugins/error-recovery-plugin.d.ts

@@ -1,7 +0,0 @@
-import { Player, PlayerPlugin } from "@player-ui/player";
-export declare class ErrorRecoveryPlugin implements PlayerPlugin {
-    readonly name = "ErrorRecoveryPlugin";
-    /** */
-    apply(player: Player): void;
-}
-//# sourceMappingURL=error-recovery-plugin.d.ts.map