@inkeep/agents-core 0.0.0-dev-20260331194528 → 0.0.0-dev-20260331210432

package/dist/data-access/manage/tools.js

@@ -23,20 +23,45 @@ import "../../utils/index.js";
 import { isGithubWorkAppTool } from "../runtime/github-work-app-installations.js";
 import { getCredentialReference, getUserScopedCredentialReference } from "./credentialReferences.js";
 import { and, count, desc, eq } from "drizzle-orm";
+import { UnauthorizedError } from "@modelcontextprotocol/sdk/client/auth.js";
+import { StreamableHTTPError } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 import { ErrorCode, McpError } from "@modelcontextprotocol/sdk/types.js";
 
 //#region src/data-access/manage/tools.ts
 /**
-* Check if an error is a timeout/connection error.
+* Check if an error is a timeout/connection error (transient, not auth-related).
 * Uses MCP SDK ErrorCode for proper type safety.
 */
 function isTimeoutOrConnectionError(error) {
-	if (error instanceof McpError) return error.code === ErrorCode.RequestTimeout || error.code === ErrorCode.ConnectionClosed;
+	if (error instanceof McpError) return error.code === ErrorCode.RequestTimeout || error.code === ErrorCode.ConnectionClosed || error.code === ErrorCode.InternalError;
+	if (error instanceof StreamableHTTPError) return error.code !== void 0 && error.code >= 500;
 	if (error instanceof Error) {
 		const message = error.message.toLowerCase();
 		const cause = error.cause;
-		if (message.includes("timed out") || message.includes("timeout")) return true;
-		if (cause?.code === "ETIMEDOUT" || cause?.code === "ECONNABORTED" || cause?.code === "ECONNRESET") return true;
+		if (message.includes("timed out") || message.includes("timeout") || message.includes("fetch failed")) return true;
+		if (cause?.code && [
+			"ETIMEDOUT",
+			"ECONNABORTED",
+			"ECONNRESET",
+			"ECONNREFUSED",
+			"ENOTFOUND",
+			"EHOSTUNREACH",
+			"ENETUNREACH",
+			"EPIPE"
+		].includes(cause.code)) return true;
+	}
+	return false;
+}
+/**
+* Check if an error indicates the credential is invalid/expired/revoked.
+* These errors mean the user genuinely needs to re-authenticate.
+*/
+function isAuthenticationError(error) {
+	if (error instanceof UnauthorizedError) return true;
+	if (error instanceof StreamableHTTPError) return error.code === 401 || error.code === 403;
+	if (error instanceof Error) {
+		const message = error.message.toLowerCase();
+		if (message.includes("unauthorized") || message.includes("forbidden") || message.includes("invalid_token") || message.includes("token expired") || message.includes("invalid_grant")) return true;
 	}
 	return false;
 }
