refactor: Update pull request function names to use consistent capitalization

aryasoni98 · aryasoni98 · commit 4c3afa28bca9 · 2025-04-09T19:00:10.000+02:00
diff --git a/pkg/github/pullrequests.go b/pkg/github/pullrequests.go
@@ -13,8 +13,8 @@ import (
 	"github.com/mark3labs/mcp-go/server"
 )
 
-// getPullRequest creates a tool to get details of a specific pull request.
-func getPullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// GetPullRequest creates a tool to get details of a specific pull request.
+func GetPullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("get_pull_request",
 			mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_DESCRIPTION", "Get details of a specific pull request")),
 			mcp.WithString("owner",
@@ -67,8 +67,8 @@ func getPullRequest(client *github.Client, t translations.TranslationHelperFunc)
 		}
 }
 
-// listPullRequests creates a tool to list and filter repository pull requests.
-func listPullRequests(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// ListPullRequests creates a tool to list and filter repository pull requests.
+func ListPullRequests(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("list_pull_requests",
 			mcp.WithDescription(t("TOOL_LIST_PULL_REQUESTS_DESCRIPTION", "List and filter repository pull requests")),
 			mcp.WithString("owner",
@@ -165,8 +165,8 @@ func listPullRequests(client *github.Client, t translations.TranslationHelperFun
 		}
 }
 
-// mergePullRequest creates a tool to merge a pull request.
-func mergePullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// MergePullRequest creates a tool to merge a pull request.
+func MergePullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("merge_pull_request",
 			mcp.WithDescription(t("TOOL_MERGE_PULL_REQUEST_DESCRIPTION", "Merge a pull request")),
 			mcp.WithString("owner",
@@ -245,8 +245,8 @@ func mergePullRequest(client *github.Client, t translations.TranslationHelperFun
 		}
 }
 
-// getPullRequestFiles creates a tool to get the list of files changed in a pull request.
-func getPullRequestFiles(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// GetPullRequestFiles creates a tool to get files in a pull request.
+func GetPullRequestFiles(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("get_pull_request_files",
 			mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_FILES_DESCRIPTION", "Get the list of files changed in a pull request")),
 			mcp.WithString("owner",
@@ -300,8 +300,8 @@ func getPullRequestFiles(client *github.Client, t translations.TranslationHelper
 		}
 }
 
-// getPullRequestStatus creates a tool to get the combined status of all status checks for a pull request.
-func getPullRequestStatus(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// GetPullRequestStatus creates a tool to get the status of a pull request.
+func GetPullRequestStatus(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("get_pull_request_status",
 			mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_STATUS_DESCRIPTION", "Get the combined status of all status checks for a pull request")),
 			mcp.WithString("owner",
@@ -369,8 +369,8 @@ func getPullRequestStatus(client *github.Client, t translations.TranslationHelpe
 		}
 }
 
-// updatePullRequestBranch creates a tool to update a pull request branch with the latest changes from the base branch.
-func updatePullRequestBranch(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// UpdatePullRequestBranch creates a tool to update a pull request branch.
+func UpdatePullRequestBranch(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("update_pull_request_branch",
 			mcp.WithDescription(t("TOOL_UPDATE_PULL_REQUEST_BRANCH_DESCRIPTION", "Update a pull request branch with the latest changes from the base branch")),
 			mcp.WithString("owner",
@@ -439,8 +439,8 @@ func updatePullRequestBranch(client *github.Client, t translations.TranslationHe
 		}
 }
 
-// getPullRequestComments creates a tool to get the review comments on a pull request.
-func getPullRequestComments(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// GetPullRequestComments creates a tool to get comments on a pull request.
+func GetPullRequestComments(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("get_pull_request_comments",
 			mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_COMMENTS_DESCRIPTION", "Get the review comments on a pull request")),
 			mcp.WithString("owner",
@@ -499,8 +499,8 @@ func getPullRequestComments(client *github.Client, t translations.TranslationHel
 		}
 }
 
-// getPullRequestReviews creates a tool to get the reviews on a pull request.
-func getPullRequestReviews(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// GetPullRequestReviews creates a tool to get reviews on a pull request.
+func GetPullRequestReviews(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("get_pull_request_reviews",
 			mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_REVIEWS_DESCRIPTION", "Get the reviews on a pull request")),
 			mcp.WithString("owner",
@@ -553,8 +553,8 @@ func getPullRequestReviews(client *github.Client, t translations.TranslationHelp
 		}
 }
 
-// createPullRequestReview creates a tool to submit a review on a pull request.
-func createPullRequestReview(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// CreatePullRequestReview creates a tool to create a review on a pull request.
+func CreatePullRequestReview(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("create_pull_request_review",
 			mcp.WithDescription(t("TOOL_CREATE_PULL_REQUEST_REVIEW_DESCRIPTION", "Create a review on a pull request")),
 			mcp.WithString("owner",
@@ -745,8 +745,8 @@ func createPullRequestReview(client *github.Client, t translations.TranslationHe
 		}
 }
 
-// createPullRequest creates a tool to create a new pull request.
-func createPullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
+// CreatePullRequest creates a tool to create a new pull request.
+func CreatePullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 	return mcp.NewTool("create_pull_request",
 			mcp.WithDescription(t("TOOL_CREATE_PULL_REQUEST_DESCRIPTION", "Create a new pull request in a GitHub repository")),
 			mcp.WithString("owner",

Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@ import (`
`13`	`13`	`"github.com/mark3labs/mcp-go/server"`
`14`	`14`	`)`
`15`	`15`
`16`		`-// getPullRequest creates a tool to get details of a specific pull request.`
`17`		`-func getPullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`16`	`+// GetPullRequest creates a tool to get details of a specific pull request.`
	`17`	`+func GetPullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`18`	`18`	`return mcp.NewTool("get_pull_request",`
`19`	`19`	`mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_DESCRIPTION", "Get details of a specific pull request")),`
`20`	`20`	`mcp.WithString("owner",`
`@@ -67,8 +67,8 @@ func getPullRequest(client *github.Client, t translations.TranslationHelperFunc)`
`67`	`67`	`}`
`68`	`68`	`}`
`69`	`69`
`70`		`-// listPullRequests creates a tool to list and filter repository pull requests.`
`71`		`-func listPullRequests(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`70`	`+// ListPullRequests creates a tool to list and filter repository pull requests.`
	`71`	`+func ListPullRequests(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`72`	`72`	`return mcp.NewTool("list_pull_requests",`
`73`	`73`	`mcp.WithDescription(t("TOOL_LIST_PULL_REQUESTS_DESCRIPTION", "List and filter repository pull requests")),`
`74`	`74`	`mcp.WithString("owner",`
`@@ -165,8 +165,8 @@ func listPullRequests(client *github.Client, t translations.TranslationHelperFun`
`165`	`165`	`}`
`166`	`166`	`}`
`167`	`167`
`168`		`-// mergePullRequest creates a tool to merge a pull request.`
`169`		`-func mergePullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`168`	`+// MergePullRequest creates a tool to merge a pull request.`
	`169`	`+func MergePullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`170`	`170`	`return mcp.NewTool("merge_pull_request",`
`171`	`171`	`mcp.WithDescription(t("TOOL_MERGE_PULL_REQUEST_DESCRIPTION", "Merge a pull request")),`
`172`	`172`	`mcp.WithString("owner",`
`@@ -245,8 +245,8 @@ func mergePullRequest(client *github.Client, t translations.TranslationHelperFun`
`245`	`245`	`}`
`246`	`246`	`}`
`247`	`247`
`248`		`-// getPullRequestFiles creates a tool to get the list of files changed in a pull request.`
`249`		`-func getPullRequestFiles(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`248`	`+// GetPullRequestFiles creates a tool to get files in a pull request.`
	`249`	`+func GetPullRequestFiles(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`250`	`250`	`return mcp.NewTool("get_pull_request_files",`
`251`	`251`	`mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_FILES_DESCRIPTION", "Get the list of files changed in a pull request")),`
`252`	`252`	`mcp.WithString("owner",`
`@@ -300,8 +300,8 @@ func getPullRequestFiles(client *github.Client, t translations.TranslationHelper`
`300`	`300`	`}`
`301`	`301`	`}`
`302`	`302`
`303`		`-// getPullRequestStatus creates a tool to get the combined status of all status checks for a pull request.`
`304`		`-func getPullRequestStatus(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`303`	`+// GetPullRequestStatus creates a tool to get the status of a pull request.`
	`304`	`+func GetPullRequestStatus(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`305`	`305`	`return mcp.NewTool("get_pull_request_status",`
`306`	`306`	`mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_STATUS_DESCRIPTION", "Get the combined status of all status checks for a pull request")),`
`307`	`307`	`mcp.WithString("owner",`
`@@ -369,8 +369,8 @@ func getPullRequestStatus(client *github.Client, t translations.TranslationHelpe`
`369`	`369`	`}`
`370`	`370`	`}`
`371`	`371`
`372`		`-// updatePullRequestBranch creates a tool to update a pull request branch with the latest changes from the base branch.`
`373`		`-func updatePullRequestBranch(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`372`	`+// UpdatePullRequestBranch creates a tool to update a pull request branch.`
	`373`	`+func UpdatePullRequestBranch(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`374`	`374`	`return mcp.NewTool("update_pull_request_branch",`
`375`	`375`	`mcp.WithDescription(t("TOOL_UPDATE_PULL_REQUEST_BRANCH_DESCRIPTION", "Update a pull request branch with the latest changes from the base branch")),`
`376`	`376`	`mcp.WithString("owner",`
`@@ -439,8 +439,8 @@ func updatePullRequestBranch(client *github.Client, t translations.TranslationHe`
`439`	`439`	`}`
`440`	`440`	`}`
`441`	`441`
`442`		`-// getPullRequestComments creates a tool to get the review comments on a pull request.`
`443`		`-func getPullRequestComments(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`442`	`+// GetPullRequestComments creates a tool to get comments on a pull request.`
	`443`	`+func GetPullRequestComments(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`444`	`444`	`return mcp.NewTool("get_pull_request_comments",`
`445`	`445`	`mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_COMMENTS_DESCRIPTION", "Get the review comments on a pull request")),`
`446`	`446`	`mcp.WithString("owner",`
`@@ -499,8 +499,8 @@ func getPullRequestComments(client *github.Client, t translations.TranslationHel`
`499`	`499`	`}`
`500`	`500`	`}`
`501`	`501`
`502`		`-// getPullRequestReviews creates a tool to get the reviews on a pull request.`
`503`		`-func getPullRequestReviews(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`502`	`+// GetPullRequestReviews creates a tool to get reviews on a pull request.`
	`503`	`+func GetPullRequestReviews(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`504`	`504`	`return mcp.NewTool("get_pull_request_reviews",`
`505`	`505`	`mcp.WithDescription(t("TOOL_GET_PULL_REQUEST_REVIEWS_DESCRIPTION", "Get the reviews on a pull request")),`
`506`	`506`	`mcp.WithString("owner",`
`@@ -553,8 +553,8 @@ func getPullRequestReviews(client *github.Client, t translations.TranslationHelp`
`553`	`553`	`}`
`554`	`554`	`}`
`555`	`555`
`556`		`-// createPullRequestReview creates a tool to submit a review on a pull request.`
`557`		`-func createPullRequestReview(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`556`	`+// CreatePullRequestReview creates a tool to create a review on a pull request.`
	`557`	`+func CreatePullRequestReview(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`558`	`558`	`return mcp.NewTool("create_pull_request_review",`
`559`	`559`	`mcp.WithDescription(t("TOOL_CREATE_PULL_REQUEST_REVIEW_DESCRIPTION", "Create a review on a pull request")),`
`560`	`560`	`mcp.WithString("owner",`
`@@ -745,8 +745,8 @@ func createPullRequestReview(client *github.Client, t translations.TranslationHe`
`745`	`745`	`}`
`746`	`746`	`}`
`747`	`747`
`748`		`-// createPullRequest creates a tool to create a new pull request.`
`749`		`-func createPullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
	`748`	`+// CreatePullRequest creates a tool to create a new pull request.`
	`749`	`+func CreatePullRequest(client *github.Client, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {`
`750`	`750`	`return mcp.NewTool("create_pull_request",`
`751`	`751`	`mcp.WithDescription(t("TOOL_CREATE_PULL_REQUEST_DESCRIPTION", "Create a new pull request in a GitHub repository")),`
`752`	`752`	`mcp.WithString("owner",`