Skip local error reporting if the user has no API key

Author: onmyraedar

edsl/coop/coop.py

@@ -214,6 +214,11 @@ def get_progress_bar_url(self):
     ################
     # BASIC METHODS
     ################
+    @property
+    def has_api_key(self) -> bool:
+        """True if a non-empty API key is set (constructor arg, class default, or env/config)."""
+        return bool(self.api_key)
+
     @property
     def headers(self) -> dict:
         """
@@ -260,9 +265,9 @@ def _send_server_request(
             if "json_string" in log_payload and log_payload["json_string"]:
                 json_str = log_payload["json_string"]
                 if len(json_str) > 200:
-                    log_payload[
-                        "json_string"
-                    ] = f"{json_str[:200]}... (truncated, total length: {len(json_str)})"
+                    log_payload["json_string"] = (
+                        f"{json_str[:200]}... (truncated, total length: {len(json_str)})"
+                    )
             self._logger.info(f"Request payload: {log_payload}")
 
         try:
@@ -3632,15 +3637,11 @@ def _collect_filestores(d: dict, path: str = ""):
             else:
                 for key, value in d.items():
                     if isinstance(value, dict):
-                        _collect_filestores(
-                            value, f"{path}.{key}" if path else key
-                        )
+                        _collect_filestores(value, f"{path}.{key}" if path else key)
                     elif isinstance(value, list):
                         for i, item in enumerate(value):
                             if isinstance(item, dict):
-                                _collect_filestores(
-                                    item, f"{path}.{key}[{i}]"
-                                )
+                                _collect_filestores(item, f"{path}.{key}[{i}]")
 
         _collect_filestores(modified_dict)
 
@@ -3741,11 +3742,7 @@ def _upload_one(d: dict) -> dict | None:
             if original_object is not None and path:
                 try:
                     clean_path = path.lstrip(".")
-                    keys = (
-                        clean_path.split(".")
-                        if "." in clean_path
-                        else [clean_path]
-                    )
+                    keys = clean_path.split(".") if "." in clean_path else [clean_path]
                     keys = [k for k in keys if k]
                     current_obj = original_object
 
@@ -3765,7 +3762,9 @@ def _upload_one(d: dict) -> dict | None:
                             idx_val = int(idx_str.rstrip("]"))
                             if key_name:
                                 if hasattr(current_obj, key_name):
-                                    current_obj = getattr(current_obj, key_name)[idx_val]
+                                    current_obj = getattr(current_obj, key_name)[
+                                        idx_val
+                                    ]
                                 else:
                                     current_obj = current_obj[key_name][idx_val]
                             else:
@@ -4012,9 +4011,7 @@ def push(
                     value_type = (
                         "inf"
                         if math.isinf(value)
-                        else "nan"
-                        if math.isnan(value)
-                        else "invalid"
+                        else "nan" if math.isnan(value) else "invalid"
                     )
                     error_msg += f"  • {path}: {value} ({value_type})\n"
 
@@ -4462,6 +4459,7 @@ async def report_error(self, error: Exception) -> None:
         This method provides a non-blocking way to report errors that occur during
         EDSL operations. It sends error reports to the server for monitoring and
         debugging purposes, while also printing to stderr for immediate feedback.
+        If no API key is configured, returns immediately without contacting the server.
 
         Duplicate errors (same error type and message) are not reported if they
         occurred within the past minute to prevent spam.
@@ -4476,6 +4474,9 @@ async def report_error(self, error: Exception) -> None:
             ... except Exception as e:
             ...     await coop.report_error(e)
         """
+        if not self.has_api_key:
+            return
+
         import sys
         import traceback
         import httpx