Merge branch 'main' of github.com:aryasoni98/github-mcp-server

Author: aryasoni98

.github/workflows/lint.yaml

@@ -24,11 +24,6 @@ jobs:
           go mod verify
           go mod download
 
-          LINT_VERSION=2.1.6
-          curl -fsSL https://github.com/golangci/golangci-lint/releases/download/v${LINT_VERSION}/golangci-lint-${LINT_VERSION}-linux-amd64.tar.gz | \
-            tar xz --strip-components 1 --wildcards \*/golangci-lint
-          mkdir -p bin && mv golangci-lint bin/
-
       - name: Run checks
         run: |
           STATUS=0
@@ -41,10 +36,10 @@ jobs:
               STATUS=1
             fi
           }
-
-          assert-nothing-changed go fmt ./...
           assert-nothing-changed go mod tidy
-
-          bin/golangci-lint run --timeout=3m || STATUS=$?
-
           exit $STATUS
+
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@4afd733a84b1f43292c63897423277bb7f4313a9
+        with:
+          version: v2.1.6

README.md

@@ -636,6 +636,39 @@ export GITHUB_MCP_TOOL_ADD_ISSUE_COMMENT_DESCRIPTION="an alternative description
   - `secret_type`: The secret types to be filtered for in a comma-separated list (string, optional)
   - `resolution`: The resolution status (string, optional)
 
+### Notifications
+
+- **list_notifications** – List notifications for a GitHub user
+  - `filter`: Filter to apply to the response (`default`, `include_read_notifications`, `only_participating`)
+  - `since`: Only show notifications updated after the given time (ISO 8601 format)
+  - `before`: Only show notifications updated before the given time (ISO 8601 format)
+  - `owner`: Optional repository owner (string)
+  - `repo`: Optional repository name (string)
+  - `page`: Page number (number, optional)
+  - `perPage`: Results per page (number, optional)
+
+
+- **get_notification_details** – Get detailed information for a specific GitHub notification
+  - `notificationID`: The ID of the notification (string, required)
+
+- **dismiss_notification** – Dismiss a notification by marking it as read or done
+  - `threadID`: The ID of the notification thread (string, required)
+  - `state`: The new state of the notification (`read` or `done`)
+
+- **mark_all_notifications_read** – Mark all notifications as read
+  - `lastReadAt`: Describes the last point that notifications were checked (optional, RFC3339/ISO8601 string, default: now)
+  - `owner`: Optional repository owner (string)
+  - `repo`: Optional repository name (string)
+
+- **manage_notification_subscription** – Manage a notification subscription (ignore, watch, or delete) for a notification thread
+  - `notificationID`: The ID of the notification thread (string, required)
+  - `action`: Action to perform: `ignore`, `watch`, or `delete` (string, required)
+
+- **manage_repository_notification_subscription** – Manage a repository notification subscription (ignore, watch, or delete)
+  - `owner`: The account owner of the repository (string, required)
+  - `repo`: The name of the repository (string, required)
+  - `action`: Action to perform: `ignore`, `watch`, or `delete` (string, required)
+
 ## Resources
 
 ### Repository Content

e2e/README.md

@@ -90,3 +90,7 @@ The current test suite is intentionally very limited in scope. This is because t
 The tests are quite repetitive and verbose. This is intentional as we want to see them develop more before committing to abstractions.
 
 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.
+
+### Global State Mutation Tests
+
+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.