replace common.Undo behaviour with undo package equivalent

pkg/compute/deploy.go

@@ -20,6 +20,7 @@ import (
 	"github.com/fastly/cli/pkg/config"
 	"github.com/fastly/cli/pkg/errors"
 	"github.com/fastly/cli/pkg/text"
+	"github.com/fastly/cli/pkg/undo"
 	"github.com/fastly/go-fastly/v3/fastly"
 	"github.com/kennygrant/sanitize"
 )
@@ -174,7 +175,7 @@ func (c *DeployCommand) Exec(in io.Reader, out io.Writer) (err error) {
 		progress = text.NewQuietProgress(out)
 	}
 
-	undoStack := common.NewUndoStack()
+	undoStack := undo.NewStack()
 
 	defer func() {
 		if err != nil {
@@ -562,7 +563,7 @@ func cfgBackend(backend string, backendPort uint, out io.Writer, in io.Reader, f
 
 // createDomain creates the given domain and handle unrolling the stack in case
 // of an error (i.e. will ensure the domain is deleted if there is an error).
-func createDomain(progress text.Progress, client api.Interface, serviceID string, version int, domain string, undoStack common.Undoer) error {
+func createDomain(progress text.Progress, client api.Interface, serviceID string, version int, domain string, undoStack undo.Stacker) error {
 	progress.Step("Creating domain...")
 
 	undoStack.Push(func() error {
@@ -587,7 +588,7 @@ func createDomain(progress text.Progress, client api.Interface, serviceID string
 
 // createBackend creates the given domain and handle unrolling the stack in case
 // of an error (i.e. will ensure the backend is deleted if there is an error).
-func createBackend(progress text.Progress, client api.Interface, serviceID string, version int, backend string, backendPort uint, undoStack common.Undoer) error {
+func createBackend(progress text.Progress, client api.Interface, serviceID string, version int, backend string, backendPort uint, undoStack undo.Stacker) error {
 	progress.Step("Creating backend...")
 
 	undoStack.Push(func() error {

pkg/undo/doc.go

@@ -0,0 +1,2 @@
+// Package undo contains abstractions for working with a stack of state changes.
+package undo

pkg/undo/undo.go

@@ -59,7 +59,7 @@ func (s *Stack) Len() int {
 // calling each Fn function state in FIFO order. If any Fn returns an
 // error, it gets logged to the provided writer. Should be deferrerd, such as:
 //
-//     undoStack := common.NewStack()
+//     undoStack := undo.NewStack()
 //     defer func() { undoStack.RunIfError(w, err) }()
 //
 func (s *Stack) RunIfError(w io.Writer, err error) {