@trops/dash-core 0.1.436 → 0.1.437

package/dist/index.js

@@ -15413,18 +15413,14 @@ var LayoutBuilderConfigModal = function LayoutBuilderConfigModal(_ref) {
     return s.key === activeSection;
   }) || sections[0];
 
-  // Footer label: derive @scope/package straight from the layout
-  // item's scoped component id. Strict — no workspace fallback.
+  // Footer label: full scoped id (`scope.package.Component`) so the
+  // user always sees the underlying widget identity, even when the
+  // primary title is a custom per-instance label. No workspace fallback.
   var footerPackageLabel = function () {
     var scopedId = (itemSelected === null || itemSelected === void 0 ? void 0 : itemSelected.component) || "";
     if (typeof scopedId !== "string") return "";
     var parts = scopedId.split(".");
-    if (parts.length !== 3) return "";
-    var _parts = _slicedToArray(parts, 2),
-      scope = _parts[0],
-      pkg = _parts[1];
-    if (!scope || !pkg) return "";
-    return "@".concat(scope, "/").concat(pkg);
+    return parts.length === 3 ? scopedId : "";
   }();
   var componentName = itemSelected ? itemSelected["component"] : "";
   var footerLeftContent = footerPackageLabel ? /*#__PURE__*/jsxRuntime.jsxs("span", {
@@ -19727,24 +19723,40 @@ var WidgetCardHeader = function WidgetCardHeader(_ref) {
   // Detect missing widgets (component key exists but not in ComponentManager)
   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.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);
+  // Display name priority: per-instance user title → widget's default
+  // title → developer's friendly displayName → widget config name →
+  // bare component name (last segment) → full scoped id (last resort).
+  // Mirrors WidgetsTab so the same widget shows the same name in both
+  // surfaces.
+  var widgetName = function (_widgetItem$userPrefs, _widgetConfig$userCon) {
+    var prefsTitle = widgetItem === null || widgetItem === void 0 || (_widgetItem$userPrefs = widgetItem.userPrefs) === null || _widgetItem$userPrefs === void 0 ? void 0 : _widgetItem$userPrefs.title;
+    if (typeof prefsTitle === "string" && prefsTitle.trim()) return prefsTitle;
+    var configDefault = widgetConfig === null || widgetConfig === void 0 || (_widgetConfig$userCon = widgetConfig.userConfig) === null || _widgetConfig$userCon === void 0 || (_widgetConfig$userCon = _widgetConfig$userCon.title) === null || _widgetConfig$userCon === void 0 ? void 0 : _widgetConfig$userCon.defaultValue;
+    if (typeof configDefault === "string" && configDefault.trim()) {
+      return configDefault;
+    }
+    if (widgetConfig !== null && widgetConfig !== void 0 && widgetConfig.displayName) return widgetConfig.displayName;
+    if (widgetConfig !== null && widgetConfig !== void 0 && widgetConfig.name) return widgetConfig.name;
+    if (widgetItem !== null && widgetItem !== void 0 && widgetItem.name) return widgetItem.name;
+    var comp = widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.component;
+    if (typeof comp === "string") {
+      var parts = comp.split(".");
+      if (parts.length === 3) return parts[2];
+      return comp;
+    }
+    return "Widget";
+  }();
 
-  // Derive the "@scope/package" label from the layout item's scoped
-  // component id (`scope.package.Component`). Strict — the layout item
-  // is the source of truth. If the item isn't scoped (a legacy layout
-  // that LayoutModel couldn't migrate, or a missing widget) the label
-  // is empty rather than guessed from the workspace category.
+  // Show the full scoped registry id (`scope.package.Component`) as
+  // the subtitle. The friendly title on line 1 may be a custom user
+  // title that doesn't reveal which widget is mounted; the subtitle
+  // pins down the underlying identity. Strict — the layout item is
+  // the source of truth.
   var packageLabel = function () {
     var scopedId = (widgetItem === null || widgetItem === void 0 ? void 0 : widgetItem.component) || (widgetConfig === null || widgetConfig === void 0 ? void 0 : widgetConfig.id) || "";
     if (typeof scopedId !== "string") return "";
     var parts = scopedId.split(".");
-    if (parts.length !== 3) return "";
-    var _parts = _slicedToArray(parts, 2),
-      scope = _parts[0],
-      pkg = _parts[1];
-    if (!scope || !pkg) return "";
-    return "@".concat(scope, "/").concat(pkg);
+    return parts.length === 3 ? scopedId : "";
   }();
 
   // Get provider requirements from widget config (not from item directly)
@@ -55224,21 +55236,53 @@ function _arrayLikeToArray$5(r, a) { (null == a || a > r.length) && (a = r.lengt
 var ALL_WIDGETS_ID = "__ALL__";
 
 /**
- * Build the scoped registry identifier for a widget. Surfaces the
- * `scope.packageName.component` triple in the settings UI so users
- * can verify what a widget's registry identity is — useful when
- * diagnosing dashboard-install warnings ("why didn't this widget
- * install?" → scoped id in the warning vs scoped id shown here).
- * Returns just the component name when scope/package metadata is
- * unavailable (e.g. bare built-ins).
+ * Pick the most user-friendly display name for a widget instance.
+ * Priority: per-instance user title → widget's default title → widget
+ * developer's friendly displayName → widget config name → the bare
+ * component name (last segment of the scoped id) → the full scoped
+ * id (last resort, never preferred — but always present).
+ *
+ * This is what shows on the primary line of the WidgetsTab list and
+ * detail header. The full scoped registry id (`scope.package.X`) is
+ * surfaced separately via `pickWidgetRef` below, so the user sees
+ * BOTH a friendly name and an unambiguous reference.
  */
-function buildScopedId(widget) {
-  if (!(widget !== null && widget !== void 0 && widget.component)) return null;
-  var scope = widget.scope ? String(widget.scope).replace(/^@/, "") : null;
-  var pkg = widget.packageName ? scope ? String(widget.packageName).replace(new RegExp("^@?".concat(scope, "/")), "") : String(widget.packageName).replace(/^@/, "") : null;
-  if (scope && pkg) return "".concat(scope, ".").concat(pkg, ".").concat(widget.component);
-  if (pkg) return "".concat(pkg, ".").concat(widget.component);
-  return widget.component;
+function pickDisplayName(item, cfg) {
+  var _item$userPrefs, _item$userConfig;
+  var prefsTitle = item === null || item === void 0 || (_item$userPrefs = item.userPrefs) === null || _item$userPrefs === void 0 ? void 0 : _item$userPrefs.title;
+  if (typeof prefsTitle === "string" && prefsTitle.trim()) return prefsTitle;
+  var configDefault = item === null || item === void 0 || (_item$userConfig = item.userConfig) === null || _item$userConfig === void 0 || (_item$userConfig = _item$userConfig.title) === null || _item$userConfig === void 0 ? void 0 : _item$userConfig.defaultValue;
+  if (typeof configDefault === "string" && configDefault.trim()) {
+    return configDefault;
+  }
+  if (cfg !== null && cfg !== void 0 && cfg.displayName) return cfg.displayName;
+  if (cfg !== null && cfg !== void 0 && cfg.name) return cfg.name;
+  var bare = bareComponentName(item === null || item === void 0 ? void 0 : item.component);
+  if (bare) return bare;
+  return (item === null || item === void 0 ? void 0 : item.component) || "Widget";
+}
+
+/**
+ * Format the full scoped registry id (`scope.package.Component`) for
+ * the subtitle. The friendly display name on line 1 may be a custom
+ * per-instance title that doesn't reveal which widget is actually
+ * mounted — the subtitle pins down the underlying widget identity so
+ * the user can always see "this 'Q4 Pipeline' is actually a
+ * `trops.pipeline.SalesPipeline`".
+ *
+ * Falls back to the cfg fields when the layout item is non-canonical
+ * (legacy bare names that LayoutModel couldn't migrate). Returns
+ * null when even those are missing — caller hides the subtitle.
+ */
+function pickWidgetRef(item, cfg) {
+  if (parseScopedComponentId(item === null || item === void 0 ? void 0 : item.component)) return item.component;
+  var scope = (cfg === null || cfg === void 0 ? void 0 : cfg.scope) || (item === null || item === void 0 ? void 0 : item.scope);
+  var pkg = (cfg === null || cfg === void 0 ? void 0 : cfg.packageName) || (item === null || item === void 0 ? void 0 : item.packageName);
+  var comp = bareComponentName(item === null || item === void 0 ? void 0 : item.component) || (cfg === null || cfg === void 0 ? void 0 : cfg.name);
+  if (!scope || !pkg || !comp) return (item === null || item === void 0 ? void 0 : item.component) || null;
+  var bareScope = String(scope).replace(/^@/, "");
+  var barePkg = String(pkg).replace(new RegExp("^@?".concat(bareScope, "/")), "");
+  return "".concat(bareScope, ".").concat(barePkg, ".").concat(comp);
 }
 
 /**
@@ -55285,13 +55329,11 @@ var WidgetsTab = function WidgetsTab(_ref) {
         result.push({
           id: id,
           component: item.component,
-          displayName: item.name || cfg.name || cfg.displayName || item.component,
+          displayName: pickDisplayName(item, cfg),
+          widgetRef: pickWidgetRef(item, cfg),
           section: section,
           userConfig: cfg.userConfig || {},
           userPrefs: item.userPrefs || {},
-          // Identity fields for the registry-identifier label. Prefer
-          // values from the widget's component manager entry since the
-          // layout item itself often doesn't carry scope/packageName.
           scope: cfg.scope || item.scope || null,
           packageName: cfg.packageName || cfg.name || item.packageName || null
         });
@@ -55429,6 +55471,10 @@ var WidgetsTab = function WidgetsTab(_ref) {
                   className: "text-[10px] bg-amber-500/20 text-amber-300 px-1.5 rounded",
                   children: stagedForWidget
                 })]
+              }), w.widgetRef && /*#__PURE__*/jsxRuntime.jsx("div", {
+                className: "text-[10px] text-gray-500 font-mono truncate mt-0.5",
+                title: w.widgetRef,
+                children: w.widgetRef
               }), /*#__PURE__*/jsxRuntime.jsx("div", {
                 className: "text-[10px] text-gray-600 mt-0.5",
                 children: fieldCount === 0 ? "No configurable fields" : "".concat(fieldCount, " field").concat(fieldCount === 1 ? "" : "s")
@@ -55461,20 +55507,19 @@ function SingleWidgetPane(_ref4) {
     effectivePrefs = _ref4.effectivePrefs,
     onFieldChange = _ref4.onFieldChange;
   var hasFields = Object.keys(widget.userConfig).length > 0;
-  var scopedId = buildScopedId(widget);
   return /*#__PURE__*/jsxRuntime.jsxs("div", {
     children: [/*#__PURE__*/jsxRuntime.jsxs("div", {
       className: "mb-3",
       children: [/*#__PURE__*/jsxRuntime.jsx("div", {
         className: "text-gray-200 font-semibold",
         children: widget.displayName
-      }), /*#__PURE__*/jsxRuntime.jsxs("div", {
-        className: "text-xs text-gray-500",
-        children: [widget.component, " \xB7 ", widget.section]
-      }), scopedId && scopedId !== widget.component && /*#__PURE__*/jsxRuntime.jsx("code", {
-        className: "block mt-1 text-[11px] text-gray-500 font-mono truncate",
-        title: scopedId,
-        children: scopedId
+      }), widget.widgetRef && /*#__PURE__*/jsxRuntime.jsx("div", {
+        className: "text-xs text-gray-500 font-mono truncate mt-0.5",
+        title: widget.widgetRef,
+        children: widget.widgetRef
+      }), /*#__PURE__*/jsxRuntime.jsx("div", {
+        className: "text-xs text-gray-600 mt-0.5",
+        children: widget.section
       })]
     }), hasFields ? /*#__PURE__*/jsxRuntime.jsx(PanelEditForm, {
       userConfig: widget.userConfig,
@@ -55720,11 +55765,11 @@ var DashboardConfigModal = function DashboardConfigModal(_ref) {
     var derivePackage = function derivePackage(item) {
       var scopedId = typeof (item === null || item === void 0 ? void 0 : item.component) === "string" ? item.component : "";
       var parts = scopedId.split(".");
-      if (parts.length === 3) {
-        var _parts = _slicedToArray(parts, 2),
-          scope = _parts[0],
-          pkg = _parts[1];
-        if (scope && pkg) return "@".concat(scope, "/").concat(pkg);
+      if (parts.length === 3 && parts[0] && parts[1]) {
+        // Group rows by `@scope/package` so two widgets from the same
+        // package show up under one heading. The full scoped id is
+        // surfaced per-row in the table (column-level identity).
+        return "@".concat(parts[0], "/").concat(parts[1]);
       }
       // Defensive: an explicit packageId on the item still wins for
       // legacy items the migration couldn't auto-resolve.