fix: use get_span_context() to handle NonRecordingSpan

[nikhilNava]

Summary

When the token resolver returns None on the first turn, the TracerProvider produces a NonRecordingSpan. The .context attribute only exists on concrete ReadableSpan implementations, causing AttributeError crashes on the first agent message.

Changes

Replace all span.context accesses with the public get_span_context() interface method, which is available on all Span types including NonRecordingSpan. Also guard the .name access in _end() with getattr() since NonRecordingSpan lacks that attribute as well.

Files modified:

• opentelemetry_scope.py: use get_span_context() in span start/end logging
• enriched_span.py: delegate context property via get_span_context()
• agent365_exporter.py: use get_span_context() in _map_span()
• Test mocks updated: set get_span_context() return value on all mock spans
• New test file: test_nonrecording_span.py with 6 targeted tests

Source

Ported from: microsoft/opentelemetry-distro-python#168

Testing

All 227 observability core tests pass.

[github-actions]

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[Copilot]

Pull request overview

This PR hardens the observability pipeline against NonRecordingSpan instances by switching span-context access from the .context attribute (not present on NonRecordingSpan) to the get_span_context() API, preventing first-turn AttributeError crashes.

Changes:

• Replace .context usage with get_span_context() in span logging, span wrappers, and exporter mapping.
• Guard span .name access during scope shutdown to handle spans that don’t expose name.
• Update span mocks and add a dedicated test module covering NonRecordingSpan scenarios.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/observability/core/test_nonrecording_span.py	Adds targeted tests to ensure NonRecordingSpan does not crash scope lifecycle, span wrapping, or exporter mapping.
tests/observability/core/test_agent365_exporter.py	Updates mock spans to provide get_span_context() for exporter tests.
tests/observability/core/exporters/test_payload_chunking.py	Updates mock spans to provide get_span_context() for payload chunking tests.
tests/observability/core/exporters/test_enriching_span_processor.py	Updates mock spans to provide get_span_context() for processor tests.
tests/observability/core/exporters/test_enriched_span.py	Updates mock spans to provide get_span_context() for enriched span tests.
libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/opentelemetry_scope.py	Uses get_span_context() for logging span IDs and guards span name access in _end().
libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/enriched_span.py	Makes EnrichedReadableSpan.context delegate via get_span_context().
libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/agent365_exporter.py	Uses get_span_context() in _map_span() to extract trace/span IDs.