fix(logging): remove redundant string concatenations in warning messages and error formatting for clarity

Author: nikmd1306

sgr_agent_core/agent_config.py

@@ -70,7 +70,7 @@ def _definitions_from_dict(cls, data: dict) -> Self:
         # Check for agents that will be overridden
         overridden = set(cls._instance.agents.keys()) & set(custom_agents.keys())
         if overridden:
-            logger.warning(f"Loaded agents will override existing agents: " f"{', '.join(sorted(overridden))}")
+            logger.warning(f"Loaded agents will override existing agents: {', '.join(sorted(overridden))}")
 
         cls._instance.agents.update(custom_agents)
 
@@ -90,7 +90,7 @@ def _definitions_from_dict(cls, data: dict) -> Self:
         # Check for tools that will be overridden
         overridden_tools = set(cls._instance.tools.keys()) & set(custom_tools.keys())
         if overridden_tools:
-            logger.warning(f"Loaded tools will override existing tools: " f"{', '.join(sorted(overridden_tools))}")
+            logger.warning(f"Loaded tools will override existing tools: {', '.join(sorted(overridden_tools))}")
 
         cls._instance.tools.update(custom_tools)
         return cls._instance

sgr_agent_core/services/tool_instantiator.py

@@ -391,7 +391,7 @@ def build_model(self, content: str) -> "BaseTool":
         except ValidationError as e:
             for err in e.errors():
                 self.errors.append(
-                    f"pydantic validation error - type: {err['type']} - " f"field: {err['loc']} - {err['msg']}"
+                    f"pydantic validation error - type: {err['type']} - field: {err['loc']} - {err['msg']}"
                 )
             raise ValueError("Failed to build model") from e
         except JSONDecodeError as e: