docs(github): remove stale PAT/notifications references from GitHub MCP server docs#996
Draft
jottakka wants to merge 1 commit into
Draft
docs(github): remove stale PAT/notifications references from GitHub MCP server docs#996jottakka wants to merge 1 commit into
jottakka wants to merge 1 commit into
Conversation
…CP server docs The GitHub MCP server no longer ships notification tools (GetNotificationSummary, ListNotifications), which were the only tools that required a classic Personal Access Token. Remove the now-stale PAT and notifications content from both the generated toolkit docs (github.json documentation chunks) and the hand-authored GitHub auth provider page: - Drop the 'Special: Notifications' permissions block and the GITHUB_CLASSIC_PERSONAL_ACCESS_TOKEN secret row/callout. - Remove the Notifications rows from the permissions and troubleshooting tables and the notifications FAQ entry; rename the trailing permissions tab to 'Act on Behalf of User'. - Fix the ListIssues 'since' parameter description that wrongly referenced notifications. - Clarify that GITHUB_SERVER_URL is the only (optional) secret, since the PAT was the only required one.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
2 participants
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.
What & why
The GitHub MCP server docs still described notification tools and a classic Personal Access Token (PAT) requirement, but those tools (
GetNotificationSummary/ListNotifications) have been removed from the toolkit. The PAT (GITHUB_CLASSIC_PERSONAL_ACCESS_TOKEN) was only ever needed for those notification tools, so all of that content is now stale and misleading.This PR removes the stale content from both the generated toolkit docs and the hand-authored GitHub auth provider page, and tidies up the affected sections.
How docs are generated (context)
The GitHub MCP server page is rendered from
toolkit-docs-generator/data/toolkits/github.json. The prose (documentationChunks) is hand-curated content that the generator carries forward from the committed JSON when no custom-sections file is supplied (the scheduledgenerate-toolkit-docsworkflow runs with--skip-unchangedand no--custom-sections). Editing the committed JSON directly is therefore the durable fix. Tool definitions/parameters come from the Engine API.Changes
Generated docs —
toolkit-docs-generator/data/toolkits/github.jsonGITHUB_CLASSIC_PERSONAL_ACCESS_TOKENsecrets-table row and the "For Notifications Tools" PAT warning callout.GITHUB_SERVER_URLis the only (optional) secret, now that the PAT is gone.ListIssuessinceparameter description that incorrectly said "notifications" instead of "issues".Hand-authored docs —
app/en/references/auth-providers/github/page.mdxHeading text for anchored sections (e.g.
#creating-a-github-app,#configuring-github-auth-in-arcade,#why-arcade-uses-github-apps-not-oauth-apps,#tool-permissions-reference) was preserved to avoid breaking internal links.Verification
github.jsonre-validated as parseable JSON; no remainingnotification/PAT/CLASSICreferences in either file.tests/internal-link-check.test.ts,tests/broken-link-check.test.ts,tests/sitemap.test.ts,tests/filter-params.test.tspass.tests/stale-summaries.test.tsandtests/merger/secret-coherence.test.tspass.