Skip to content

feat: add Paramore.Brighter.Extensions.OpenTelemetry package#4020

Merged
iancooper merged 3 commits into
BrighterCommand:masterfrom
thomhurst:feat/extensions-opentelemetry
Feb 19, 2026
Merged

feat: add Paramore.Brighter.Extensions.OpenTelemetry package#4020
iancooper merged 3 commits into
BrighterCommand:masterfrom
thomhurst:feat/extensions-opentelemetry

Conversation

@thomhurst

Copy link
Copy Markdown
Contributor

Summary

  • Adds a new Paramore.Brighter.Extensions.OpenTelemetry package that provides a single OpenTelemetryBuilder.AddBrighterInstrumentation() extension method
  • Wires both tracing and metrics in one call by delegating to the existing TracerProviderBuilder and MeterProviderBuilder extensions from Extensions.Diagnostics
  • No exporters bundled — follows OTel best practices where users choose their own exporters

Usage

services.AddOpenTelemetry()
    .ConfigureResource(r => r.AddService("MyService"))
    .AddBrighterInstrumentation()                      // one call for tracing + metrics
    .WithTracing(t => t.AddOtlpExporter())             // user chooses exporters
    .WithMetrics(m => m.AddOtlpExporter());

Files changed

  • New: src/Paramore.Brighter.Extensions.OpenTelemetry/Paramore.Brighter.Extensions.OpenTelemetry.csproj — targets net8.0;net9.0;net10.0, references Extensions.Diagnostics + OpenTelemetry.Extensions.Hosting
  • New: src/Paramore.Brighter.Extensions.OpenTelemetry/OpenTelemetryBuilderExtensions.cs — the extension method
  • Modified: Brighter.slnx — added new project to solution

Test plan

  • dotnet build src/Paramore.Brighter.Extensions.OpenTelemetry/ succeeds with 0 warnings, 0 errors across all 3 TFMs
  • Full solution build (dotnet build Brighter.slnx) passes in CI

🤖 Generated with Claude Code

Adds a new package that provides a single-call OpenTelemetryBuilder.AddBrighterInstrumentation()
extension method, wiring both tracing and metrics in one call. This delegates to the existing
TracerProviderBuilder and MeterProviderBuilder extensions from Extensions.Diagnostics.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
codescene-delta-analysis[bot]

This comment was marked as outdated.

@iancooper

Copy link
Copy Markdown
Member

Thanks @thomhurst. Trying to clear some build/test issues (capitalizing on your work to allow us to run in parallel), then we can address this. Can you add a (short) ADR, just to follow the process? It helps us as we can generate documentation from the ADRs and we need to do a batch of that with the next release.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
codescene-delta-analysis[bot]

This comment was marked as outdated.

@thomhurst

Copy link
Copy Markdown
Contributor Author

Sure thing! ADR committed.

Let me know if ya need any help with tests

@iancooper iancooper merged commit 0b0519a into BrighterCommand:master Feb 19, 2026
3 of 7 checks passed

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gates Passed
4 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

DevJonny pushed a commit to DevJonny/Brighter that referenced this pull request Feb 28, 2026
…rCommand#4020)

* feat: add Paramore.Brighter.Extensions.OpenTelemetry package

Adds a new package that provides a single-call OpenTelemetryBuilder.AddBrighterInstrumentation()
extension method, wiring both tracing and metrics in one call. This delegates to the existing
TracerProviderBuilder and MeterProviderBuilder extensions from Extensions.Diagnostics.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* docs: add ADR 0039 for OpenTelemetry builder extension

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Ian Cooper <ian_hammond_cooper@yahoo.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants