docs: clarify BYOK and MCP registry enforcement behavior#26900
Merged
Conversation
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/4288c841-4380-44f6-beaf-7db2c8840676 Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
pelikhan
April 17, 2026 15:18
View session
pelikhan
approved these changes
Apr 17, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Documentation updates clarifying that features.byok-copilot and MCP registry are convenience/metadata fields rather than enforcement boundaries, with an added link to official GitHub Copilot BYOK documentation.
Changes:
- Clarify MCP
registryfield is informational and not enforced by gh-aw across MCP reference/guides. - Clarify
features.byok-copilotis a convenience extension point (not enforcement) and link to official Copilot BYOK docs.
Show a summary per file
| File | Description |
|---|---|
| docs/src/content/docs/reference/tools.md | Clarifies registry is informational and not enforced by gh-aw in the tools reference. |
| docs/src/content/docs/reference/frontmatter.md | Adds IMPORTANT note that byok-copilot is not an enforcement boundary and links to official Copilot BYOK docs. |
| docs/src/content/docs/guides/mcps.md | Clarifies registry-based MCP server references are informational and not enforced by gh-aw. |
| docs/src/content/docs/guides/getting-started-mcp.md | Clarifies registry usage is informational and not enforced by gh-aw in the getting-started guide. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 4/4 changed files
- Comments generated: 1
| > `byok-copilot` is a gh-aw convenience extension point, not an enforcement boundary. gh-aw does not enforce Copilot BYOK usage. | ||
| > | ||
| > For Copilot BYOK setup and policy details, see [Using your LLM provider API keys with Copilot](https://docs.github.com/en/copilot/how-tos/administer-copilot/manage-for-enterprise/use-your-own-api-keys). | ||
|
|
There was a problem hiding this comment.
Line 440 appears to contain trailing whitespace (two spaces on an otherwise blank line). This can cause unintended Markdown line breaks and may trip markdown/whitespace linters; please replace it with a truly empty line.
Suggested change
Closed
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.
Summary
features.byok-copilotis a convenience extension point and not an enforcement boundaryregistryusage is informational and not enforced by gh-aw in MCP docsValidation
make agent-finish(fails due to pre-existing testifylint issues inpkg/constants/spec_test.go, unrelated to these documentation-only changes)parallel_validation(Code Review ✅, CodeQL ✅)