feat: split datacenter name input

Author: apricote

internal/cmd/primaryip/create.go

@@ -2,6 +2,7 @@ package primaryip
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/spf13/cobra"
 
@@ -34,10 +35,10 @@ See https://docs.hetzner.cloud/changelog#2025-12-16-phasing-out-datacenters`,
 
 		cmd.Flags().Int64("assignee-id", 0, "Assignee (usually a Server) to assign Primary IP to")
 
-		cmd.Flags().String("location", "", "Location of Primary IP")
+		cmd.Flags().String("location", "", "Location (ID or name) of Primary IP")
 		_ = cmd.RegisterFlagCompletionFunc("location", cmpl.SuggestCandidatesF(client.Location().Names))
 
-		cmd.Flags().String("datacenter", "", "Datacenter (ID or name) (deprecated)")
+		cmd.Flags().String("datacenter", "", "Datacenter (name) (deprecated)")
 		_ = cmd.RegisterFlagCompletionFunc("datacenter", cmpl.SuggestCandidatesF(client.Datacenter().Names))
 
 		cmd.Flags().StringToString("label", nil, "User-defined labels ('key=value') (can be specified multiple times)")
@@ -70,7 +71,6 @@ See https://docs.hetzner.cloud/changelog#2025-12-16-phasing-out-datacenters`,
 			Type:         hcloud.PrimaryIPType(typ),
 			Name:         name,
 			AssigneeType: "server",
-			Datacenter:   datacenter,
 			Labels:       labels,
 		}
 		if assigneeID != 0 {
@@ -91,6 +91,17 @@ See https://docs.hetzner.cloud/changelog#2025-12-16-phasing-out-datacenters`,
 		}
 		if cmd.Flags().Changed("datacenter") {
 			cmd.PrintErrln("Warning: The --datacenter flag is deprecated. Use --location or --assignee-id instead.")
+
+			// Backward compatible datacenter argument.
+			// datacenter hel1-dc2 => location hel1
+			parts := strings.Split(datacenter, "-")
+
+			if len(parts) != 2 {
+				return nil, nil, fmt.Errorf("Datacenter name is not valid, expected format $LOCATION-$DATACENTER, but got: %s", datacenter)
+			}
+
+			locationName := parts[0]
+			createOpts.Location = locationName
 		}
 
 		result, _, err := s.Client().PrimaryIP().Create(s, createOpts)

internal/cmd/server/create.go

@@ -9,6 +9,7 @@ import (
 	"mime/multipart"
 	"net/textproto"
 	"os"
+	"strconv"
 	"strings"
 	"time"
 
@@ -439,16 +440,36 @@ func createOptsFromFlags(
 	if datacenterIDOrName != "" {
 		cmd.PrintErrln("Warning: The --datacenter flag is deprecated. Use --location instead.")
 
-		var datacenter *hcloud.Datacenter
-		datacenter, _, err = s.Client().Datacenter().Get(s, datacenterIDOrName)
+		// If parseable as ID -> GetByID and use Location Name
+		// Else -> Backwards-compatible split
+		var datacenterID int64
+		datacenterID, err = strconv.ParseInt(datacenterIDOrName, 10, 64)
+
 		if err != nil {
-			return
-		}
-		if datacenter == nil {
-			err = fmt.Errorf("Datacenter not found: %s", datacenterIDOrName)
-			return
+			// Input was a valid number/ID
+			var datacenter *hcloud.Datacenter
+			datacenter, _, err = s.Client().Datacenter().GetByID(s, datacenterID)
+			if err != nil {
+				return
+			}
+			if datacenter == nil {
+				err = fmt.Errorf("Datacenter not found: %s", datacenterIDOrName)
+				return
+			}
+			createOpts.Location = datacenter.Location
+		} else {
+			// Input was not a valid number/ID, probably DC name
+
+			// Backward compatible datacenter argument.
+			// datacenter hel1-dc2 => location hel1
+			parts := strings.Split(datacenterIDOrName, "-")
+
+			if len(parts) != 2 {
+				err = fmt.Errorf("Datacenter name is not valid, expected format $LOCATION-$DATACENTER, but got: %s", datacenterIDOrName)
+			}
+
+			createOpts.Location = &hcloud.Location{Name: parts[0]}
 		}
-		createOpts.Datacenter = datacenter
 	}
 
 	if locationIDOrName != "" {