move checkPortAvailable to its own package

carole-lavillonniere · carole-lavillonniere · commit f4f420e4f11c · 2026-02-19T15:27:06.000+01:00
diff --git a/internal/container/start.go b/internal/container/start.go
@@ -3,7 +3,6 @@ package container
 import (
 	"context"
 	"fmt"
-	"net"
 	"net/http"
 	"os"
 	stdruntime "runtime"
@@ -14,6 +13,7 @@ import (
 	"github.com/localstack/lstk/internal/auth"
 	"github.com/localstack/lstk/internal/config"
 	"github.com/localstack/lstk/internal/output"
+	"github.com/localstack/lstk/internal/ports"
 	"github.com/localstack/lstk/internal/runtime"
 )
 
@@ -142,7 +142,7 @@ func selectContainersToStart(ctx context.Context, rt runtime.Runtime, sink outpu
 			output.EmitLog(sink, fmt.Sprintf("%s is already running", c.Name))
 			continue
 		}
-		if err := checkPortAvailable(c.Port); err != nil {
+		if err := ports.CheckAvailable(c.Port); err != nil {
 			configPath, pathErr := config.ConfigFilePath()
 			if pathErr != nil {
 				return nil, err
@@ -154,18 +154,6 @@ func selectContainersToStart(ctx context.Context, rt runtime.Runtime, sink outpu
 	return filtered, nil
 }
 
-func checkPortAvailable(port string) error {
-	conn, err := net.DialTimeout("tcp", "localhost:"+port, time.Second)
-	if err != nil {
-		return nil
-	}
-	err = conn.Close()
-	if err != nil {
-		return nil
-	}
-	return fmt.Errorf("port %s already in use", port)
-}
-
 func validateLicense(ctx context.Context, rt runtime.Runtime, sink output.Sink, platformClient api.PlatformAPI, containerConfig runtime.ContainerConfig, token string) error {
 	version := containerConfig.Tag
 	if version == "" || version == "latest" {
diff --git a/internal/ports/ports.go b/internal/ports/ports.go
@@ -0,0 +1,16 @@
+package ports
+
+import (
+	"fmt"
+	"net"
+	"time"
+)
+
+func CheckAvailable(port string) error {
+	conn, err := net.DialTimeout("tcp", "localhost:"+port, time.Second)
+	if err != nil {
+		return nil
+	}
+	_ = conn.Close()
+	return fmt.Errorf("port %s already in use", port)
+}
