Skip to content

docs: fix alt text and dead links in README and ADDITIONAL.md - #5467

Merged
cliffhall merged 1 commit into
v2/mainfrom
v2/docs/5212-readme-additional-link-fixes
Nov 5, 2026
Merged

cliffhall merged 1 commit into
v2/mainfrom
v2/docs/5212-readme-additional-link-fixes

Conversation

@cliffhall

Copy link
Copy Markdown
Member

Closes #5212

Description

Four corrections to existing entries, each URL checked before changing it:

  1. llm-analysis-assistant (ADDITIONAL.md): the icon's alt text read Langfuse Logo, copied from another entry. It now reads llm-analysis-assistant Logo.
  2. OpenTools (ADDITIONAL.md): the attribution linked github.com/opentoolsteam, which is gone. It now links github.com/opentools, the GitHub org that opentools.com itself links to (its profile's homepage is opentools.com).
  3. AgentR (ADDITIONAL.md): agentr.dev/home is dead. The attribution keeps the name and drops the link; the SDK repo it describes is live and unchanged.
  4. Community (README.md): github.com/orgs/modelcontextprotocol/discussions is 404. It now links the live discussions on modelcontextprotocol/modelcontextprotocol.

Server Details

  • Server: none (repository-wide)
  • Changes to: README.md, ADDITIONAL.md (docs only)

Motivation and Context

#5212, carrying on the prior work in #4169, #4675 and #4884 (thanks to their authors). #4884 proposed agentr.dev/ in place of /home; that root is dead too, so the link is dropped instead.

How Has This Been Tested?

No client-observable surface, so the probe is the URL checks (curl -sL -w '%{http_code}', 2026-11-04):

URL Result
https://github.com/opentoolsteam 404 (and gh api users/opentoolsteam → 404)
https://github.com/opentools org OpenTools, homepage https://opentools.com; opentools.com's footer links it
https://agentr.dev/home, https://agentr.dev/ 402, body DEPLOYMENT_DISABLED
https://github.com/universal-mcp/universal-mcp 200 (entry kept)
https://github.com/orgs/modelcontextprotocol/discussions 404
https://github.com/modelcontextprotocol/modelcontextprotocol/discussions 200

npm run local:gate exited 0.

Breaking Changes

None.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Protocol Documentation (not applicable: no protocol feature touched)
  • My changes follow MCP security best practices (not applicable: docs links only)
  • I have updated the server's README accordingly (not applicable: no server changed)
  • I have added a changeset (npm run changeset) if this changes what a TypeScript server publishes (not applicable: no published package changes)
  • I have tested this with an LLM client (not applicable: no server-facing change)
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling (not applicable: docs only)
  • I have documented all environment variables and configuration options (not applicable: none added)

🤖 Generated with Claude Code

- llm-analysis-assistant: the icon's alt text named Langfuse (copied from
  another entry); it now names the project.
- OpenTools: github.com/opentoolsteam is 404. Point the attribution at
  github.com/opentools, the org opentools.com itself links to.
- AgentR: agentr.dev returns 402 DEPLOYMENT_DISABLED on / and /home, so
  the attribution keeps the name and drops the dead link.
- README Community: github.com/orgs/modelcontextprotocol/discussions is
  404; link the live discussions on modelcontextprotocol/modelcontextprotocol.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: cliffhall <cliff@futurescale.com>
@cliffhall cliffhall added the v2 label Nov 5, 2026
@changeset-bot

changeset-bot Bot commented Nov 5, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 8c644a0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues are reported for these documentation-only corrections to links, attribution, and icon alt text.

No open issues 🎉

What changed in this PR

Addresses #5212 with four documentation corrections to existing community and resource entries. No server behavior or published packages change.

Changes:

  • Corrects the llm-analysis-assistant icon’s alt text.
  • Updates the OpenTools attribution and removes AgentR’s dead website link while retaining its SDK link.
  • Points the Community link to the MCP specification repository’s discussions.
File Description
README.md Corrects the GitHub Discussions destination.
ADDITIONAL.md Fixes icon alt text and two project attributions.

Review effort: Balanced

@cliffhall

Copy link
Copy Markdown
Member Author

Copilot round 1: clean (no findings, no suppressed comments). Loop converged on the first clean round.

@cliffhall
cliffhall merged commit c11c81d into v2/main Nov 5, 2026
41 checks passed
@cliffhall
cliffhall deleted the v2/docs/5212-readme-additional-link-fixes branch November 5, 2026 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

README / ADDITIONAL.md: small link and alt-text corrections

2 participants