@trops/dash-core 0.1.420 → 0.1.422

package/dist/index.js

@@ -18812,7 +18812,32 @@ var WidgetCardHeader = function WidgetCardHeader(_ref) {
   var isWidgetMissing = (widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.component) && !widgetConfig;
 
   // Get widget name from config or item
-  var widgetName = (widgetConfig === null || widgetConfig === void 0 ? void 0 : widgetConfig.name) || (widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.name) || (widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.component);
+  var widgetName = (widgetConfig === null || widgetConfig === void 0 ? void 0 : widgetConfig.displayName) || (widgetConfig === null || widgetConfig === void 0 ? void 0 : widgetConfig.name) || (widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.name) || (widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.component);
+
+  // Build a scope/package subtitle so ambiguous component names like
+  // `ProspectListColumn` (which might come from `@ai-built/…` or
+  // `@trops/pipeline`) are disambiguated in the layout builder. Derives
+  // from whatever identity the config / layout item carries:
+  //   - widgetConfig.id:   e.g. "@ai-built/prospectlistcolumn.ProspectListColumn"
+  //   - widgetConfig.package: e.g. "@ai-built/prospectlistcolumn"
+  //   - widgetItem.workspace: fallback hint ("ai-built" / "@trops/pipeline")
+  // Falls back to empty string so we can skip rendering if we have
+  // nothing meaningful beyond the component name itself.
+  var packageLabel = function () {
+    var dropTrailingComponent = function dropTrailingComponent(s) {
+      if (typeof s !== "string") return "";
+      var lastDot = s.lastIndexOf(".");
+      return lastDot > 0 ? s.slice(0, lastDot) : s;
+    };
+    var fromId = dropTrailingComponent((widgetConfig === null || widgetConfig === void 0 ? void 0 : widgetConfig.id) || "");
+    if (fromId) return fromId;
+    if (widgetConfig !== null && widgetConfig !== void 0 && widgetConfig["package"]) return String(widgetConfig["package"]);
+    var wsHint = widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.workspace;
+    if (typeof wsHint === "string" && wsHint && wsHint !== "layout") {
+      return wsHint.startsWith("@") ? wsHint : "@".concat(wsHint);
+    }
+    return "";
+  }();
 
   // Get provider requirements from widget config (not from item directly)
   // Filter out providerClass: "api" so only user-configurable providers show badges
@@ -18951,11 +18976,18 @@ var WidgetCardHeader = function WidgetCardHeader(_ref) {
       children: [/*#__PURE__*/jsxRuntime.jsx(WidgetIcon, {
         icon: isWidgetMissing ? "triangle-exclamation" : widgetConfig === null || widgetConfig === void 0 ? void 0 : widgetConfig.icon,
         className: "h-4 w-4 flex-shrink-0 ".concat(isWidgetMissing ? "text-amber-500" : "text-white/60")
-      }), /*#__PURE__*/jsxRuntime.jsxs("span", {
-        className: "font-medium text-sm text-gray-100 truncate",
-        children: [widgetName || cellNumber || "Empty", isWidgetMissing && /*#__PURE__*/jsxRuntime.jsx("span", {
-          className: "text-amber-500/70 font-normal ml-1",
-          children: "(not found)"
+      }), /*#__PURE__*/jsxRuntime.jsxs("div", {
+        className: "flex flex-col min-w-0 leading-tight",
+        children: [/*#__PURE__*/jsxRuntime.jsxs("span", {
+          className: "font-medium text-sm text-gray-100 truncate",
+          children: [widgetName || cellNumber || "Empty", isWidgetMissing && /*#__PURE__*/jsxRuntime.jsx("span", {
+            className: "text-amber-500/70 font-normal ml-1",
+            children: "(not found)"
+          })]
+        }), packageLabel && /*#__PURE__*/jsxRuntime.jsxs("span", {
+          className: "text-[10px] text-gray-500 truncate",
+          title: "".concat(packageLabel, " \xB7 ").concat((widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.component) || ""),
+          children: [packageLabel, widgetItem !== null && widgetItem !== void 0 && widgetItem.component && widgetItem.component !== widgetName ? " \xB7 ".concat(widgetItem.component) : ""]
         })]
       })]
     }), (providerRequirements.length > 0 || overflowActions.length > 0 || onMoreOptions) && /*#__PURE__*/jsxRuntime.jsxs("div", {
@@ -20710,6 +20742,17 @@ var LayoutBuilder = function LayoutBuilder(_ref) {
     wsRef.current = currentWorkspace;
   }, [currentWorkspace]);
 
+  // Stable ref to onWorkspaceChange so the window-event handlers below can
+  // propagate mutations upstream without resubscribing on every render.
+  // Without this, swap/place handlers only update LayoutBuilder-local
+  // state — the parent (PinnedSidebar / PageLayoutBuilder) never learns
+  // about the change, so the next parent re-render reverts it via the
+  // `[workspace]` useEffect above and the save path reads stale state.
+  var onWorkspaceChangeRef = React.useRef(onWorkspaceChange);
+  React.useEffect(function () {
+    onWorkspaceChangeRef.current = onWorkspaceChange;
+  }, [onWorkspaceChange]);
+
   // Listen for AI widget builder placement — modifies layout state directly
   React.useEffect(function () {
     var handler = function handler(e) {
@@ -20746,6 +20789,9 @@ var LayoutBuilder = function LayoutBuilder(_ref) {
         var newWorkspace = JSON.parse(JSON.stringify(ws));
         newWorkspace.layout = newLayout;
         setCurrentWorkspace(newWorkspace);
+        if (typeof onWorkspaceChangeRef.current === "function") {
+          onWorkspaceChangeRef.current(newWorkspace);
+        }
       } catch (err) {
       }
     };
@@ -20780,6 +20826,9 @@ var LayoutBuilder = function LayoutBuilder(_ref) {
         var newWorkspace = JSON.parse(JSON.stringify(ws));
         newWorkspace.layout = newLayout;
         setCurrentWorkspace(newWorkspace);
+        if (typeof onWorkspaceChangeRef.current === "function") {
+          onWorkspaceChangeRef.current(newWorkspace);
+        }
       } catch (err) {
       }
     };
@@ -55809,13 +55858,31 @@ function _objectSpread$7(e) { for (var r = 1; r < arguments.length; r++) { var t
  * widget currently in the workspace's layout tree. Keys drive
  * listener pruning; the itemId set drives provider-binding pruning
  * (selectedProviders is keyed by raw widgetId).
+ *
+ * Listener event strings are written as `${component}[${itemId}].${event}`
+ * where `itemId` is whichever id-ish field the emitter had at wire time
+ * (in practice `itemIdOf` prefers numeric `id`). `canonicalItemKey`'s
+ * preference order (uuidString → uuid → id) doesn't match that, and
+ * `LayoutModel` decorates every persisted item with a composite
+ * `uuid = "${dashboardId}-${component}-${id}"` while leaving
+ * `uuidString` unset — so canonicalItemKey would produce
+ * `Component|{dashId}-{Component}-{123}` while the event string produces
+ * `Component|123`. Record every id-ish form as a separate live key so
+ * reconciliation is tolerant of whichever shape the stored binding
+ * carries.
  */
 function collectLiveIdentity(workspace) {
   var liveCanonicalKeys = new Set();
   var liveItemIds = new Set();
   forEachWidget(workspace, function (item) {
-    var key = canonicalItemKey(item);
-    if (key) liveCanonicalKeys.add(key);
+    var comp = item.component;
+    if (comp) {
+      if (item.id != null) liveCanonicalKeys.add("".concat(comp, "|").concat(item.id));
+      if (item.uuidString) {
+        liveCanonicalKeys.add("".concat(comp, "|").concat(item.uuidString));
+      }
+      if (item.uuid) liveCanonicalKeys.add("".concat(comp, "|").concat(item.uuid));
+    }
     // Track every id-ish the runtime might key by. selectedProviders
     // uses `item.uuidString` or `item.uuid` or `item.id` depending on
     // when the binding was written, so accept any of them.