Fix missing imports in examples, use internal log level filter

- Add DataverseClient import to README and SKILL.md telemetry snippets
- Stop calling logger.setLevel() which mutates global logging state;
  use internal _log_level threshold to filter emissions instead
- Update test to check _log_level instead of logger.level

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: tpellissier

README.md

@@ -391,6 +391,7 @@ client.files.upload(
 ### Telemetry & observability
 
 ```python
+from PowerPlatform.Dataverse.client import DataverseClient
 from PowerPlatform.Dataverse.core.telemetry import TelemetryConfig, TelemetryHook
 from PowerPlatform.Dataverse.core.config import DataverseConfig
 

src/PowerPlatform/Dataverse/claude_skill/dataverse-sdk-use/SKILL.md

@@ -321,6 +321,7 @@ client.files.upload(
 The SDK supports opt-in telemetry via hooks, OpenTelemetry, and Python logging:
 
 ```python
+from PowerPlatform.Dataverse.client import DataverseClient
 from PowerPlatform.Dataverse.core.telemetry import TelemetryConfig, TelemetryHook
 from PowerPlatform.Dataverse.core.config import DataverseConfig
 

src/PowerPlatform/Dataverse/core/telemetry.py

@@ -219,6 +219,7 @@ def __init__(self, config: TelemetryConfig) -> None:
         self._tracer: Any = None
         self._meter: Any = None
         self._logger: Optional[logging.Logger] = None
+        self._log_level: int = logging.WARNING
         self._hooks = list(config.hooks)
 
         # Metric instruments (populated in _initialize)
@@ -261,8 +262,7 @@ def _initialize(self) -> None:
 
         if self._config.enable_logging:
             self._logger = logging.getLogger(self._config.logger_name)
-            level = getattr(logging, self._config.log_level.upper(), logging.WARNING)
-            self._logger.setLevel(level)
+            self._log_level = getattr(logging, self._config.log_level.upper(), logging.WARNING)
 
     def _setup_metrics(self) -> None:
         if not self._meter:
@@ -426,23 +426,24 @@ def record_response(
             except Exception:
                 pass
 
-        # 3. Logging
+        # 3. Logging (use _log_level as internal filter to avoid mutating global logger state)
         if self._logger:
             try:
                 level = logging.WARNING if (status_code >= 400 or error) else logging.DEBUG
-                self._logger.log(
-                    level,
-                    "%s %s %s %.1fms",
-                    ctx.operation,
-                    ctx.method,
-                    status_code,
-                    duration_ms,
-                    extra={
-                        "client_request_id": ctx.client_request_id,
-                        "correlation_id": ctx.correlation_id,
-                        "service_request_id": service_request_id,
-                    },
-                )
+                if level >= self._log_level:
+                    self._logger.log(
+                        level,
+                        "%s %s %s %.1fms",
+                        ctx.operation,
+                        ctx.method,
+                        status_code,
+                        duration_ms,
+                        extra={
+                            "client_request_id": ctx.client_request_id,
+                            "correlation_id": ctx.correlation_id,
+                            "service_request_id": service_request_id,
+                        },
+                    )
             except Exception:
                 pass
 

tests/unit/core/test_telemetry.py

@@ -193,7 +193,7 @@ def test_logging_enabled_creates_logger(self):
     def test_invalid_log_level_falls_back_to_warning(self):
         mgr = TelemetryManager(TelemetryConfig(enable_logging=True, log_level="VERBOSE"))
         self.assertIsNotNone(mgr._logger)
-        self.assertEqual(mgr._logger.level, logging.WARNING)
+        self.assertEqual(mgr._log_level, logging.WARNING)
 
     def test_factory_with_hooks_only_returns_real_manager(self):
         hook = TelemetryHook()