@trops/dash-core 0.1.612 → 0.1.613

package/dist/index.js

@@ -46051,6 +46051,55 @@ var NewProviderPicker = function NewProviderPicker(_ref2) {
   });
 };
 
+/**
+ * providerType.js
+ *
+ * Helpers for assigning a stable, UNIQUE `type` to a provider.
+ *
+ * A provider's identity in the widget system is its `type` string (widgets
+ * declare `requiredProviders: [{ type }]`, runtime resolves by type). Catalog
+ * providers each carry a unique type (`slack`, `github`, …). Hand-rolled
+ * custom MCP servers historically all shared `type: "custom"`, which conflates
+ * multiple customs at binding/runtime. These helpers give each NEW custom
+ * provider a unique slug derived from its name so it behaves like a catalog
+ * provider.
+ *
+ * Forward-only: existing providers are never renamed here (that would break
+ * widgets already built against their type). Only the create path uses this.
+ */
+
+/**
+ * Convert a provider name into a kebab-case type slug.
+ * "My Granola Server" → "my-granola-server". Falls back to a sane default
+ * when the name has no slug-able characters.
+ */
+function slugifyProviderType(name) {
+  var slug = String(name || "").toLowerCase().trim().replace(/[^\w\s-]/g, "") // drop punctuation/symbols
+  .replace(/[\s_]+/g, "-") // whitespace/underscores → hyphen
+  .replace(/-+/g, "-") // collapse repeats
+  .replace(/^-+|-+$/g, ""); // trim hyphens
+  return slug || "custom-provider";
+}
+
+/**
+ * Produce a slug type unique against the already-configured provider types.
+ * Appends `-2`, `-3`, … on collision so two providers can never share a type.
+ *
+ * @param {string} name provider instance name
+ * @param {Iterable<string>} existingTypes types already in use (e.g.
+ *   Object.values(providers).map(p => p.type))
+ * @returns {string}
+ */
+function uniqueProviderType(name) {
+  var existingTypes = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : [];
+  var base = slugifyProviderType(name);
+  var taken = new Set(existingTypes || []);
+  if (!taken.has(base)) return base;
+  var i = 2;
+  while (taken.has("".concat(base, "-").concat(i))) i++;
+  return "".concat(base, "-").concat(i);
+}
+
 var ProvidersSection = function ProvidersSection(_ref) {
   var _ref$dashApi = _ref.dashApi,
     dashApi = _ref$dashApi === void 0 ? null : _ref$dashApi,
@@ -46275,8 +46324,21 @@ var ProvidersSection = function ProvidersSection(_ref) {
   function handleMcpSave(providerName, providerType, mcpCredentials, mcpConfig) {
     var allowedTools = arguments.length > 4 && arguments[4] !== undefined ? arguments[4] : null;
     if (!dashApi || !appId) return;
+    // Forward-only unique types: a hand-rolled custom MCP server arrives with
+    // the generic default type "custom", which conflates multiple customs at
+    // widget-binding/runtime time. Give each NEW one a unique slug so it has a
+    // distinct identity like catalog providers. Catalog installs pass a real
+    // id (not "custom") and are untouched; the edit path preserves type, so no
+    // existing provider is ever renamed.
+    var resolvedType = providerType;
+    if (resolvedType === "custom") {
+      var existingTypes = Object.values(providers || {}).map(function (p) {
+        return p.type;
+      });
+      resolvedType = uniqueProviderType(providerName, existingTypes);
+    }
     dashApi.saveProvider(appId, providerName, {
-      providerType: providerType,
+      providerType: resolvedType,
       credentials: mcpCredentials,
       providerClass: "mcp",
       mcpConfig: mcpConfig,