fast-agent-mcp 0.2.48__tar.gz → 0.2.49__tar.gz

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fast-agent-mcp
-Version: 0.2.48
+Version: 0.2.49
 Summary: Define, Prompt and Test MCP enabled Agents and Workflows
 Author-email: Shaun Smith <fastagent@llmindset.co.uk>
 License:                                  Apache License

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "fast-agent-mcp"
-version = "0.2.48"
+version = "0.2.49"
 description = "Define, Prompt and Test MCP enabled Agents and Workflows"
 readme = "README.md"
 license = { file = "LICENSE" }

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/src/mcp_agent/config.py

@@ -8,6 +8,7 @@ import re
 from pathlib import Path
 from typing import Any, Dict, List, Literal, Optional, Tuple
 
+from mcp import Implementation
 from pydantic import BaseModel, ConfigDict, field_validator
 from pydantic_settings import BaseSettings, SettingsConfigDict
 
@@ -61,11 +62,9 @@ class MCPServerSettings(BaseModel):
     Represents the configuration for an individual server.
     """
 
-    # TODO: saqadri - server name should be something a server can provide itself during initialization
     name: str | None = None
     """The name of the server."""
 
-    # TODO: saqadri - server description should be something a server can provide itself during initialization
     description: str | None = None
     """The description of the server."""
 
@@ -108,6 +107,8 @@ class MCPServerSettings(BaseModel):
     cwd: str | None = None
     """Working directory for the executed server command."""
 
+    implementation: Implementation | None = None
+
 
 class MCPSettings(BaseModel):
     """Configuration for all MCP servers."""

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/src/mcp_agent/llm/model_factory.py

@@ -143,6 +143,7 @@ class ModelFactory:
         "gemini2": "gemini-2.0-flash",
         "gemini25": "gemini-2.5-flash-preview-05-20",
         "gemini25pro": "gemini-2.5-pro-preview-05-06",
+        "kimi": "groq.moonshotai/kimi-k2-instruct",
     }
 
     # Mapping of providers to their LLM classes

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/src/mcp_agent/llm/providers/augmented_llm_azure.py

@@ -18,7 +18,7 @@ def _extract_resource_name(url: str) -> str | None:
     return host.replace(suffix, "") if host.endswith(suffix) else None
 
 
-DEFAULT_AZURE_API_VERSION = "2023-05-15"
+DEFAULT_AZURE_API_VERSION = "2024-10-21"
 
 
 class AzureOpenAIAugmentedLLM(OpenAIAugmentedLLM):

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/src/mcp_agent/llm/providers/augmented_llm_openai.py

@@ -412,7 +412,9 @@ class OpenAIAugmentedLLM(AugmentedLLM[ChatCompletionMessageParam, ChatCompletion
                     )
 
                 tool_results = []
+                
                 for tool_call in message.tool_calls:
+                    
                     self.show_tool_call(
                         available_tools,
                         tool_call.function.name,
@@ -428,12 +430,20 @@ class OpenAIAugmentedLLM(AugmentedLLM[ChatCompletionMessageParam, ChatCompletion
                             else from_json(tool_call.function.arguments, allow_partial=True),
                         ),
                     )
-                    result = await self.call_tool(tool_call_request, tool_call.id)
-                    self.show_tool_result(result)
-
-                    tool_results.append((tool_call.id, result))
-                    responses.extend(result.content)
-                messages.extend(OpenAIConverter.convert_function_results_to_openai(tool_results))
+                    
+                    try:
+                        result = await self.call_tool(tool_call_request, tool_call.id)
+                        self.show_tool_result(result)
+                        tool_results.append((tool_call.id, result))
+                        responses.extend(result.content)
+                    except Exception as e:
+                        self.logger.error(f"Tool call {tool_call.id} failed with error: {e}")
+                        # Still add the tool_call_id with an error result to prevent missing responses
+                        error_result = CallToolResult(content=[TextContent(type="text", text=f"Tool call failed: {str(e)}")])
+                        tool_results.append((tool_call.id, error_result))
+                
+                converted_messages = OpenAIConverter.convert_function_results_to_openai(tool_results)
+                messages.extend(converted_messages)
 
                 self.logger.debug(
                     f"Iteration {i}: Tool call results: {str(tool_results) if tool_results else 'None'}"

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/src/mcp_agent/llm/providers/multipart_converter_openai.py

@@ -441,9 +441,6 @@ class OpenAIConverter:
         # Convert to OpenAI format
         user_message = OpenAIConverter.convert_to_openai(non_text_multipart)
 
-        # We need to add tool_call_id manually
-        user_message["tool_call_id"] = tool_call_id
-
         return (tool_message, [user_message])
 
     @staticmethod
@@ -461,22 +458,42 @@ class OpenAIConverter:
         Returns:
             List of OpenAI API messages for tool responses
         """
-        messages = []
+        tool_messages = []
+        user_messages = []
+        has_mixed_content = False
 
         for tool_call_id, result in results:
