Skip to content

feat: Add tool for discussion comment write operations #2427

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kerobbi merged 12 commits into from
May 15, 2026
Merged

feat: Add tool for discussion comment write operations #2427

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6411325
Add discussion comment write operation tools
RossTarrant May 5, 2026
192ca0b
Address comments from Copilot review
RossTarrant May 5, 2026
cbd22dd
Update includeReplies description to specify GitHub API maximum repli…
RossTarrant May 5, 2026
f2a9baf
Consolidate into single tool
RossTarrant May 8, 2026
b4aedb5
add tests cases for checking param presence
RossTarrant May 8, 2026
6b6bc17
Enhance validation on discussion comment operations
RossTarrant May 8, 2026
a1e3b45
Enhance discussion_write tool description
RossTarrant May 13, 2026
1a75379
Remove redundant param
RossTarrant May 13, 2026
91b179b
Refactor tests
RossTarrant May 13, 2026
f5c918b
Merge branch 'rosstarrant/add-discussion-comment-write-ops' of github…
RossTarrant May 13, 2026
0a37d35
Fix failing build
RossTarrant May 13, 2026
4a488cc
Merge branch 'main' into rosstarrant/add-discussion-comment-write-ops
RossTarrant May 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update includeReplies description to specify GitHub API maximum repli… 
…es limit
  • Loading branch information
@RossTarrant
RossTarrant committed May 5, 2026
commit cbd22dd93548ef3c1c80cac92201a28009f71dbb
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -752,7 +752,7 @@ The following sets of tools are available:
- **Required OAuth Scopes**: `repo`
- `after`: Cursor for pagination. Use the endCursor from the previous page's PageInfo for GraphQL APIs. (string, optional)
- `discussionNumber`: Discussion Number (number, required)
- `includeReplies`: When true, each top-level comment will include its replies nested within it. Defaults to false. (boolean, optional)
- `includeReplies`: When true, each top-level comment will include its replies nested within it (up to 100 replies per comment, which is the GitHub API maximum). Defaults to false. (boolean, optional)
- `owner`: Repository owner (string, required)
- `perPage`: Results per page for pagination (min 1, max 100) (number, optional)
- `repo`: Repository name (string, required)
Expand Down
2 changes: 1 addition & 1 deletion pkg/github/__toolsnaps__/get_discussion_comments.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
"type": "number"
},
"includeReplies": {
"description": "When true, each top-level comment will include its replies nested within it. Defaults to false.",
"description": "When true, each top-level comment will include its replies nested within it (up to 100 replies per comment, which is the GitHub API maximum). Defaults to false.",
"type": "boolean"
},
"owner": {
Expand Down
2 changes: 1 addition & 1 deletion pkg/github/discussions.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -408,7 +408,7 @@ func GetDiscussionComments(t translations.TranslationHelperFunc) inventory.Serve
},
"includeReplies": {
Type: "boolean",
Description: "When true, each top-level comment will include its replies nested within it. Defaults to false.",
Description: "When true, each top-level comment will include its replies nested within it (up to 100 replies per comment, which is the GitHub API maximum). Defaults to false.",
},
},
Required: []string{"owner", "repo", "discussionNumber"},
Expand Down
Loading