Apr 29, 2025 · Apr 16, 2025 · Apr 29, 2025 · Apr 21, 2025
diff --git a/pkg/github/repositories.go b/pkg/github/repositories.go
 // CreateOrUpdateFile creates a tool to create or update a file in a GitHub repository.
 func CreateOrUpdateFile(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
 return mcp.NewTool("create_or_update_file",
 mcp.WithDescription(t("TOOL_CREATE_OR_UPDATE_FILE_DESCRIPTION", "Create or update a single file in a GitHub repository")),
 mcp.WithDescription(t("TOOL_CREATE_OR_UPDATE_FILE_DESCRIPTION", "Create or update a single file in a GitHub repository. If updating, you must provide the SHA of the file you want to update.")),
 mcp.WithToolAnnotation(mcp.ToolAnnotation{
 Title:        t("TOOL_CREATE_OR_UPDATE_FILE_USER_TITLE", "Create or update file"),
 ReadOnlyHint: false,
Original file line number	Diff line number	Diff line change
Expand Up		@@ -228,7 +228,7 @@ func ListBranches(getClient GetClientFn, t translations.TranslationHelperFunc) (
		// CreateOrUpdateFile creates a tool to create or update a file in a GitHub repository.
		func CreateOrUpdateFile(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) {
		return mcp.NewTool("create_or_update_file",
		mcp.WithDescription(t("TOOL_CREATE_OR_UPDATE_FILE_DESCRIPTION", "Create or update a single file in a GitHub repository")),
		mcp.WithDescription(t("TOOL_CREATE_OR_UPDATE_FILE_DESCRIPTION", "Create or update a single file in a GitHub repository. If updating, you must provide the SHA of the file you want to update.")),
Copy link Preview CopilotAIApr 21, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. The updated tool description indicates that a SHA is required when updating a file, but there is no validation enforcing this constraint. Consider adding a check to ensure that the SHA field is provided when processing an update. Copilot uses AI. Check for mistakes.
		mcp.WithToolAnnotation(mcp.ToolAnnotation{
		Title: t("TOOL_CREATE_OR_UPDATE_FILE_USER_TITLE", "Create or update file"),
		ReadOnlyHint: false,
Expand Down