[EFCore] [SqlClient] Remove SetDbQueryParameters

[martincostello]

Fixes #3078

Changes

Make the new SetDbQueryParameters property internal and gate behind an environment variable instead.

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]

Pull Request Overview

This PR makes the SetDbQueryParameters property internal and configures it through environment variables instead of exposing it as a public API. The change affects both SqlClient and EntityFrameworkCore instrumentation libraries.

Key changes:

• Made SetDbQueryParameters property internal instead of public
• Added environment variable configuration for both libraries
• Updated documentation to remove public API usage examples

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
SqlClientTraceInstrumentationOptions.cs	Made SetDbQueryParameters internal and added environment variable configuration
EntityFrameworkInstrumentationOptions.cs	Made SetDbQueryParameters internal and added environment variable configuration
SqlClient README.md	Removed public API documentation section
EntityFrameworkCore README.md	Removed public API documentation section
Test files	Added tests for environment variable configuration
PublicAPI.Unshipped.txt	Removed public API entries
CHANGELOG.md	Updated to reference environment variable instead of public option

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.87%. Comparing base (c8c60c4) to head (ad04bce).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3081      +/-   ##
==========================================
+ Coverage   69.74%   69.87%   +0.12%     
==========================================
  Files         410      410              
  Lines       16264    16239      -25     
==========================================
+ Hits        11344    11347       +3     
+ Misses       4920     4892      -28     

Flag	Coverage Δ
unittests-Instrumentation.EntityFrameworkCore	80.80% <100.00%> (+0.23%)	⬆️
unittests-Instrumentation.SqlClient	87.29% <100.00%> (+0.61%)	⬆️

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

Files with missing lines	Coverage Δ
...eworkCore/EntityFrameworkInstrumentationOptions.cs	100.00% <100.00%> (ø)
....SqlClient/SqlClientTraceInstrumentationOptions.cs	100.00% <100.00%> (ø)

... and 8 files with indirect coverage changes

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.