From 2cece8f979aa4d08dc34077dbf6b8b45c31ea194 Mon Sep 17 00:00:00 2001 From: Sam Morrow Date: Thu, 17 Apr 2025 16:05:09 +0200 Subject: [PATCH 1/3] fix: incorrect env var in README.md --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 5493adba..bf0bcce3 100644 --- a/README.md +++ b/README.md @@ -192,7 +192,7 @@ docker run -i --rm \ ## GitHub Enterprise Server -The flag `--gh-host` and the environment variable `GH_HOST` can be used to set +The flag `--gh-host` and the environment variable `GITHUB_HOST` can be used to set the GitHub Enterprise Server hostname. ## i18n / Overriding Descriptions From 495c0cb428815a63f8c6585625b60a2080973598 Mon Sep 17 00:00:00 2001 From: Simon Taranto Date: Thu, 17 Apr 2025 14:09:38 -0600 Subject: [PATCH 2/3] chore: Update README.md to explain Agent mode will start the server (#305) * Update README.md to explain Agent mode will start the server * Update README.md Co-authored-by: Sam Morrow --------- Co-authored-by: Sam Morrow --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index bf0bcce3..5977763b 100644 --- a/README.md +++ b/README.md @@ -23,7 +23,7 @@ The MCP server can use many of the GitHub APIs, so enable the permissions that y ### Usage with VS Code -For quick installation, use one of the one-click install buttons at the top of this README. +For quick installation, use one of the one-click install buttons at the top of this README. Once you complete that flow, toggle Agent mode (located by the Copilot Chat text input) and the server will start. For manual installation, add the following JSON block to your User Settings (JSON) file in VS Code. You can do this by pressing `Ctrl + Shift + P` and typing `Preferences: Open User Settings (JSON)`. From 9fa582d8d63522d70ce8f3af58265effb9645323 Mon Sep 17 00:00:00 2001 From: Toby Padilla Date: Mon, 21 Apr 2025 13:12:56 -0600 Subject: [PATCH 3/3] fix: use anyOf instead of nullable type array Windsurf is erroring on the `create_pull_request` tool because we're using an array of types to make fields nullable. This PR uses `anyOf` instead and should fix the issue. --- pkg/github/pullrequests.go | 25 ++++++++++++++++++++----- 1 file changed, 20 insertions(+), 5 deletions(-) diff --git a/pkg/github/pullrequests.go b/pkg/github/pullrequests.go index b1584c0e..1ecd209e 100644 --- a/pkg/github/pullrequests.go +++ b/pkg/github/pullrequests.go @@ -915,23 +915,38 @@ func CreatePullRequestReview(getClient GetClientFn, t translations.TranslationHe "description": "path to the file", }, "position": map[string]interface{}{ - "type": []string{"number", "null"}, + "anyOf": []interface{}{ + map[string]string{"type": "number"}, + map[string]string{"type": "null"}, + }, "description": "position of the comment in the diff", }, "line": map[string]interface{}{ - "type": []string{"number", "null"}, + "anyOf": []interface{}{ + map[string]string{"type": "number"}, + map[string]string{"type": "null"}, + }, "description": "line number in the file to comment on. For multi-line comments, the end of the line range", }, "side": map[string]interface{}{ - "type": []string{"string", "null"}, + "anyOf": []interface{}{ + map[string]string{"type": "string"}, + map[string]string{"type": "null"}, + }, "description": "The side of the diff on which the line resides. For multi-line comments, this is the side for the end of the line range. (LEFT or RIGHT)", }, "start_line": map[string]interface{}{ - "type": []string{"number", "null"}, + "anyOf": []interface{}{ + map[string]string{"type": "number"}, + map[string]string{"type": "null"}, + }, "description": "The first line of the range to which the comment refers. Required for multi-line comments.", }, "start_side": map[string]interface{}{ - "type": []string{"string", "null"}, + "anyOf": []interface{}{ + map[string]string{"type": "string"}, + map[string]string{"type": "null"}, + }, "description": "The side of the diff on which the start line resides for multi-line comments. (LEFT or RIGHT)", }, "body": map[string]interface{}{