@slashfi/agents-sdk 0.71.4 → 0.73.0

package/src/registry-consumer.ts

@@ -39,6 +39,7 @@ import type {
   ResolvedRegistry,
 } from "./define-config.js";
 import type { CallAgentRequest } from "./call-agent-schema.js";
+import type { FetchFn } from "./fetch-types.js";
 import type { SecuritySchemeSummary, CallAgentResponse } from "./types.js";
 import {
   isSecretUri,
@@ -331,7 +332,7 @@ async function defaultSecretResolver(
 async function listFromMcpServer(
   url: string,
   auth: { token?: string; headers?: Record<string, string> },
-  fetchFn: typeof globalThis.fetch,
+  fetchFn: FetchFn,
 ): Promise<AgentListing[]> {
   const serverUrl = url.replace(/\/$/, "");
 
@@ -409,7 +410,7 @@ function issuerFromMcpUrlAndServerInfo(
 async function discoverRegistryViaMcp(
   registryUrl: string,
   authHeaders: Record<string, string>,
-  fetchFn: typeof globalThis.fetch,
+  fetchFn: FetchFn,
 ): Promise<RegistryConfiguration> {
   const serverUrl = registryUrl.replace(/\/$/, "");
   const headers: Record<string, string> = {
@@ -467,7 +468,7 @@ async function callMcpTool(
   toolName: string,
   params: Record<string, unknown>,
   auth: { token?: string; headers?: Record<string, string> },
-  fetchFn: typeof globalThis.fetch,
+  fetchFn: FetchFn,
 ): Promise<unknown> {
   const serverUrl = url.replace(/\/$/, "");
   const headers: Record<string, string> = {
@@ -539,7 +540,7 @@ async function callHttpsTool(
   _toolName: string,
   params: Record<string, unknown>,
   auth: { token?: string; headers?: Record<string, string> },
-  fetchFn: typeof globalThis.fetch,
+  fetchFn: FetchFn,
 ): Promise<unknown> {
   const method = (params.method as string) ?? "GET";
   const path = (params.path as string) ?? "";
@@ -582,8 +583,13 @@ export interface RegistryConsumerOptions {
   /** Bearer token for authenticated registries */
   token?: string;
 
-  /** Custom fetch implementation */
-  fetch?: typeof globalThis.fetch;
+  /**
+   * Custom fetch implementation. Forwarded to every outbound HTTP call the
+   * consumer makes (discovery, jwks, callRegistry, secret resolve). Hosts
+   * running in long-lived servers should pass a hardened fetch to avoid
+   * dead-socket hangs on rolling deploys.
+   */
+  fetch?: FetchFn;
 }
 
 // ============================================
@@ -651,7 +657,7 @@ export async function createRegistryConsumer(
   config: ConsumerConfig,
   options: RegistryConsumerOptions = {},
 ): Promise<RegistryConsumer> {
-  const fetchFn = options.fetch ?? globalThis.fetch;
+  const fetchFn: FetchFn = options.fetch ?? globalThis.fetch;
   const resolveSecretFn = options.resolveSecret ?? defaultSecretResolver;
 
   // Normalize registries

package/src/registry.ts

@@ -7,6 +7,7 @@
 import { dirname, resolve } from "node:path";
 import type { AgentEvent, BaseEvent, CallAgentToolCallEvent, CustomEventMap, EventCallback, EventType, ListAgentsResult, ListAgentsToolCallEvent } from "./events.js";
 import { createEventBus } from "./events.js";
+import type { Logger } from "./logger.js";
 import type { SerializedAgentDefinition } from "./serialized.js";
 import type {
   AgentAction,
@@ -87,7 +88,11 @@ export interface AgentRegistryOptions {
   contextFactory?: ContextFactory;
   /** Lifecycle middleware hooks */
   middleware?: RegistryMiddleware;
-
+  /**
+   * Structured logger for SDK-internal events (listener errors, etc.).
+   * Defaults to the module-level default logger (JSON to stdout/stderr).
+   */
+  logger?: Logger;
 }
 
 /**
@@ -231,7 +236,7 @@ export function createAgentRegistry(
 ): AgentRegistry {
   const { defaultVisibility = "internal" } = options;
   const agents = new Map<string, AgentDefinition>();
-  const eventBus = createEventBus();
+  const eventBus = createEventBus({ logger: options.logger });
 
   /**
    * Check if agent supports the requested action.

package/src/server.ts

@@ -44,6 +44,7 @@ import {
   verifyJwtFromIssuer,
   verifyJwtLocal,
 } from "./jwt.js";
+import { getDefaultLogger, type Logger } from "./logger.js";
 import { type OIDCProviderConfig, createOIDCSignIn } from "./oidc-signin.js";
 import type { AgentRegistry } from "./registry.js";
 import {
@@ -206,6 +207,12 @@ export interface AgentServerOptions {
     /** OAuth callback URL for shared OAuth flows */
     oauthCallbackUrl?: string;
   };
+  /**
+   * Structured logger for server-side errors (tool-call failures, JWT
+   * exchange errors, request errors, etc.). Defaults to the module-level
+   * default logger (single-line JSON).
+   */
+  logger?: Logger;
 }
 
 export interface AgentServer {
@@ -522,6 +529,7 @@ export function createAgentServer(
     secretStore,
     oauthIdentityProvider,
   } = options;
+  const logger = options.logger ?? getDefaultLogger();
 
   // Build tool definitions and validation schemas from overrides
   const toolDefs = getToolDefinitions(options.schemas);
@@ -607,7 +615,13 @@ export function createAgentServer(
           const result = await handleToolCall(name, args ?? {}, auth);
           return jsonRpcSuccess(request.id, result);
         } catch (err) {
-          console.error("[server] Tool call error:", err);
+          logger.error("mcp_tool_call_error", {
+            component: "agents-sdk.server",
+            tool: name,
+            caller_id: auth?.callerId,
+            caller_type: auth?.callerType,
+            error: err,
+          });
           return jsonRpcSuccess(
             request.id,
             mcpResult(
@@ -907,22 +921,24 @@ export function createAgentServer(
                 callerType: "system",
               });
               if (addResult.success) {
-                console.error(
-                  `[jwt_exchange] Reverse connection stored for ${assertionPayload.iss}`,
-                );
+                logger.info("jwt_exchange_reverse_connection_stored", {
+                  component: "agents-sdk.server",
+                  issuer: assertionPayload.iss,
+                });
               } else {
-                console.error(
-                  "[jwt_exchange] Reverse connection failed:",
-                  addResult.error,
-                );
+                logger.error("jwt_exchange_reverse_connection_failed", {
+                  component: "agents-sdk.server",
+                  issuer: assertionPayload.iss,
+                  error: addResult.error,
+                });
               }
             }
           }
         } catch (reverseErr) {
-          console.error(
-            "[jwt_exchange] Reverse registration check failed:",
-            reverseErr,
-          );
+          logger.error("jwt_exchange_reverse_registration_failed", {
+            component: "agents-sdk.server",
+            error: reverseErr,
+          });
         }
 
         if (!exchangeResult) {
@@ -988,7 +1004,10 @@ export function createAgentServer(
 
         return jsonResponse(exchangeResult);
       } catch (err) {
-        console.error("[oauth] JWT exchange error:", err);
+        logger.error("oauth_jwt_exchange_error", {
+          component: "agents-sdk.server",
+          error: err,
+        });
         return jsonResponse(
           {
             error: "server_error",
@@ -1061,7 +1080,10 @@ export function createAgentServer(
           refresh_token: tokenResult.refreshToken,
         });
       } catch (err) {
-        console.error("[oauth] Token error:", err);
+        logger.error("oauth_token_error", {
+          component: "agents-sdk.server",
+          error: err,
+        });
         return jsonResponse(
           { error: "server_error", error_description: "Token exchange failed" },
           500,
@@ -1190,36 +1212,31 @@ export function createAgentServer(
         const allIssuerUrls = [
           ...new Set([...storeIssuers, ...configIssuerUrls]),
         ];
-        console.log(
-          "[oauth/authorize] storeIssuers:",
-          storeIssuers.length,
-          "configIssuers:",
-          configIssuerUrls.length,
-          "total:",
-          allIssuerUrls.length,
-          "urls:",
-          allIssuerUrls,
-        );
+        logger.debug("oauth_authorize_issuer_check", {
+          component: "agents-sdk.server",
+          store_issuers: storeIssuers.length,
+          config_issuers: configIssuerUrls.length,
+          total_issuers: allIssuerUrls.length,
+          issuer_urls: allIssuerUrls,
+        });
         for (const issuerUrl of allIssuerUrls) {
           try {
             const result = await verifyJwtFromIssuer(token, issuerUrl);
-            console.log(
-              "[oauth/authorize] verify",
-              issuerUrl,
-              "->",
-              result ? "OK" : "null",
-            );
+            logger.debug("oauth_authorize_verify", {
+              component: "agents-sdk.server",
+              issuer: issuerUrl,
+              ok: !!result,
+            });
             if (result) {
               claims = result as unknown as Record<string, unknown>;
               break;
             }
           } catch (e: unknown) {
-            console.log(
-              "[oauth/authorize] verify",
-              issuerUrl,
-              "-> ERROR:",
-              e instanceof Error ? e.message : String(e),
-            );
+            logger.debug("oauth_authorize_verify_error", {
+              component: "agents-sdk.server",
+              issuer: issuerUrl,
+              error: e,
+            });
           }
         }
         if (!claims) {
@@ -1500,7 +1517,12 @@ export function createAgentServer(
             const res = jsonResponse(jsonRpcSuccess(body.id, result));
             return cors ? addCors(res) : res;
           } catch (err) {
-            console.error("[server] Scoped tool call error:", err);
+            logger.error("mcp_scoped_tool_call_error", {
+              component: "agents-sdk.server",
+              caller_id: effectiveAuth?.callerId,
+              caller_type: effectiveAuth?.callerType,
+              error: err,
+            });
             const res = jsonResponse(
               jsonRpcSuccess(
                 body.id,
@@ -1534,7 +1556,12 @@ export function createAgentServer(
       );
       return cors ? addCors(res) : res;
     } catch (err) {
-      console.error("[server] Request error:", err);
+      logger.error("mcp_request_error", {
+        component: "agents-sdk.server",
+        method: req.method,
+        url: req.url,
+        error: err,
+      });
       const res = jsonResponse(
         {
           jsonrpc: "2.0",
@@ -1586,9 +1613,12 @@ export function createAgentServer(
         fetch,
       });
       this.url = `http://${hostname}:${port}`;
-      console.log(
-        `[agents-sdk] Server listening on http://${hostname}:${port}`,
-      );
+      logger.info("agent_server_listening", {
+        component: "agents-sdk.server",
+        url: this.url,
+        hostname,
+        port,
+      });
     },
 
     async stop() {
@@ -1644,7 +1674,11 @@ export function createAgentServer(
           issuer: issuerUrl,
           scopes: scopes ?? ["*"],
         });
-        console.error(`[agent-server] Added trusted issuer: ${issuerUrl}`);
+        logger.info("trusted_issuer_added", {
+          component: "agents-sdk.server",
+          issuer: issuerUrl,
+          scopes: scopes ?? ["*"],
+        });
       }
     },
   };