Skip to content

Commit d62f05f

Browse files
SamMorrowDrumsCopilot
SamMorrowDrums
and
Copilot
authored
refactor(pull-requests): migrate consolidated PR tools to typed inputs and outputs
Convert pull_request_read, create_pull_request, update_pull_request, merge_pull_request, update_pull_request_branch, pull_request_review_write (both feature variants), add_comment_to_pending_review and add_reply_to_pull_request_comment to NewTool with concrete inputs and output DTOs. Input normalizers replay legacy handler checks so legacy coercions and error text are preserved. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 389fc36 commit d62f05f

20 files changed

Lines changed: 4131 additions & 150 deletions

‎pkg/github/__toolsnaps__/add_comment_to_pending_review.snap‎

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,5 +70,17 @@
7070
],
7171
"type": "object"
7272
},
73-
"name": "add_comment_to_pending_review"
73+
"name": "add_comment_to_pending_review",
74+
"outputSchema": {
75+
"additionalProperties": false,
76+
"properties": {
77+
"message": {
78+
"type": "string"
79+
}
80+
},
81+
"required": [
82+
"message"
83+
],
84+
"type": "object"
85+
}
7486
}

‎pkg/github/__toolsnaps__/add_comment_to_pending_review_typed.snap‎

Lines changed: 100 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,100 @@
1+
{
2+
"annotations": {
3+
"idempotentHint": false,
4+
"readOnlyHint": false,
5+
"title": "Add review comment to the requester's latest pending pull request review"
6+
},
7+
"description": "Add review 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).",
8+
"icons": [
9+
{
10+
"mimeType": "image/png",
11+
"src": "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABgAAAAYCAYAAADgdz34AAAABmJLR0QA/wD/AP+gvaeTAAACwUlEQVRIie2Vz28UZRjHP993pi0QIC3YahNjirtmd3bS3Q1eUHvQEPUiEv8A4kXjwRJ78MCFBLjBBRKCHowHE38cNCbGGx6IUoKiodtNpoNmTJp4oSJNQ3pw29l5POxus2wo3QTwxPc0887zfD7zvu9kXnjEUfdNrjj5vJOmMP4e9JrfR1G02tuQD8tvgpck0dxCPwK30ViqnJTcr4bOmfRlI/PrhUJ5313woDpDpu8ss7f6nYHrvDnGcYlPsoY/bKaXwHY3HWfvgmNnMX0zvMM7069A3c3pkEYWa7UVgFxQPSfs7SSeH3k2rEy5jMubMBoG1yQ+SBbm53of+gCybMkk/H8VAFdbZisZLAFsJ11oyL+BUURcwrjWAZixXeIwxs/5UuVAr0QAYRjubGR+HWy3mb6QCIBXQe8nce0jgIkwfMo3/xLG085x8I9ofkMyUa0O+w2rgS0mcf3lboEDiKJo1cvsIDDr4D1DhTb8407hYhTdTJW+AvrdMnuhG9Je1m9BBzbfjXbyQcXypeqJLQt7+0rVE/mgYr3j7l7FDzOPBY8FDx6vc1EolPeNjI5/Jpgw7Lm9o+Pry//c/K0PhnLFyrSMDxE79jwxvn9079gvt28vrUD7V1EoFHalbltd2C7DfS4sAF4DTSdx7cL96LliZVriPPADuL+geRh0Z8il5SiKVn2ATENvCCYw78U/b8xdBcgHlYuGHQXuK5A4ClxM4vnXW8Lqp5JdWWt6h4CvWnsgxgDSbRZ3Gg0tCJ7sY4nGwDZOt/WBZtzN9FswLgM2sGann5mcPDaw5pWEHQH7cUu86SdkR3LF6tfrA814MNVpwNrM1leUxPXrSKfMeHcwdctyNotY8c3NbMX3LJsB3ZHsymDqlkHvYHYyievXWxPpSj4o75eYIuPWZof+vRKG4c61pncIx6gZsx34/5L/ACy3ElqUYhuvAAAAAElFTkSuQmCC",
12+
"theme": "light"
13+
},
14+
{
15+
"mimeType": "image/png",
16+
"src": "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABgAAAAYCAYAAADgdz34AAAABmJLR0QA/wD/AP+gvaeTAAABn0lEQVRIie2Vvy5EURDGvyMUkhUU/pS2Q2clwvIKqL0Cm9ArrBcQShJPoLFR2YaIhIjQ7a7E39IiCgqVn8KsPY697E3oTHPumTPf990z986M9Mfm/A0wKGlMUlnSlnPuOQQAE5LOnXOFWErAIvBK1S6BZBAzZ2fzcckHjXwVaAXSwD2wWYN8A2iKK1ABt3m+ZeDRnseIthdgDxioxd1o662tfZIO7Lnf8xcklST1StqRdORxNEualHQIDDvnTmvdIGE5vwdWgLy93bQX0w0UgSdgKMC3AdfA7ndpSgKbduUbYBoI/7Ju4BiYrYFfAl4iBbxAgOyPgV9xWYDQ3xCXKK79C/wL/KJZoeW8QpsJCy0C54AMULaGmQu7sIAW4MpaxTKwbQU3U4dAxmLzwLpxXAIJP2jKgkY8Xx4o1SFwBmx7+7RxTUnVb9Bpa9HDFiR1/SRgWH+6FT3/h2qK6sBpB0aBB7yB880NcpaWtGHXjCsVBmb5PDIvgJ46BJKW84q9AguV87Adp/Q+9O8UMfQjRBKSxiV1SNp3zp3Ug/sVewPruexhKwhGXQAAAABJRU5ErkJggg==",
17+
"theme": "dark"
18+
}
19+
],
20+
"inputSchema": {
21+
"properties": {
22+
"body": {
23+
"description": "The text of the review comment",
24+
"type": "string"
25+
},
26+
"line": {
27+
"description": "The line of the blob in the pull request diff that the comment applies to. For multi-line comments, the last line of the range",
28+
"type": "number"
29+
},
30+
"owner": {
31+
"description": "Repository owner",
32+
"type": "string",
33+
"x-mcp-header": "owner"
34+
},
35+
"path": {
36+
"description": "The relative path to the file that necessitates a comment",
37+
"type": "string"
38+
},
39+
"pullNumber": {
40+
"description": "Pull request number",
41+
"type": "number"
42+
},
43+
"repo": {
44+
"description": "Repository name",
45+
"type": "string",
46+
"x-mcp-header": "repo"
47+
},
48+
"side": {
49+
"description": "The side of the diff to comment on. LEFT indicates the previous state, RIGHT indicates the new state",
50+
"enum": [
51+
"LEFT",
52+
"RIGHT"
53+
],
54+
"type": "string"
55+
},
56+
"startLine": {
57+
"description": "For multi-line comments, the first line of the range that the comment applies to",
58+
"type": "number"
59+
},
60+
"startSide": {
61+
"description": "For multi-line comments, the starting side of the diff that the comment applies to. LEFT indicates the previous state, RIGHT indicates the new state",
62+
"enum": [
63+
"LEFT",
64+
"RIGHT"
65+
],
66+
"type": "string"
67+
},
68+
"subjectType": {
69+
"description": "The level at which the comment is targeted",
70+
"enum": [
71+
"FILE",
72+
"LINE"
73+
],
74+
"type": "string"
75+
}
76+
},
77+
"required": [
78+
"owner",
79+
"repo",
80+
"pullNumber",
81+
"path",
82+
"body",
83+
"subjectType"
84+
],
85+
"type": "object"
86+
},
87+
"name": "add_comment_to_pending_review",
88+
"outputSchema": {
89+
"additionalProperties": false,
90+
"properties": {
91+
"message": {
92+
"type": "string"
93+
}
94+
},
95+
"required": [
96+
"message"
97+
],
98+
"type": "object"
99+
}
100+
}

‎pkg/github/__toolsnaps__/add_reply_to_pull_request_comment.snap‎

Lines changed: 63 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,5 +50,67 @@
5050
],
5151
"type": "object"
5252
},
53-
"name": "add_reply_to_pull_request_comment"
53+
"name": "add_reply_to_pull_request_comment",
54+
"outputSchema": {
55+
"oneOf": [
56+
{
57+
"additionalProperties": false,
58+
"properties": {
59+
"id": {
60+
"type": "string"
61+
},
62+
"url": {
63+
"type": "string"
64+
}
65+
},
66+
"required": [
67+
"id",
68+
"url"
69+
],
70+
"type": "object"
71+
},
72+
{
73+
"additionalProperties": false,
74+
"properties": {
75+
"comment": {
76+
"additionalProperties": false,
77+
"properties": {
78+
"id": {
79+
"type": "string"
80+
},
81+
"url": {
82+
"type": "string"
83+
}
84+
},
85+
"required": [
86+
"id",
87+
"url"
88+
],
89+
"type": "object"
90+
},
91+
"reaction": {
92+
"additionalProperties": false,
93+
"properties": {
94+
"id": {
95+
"type": "string"
96+
},
97+
"url": {
98+
"type": "string"
99+
}
100+
},
101+
"required": [
102+
"id",
103+
"url"
104+
],
105+
"type": "object"
106+
}
107+
},
108+
"required": [
109+
"comment",
110+
"reaction"
111+
],
112+
"type": "object"
113+
}
114+
]
115+
}
54116
}

