fix: better error messages when not passing an id

not passing an id for a resource that requires it results
in an esoteric error message.

Author: toumorokoshi

cmd/aepcli/main.go

@@ -112,10 +112,10 @@ func aepcli(args []string) error {
 	s = service.NewService(api, headersMap)
 
 	result, err := s.ExecuteCommand(additionalArgs)
+	fmt.Println(result)
 	if err != nil {
-		return fmt.Errorf("unable to execute command: %w", err)
+		return err
 	}
-	fmt.Println(result)
 	return nil
 }
 

internal/service/resource_definition.go

@@ -158,15 +158,13 @@ func ExecuteResourceCommand(r *api.Resource, args []string) (*http.Request, stri
 		}
 		c.AddCommand(customCmd)
 	}
-	var stderr strings.Builder
 	var stdout strings.Builder
 	c.SetOut(&stdout)
-	c.SetErr(&stderr)
 	c.SetArgs(args)
 	if err := c.Execute(); err != nil {
-		return nil, stdout.String() + stderr.String(), err
+		return nil, stdout.String(), err
 	}
-	return req, stdout.String() + stderr.String(), err
+	return req, stdout.String(), err
 }
 
 func addSchemaFlags(c *cobra.Command, schema openapi.Schema, args map[string]interface{}) error {

internal/service/service.go

@@ -39,7 +39,7 @@ func (s *Service) ExecuteCommand(args []string) (string, error) {
 	}
 	req, output, err := ExecuteResourceCommand(r, args[1:])
 	if err != nil {
-		return "", fmt.Errorf("unable to execute command: %v", err)
+		return output, err
 	}
 	if req == nil {
 		return output, nil