Fixed bug where full tool list not passed to LLM. Better debug output.

Author: steviec

src/commands/evals/providers/anthropic-provider.ts

@@ -23,18 +23,10 @@ export class AnthropicProvider extends LlmProvider {
       const toolsResponse = await mcpClient.sdk.listTools();
       const mcpTools = toolsResponse.tools || [];
 
-      // Filter tools based on allowed list if specified
-      let toolsToUse = mcpTools;
-      if (config.allowedTools !== undefined) {
-        toolsToUse = mcpTools.filter((tool: { name: string }) =>
-          config.allowedTools!.includes(tool.name)
-        );
-      }
-
       // Convert MCP tools to AI SDK format using tool() helper
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const aiTools: Record<string, any> = {};
-      for (const mcpTool of toolsToUse) {
+      for (const mcpTool of mcpTools) {
         aiTools[mcpTool.name] = tool({
           description: mcpTool.description,
           parameters: jsonSchema(mcpTool.inputSchema as object),

src/shared/display/utils/conversationFormatter.ts

@@ -109,15 +109,15 @@ export function formatContent(content: any): string[] {
  * Format tool call
  */
 export function formatToolCall(toolName: string, args: any): string {
-  const argsStr = JSON.stringify(args);
+  const argsStr = JSON.stringify(args, null, 2);
   return formatText(`🔧 Tool Call: ${toolName}(${argsStr})`);
 }
 
 /**
  * Format tool result
  */
 export function formatToolResult(result: any): string {
-  const resultStr = JSON.stringify(result);
-  const truncated = resultStr.length > 200 ? `${resultStr.substring(0, 200)}...` : resultStr;
-  return formatText(`✅ Tool Result: ${truncated}`);
+  return formatText(
+    `✅ Tool Result: ${JSON.stringify(result.structuredContent ?? result.content, null, 2)}`
+  );
 }