Skip to content

Update docs with the contextual token resolver config#190

Merged
rads-1996 merged 3 commits into
microsoft:mainfrom
rads-1996:update-readme-with-token-resolver
Jun 5, 2026
Merged

Update docs with the contextual token resolver config#190
rads-1996 merged 3 commits into
microsoft:mainfrom
rads-1996:update-readme-with-token-resolver

Conversation

@rads-1996
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings June 5, 2026 15:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the documentation for Agent 365 (A365) configuration to include the new contextual token resolver option, clarifying its behavior and precedence relative to the existing token resolver.

Changes:

  • Document a365_contextual_token_resolver in the main README configuration table, including precedence rules.
  • Update the A365 migration guide example and checklist to reference the contextual resolver.

Reviewed changes

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

File Description
README.md Adds a365_contextual_token_resolver to the configuration reference and clarifies precedence vs a365_token_resolver.
MIGRATION_A365.md Updates migration example and checklist to include/mention the contextual resolver option.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread MIGRATION_A365.md Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

Performance comparison

Threshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.

Scenario Gating Baseline (ops/s) Candidate (ops/s) Δ % Status
azure_monitor_log yes 35,634.1 35,345.7 +0.82%
azure_monitor_span yes 162,364.0 163,158.8 -0.49%
otel_log no 42,890.8 42,744.2 +0.34%
otel_span no 43,890.4 44,338.0 -1.01%

nikhilNava
nikhilNava previously approved these changes Jun 5, 2026
Copy link
Copy Markdown
Collaborator

@nikhilNava nikhilNava left a comment

Choose a reason for hiding this comment

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

Approved with comments

Comment thread MIGRATION_A365.md Outdated
@rads-1996 rads-1996 merged commit 7b20166 into microsoft:main Jun 5, 2026
11 checks passed
@rads-1996 rads-1996 deleted the update-readme-with-token-resolver branch June 5, 2026 18:30
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.

4 participants