@trops/dash-core 0.1.432 → 0.1.434

package/dist/index.js

@@ -18054,7 +18054,6 @@ var EnhancedWidgetDropdown = function EnhancedWidgetDropdown(_ref) {
       padding: true,
       height: "h-full",
       children: filteredWidgets.map(function (widget) {
-        var _widget$providers;
         return /*#__PURE__*/jsxRuntime.jsx(DashReact.MenuItem3, {
           onClick: function onClick() {
             return handleWidgetSelect(widget);
@@ -18074,15 +18073,22 @@ var EnhancedWidgetDropdown = function EnhancedWidgetDropdown(_ref) {
               }), widget.description && /*#__PURE__*/jsxRuntime.jsx("div", {
                 className: "text-xs opacity-50 truncate",
                 children: widget.description
-              }), ((_widget$providers = widget.providers) === null || _widget$providers === void 0 ? void 0 : _widget$providers.length) > 0 && /*#__PURE__*/jsxRuntime.jsx("div", {
-                className: "flex flex-wrap gap-1 mt-0.5",
-                children: widget.providers.map(function (p) {
-                  return /*#__PURE__*/jsxRuntime.jsx("span", {
-                    className: "text-[10px] px-1.5 py-0.5 rounded bg-blue-500/20 text-blue-300",
-                    children: p.type
-                  }, p.type);
-                })
-              })]
+              }), function () {
+                // Same defensive null-safe iteration as WidgetSidebar:
+                // a sparse `providers` array (null entries from a
+                // malformed `.dash.js`) used to crash the chip render.
+                var provs = getUserConfigurableProviders(widget.providers);
+                if (provs.length === 0) return null;
+                return /*#__PURE__*/jsxRuntime.jsx("div", {
+                  className: "flex flex-wrap gap-1 mt-0.5",
+                  children: provs.map(function (p) {
+                    return /*#__PURE__*/jsxRuntime.jsx("span", {
+                      className: "text-[10px] px-1.5 py-0.5 rounded bg-blue-500/20 text-blue-300",
+                      children: p.type
+                    }, p.type);
+                  })
+                });
+              }()]
             }), /*#__PURE__*/jsxRuntime.jsxs("div", {
               className: "flex items-center gap-1.5 flex-shrink-0",
               children: [installedPackageNames.has(widget.packageName) && /*#__PURE__*/jsxRuntime.jsx("span", {
@@ -26686,7 +26692,11 @@ function renderComponent(component, id) {
       // Check if widget exists before attempting render —
       // WidgetFactory.render() always returns a React element (truthy),
       // so the old ternary fallback could never fire.
-      if (!isWidgetResolvable(component)) {
+      // Pass `params` (the layout item) so `packageId` /
+      // `_sourcePackage` can disambiguate when a legacy layout
+      // references a bare component name and multiple installed
+      // packages register a widget with the same trailing name.
+      if (!isWidgetResolvable(component, params)) {
         return /*#__PURE__*/jsxRuntime.jsx(WidgetNotFound, {
           component: component
         });
@@ -27491,20 +27501,25 @@ function addChildToLayoutItem(childComponent, layoutItem, workspace) {
 }
 
 /**
- * Check if a widget component key is resolvable in ComponentManager
+ * Check if a widget component key is resolvable in ComponentManager.
+ * Routes through `ComponentManager.resolve` so a legacy bare component
+ * key (e.g. `"ProspectWorkspace"`) finds its registered scoped form
+ * (`"ai-built.pipeline.ProspectWorkspace"`). Without this routing the
+ * function would return false for every legacy-layout widget after the
+ * scoped-IDs migration, and the renderer would show `WidgetNotFound`
+ * for working dashboards.
+ *
  * @param {string} componentKey the component key to check
+ * @param {object} [data] the layout item — used for `packageId`
+ *                        disambiguation when multiple registered
+ *                        widgets share the same bare name
  * @returns {boolean} true if the widget can be rendered
  */
-function isWidgetResolvable(componentKey) {
+function isWidgetResolvable(componentKey, data) {
   // Layout containers are always resolvable (handled specially by WidgetFactory)
   if (ComponentManager.isLayoutContainer(componentKey)) return true;
-  var m = ComponentManager.componentMap();
-  if (!m) return false;
-  // Exact key match (scoped id)
-  if (m[componentKey] && typeof m[componentKey].component === "function") {
-    return true;
-  }
-  return false;
+  var config = ComponentManager.resolve(componentKey, data);
+  return !!(config && typeof config.component === "function");
 }
 
 // export {
@@ -31447,31 +31462,33 @@ function bareComponentName(idOrName) {
   return parts[parts.length - 1] || "";
 }
 
-var _componentMap = {};
-
 /**
- * Resolve the registry key for a component lookup. Returns null if no
+ * Resolve a component name to a registry key. Returns null if no
  * match exists.
  *
+ * Pure function — depends only on the passed-in `componentMap` and
+ * `data`. Lives outside ComponentManager so it can be tested without
+ * pulling in the full Component/Layout import chain (and so other
+ * call sites can route through it without circular deps).
+ *
  * Lookup order (the LAYOUT ITEM is the source of truth):
  *   1. EXACT match on `component` — covers the new scoped form
- *      (`scope.package.X`) and any legacy `.dash.js` that already set
- *      `config.id` to a scoped value.
+ *      (`scope.package.X`) and any legacy `.dash.js` that already
+ *      set `config.id` to a scoped value.
  *   2. If `component` is bare (no dots) AND we have a packageId hint
  *      on the layout item, build the scoped id and try that.
  *   3. Bare-name fallback: scan the map for any key ending in
  *      `.${component}`. If exactly one matches, use it. If multiple
- *      match (the collision case), prefer the one matching the layout
- *      item's `packageId` / `_sourcePackage`; otherwise fall through
- *      to the first match (deterministic, but also logs a warning so
- *      callers can spot the ambiguity).
+ *      match (the collision case), prefer the one matching the
+ *      layout item's `packageId` / `_sourcePackage`; otherwise fall
+ *      through to the first match (deterministic, but also logs a
+ *      warning so callers can spot the ambiguity).
  *
- * Step (3) is the back-compat path for layouts authored before scoped
- * registration landed. New layouts ALWAYS resolve via step (1) — the
- * `component` field is already scoped.
+ * Step (3) is the back-compat path for layouts authored before
+ * scoped registration landed. New layouts ALWAYS resolve via step (1).
  */
 function resolveComponentKey(componentMap, component, data) {
-  if (!component) return null;
+  if (!componentMap || !component) return null;
   if (component in componentMap) return component;
   if (typeof component !== "string") return null;
   if (component.includes(".")) return null;
@@ -31492,6 +31509,8 @@ function resolveComponentKey(componentMap, component, data) {
   }
   return matches[0];
 }
+
+var _componentMap = {};
 var _containerComponent = null;
 var _gridContainerComponent = null;
 var ComponentManager = {
@@ -31527,6 +31546,25 @@ var ComponentManager = {
   componentMap: function componentMap() {
     return _componentMap;
   },
+  /**
+   * Resolve a component name to its registered config — single source
+   * of truth for every render-path lookup. Routes through
+   * `resolveComponentKey` so legacy bare names (`"ProspectWorkspace"`)
+   * still find their registered scoped counterpart
+   * (`"ai-built.pipeline.ProspectWorkspace"`). Returns null when the
+   * widget isn't registered. Pass the layout item as `data` to use
+   * its `packageId` / `_sourcePackage` for disambiguation.
+   *
+   * @param {string} component
+   * @param {object} [data]
+   * @returns {object|null} the live registered config, or null
+   */
+  resolve: function resolve(component, data) {
+    var m = _componentMap;
+    if (!m) return null;
+    var key = resolveComponentKey(m, component, data);
+    return key ? m[key] || null : null;
+  },
   /**
    * The method for registering the widget into the Dashboard application
    * This is a requirement for the widget to be included into the Dash
@@ -51433,14 +51471,17 @@ var NotificationsSection = function NotificationsSection(_ref) {
     });
   }, []);
 
-  // Collect all widget instances with notifications from workspaces
+  // Collect all widget instances with notifications from workspaces.
+  // Route through `ComponentManager.resolve` so a legacy layout
+  // referencing a bare component name still finds its registered
+  // scoped form (post-v0.1.432). Direct `componentMap[item.component]`
+  // returns undefined for bare names after the migration.
   var widgetInstances = [];
-  var componentMap = ComponentManager.componentMap() || {};
   workspaces.forEach(function (ws) {
     var items = flattenLayout(ws.layout);
     items.forEach(function (item) {
       var _config$notifications;
-      var config = componentMap[item.component];
+      var config = ComponentManager.resolve(item.component, item);
       if ((config === null || config === void 0 || (_config$notifications = config.notifications) === null || _config$notifications === void 0 ? void 0 : _config$notifications.length) > 0) {
         var _item$userPrefs;
         widgetInstances.push({
@@ -53657,7 +53698,13 @@ var DraggableWidgetItem = function DraggableWidgetItem(_ref) {
     _useDrag2 = _slicedToArray(_useDrag, 2),
     isDragging = _useDrag2[0].isDragging,
     drag = _useDrag2[1];
-  var providerTypes = (widget.providers || []).map(function (p) {
+
+  // Route through getUserConfigurableProviders — drops the `api`
+  // class AND null/undefined entries that occasionally appear in
+  // sparse `widget.providers` arrays. Without this, a single null
+  // slot crashes the whole sidebar with `Cannot read properties of
+  // null (reading 'type')`.
+  var providerTypes = getUserConfigurableProviders(widget.providers).map(function (p) {
     return p.type;
   });
   var eventCount = (widget.events || []).length;
@@ -54231,12 +54278,18 @@ var WidgetSidebar = function WidgetSidebar(_ref4) {
     }))).sort();
   }, [allWidgets]);
 
-  // Derive unique provider types for dropdown
+  // Derive unique provider types for the filter dropdown. Route
+  // through getUserConfigurableProviders so a single null entry in
+  // any widget's `providers` array doesn't crash the whole sidebar
+  // (it filters non-objects AND drops the `api` provider class that
+  // isn't user-configurable). Observed prod crash:
+  // `TypeError: Cannot read properties of null (reading 'type')` →
+  // unmounted WidgetSidebar via React error boundary.
   var uniqueProviders = React.useMemo(function () {
     var types = new Set();
     allWidgets.forEach(function (_ref7) {
       var widget = _ref7.widget;
-      return (widget.providers || []).forEach(function (p) {
+      return getUserConfigurableProviders(widget === null || widget === void 0 ? void 0 : widget.providers).forEach(function (p) {
         return types.add(p.type);
       });
     });
@@ -54258,12 +54311,13 @@ var WidgetSidebar = function WidgetSidebar(_ref4) {
       if (filterAuthor !== "all") {
         if ((widget["package"] || widget.author || "Other") !== filterAuthor) return false;
       }
-      // Provider filter
+      // Provider filter — same null-safety as `uniqueProviders` above.
       if (filterProvider !== "all") {
+        var configurable = getUserConfigurableProviders(widget === null || widget === void 0 ? void 0 : widget.providers);
         if (filterProvider === "none") {
-          if (widget.providers && widget.providers.length > 0) return false;
+          if (configurable.length > 0) return false;
         } else {
-          if (!widget.providers || !widget.providers.some(function (p) {
+          if (!configurable.some(function (p) {
             return p.type === filterProvider;
           })) return false;
         }
@@ -57841,9 +57895,13 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
     return getUnresolvedProviders({
       workspace: workspaceSelected,
       appProviders: (appContext === null || appContext === void 0 ? void 0 : appContext.providers) || {},
+      // Use ComponentManager.config so a legacy layout referencing
+      // a bare component name (`"PipelineKanban"`) still resolves
+      // to its scoped registration. Direct `componentMap[name]`
+      // returns undefined after the v0.1.432 scoped-IDs migration.
       getWidgetRequirements: function getWidgetRequirements(name) {
-        var _ComponentManager$com;
-        return name && ((_ComponentManager$com = ComponentManager.componentMap()[name]) === null || _ComponentManager$com === void 0 ? void 0 : _ComponentManager$com.providers) || [];
+        var _ComponentManager$con;
+        return name && ((_ComponentManager$con = ComponentManager.config(name)) === null || _ComponentManager$con === void 0 ? void 0 : _ComponentManager$con.providers) || [];
       }
     });
   }, [workspaceSelected, appContext === null || appContext === void 0 ? void 0 : appContext.providers]);
@@ -58845,13 +58903,18 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
           isOpen: isConfigModalOpen,
           setIsOpen: setIsConfigModalOpen,
           workspace: workspaceSelected,
-          appProviders: (appContext === null || appContext === void 0 ? void 0 : appContext.providers) || {},
+          appProviders: (appContext === null || appContext === void 0 ? void 0 : appContext.providers) || {}
+          // Use ComponentManager.config so legacy bare component
+          // refs in `workspaceSelected.layout` still resolve to
+          // their registered scoped form (post-v0.1.432). Direct
+          // `componentMap[name]` returns undefined for those.
+          ,
           getWidgetRequirements: function getWidgetRequirements(name) {
-            var _ComponentManager$com2;
-            return name && ((_ComponentManager$com2 = ComponentManager.componentMap()[name]) === null || _ComponentManager$com2 === void 0 ? void 0 : _ComponentManager$com2.providers) || [];
+            var _ComponentManager$con2;
+            return name && ((_ComponentManager$con2 = ComponentManager.config(name)) === null || _ComponentManager$con2 === void 0 ? void 0 : _ComponentManager$con2.providers) || [];
           },
           getWidgetConfig: function getWidgetConfig(name) {
-            return name && ComponentManager.componentMap()[name] || null;
+            return name && ComponentManager.config(name) || null;
           },
           onSaveBindings: handleBulkProviderBindings,
           onSaveListeners: handleBulkListenerBindings,