[docs] Update glossary - daily scan 2026-07-09#9772
Merged
Evangelink merged 5 commits intoJul 9, 2026
Conversation
- AzureFoundry: add DefaultAzureCredential/managed identity auth details and required environment variables (PR #9707) - MSTestTestFramework: new entry documenting the native MTP ITestFramework for MSTest introduced by RFC 018 (PRs #9706, #9743, #9748, #9755) - VSTestBridge: note MSTest no longer depends on it on the MTP path as of MSTest 4.3 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Match the existing glossary convention of referencing RFC files as plain backtick paths instead of clickable links whose target (docs/RFCs/...) resolved incorrectly relative to docs/glossary.md. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the repository glossary to reflect recent changes in MSTest’s native Microsoft.Testing.Platform (MTP) integration and AzureFoundry authentication behavior.
Changes:
- Expanded AzureFoundry entry to document managed identity /
DefaultAzureCredentialauthentication and related environment variables. - Added MSTestTestFramework entry describing MSTest’s native
ITestFrameworkpath on MTP and the key native seams involved. - Updated VSTestBridge entry to clarify MSTest’s removal of the bridge dependency on the MTP path as of MSTest 4.3.
Show a summary per file
| File | Description |
|---|---|
| docs/glossary.md | Adds/updates glossary entries for AzureFoundry auth, MSTestTestFramework native MTP integration, and VSTestBridge dependency notes. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Low
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Note
🤖 Automated review by GitHub Copilot. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.
Documentation-only PR — limited-scope review
This PR only modifies docs/glossary.md. Most code-focused review dimensions (threading, security, API compat, performance, etc.) are N/A.
Dimension 17 — Documentation Accuracy
Verified ✓:
MSTestTestFramework,MtpUnitTestElementSink,MtpTestResultRecorder,MSTestTestNodeConverter— all class names confirmed in source (src/Adapter/MSTest.TestAdapter/TestingPlatformAdapter/).FrameworkTestResultis a valid using-alias forMicrosoft.VisualStudio.TestTools.UnitTesting.TestResultin the relevant files.- RFC 018 (
docs/RFCs/018-Native-MTP-Integration-For-MSTest.md) exists in the repo. - Environment variables (
AZURE_OPENAI_ENDPOINT,AZURE_OPENAI_DEPLOYMENT_NAME,AZURE_OPENAI_API_KEY,AZURE_CLIENT_ID) confirmed inOpenAIChatClientProvider.csandPACKAGE.md. DefaultAzureCredentialusage confirmed in source.- Cross-reference anchors (
#vstestbridge↔#mstesttestframework) resolve correctly. - RFC reference format (backtick paths) is consistent with existing glossary entries.
- Markdown table is well-formed.
One NIT (inline comment): The MSTestTestFramework entry's claim that all five listed MTP interfaces are read "from MTP's IServiceProvider directly" is slightly imprecise — IMessageBus and ITestExecutionFilter come from the ExecuteRequestContext/request objects rather than IServiceProvider.
Dimension 21 — Scope & PR Discipline
✓ Appropriately scoped single-file documentation update. References issue #9766 correctly. PR description accurately summarizes what changed.
IMessageBus and ITestExecutionFilter are read from the per-request ExecuteRequestContext/request objects, not from IServiceProvider. Addresses review feedback on PR #9772. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Add explicit subject to the AzureFoundry description for grammatical consistency with neighboring entries. Addresses review feedback on PR #9772. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
0101
approved these changes
Jul 9, 2026
Evangelink
deleted the
docs/glossary-daily-scan-2026-07-09-8d9172e177506ef8
branch
July 9, 2026 09:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #9766.
Applies the daily glossary scan for 2026-07-09 originally produced by the Glossary Maintainer workflow (which could not open the PR itself due to GitHub Actions PR-creation restrictions), rebased onto the latest
main.Changes to
docs/glossary.mdAdded
ITestFrameworkfor MSTest on MTP introduced by RFC 018 (PRs Native MTP integration for MSTest — RFC 018 + Phases 1–5 (opt-in native path) #9706, Native MTP framework for MSTest — Phases 3–5 (native filter/runsettings/context, opt-in) #9743, Native MTP integration for MSTest (Phase 6a): native providers, config, resources #9748, Drop MSTest's VSTest bridge dependency on the MTP path #9755). Describes the three native seams (MtpUnitTestElementSink,MtpTestResultRecorder,MSTestTestNodeConverter) that replace the former VSTest bridge boundary, and notes MSTest 4.3 no longer depends onMicrosoft.Testing.Extensions.VSTestBridgeon the MTP path.Updated
DefaultAzureCredential/ managed-identity authentication details and the three environment variables (AZURE_OPENAI_ENDPOINT,AZURE_OPENAI_DEPLOYMENT_NAME,AZURE_OPENAI_API_KEY) introduced in PR Support managed identity auth in AzureFoundry provider #9707.Reviewer notes
docs/RFCs/...paths) rather than clickable links pointing atdocs/RFCs/..., which resolved incorrectly relative todocs/glossary.md.