refactor: replace util.SliceContainsString with slices.Contains & make fmt

cmd/filters/list.go

@@ -15,6 +15,7 @@ package filters
 
 import (
 	"fmt"
+	"slices"
 
 	"github.com/fatih/color"
 	"github.com/k8sgpt-ai/k8sgpt/pkg/analyzer"
@@ -40,10 +41,9 @@ var listCmd = &cobra.Command{
 		inactiveFilters := util.SliceDiff(availableFilters, activeFilters)
 		fmt.Print(color.YellowString("Active: \n"))
 		for _, filter := range activeFilters {
-
 			// if the filter is an integration, mark this differently
 			// but if the integration is inactive, remove
-			if util.SliceContainsString(integrationFilters, filter) {
+			if slices.Contains(integrationFilters, filter) {
 				fmt.Printf("> %s\n", color.BlueString("%s (integration)", filter))
 			} else {
 				// This strange bit of logic will loop through every integration via
@@ -60,13 +60,12 @@ var listCmd = &cobra.Command{
 			fmt.Print(color.YellowString("Unused: \n"))
 			for _, filter := range inactiveFilters {
 				// if the filter is an integration, mark this differently
-				if util.SliceContainsString(integrationFilters, filter) {
+				if slices.Contains(integrationFilters, filter) {
 					fmt.Printf("> %s\n", color.BlueString("%s (integration)", filter))
 				} else {
 					fmt.Printf("> %s\n", color.RedString(filter))
 				}
 			}
 		}
-
 	},
 }

cmd/serve/serve.go

@@ -79,13 +79,13 @@ var ServeCmd = &cobra.Command{
 			envIsSet := backend != "" || password != "" || model != ""
 			if envIsSet {
 				aiProvider = &ai.AIProvider{
-					Name:        backend,
-					Password:    password,
-					Model:       model,
-					BaseURL:     baseURL,
-					Engine:      engine,
+					Name:          backend,
+					Password:      password,
+					Model:         model,
+					BaseURL:       baseURL,
+					Engine:        engine,
 					ProxyEndpoint: proxyEndpoint,
-					Temperature: temperature(),
+					Temperature:   temperature(),
 				}
 
 				configAI.Providers = append(configAI.Providers, *aiProvider)

pkg/ai/azureopenai.go

@@ -34,7 +34,7 @@ func (c *AzureAIClient) Configure(config IAIConfig) error {
 		return azureModelMapping[model]
 
 	}
-	
+
 	if proxyEndpoint != "" {
 		proxyUrl, err := url.Parse(proxyEndpoint)
 		if err != nil {

pkg/ai/openai.go

@@ -63,7 +63,7 @@ func (c *OpenAIClient) Configure(config IAIConfig) error {
 			Transport: transport,
 		}
 	}
-	
+
 	client := openai.NewClientWithConfig(defaultConfig)
 	if client == nil {
 		return errors.New("error creating OpenAI client")

pkg/util/util.go

@@ -33,15 +33,6 @@ import (
 
 var anonymizePattern = []rune("abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789!@#$%^&*()-_=+[]{}|;':\",./<>?")
 
-func SliceContainsString(slice []string, s string) bool {
-	for _, item := range slice {
-		if item == s {
-			return true
-		}
-	}
-	return false
-}
-
 func GetParent(client *kubernetes.Client, meta metav1.ObjectMeta) (string, bool) {
 	if meta.OwnerReferences != nil {
 		for _, owner := range meta.OwnerReferences {
@@ -183,7 +174,8 @@ func GetCacheKey(provider string, language string, sEnc string) string {
 
 func GetPodListByLabels(client k.Interface,
 	namespace string,
-	labels map[string]string) (*v1.PodList, error) {
+	labels map[string]string,
+) (*v1.PodList, error) {
 	pods, err := client.CoreV1().Pods(namespace).List(context.Background(), metav1.ListOptions{
 		LabelSelector: metav1.FormatLabelSelector(&metav1.LabelSelector{
 			MatchLabels: labels,
@@ -207,7 +199,7 @@ func FileExists(path string) (bool, error) {
 }
 
 func EnsureDirExists(dir string) error {
-	err := os.MkdirAll(dir, 0755)
+	err := os.MkdirAll(dir, 0o755)
 
 	if errors.Is(err, os.ErrExist) {
 		return nil

pkg/util/util_test.go

@@ -26,29 +26,6 @@ import (
 	"k8s.io/client-go/kubernetes/fake"
 )
 
-func TestSliceContainsString(t *testing.T) {
-	tests := []struct {
-		slice    []string
-		s        string
-		expected bool
-	}{
-		{
-			expected: false,
-		},
-		{
-			slice:    []string{"temp", "value"},
-			s:        "value",
-			expected: true,
-		},
-	}
-	for _, tt := range tests {
-		tt := tt
-		t.Run(tt.s, func(t *testing.T) {
-			require.Equal(t, tt.expected, SliceContainsString(tt.slice, tt.s))
-		})
-	}
-}
-
 func TestGetParent(t *testing.T) {
 	ownerName := "test-name"
 	namespace := "test"
@@ -409,6 +386,7 @@ func TestGetPodListByLabels(t *testing.T) {
 		})
 	}
 }
+
 func TestFileExists(t *testing.T) {
 	tests := []struct {
 		filePath  string