Skip to content

Conversation

@TheoBrigitte
Copy link
Contributor

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

@williammartin williammartin merged commit 414309c into github:main May 26, 2025
9 checks passed
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

2 participants