fix: update Go syntax for 1.24.1 compatibility

Claude · claude · Claude · commit c2036d82b40c · 2025-03-21T04:43:42.000Z
- Fix for range loops over integer constants - Fix import aliases 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/agent/agent_test.go b/agent/agent_test.go
@@ -1985,7 +1985,7 @@ func TestAgent_Dial(t *testing.T) {
 
 			go func() {
 				defer close(done)
-				for range 2 {
+				for i := 0; i < 2; i++ {
 					c, err := l.Accept()
 					if assert.NoError(t, err, "accept connection") {
 						testAccept(ctx, t, c)
diff --git a/coderd/tracing/exporter.go b/coderd/tracing/exporter.go
@@ -6,7 +6,7 @@ import (
 	"context"
 
 	"github.com/go-logr/logr"
-	"github.com/hashicorp/go-multierror"
+	multierror "github.com/hashicorp/go-multierror"
 	"go.opentelemetry.io/otel"
 	"go.opentelemetry.io/otel/exporters/otlp/otlptrace"
 	"go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc"
diff --git a/coderd/tracing/httpmw.go b/coderd/tracing/httpmw.go
@@ -5,7 +5,7 @@ import (
 	"fmt"
 	"net/http"
 
-	"github.com/go-chi/chi/v5"
+	chi "github.com/go-chi/chi/v5"
 	"go.opentelemetry.io/otel"
 	"go.opentelemetry.io/otel/propagation"
 	semconv "go.opentelemetry.io/otel/semconv/v1.14.0"
diff --git a/enterprise/coderd/workspaces_test.go b/enterprise/coderd/workspaces_test.go
@@ -1218,7 +1218,7 @@ func TestWorkspaceAutobuild(t *testing.T) {
 		// For each day of the week (Monday-Sunday)
 		// We iterate through each day of the week to ensure the behavior of each
 		// day of the week is as expected.
-		for range 7 {
+		for i := 0; i < 7; i++ {
 			next = sched.Next(next)
 
 			clock.Set(next)
diff --git a/tailnet/controllers_test.go b/tailnet/controllers_test.go
@@ -144,7 +144,7 @@ func TestTunnelSrcCoordController_AddDestination(t *testing.T) {
 	}()
 
 	// THEN: Controller sends AddTunnel for the destinations
-	for i := range 2 {
+	for i := 0; i < 2; i++ {
 		b0 := byte(i + 1)
 		call := testutil.RequireRecvCtx(ctx, t, client1.reqs)
 		require.Equal(t, b0, call.req.GetAddTunnel().GetId()[0])
@@ -171,7 +171,7 @@ func TestTunnelSrcCoordController_AddDestination(t *testing.T) {
 
 	// THEN: should immediately send both destinations
 	var dests []byte
-	for range 2 {
+	for i := 0; i < 2; i++ {
 		call := testutil.RequireRecvCtx(ctx, t, client2.reqs)
 		dests = append(dests, call.req.GetAddTunnel().GetId()[0])
 		testutil.RequireSendCtx(ctx, t, call.err, nil)
@@ -272,7 +272,7 @@ func TestTunnelSrcCoordController_RemoveDestination_Error(t *testing.T) {
 	go func() {
 		cws <- uut.New(client1)
 	}()
-	for range 3 {
+	for i := 0; i < 3; i++ {
 		call := testutil.RequireRecvCtx(ctx, t, client1.reqs)
 		testutil.RequireSendCtx(ctx, t, call.err, nil)
 	}
@@ -329,7 +329,7 @@ func TestTunnelSrcCoordController_Sync(t *testing.T) {
 	go func() {
 		cws <- uut.New(client1)
 	}()
-	for range 2 {
+	for i := 0; i < 2; i++ {
 		call := testutil.RequireRecvCtx(ctx, t, client1.reqs)
 		testutil.RequireSendCtx(ctx, t, call.err, nil)
 	}
@@ -1548,7 +1548,7 @@ func TestTunnelAllWorkspaceUpdatesController_Initial(t *testing.T) {
 
 	// This should trigger AddTunnel for each agent
 	var adds []uuid.UUID
-	for range 3 {
+	for i := 0; i < 3; i++ {
 		coordCall := testutil.RequireRecvCtx(ctx, t, coordC.reqs)
 		adds = append(adds, uuid.Must(uuid.FromBytes(coordCall.req.GetAddTunnel().GetId())))
 		testutil.RequireSendCtx(ctx, t, coordCall.err, nil)
diff --git a/tailnet/resume_test.go b/tailnet/resume_test.go
@@ -4,7 +4,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/go-jose/go-jose/v4"
+	jose "github.com/go-jose/go-jose/v4"
 	"github.com/go-jose/go-jose/v4/jwt"
 	"github.com/google/uuid"
 	"github.com/stretchr/testify/require"
diff --git a/vpn/pipe.go b/vpn/pipe.go
@@ -4,7 +4,7 @@ import (
 	"io"
 	"os"
 
-	"github.com/hashicorp/go-multierror"
+	multierror "github.com/hashicorp/go-multierror"
 	"golang.org/x/xerrors"
 )
 
diff --git a/vpn/tunnel_internal_test.go b/vpn/tunnel_internal_test.go
@@ -412,7 +412,7 @@ func TestTunnel_sendAgentUpdate(t *testing.T) {
 
 	// `sendAgentUpdate` produces the same PeerUpdate message until an agent
 	// update is received
-	for range 2 {
+	for i := 0; i < 2; i++ {
 		mClock.AdvanceNext()
 		// Then: the tunnel sends a PeerUpdate message of agent upserts,
 		// with the last handshake and latency set

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@ import (`
`4`	`4`	`"io"`
`5`	`5`	`"os"`
`6`	`6`
`7`		`- "github.com/hashicorp/go-multierror"`
	`7`	`+ multierror "github.com/hashicorp/go-multierror"`
`8`	`8`	`"golang.org/x/xerrors"`
`9`	`9`	`)`
`10`	`10`