@trops/dash-core 0.1.431 → 0.1.433

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 {
@@ -27686,6 +27701,16 @@ var LayoutModel = function LayoutModel(layoutItem, workspaceLayout, dashboardId)
       if (Array.isArray(widgetConfig.eventHandlers)) {
         layout.eventHandlers = widgetConfig.eventHandlers;
       }
+      // Migrate legacy bare `component` references to the canonical
+      // scoped form (`scope.package.Component`). ComponentManager.config
+      // returns the resolved scoped key on `widgetConfig.component`, so
+      // we just lift it. Idempotent: items already scoped pass through
+      // unchanged because resolveComponentKey returns the input verbatim
+      // when it's already in the map. Persisted on next save — no
+      // separate migration step needed.
+      if (typeof widgetConfig.component === "string" && widgetConfig.component && widgetConfig.component !== layout.component) {
+        layout.component = widgetConfig.component;
+      }
     }
 
     // Merge user-entered config values (from EnhancedWidgetDropdown) into userPrefs
@@ -31371,6 +31396,120 @@ var ContextModel = /*#__PURE__*/function () {
   }]);
 }();
 
+/**
+ * scopedComponentId.js
+ *
+ * Single source of truth for the canonical scoped component id used by
+ * ComponentManager registration, layout items, and publish-time scope
+ * remap. Format: `scope.packageName.ComponentName`.
+ *
+ * Why a 3-part dotted form (and not `@scope/pkg.Component`)?
+ * - Matches existing `config.id` shape that `.dash.js` files already
+ *   set when authored explicitly (see ComponentManager.registerWidget:
+ *   `const registrationKey = config.id || widgetKey;`).
+ * - Trivial to parse (`split(".")` — three parts, ordered).
+ * - Avoids slashes inside object keys, which some downstream serializers
+ *   (older dash-registry indexers) historically choked on.
+ *
+ * The upstream package id may arrive in either of two shapes:
+ *   - "@scope/pkg" (npm-style)
+ *   - "scope/pkg"  (bare scope)
+ * Both produce the same scoped id.
+ */
+
+/**
+ * Build the canonical scoped component id from a package name and a
+ * bare component name.
+ *
+ * @param {string} packageName e.g. "@ai-built/pipeline" or "ai-built/pipeline"
+ * @param {string} componentName e.g. "ProspectListColumn"
+ * @returns {string} e.g. "ai-built.pipeline.ProspectListColumn"
+ */
+function makeScopedComponentId(packageName, componentName) {
+  if (!componentName) return "";
+  if (!packageName) return componentName;
+  var cleaned = String(packageName).replace(/^@/, "").replace(/\//g, ".");
+  return "".concat(cleaned, ".").concat(componentName);
+}
+
+/**
+ * Parse a scoped component id into its three parts. Returns null when
+ * the input isn't a 3-part dotted id (e.g. legacy bare names).
+ *
+ * @param {string} scopedId
+ * @returns {{scope: string, packageName: string, componentName: string} | null}
+ */
+function parseScopedComponentId(scopedId) {
+  if (typeof scopedId !== "string") return null;
+  var parts = scopedId.split(".");
+  if (parts.length !== 3) return null;
+  return {
+    scope: parts[0],
+    packageName: parts[1],
+    componentName: parts[2]
+  };
+}
+
+/**
+ * Pull the bare component name from a scoped or unscoped id.
+ *
+ * @param {string} idOrName
+ * @returns {string}
+ */
+function bareComponentName(idOrName) {
+  if (typeof idOrName !== "string") return "";
+  var parts = idOrName.split(".");
+  return parts[parts.length - 1] || "";
+}
+
+/**
+ * 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.
+ *   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).
+ *
+ * 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 (!componentMap || !component) return null;
+  if (component in componentMap) return component;
+  if (typeof component !== "string") return null;
+  if (component.includes(".")) return null;
+  var packageId = (data === null || data === void 0 ? void 0 : data.packageId) || (data === null || data === void 0 ? void 0 : data._sourcePackage) || (data === null || data === void 0 ? void 0 : data.packageName) || null;
+  if (packageId) {
+    var scoped = makeScopedComponentId(packageId, component);
+    if (scoped in componentMap) return scoped;
+  }
+  var suffix = ".".concat(component);
+  var matches = Object.keys(componentMap).filter(function (k) {
+    return k.endsWith(suffix);
+  });
+  if (matches.length === 0) return null;
+  if (matches.length === 1) return matches[0];
+  if (packageId) {
+    var target = makeScopedComponentId(packageId, component);
+    if (matches.includes(target)) return target;
+  }
+  return matches[0];
+}
+
 var _componentMap = {};
 var _containerComponent = null;
 var _gridContainerComponent = null;
@@ -31407,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
@@ -31475,15 +31633,18 @@ var ComponentManager = {
    * @returns {Widget} the Widget in the component map
    */
   getComponent: function getComponent(component) {
+    var data = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
     try {
       // console.log("get component");
       if (component && this.componentMap()) {
         if (ComponentManager.isLayoutContainer(component) === false) {
           var m = this.componentMap();
-          // Try exact match first (works for both scoped ids and legacy names)
-          var cmp = component in m ? m[component] : null;
-          if (cmp !== null) {
-            cmp["componentName"] = component;
+          // Resolve through the scoped/bare lookup pipeline so layouts
+          // authored under either format land at the same registry key.
+          var resolvedKey = resolveComponentKey(m, component, data);
+          var cmp = resolvedKey ? m[resolvedKey] : null;
+          if (cmp !== null && cmp !== undefined) {
+            cmp["componentName"] = resolvedKey;
             return cmp;
           }
         } else {
@@ -31661,17 +31822,20 @@ var ComponentManager = {
 
         // get the component configuration from the map
         var components = this.map();
-        if (component in components) {
+        var resolvedKey = resolveComponentKey(components, component, data);
+        if (resolvedKey && resolvedKey in components) {
           // let c = deepCopy(components['component']);
 
           // we have to make sure that we remove the component if this is a context
 
-          var tempComponent = components[component];
+          var tempComponent = components[resolvedKey];
           delete tempComponent["component"];
           var c = JSON.parse(JSON.stringify(tempComponent));
 
-          // tack on the component name
-          c["component"] = component;
+          // Carry the canonical scoped id forward so callers (LayoutModel,
+          // ComponentManager.getComponent) can rewrite layout items'
+          // `component` to the scoped form on first load.
+          c["component"] = resolvedKey;
 
           // if no userConfig key. let's add it for the next step
           if ("userConfig" in c === false) {
@@ -51307,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({
@@ -53531,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;
@@ -54105,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);
       });
     });
@@ -54132,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;
         }
@@ -57715,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]);
@@ -58719,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,
@@ -62322,6 +62511,7 @@ exports.WorkspaceModel = WorkspaceModel;
 exports.WorkspaceScopeContext = WorkspaceScopeContext;
 exports.addChildToLayoutItem = addChildToLayoutItem;
 exports.addItemToItemLayout = addItemToItemLayout;
+exports.bareComponentName = bareComponentName;
 exports.buildMcpConfigFromOverrides = buildMcpConfigFromOverrides;
 exports.canHaveChildren = canHaveChildren;
 exports.changeDirectionForLayoutItem = changeDirectionForLayoutItem;
@@ -62367,10 +62557,12 @@ exports.isWidgetResolvable = isWidgetResolvable;
 exports.isWorkspace = isWorkspace;
 exports.layoutItemHasWorkspaceAsChild = layoutItemHasWorkspaceAsChild;
 exports.loadWidgetBundle = loadWidgetBundle;
+exports.makeScopedComponentId = makeScopedComponentId;
 exports.mcpJsonToFormState = mcpJsonToFormState;
 exports.moveWidgetAcrossContainers = moveWidgetAcrossContainers;
 exports.numChildrenForLayout = numChildrenForLayout;
 exports.parse = parse;
+exports.parseScopedComponentId = parseScopedComponentId;
 exports.removeItemFromLayout = removeItemFromLayout;
 exports.renderComponent = renderComponent;
 exports.renderGridLayout = renderGridLayout;