use switch case instead of if

Author: SamMorrowDrums

pkg/github/notifications.go

@@ -171,18 +171,18 @@ func DismissNotification(getclient GetClientFn, t translations.TranslationHelper
 			}
 
 			var resp *github.Response
-
-			if state == "done" {
+			switch state {
+			case "done":
 				// for some inexplicable reason, the API seems to have threadID as int64 and string depending on the endpoint
 				var threadIDInt int64
 				threadIDInt, err = strconv.ParseInt(threadID, 10, 64)
 				if err != nil {
 					return mcp.NewToolResultError(fmt.Sprintf("invalid threadID format: %v", err)), nil
 				}
 				resp, err = client.Activity.MarkThreadDone(ctx, threadIDInt)
-			} else if state == "read" {
+			case "read":
 				resp, err = client.Activity.MarkThreadRead(ctx, threadID)
-			} else {
+			default:
 				return mcp.NewToolResultError("Invalid state. Must be one of: read, done."), nil
 			}