pylint fixes

Author: guenp

src/telemetric/ga4/analytics.py

@@ -58,10 +58,10 @@ def track_event(
         }
 
         url = f"{self.proxy_url}/track"
-        _log.debug(f"POST request to: {url}")
+        _log.debug("POST request to: %s", url)
         _log.debug(payload)
 
         try:  # noqa: SIM105
             requests.post(f"{self.proxy_url}/track", json=payload, timeout=2)
-        except Exception:
+        except (requests.RequestException, OSError):
             pass

src/telemetric/ga4/ga4_proxy.py

@@ -99,7 +99,7 @@ async def track_event(request: Request):  # type: ignore[no-untyped-def]
             {"status": "error", "message": "GA4 request failed"}, status_code=502
         )
 
-    except Exception as e:
+    except (requests.RequestException, ValueError, KeyError) as e:
         return JSONResponse({"status": "error", "message": str(e)}, status_code=500)
 
 

src/telemetric/ga4/stats_uploader.py

@@ -43,15 +43,15 @@ def upload_function_stats(  # type: ignore[no-untyped-def]
             return False
 
         # Get function counts: (total_calls, error_calls, invalid_args)
-        counts = wrapped_func._get_counts()
+        counts = wrapped_func._get_counts()  # pylint: disable=protected-access
         total_calls, error_calls, invalid_args = counts
 
         # Skip functions that haven't been called
         if total_calls == 0:
             return True
 
         # Get parameter statistics
-        param_stats = wrapped_func._get_param_stats()
+        param_stats = wrapped_func._get_param_stats()  # pylint: disable=protected-access
 
         # Build event parameters
         event_params = {
@@ -122,7 +122,7 @@ def upload_all_stats(
         skipped = 0
 
         for wrapped_func in _wrapped:
-            counts = wrapped_func._get_counts()
+            counts = wrapped_func._get_counts()  # pylint: disable=protected-access
             total_calls = counts[0]
 
             if skip_uncalled and total_calls == 0:
@@ -135,9 +135,9 @@ def upload_all_stats(
         # Upload summary statistics
         summary_params = {
             "total_wrapped_functions": len(_wrapped),
-            "functions_called": sum(1 for f in _wrapped if f._get_counts()[0] > 0),
-            "total_function_calls": sum(f._get_counts()[0] for f in _wrapped),
-            "total_errors": sum(f._get_counts()[1] for f in _wrapped),
+            "functions_called": sum(1 for f in _wrapped if f._get_counts()[0] > 0),  # pylint: disable=protected-access
+            "total_function_calls": sum(f._get_counts()[0] for f in _wrapped),  # pylint: disable=protected-access
+            "total_errors": sum(f._get_counts()[1] for f in _wrapped),  # pylint: disable=protected-access
         }
 
         if package_name:

src/telemetric/path_finder.py

@@ -12,6 +12,7 @@
 
 
 class TelemetryMetaFinder(MetaPathFinder):
+    """MetaPathFinder implementation that overrides spec loaders with telemetry-enabled loaders."""
     def __init__(self, module_names: list[str], *args, **kwargs) -> None:  # type: ignore[no-untyped-def]
         """MetaPathFinder implementation that overrides a spec loader
         of type SourceFileLoader with a TelemetrySpanLoader.
@@ -42,6 +43,7 @@ def find_spec(self, fullname: str, path, target=None):  # type: ignore[no-untype
 
 
 class TelemetrySpanSourceFileLoader(SourceFileLoader):
+    """SourceFileLoader that automatically adds telemetry decorators to functions and methods."""
     def exec_module(self, module) -> None:  # type: ignore[no-untyped-def]
         super().exec_module(module)
         functions = inspect.getmembers(module, predicate=inspect.isfunction)

src/telemetric/span.py

@@ -30,14 +30,14 @@ def span(func):  # type: ignore[no-untyped-def]
 
     @wraps(func)
     def span_wrapper(*args, **kwargs):  # type: ignore[no-untyped-def]
-        with tracer.start_as_current_span(func_name) as span:
-            span.set_attribute("num_args", len(args))
-            span.set_attribute("num_kwargs", len(kwargs))
+        with tracer.start_as_current_span(func_name) as current_span:
+            current_span.set_attribute("num_args", len(args))
+            current_span.set_attribute("num_kwargs", len(kwargs))
             for n, arg in enumerate(args):
-                span.set_attribute(f"args.{n}", _serialize(arg))  # type: ignore[no-untyped-call]
+                current_span.set_attribute(f"args.{n}", _serialize(arg))  # type: ignore[no-untyped-call]
             for k, v in kwargs.items():
-                span.set_attribute(f"kwargs.{k}", v)
-            span.set_status(trace.StatusCode.OK)
+                current_span.set_attribute(f"kwargs.{k}", v)
+            current_span.set_status(trace.StatusCode.OK)
             return func(*args, **kwargs)
 
     return span_wrapper

src/telemetric/statswrapper/__init__.py

@@ -1,4 +1,5 @@
 # Copyright (c) 2025 Scientific Python. All rights reserved.
+# pylint: disable=import-error,no-name-in-module
 from __future__ import annotations
 
 import functools
@@ -14,17 +15,17 @@
 _wrapped: list[_StatsWrapper] = []
 
 
-def print_all_stats(skip_uncalled: bool = True) -> None:  # noqa: ARG001
+def print_all_stats(skip_uncalled: bool = True) -> None:  # noqa: ARG001  # pylint: disable=unused-argument
     print()  # noqa: T201
     print("Statistics for argument usage of wrapped functions")  # noqa: T201
     print("--------------------------------------------------")  # noqa: T201
-    sorted_w = sorted(_wrapped, key=lambda x: x._get_counts()[0], reverse=True)
+    sorted_w = sorted(_wrapped, key=lambda x: x._get_counts()[0], reverse=True)  # pylint: disable=protected-access
     for func in sorted_w:
-        counts = func._get_counts()
+        counts = func._get_counts()  # pylint: disable=protected-access
         if counts[0] == 0:
             continue
         counts = f"{counts[0]},{counts[1]},{counts[2]}"
-        stats = func._get_param_stats()
+        stats = func._get_param_stats()  # pylint: disable=protected-access
         argcounts = []
         for name, n_uses, _, _ in stats:
             if name is None:
@@ -146,7 +147,7 @@ def stats_deco_auto(func, /, *, track_positional_use: bool = False):  # type: ig
             pass
 
     new = stats_wrapper(func, *args, **kwargs)
-    new._set_npos(len(args) + positional_kws)
+    new._set_npos(len(args) + positional_kws)  # pylint: disable=protected-access
 
     functools.update_wrapper(new, func)
     if hasattr(func, "__code__"):