@player-ui/player 0.12.0-next.9 → 0.12.0

package/package.json

@@ -6,12 +6,12 @@
     "types"
   ],
   "name": "@player-ui/player",
-  "version": "0.12.0-next.9",
+  "version": "0.12.0",
   "main": "dist/cjs/index.cjs",
   "dependencies": {
-    "@player-ui/partial-match-registry": "0.12.0-next.9",
-    "@player-ui/make-flow": "0.12.0-next.9",
-    "@player-ui/types": "0.12.0-next.9",
+    "@player-ui/partial-match-registry": "0.12.0",
+    "@player-ui/make-flow": "0.12.0",
+    "@player-ui/types": "0.12.0",
     "@types/dlv": "^1.1.4",
     "dequal": "^2.0.2",
     "dlv": "^1.1.3",

package/src/__tests__/player.test.ts

@@ -595,3 +595,54 @@ describe("failure cases", () => {
     await expect(response).rejects.toThrowError("Custom Error");
   });
 });
+
+test("it applies default-view-plugins first", () => {
+  let taps: Array<{ [key: string]: any; name: string }> = [];
+  class TestPlugin {
+    name = "test-plugin";
+
+    apply(player: Player) {
+      player.hooks.view.tap(this.name, (view) => {
+        view.hooks.resolver.tap(this.name, () => {
+          return;
+        });
+        // @ts-expect-error private property
+        taps = view.hooks.resolver!.taps;
+      });
+    }
+  }
+  const player = new Player({ plugins: [new TestPlugin()] });
+
+  player.start(makeFlow({ type: "text", id: "text", value: "View" }));
+
+  const tapsByName = taps.map((tap) => tap!.name);
+  expect(tapsByName.pop()).toBe("test-plugin");
+});
+
+test("allows custom plugin to move before default", () => {
+  let taps: Array<{ [key: string]: any; name: string }> = [];
+  class TestPlugin {
+    name = "test-plugin";
+
+    apply(player: Player) {
+      player.hooks.view.tap(this.name, (view) => {
+        view.hooks.resolver.tap(
+          { name: this.name, before: "applicability" },
+          () => {
+            return;
+          },
+        );
+        // @ts-expect-error private property
+        taps = view.hooks.resolver!.taps;
+      });
+    }
+  }
+  const player = new Player({ plugins: [new TestPlugin()] });
+
+  player.start(makeFlow({ type: "text", id: "text", value: "View" }));
+
+  const tapsByName = taps.map((tap) => tap!.name);
+  expect(tapsByName.indexOf("test-plugin")).toBeLessThan(
+    tapsByName.indexOf("applicability"),
+  );
+});

package/src/player.ts

@@ -420,9 +420,12 @@ export class Player {
       },
       constants: this.constantsController,
     });
-    viewController.hooks.view.tap("player", (view) => {
-      validationController.onView(view);
-      this.hooks.view.call(view);
+
+    this.hooks.viewController.tap("player", (vc) => {
+      vc.hooks.view.tap("player", (view) => {
+        validationController.onView(view);
+        this.hooks.view.call(view);
+      });
     });
     this.hooks.viewController.call(viewController);