chuk-tool-processor 0.6.3__tar.gz → 0.6.5__tar.gz

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: chuk-tool-processor
-Version: 0.6.3
+Version: 0.6.5
 Summary: Async-native framework for registering, discovering, and executing tools referenced in LLM responses
 Author-email: CHUK Team <chrishayuk@somejunkmailbox.com>
 Maintainer-email: CHUK Team <chrishayuk@somejunkmailbox.com>
@@ -20,7 +20,7 @@ Classifier: Framework :: AsyncIO
 Classifier: Typing :: Typed
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
-Requires-Dist: chuk-mcp>=0.5
+Requires-Dist: chuk-mcp>=0.5.2
 Requires-Dist: dotenv>=0.9.9
 Requires-Dist: pydantic>=2.11.3
 Requires-Dist: uuid>=1.30

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "chuk-tool-processor"
-version = "0.6.3"
+version = "0.6.5"
 description = "Async-native framework for registering, discovering, and executing tools referenced in LLM responses"
 readme = "README.md"
 requires-python = ">=3.11"
@@ -41,7 +41,7 @@ classifiers = [
     "Typing :: Typed",
 ]
 dependencies = [
-    "chuk-mcp>=0.5",
+    "chuk-mcp>=0.5.2",
     "dotenv>=0.9.9",
     "pydantic>=2.11.3",
     "uuid>=1.30",

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/src/chuk_tool_processor/logging/helpers.py

@@ -58,15 +58,15 @@ async def log_context_span(
     prev = log_context.get_copy()
     log_context.update(span_ctx)
 
-    logger.info("Starting %s", operation)
+    logger.debug("Starting %s", operation)
     try:
         yield
         if log_duration:
-            logger.info(
+            logger.debug(
                 "Completed %s", operation, extra={"context": {"duration": time.time() - start}}
             )
         else:
-            logger.info("Completed %s", operation)
+            logger.debug("Completed %s", operation)
     except Exception as exc:
         logger.exception(
             "Error in %s: %s", operation, exc, extra={"context": {"duration": time.time() - start}}
@@ -97,10 +97,10 @@ async def request_logging(
     logger = get_logger("chuk_tool_processor.request")
     request_id = log_context.start_request(request_id)
     start = time.time()
-    logger.info("Starting request %s", request_id)
+    logger.debug("Starting request %s", request_id)
     try:
         yield request_id
-        logger.info(
+        logger.debug(
             "Completed request %s",
             request_id,
             extra={"context": {"duration": time.time() - start}},
@@ -184,4 +184,4 @@ async def log_tool_call(tool_call: Any, tool_result: Any) -> None:
     if tool_result.error:
         logger.error("Tool %s failed: %s", tool_call.tool, tool_result.error, extra={"context": ctx})
     else:
-        logger.info("Tool %s succeeded in %.3fs", tool_call.tool, dur, extra={"context": ctx})
+        logger.debug("Tool %s succeeded in %.3fs", tool_call.tool, dur, extra={"context": ctx})

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/src/chuk_tool_processor/mcp/register_mcp_tools.py

@@ -95,7 +95,7 @@ async def register_mcp_tools(
         except Exception as exc:
             logger.error("Failed to register MCP tool '%s': %s", tool_name, exc)
 
-    logger.info("MCP registration complete - %d tool(s) available", len(registered))
+    logger.debug("MCP registration complete - %d tool(s) available", len(registered))
     return registered
 
 
@@ -134,14 +134,14 @@ async def update_mcp_tools_stream_manager(
                 if tool and hasattr(tool, 'set_stream_manager'):
                     tool.set_stream_manager(new_stream_manager)
                     updated_count += 1
-                    logger.debug(f"Updated StreamManager for tool '{namespace}:{tool_name}'")
+                    logger.debug("Updated StreamManager for tool '%s:%s'", namespace, tool_name)
             except Exception as e:
-                logger.warning(f"Failed to update StreamManager for tool '{namespace}:{tool_name}': {e}")
+                logger.warning("Failed to update StreamManager for tool '%s:%s': %s", namespace, tool_name, e)
         
         action = "connected" if new_stream_manager else "disconnected"
-        logger.info(f"StreamManager {action} for {updated_count} tools in namespace '{namespace}'")
+        logger.debug("StreamManager %s for %d tools in namespace '%s'", action, updated_count, namespace)
         
     except Exception as e:
-        logger.error(f"Failed to update tools in namespace '{namespace}': {e}")
+        logger.error("Failed to update tools in namespace '%s': %s", namespace, e)
     
     return updated_count

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/src/chuk_tool_processor/mcp/setup_mcp_http_streamable.py

@@ -110,8 +110,8 @@ async def setup_mcp_http_streamable(
         max_retries=max_retries,
     )
 
-    logger.info(
-        "MCP (HTTP Streamable) initialised - %s tool%s registered into namespace '%s'",
+    logger.debug(
+        "MCP (HTTP Streamable) initialised - %d tool%s registered into namespace '%s'",
         len(registered),
         "" if len(registered) == 1 else "s",
         namespace,

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/src/chuk_tool_processor/mcp/setup_mcp_sse.py

@@ -89,8 +89,8 @@ async def setup_mcp_sse(  # noqa: C901 - long but just a config facade
         max_retries=max_retries,
     )
 
-    logger.info(
-        "MCP (SSE) initialised - %s tool%s registered into namespace '%s'",
+    logger.debug(
+        "MCP (SSE) initialised - %d tool%s registered into namespace '%s'",
         len(registered),
         "" if len(registered) == 1 else "s",
         namespace,

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/src/chuk_tool_processor/mcp/setup_mcp_stdio.py

@@ -73,8 +73,8 @@ async def setup_mcp_stdio(  # noqa: C901 - long but just a config facade
         max_retries=max_retries,
     )
 
-    logger.info(
-        "MCP (stdio) initialised - %s tool%s registered into namespace '%s'",
+    logger.debug(
+        "MCP (stdio) initialised - %d tool%s registered into namespace '%s'",
         len(registered),
         "" if len(registered) == 1 else "s",
         namespace,

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/src/chuk_tool_processor/mcp/stream_manager.py

@@ -73,7 +73,7 @@ class StreamManager:
             )
             return inst
         except asyncio.TimeoutError:
-            logger.error(f"StreamManager initialization timed out after {initialization_timeout}s")
+            logger.error("StreamManager initialization timed out after %ss", initialization_timeout)
             raise RuntimeError(f"StreamManager initialization timed out after {initialization_timeout}s")
 
     @classmethod
@@ -99,7 +99,7 @@ class StreamManager:
             )
             return inst
         except asyncio.TimeoutError:
-            logger.error(f"SSE StreamManager initialization timed out after {initialization_timeout}s")
+            logger.error("SSE StreamManager initialization timed out after %ss", initialization_timeout)
             raise RuntimeError(f"SSE StreamManager initialization timed out after {initialization_timeout}s")
 
     @classmethod
@@ -125,7 +125,7 @@ class StreamManager:
             )
             return inst
         except asyncio.TimeoutError:
-            logger.error(f"HTTP Streamable StreamManager initialization timed out after {initialization_timeout}s")
+            logger.error("HTTP Streamable StreamManager initialization timed out after %ss", initialization_timeout)
             raise RuntimeError(f"HTTP Streamable StreamManager initialization timed out after {initialization_timeout}s")
 
     # ------------------------------------------------------------------ #
@@ -196,7 +196,7 @@ class StreamManager:
                         else:
                             sse_url = "http://localhost:8000"
                             api_key = None
-                            logger.warning(f"No URL configured for SSE transport, using default: {sse_url}")
+                            logger.warning("No URL configured for SSE transport, using default: %s", sse_url)
                         
                         transport = SSETransport(
                             sse_url,
@@ -215,7 +215,7 @@ class StreamManager:
                             http_url = "http://localhost:8000"
                             api_key = None
                             session_id = None
-                            logger.warning(f"No URL configured for HTTP Streamable transport, using default: {http_url}")
+                            logger.warning("No URL configured for HTTP Streamable transport, using default: %s", http_url)
                         
                         transport = HTTPStreamableTransport(
                             http_url,
@@ -252,13 +252,13 @@ class StreamManager:
                             "status": status,
                         }
                     )
-                    logger.info("Initialised %s - %d tool(s)", server_name, len(tools))
+                    logger.debug("Initialised %s - %d tool(s)", server_name, len(tools))
                 except asyncio.TimeoutError:
                     logger.error("Timeout initialising %s", server_name)
                 except Exception as exc:
                     logger.error("Error initialising %s: %s", server_name, exc)
 
-            logger.info(
+            logger.debug(
                 "StreamManager ready - %d server(s), %d tool(s)",
                 len(self.transports),
                 len(self.all_tools),
@@ -307,13 +307,13 @@ 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.debug("Initialised SSE %s - %d tool(s)", name, len(tools))
                 except asyncio.TimeoutError:
                     logger.error("Timeout initialising SSE %s", name)
                 except Exception as exc:
                     logger.error("Error initialising SSE %s: %s", name, exc)
 
-            logger.info(
+            logger.debug(
                 "StreamManager ready - %d SSE server(s), %d tool(s)",
                 len(self.transports),
                 len(self.all_tools),
@@ -364,13 +364,13 @@ class StreamManager:
                     self.server_info.append(
                         {"id": idx, "name": name, "tools": len(tools), "status": status}
                     )
-                    logger.info("Initialised HTTP Streamable %s - %d tool(s)", name, len(tools))
+                    logger.debug("Initialised HTTP Streamable %s - %d tool(s)", name, len(tools))
                 except asyncio.TimeoutError:
                     logger.error("Timeout initialising HTTP Streamable %s", name)
                 except Exception as exc:
                     logger.error("Error initialising HTTP Streamable %s: %s", name, exc)
 
-            logger.info(
+            logger.debug(
                 "StreamManager ready - %d HTTP Streamable server(s), %d tool(s)",
                 len(self.transports),
                 len(self.all_tools),
@@ -395,20 +395,20 @@ class StreamManager:
             return []
             
         if server_name not in self.transports:
-            logger.error(f"Server '{server_name}' not found in transports")
+            logger.error("Server '%s' not found in transports", server_name)
             return []
         
         transport = self.transports[server_name]
         
         try:
             tools = await asyncio.wait_for(transport.get_tools(), timeout=10.0)
-            logger.debug(f"Found {len(tools)} tools for server {server_name}")
+            logger.debug("Found %d tools for server %s", len(tools), server_name)
             return tools
         except asyncio.TimeoutError:
-            logger.error(f"Timeout listing tools for server {server_name}")
+            logger.error("Timeout listing tools for server %s", server_name)
             return []
         except Exception as e:
-            logger.error(f"Error listing tools for server {server_name}: {e}")
+            logger.error("Error listing tools for server %s: %s", server_name, e)
             return []
 
     # ------------------------------------------------------------------ #
@@ -541,7 +541,7 @@ class StreamManager:
             self._closed = True
             return
         
-        logger.debug(f"Closing {len(self.transports)} transports...")
+        logger.debug("Closing %d transports...", len(self.transports))
         
         try:
             # Use shield to protect the cleanup operation from cancellation
@@ -551,7 +551,7 @@ class StreamManager:
             logger.debug("Close operation cancelled, performing synchronous cleanup")
             self._sync_cleanup()
         except Exception as e:
-            logger.debug(f"Error during close: {e}")
+            logger.debug("Error during close: %s", e)
             self._sync_cleanup()
         finally:
             self._closed = True
@@ -565,7 +565,7 @@ class StreamManager:
         try:
             await self._concurrent_close(transport_items, close_results)
         except Exception as e:
-            logger.debug(f"Concurrent close failed: {e}, falling back to sequential close")
+            logger.debug("Concurrent close failed: %s, falling back to sequential close", e)
             # Strategy 2: Fall back to sequential close
             await self._sequential_close(transport_items, close_results)
         
@@ -575,7 +575,7 @@ class StreamManager:
         # Log summary
         if close_results:
             successful_closes = sum(1 for _, success, _ in close_results if success)
-            logger.debug(f"Transport cleanup: {successful_closes}/{len(close_results)} closed successfully")
+            logger.debug("Transport cleanup: %d/%d closed successfully", successful_closes, len(close_results))
 
     async def _concurrent_close(self, transport_items: List[Tuple[str, MCPBaseTransport]], close_results: List) -> None:
         """Try to close all transports concurrently."""
@@ -602,10 +602,10 @@ class StreamManager:
                 for i, (name, _) in enumerate(close_tasks):
                     result = results[i] if i < len(results) else None
                     if isinstance(result, Exception):
-                        logger.debug(f"Transport {name} close failed: {result}")
+                        logger.debug("Transport %s close failed: %s", name, result)
                         close_results.append((name, False, str(result)))
                     else:
-                        logger.debug(f"Transport {name} closed successfully")
+                        logger.debug("Transport %s closed successfully", name)
                         close_results.append((name, True, None))
                         
             except asyncio.TimeoutError:
@@ -632,16 +632,16 @@ class StreamManager:
                     self._close_single_transport(name, transport),
                     timeout=0.5  # Short timeout per transport
                 )
-                logger.debug(f"Closed transport: {name}")
+                logger.debug("Closed transport: %s", name)
                 close_results.append((name, True, None))
             except asyncio.TimeoutError:
-                logger.debug(f"Transport {name} close timed out (normal during shutdown)")
+                logger.debug("Transport %s close timed out (normal during shutdown)", name)
                 close_results.append((name, False, "timeout"))
             except asyncio.CancelledError:
-                logger.debug(f"Transport {name} close cancelled during event loop shutdown")
+                logger.debug("Transport %s close cancelled during event loop shutdown", name)
                 close_results.append((name, False, "cancelled"))
             except Exception as e:
-                logger.debug(f"Error closing transport {name}: {e}")
+                logger.debug("Error closing transport %s: %s", name, e)
                 close_results.append((name, False, str(e)))
 
     async def _close_single_transport(self, name: str, transport: MCPBaseTransport) -> None:
@@ -650,9 +650,9 @@ class StreamManager:
             if hasattr(transport, 'close') and callable(transport.close):
                 await transport.close()
             else:
-                logger.debug(f"Transport {name} has no close method")
+                logger.debug("Transport %s has no close method", name)
         except Exception as e:
-            logger.debug(f"Error closing transport {name}: {e}")
+            logger.debug("Error closing transport %s: %s", name, e)
             raise
 
     def _sync_cleanup(self) -> None:
@@ -660,9 +660,9 @@ class StreamManager:
         try:
             transport_count = len(self.transports)
             self._cleanup_state()
-            logger.debug(f"Synchronous cleanup completed for {transport_count} transports")
+            logger.debug("Synchronous cleanup completed for %d transports", transport_count)
         except Exception as e:
-            logger.debug(f"Error during synchronous cleanup: {e}")
+            logger.debug("Error during synchronous cleanup: %s", e)
 
     def _cleanup_state(self) -> None:
         """Clean up internal state synchronously."""
@@ -673,7 +673,7 @@ class StreamManager:
             self.all_tools.clear()
             self.server_names.clear()
         except Exception as e:
-            logger.debug(f"Error during state cleanup: {e}")
+            logger.debug("Error during state cleanup: %s", e)
 
     # ------------------------------------------------------------------ #
     #  backwards-compat: streams helper                                  #

{chuk_tool_processor-0.6.3 → chuk_tool_processor-0.6.5}/src/chuk_tool_processor/mcp/transport/http_streamable_transport.py

@@ -107,7 +107,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
         start_time = time.time()
         
         try:
-            logger.info(f"Initializing HTTP Streamable transport to {self.url}")
+            logger.debug("Initializing HTTP Streamable transport to %s", self.url)
             
             # Create HTTP parameters for chuk-mcp (following SSE pattern)
             headers = {}
@@ -117,7 +117,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
             
             if self.session_id:
                 headers["X-Session-ID"] = self.session_id
-                logger.debug(f"Using session ID: {self.session_id}")
+                logger.debug("Using session ID: %s", self.session_id)
             
             http_params = StreamableHTTPParameters(
                 url=self.url,
@@ -154,7 +154,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
                 self._metrics["initialization_time"] = init_time
                 self._metrics["last_ping_time"] = ping_time
                 
-                logger.info(f"HTTP Streamable transport initialized successfully in {init_time:.3f}s (ping: {ping_time:.3f}s)")
+                logger.debug("HTTP Streamable transport initialized successfully in %.3fs (ping: %.3fs)", init_time, ping_time)
                 return True
             else:
                 logger.warning("HTTP connection established but ping failed")
@@ -164,12 +164,12 @@ class HTTPStreamableTransport(MCPBaseTransport):
                 return True
 
         except asyncio.TimeoutError:
-            logger.error(f"HTTP Streamable initialization timed out after {self.connection_timeout}s")
+            logger.error("HTTP Streamable initialization timed out after %ss", self.connection_timeout)
             logger.error("This may indicate the server is not responding to MCP initialization")
             await self._cleanup()
             return False
         except Exception as e:
-            logger.error(f"Error initializing HTTP Streamable transport: {e}", exc_info=True)
+            logger.error("Error initializing HTTP Streamable transport: %s", e, exc_info=True)
             await self._cleanup()
             return False
 
@@ -180,10 +180,11 @@ class HTTPStreamableTransport(MCPBaseTransport):
         
         # Log final metrics (enhanced from SSE)
         if self.enable_metrics and self._metrics["total_calls"] > 0:
-            logger.info(
-                f"HTTP Streamable transport closing - Total calls: {self._metrics['total_calls']}, "
-                f"Success rate: {(self._metrics['successful_calls']/self._metrics['total_calls']*100):.1f}%, "
-                f"Avg response time: {self._metrics['avg_response_time']:.3f}s"
+            logger.debug(
+                "HTTP Streamable transport closing - Total calls: %d, Success rate: %.1f%%, Avg response time: %.3fs",
+                self._metrics["total_calls"],
+                (self._metrics["successful_calls"] / self._metrics["total_calls"] * 100),
+                self._metrics["avg_response_time"]
             )
             
         try:
@@ -192,7 +193,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
                 logger.debug("HTTP Streamable context closed")
                 
         except Exception as e:
-            logger.debug(f"Error during transport close: {e}")
+            logger.debug("Error during transport close: %s", e)
         finally:
             await self._cleanup()
 
@@ -219,14 +220,14 @@ class HTTPStreamableTransport(MCPBaseTransport):
             if self.enable_metrics:
                 ping_time = time.time() - start_time
                 self._metrics["last_ping_time"] = ping_time
-                logger.debug(f"Ping completed in {ping_time:.3f}s: {result}")
+                logger.debug("Ping completed in %.3fs: %s", ping_time, result)
             
             return bool(result)
         except asyncio.TimeoutError:
             logger.error("Ping timed out")
             return False
         except Exception as e:
-            logger.error(f"Ping failed: {e}")
+            logger.error("Ping failed: %s", e)
             return False
 
     async def get_tools(self) -> List[Dict[str, Any]]:
@@ -248,12 +249,12 @@ class HTTPStreamableTransport(MCPBaseTransport):
             elif isinstance(tools_response, list):
                 tools = tools_response
             else:
-                logger.warning(f"Unexpected tools response type: {type(tools_response)}")
+                logger.warning("Unexpected tools response type: %s", type(tools_response))
                 tools = []
             
             if self.enable_metrics:
                 response_time = time.time() - start_time
-                logger.debug(f"Retrieved {len(tools)} tools in {response_time:.3f}s")
+                logger.debug("Retrieved %d tools in %.3fs", len(tools), response_time)
             
             return tools
             
@@ -261,7 +262,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
             logger.error("Get tools timed out")
             return []
         except Exception as e:
-            logger.error(f"Error getting tools: {e}")
+            logger.error("Error getting tools: %s", e)
             return []
 
     async def call_tool(self, tool_name: str, arguments: Dict[str, Any], 
@@ -280,7 +281,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
             self._metrics["total_calls"] += 1
 
         try:
-            logger.debug(f"Calling tool '{tool_name}' with timeout {tool_timeout}s")
+            logger.debug("Calling tool '%s' with timeout %ss", tool_name, tool_timeout)
             
             raw_response = await asyncio.wait_for(
                 send_tools_call(
@@ -299,9 +300,9 @@ class HTTPStreamableTransport(MCPBaseTransport):
                 self._update_metrics(response_time, not result.get("isError", False))
                 
             if not result.get("isError", False):
-                logger.debug(f"Tool '{tool_name}' completed successfully in {response_time:.3f}s")
+                logger.debug("Tool '%s' completed successfully in %.3fs", tool_name, response_time)
             else:
-                logger.warning(f"Tool '{tool_name}' failed in {response_time:.3f}s: {result.get('error', 'Unknown error')}")
+                logger.warning("Tool '%s' failed in %.3fs: %s", tool_name, response_time, result.get('error', 'Unknown error'))
             
             return result
 
@@ -311,7 +312,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
                 self._update_metrics(response_time, False)
                 
             error_msg = f"Tool execution timed out after {tool_timeout}s"
-            logger.error(f"Tool '{tool_name}' {error_msg}")
+            logger.error("Tool '%s' %s", tool_name, error_msg)
             return {
                 "isError": True,
                 "error": error_msg
@@ -322,7 +323,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
                 self._update_metrics(response_time, False)
                 
             error_msg = f"Tool execution failed: {str(e)}"
-            logger.error(f"Tool '{tool_name}' error: {error_msg}")
+            logger.error("Tool '%s' error: %s", tool_name, error_msg)
             return {
                 "isError": True,
                 "error": error_msg
@@ -359,7 +360,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
             logger.error("List resources timed out")
             return {}
         except Exception as e:
-            logger.debug(f"Error listing resources: {e}")
+            logger.debug("Error listing resources: %s", e)
             return {}
 
     async def list_prompts(self) -> Dict[str, Any]:
@@ -381,7 +382,7 @@ class HTTPStreamableTransport(MCPBaseTransport):
             logger.error("List prompts timed out")
             return {}
         except Exception as e:
-            logger.debug(f"Error listing prompts: {e}")
+            logger.debug("Error listing prompts: %s", e)
             return {}
 
     def _normalize_tool_response(self, raw_response: Dict[str, Any]) -> Dict[str, Any]: