From 1b395ecc916ff93dddf01f3204acb61120f0a0a4 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Thu, 29 Jan 2026 16:30:34 +0000
Subject: [PATCH] Move SQLToolService logs to debug level

Changed several INFO level log messages in SQLToolService to DEBUG level as intended by previous TODOs. This reduces log noise during normal operation. Removed the associated TODO comments.

References: #123

Co-authored-by: emooreatx <3317461+emooreatx@users.noreply.github.com>
---
 ciris_adapters/external_data_sql/service.py | 21 +++++++--------------
 1 file changed, 7 insertions(+), 14 deletions(-)

diff --git a/ciris_adapters/external_data_sql/service.py b/ciris_adapters/external_data_sql/service.py
index d3278257e6..2178937bdd 100644
--- a/ciris_adapters/external_data_sql/service.py
+++ b/ciris_adapters/external_data_sql/service.py
@@ -120,8 +120,7 @@ async def initialize(self) -> None:
         # Note: BaseService doesn't have initialize(), but this is required by framework
         # await super().initialize()  # Skip if not implemented
 
-        # TODO: Move to debug level after tool registration is confirmed working
-        self._logger.info(
+        self._logger.debug(
             f"SQLToolService initializing with config: {self._config.connector_id if self._config else 'None'}"
         )
         self._logger.info(f"SQLToolService has {len(self._tool_schemas)} tool schemas ready")
@@ -155,8 +154,7 @@ async def initialize(self) -> None:
             # Connect to database
             await self._connect()
 
-            # TODO: Move to debug level after tool registration is confirmed working
-            self._logger.info(f"SQLToolService initialized successfully - tools: {await self.list_tools()}")
+            self._logger.debug(f"SQLToolService initialized successfully - tools: {await self.list_tools()}")
 
     async def _connect(self) -> None:
         """Establish database connection."""
@@ -218,8 +216,7 @@ async def list_tools(self) -> List[str]:
         Returns:
             List of generic SQL tool names
         """
-        # TODO: Move to debug level after tool registration is confirmed working
-        self._logger.info(f"list_tools() called on SQLToolService")
+        self._logger.debug(f"list_tools() called on SQLToolService")
         tools = [
             "initialize_sql_connector",
             "get_sql_service_metadata",
@@ -231,8 +228,7 @@ async def list_tools(self) -> List[str]:
             "sql_get_stats",
             "sql_query",
         ]
-        # TODO: Move to debug level after tool registration is confirmed working
-        self._logger.info(f"list_tools() returning {len(tools)} tools: {tools}")
+        self._logger.debug(f"list_tools() returning {len(tools)} tools: {tools}")
         return tools
 
     async def execute_tool(self, tool_name: str, parameters: JSONDict) -> ToolExecutionResult:
@@ -245,8 +241,7 @@ async def execute_tool(self, tool_name: str, parameters: JSONDict) -> ToolExecut
         Returns:
             Tool execution result
         """
-        # TODO: Move to debug level after tool registration is confirmed working
-        self._logger.info(f"execute_tool() called with tool_name={tool_name}, parameters={list(parameters.keys())}")
+        self._logger.debug(f"execute_tool() called with tool_name={tool_name}, parameters={list(parameters.keys())}")
 
         # Extract or generate correlation ID for result tracking
         import uuid
@@ -351,8 +346,7 @@ async def _initialize_connector(self, parameters: JSONDict) -> ToolExecutionResu
         Returns:
             ToolExecutionResult with connector configuration details
         """
-        # TODO: Move to debug level after tool registration is confirmed working
-        self._logger.info(f"_initialize_connector called with parameters: {list(parameters.keys())}")
+        self._logger.debug(f"_initialize_connector called with parameters: {list(parameters.keys())}")
 
         # Extract or generate correlation ID
         import uuid
@@ -494,8 +488,7 @@ async def _get_sql_metadata(self, parameters: JSONDict) -> ToolExecutionResult:
         Returns:
             ToolExecutionResult with connector metadata
         """
-        # TODO: Move to debug level after tool registration is confirmed working
-        self._logger.info(f"_get_sql_metadata called with parameters: {list(parameters.keys())}")
+        self._logger.debug(f"_get_sql_metadata called with parameters: {list(parameters.keys())}")
 
         # Extract or generate correlation ID
         import uuid