[Exporter.Prometheus] Separate enums

[martincostello]

Changes

Separate the enums for the Prometheus translation strategy so the two different exporter packages don't have a public type with the same name.

Merge requirement checklist

• CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
• Unit tests added/updated
• Appropriate CHANGELOG.md files updated for non-trivial changes
• Changes in public API reviewed (if applicable)

[Copilot]

🟡 Changes recommended

Both breaking-change entries need the repository-required colon in their prefixes.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Separates Prometheus translation strategy APIs by exporter to prevent public type-name collisions.

Changes:

• Adds exporter-specific public enums backed by an internal shared enum.
• Updates options, API baselines, and tests.
• Documents the breaking renames.

File summaries

File	Description
test/...HttpListener.Tests/PrometheusTranslationStrategyTests.cs	Updates default enum assertion.
test/...HttpListener.Tests/PrometheusProtocolTests.cs	Tests both exporter-specific enums.
test/...AspNetCore.Tests/PrometheusIntegrationTests.cs	Uses the ASP.NET Core enum.
test/...AspNetCore.Tests/PrometheusExporterMiddlewareTests.cs	Updates strategy scenarios.
test/...AspNetCore.Tests/PrometheusExporterMeterProviderBuilderExtensionsTests.cs	Updates option tests.
src/...HttpListener/PrometheusHttpListenerTranslationStrategy.cs	Adds the HttpListener enum.
src/...HttpListener/PrometheusHttpListenerOptions.cs	Exposes the new enum.
src/...HttpListener/PrometheusHttpListenerMeterProviderBuilderExtensions.cs	Converts to the internal enum.
src/...HttpListener/Internal/Shared/PrometheusTranslationStrategy.cs	Makes the shared enum internal.
src/...HttpListener/CHANGELOG.md	Documents the rename.
src/...HttpListener/.publicApi/PublicAPI.Unshipped.txt	Updates the public API baseline.
src/...AspNetCore/PrometheusAspNetCoreTranslationStrategy.cs	Adds the ASP.NET Core enum.
src/...AspNetCore/PrometheusAspNetCoreOptions.cs	Bridges public and internal enums.
src/...AspNetCore/CHANGELOG.md	Documents the rename.
src/...AspNetCore/.publicApi/PublicAPI.Unshipped.txt	Updates the public API baseline.

Review details

• Files reviewed: 15/15 changed files
• Comments generated: 2
• Review effort level: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.86%. Comparing base (6265592) to head (4c1f417).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7751      +/-   ##
==========================================
- Coverage   91.87%   91.86%   -0.02%     
==========================================
  Files         337      337              
  Lines       18380    18380              
==========================================
- Hits        16887    16884       -3     
- Misses       1493     1496       +3     

Flag	Coverage Δ
unittests-Project-Experimental	91.96% <100.00%> (-0.03%)	⬇️
unittests-Project-Stable	91.97% <100.00%> (-0.03%)	⬇️
unittests-UnstableCoreLibraries-Experimental	50.73% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...ometheus.AspNetCore/PrometheusAspNetCoreOptions.cs	100.00% <100.00%> (ø)
...theusHttpListenerMeterProviderBuilderExtensions.cs	94.59% <100.00%> (ø)
...heus.HttpListener/PrometheusHttpListenerOptions.cs	100.00% <ø> (ø)

... and 1 file with indirect coverage changes

[Kielek]

LGTM. I will support this. It occurs in Splunk/Cisco internal Codex scans. I suppose it might occurs also in other similar tools.

I understand that it is breaking change, but we are still in pre-release mode. It is last moment to adjust.