-            converted = OpenAIConverter.convert_tool_result_to_openai(
-                tool_result=result,
-                tool_call_id=tool_call_id,
-                concatenate_text_blocks=concatenate_text_blocks,
-            )
-
-            # Handle the case where we have mixed content and get back a tuple
-            if isinstance(converted, tuple):
-                tool_message, additional_messages = converted
-                messages.append(tool_message)
-                messages.extend(additional_messages)
-            else:
-                # Single message case (text-only)
-                messages.append(converted)
+            try:
+                converted = OpenAIConverter.convert_tool_result_to_openai(
+                    tool_result=result,
+                    tool_call_id=tool_call_id,
+                    concatenate_text_blocks=concatenate_text_blocks,
+                )
 
+                # Handle the case where we have mixed content and get back a tuple
+                if isinstance(converted, tuple):
+                    tool_message, additional_messages = converted
+                    tool_messages.append(tool_message)
+                    user_messages.extend(additional_messages)
+                    has_mixed_content = True
+                else:
+                    # Single message case (text-only)
+                    tool_messages.append(converted)
+            except Exception as e:
+                _logger.error(f"Failed to convert tool_call_id={tool_call_id}: {e}")
+                # Create a basic tool response to prevent missing tool_call_id error
+                fallback_message = {
+                    "role": "tool",
+                    "tool_call_id": tool_call_id,
+                    "content": f"[Conversion error: {str(e)}]",
+                }
+                tool_messages.append(fallback_message)
+
+        # CONDITIONAL REORDERING: Only reorder if there are user messages (mixed content)
+        if has_mixed_content and user_messages:
+            # Reorder: All tool messages first (OpenAI sequence), then user messages (vision context)
+            messages = tool_messages + user_messages
+        else:
+            # Pure tool responses - keep original order to preserve context (snapshots, etc.)
+            messages = tool_messages
         return messages

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/src/mcp_agent/logging/rich_progress.py

@@ -21,7 +21,7 @@ class RichProgressDisplay:
         self._progress = Progress(
             SpinnerColumn(spinner_name="simpleDotsScrolling"),
             TextColumn(
-                "[progress.description]{task.description}|",
+                "[progress.description]{task.description}▎",
                 #                table_column=Column(max_width=16),
             ),
             TextColumn(text_format="{task.fields[target]:<16}", style="Bold Blue"),
@@ -77,7 +77,7 @@ class RichProgressDisplay:
             ProgressAction.LOADED: "dim green",
             ProgressAction.INITIALIZED: "dim green",
             ProgressAction.CHATTING: "bold blue",
-            ProgressAction.STREAMING: "bold blue",  # Same color as chatting
+            ProgressAction.STREAMING: "bold green",  # Assistant Colour
             ProgressAction.ROUTING: "bold blue",
             ProgressAction.PLANNING: "bold blue",
             ProgressAction.READY: "dim green",
@@ -107,10 +107,19 @@ class RichProgressDisplay:
         # Ensure no None values in the update
         # For streaming, use custom description immediately to avoid flashing
         if event.action == ProgressAction.STREAMING and event.streaming_tokens:
-            formatted_tokens = f"↓ {event.streaming_tokens.strip()}".ljust(15)
+            # Account for [dim][/dim] tags (11 characters) in padding calculation
+            formatted_tokens = f"▎[dim]◀[/dim] {event.streaming_tokens.strip()}".ljust(17 + 11)
             description = f"[{self._get_action_style(event.action)}]{formatted_tokens}"
+        elif event.action == ProgressAction.CHATTING:
+            # Add special formatting for chatting with dimmed arrow
+            formatted_text = f"▎[dim]▶[/dim] {event.action.value.strip()}".ljust(17 + 11)
+            description = f"[{self._get_action_style(event.action)}]{formatted_text}"
+        elif event.action == ProgressAction.CALLING_TOOL:
+            # Add special formatting for calling tool with dimmed arrow
+            formatted_text = f"▎[dim]◀[/dim] {event.action.value}".ljust(17 + 11)
+            description = f"[{self._get_action_style(event.action)}]{formatted_text}"
         else:
-            description = f"[{self._get_action_style(event.action)}]{event.action.value:<15}"
+            description = f"[{self._get_action_style(event.action)}]▎ {event.action.value:<15}"
 
         self._progress.update(
             task_id,

{fast_agent_mcp-0.2.48 → fast_agent_mcp-0.2.49}/src/mcp_agent/mcp/mcp_agent_client_session.py

@@ -66,9 +66,6 @@ class MCPAgentClientSession(ClientSession, ContextDependent):
         # Extract server_name if provided in kwargs
         from importlib.metadata import version
 
-        version = version("fast-agent-mcp") or "dev"
-        fast_agent: Implementation = Implementation(name="fast-agent-mcp", version=version)
-
         self.session_server_name = kwargs.pop("server_name", None)
         # Extract the notification callbacks if provided
         self._tool_list_changed_callback = kwargs.pop("tool_list_changed_callback", None)
@@ -83,6 +80,11 @@ class MCPAgentClientSession(ClientSession, ContextDependent):
         # Extract custom elicitation handler if provided
         custom_elicitation_handler = kwargs.pop("elicitation_handler", None)
 
+        version = version("fast-agent-mcp") or "dev"
+        fast_agent: Implementation = Implementation(name="fast-agent-mcp", version=version)
+        if self.server_config and self.server_config.implementation:
+            fast_agent = self.server_config.implementation
+
         # Only register callbacks if the server_config has the relevant settings
         list_roots_cb = list_roots if (self.server_config and self.server_config.roots) else None