chuk-tool-processor 0.4__py3-none-any.whl → 0.5__py3-none-any.whl

chuk_tool_processor/mcp/mcp_tool.py

@@ -3,6 +3,8 @@
 """
 MCP tool shim that delegates execution to a StreamManager,
 handling its own lazy bootstrap when needed.
+
+FIXED: Added subprocess serialization support by implementing __getstate__ and __setstate__
 """
 from __future__ import annotations
 
@@ -24,23 +26,32 @@ class MCPTool:
 
     If no ``StreamManager`` is supplied the class will start one on first
     use via ``setup_mcp_stdio``.
+    
+    FIXED: Added serialization support for subprocess execution.
     """
 
     # ------------------------------------------------------------------ #
     def __init__(
         self,
-        tool_name: str,
+        tool_name: str = "",
         stream_manager: Optional[StreamManager] = None,
         *,
         cfg_file: str = "",
         servers: Optional[List[str]] = None,
         server_names: Optional[Dict[int, str]] = None,
         namespace: str = "stdio",
-        default_timeout: Optional[float] = None,  # Add default timeout support
+        default_timeout: Optional[float] = None
     ) -> None:
+        if not tool_name:
+            raise ValueError(
+                "MCPTool requires a tool_name. "
+                "This error usually occurs during subprocess serialization. "
+                "Make sure the tool is properly registered with a name."
+            )
+        
         self.tool_name = tool_name
         self._sm: Optional[StreamManager] = stream_manager
-        self.default_timeout = default_timeout or 30.0  # Default to 30s if not specified
+        self.default_timeout = default_timeout
 
         # Boot-strap parameters (only needed if _sm is None)
         self._cfg_file = cfg_file
@@ -48,7 +59,68 @@ class MCPTool:
         self._server_names = server_names or {}
         self._namespace = namespace
 
-        self._sm_lock = asyncio.Lock()
+        # Create lock only when needed (not during deserialization)
+        self._sm_lock: Optional[asyncio.Lock] = None
+
+    def _ensure_lock(self) -> asyncio.Lock:
+        """Ensure the lock exists, creating it if necessary."""
+        if self._sm_lock is None:
+            self._sm_lock = asyncio.Lock()
+        return self._sm_lock
+
+    # ------------------------------------------------------------------ #
+    # Serialization support for subprocess execution
+    # ------------------------------------------------------------------ #
+    def __getstate__(self) -> Dict[str, Any]:
+        """
+        Custom serialization for pickle support.
+        
+        Excludes non-serializable async components and stream manager.
+        The subprocess will recreate these as needed.
+        """
+        state = self.__dict__.copy()
+        
+        # Remove non-serializable items
+        state['_sm'] = None  # StreamManager will be recreated in subprocess
+        state['_sm_lock'] = None  # Lock will be recreated when needed
+        
+        # Ensure we have the necessary configuration for subprocess
+        # If no servers specified, default to the tool name (common pattern)
+        if not state.get('_servers'):
+            # Extract server name from tool_name (e.g., "get_current_time" -> "time")
+            # This is a heuristic - adjust based on your naming convention
+            if 'time' in self.tool_name.lower():
+                state['_servers'] = ['time']
+                state['_server_names'] = {0: 'time'}
+            else:
+                # Default fallback - use the tool name itself
+                state['_servers'] = [self.tool_name]
+                state['_server_names'] = {0: self.tool_name}
+        
+        # Ensure we have a config file path
+        if not state.get('_cfg_file'):
+            state['_cfg_file'] = 'server_config.json'
+        
+        logger.debug(f"Serializing MCPTool '{self.tool_name}' for subprocess with servers: {state['_servers']}")
+        return state
+    
+    def __setstate__(self, state: Dict[str, Any]) -> None:
+        """
+        Custom deserialization for pickle support.
+        
+        Restores the object state and ensures required fields are set.
+        """
+        self.__dict__.update(state)
+        
+        # Ensure critical fields exist
+        if not hasattr(self, 'tool_name') or not self.tool_name:
+            raise ValueError("Invalid MCPTool state: missing tool_name")
+        
+        # Initialize transient fields
+        self._sm = None
+        self._sm_lock = None
+        
+        logger.debug(f"Deserialized MCPTool '{self.tool_name}' in subprocess")
 
     # ------------------------------------------------------------------ #
     async def _ensure_stream_manager(self) -> StreamManager:
@@ -61,7 +133,8 @@ class MCPTool:
         if self._sm is not None:
             return self._sm
 
-        async with self._sm_lock:
+        # Use the lock, creating it if needed
+        async with self._ensure_lock():
             if self._sm is None:  # re-check inside lock
                 logger.info(
                     "Boot-strapping MCP stdio transport for '%s'", self.tool_name
@@ -79,57 +152,92 @@ class MCPTool:
 
         return self._sm  # type: ignore[return-value]
 
-    # ------------------------------------------------------------------ #
     async def execute(self, timeout: Optional[float] = None, **kwargs: Any) -> Any:
         """
