Skip to content

[cDAC] Consolidate metadata contract APIs - #131572

Draft
rcj1 with Copilot wants to merge 3 commits into
mainfrom
copilot/consolidate-metadata-apis
Draft

[cDAC] Consolidate metadata contract APIs#131572
rcj1 with Copilot wants to merge 3 commits into
mainfrom
copilot/consolidate-metadata-apis

Conversation

Copilot AI commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Consolidates duplicate EcmaMetadata APIs while preserving read-only, saved-copy, and live read-write metadata semantics.

Changes

  • Replace separate address APIs with GetMetadataAddress(ModuleHandle, MetadataAddressKind).
  • Fold read-write blob access into GetMetadata with explicit read-write validation.
  • Update DacDbi and SOS consumers to use the consolidated surface.
  • Update contract documentation and focused unit/dump tests.

Copilot AI and others added 3 commits July 30, 2026 05:00
Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Co-authored-by: rcj1 <77995559+rcj1@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 30, 2026 05:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 5 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-Diagnostics-coreclr NO-REVIEW Experimental/testing PR, do NOT review it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants