@player-ui/player 0.7.0 → 0.7.1-next.0

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@player-ui/player",
-  "version": "0.7.0",
+  "version": "0.7.1-next.0",
   "private": false,
   "publishConfig": {
     "registry": "https://registry.npmjs.org"
   },
   "peerDependencies": {},
   "dependencies": {
-    "@player-ui/partial-match-registry": "0.7.0",
-    "@player-ui/types": "0.7.0",
+    "@player-ui/partial-match-registry": "0.7.1-next.0",
+    "@player-ui/types": "0.7.1-next.0",
     "dequal": "^2.0.2",
     "p-defer": "^3.0.0",
     "queue-microtask": "^1.2.3",

package/src/controllers/validation/controller.ts

@@ -184,13 +184,10 @@ class ValidatedBinding {
   public getAll(): Array<ValidationResponse> {
     return this.applicableValidations.reduce((all, statefulObj) => {
       if (statefulObj.state === 'active' && statefulObj.response) {
-        return [
-          ...all,
-          {
-            ...statefulObj.response,
-            blocking: this.checkIfBlocking(statefulObj),
-          },
-        ];
+        all.push({
+          ...statefulObj.response,
+          blocking: this.checkIfBlocking(statefulObj),
+        });
       }
 
       return all;
@@ -604,18 +601,18 @@ export class ValidationController implements BindingTracker {
       // Get all of the validations from each provider
       const possibleValidations = this.getValidationProviders().reduce<
         Array<ValidationObjectWithSource>
-      >(
-        (vals, provider) => [
-          ...vals,
+      >((vals, provider) => {
+        vals.push(
           ...(provider.provider
             .getValidationsForBinding?.(binding)
             ?.map((valObj) => ({
               ...valObj,
               [VALIDATION_PROVIDER_NAME_SYMBOL]: provider.source,
-            })) ?? []),
-        ],
-        []
-      );
+            })) ?? [])
+        );
+
+        return vals;
+      }, []);
 
       if (possibleValidations.length === 0) {
         return;

package/src/controllers/view/controller.ts

@@ -41,6 +41,8 @@ export class ViewController {
   private pendingUpdate?: {
     /** pending data binding changes */
     changedBindings?: Set<BindingInstance>;
+    /** Whether we have a microtask queued to handle this pending update */
+    scheduled?: boolean;
   };
 
   public currentView?: ViewInstance;
@@ -53,10 +55,11 @@ export class ViewController {
   ) {
     this.viewOptions = options;
     this.viewMap = initialViews.reduce<Record<string, View>>(
-      (viewMap, view) => ({
-        ...viewMap,
-        [view.id]: view,
-      }),
+      (viewMap, view) => {
+        // eslint-disable-next-line no-param-reassign
+        viewMap[view.id] = view;
+        return viewMap;
+      },
       {}
     );
 
@@ -76,19 +79,25 @@ export class ViewController {
     );
 
     /** Trigger a view update */
-    const update = (updates: Set<BindingInstance>) => {
+    const update = (updates: Set<BindingInstance>, silent = false) => {
       if (this.currentView) {
         if (this.optimizeUpdates) {
-          this.queueUpdate(updates);
+          this.queueUpdate(updates, silent);
         } else {
           this.currentView.update();
         }
       }
     };
 
-    options.model.hooks.onUpdate.tap('viewController', (updates) => {
-      update(new Set(updates.map((t) => t.binding)));
-    });
+    options.model.hooks.onUpdate.tap(
+      'viewController',
+      (updates, updateOptions) => {
+        update(
+          new Set(updates.map((t) => t.binding)),
+          updateOptions?.silent ?? false
+        );
+      }
+    );
 
     options.model.hooks.onDelete.tap('viewController', (binding) => {
       const parentBinding = binding.parent();
@@ -103,14 +112,21 @@ export class ViewController {
     });
   }
 
-  private queueUpdate(bindings: Set<BindingInstance>) {
+  private queueUpdate(bindings: Set<BindingInstance>, silent = false) {
     if (this.pendingUpdate?.changedBindings) {
+      // If there's already a pending update, just add to it don't worry about silent updates here yet
       this.pendingUpdate.changedBindings = new Set([
         ...this.pendingUpdate.changedBindings,
         ...bindings,
       ]);
     } else {
-      this.pendingUpdate = { changedBindings: bindings };
+      this.pendingUpdate = { changedBindings: bindings, scheduled: false };
+    }
+
+    // If there's no pending update, schedule one only if this one isn't silent
+    // otherwise if this is silent, we'll just wait for the next non-silent update and make sure our bindings are included
+    if (!this.pendingUpdate.scheduled && !silent) {
+      this.pendingUpdate.scheduled = true;
       queueMicrotask(() => {
         const updates = this.pendingUpdate?.changedBindings;
         this.pendingUpdate = undefined;

package/src/player.ts

@@ -30,8 +30,8 @@ import type {
 import { NOT_STARTED_STATE } from './types';
 
 // Variables injected at build time
-const PLAYER_VERSION = '0.7.0';
-const COMMIT = '00b3c9d34a6983cf364452a8b24b30e0c369a222';
+const PLAYER_VERSION = '0.7.1-next.0';
+const COMMIT = 'b0282a1b53e7a91069e97814edec091b3de9ec5f';
 
 export interface PlayerPlugin {
   /**