Skip to content

fix(everything): clarify resource link ID mapping#4503

Open
Joosboy wants to merge 1 commit into
modelcontextprotocol:mainfrom
Joosboy:docs-everything-resource-link-ids
Open

fix(everything): clarify resource link ID mapping#4503
Joosboy wants to merge 1 commit into
modelcontextprotocol:mainfrom
Joosboy:docs-everything-resource-link-ids

Conversation

@Joosboy

@Joosboy Joosboy commented Jul 9, 2026

Copy link
Copy Markdown

Summary

  • Clarify the get-resource-links resource ID mapping in the Everything server docs.
  • Rename the local boolean so the implementation reads as "even IDs are text, odd IDs are blob" instead of using a misleading isOdd name.

Why

Issue #475 notes that odd/even resource wording can be confusing. The current Everything server uses dynamic text/{resourceId} and blob/{resourceId} templates; this keeps the existing behavior and documents the mapping used by get-resource-links.

Validation

  • npm test --workspace @modelcontextprotocol/server-everything -- __tests__/tools.test.ts
  • npm run build --workspace @modelcontextprotocol/server-everything
  • npx prettier --check src/everything/tools/get-resource-links.ts src/everything/docs/features.md

Related issue

Refs #475

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant