Skip to content

Commit 3778e3e

Browse files
committed
remove unnecessary mcp tool args
1 parent 6bf3367 commit 3778e3e

File tree

2 files changed

+4
-28
lines changed

2 files changed

+4
-28
lines changed

config.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,14 +78,14 @@ amp:
7878
instructions:
7979
- "Use this tool to leave a general review comment on the pull request"
8080
- "This will post your comment as the top level comment on a review"
81-
- "Required: message, owner, repo, pr_number"
81+
- "Required: message"
8282
- "Example: 'The changes look good, but there are a few issues that need to be addressed'"
8383
- name: leave_inline_comment
8484
description: Leave inline comments on specific lines in pull requests
8585
instructions:
8686
- "Use this tool to leave comments on specific lines of code"
8787
- "This will post your comment directly on the line in the diff view via PR review"
88-
- "Required: message, owner, repo, pr_number, path, line"
88+
- "Required: message, path, line"
8989
- "Optional: suggested_fix - provide exact replacement code for obvious/trivial fixes only"
9090
- "For suggested_fix: only include the raw code replacement, no markdown formatting or explanations"
9191
- "Use suggested_fix for simple fixes like typos, missing semicolons, variable name corrections, etc."

src/mcp/server.ts

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -60,20 +60,8 @@ class GitHubMCPServer {
6060
type: 'string',
6161
description: 'The comment message',
6262
},
63-
owner: {
64-
type: 'string',
65-
description: 'Repository owner',
66-
},
67-
repo: {
68-
type: 'string',
69-
description: 'Repository name',
70-
},
71-
pr_number: {
72-
type: 'number',
73-
description: 'Pull request number',
74-
},
7563
},
76-
required: ['message', 'owner', 'repo', 'pr_number'],
64+
required: ['message'],
7765
},
7866
},
7967
{
@@ -86,18 +74,6 @@ class GitHubMCPServer {
8674
type: 'string',
8775
description: 'The comment message explaining the issue or feedback',
8876
},
89-
owner: {
90-
type: 'string',
91-
description: 'Repository owner',
92-
},
93-
repo: {
94-
type: 'string',
95-
description: 'Repository name',
96-
},
97-
pr_number: {
98-
type: 'number',
99-
description: 'Pull request number',
100-
},
10177
path: {
10278
type: 'string',
10379
description: 'File path for the inline comment',
@@ -111,7 +87,7 @@ class GitHubMCPServer {
11187
description: 'Optional code suggestion to replace the current line(s). Only provide this for obvious/trivial fixes. The suggestion should be the exact replacement code without any markdown formatting.',
11288
},
11389
},
114-
required: ['message', 'owner', 'repo', 'pr_number', 'path', 'line'],
90+
required: ['message', 'path', 'line'],
11591
},
11692
},
11793
{

0 commit comments

Comments
 (0)