Skip to content

feat: Update MCP tool response format, remove urls tool#62

Merged
ka2n merged 4 commits into
mainfrom
feat/mcp-tool-response
May 27, 2025
Merged

feat: Update MCP tool response format, remove urls tool#62
ka2n merged 4 commits into
mainfrom
feat/mcp-tool-response

Conversation

@ka2n

@ka2n ka2n commented May 27, 2025

Copy link
Copy Markdown
Owner
  • test: use embedded resource
  • feat(mcp): remove fetch_library_urls and update fetch_library_docs response format

Copilot AI review requested due to automatic review settings May 27, 2025 07:44
@ka2n ka2n added the enhancement New feature or request label May 27, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes the deprecated "fetch_library_urls" tool and updates the response format for the "fetch_library_docs" tool to use resource objects instead of plain text.

  • Removed the SearchURLs function and its corresponding import and tool registration.
  • Updated the fetch_library_docs response format to return resource objects for various document types (README, documentation, homepage, registry, repository).
  • Removed the "fetch_library_urls" tool reference from the README.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
mcp/tools.go Removed the obsolete fetch_library_urls tool and updated the response format of fetch_library_docs to use mcp.NewToolResultResource.
README.md Removed the reference to the now-deleted fetch_library_urls tool.

@ka2n ka2n requested a review from Copilot May 27, 2025 07:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes the outdated fetch_library_urls tool and enhances the fetch_library_docs tool by updating its response format and improving its descriptions. Key changes include the deletion of the fetch_library_urls functionality in mcp/tools.go, updating the fetch_library_docs tool to use resource-based results, and updating the README.md to remove the obsolete tool reference.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
mcp/tools.go Removed fetch_library_urls and updated fetch_library_docs response format and descriptions.
README.md Removed the reference to the fetch_library_urls tool.

Comment thread mcp/tools.go Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ka2n ka2n enabled auto-merge May 27, 2025 10:23
@ka2n ka2n merged commit 390971c into main May 27, 2025
2 checks passed
@ka2n ka2n deleted the feat/mcp-tool-response branch May 27, 2025 10:24
@github-actions github-actions Bot mentioned this pull request May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants