feat: Add tool annotations for improved LLM tool understanding#2010
Closed
bryankthompson wants to merge 1 commit intoscreenpipe:mainfrom
Closed
feat: Add tool annotations for improved LLM tool understanding#2010bryankthompson wants to merge 1 commit intoscreenpipe:mainfrom
bryankthompson wants to merge 1 commit intoscreenpipe:mainfrom
Conversation
Add readOnlyHint and destructiveHint annotations to all tools to help LLMs better understand tool behavior and make safer decisions. Changes: - Added readOnlyHint: true to read-only tools (search-content, find-elements) - Added destructiveHint: true to tools that modify state (pixel-control, click-element, fill-element, scroll-element, open-application, open-url) - Added title annotations for human-readable display - Bump MCP SDK from unversioned to >=1.8.0 for annotation support This improves tool safety metadata for MCP clients. Co-Authored-By: Claude <[email protected]>
Contributor
🧪 testing bounty created!a testing bounty has been created for this PR: view testing issue testers will be awarded $20 each for providing quality test reports. please check the issue for testing requirements. |
Closed
11 tasks
11 tasks
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds MCP tool annotations (
readOnlyHint,destructiveHint,title) to all 8 tools in the screenpipe MCP server to help LLMs better understand tool behavior and make safer decisions about tool execution.Changes
Added
readOnlyHint: trueto read-only tools:search-content- Search recorded contentfind-elements- Find UI elements (macOS)Added
destructiveHint: trueto tools that interact with the system:pixel-control- Mouse/keyboard controlclick-element- Click UI elements (macOS)fill-element- Type text into elements (macOS)scroll-element- Scroll elements (macOS)open-application- Open applicationsopen-url- Open URLs in browserAdded
titleannotations for human-readable displayBump MCP SDK dependency from unversioned to
>=1.8.0for annotation supportWhy This Matters
readOnlyHinttools can be called freely for information gatheringdestructiveHinttools signal to LLMs that extra caution is warrantedTesting
uv sync)tools/listresponseBefore/After
Before:
After:
🤖 Generated with Claude Code