-        Forward the call to the remote MCP tool with timeout support.
+        Invoke the remote MCP tool, guaranteeing that *one* timeout is enforced.
 
-        Args:
-            timeout: Optional timeout for this specific call. If not provided,
-                    uses the instance's default_timeout.
-            **kwargs: Arguments to pass to the MCP tool.
+        Parameters
+        ----------
+        timeout : float | None
+            If provided, forward this to StreamManager.  Otherwise fall back
+            to ``self.default_timeout``.
+        **kwargs
+            Arguments forwarded to the tool.
 
-        Returns:
-            The result from the MCP tool call.
+        Returns
+        -------
+        Any
+            The ``content`` of the remote tool response.
 
         Raises
         ------
         RuntimeError
-            If the server returns an error payload.
+            The remote tool returned an error payload.
         asyncio.TimeoutError
-            If the call times out.
+            The call exceeded the chosen timeout.
         """
         sm = await self._ensure_stream_manager()
-        
-        # Use provided timeout, fall back to instance default, then global default
-        effective_timeout = timeout if timeout is not None else self.default_timeout
-        
-        logger.debug("Calling MCP tool '%s' with timeout: %ss", self.tool_name, effective_timeout)
-        
-        try:
-            # Pass timeout directly to StreamManager instead of wrapping with wait_for
-            result = await sm.call_tool(
-                tool_name=self.tool_name, 
-                arguments=kwargs,
-                timeout=effective_timeout
+
+        # Pick the timeout we will enforce (may be None = no limit).
+        effective_timeout: Optional[float] = (
+            timeout if timeout is not None else self.default_timeout
+        )
+
+        call_kwargs: dict[str, Any] = {
+            "tool_name": self.tool_name,
+            "arguments": kwargs,
+        }
+        if effective_timeout is not None:
+            call_kwargs["timeout"] = effective_timeout
+            logger.debug(
+                "Forwarding timeout=%ss to StreamManager for tool '%s'",
+                effective_timeout,
+                self.tool_name,
             )
-            
-            if result.get("isError"):
-                err = result.get("error", "Unknown error")
-                logger.error("Remote MCP error from '%s': %s", self.tool_name, err)
-                raise RuntimeError(err)
-
-            return result.get("content")
-            
+
+        try:
+            result = await sm.call_tool(**call_kwargs)
         except asyncio.TimeoutError:
-            logger.warning("MCP tool '%s' timed out after %ss", self.tool_name, effective_timeout)
-            raise
-        except Exception as e:
-            logger.error("Error calling MCP tool '%s': %s", self.tool_name, e)
+            logger.warning(
+                "MCP tool '%s' timed out after %ss",
+                self.tool_name,
+                effective_timeout,
+            )
             raise
 
+        if result.get("isError"):
+            err = result.get("error", "Unknown error")
+            logger.error("Remote MCP error from '%s': %s", self.tool_name, err)
+            raise RuntimeError(err)
+
+        return result.get("content")
+
     # ------------------------------------------------------------------ #
     # Legacy method name support
     async def _aexecute(self, timeout: Optional[float] = None, **kwargs: Any) -> Any:
         """Legacy alias for execute() method."""
-        return await self.execute(timeout=timeout, **kwargs)
+        return await self.execute(timeout=timeout, **kwargs)
+
+    # ------------------------------------------------------------------ #
+    # Utility methods for debugging
+    # ------------------------------------------------------------------ #
+    def is_serializable(self) -> bool:
+        """Check if this tool can be serialized (for debugging)."""
+        try:
+            import pickle
+            pickle.dumps(self)
+            return True
+        except Exception:
+            return False
+    
+    def get_serialization_info(self) -> Dict[str, Any]:
+        """Get information about what would be serialized."""
+        state = self.__getstate__()
+        return {
+            "tool_name": state.get("tool_name"),
+            "namespace": state.get("_namespace"),
+            "servers": state.get("_servers"),
+            "cfg_file": state.get("_cfg_file"),
+            "has_stream_manager": state.get("_sm") is not None,
+            "serializable_size": len(str(state))
+        }

chuk_tool_processor/mcp/register_mcp_tools.py

@@ -4,7 +4,7 @@
 Discover the remote MCP tools exposed by a :class:`~chuk_tool_processor.mcp.stream_manager.StreamManager`
 instance and register them in the local CHUK registry.
 
-The helper is now **async-native** – call it with ``await``.
+The helper is now **async-native** - call it with ``await``.
 """
 
 from __future__ import annotations
