@trops/dash-core 0.1.429 → 0.1.431

package/dist/index.esm.js

@@ -28734,8 +28734,16 @@ var WorkspaceModel = function WorkspaceModel(workspaceItem) {
   // Skip items already produced by LayoutModel (idempotent: LayoutModel
   // is safe to call on its own output).
   var wsId = "id" in obj ? obj["id"] : workspace.id;
+  // LayoutModel returns null when an item can't be normalized (e.g.
+  // throw inside its catch). A null in the layout array crashes
+  // downstream forEach/map consumers with `Cannot read properties of
+  // null (reading 'type')`. Filter nulls so the renderer never sees
+  // them — the original raw item is already lost at that point, so
+  // dropping it is the only safe action.
   workspace.layout = rawLayout.map(function (item) {
     return LayoutModel(item, rawLayout, wsId);
+  }).filter(function (item) {
+    return item != null;
   });
   workspace.pages = "pages" in obj ? obj["pages"] : [];
   workspace.activePageId = "activePageId" in obj ? obj["activePageId"] : null;
@@ -57406,26 +57414,36 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
     try {
       var _message$workspaces;
       var workspaces = deepCopy(message["workspaces"]);
+      // LayoutModel returns null when normalization throws (e.g. a
+      // widget config that references a component the registry can't
+      // resolve yet — common right after a fresh dashboard install
+      // where some widgets are still downloading). Filter nulls so
+      // every renderer that walks the layout sees only well-formed
+      // items and never crashes on `Cannot read properties of null
+      // (reading 'type')` or similar.
       var workspacesTemp = workspaces.map(function (ws) {
-        var tempLayout = ws["layout"].map(function (layoutOG) {
+        ws["layout"] = (ws["layout"] || []).map(function (layoutOG) {
           return LayoutModel(layoutOG, workspaces, ws["id"]);
+        }).filter(function (item) {
+          return item != null;
         });
-        ws["layout"] = tempLayout;
-        // Normalize page layouts too
         if (ws.pages && Array.isArray(ws.pages)) {
           ws.pages = ws.pages.map(function (page) {
             if (page.layout && Array.isArray(page.layout)) {
               page.layout = page.layout.map(function (layoutOG) {
                 return LayoutModel(layoutOG, workspaces, ws["id"]);
+              }).filter(function (item) {
+                return item != null;
               });
             }
             return page;
           });
         }
-        // Normalize sidebar layout
         if (ws.sidebarLayout && Array.isArray(ws.sidebarLayout)) {
           ws.sidebarLayout = ws.sidebarLayout.map(function (layoutOG) {
             return LayoutModel(layoutOG, workspaces, ws["id"]);
+          }).filter(function (item) {
+            return item != null;
           });
         }
         return WorkspaceModel(ws);
@@ -58209,13 +58227,16 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
   }
   function handleSaveWorkspaceComplete(e, message) {
 
-    // Reconstruct workspaces through LayoutModel (same as load path)
+    // Reconstruct workspaces through LayoutModel (same as load path).
+    // Filter nulls so a partially-failed normalize doesn't poison the
+    // layout array — see handleLoadWorkspacesComplete for the rationale.
     var workspaces = deepCopy(message["workspaces"]);
     var workspacesTemp = workspaces.map(function (ws) {
-      var tempLayout = ws["layout"].map(function (layoutOG) {
+      ws["layout"] = (ws["layout"] || []).map(function (layoutOG) {
         return LayoutModel(layoutOG, workspaces, ws["id"]);
+      }).filter(function (item) {
+        return item != null;
       });
-      ws["layout"] = tempLayout;
       return WorkspaceModel(ws);
     });
     pub.pub("dashboard.workspaceChange", {