chore: improve logging

Author: lpm0073

Dockerfile

@@ -9,9 +9,9 @@ LABEL maintainer="Lawrence McDaniel <lpm0073@gmail.com>" \
   org.opencontainers.image.title="StackademyAssistent" \
   org.opencontainers.image.version="0.1.0" \
   org.opencontainers.image.authors="Lawrence McDaniel <lpm0073@gmail.com>" \
-  org.opencontainers.image.url="https://FullStackWithLawrence.github.io/smarter/" \
+  org.opencontainers.image.url="https://FullStackWithLawrence.github.io/agentic-ai-workflow/" \
   org.opencontainers.image.source="https://github.com/FullStackWithLawrence/agentic-ai-workflow" \
-  org.opencontainers.image.documentation="https://FullStackWithLawrence.github.io/smarter/"
+  org.opencontainers.image.documentation="https://FullStackWithLawrence.github.io/agentic-ai-workflow/"
 
 
 FROM base AS requirements

app/database.py

@@ -41,6 +41,11 @@ def __init__(self):
                 "MYSQL_HOST, MYSQL_USER, MYSQL_PASSWORD, MYSQL_DATABASE"
             )
 
+    @property
+    def connection_string(self) -> str:
+        """Return the database connection string."""
+        return f"{self.user}@{self.host}:{self.port}/{self.database}"
+
     def get_connection(self) -> pymysql.Connection:
         """
         Create and return a new MySQL connection.

app/logging_config.py

@@ -27,6 +27,8 @@ def setup_logging(level: int = LOGGING_LEVEL) -> logging.Logger:
             handlers=[logging.StreamHandler(sys.stdout)],  # This logs to console
         )
 
+    logging.getLogger("httpx").setLevel(logging.WARNING)
+
     return logging.getLogger(__name__)
 
 

app/stackademy.py

@@ -67,7 +67,6 @@ def _log_success(self, message: str) -> None:
     def tool_factory_get_courses(self) -> ChatCompletionFunctionToolParam:
         """LLM Factory function to create a tool for getting courses"""
         schema = StackademyGetCoursesParams.model_json_schema()
-        schema["required"] = []  # Both parameters are optional
         return ChatCompletionFunctionToolParam(
             type="function",
             function={
@@ -80,7 +79,6 @@ def tool_factory_get_courses(self) -> ChatCompletionFunctionToolParam:
     def tool_factory_register(self) -> ChatCompletionFunctionToolParam:
         """LLMFactory function to create a tool for registering a user"""
         schema = StackademyRegisterCourseParams.model_json_schema()
-        schema["required"] = ["course_code", "email", "full_name"]  # All parameters are required
         return ChatCompletionFunctionToolParam(
             type="function",
             function={
@@ -115,7 +113,7 @@ def get_courses(self, description: Optional[str] = None, max_cost: Optional[floa
         Returns:
             List[Dict[str, Any]]: List of courses matching the criteria
         """
-        # Base query
+
         query = """
         SELECT
             c.course_code,
@@ -128,7 +126,6 @@ def get_courses(self, description: Optional[str] = None, max_cost: Optional[floa
         LEFT JOIN courses prerequisite ON c.prerequisite_id = prerequisite.course_id
         """
 
-        # Build WHERE clause dynamically
         where_conditions = []
         params = []
 
@@ -140,14 +137,15 @@ def get_courses(self, description: Optional[str] = None, max_cost: Optional[floa
             where_conditions.append("c.cost <= %s")
             params.append(max_cost)
 
-        # Add WHERE clause if we have conditions
         if where_conditions:
             query += " WHERE " + " AND ".join(where_conditions)
 
         query += " ORDER BY c.prerequisite_id"
-        logger.info("get_courses() executing db query with params: %s", params)
+
         try:
-            return self.db.execute_query(query, tuple(params))
+            retval = self.db.execute_query(query, tuple(params))
+            logger.info("get_courses() retrieved %d rows from %s", len(retval), self.db.connection_string)
+            return retval
         # pylint: disable=broad-except
         except Exception as e:
             logger.error("Failed to retrieve courses: %s", e)
@@ -190,9 +188,9 @@ def register_course(self, course_code: str, email: str, full_name: str) -> bool:
         if MISSING in (course_code, email, full_name):
             raise ConfigurationException("Missing required registration parameters.")
 
-        full_name = full_name.title().strip()
-        email = email.lower().strip()
-        course_code = course_code.upper().strip()
+        full_name = full_name.title().strip() if isinstance(full_name, str) else full_name
+        email = email.lower().strip() if isinstance(email, str) else email
+        course_code = course_code.upper().strip() if isinstance(course_code, str) else course_code
 
         logger.info("Registering %s (%s) for course %s...", full_name, email, course_code)
         if not self.verify_course(course_code):