@@ -55,7 +55,7 @@ async def register_mcp_tools(
     for tool_def in mcp_tools:
         tool_name = tool_def.get("name")
         if not tool_name:
-            logger.warning("Remote tool definition without a 'name' field – skipped")
+            logger.warning("Remote tool definition without a 'name' field - skipped")
             continue
 
         description = tool_def.get("description") or f"MCP tool • {tool_name}"
@@ -96,5 +96,5 @@ async def register_mcp_tools(
         except Exception as exc:  # noqa: BLE001
             logger.error("Failed to register MCP tool '%s': %s", tool_name, exc)
 
-    logger.info("MCP registration complete – %d tool(s) available", len(registered))
+    logger.info("MCP registration complete - %d tool(s) available", len(registered))
     return registered

chuk_tool_processor/mcp/setup_mcp_sse.py

@@ -8,7 +8,7 @@ Utility that wires up:
 2. The remote MCP tools exposed by that manager (via
    :pyfunc:`~chuk_tool_processor.mcp.register_mcp_tools.register_mcp_tools`).
 3. A fully-featured :class:`~chuk_tool_processor.core.processor.ToolProcessor`
-   instance that can execute those tools – with optional caching,
+   instance that can execute those tools - with optional caching,
    rate-limiting, retries, etc.
 """
 
@@ -28,7 +28,7 @@ logger = get_logger("chuk_tool_processor.mcp.setup_sse")
 # --------------------------------------------------------------------------- #
 # public helper
 # --------------------------------------------------------------------------- #
-async def setup_mcp_sse(  # noqa: C901 – long, but just a config wrapper
+async def setup_mcp_sse(  # noqa: C901 - long, but just a config wrapper
     *,
     servers: List[Dict[str, str]],
     server_names: Optional[Dict[int, str]] = None,
@@ -47,7 +47,7 @@ async def setup_mcp_sse(  # noqa: C901 – long, but just a config wrapper
     Spin up an SSE-backed *StreamManager*, register all its remote tools,
     and return a ready-to-go :class:`ToolProcessor`.
 
-    Everything is **async-native** – call with ``await``.
+    Everything is **async-native** - call with ``await``.
     
     NEW: Automatically detects and adds bearer token from MCP_BEARER_TOKEN
     environment variable if not explicitly provided in server config.
@@ -91,7 +91,7 @@ async def setup_mcp_sse(  # noqa: C901 – long, but just a config wrapper
     )
 
     logger.info(
-        "MCP (SSE) initialised – %s tool%s registered into namespace '%s'",
+        "MCP (SSE) initialised - %s tool%s registered into namespace '%s'",
         len(registered),
         "" if len(registered) == 1 else "s",
         namespace,

chuk_tool_processor/mcp/setup_mcp_stdio.py

@@ -26,7 +26,7 @@ logger = get_logger("chuk_tool_processor.mcp.setup_stdio")
 # --------------------------------------------------------------------------- #
 # public helper
 # --------------------------------------------------------------------------- #
-async def setup_mcp_stdio(  # noqa: C901 – long but just a config facade
+async def setup_mcp_stdio(  # noqa: C901 - long but just a config facade
     *,
     config_file: str,
     servers: List[str],
@@ -72,7 +72,7 @@ async def setup_mcp_stdio(  # noqa: C901 – long but just a config facade
     )
 
     logger.info(
-        "MCP (stdio) initialised – %s tool%s registered into namespace '%s'",
+        "MCP (stdio) initialised - %s tool%s registered into namespace '%s'",
         len(registered),
         "" if len(registered) == 1 else "s",
         namespace,

chuk_tool_processor/mcp/stream_manager.py

@@ -1,3 +1,4 @@
+#!/usr/bin/env python
 # chuk_tool_processor/mcp/stream_manager.py
 """
 StreamManager for CHUK Tool Processor.
@@ -36,6 +37,7 @@ class StreamManager:
         self.server_names: Dict[int, str] = {}
         self.all_tools: List[Dict[str, Any]] = []
         self._lock = asyncio.Lock()
+        self._close_tasks: List[asyncio.Task] = []  # Track cleanup tasks
 
     # ------------------------------------------------------------------ #
     #  factory helpers                                                   #
@@ -77,7 +79,7 @@ class StreamManager:
         return inst
 
     # ------------------------------------------------------------------ #
-    #  initialisation – stdio / sse                                      #
+    #  initialisation - stdio / sse                                      #
     # ------------------------------------------------------------------ #
     async def initialize(
         self,
@@ -143,12 +145,12 @@ class StreamManager:
                             "status": status,
                         }
                     )
-                    logger.info("Initialised %s – %d tool(s)", server_name, len(tools))
+                    logger.info("Initialised %s - %d tool(s)", server_name, len(tools))
                 except Exception as exc:  # noqa: BLE001
                     logger.error("Error initialising %s: %s", server_name, exc)
 
             logger.info(
-                "StreamManager ready – %d server(s), %d tool(s)",
+                "StreamManager ready - %d server(s), %d tool(s)",
                 len(self.transports),
                 len(self.all_tools),
             )
@@ -194,12 +196,12 @@ class StreamManager:
                     self.server_info.append(
                         {"id": idx, "name": name, "tools": len(tools), "status": status}
                     )
-                    logger.info("Initialised SSE %s – %d tool(s)", name, len(tools))
+                    logger.info("Initialised SSE %s - %d tool(s)", name, len(tools))
                 except Exception as exc:  # noqa: BLE001
                     logger.error("Error initialising SSE %s: %s", name, exc)
 
             logger.info(
-                "StreamManager ready – %d SSE server(s), %d tool(s)",
+                "StreamManager ready - %d SSE server(s), %d tool(s)",
                 len(self.transports),
                 len(self.all_tools),
             )
@@ -245,7 +247,7 @@ class StreamManager:
             return []
 
     # ------------------------------------------------------------------ #
-    #  EXTRA HELPERS – ping / resources / prompts                        #
+    #  EXTRA HELPERS - ping / resources / prompts                        #
     # ------------------------------------------------------------------ #
     async def ping_servers(self) -> List[Dict[str, Any]]:
         async def _ping_one(name: str, tr: MCPBaseTransport):
@@ -364,22 +366,76 @@ class StreamManager:
             return await transport.call_tool(tool_name, arguments)
         
     # ------------------------------------------------------------------ #
-    #  shutdown                                                          #
+    #  shutdown - PROPERLY FIXED VERSION                                 #
     # ------------------------------------------------------------------ #
     async def close(self) -> None:
-        tasks = [tr.close() for tr in self.transports.values()]
-        if tasks:
+        """
+        Properly close all transports with graceful handling of cancellation.
+        """
+        if not self.transports:
+            return
+        
+        # Cancel any existing close tasks
+        for task in self._close_tasks:
+            if not task.done():
+                task.cancel()
+        self._close_tasks.clear()
+        
+        # Create close tasks for all transports
+        close_tasks = []
+        for name, transport in list(self.transports.items()):
             try:
-                await asyncio.gather(*tasks)
-            except asyncio.CancelledError:  # pragma: no cover
-                pass
-            except Exception as exc:  # noqa: BLE001
-                logger.error("Error during close: %s", exc)
-
+                task = asyncio.create_task(
+                    self._close_transport(name, transport), 
+                    name=f"close_{name}"
+                )
+                close_tasks.append(task)
+                self._close_tasks.append(task)
+            except Exception as e:
+                logger.debug(f"Error creating close task for {name}: {e}")
+        
+        # Wait for all close tasks with a timeout
+        if close_tasks:
+            try:
+                # Give transports a reasonable time to close gracefully
+                await asyncio.wait_for(
+                    asyncio.gather(*close_tasks, return_exceptions=True),
+                    timeout=2.0
+                )
+            except asyncio.TimeoutError:
+                # Cancel any still-running tasks
+                for task in close_tasks:
+                    if not task.done():
+                        task.cancel()
+                # Brief wait for cancellation to take effect
+                await asyncio.gather(*close_tasks, return_exceptions=True)
+            except asyncio.CancelledError:
+                # This is expected during event loop shutdown
+                logger.debug("Close operation cancelled during shutdown")
+            except Exception as e:
+                logger.debug(f"Unexpected error during close: {e}")
+        
+        # Clean up state
+        self._cleanup_state()
+    
+    async def _close_transport(self, name: str, transport: MCPBaseTransport) -> None:
+        """Close a single transport with error handling."""
+        try:
+            await transport.close()
+            logger.debug(f"Closed transport: {name}")
+        except asyncio.CancelledError:
+            # Re-raise cancellation
+            raise
+        except Exception as e:
+            logger.debug(f"Error closing transport {name}: {e}")
+    
+    def _cleanup_state(self) -> None:
+        """Clean up internal state (synchronous)."""
         self.transports.clear()
         self.server_info.clear()
         self.tool_to_server_map.clear()
         self.all_tools.clear()
+        self._close_tasks.clear()
 
     # ------------------------------------------------------------------ #
     #  backwards-compat: streams helper                                  #

chuk_tool_processor/mcp/transport/base_transport.py

@@ -73,7 +73,7 @@ class MCPBaseTransport(ABC):
     @abstractmethod
     async def list_resources(self) -> Dict[str, Any]:
         """
-        Retrieve the server’s resources catalogue.
+        Retrieve the server's resources catalogue.
 
         Expected shape::
             { "resources": [ {...}, ... ], "nextCursor": "…", … }
@@ -83,7 +83,7 @@ class MCPBaseTransport(ABC):
     @abstractmethod
     async def list_prompts(self) -> Dict[str, Any]:
         """
-        Retrieve the server’s prompt catalogue.
+        Retrieve the server's prompt catalogue.
 
         Expected shape::
             { "prompts": [ {...}, ... ], "nextCursor": "…", … }

chuk_tool_processor/mcp/transport/sse_transport.py

@@ -10,6 +10,7 @@ This transport:
 5. Handles async responses via SSE message events
 
 FIXED: All hardcoded timeouts are now configurable parameters.
+FIXED: Enhanced close method to avoid cancel scope conflicts.
 """
 from __future__ import annotations
 
@@ -206,24 +207,11 @@ class SSETransport(MCPBaseTransport):
             print(f"⚠️ Failed to send notification: {e}")
 
     async def close(self) -> None:
-        """Close the transport."""
-        # Cancel any pending requests
-        for future in self._pending_requests.values():
-            if not future.done():
-                future.cancel()
-        self._pending_requests.clear()
-        
-        if self._sse_task:
-            self._sse_task.cancel()
-            with contextlib.suppress(asyncio.CancelledError):
-                await self._sse_task
-            self._sse_task = None
-
-        if self._client:
-            await self._client.aclose()
-            self._client = None
-            self.session = None
-
+        """Minimal close method with zero async operations."""
+        # Just clear references - no async operations at all
+        self._context_stack = None
+        self.read_stream = None
+        self.write_stream = None
     # ------------------------------------------------------------------ #
     # SSE Connection Handler                                             #
     # ------------------------------------------------------------------ #
@@ -351,7 +339,7 @@ class SSETransport(MCPBaseTransport):
         """
         # NEW: Ensure initialization before tool calls
         if not self._initialized.is_set():
-            return {"isError": True, "error": "MCP session not initialized"}
+            return {"isError": True, "error": "SSE transport not implemented"}
             
         if not self._message_url:
             return {"isError": True, "error": "No message endpoint available"}

chuk_tool_processor/mcp/transport/stdio_transport.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 from contextlib import AsyncExitStack
 import json
 from typing import Dict, Any, List, Optional
+import asyncio
 
 # ------------------------------------------------------------------ #
 #  Local import                                                      #
@@ -71,14 +72,11 @@ class StdioTransport(MCPBaseTransport):
             return False
 
     async def close(self) -> None:
-        if self._context_stack:
-            try:
-                await self._context_stack.__aexit__(None, None, None)
-            except Exception:
-                pass
+        """Minimal close method with zero async operations."""
+        # Just clear references - no async operations at all
+        self._context_stack = None
         self.read_stream = None
         self.write_stream = None
-        self._context_stack = None
 
     # --------------------------------------------------------------------- #
     #  Utility                                                              #
@@ -131,7 +129,7 @@ class StdioTransport(MCPBaseTransport):
     def get_streams(self):
         """
         Expose the low-level streams so legacy callers can access them
-        directly.  The base-class’ default returns an empty list; here we
+        directly.  The base-class' default returns an empty list; here we
         return a single-element list when the transport is active.
         """
         if self.read_stream and self.write_stream:
@@ -145,7 +143,7 @@ class StdioTransport(MCPBaseTransport):
         self, tool_name: str, arguments: Dict[str, Any]
     ) -> Dict[str, Any]:
         """
-        Execute *tool_name* with *arguments* and normalise the server’s reply.
+        Execute *tool_name* with *arguments* and normalise the server's reply.
 
         The echo-server often returns:
         {
@@ -194,4 +192,4 @@ class StdioTransport(MCPBaseTransport):
             import logging
 
             logging.error(f"Error calling tool {tool_name}: {e}")
-            return {"isError": True, "error": str(e)}
+            return {"isError": True, "error": str(e)}

chuk_tool_processor/models/validated_tool.py

@@ -35,7 +35,7 @@ T_Validated = TypeVar("T_Validated", bound="ValidatedTool")
 
 
 # --------------------------------------------------------------------------- #
-# Helper mix-in – serialise a *class* into assorted formats
+# Helper mix-in - serialise a *class* into assorted formats
 # --------------------------------------------------------------------------- #
 class _ExportMixin:
     """Static helpers that expose a tool class in other specs."""
@@ -79,7 +79,7 @@ class _ExportMixin:
         return cls.Arguments.model_json_schema()  # type: ignore[attr-defined]
 
     # ------------------------------------------------------------------ #
-    # Tiny XML tag – handy for unit-tests / demos
+    # Tiny XML tag - handy for unit-tests / demos
     # ------------------------------------------------------------------ #
     @classmethod
     def to_xml_tag(cls: type[T_Validated], **arguments: Any) -> str:
@@ -96,9 +96,9 @@ class ValidatedTool(_ExportMixin, BaseModel):
     """Pydantic-validated base for new async-native tools."""
 
     # ------------------------------------------------------------------ #
-    # Inner models – override in subclasses
+    # Inner models - override in subclasses
     # ------------------------------------------------------------------ #
-    class Arguments(BaseModel):  # noqa: D401 – acts as a namespace
+    class Arguments(BaseModel):  # noqa: D401 - acts as a namespace
         """Input model"""
 
     class Result(BaseModel):  # noqa: D401
@@ -124,14 +124,14 @@ class ValidatedTool(_ExportMixin, BaseModel):
     # ------------------------------------------------------------------ #
     # Sub-classes must implement this
     # ------------------------------------------------------------------ #
-    async def _execute(self, **_kwargs: Any):  # noqa: D401 – expected override
+    async def _execute(self, **_kwargs: Any):  # noqa: D401 - expected override
         raise NotImplementedError("Tool must implement async _execute()")
 
 
 # --------------------------------------------------------------------------- #
 # Decorator to retrofit validation onto classic "imperative" tools
 # --------------------------------------------------------------------------- #
-def with_validation(cls):  # noqa: D401 – factory
+def with_validation(cls):  # noqa: D401 - factory
     """
     Decorator that wraps an existing async ``execute`` method with:
 

chuk_tool_processor/plugins/discovery.py

@@ -58,13 +58,13 @@ class PluginDiscovery:
     """
     Recursively scans *package_paths* for plugin classes and registers them.
 
-    * Parser plugins – concrete subclasses of :class:`ParserPlugin`
+    * Parser plugins - concrete subclasses of :class:`ParserPlugin`
       with an **async** ``try_parse`` coroutine.
 
-    * Execution strategies – concrete subclasses of
+    * Execution strategies - concrete subclasses of
       :class:`ExecutionStrategy`.
 
-    * Explicitly-decorated plugins – classes tagged with ``@plugin(...)``.
+    * Explicitly-decorated plugins - classes tagged with ``@plugin(...)``.
     """
 
     # ------------------------------------------------------------------ #

chuk_tool_processor/plugins/parsers/base.py

@@ -16,7 +16,7 @@ class ParserPlugin(ABC):
     Every parser plugin **must** implement the async ``try_parse`` coroutine.
 
     The processor awaits it and expects *a list* of :class:`ToolCall`
-    objects. If the plugin doesn’t recognise the input it should return an
+    objects. If the plugin doesn't recognise the input it should return an
     empty list.
     """