Skip to content

Conversation

@YuriNachos
Copy link

Summary

Fixes #1652

The get_markdown function in deploy/docker/server.py was missing a docstring, which caused the MCP tool 'md' to have no description when exposed via _list_tools().

List of files changed and why

  • deploy/docker/server.py - Added docstring to get_markdown function to provide description for MCP tool listing

How Has This Been Tested?

The fix adds a docstring that is properly picked up by inspect.getdoc() in the _list_tools() function. The docstring describes:

  • The tool's purpose (converting web pages to Markdown)
  • Supported extraction modes (fit, raw, bm25, llm)
  • Usage guidance for LLMs and developers

Checklist:

  • My code follows the style guidelines of this project (follows existing docstring format in generate_html)
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (N/A - docstring is documentation)
  • I have added/updated unit tests that prove my fix is effective or that my feature works (N/A - simple docstring addition)
  • New and existing unit tests pass locally with my changes

🤖 Generated with Claude Code

Fixes unclecode#1652

The `get_markdown` function was missing a docstring, which caused the MCP
tool 'md' to have no description when exposed via `_list_tools()`. This
made it difficult for LLMs and developers to understand the tool's purpose.

Added comprehensive docstring describing the tool's functionality and
supported extraction modes.

Co-Authored-By: Claude <noreply@anthropic.com>
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.

[Bug]: MCP tool md has no description due to missing docstring

1 participant