Modify project destroy to require -p flag and delete projects created in UI without data source

.changelog/4212.txt

@@ -0,0 +1,7 @@
+```release-note:improvement
+cli: `project destroy` requires the `-project` or `-p` flag regardless of where it's run.
+```
+
+```release-note:bug
+cli: `project destroy` now successfully destroys a project created in the UI without a remote source or local hcl file.
+```

internal/cli/project_destroy.go

@@ -1,11 +1,12 @@
 package cli
 
 import (
+ "strings"
+
  "github.com/hashicorp/waypoint-plugin-sdk/terminal"
  "github.com/hashicorp/waypoint/internal/clierrors"
  "github.com/hashicorp/waypoint/internal/pkg/flag"
  pb "github.com/hashicorp/waypoint/pkg/server/gen"
- "strings"
 )
 
 type ProjectDestroyCommand struct {
@@ -24,20 +25,21 @@ func (c *ProjectDestroyCommand) Run(args []string) int {
  return 1
  }
 
- if c.project == nil {
- c.ui.Output("The -project flag must be set.", terminal.WithErrorStyle())
+ if c.flagProject == "" {
+ c.ui.Output("Must explicitly set -project (-p) flag to destroy project.", terminal.WithErrorStyle())
  return 1
  }
 
- // Get the project we're destroying
+ // Verify the project we're destroying exists
  project, err := c.project.Client().GetProject(c.Ctx, &pb.GetProjectRequest{
  Project: c.project.Ref(),
  })
  if err != nil {
+ c.ui.Output("Project %q not found.", c.project.Ref().Project, terminal.WithErrorStyle())
  return 1
  }
 
- // Confirmation is required for destroying a project &/or its resources
+ // Confirmation required without `-auto-approve` flag
  if !c.confirm {
  proceed, err := c.ui.Input(&terminal.Input{
  Prompt: "Do you really want to destroy project \"" + project.Project.Name + "\" and its resources? Only 'yes' will be accepted to approve: ",
@@ -56,12 +58,19 @@ func (c *ProjectDestroyCommand) Run(args []string) int {
  return 1
  }
  }
- _, err = c.project.DestroyProject(c.Ctx, &pb.Job_DestroyProjectOp{
- Project: &pb.Ref_Project{Project: project.Project.Name},
- SkipDestroyResources: c.skipDestroyResources,
- })
+
+ if project.Project.DataSource == nil {
+ _, err = c.project.Client().DestroyProject(c.Ctx, &pb.DestroyProjectRequest{
+ Project: c.project.Ref(),
+ })
+ } else {
+ _, err = c.project.DestroyProject(c.Ctx, &pb.Job_DestroyProjectOp{
+ Project: &pb.Ref_Project{Project: project.Project.Name},
+ SkipDestroyResources: c.skipDestroyResources,
+ })
+ }
  if err != nil {
- c.ui.Output("Error destroying project: %s", err.Error(), terminal.WithErrorStyle())
+ c.ui.Output("Error destroying project %q: %s", project.Project.Name, err.Error(), terminal.WithErrorStyle())
  return 1
  }
  c.ui.Output("Project %q destroyed!", project.Project.Name, terminal.WithSuccessStyle())

internal/runner/accept.go

@@ -173,7 +173,7 @@ func (r *Runner) accept(ctx context.Context, id string) error {
  // Grab this lock before updating canceled. You don't
  // need to have this lock to touch canceled (we use atomic ops)
  // but it is used when the streamCancel is being reset so that
- // we don't set it up and race with cancelation.
+ // we don't set it up and race with cancellation.
  streamCtxLock.Lock()
  defer streamCtxLock.Unlock()