Do not cache the toolsList per default (#128)

Author: mjakl

.changeset/ten-lizards-deny.md

@@ -0,0 +1,5 @@
+---
+'@openai/agents-core': minor
+---
+
+Don't enable `cacheToolsList` per default for MCP servers

packages/agents-core/src/mcp.ts

@@ -45,7 +45,7 @@ export abstract class BaseMCPServerStdio implements MCPServer {
   constructor(options: MCPServerStdioOptions) {
     this.logger =
       options.logger ?? getLogger(DEFAULT_STDIO_MCP_CLIENT_LOGGER_NAME);
-    this.cacheToolsList = options.cacheToolsList ?? true;
+    this.cacheToolsList = options.cacheToolsList ?? false;
   }
 
   abstract get name(): string;
@@ -78,7 +78,7 @@ export abstract class BaseMCPServerStreamableHttp implements MCPServer {
     this.logger =
       options.logger ??
       getLogger(DEFAULT_STREAMABLE_HTTP_MCP_CLIENT_LOGGER_NAME);
-    this.cacheToolsList = options.cacheToolsList ?? true;
+    this.cacheToolsList = options.cacheToolsList ?? false;
   }
 
   abstract get name(): string;

packages/agents-core/test/mcpCache.test.ts

@@ -9,6 +9,7 @@ class StubServer extends NodeMCPServerStdio {
   constructor(name: string, tools: any[]) {
     super({ command: 'noop', name });
     this.toolList = tools;
+    this.cacheToolsList = true;
   }
   async connect(): Promise<void> {}
   async close(): Promise<void> {}