Detect SDK handler in install_log_bridge

[chris-colinsky]

Summary

• install_log_bridge's idempotency check only recognized opentelemetry.instrumentation.logging.handler.LoggingHandler (what OA itself installs). If an application already attached opentelemetry.sdk._logs.LoggingHandler against the same LoggerProvider in its own logging setup, OA's bridge added a second OTel-Logs handler and every log record shipped to OTLP twice. Console handlers are independent so the duplication was OTLP-only — slow to diagnose.
• Widen the check to detect either OTel-Logs handler class wired to the same provider; skip addHandler accordingly. The openarmature.correlation_id LogRecord factory still installs unconditionally. Provider-scoped: applications running a separate logs pipeline against a DIFFERENT LoggerProvider still get OA's bridge attached against the OA provider.
• Document the collision shape in docs/agent/non-obvious-shapes.md and regenerate the bundled AGENTS.md.

Test plan

• New unit test: SDK handler pre-attached against the same provider → OA skips its own addHandler, factory still installs.
• New unit test: pre-attached handler against a DIFFERENT provider → OA does attach (no false-positive dedup).
• Existing idempotency test still passes (instrumentation-handler marker path).
• uv run pyright clean.
• uv run pytest tests/unit/test_observability_otel.py -q green.

[Copilot]

Pull request overview

Fixes a duplicate-OTLP-export bug where install_log_bridge only recognized the instrumentation LoggingHandler class and would attach a second OTel-Logs handler against the same LoggerProvider when an application had already attached the SDK LoggingHandler.

Changes:

• Add _otel_logs_handler_already_bridges helper that detects either OTel LoggingHandler class (SDK or instrumentation) wired to the same provider and skips the duplicate addHandler.
• Add two unit tests covering same-provider skip and different-provider attach.
• Document the collision in docs/agent/non-obvious-shapes.md and regenerate src/openarmature/AGENTS.md.

Reviewed changes

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

File	Description
src/openarmature/observability/otel/logs.py	Widens idempotency check to cover both OTel-Logs handler classes against the same provider.
tests/unit/test_observability_otel.py	Adds two tests: SDK-handler same-provider skip, SDK-handler different-provider attach.
docs/agent/non-obvious-shapes.md	Documents the dual-LoggingHandler collision and helper's provider-scoped dedup behavior.
src/openarmature/AGENTS.md	Regenerated bundle reflecting the new non-obvious shape entry.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.