@opengeni/core 0.4.4 → 0.4.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opengeni/core",
-  "version": "0.4.4",
+  "version": "0.4.6",
   "description": "OpenGeni framework-agnostic core: the domain, access, and billing layers (neutral access, off-HTTP V2 surface). Behavior-preserving extraction from apps/api — keeps Hono's HTTPException for error throwing (typed-errors cleanup deferred).",
   "license": "Apache-2.0",
   "repository": {
@@ -38,14 +38,14 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0",
     "@opengeni/codex": "^0.2.1",
-    "@opengeni/config": "^0.3.0",
+    "@opengeni/config": "^0.4.0",
     "@opengeni/contracts": "^0.9.0",
-    "@opengeni/db": "^0.6.0",
-    "@opengeni/documents": "^0.2.7",
-    "@opengeni/events": "^0.2.7",
-    "@opengeni/observability": "^0.2.1",
-    "@opengeni/runtime": "^0.5.0",
-    "@opengeni/storage": "^0.2.7",
+    "@opengeni/db": "^0.6.1",
+    "@opengeni/documents": "^0.2.8",
+    "@opengeni/events": "^0.2.8",
+    "@opengeni/observability": "^0.3.0",
+    "@opengeni/runtime": "^0.6.1",
+    "@opengeni/storage": "^0.2.8",
     "hono": "^4.12.18"
   }
 }

package/src/domain/capabilities.ts

@@ -436,7 +436,7 @@ export async function validateMcpCapabilityConnection(
     };
   } catch (error) {
     throw new HTTPException(422, {
-      message: `MCP capability "${item.name}" could not be enabled because OpenGeni could not initialize ${item.endpointUrl}: ${mcpProbeErrorMessage(error)}`,
+      message: `MCP capability "${item.name}" could not be enabled because ${mcpProbeErrorMessage(error, item.endpointUrl)}`,
     });
   }
 }
@@ -461,9 +461,16 @@ async function probeStreamableHttpMcpServer(input: McpCapabilityProbeInput): Pro
   }
 }
 
-function mcpProbeErrorMessage(error: unknown): string {
+function mcpProbeErrorMessage(error: unknown, endpointUrl: string): string {
   const message = error instanceof Error ? error.message : String(error);
-  return message.replace(/\s+/g, " ").trim().slice(0, 500) || "unknown error";
+  const normalized = message.replace(/\s+/g, " ").trim();
+  if (
+    /404|405|not found|unexpected token|not valid json|invalid json|failed to parse|streamable http error|unable to connect|fetch failed|econnrefused|enotfound|timeout|aborted/i
+      .test(normalized)
+  ) {
+    return `OpenGeni could not reach a valid Streamable HTTP MCP server at ${endpointUrl}. Check the endpoint URL or choose a different catalog entry.`;
+  }
+  return `OpenGeni could not initialize ${endpointUrl}: ${normalized.slice(0, 500) || "unknown error"}`;
 }
 
 export async function disableCapability(input: {
@@ -785,7 +792,7 @@ async function readSkillMetadata(url: URL, fallbackName: string): Promise<{ name
   return { name, description, category };
 }
 
-function applyCapabilityEnablement(
+export function applyCapabilityEnablement(
   item: CapabilityCatalogItem,
   installation: CapabilityInstallation | undefined,
   activePackIds: Set<string>,
@@ -813,9 +820,28 @@ function applyCapabilityEnablement(
     ...item,
     enabled,
     enabledReason: enabled ? "enabled" : null,
+    connectionRef: enabled && installation ? installationConnectionRef(installation.config) : null,
   };
 }
 
+/**
+ * The connection an installation was enabled with, when the enable-time
+ * connectionRef fully resolved to one (config.connectionRef is set only by
+ * the enable path — see enableCapability). Headers-enabled and credential-
+ * free installations never set it, so this returns null for them.
+ */
+function installationConnectionRef(config: Record<string, unknown>): CapabilityCatalogItem["connectionRef"] {
+  const ref = config.connectionRef;
+  if (!ref || typeof ref !== "object") {
+    return null;
+  }
+  const { connectionId, providerDomain, kind } = ref as Record<string, unknown>;
+  if (typeof connectionId !== "string" || typeof providerDomain !== "string" || typeof kind !== "string") {
+    return null;
+  }
+  return { connectionId, providerDomain, kind };
+}
+
 function dedupeCatalogItems(items: CapabilityCatalogItem[]): CapabilityCatalogItem[] {
   const byId = new Map<string, CapabilityCatalogItem>();
   for (const item of items) {