Skip to content

Commit 5878c21

Browse files
SamMorrowDrumsCopilot
SamMorrowDrums
and
Copilot
committed
refactor: reuse existing helpers in granular toolsets
Refactor granular issue and PR tools to delegate to existing tested helper functions instead of reimplementing logic from scratch: - Sub-issue tools (add/remove/reprioritize) now delegate to existing REST-based AddSubIssue, RemoveSubIssue, ReprioritizeSubIssue helpers - PR review tools (create/submit/delete) now delegate to existing CreatePullRequestReview, SubmitPendingPullRequestReview, DeletePendingPullRequestReview helpers (fixes viewer filtering bug) - Review comment tool now uses viewer-safe pattern from AddCommentToPendingReview (query viewer, filter by author, validate PENDING state, pass PullRequestReviewID) - Fix milestone param to use RequiredInt instead of float64 cast - Fix line/startLine params to use OptionalIntParam - Draft state tool uses typed GraphQL inputs matching existing patterns - Remove duplicate GraphQL types and helper functions - Add toolsnap tests for all 20 granular tools - Update generated docs Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent bf18957 commit 5878c21

24 files changed

+1058
-410
lines changed

‎README.md‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -913,7 +913,7 @@ The following sets of tools are available:
913913
- `owner`: Repository owner (username or organization) (string, required)
914914
- `replace_parent`: If true, reparent the sub-issue if it already has a parent (boolean, optional)
915915
- `repo`: Repository name (string, required)
916-
- `sub_issue_id`: The global node ID of the issue to add as a sub-issue (number, required)
916+
- `sub_issue_id`: The ID of the sub-issue to add. ID is not the same as issue number (number, required)
917917

918918
- **create_issue** - Create Issue
919919
- **Required OAuth Scopes**: `repo`
@@ -927,16 +927,16 @@ The following sets of tools are available:
927927
- `issue_number`: The parent issue number (number, required)
928928
- `owner`: Repository owner (username or organization) (string, required)
929929
- `repo`: Repository name (string, required)
930-
- `sub_issue_id`: The global node ID of the sub-issue to remove (number, required)
930+
- `sub_issue_id`: The ID of the sub-issue to remove. ID is not the same as issue number (number, required)
931931

932932
- **reprioritize_sub_issue** - Reprioritize Sub-Issue
933933
- **Required OAuth Scopes**: `repo`
934-
- `after_id`: The global node ID of the sub-issue to place this after (optional) (number, optional)
935-
- `before_id`: The global node ID of the sub-issue to place this before (optional) (number, optional)
934+
- `after_id`: The ID of the sub-issue to place this after (either after_id OR before_id should be specified) (number, optional)
935+
- `before_id`: The ID of the sub-issue to place this before (either after_id OR before_id should be specified) (number, optional)
936936
- `issue_number`: The parent issue number (number, required)
937937
- `owner`: Repository owner (username or organization) (string, required)
938938
- `repo`: Repository name (string, required)
939-
- `sub_issue_id`: The global node ID of the sub-issue to reorder (number, required)
939+
- `sub_issue_id`: The ID of the sub-issue to reorder. ID is not the same as issue number (number, required)
940940

941941
- **update_issue_assignees** - Update Issue Assignees
942942
- **Required OAuth Scopes**: `repo`
@@ -962,7 +962,7 @@ The following sets of tools are available:
962962
- **update_issue_milestone** - Update Issue Milestone
963963
- **Required OAuth Scopes**: `repo`
964964
- `issue_number`: The issue number to update (number, required)
965-
- `milestone`: The milestone number to set on the issue (number, required)
965+
- `milestone`: The milestone number to set on the issue (integer, required)
966966
- `owner`: Repository owner (username or organization) (string, required)
967967
- `repo`: Repository name (string, required)
968968

‎pkg/github/__toolsnaps__/add_pull_request_review_comment.snap‎

