feat: include uuid of created resource in create output

Author: kangasta

CHANGELOG.md

@@ -6,6 +6,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Added
+- Include UUID (or address) of created resource in create command output
+
 ### Changed
 - New go-api version v4.3.0
 

internal/commands/ipaddress/assign.go

@@ -92,5 +92,7 @@ func (s *assignCommand) ExecuteWithoutArguments(exec commands.Executor) (output.
 	}
 	logline.SetMessage(fmt.Sprintf("%s: success", msg))
 	logline.MarkDone()
-	return output.OnlyMarshaled{Value: res}, nil
+	return output.MarshaledWithHumanDetails{Value: res, Details: []output.DetailRow{
+		{Title: "IP Address", Value: res.Address, Colour: ui.DefaultAddressColours},
+	}}, nil
 }

internal/commands/network/create.go

@@ -120,5 +120,7 @@ func (s *createCommand) ExecuteWithoutArguments(exec commands.Executor) (output.
 	}
 	logline.SetMessage(fmt.Sprintf("%s: success", msg))
 	logline.MarkDone()
-	return output.OnlyMarshaled{Value: res}, nil
+	return output.MarshaledWithHumanDetails{Value: res, Details: []output.DetailRow{
+		{Title: "UUID", Value: res.UUID, Colour: ui.DefaultUUUIDColours},
+	}}, nil
 }

internal/commands/networkinterface/create.go

@@ -2,13 +2,15 @@ package networkinterface
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/UpCloudLtd/upcloud-cli/internal/commands"
 	"github.com/UpCloudLtd/upcloud-cli/internal/completion"
 	"github.com/UpCloudLtd/upcloud-cli/internal/config"
 	"github.com/UpCloudLtd/upcloud-cli/internal/output"
 	"github.com/UpCloudLtd/upcloud-cli/internal/resolver"
 	"github.com/UpCloudLtd/upcloud-cli/internal/ui"
+	"github.com/jedib0t/go-pretty/v6/text"
 
 	"github.com/UpCloudLtd/upcloud-go-api/v4/upcloud"
 	"github.com/UpCloudLtd/upcloud-go-api/v4/upcloud/request"
@@ -107,5 +109,23 @@ func (s *createCommand) ExecuteSingleArgument(exec commands.Executor, arg string
 	logline.SetMessage(fmt.Sprintf("%s: done", msg))
 	logline.MarkDone()
 
-	return output.OnlyMarshaled{Value: res}, nil
+	return output.MarshaledWithHumanDetails{Value: res, Details: []output.DetailRow{
+		{Title: "MAC Address", Value: res.MAC},
+		{Title: "IP Addresses", Value: res, Format: formatIPAddresses},
+	}}, nil
+}
+
+func formatIPAddresses(val interface{}) (text.Colors, string, error) {
+	iface, ok := val.(*upcloud.Interface)
+	if !ok {
+		return nil, "", fmt.Errorf("cannot parse IP addresses from %T, expected *upcloud.Interface", val)
+	}
+
+	strs := make([]string, len(iface.IPAddresses))
+
+	for i, ipa := range iface.IPAddresses {
+		strs[i] = ui.DefaultAddressColours.Sprint(ipa.Address)
+	}
+
+	return nil, strings.Join(strs, ",\n"), nil
 }

internal/commands/router/create.go

@@ -59,5 +59,7 @@ func (s *createCommand) ExecuteWithoutArguments(exec commands.Executor) (output.
 	logline.SetMessage(fmt.Sprintf("%s: done", msg))
 	logline.MarkDone()
 
-	return output.OnlyMarshaled{Value: res}, nil
+	return output.MarshaledWithHumanDetails{Value: res, Details: []output.DetailRow{
+		{Title: "UUID", Value: res.UUID, Colour: ui.DefaultUUUIDColours},
+	}}, nil
 }

internal/commands/server/create.go

@@ -400,7 +400,7 @@ func formatIPAddresses(val interface{}) (text.Colors, string, error) {
 
 	var strs []string
 	for ipa := range addresses {
-		strs = append(strs, ipa)
+		strs = append(strs, ui.DefaultAddressColours.Sprint(ipa))
 	}
 
 	return nil, strings.Join(strs, ",\n"), nil

internal/commands/storage/backup_create.go

@@ -74,5 +74,7 @@ func (s *createBackupCommand) Execute(exec commands.Executor, uuid string) (outp
 	logline.SetMessage(fmt.Sprintf("%s: success", msg))
 	logline.MarkDone()
 
-	return output.OnlyMarshaled{Value: res}, nil
+	return output.MarshaledWithHumanDetails{Value: res, Details: []output.DetailRow{
+		{Title: "UUID", Value: res.UUID, Colour: ui.DefaultUUUIDColours},
+	}}, nil
 }

internal/commands/storage/create.go

@@ -112,5 +112,7 @@ func (s *createCommand) ExecuteWithoutArguments(exec commands.Executor) (output.
 	logline.SetMessage(fmt.Sprintf("%s: success", msg))
 	logline.MarkDone()
 
-	return output.OnlyMarshaled{Value: res}, nil
+	return output.MarshaledWithHumanDetails{Value: res, Details: []output.DetailRow{
+		{Title: "UUID", Value: res.UUID, Colour: ui.DefaultUUUIDColours},
+	}}, nil
 }