Skip to content

fix(fetch): fall back without readability js#4200

Open
he-yufeng wants to merge 1 commit into
modelcontextprotocol:mainfrom
he-yufeng:fix/fetch-readability-node-fallback
Open

fix(fetch): fall back without readability js#4200
he-yufeng wants to merge 1 commit into
modelcontextprotocol:mainfrom
he-yufeng:fix/fetch-readability-node-fallback

Conversation

@he-yufeng
Copy link
Copy Markdown

Summary:

  • avoid the optional Readability.js path when Node.js is not available
  • add a --no-readability-js switch for environments where the Node.js simplifier is installed but problematic
  • document the fallback behavior and cover both default and opt-out paths in fetch tests

Fixes #4199.

To verify:

  • uv run pytest tests -q
  • uv run ruff check src tests
  • uv run pyright src
  • git diff --check

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.

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

1 participant