Lines changed: 75 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,75 @@
1+
{
2+
"annotations": {
3+
"destructiveHint": false,
4+
"openWorldHint": true,
5+
"title": "Add Pull Request Review Comment"
6+
},
7+
"description": "Add a review comment to the current user's pending pull request review.",
8+
"inputSchema": {
9+
"properties": {
10+
"body": {
11+
"description": "The comment body",
12+
"type": "string"
13+
},
14+
"line": {
15+
"description": "The line number in the diff to comment on (optional)",
16+
"type": "number"
17+
},
18+
"owner": {
19+
"description": "Repository owner (username or organization)",
20+
"type": "string"
21+
},
22+
"path": {
23+
"description": "The relative path of the file to comment on",
24+
"type": "string"
25+
},
26+
"pullNumber": {
27+
"description": "The pull request number",
28+
"minimum": 1,
29+
"type": "number"
30+
},
31+
"repo": {
32+
"description": "Repository name",
33+
"type": "string"
34+
},
35+
"side": {
36+
"description": "The side of the diff to comment on (optional)",
37+
"enum": [
38+
"LEFT",
39+
"RIGHT"
40+
],
41+
"type": "string"
42+
},
43+
"startLine": {
44+
"description": "The start line of a multi-line comment (optional)",
45+
"type": "number"
46+
},
47+
"startSide": {
48+
"description": "The start side of a multi-line comment (optional)",
49+
"enum": [
50+
"LEFT",
51+
"RIGHT"
52+
],
53+
"type": "string"
54+
},
55+
"subjectType": {
56+
"description": "The subject type of the comment",
57+
"enum": [
58+
"FILE",
59+
"LINE"
60+
],
61+
"type": "string"
62+
}
63+
},
64+
"required": [
65+
"owner",
66+
"repo",
67+
"pullNumber",
68+
"path",
69+
"body",
70+
"subjectType"
71+
],
72+
"type": "object"
73+
},
74+
"name": "add_pull_request_review_comment"
75+
}

‎pkg/github/__toolsnaps__/add_sub_issue.snap‎

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
{
2+
"annotations": {
3+
"destructiveHint": false,
4+
"openWorldHint": true,
5+
"title": "Add Sub-Issue"
6+
},
7+
"description": "Add a sub-issue to a parent issue.",
8+
"inputSchema": {
9+
"properties": {
10+
"issue_number": {
11+
"description": "The parent issue number",
12+
"minimum": 1,
13+
"type": "number"
14+
},
15+
"owner": {
16+
"description": "Repository owner (username or organization)",
17+
"type": "string"
18+
},
19+
"replace_parent": {
20+
"description": "If true, reparent the sub-issue if it already has a parent",
21+
"type": "boolean"
22+
},
23+
"repo": {
24+
"description": "Repository name",
25+
"type": "string"
26+
},
27+
"sub_issue_id": {
28+
"description": "The ID of the sub-issue to add. ID is not the same as issue number",
29+
"type": "number"
30+
}
31+
},
32+
"required": [
33+
"owner",
34+
"repo",
35+
"issue_number",
36+
"sub_issue_id"
37+
],
38+
"type": "object"
39+
},
40+
"name": "add_sub_issue"
41+
}

‎pkg/github/__toolsnaps__/create_issue.snap‎

