Inline optional bool function for all notifications

Author: martinajir

pkg/github/notifications.go

@@ -36,9 +36,9 @@ func ListNotifications(getClient GetClientFn, t translations.TranslationHelperFu
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
-			all, err := OptionalBoolParamWithDefault(request, "all", false) // Default to false unless specified
-			if err != nil {
-				return mcp.NewToolResultError(err.Error()), nil
+			all := false
+			if val, err := OptionalParam[bool](request, "all"); err == nil {
+				all = val
 			}
 
 			opts := &github.NotificationListOptions{

pkg/github/server.go

@@ -130,20 +130,6 @@ func OptionalIntParam(r mcp.CallToolRequest, p string) (int, error) {
 	return int(v), nil
 }
 
-// OptionalBoolParamWithDefault is a helper function that retrieves a boolean parameter from the request.
-// If the parameter is not present, it returns the provided default value. If the parameter is present,
-// it validates its type and returns the value.
-func OptionalBoolParamWithDefault(request mcp.CallToolRequest, s string, b bool) (bool, error) {
-	v, err := OptionalParam[bool](request, s)
-	if err != nil {
-		return false, err
-	}
-	if !b {
-		return b, nil
-	}
-	return v, nil
-}
-
 // OptionalIntParamWithDefault is a helper function that can be used to fetch a requested parameter from the request
 // similar to optionalIntParam, but it also takes a default value.
 func OptionalIntParamWithDefault(r mcp.CallToolRequest, p string, d int) (int, error) {