Skip to content

fix(fetch): avoid Node readability backend by default - #4338

Closed
go165 wants to merge 1 commit into
modelcontextprotocol:mainfrom
go165:go165/fetch-python-readability-default
Closed

go165 wants to merge 1 commit into
modelcontextprotocol:mainfrom
go165:go165/fetch-python-readability-default

Conversation

@go165

@go165 go165 commented Jun 13, 2026

Copy link
Copy Markdown

Summary

Verification

  • /tmp/mcp-fetch-test-venv/bin/python -m pytest src/fetch/tests/test_server.py
  • /tmp/mcp-fetch-test-venv/bin/python -m compileall -q src/fetch/src/mcp_server_fetch src/fetch/tests
  • /tmp/mcp-fetch-test-venv/bin/python -m ruff check src/fetch/src/mcp_server_fetch/server.py src/fetch/tests/test_server.py
  • git diff --check

@cliffhall

Copy link
Copy Markdown
Member

Thank you for this pull request, and apologies for the long wait for a response.

Since it was opened, the reference servers have been rebuilt for the 2026-07-28 MCP specification (#4857) and released as v2, so this diff no longer applies to the code as it stands. This repository also now accepts issues, not pull requests, from anyone but its maintainers: design and implementation are done by the maintainers through a prompt-driven workflow, so outside PRs are closed rather than reviewed. The policy is in CONTRIBUTING.md.

This is not a judgment on the change or on you. The problem it addresses still applies to v2 and is tracked in #4199, which links back to this PR as a prototype and credits you. Please add anything further there.

This is part of the backlog triage in #4875.

@cliffhall cliffhall closed this Oct 20, 2026
@cliffhall cliffhall mentioned this pull request Oct 20, 2026
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

closed-superseded-v2 Closed: fixed or made obsolete by the v2 / 2026-07-28 refactor (#4875)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mcp-server-fetch hard-codes use_readability=True which requires Node.js (undeclared); fails silently when Node is unavailable or misconfigured

2 participants