Lines changed: 6 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,18 @@
11
{
22
"annotations": {
3-
"title": "Open new issue",
4-
"readOnlyHint": false
3+
"destructiveHint": false,
4+
"openWorldHint": true,
5+
"title": "Create Issue"
56
},
6-
"description": "Create a new issue in a GitHub repository.",
7+
"description": "Create a new issue in a GitHub repository with a title and optional body.",
78
"inputSchema": {
89
"properties": {
9-
"assignees": {
10-
"description": "Usernames to assign to this issue",
11-
"items": {
12-
"type": "string"
13-
},
14-
"type": "array"
15-
},
1610
"body": {
17-
"description": "Issue body content",
11+
"description": "Issue body content (optional)",
1812
"type": "string"
1913
},
20-
"labels": {
21-
"description": "Labels to apply to this issue",
22-
"items": {
23-
"type": "string"
24-
},
25-
"type": "array"
26-
},
27-
"milestone": {
28-
"description": "Milestone number",
29-
"type": "number"
30-
},
3114
"owner": {
32-
"description": "Repository owner",
15+
"description": "Repository owner (username or organization)",
3316
"type": "string"
3417
},
3518
"repo": {
@@ -39,10 +22,6 @@
3922
"title": {
4023
"description": "Issue title",
4124
"type": "string"
42-
},
43-
"type": {
44-
"description": "Type of this issue",
45-
"type": "string"
4625
}
4726
},
4827
"required": [

‎pkg/github/__toolsnaps__/create_pull_request_review.snap‎

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
{
2+
"annotations": {
3+
"destructiveHint": false,
4+
"openWorldHint": true,
5+
"title": "Create Pull Request Review"
6+
},
7+
"description": "Create a review on a pull request. If event is provided, the review is submitted immediately; otherwise a pending review is created.",
8+
"inputSchema": {
9+
"properties": {
10+
"body": {
11+
"description": "The review body text (optional)",
12+
"type": "string"
13+
},
14+
"commitID": {
15+
"description": "The SHA of the commit to review (optional, defaults to latest)",
16+
"type": "string"
17+
},
18+
"event": {
19+
"description": "The review action to perform. If omitted, creates a pending review.",
20+
"enum": [
21+
"APPROVE",
22+
"REQUEST_CHANGES",
23+
"COMMENT"
24+
],
25+
"type": "string"
26+
},
27+
"owner": {
28+
"description": "Repository owner (username or organization)",
29+
"type": "string"
30+
},
31+
"pullNumber": {
32+
"description": "The pull request number",
33+
"minimum": 1,
34+
"type": "number"
35+
},
36+
"repo": {
37+
"description": "Repository name",
38+
"type": "string"
39+
}
40+
},
41+
"required": [
42+
"owner",
43+
"repo",
44+
"pullNumber"
45+
],
46+
"type": "object"
47+
},
48+
"name": "create_pull_request_review"
49+
}

‎pkg/github/__toolsnaps__/delete_pending_pull_request_review.snap‎

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
{
2+
"annotations": {
3+
"destructiveHint": true,
4+
"openWorldHint": true,
5+
"title": "Delete Pending Pull Request Review"
6+
},
7+
"description": "Delete a pending pull request review.",
8+
"inputSchema": {
9+
"properties": {
10+
"owner": {
11+
"description": "Repository owner (username or organization)",
12+
"type": "string"
13+
},
14+
"pullNumber": {
15+
"description": "The pull request number",
16+
"minimum": 1,
17+
"type": "number"
18+
},
19+
"repo": {
20+
"description": "Repository name",
21+
"type": "string"
22+
}
23+
},
24+
"required": [
25+
"owner",
26+
"repo",
27+
"pullNumber"
28+
],
29+
"type": "object"
30+
},
31+
"name": "delete_pending_pull_request_review"
32+
}

‎pkg/github/__toolsnaps__/remove_sub_issue.snap‎

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
{
2+
"annotations": {
3+
"destructiveHint": true,
4+
"openWorldHint": true,
5+
"title": "Remove Sub-Issue"
6+
},
7+
"description": "Remove a sub-issue from a parent issue.",
8+
"inputSchema": {
9+
"properties": {
10+
"issue_number": {
11+
"description": "The parent issue number",
12+
"minimum": 1,
13+
"type": "number"
14+
},
15+
"owner": {
16+
"description": "Repository owner (username or organization)",
17+
"type": "string"
18+
},
19+
"repo": {
20+
"description": "Repository name",
21+
"type": "string"
22+
},
23+
"sub_issue_id": {
24+
"description": "The ID of the sub-issue to remove. ID is not the same as issue number",
25+
"type": "number"
26+
}
27+
},
28+
"required": [
29+
"owner",
30+
"repo",
31+
"issue_number",
32+
"sub_issue_id"
33+
],
34+
"type": "object"
35+
},
36+
"name": "remove_sub_issue"
37+
}

‎pkg/github/__toolsnaps__/reprioritize_sub_issue.snap‎

Lines changed: 45 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,45 @@
1+
{
2+
"annotations": {
3+
"destructiveHint": false,
4+
"openWorldHint": true,
5+
"title": "Reprioritize Sub-Issue"
6+
},
7+
"description": "Reprioritize (reorder) a sub-issue relative to other sub-issues.",
8+
"inputSchema": {
9+
"properties": {
10+
"after_id": {
11+
"description": "The ID of the sub-issue to place this after (either after_id OR before_id should be specified)",
12+
"type": "number"
13+
},
14+
"before_id": {
15+
"description": "The ID of the sub-issue to place this before (either after_id OR before_id should be specified)",
16+
"type": "number"
17+
},
18+
"issue_number": {
19+
"description": "The parent issue number",
20+
"minimum": 1,
21+
"type": "number"
22+
},
23+
"owner": {
24+
"description": "Repository owner (username or organization)",
25+
"type": "string"
26+
},
27+
"repo": {
28+
"description": "Repository name",
29+
"type": "string"
30+
},
31+
"sub_issue_id": {
32+
"description": "The ID of the sub-issue to reorder. ID is not the same as issue number",
33+
"type": "number"
34+
}
35+
},
36+
"required": [
37+
"owner",
38+
"repo",
39+
"issue_number",
40+
"sub_issue_id"
41+
],
42+
"type": "object"
43+
},
44+
"name": "reprioritize_sub_issue"
45+
}

0 commit comments

Comments
 (0)