Address feedback comments

Author: psx95

samples/adk-sql-agent/main.py

@@ -51,9 +51,6 @@
 # Set web=True if you intend to serve a web interface, False otherwise
 SERVE_WEB_INTERFACE = True
 
-logging.getLogger("opentelemetry").setLevel(logging.DEBUG)
-
-
 # [START opentelemetry_adk_otel_setup]
 def setup_opentelemetry() -> None:
     credentials, project_id = google.auth.default()
@@ -121,7 +118,7 @@ def main() -> None:
     # For this example this would be the current directory containing main.py.
     # Note: Calling this method attempts to set the global tracer provider, which has already been
     # set by the setup_opentelemetry() function.
-    app: FastAPI = get_fast_api_app(
+    app = get_fast_api_app(
         agents_dir=AGENT_DIR,
         session_service_uri=SESSION_DB_URL,
         allow_origins=ALLOWED_ORIGINS,

samples/adk-sql-agent/sql_agent/agent.py

@@ -40,8 +40,8 @@
 You should always check if database for current session exist before running sql queries by calling create_database_tool.
 If you make a mistake, try to recover."""
 
-INTRO_TEXT = """\
-Starting agent using ephemeral SQLite DB {dbpath}. This demo allows you to chat with an Agent
+DESCRIPTION = """\
+Starting agent using ephemeral SQLite DB. This demo allows you to chat with an Agent
 that has full access to an ephemeral SQLite database. The database is initially empty. It is
 built with the the LangGraph prebuilt **ReAct Agent** and the **SQLDatabaseToolkit**. Here are some samples you can try:
 
@@ -64,7 +64,7 @@
 root_agent = Agent(
     name="weather_time_agent",
     model="gemini-2.0-flash",
-    description=INTRO_TEXT,
+    description=DESCRIPTION,
     instruction=SYSTEM_PROMPT,
     tools=[run_sql_tool, create_database_tool],
 )

samples/adk-sql-agent/sql_agent/tools.py

@@ -34,15 +34,15 @@ class SqlRunResult(TypedDict):
     """The rows returned by the SQL query"""
 
 @tracer.start_as_current_span("create_database")
-def create_database_tool(tool_context: ToolContext):
+def create_database_tool(tool_context: ToolContext) -> None:
     """Creates a temporary file in the /tmp directory to hold an ephemeral
     sqlite3 database if a database is not found for the current session.
     """
     if not SESSION_DB_KEY in tool_context.state:
         _, path = tempfile.mkstemp(suffix=".db")
         # No scope prefix in the state data indicates that it will be persisted for
         # current session.
-        # See https://deepwiki.com/google/adk-python/3.4-state-management.
+        # See https://google.github.io/adk-docs/sessions/state/.
         tool_context.state[SESSION_DB_KEY] = path
 
 @tracer.start_as_current_span("run_sql")