style: fix error [SIM103] - Return the condition bool(<CONDITION>) directly.

Used Ruff (vscode and pre-commit) to:
- Black-compatible code formatting.
- fix all auto-fixable violations.
- isort-compatible import sorting.
- flake8-simplify manual fixes.

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>

Author: pvital

src/instana/collector/host.py

@@ -70,9 +70,7 @@ def prepare_and_report_data(self) -> None:
 
     def should_send_snapshot_data(self) -> bool:
         delta = int(time()) - self.snapshot_data_last_sent
-        if delta > self.snapshot_data_interval:
-            return True
-        return False
+        return delta > self.snapshot_data_interval
 
     def prepare_payload(self) -> DefaultDict[Any, Any]:
         payload = DictionaryOfStan()

src/instana/span/stack_trace.py

@@ -26,54 +26,50 @@
 def _should_collect_stack(level: str, is_errored: bool) -> bool:
     """
     Determine if stack trace should be collected based on level and error state.
-    
+
     Args:
         level: Stack trace collection level ("all", "error", or "none")
         is_errored: Whether the span has errors (ec > 0)
-    
+
     Returns:
         True if stack trace should be collected, False otherwise
     """
     if level == "all":
         return True
-    if level == "error" and is_errored:
-        return True
-    return False
+    return bool(level == "error" and is_errored)
 
 
 def _should_exclude_frame(frame) -> bool:
     """
     Check if a frame should be excluded from the stack trace.
-    
+
     Frames are excluded if they are part of Instana's internal code,
     unless INSTANA_DEBUG is set.
-    
+
     Args:
         frame: A frame from traceback.extract_stack()
-    
+
     Returns:
         True if frame should be excluded, False otherwise
     """
     if "INSTANA_DEBUG" in os.environ:
         return False
     if _re_tracer_frame.search(frame[0]):
         return True
-    if _re_with_stan_frame.search(frame[2]):
-        return True
-    return False
+    return bool(_re_with_stan_frame.search(frame[2]))
 
 
 def _apply_stack_limit(
     sanitized_stack: List[dict], limit: int, use_full_stack: bool
 ) -> List[dict]:
     """
     Apply frame limit to the sanitized stack.
-    
+
     Args:
         sanitized_stack: List of stack frames
         limit: Maximum number of frames to include
         use_full_stack: If True, ignore the limit
-    
+
     Returns:
         Limited stack trace
     """
@@ -84,20 +80,18 @@ def _apply_stack_limit(
     return sanitized_stack[(limit * -1) :]
 
 
-def add_stack(
-    level: str, limit: int, is_errored: bool = False
-) -> Optional[List[dict]]:
+def add_stack(level: str, limit: int, is_errored: bool = False) -> Optional[List[dict]]:
     """
     Capture and return a stack trace based on configuration.
-    
+
     This function collects the current call stack, filters out Instana
     internal frames, and applies the configured limit.
-    
+
     Args:
         level: Stack trace collection level ("all", "error", or "none")
         limit: Maximum number of frames to include (1-40)
         is_errored: Whether the span has errors (ec > 0)
-    
+
     Returns:
         List of stack frames in format [{"c": file, "n": line, "m": method}, ...]
         or None if stack trace should not be collected
@@ -134,25 +128,21 @@ def add_stack(
 def add_stack_trace_if_needed(span: "InstanaSpan") -> None:
     """
     Add stack trace to span based on configuration before span ends.
-    
+
     This function checks if the span is an EXIT span and if so, captures
     a stack trace based on the configured level and limit. It supports
     technology-specific configuration overrides via get_stack_trace_config().
-    
+
     Args:
         span: The InstanaSpan to potentially add stack trace to
     """
     if span.name in EXIT_SPANS:
         # Get configuration from agent options (with technology-specific overrides)
         options = span._span_processor.agent.options
         level, limit = options.get_stack_trace_config(span.name)
-        
+
         # Check if span is errored
         is_errored = span.attributes.get("ec", 0) > 0
-        
+
         # Capture stack trace using add_stack function
-        span.stack = add_stack(
-            level=level,
-            limit=limit,
-            is_errored=is_errored
-        )
+        span.stack = add_stack(level=level, limit=limit, is_errored=is_errored)