Skip to content

docs: enrich tool docstrings to provide workflow context#16

Merged
rnovatorov merged 1 commit into
mainfrom
rnovatorov/fix-issue-13
Apr 24, 2026
Merged

docs: enrich tool docstrings to provide workflow context#16
rnovatorov merged 1 commit into
mainfrom
rnovatorov/fix-issue-13

Conversation

@rnovatorov
Copy link
Copy Markdown
Member

Remove global MCP instructions and move guidance directly into tool docstrings. This ensures that clients like Claude Desktop, which ignore metadata-level instructions, still provide the LLM with the necessary context to use the tools effectively.

The docstrings now include 'Tips' and 'Related tools' sections, forming a forward-pointing DAG to guide the LLM through Enapter workflows.

Fixes #13

Remove global MCP instructions and move guidance directly into tool
docstrings. This ensures that clients like Claude Desktop, which ignore
metadata-level instructions, still provide the LLM with the necessary
context to use the tools effectively.

The docstrings now include 'Tips' and 'Related tools' sections, forming
a forward-pointing DAG to guide the LLM through Enapter workflows.

Fixes #13
Copilot AI review requested due to automatic review settings April 24, 2026 10:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR moves workflow guidance from server-level MCP instructions metadata into individual tool docstrings so that MCP clients that ignore metadata-level instructions still receive operational context when calling tools.

Changes:

  • Removed the global INSTRUCTIONS block previously passed to fastmcp.FastMCP.
  • Added richer, workflow-oriented docstrings (with “Tips” and “Related tools”) to the exposed tools: search_sites, search_devices, read_blueprint, search_command_executions, and get_historical_telemetry.
  • Explicitly disables server-level instructions by passing instructions=None when creating the FastMCP server.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rnovatorov rnovatorov merged commit 331214a into main Apr 24, 2026
7 checks passed
@rnovatorov rnovatorov deleted the rnovatorov/fix-issue-13 branch April 24, 2026 10:53
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 Instructions Are Ignored By Some Clients

2 participants