Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commite9f748f

Browse files
SamMorrowDrumswilliammartin
authored andcommitted
Add additional tools and tests for notifications
1 parent5771908 commite9f748f

File tree

7 files changed

+1060
-121
lines changed

7 files changed

+1060
-121
lines changed

‎README.md

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -581,6 +581,39 @@ export GITHUB_MCP_TOOL_ADD_ISSUE_COMMENT_DESCRIPTION="an alternative description
581581
-`secret_type`: The secret types to be filtered for in a comma-separated list (string, optional)
582582
-`resolution`: The resolution status (string, optional)
583583

584+
###Notifications
585+
586+
-**list_notifications** – List notifications for a GitHub user
587+
-`filter`: Filter to apply to the response (`default`,`include_read_notifications`,`only_participating`)
588+
-`since`: Only show notifications updated after the given time (ISO 8601 format)
589+
-`before`: Only show notifications updated before the given time (ISO 8601 format)
590+
-`owner`: Optional repository owner (string)
591+
-`repo`: Optional repository name (string)
592+
-`page`: Page number (number, optional)
593+
-`perPage`: Results per page (number, optional)
594+
595+
596+
-**get_notification_details** – Get detailed information for a specific GitHub notification
597+
-`notificationID`: The ID of the notification (string, required)
598+
599+
-**dismiss_notification** – Dismiss a notification by marking it as read or done
600+
-`threadID`: The ID of the notification thread (string, required)
601+
-`state`: The new state of the notification (`read` or`done`)
602+
603+
-**mark_all_notifications_read** – Mark all notifications as read
604+
-`lastReadAt`: Describes the last point that notifications were checked (optional, RFC3339/ISO8601 string, default: now)
605+
-`owner`: Optional repository owner (string)
606+
-`repo`: Optional repository name (string)
607+
608+
-**manage_notification_subscription** – Manage a notification subscription (ignore, watch, or delete) for a notification thread
609+
-`notificationID`: The ID of the notification thread (string, required)
610+
-`action`: Action to perform:`ignore`,`watch`, or`delete` (string, required)
611+
612+
-**manage_repository_notification_subscription** – Manage a repository notification subscription (ignore, watch, or delete)
613+
-`owner`: The account owner of the repository (string, required)
614+
-`repo`: The name of the repository (string, required)
615+
-`action`: Action to perform:`ignore`,`watch`, or`delete` (string, required)
616+
584617
##Resources
585618

586619
###Repository Content

‎e2e/README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,3 +90,7 @@ The current test suite is intentionally very limited in scope. This is because t
9090
The tests are quite repetitive and verbose. This is intentional as we want to see them develop more before committing to abstractions.
9191

9292
Currently, visibility into failures is not particularly good. We're hoping that we can pull apart the mcp-go client and have it hook into streams representing stdio without requiring an exec. This way we can get breakpoints in the debugger easily.
93+
94+
###Global State Mutation Tests
95+
96+
Some tools (such as those that mark all notifications as read) would change the global state for the tester, and are also not idempotent, so they offer little value for end to end tests and instead should rely on unit testing and manual verifications.

‎e2e/e2e_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,8 @@ func getRESTClient(t *testing.T) *gogithub.Client {
6262

6363
// Create a new GitHub client with the token
6464
ghClient:=gogithub.NewClient(nil).WithAuthToken(token)
65-
ifhost:=getE2EHost();host!="https://github.com" {
65+
66+
ifhost:=getE2EHost();host!=""&&host!="https://github.com" {
6667
varerrerror
6768
// Currently this works for GHEC because the API is exposed at the api subdomain and the path prefix
6869
// but it would be preferable to extract the host parsing from the main server logic, and use it here.

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp