PR review tool #59
Labels
No labels
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/OpenSpec
Kind/Security
Kind/Testing
Priority/Critical
Priority/High
Priority/Low
Priority/Medium
RFC - Request For Comments
Reviewed/Confirmed
Reviewed/Duplicate
Reviewed/Invalid
Reviewed/Won't Fix
Status/Abandoned
Status/Blocked
Status/Need More Info
hermes-attempted
hermes-needs-clarification
hermes-ready
hermes-review
hermes-wip
human-required
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
agentic-forges/forgejo-mcp#59
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
HI, thanks for taking this on!
I think that this repo was forked for the gitea MCP server. There are (a lot) of new tools on the gitea repo, would it be possible to import them to forgejo MCP server? Especially the latest commit with the PR review :)?
Hi @MalcolmMielle, thanks for this request!
I've started working on this. Here's what I'm planning to add — 6 new write-side PR review tools to complement the existing read-only ones (
list_pull_reviews,get_pull_review,list_pull_review_comments):create_pull_reviewsubmit_pull_reviewdismiss_pull_reviewdelete_pull_reviewcreate_review_requestsdelete_review_requestsAll of these are already supported by the Forgejo SDK — this is about exposing them as MCP tools.
Is this what you had in mind? Are there any other tools from the upstream Gitea MCP server you'd particularly like to see prioritized? Happy to adjust scope based on your feedback.
Thanks for your quick answer!
I think the one I was looking at was
create_pull_request_review. If I understand correctly, it's about sending PR review with per line comments?I think the one your proposed also look great
Implementation is ready on
main(commit578643c). Here are the 6 new tools:create_pull_reviewsubmit_pull_reviewdismiss_pull_reviewdelete_pull_reviewcreate_review_requestsdelete_review_requestsHow to test
The easiest way is with the MCP Inspector:
This gives you a web UI to call each tool individually with parameters and inspect responses.
Test walkthrough
You'll need a Forgejo instance with a repo that has at least one open PR.
1. Create a review with inline comments (the key use case from your request):
Verify the review appears in the PR timeline and the inline comment shows on the correct line in the diff view.
2. Approve a PR:
3. Request a review from someone:
4. Dismiss a review:
@MalcolmMielle — would you be able to give these a spin and let us know if they work for your use case? Especially the inline comments feature (test 1 above) since that's what you were most interested in. Any feedback is welcome!
Super! I'll give it a try and come back to you if I have questions/comments