@@ -243,6 +268,18 @@ const dbResultToMcpTool = async (dbResult, dbClient, credentialStoreRegistry, re
 		if (isTimeoutOrConnectionError(error)) {
 			status = "unavailable";
 			lastErrorComputed = `Connection failed - the MCP server may be slow or temporarily unreachable.${error instanceof McpError ? ` (MCP error ${error.code})` : ""} ${errorMessage}`;
+		} else if (isAuthenticationError(error)) {
+			status = "needs_auth";
+			lastErrorComputed = `Authentication required - OAuth login needed. ${errorMessage}`;
+		} else if (credentialReference) {
+			logger.warn({
+				toolId: dbResult.id,
+				credentialId: credentialReference.id,
+				errorCode: error instanceof McpError ? error.code : void 0,
+				errorMessage
+			}, "MCP server discovery failed with existing credential — treating as transient");
+			status = "unavailable";
+			lastErrorComputed = `Server temporarily unavailable. ${errorMessage}`;
 		} else {
 			const toolNeedsAuth = await detectAuthenticationRequired({
 				serverUrl: mcpServerUrl,

package/dist/data-access/runtime/apiKeys.d.ts

@@ -12,10 +12,10 @@ declare const getApiKeyById: (db: AgentsRunDatabaseClient) => (params: {
   name: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
+  expiresAt: string | null;
   tenantId: string;
   agentId: string;
-  expiresAt: string | null;
+  projectId: string;
   publicId: string;
   keyHash: string;
   keyPrefix: string;
@@ -26,10 +26,10 @@ declare const getApiKeyByPublicId: (db: AgentsRunDatabaseClient) => (publicId: s
   name: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
+  expiresAt: string | null;
   tenantId: string;
   agentId: string;
-  expiresAt: string | null;
+  projectId: string;
   publicId: string;
   keyHash: string;
   keyPrefix: string;
@@ -43,10 +43,10 @@ declare const listApiKeys: (db: AgentsRunDatabaseClient) => (params: {
   name: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
+  expiresAt: string | null;
   tenantId: string;
   agentId: string;
-  expiresAt: string | null;
+  projectId: string;
   publicId: string;
   keyHash: string;
   keyPrefix: string;
@@ -70,10 +70,10 @@ declare const createApiKey: (db: AgentsRunDatabaseClient) => (params: ApiKeyInse
   name: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
+  expiresAt: string | null;
   tenantId: string;
   agentId: string;
-  expiresAt: string | null;
+  projectId: string;
   publicId: string;
   keyHash: string;
   keyPrefix: string;

package/dist/data-access/runtime/apps.d.ts

@@ -8,11 +8,11 @@ declare const getAppById: (db: AgentsRunDatabaseClient) => (id: string) => Promi
   type: AppType;
   id: string;
   name: string;
-  description: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string | null;
   tenantId: string | null;
+  description: string | null;
+  projectId: string | null;
   lastUsedAt: string | null;
   defaultProjectId: string | null;
   defaultAgentId: string | null;
@@ -67,11 +67,11 @@ declare const createApp: (db: AgentsRunDatabaseClient) => (params: AppInsert) =>
   type: AppType;
   id: string;
   name: string;
-  description: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string | null;
   tenantId: string | null;
+  description: string | null;
+  projectId: string | null;
   lastUsedAt: string | null;
   defaultProjectId: string | null;
   defaultAgentId: string | null;

package/dist/data-access/runtime/conversations.d.ts

@@ -21,16 +21,16 @@ declare const createConversation: (db: AgentsRunDatabaseClient) => (params: Conv
     name: string;
     hash: string;
   } | null;
-  title: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
   userId: string | null;
+  metadata: ConversationMetadata | null;
+  tenantId: string;
   agentId: string | null;
   activeSubAgentId: string;
+  title: string | null;
   lastContextResolution: string | null;
-  metadata: ConversationMetadata | null;
+  projectId: string;
 }>;
 declare const updateConversation: (db: AgentsRunDatabaseClient) => (params: {
   scopes: ProjectScopeConfig;
@@ -90,16 +90,16 @@ declare const getConversation: (db: AgentsRunDatabaseClient) => (params: {
     name: string;
     hash: string;
   } | null;
-  title: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
   userId: string | null;
+  metadata: ConversationMetadata | null;
+  tenantId: string;
   agentId: string | null;
   activeSubAgentId: string;
+  title: string | null;
   lastContextResolution: string | null;
-  metadata: ConversationMetadata | null;
+  projectId: string;
 } | undefined>;
 declare const createOrGetConversation: (db: AgentsRunDatabaseClient) => (input: ConversationInsert) => Promise<{
   activeSubAgentId: string;
@@ -126,16 +126,16 @@ declare const createOrGetConversation: (db: AgentsRunDatabaseClient) => (input:
     name: string;
     hash: string;
   } | null;
-  title: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
   userId: string | null;
+  metadata: ConversationMetadata | null;
+  tenantId: string;
   agentId: string | null;
   activeSubAgentId: string;
+  title: string | null;
   lastContextResolution: string | null;
-  metadata: ConversationMetadata | null;
+  projectId: string;
 }>;
 /**
  * Get conversation history with filtering and context management
@@ -158,16 +158,16 @@ declare const getActiveAgentForConversation: (db: AgentsRunDatabaseClient) => (p
     name: string;
     hash: string;
   } | null;
-  title: string | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
   userId: string | null;
+  metadata: ConversationMetadata | null;
+  tenantId: string;
   agentId: string | null;
   activeSubAgentId: string;
+  title: string | null;
   lastContextResolution: string | null;
-  metadata: ConversationMetadata | null;
+  projectId: string;
 } | undefined>;
 /**
  * Set active agent for a conversation (upsert operation)

package/dist/data-access/runtime/messages.d.ts

@@ -11,20 +11,20 @@ declare const getMessageById: (db: AgentsRunDatabaseClient) => (params: {
   messageId: string;
 }) => Promise<{
   id: string;
-  conversationId: string;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
   metadata: MessageMetadata | null;
-  content: MessageContent;
   role: string;
+  tenantId: string;
+  projectId: string;
+  conversationId: string;
   fromSubAgentId: string | null;
   toSubAgentId: string | null;
   fromExternalAgentId: string | null;
   toExternalAgentId: string | null;
   fromTeamAgentId: string | null;
   toTeamAgentId: string | null;
+  content: MessageContent;
   visibility: string;
   messageType: string;
   taskId: string | null;
@@ -145,20 +145,20 @@ declare const createMessage: (db: AgentsRunDatabaseClient) => (params: {
   data: Omit<MessageInsert, "tenantId" | "projectId">;
 }) => Promise<{
   id: string;
-  conversationId: string;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
   metadata: MessageMetadata | null;
-  content: MessageContent;
   role: string;
+  tenantId: string;
+  projectId: string;
+  conversationId: string;
   fromSubAgentId: string | null;
   toSubAgentId: string | null;
   fromExternalAgentId: string | null;
   toExternalAgentId: string | null;
   fromTeamAgentId: string | null;
   toTeamAgentId: string | null;
+  content: MessageContent;
   visibility: string;
   messageType: string;
   taskId: string | null;
@@ -198,20 +198,20 @@ declare const deleteMessage: (db: AgentsRunDatabaseClient) => (params: {
   messageId: string;
 }) => Promise<{
   id: string;
-  conversationId: string;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
   metadata: MessageMetadata | null;
-  content: MessageContent;
   role: string;
+  tenantId: string;
+  projectId: string;
+  conversationId: string;
   fromSubAgentId: string | null;
   toSubAgentId: string | null;
   fromExternalAgentId: string | null;
   toExternalAgentId: string | null;
   fromTeamAgentId: string | null;
   toTeamAgentId: string | null;
+  content: MessageContent;
   visibility: string;
   messageType: string;
   taskId: string | null;

package/dist/data-access/runtime/tasks.d.ts

@@ -15,13 +15,13 @@ declare const createTask: (db: AgentsRunDatabaseClient) => (params: TaskInsert)
   } | null;
   createdAt: string;
   updatedAt: string;
-  projectId: string;
-  tenantId: string;
-  agentId: string;
   metadata: TaskMetadataConfig | null;
-  subAgentId: string;
   status: string;
+  tenantId: string;
+  agentId: string;
+  projectId: string;
   contextId: string;
+  subAgentId: string;
 }>;
 declare const getTask: (db: AgentsRunDatabaseClient) => (params: {
   id: string;

package/dist/utils/mcp-client.d.ts

@@ -1,7 +1,7 @@
 import { MCPTransportType } from "../types/utility.js";
+import { StreamableHTTPClientTransportOptions } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 import { ClientCapabilities } from "@modelcontextprotocol/sdk/types.js";
 import { SSEClientTransportOptions } from "@modelcontextprotocol/sdk/client/sse.js";
-import { StreamableHTTPClientTransportOptions } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 
 //#region src/utils/mcp-client.d.ts
 declare const activeMcpClients: Set<McpClient>;

package/dist/utils/mcp-client.js

@@ -1,10 +1,10 @@
 import { MCPTransportType } from "../types/utility.js";
 import { MCP_TOOL_CONNECTION_TIMEOUT_MS, MCP_TOOL_INITIAL_RECONNECTION_DELAY_MS, MCP_TOOL_MAX_RECONNECTION_DELAY_MS, MCP_TOOL_MAX_RETRIES, MCP_TOOL_RECONNECTION_DELAY_GROWTH_FACTOR } from "../constants/execution-limits-shared/index.js";
 import { z } from "@hono/zod-openapi";
+import { StreamableHTTPClientTransport } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 import { CallToolResultSchema } from "@modelcontextprotocol/sdk/types.js";
 import { Client } from "@modelcontextprotocol/sdk/client/index.js";
 import { SSEClientTransport } from "@modelcontextprotocol/sdk/client/sse.js";
-import { StreamableHTTPClientTransport } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 import { DEFAULT_REQUEST_TIMEOUT_MSEC } from "@modelcontextprotocol/sdk/shared/protocol.js";
 import { tool } from "ai";
 import { asyncExitHook } from "exit-hook";

package/dist/validation/drizzle-schema-helpers.d.ts

@@ -1,10 +1,10 @@
 import { z } from "@hono/zod-openapi";
-import * as drizzle_zod1 from "drizzle-zod";
+import * as drizzle_zod365 from "drizzle-zod";
 import { AnySQLiteTable } from "drizzle-orm/sqlite-core";
 
 //#region src/validation/drizzle-schema-helpers.d.ts
-declare function createSelectSchemaWithModifiers<T extends AnySQLiteTable>(table: T, overrides?: Partial<Record<keyof T['_']['columns'], (schema: z.ZodTypeAny) => z.ZodTypeAny>>): drizzle_zod1.BuildSchema<"select", T["_"]["columns"], drizzle_zod1.BuildRefine<T["_"]["columns"], undefined>, undefined>;
-declare function createInsertSchemaWithModifiers<T extends AnySQLiteTable>(table: T, overrides?: Partial<Record<keyof T['_']['columns'], (schema: z.ZodTypeAny) => z.ZodTypeAny>>): drizzle_zod1.BuildSchema<"insert", T["_"]["columns"], drizzle_zod1.BuildRefine<Pick<T["_"]["columns"], keyof T["$inferInsert"]>, undefined>, undefined>;
+declare function createSelectSchemaWithModifiers<T extends AnySQLiteTable>(table: T, overrides?: Partial<Record<keyof T['_']['columns'], (schema: z.ZodTypeAny) => z.ZodTypeAny>>): drizzle_zod365.BuildSchema<"select", T["_"]["columns"], drizzle_zod365.BuildRefine<T["_"]["columns"], undefined>, undefined>;
+declare function createInsertSchemaWithModifiers<T extends AnySQLiteTable>(table: T, overrides?: Partial<Record<keyof T['_']['columns'], (schema: z.ZodTypeAny) => z.ZodTypeAny>>): drizzle_zod365.BuildSchema<"insert", T["_"]["columns"], drizzle_zod365.BuildRefine<Pick<T["_"]["columns"], keyof T["$inferInsert"]>, undefined>, undefined>;
 declare const createSelectSchema: typeof createSelectSchemaWithModifiers;
 declare const createInsertSchema: typeof createInsertSchemaWithModifiers;
 /**

package/dist/validation/schemas/skills.d.ts

@@ -337,8 +337,8 @@ declare const SkillFileInsertSchema: z.ZodObject<{
 }>;
 declare const SkillInsertSchema: z.ZodObject<{
   name: z.ZodString;
-  description: z.ZodString;
   metadata: z.ZodOptional<z.ZodNullable<z.ZodRecord<z.ZodString, z.ZodString>>>;
+  description: z.ZodString;
   content: z.ZodString;
 }, {
   out: {};
@@ -353,10 +353,10 @@ declare const SkillUpdateSchema: z.ZodObject<{
 declare const SkillApiSelectSchema: z.ZodObject<{
   id: z.ZodString;
   name: z.ZodString;
-  description: z.ZodString;
   createdAt: z.ZodString;
   updatedAt: z.ZodString;
   metadata: z.ZodNullable<z.ZodRecord<z.ZodString, z.ZodString>>;
+  description: z.ZodString;
   content: z.ZodString;
 }, z.core.$strip>;
 declare const SkillApiInsertSchema: z.ZodPipe<z.ZodPipe<z.ZodObject<{
@@ -426,10 +426,10 @@ declare const SkillFileApiUpdateSchema: z.ZodObject<{
 declare const SkillWithFilesApiSelectSchema: z.ZodObject<{
   id: z.ZodString;
   name: z.ZodString;
-  description: z.ZodString;
   createdAt: z.ZodString;
   updatedAt: z.ZodString;
   metadata: z.ZodNullable<z.ZodRecord<z.ZodString, z.ZodString>>;
+  description: z.ZodString;
   content: z.ZodString;
   files: z.ZodArray<z.ZodObject<{
     id: z.ZodString;
@@ -516,10 +516,10 @@ declare const SubAgentSkillApiUpdateSchema: z.ZodObject<{
 declare const SubAgentSkillWithIndexSchema: z.ZodObject<{
   id: z.ZodString;
   name: z.ZodString;
-  description: z.ZodString;
   createdAt: z.ZodString;
   updatedAt: z.ZodString;
   metadata: z.ZodNullable<z.ZodRecord<z.ZodString, z.ZodString>>;
+  description: z.ZodString;
   content: z.ZodString;
   subAgentSkillId: z.ZodString;
   subAgentId: z.ZodString;
@@ -540,10 +540,10 @@ declare const SkillWithFilesResponse: z.ZodObject<{
   data: z.ZodObject<{
     id: z.ZodString;
     name: z.ZodString;
-    description: z.ZodString;
     createdAt: z.ZodString;
     updatedAt: z.ZodString;
     metadata: z.ZodNullable<z.ZodRecord<z.ZodString, z.ZodString>>;
+    description: z.ZodString;
     content: z.ZodString;
     files: z.ZodArray<z.ZodObject<{
       id: z.ZodString;
@@ -559,10 +559,10 @@ declare const SkillListResponse: z.ZodObject<{
   data: z.ZodArray<z.ZodObject<{
     id: z.ZodString;
     name: z.ZodString;
-    description: z.ZodString;
     createdAt: z.ZodString;
     updatedAt: z.ZodString;
     metadata: z.ZodNullable<z.ZodRecord<z.ZodString, z.ZodString>>;
+    description: z.ZodString;
     content: z.ZodString;
   }, z.core.$strip>>;
   pagination: z.ZodObject<{
@@ -587,10 +587,10 @@ declare const SubAgentSkillWithIndexArrayResponse: z.ZodObject<{
   data: z.ZodArray<z.ZodObject<{
     id: z.ZodString;
     name: z.ZodString;
-    description: z.ZodString;
     createdAt: z.ZodString;
     updatedAt: z.ZodString;
     metadata: z.ZodNullable<z.ZodRecord<z.ZodString, z.ZodString>>;
+    description: z.ZodString;
     content: z.ZodString;
     subAgentSkillId: z.ZodString;
     subAgentId: z.ZodString;