Fix dismiss_notification to accept HTTP 204 No Content response

- Add http.StatusNoContent (204) to the list of accepted success status codes
- Add test case for 204 response when marking notification as done
- Retain existing test for 200 response for backwards compatibility

Co-authored-by: SamMorrowDrums <[email protected]>

Author: Copilot

pkg/github/notifications.go

@@ -231,7 +231,7 @@ func DismissNotification(t translations.TranslationHelperFunc) inventory.ServerT
 			}
 			defer func() { _ = resp.Body.Close() }()
 
-			if resp.StatusCode != http.StatusResetContent && resp.StatusCode != http.StatusOK {
+			if resp.StatusCode != http.StatusResetContent && resp.StatusCode != http.StatusNoContent && resp.StatusCode != http.StatusOK {
 				body, err := io.ReadAll(resp.Body)
 				if err != nil {
 					return utils.NewToolResultErrorFromErr("failed to read response body", err), nil, nil

pkg/github/notifications_test.go

@@ -472,7 +472,19 @@ func Test_DismissNotification(t *testing.T) {
 			expectRead:  true,
 		},
 		{
-			name: "mark as done",
+			name: "mark as done with 204 response",
+			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
+				DeleteNotificationsThreadsByThreadID: mockResponse(t, http.StatusNoContent, nil),
+			}),
+			requestArgs: map[string]interface{}{
+				"threadID": "123",
+				"state":    "done",
+			},
+			expectError: false,
+			expectDone:  true,
+		},
+		{
+			name: "mark as done with 200 response",
 			mockedClient: MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
 				DeleteNotificationsThreadsByThreadID: mockResponse(t, http.StatusOK, nil),
 			}),