Skip to content

Commit c64cfdc

Browse files
CopilotSamMorrowDrums
Copilot
and
SamMorrowDrums
committed
Fix test response parsing to handle PaginatedResponse wrapper - partial update
Co-authored-by: SamMorrowDrums <[email protected]>
1 parent fd29e10 commit c64cfdc

22 files changed

+207
-191
lines changed

‎pkg/github/__toolsnaps__/get_commit.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
"description": "Get details for a commit from a GitHub repository",
77
"inputSchema": {
88
"properties": {
9+
"cursor": {
10+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
11+
"type": "string"
12+
},
913
"include_diff": {
1014
"default": true,
1115
"description": "Whether to include file diffs and stats in the response. Default is true.",
@@ -15,17 +19,6 @@
1519
"description": "Repository owner",
1620
"type": "string"
1721
},
18-
"page": {
19-
"description": "Page number for pagination (min 1)",
20-
"minimum": 1,
21-
"type": "number"
22-
},
23-
"perPage": {
24-
"description": "Results per page for pagination (min 1, max 100)",
25-
"maximum": 100,
26-
"minimum": 1,
27-
"type": "number"
28-
},
2922
"repo": {
3023
"description": "Repository name",
3124
"type": "string"

‎pkg/github/__toolsnaps__/issue_read.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
"description": "Get information about a specific issue in a GitHub repository.",
77
"inputSchema": {
88
"properties": {
9+
"cursor": {
10+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
11+
"type": "string"
12+
},
913
"issue_number": {
1014
"description": "The number of the issue",
1115
"type": "number"
@@ -24,17 +28,6 @@
2428
"description": "The owner of the repository",
2529
"type": "string"
2630
},
27-
"page": {
28-
"description": "Page number for pagination (min 1)",
29-
"minimum": 1,
30-
"type": "number"
31-
},
32-
"perPage": {
33-
"description": "Results per page for pagination (min 1, max 100)",
34-
"maximum": 100,
35-
"minimum": 1,
36-
"type": "number"
37-
},
3831
"repo": {
3932
"description": "The name of the repository",
4033
"type": "string"

‎pkg/github/__toolsnaps__/list_branches.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,14 @@
66
"description": "List branches in a GitHub repository",
77
"inputSchema": {
88
"properties": {
9+
"cursor": {
10+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
11+
"type": "string"
12+
},
913
"owner": {
1014
"description": "Repository owner",
1115
"type": "string"
1216
},
13-
"page": {
14-
"description": "Page number for pagination (min 1)",
15-
"minimum": 1,
16-
"type": "number"
17-
},
18-
"perPage": {
19-
"description": "Results per page for pagination (min 1, max 100)",
20-
"maximum": 100,
21-
"minimum": 1,
22-
"type": "number"
23-
},
2417
"repo": {
2518
"description": "Repository name",
2619
"type": "string"

‎pkg/github/__toolsnaps__/list_commits.snap‎

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,28 +3,21 @@
33
"title": "List commits",
44
"readOnlyHint": true
55
},
6-
"description": "Get list of commits of a branch in a GitHub repository. Returns at least 30 results per page by default, but can return more if specified using the perPage parameter (up to 100).",
6+
"description": "Get list of commits of a branch in a GitHub repository. Returns 10 results per page. Use the cursor parameter for pagination.",
77
"inputSchema": {
88
"properties": {
99
"author": {
1010
"description": "Author username or email address to filter commits by",
1111
"type": "string"
1212
},
13+
"cursor": {
14+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
15+
"type": "string"
16+
},
1317
"owner": {
1418
"description": "Repository owner",
1519
"type": "string"
1620
},
17-
"page": {
18-
"description": "Page number for pagination (min 1)",
19-
"minimum": 1,
20-
"type": "number"
21-
},
22-
"perPage": {
23-
"description": "Results per page for pagination (min 1, max 100)",
24-
"maximum": 100,
25-
"minimum": 1,
26-
"type": "number"
27-
},
2821
"repo": {
2922
"description": "Repository name",
3023
"type": "string"

‎pkg/github/__toolsnaps__/list_issues.snap‎

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
"description": "List issues in a GitHub repository. For pagination, use the 'endCursor' from the previous response's 'pageInfo' in the 'after' parameter.",
77
"inputSchema": {
88
"properties": {
9-
"after": {
10-
"description": "Cursor for pagination. Use the endCursor from the previous page's PageInfo for GraphQL APIs.",
9+
"cursor": {
10+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
1111
"type": "string"
1212
},
1313
"direction": {
@@ -38,12 +38,6 @@
3838
"description": "Repository owner",
3939
"type": "string"
4040
},
41-
"perPage": {
42-
"description": "Results per page for pagination (min 1, max 100)",
43-
"maximum": 100,
44-
"minimum": 1,
45-
"type": "number"
46-
},
4741
"repo": {
4842
"description": "Repository name",
4943
"type": "string"

‎pkg/github/__toolsnaps__/list_notifications.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@
1010
"description": "Only show notifications updated before the given time (ISO 8601 format)",
1111
"type": "string"
1212
},
13+
"cursor": {
14+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
15+
"type": "string"
16+
},
1317
"filter": {
1418
"description": "Filter notifications to, use default unless specified. Read notifications are ones that have already been acknowledged by the user. Participating notifications are those that the user is directly involved in, such as issues or pull requests they have commented on or created.",
1519
"enum": [
@@ -23,17 +27,6 @@
2327
"description": "Optional repository owner. If provided with repo, only notifications for this repository are listed.",
2428
"type": "string"
2529
},
26-
"page": {
27-
"description": "Page number for pagination (min 1)",
28-
"minimum": 1,
29-
"type": "number"
30-
},
31-
"perPage": {
32-
"description": "Results per page for pagination (min 1, max 100)",
33-
"maximum": 100,
34-
"minimum": 1,
35-
"type": "number"
36-
},
3730
"repo": {
3831
"description": "Optional repository name. If provided with owner, only notifications for this repository are listed.",
3932
"type": "string"

‎pkg/github/__toolsnaps__/list_pull_requests.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@
1010
"description": "Filter by base branch",
1111
"type": "string"
1212
},
13+
"cursor": {
14+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
15+
"type": "string"
16+
},
1317
"direction": {
1418
"description": "Sort direction",
1519
"enum": [
@@ -26,17 +30,6 @@
2630
"description": "Repository owner",
2731
"type": "string"
2832
},
29-
"page": {
30-
"description": "Page number for pagination (min 1)",
31-
"minimum": 1,
32-
"type": "number"
33-
},
34-
"perPage": {
35-
"description": "Results per page for pagination (min 1, max 100)",
36-
"maximum": 100,
37-
"minimum": 1,
38-
"type": "number"
39-
},
4033
"repo": {
4134
"description": "Repository name",
4235
"type": "string"

‎pkg/github/__toolsnaps__/list_starred_repositories.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
"description": "List starred repositories",
77
"inputSchema": {
88
"properties": {
9+
"cursor": {
10+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
11+
"type": "string"
12+
},
913
"direction": {
1014
"description": "The direction to sort the results by.",
1115
"enum": [
@@ -14,17 +18,6 @@
1418
],
1519
"type": "string"
1620
},
17-
"page": {
18-
"description": "Page number for pagination (min 1)",
19-
"minimum": 1,
20-
"type": "number"
21-
},
22-
"perPage": {
23-
"description": "Results per page for pagination (min 1, max 100)",
24-
"maximum": 100,
25-
"minimum": 1,
26-
"type": "number"
27-
},
2821
"sort": {
2922
"description": "How to sort the results. Can be either 'created' (when the repository was starred) or 'updated' (when the repository was last pushed to).",
3023
"enum": [

‎pkg/github/__toolsnaps__/list_tags.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,14 @@
66
"description": "List git tags in a GitHub repository",
77
"inputSchema": {
88
"properties": {
9+
"cursor": {
10+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
11+
"type": "string"
12+
},
913
"owner": {
1014
"description": "Repository owner",
1115
"type": "string"
1216
},
13-
"page": {
14-
"description": "Page number for pagination (min 1)",
15-
"minimum": 1,
16-
"type": "number"
17-
},
18-
"perPage": {
19-
"description": "Results per page for pagination (min 1, max 100)",
20-
"maximum": 100,
21-
"minimum": 1,
22-
"type": "number"
23-
},
2417
"repo": {
2518
"description": "Repository name",
2619
"type": "string"

‎pkg/github/__toolsnaps__/pull_request_read.snap‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
"description": "Get information on a specific pull request in GitHub repository.",
77
"inputSchema": {
88
"properties": {
9+
"cursor": {
10+
"description": "Pagination cursor. Leave empty for the first page. Use the nextCursor value from the previous response to get the next page.",
11+
"type": "string"
12+
},
913
"method": {
1014
"description": "Action to specify what pull request data needs to be retrieved from GitHub. \nPossible options: \n 1. get - Get details of a specific pull request.\n 2. get_diff - Get the diff of a pull request.\n 3. get_status - Get status of a head commit in a pull request. This reflects status of builds and checks.\n 4. get_files - Get the list of files changed in a pull request. Use with pagination parameters to control the number of results returned.\n 5. get_review_comments - Get the review comments on a pull request. They are comments made on a portion of the unified diff during a pull request review. Use with pagination parameters to control the number of results returned.\n 6. get_reviews - Get the reviews on a pull request. When asked for review comments, use get_review_comments method.\n 7. get_comments - Get comments on a pull request. Use this if user doesn't specifically want review comments. Use with pagination parameters to control the number of results returned.\n",
1115
"enum": [
@@ -23,17 +27,6 @@
2327
"description": "Repository owner",
2428
"type": "string"
2529
},
26-
"page": {
27-
"description": "Page number for pagination (min 1)",
28-
"minimum": 1,
29-
"type": "number"
30-
},
31-
"perPage": {
32-
"description": "Results per page for pagination (min 1, max 100)",
33-
"maximum": 100,
34-
"minimum": 1,
35-
"type": "number"
36-
},
3730
"pullNumber": {
3831
"description": "Pull request number",
3932
"type": "number"

0 commit comments

Comments
 (0)