go fix ./...

to modernize

Author: fujiwara

filter.go

@@ -60,7 +60,6 @@ func runInternalFilter(ctx context.Context, src io.Reader, title string) (string
 			}
 			var found []string
 			for _, item := range items {
-				item := item
 				if item == input {
 					found = []string{item}
 					break

format.go

@@ -175,12 +175,12 @@ func (t *taskFormatterJSON) AddTask(task types.Task) {
 func (t *taskFormatterJSON) Close() {
 }
 
-func MarshalJSONForAPI(v interface{}, query *gojq.Query) ([]byte, error) {
+func MarshalJSONForAPI(v any, query *gojq.Query) ([]byte, error) {
 	b, err := json.Marshal(v)
 	if err != nil {
 		return nil, err
 	}
-	m := map[string]interface{}{}
+	m := map[string]any{}
 	if err := json.Unmarshal(b, &m); err != nil {
 		return nil, err
 	}
@@ -206,8 +206,8 @@ func MarshalJSONForAPI(v interface{}, query *gojq.Query) ([]byte, error) {
 	}
 }
 
-func UnmarshalJSONForStruct(src []byte, v interface{}) error {
-	m := map[string]interface{}{}
+func UnmarshalJSONForStruct(src []byte, v any) error {
+	m := map[string]any{}
 	if err := json.Unmarshal(src, &m); err != nil {
 		return err
 	}
@@ -233,26 +233,26 @@ func jsonKeyForStruct(s string) string {
 	return strings.ToUpper(s[:1]) + s[1:]
 }
 
-func walkMap(m map[string]interface{}, fn func(string) string) {
+func walkMap(m map[string]any, fn func(string) string) {
 	for key, value := range m {
 		delete(m, key)
 		m[fn(key)] = value
 		switch value := value.(type) {
-		case map[string]interface{}:
+		case map[string]any:
 			walkMap(value, fn)
-		case []interface{}:
+		case []any:
 			walkArray(value, fn)
 		default:
 		}
 	}
 }
 
-func walkArray(a []interface{}, fn func(string) string) {
+func walkArray(a []any, fn func(string) string) {
 	for _, value := range a {
 		switch value := value.(type) {
-		case map[string]interface{}:
+		case map[string]any:
 			walkMap(value, fn)
-		case []interface{}:
+		case []any:
 			walkArray(value, fn)
 		default:
 		}

portforward_integration_test.go

@@ -26,9 +26,7 @@ func TestTCPProxyToLocalhost(t *testing.T) {
 	backendCtx, backendCancel := context.WithCancel(context.Background())
 	defer backendCancel()
 
-	backendWg.Add(1)
-	go func() {
-		defer backendWg.Done()
+	backendWg.Go(func() {
 		for {
 			select {
 			case <-backendCtx.Done():
@@ -63,7 +61,7 @@ func TestTCPProxyToLocalhost(t *testing.T) {
 				}
 			}(conn)
 		}
-	}()
+	})
 
 	// Get a different port for proxy frontend
 	proxyListener, err := net.Listen("tcp", "0.0.0.0:0")
@@ -79,14 +77,12 @@ func TestTCPProxyToLocalhost(t *testing.T) {
 	defer cancel()
 
 	var proxyWg sync.WaitGroup
-	proxyWg.Add(1)
-	go func() {
-		defer proxyWg.Done()
+	proxyWg.Go(func() {
 		err := app.startTCPProxyToLocalhost(ctx, "0.0.0.0", proxyPort, backendPort)
 		if err != nil && err != context.DeadlineExceeded && err != context.Canceled {
 			t.Logf("Proxy ended with: %v", err)
 		}
-	}()
+	})
 
 	// Wait a bit for proxy to start
 	time.Sleep(50 * time.Millisecond)
@@ -192,11 +188,9 @@ func TestHandleProxyConnection(t *testing.T) {
 
 	// Start handleProxyConnection
 	var proxyWg sync.WaitGroup
-	proxyWg.Add(1)
-	go func() {
-		defer proxyWg.Done()
+	proxyWg.Go(func() {
 		app.handleProxyConnection(proxyCtx, serverConn, backendPort)
-	}()
+	})
 
 	// Test communication
 	testData := "test data"