Skip to content

Remove non-existent get_line_number_in_pull_request_file tool reference #436

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TheoBrigitte
Copy link

Closes: #435

The add_pull_request_review_comment_to_pending_review tool mention the get_line_number_in_pull_request_file tool to be used to get line number. But this tool does not exists.

mcp.WithDescription(t("TOOL_ADD_PULL_REQUEST_REVIEW_COMMENT_TO_PENDING_REVIEW_DESCRIPTION", "Add a comment to the requester's latest pending pull request review, a pending review needs to already exist to call this (check with the user if not sure). If you are using the LINE subjectType, use the get_line_number_in_pull_request_file tool to get an exact line number before commenting.")),

Image

@Copilot Copilot AI review requested due to automatic review settings May 25, 2025 18:57
@TheoBrigitte TheoBrigitte requested a review from a team as a code owner May 25, 2025 18:57
Copy link
Contributor

@Copilot 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

Removes a reference to the non-existent get_line_number_in_pull_request_file tool from the pull request review comment helper and reorders the translations import into the internal imports block.

  • Deleted the misleading tool recommendation from the description of add_pull_request_review_comment_to_pending_review
  • Moved the translations import to the internal imports section for clarity

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.

Non existent get_line_number_in_pull_request_file tool
1 participant