Repository navigation
Conversation
Routine external-link audit found 9 dead links across 6 files. Every old target was verified dead (HTTP 404, or sample removed upstream) and every replacement verified live (HTTP 200) at submission time. Replacements: - Deepgram streaming docs: /docs/streaming -> /docs/live-streaming-audio (sitemap-verified current page) - AssemblyAI streaming docs: /docs/real-time-speech-recognition -> /docs/streaming (sitemap-verified) - A2A protocol: google.github.io/A2A (404) -> a2a-protocol.org/latest/ (project's current official site) - GitHub MCP Server: modelcontextprotocol/servers/src/github (removed upstream) -> github/github-mcp-server (official successor) - csharp-sdk samples: AspNetCoreSseServer (renamed) -> AspNetCoreMcpServer; ProtectedMCPClient -> ProtectedMcpClient (exact path casing) - Pinecone hybrid search: /guides/get-started/key-features#hybrid-search (404) -> /guides/search/hybrid-search - Elastic .NET API docs page (whole /api/ tree removed from elasticsearch-net docs site) -> Elastic reference docs for the text field type This contribution was AI-assisted; all old and new URLs were checked programmatically.
semantic-kernel-automation
Bot
added
.NET
github-actions
Bot
changed the title
github-actions
Bot
changed the title
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are reported for these documentation link updates.
No open issues 🎉
What changed in this PR
Updates nine external links across Semantic Kernel sample documentation and the hybrid-search architecture decision record. Changes are limited to Markdown URLs and one sample link label; runtime behavior is unchanged.
Changes:
- Refresh speech-service, A2A protocol, and GitHub MCP server documentation links.
- Correct renamed MCP C# SDK sample paths and path casing.
- Update Pinecone hybrid-search and Elasticsearch text-field references.
| File | Description |
|---|---|
python/samples/concepts/mcp/README.md |
Points authentication guidance to the official GitHub MCP server. |
dotnet/samples/Demos/VoiceChat/README.md |
Updates Deepgram and AssemblyAI streaming documentation links. |
dotnet/samples/Demos/ModelContextProtocolPluginAuth/README.md |
Corrects casing in two protected-client sample links. |
dotnet/samples/Demos/ModelContextProtocolClientServer/README.md |
Updates the remote-server sample URL and link label. |
dotnet/samples/Demos/A2AClientServer/README.md |
Links to the current A2A protocol site. |
docs/decisions/0067-hybrid-search.md |
Refreshes Pinecone and Elasticsearch documentation references. |
Review effort: Balanced
This branch was successfully deployed
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.
Description
Routine external-link audit found 9 dead links across 6 files. Every old target was verified dead (HTTP 404, or the sample/directory removed upstream) and every replacement was verified live (HTTP 200) at submission time. All fixes are link-only; no code or prose meaning changed.
dotnet/samples/Demos/VoiceChat/README.mddotnet/samples/Demos/VoiceChat/README.mddotnet/samples/Demos/A2AClientServer/README.mdpython/samples/concepts/mcp/README.mdmodelcontextprotocol/servers;github/github-mcp-serveris the official successor with PAT documentationdotnet/samples/Demos/ModelContextProtocolClientServer/README.mddotnet/samples/Demos/ModelContextProtocolPluginAuth/README.md(x2)docs/decisions/0067-hybrid-search.mddocs/decisions/0067-hybrid-search.mdtextfield typeMotivation
These links are the first entry point for users trying to run the VoiceChat, A2A, and MCP samples, or evaluating vector stores via ADR-0067; each currently dead-ends.
Verification
curl -I -L(browser user-agent) returned 404 for all; the two GitHub paths were additionally confirmed against the upstream repos' Contents API (samples/AspNetCoreSseServerandsrc/githubno longer exist;ProtectedMcpClientexists only with that casing).github/github-mcp-serverconfirmed via API.This contribution was AI-assisted; all old and new URLs were checked programmatically during the audit.