‎pkg/github/__toolsnaps__/add_reply_to_pull_request_comment_typed.snap‎

Lines changed: 130 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,130 @@
1+
{
2+
"annotations": {
3+
"idempotentHint": false,
4+
"readOnlyHint": false,
5+
"title": "Add reply to pull request comment"
6+
},
7+
"description": "Add a reply and/or reaction to an existing pull request comment. This can create a new comment linked as a reply to the specified comment, add an emoji reaction to the specified comment, or do both. At least one of body or reaction is required.",
8+
"icons": [
9+
{
10+
"mimeType": "image/png",
11+
"src": "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABgAAAAYCAYAAADgdz34AAAABmJLR0QA/wD/AP+gvaeTAAACwUlEQVRIie2Vz28UZRjHP993pi0QIC3YahNjirtmd3bS3Q1eUHvQEPUiEv8A4kXjwRJ78MCFBLjBBRKCHowHE38cNCbGGx6IUoKiodtNpoNmTJp4oSJNQ3pw29l5POxus2wo3QTwxPc0887zfD7zvu9kXnjEUfdNrjj5vJOmMP4e9JrfR1G02tuQD8tvgpck0dxCPwK30ViqnJTcr4bOmfRlI/PrhUJ5313woDpDpu8ss7f6nYHrvDnGcYlPsoY/bKaXwHY3HWfvgmNnMX0zvMM7069A3c3pkEYWa7UVgFxQPSfs7SSeH3k2rEy5jMubMBoG1yQ+SBbm53of+gCybMkk/H8VAFdbZisZLAFsJ11oyL+BUURcwrjWAZixXeIwxs/5UuVAr0QAYRjubGR+HWy3mb6QCIBXQe8nce0jgIkwfMo3/xLG085x8I9ofkMyUa0O+w2rgS0mcf3lboEDiKJo1cvsIDDr4D1DhTb8407hYhTdTJW+AvrdMnuhG9Je1m9BBzbfjXbyQcXypeqJLQt7+0rVE/mgYr3j7l7FDzOPBY8FDx6vc1EolPeNjI5/Jpgw7Lm9o+Pry//c/K0PhnLFyrSMDxE79jwxvn9079gvt28vrUD7V1EoFHalbltd2C7DfS4sAF4DTSdx7cL96LliZVriPPADuL+geRh0Z8il5SiKVn2ATENvCCYw78U/b8xdBcgHlYuGHQXuK5A4ClxM4vnXW8Lqp5JdWWt6h4CvWnsgxgDSbRZ3Gg0tCJ7sY4nGwDZOt/WBZtzN9FswLgM2sGann5mcPDaw5pWEHQH7cUu86SdkR3LF6tfrA814MNVpwNrM1leUxPXrSKfMeHcwdctyNotY8c3NbMX3LJsB3ZHsymDqlkHvYHYyievXWxPpSj4o75eYIuPWZof+vRKG4c61pncIx6gZsx34/5L/ACy3ElqUYhuvAAAAAElFTkSuQmCC",
12+
"theme": "light"
13+
},
14+
{
15+
"mimeType": "image/png",
16+
"src": "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABgAAAAYCAYAAADgdz34AAAABmJLR0QA/wD/AP+gvaeTAAABn0lEQVRIie2Vvy5EURDGvyMUkhUU/pS2Q2clwvIKqL0Cm9ArrBcQShJPoLFR2YaIhIjQ7a7E39IiCgqVn8KsPY697E3oTHPumTPf990z986M9Mfm/A0wKGlMUlnSlnPuOQQAE5LOnXOFWErAIvBK1S6BZBAzZ2fzcckHjXwVaAXSwD2wWYN8A2iKK1ABt3m+ZeDRnseIthdgDxioxd1o662tfZIO7Lnf8xcklST1StqRdORxNEualHQIDDvnTmvdIGE5vwdWgLy93bQX0w0UgSdgKMC3AdfA7ndpSgKbduUbYBoI/7Ju4BiYrYFfAl4iBbxAgOyPgV9xWYDQ3xCXKK79C/wL/KJZoeW8QpsJCy0C54AMULaGmQu7sIAW4MpaxTKwbQU3U4dAxmLzwLpxXAIJP2jKgkY8Xx4o1SFwBmx7+7RxTUnVb9Bpa9HDFiR1/SRgWH+6FT3/h2qK6sBpB0aBB7yB880NcpaWtGHXjCsVBmb5PDIvgJ46BJKW84q9AguV87Adp/Q+9O8UMfQjRBKSxiV1SNp3zp3Ug/sVewPruexhKwhGXQAAAABJRU5ErkJggg==",
17+
"theme": "dark"
18+
}
19+
],
20+
"inputSchema": {
21+
"properties": {
22+
"body": {
23+
"description": "The text of the reply. Required unless reaction is provided.",
24+
"type": "string"
25+
},
26+
"commentId": {
27+
"description": "The numeric ID of the pull request review comment to reply or react to. Use the number from a #discussion_r... anchor, not the GraphQL thread node ID (PRRT_...).",
28+
"minimum": 1,
29+
"type": "number"
30+
},
31+
"owner": {
32+
"description": "Repository owner",
33+
"type": "string",
34+
"x-mcp-header": "owner"
35+
},
36+
"pullNumber": {
37+
"description": "Pull request number. Required when body is provided.",
38+
"type": "number"
39+
},
40+
"reaction": {
41+
"description": "Emoji reaction to add. Required unless body is provided.",
42+
"enum": [
43+
"+1",
44+
"-1",
45+
"laugh",
46+
"confused",
47+
"heart",
48+
"hooray",
49+
"rocket",
50+
"eyes"
51+
],
52+
"type": "string"
53+
},
54+
"repo": {
55+
"description": "Repository name",
56+
"type": "string",
57+
"x-mcp-header": "repo"
58+
}
59+
},
60+
"required": [
61+
"owner",
62+
"repo",
63+
"commentId"
64+
],
65+
"type": "object"
66+
},
67+
"name": "add_reply_to_pull_request_comment",
68+
"outputSchema": {
69+
"oneOf": [
70+
{
71+
"additionalProperties": false,
72+
"properties": {
73+
"id": {
74+
"type": "string"
75+
},
76+
"url": {
77+
"type": "string"
78+
}
79+
},
80+
"required": [
81+
"id",
82+
"url"
83+
],
84+
"type": "object"
85+
},
86+
{
87+
"additionalProperties": false,
88+
"properties": {
89+
"comment": {
90+
"additionalProperties": false,
91+
"properties": {
92+
"id": {
93+
"type": "string"
94+
},
95+
"url": {
96+
"type": "string"
97+
}
98+
},
99+
"required": [
100+
"id",
101+
"url"
102+
],
103+
"type": "object"
104+
},
105+
"reaction": {
106+
"additionalProperties": false,
107+
"properties": {
108+
"id": {
109+
"type": "string"
110+
},
111+
"url": {
112+
"type": "string"
113+
}
114+
},
115+
"required": [
116+
"id",
117+
"url"
118+
],
119+
"type": "object"
120+
}
121+
},
122+
"required": [
123+
"comment",
124+
"reaction"
125+
],
126+
"type": "object"
127+
}
128+
]
129+
}
130+
}

‎pkg/github/__toolsnaps__/create_pull_request.snap‎

Lines changed: 40 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,5 +65,44 @@
6565
],
6666
"type": "object"
6767
},
68-
"name": "create_pull_request"
68+
"name": "create_pull_request",
69+
"outputSchema": {
70+
"oneOf": [
71+
{
72+
"additionalProperties": false,
73+
"properties": {
74+
"id": {
75+
"type": "string"
76+
},
77+
"url": {
78+
"type": "string"
79+
}
80+
},
81+
"required": [
82+
"id",
83+
"url"
84+
],
85+
"type": "object"
86+
},
87+
{
88+
"additionalProperties": false,
89+
"properties": {
90+
"reason": {
91+
"type": "string"
92+
},
93+
"status": {
94+
"enum": [
95+
"awaiting_user_submission"
96+
],
97+
"type": "string"
98+
}
99+
},
100+
"required": [
101+
"status",
102+
"reason"
103+
],
104+
"type": "object"
105+
}
106+
]
107+
}
69108
}

0 commit comments

Comments
 (0)