Skip to content

feat(a365): propagate sessions and custom baggage - #242

Merged
Nikhil Navakiran (nikhilNava) merged 25 commits into
mainfrom
feature/a365-custom-baggage
Sep 23, 2026
Merged

Nikhil Navakiran (nikhilNava) merged 25 commits into
mainfrom
feature/a365-custom-baggage

Conversation

@nikhilNava

Copy link
Copy Markdown
Collaborator

Summary

  • propagate request session IDs to manual InvokeAgent, Inference, and ExecuteTool scopes
  • add opt-in custom baggage registration and recognized-GenAI span enrichment
  • preserve span/SDK precedence and document inbound baggage trust boundaries

Validation

  • npm run typecheck:baggage-public-api
  • targeted baggage tests
  • npm run build
  • npm run format
  • npm run lint (warnings only)
  • npm run test:unit
  • npm run test:functional
  • npm run test:esm-build
  • npm run docs (existing warnings only)

Parity with microsoft/opentelemetry-distro-dotnet#116. ETW logger changes are out of scope; JavaScript already omits obsolete tool-server-name data.

nikhilc-microsoft and others added 9 commits September 14, 2026 07:46
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
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 9ccc29b9-bf5d-4725-be3b-c4276b233c4c
Co-authored-by: Copilot <[email protected]>
Copilot-Session: de36d0f9-7edb-42c8-bbca-75c9812c1c31
Co-authored-by: Copilot <[email protected]>
Copilot-Session: de36d0f9-7edb-42c8-bbca-75c9812c1c31
Co-authored-by: Copilot <[email protected]>
Copilot-Session: de36d0f9-7edb-42c8-bbca-75c9812c1c31
Co-authored-by: Copilot <[email protected]>

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

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.

🟡 Changes recommended

Update the documentation to cover the full recognized GenAI operation set.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds session ID propagation to manual A365 scopes and opt-in custom baggage enrichment for recognized GenAI spans.

Changes:

  • Propagates session IDs through inference and tool scopes.
  • Adds custom baggage registration and span enrichment with precedence handling.
  • Adds tests, API typechecks, documentation, changelog, and CI validation.
File summaries
File Reviewed changes
tsconfig.baggage-public-api.json Focused public API typecheck configuration.
test/typecheck/baggageBuilder.public-api.typecheck.ts Public API compatibility checks.
test/internal/unit/a365/scopes.test.ts Session propagation coverage.
test/internal/unit/a365/baggageBuilder.test.ts Custom baggage behavior coverage.
test/internal/unit/a365/a365SpanProcessor.test.ts Span enrichment coverage.
src/a365/scopes/InferenceScope.ts Session ID propagation.
src/a365/scopes/ExecuteToolScope.ts Session ID propagation.
src/a365/processors/A365SpanProcessor.ts Custom baggage span enrichment.
src/a365/middleware/BaggageBuilder.ts Custom baggage registration and metadata handling.
src/a365/constants.ts Internal metadata constant.
package.json Public API typecheck script.
CHANGELOG.md Feature release note.
A365_DOCUMENTATION.md Session and custom baggage documentation; recognized operation coverage needs alignment.
.github/workflows/pr-validation.yml CI typecheck validation.
Review details
  • Files reviewed: 14/14 changed files
  • Comments generated: 1
  • Review effort level: Lite

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

Comment thread A365_DOCUMENTATION.md Outdated
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
nikhilc-microsoft and others added 6 commits September 18, 2026 12:01
Co-authored-by: Copilot <[email protected]>
Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: 8b01797b-8c8b-4ab7-aba9-96a5e7b7a5a0
Co-authored-by: Copilot <[email protected]>

Copilot-Session: a44c89d4-df14-4be5-8c3c-938d09824bc9
Co-authored-by: Copilot <[email protected]>

Copilot-Session: a44c89d4-df14-4be5-8c3c-938d09824bc9
Comment thread src/a365/processors/util.ts
Comment thread src/a365/middleware/BaggageBuilder.ts
Co-authored-by: Copilot <[email protected]>

Copilot-Session: a44c89d4-df14-4be5-8c3c-938d09824bc9

@rads-1996 Radhika Gupta (rads-1996) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@nikhilNava
Nikhil Navakiran (nikhilNava) merged commit c174f56 into main Sep 23, 2026
7 checks passed
@nikhilNava
Nikhil Navakiran (nikhilNava) deleted the feature/a365-custom-baggage branch September 23, 2026 15:28
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.

5 participants