Fix build: skip OTel tests when not installed, fix docs

- Add skipUnless(_OTEL_AVAILABLE) to 3 tests that require opentelemetry
- Clarify on_request_error docstring: covers both HTTP and network errors
- Fix SKILL.md: separate request vs response fields in hook data docs

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

Author: tpellissier

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

@@ -343,7 +343,8 @@ config = DataverseConfig(
 )
 ```
 
-Hook data available per request: `operation`, `table_name`, `method`, `url`, `status_code`, `duration_ms`, `service_request_id`, `client_request_id`, `correlation_id`.
+Hook data available on the **request** object: `operation`, `table_name`, `method`, `url`, `client_request_id`, `correlation_id`.
+Hook data available on the **response** object: `status_code`, `duration_ms`, `service_request_id`.
 
 Zero overhead when `TelemetryConfig` is not set.
 

src/PowerPlatform/Dataverse/core/telemetry.py

@@ -191,7 +191,12 @@ def on_request_end(self, request: RequestContext, response: ResponseContext) ->
         """Called after each HTTP request completes (success or failure)."""
 
     def on_request_error(self, request: RequestContext, error: Exception) -> None:
-        """Called when an unhandled network-level exception occurs."""
+        """Called when an error occurs during an HTTP request.
+
+        This includes both network/transport exceptions and HTTP-level
+        failures (e.g. 4xx/5xx responses surfaced as ``HttpError``).
+        Always fired before ``on_request_end``.
+        """
 
     def get_additional_headers(self) -> Dict[str, str]:
         """Return additional headers to include in outbound requests."""

tests/unit/core/test_telemetry.py

@@ -16,6 +16,7 @@
     TelemetryHook,
     TelemetryManager,
     _TrackedRequest,
+    _OTEL_AVAILABLE,
     create_telemetry_manager,
 )
 from PowerPlatform.Dataverse.data._odata import (
@@ -332,6 +333,7 @@ def test_span_not_created_without_otel(self):
             ) as tracked:
                 self.assertIsNone(tracked._span)
 
+    @unittest.skipUnless(_OTEL_AVAILABLE, "requires opentelemetry")
     def test_span_end_called_in_finally(self):
         mock_tracer = MagicMock()
         mock_span = MagicMock()
@@ -347,6 +349,7 @@ def test_span_end_called_in_finally(self):
 
         mock_span.end.assert_called_once()
 
+    @unittest.skipUnless(_OTEL_AVAILABLE, "requires opentelemetry")
     def test_record_response_sets_span_status_ok(self):
         from opentelemetry.trace import StatusCode
 
@@ -364,6 +367,7 @@ def test_record_response_sets_span_status_ok(self):
         status_arg = mock_span.set_status.call_args[0][0]
         self.assertEqual(status_arg.status_code, StatusCode.OK)
 
+    @unittest.skipUnless(_OTEL_AVAILABLE, "requires opentelemetry")
     def test_record_response_sets_span_status_error(self):
         mock_span = MagicMock()
         ctx = RequestContext(