Fix linting issues identified in CI

- Remove unused functions: getLocations and isRetryableError
- Fix gofumpt formatting issues in errors.go
- Add proper response body closing in addSSHKeyIdempotent
- Remove unused ctx parameter from handleLLAPIError
- Clean up imports to remove unused packages

All tests pass locally and linting issues are resolved.

Co-Authored-By: Alec Fong <alecsanf@usc.edu>

Author: devin-ai-integration[bot]

internal/lambdalabs/v1/errors.go

@@ -1,15 +1,14 @@
 package v1
 
 import (
-	"context"
 	"fmt"
 	"net/http"
 	"strings"
 
 	v1 "github.com/brevdev/compute/pkg/v1"
 )
 
-func handleLLAPIError(ctx context.Context, resp *http.Response, err error) error {
+func handleLLAPIError(resp *http.Response, err error) error {
 	if err == nil {
 		return nil
 	}
@@ -23,50 +22,28 @@ func handleLLErrToCloudErr(err error) error {
 	}
 
 	errStr := err.Error()
-	
+
 	if strings.Contains(errStr, "insufficient capacity") ||
 		strings.Contains(errStr, "no capacity") ||
 		strings.Contains(errStr, "capacity not available") {
 		return v1.ErrInsufficientResources
 	}
-	
+
 	if strings.Contains(errStr, "quota") ||
 		strings.Contains(errStr, "limit exceeded") ||
 		strings.Contains(errStr, "too many") {
 		return v1.ErrOutOfQuota
 	}
-	
+
 	if strings.Contains(errStr, "not found") ||
 		strings.Contains(errStr, "does not exist") {
 		return v1.ErrInstanceNotFound
 	}
-	
+
 	if strings.Contains(errStr, "service unavailable") ||
 		strings.Contains(errStr, "temporarily unavailable") {
 		return v1.ErrServiceUnavailable
 	}
-	
-	return fmt.Errorf("lambda labs error: %w", err)
-}
 
-func isRetryableError(err error) bool {
-	if err == nil {
-		return false
-	}
-	
-	errStr := err.Error()
-	
-	if strings.Contains(errStr, "unauthorized") ||
-		strings.Contains(errStr, "forbidden") ||
-		strings.Contains(errStr, "not found") ||
-		strings.Contains(errStr, "invalid") ||
-		strings.Contains(errStr, "bad request") {
-		return false
-	}
-	
-	return strings.Contains(errStr, "capacity") ||
-		strings.Contains(errStr, "timeout") ||
-		strings.Contains(errStr, "temporary") ||
-		strings.Contains(errStr, "service unavailable") ||
-		strings.Contains(errStr, "internal server error")
+	return fmt.Errorf("lambda labs error: %w", err)
 }

internal/lambdalabs/v1/instance.go

@@ -119,18 +119,21 @@ func (c *LambdaLabsClient) ListInstances(ctx context.Context, _ v1.ListInstances
 }
 
 func (c *LambdaLabsClient) addSSHKeyIdempotent(ctx context.Context, keyName, publicKey string) error {
-	_, _, err := c.client.DefaultAPI.AddSSHKey(c.makeAuthContext(ctx)).AddSSHKeyRequest(openapi.AddSSHKeyRequest{
+	_, resp, err := c.client.DefaultAPI.AddSSHKey(c.makeAuthContext(ctx)).AddSSHKeyRequest(openapi.AddSSHKeyRequest{
 		Name:      keyName,
 		PublicKey: &publicKey,
 	}).Execute()
-	
+	if resp != nil {
+		defer func() { _ = resp.Body.Close() }()
+	}
+
 	if err != nil {
 		if strings.Contains(err.Error(), "name must be unique") {
 			return nil
 		}
 		return handleLLErrToCloudErr(err)
 	}
-	
+
 	return nil
 }
 

internal/lambdalabs/v1/instancetype.go

@@ -148,14 +148,6 @@ func (c *LambdaLabsClient) GetLocations(_ context.Context, _ v1.GetLocationsArgs
 	return locations, nil
 }
 
-func getLocations() []LambdaLocation {
-	var locations []LambdaLocation
-	err := json.Unmarshal([]byte(lambdaLocationsData), &locations)
-	if err != nil {
-		return []LambdaLocation{}
-	}
-	return locations
-}
 
 func contains(slice []string, item string) bool {
 	for _, s := range slice {