Skip to content

fix(a365): preserve scope attribute precedence - #243

Draft
Nikhil Navakiran (nikhilNava) wants to merge 2 commits into
mainfrom
fix/a365-scope-attribute-precedence
Draft

Nikhil Navakiran (nikhilNava) wants to merge 2 commits into
mainfrom
fix/a365-scope-attribute-precedence

Conversation

@nikhilNava

Copy link
Copy Markdown
Collaborator

Summary

  • track attributes actually populated by manual A365 scope builders
  • prevent recordAttributes() from overwriting builder-owned values
  • continue allowing absent known attributes and last-write-wins custom attributes

Validation

  • targeted TDD: 72 tests passed
  • npm run build
  • npm run format
  • npm run lint (warnings only)
  • npm run test:unit (1096 passed, 5 todo)
  • npm run docs (existing warnings only)
  • git diff --check

Parity with microsoft/opentelemetry-distro-dotnet#107. ETW logger changes are out of scope.

nikhilc-microsoft and others added 2 commits September 15, 2026 12:39
Co-authored-by: Copilot <[email protected]>
Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c

This branch has not been deployed

No deployments
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