fix: non-zero exit on delete cancellation; remove unsupported routine delete

cmd/folder/delete.go

@@ -110,8 +110,7 @@ func runFolderDelete(cmd *cobra.Command, args []string) error {
 
 		response = strings.TrimSpace(strings.ToLower(response))
 		if response != "yes" {
-			fmt.Println("Deletion cancelled.")
-			return nil
+			return fmt.Errorf("deletion cancelled")
 		}
 	}
 

cmd/routine/routine.go

@@ -13,7 +13,6 @@ Examples:
   hevycli routine get <id>              # Get routine details
   hevycli routine create --file r.json  # Create from JSON
   hevycli routine update <id> --file r.json  # Update routine
-  hevycli routine delete <id>           # Delete routine
   hevycli routine builder               # Interactive routine builder`,
 }
 
@@ -22,5 +21,4 @@ func init() {
 	Cmd.AddCommand(getCmd)
 	Cmd.AddCommand(createCmd)
 	Cmd.AddCommand(updateCmd)
-	// deleteCmd is added in delete.go's init()
 }

cmd/workout/delete.go

@@ -101,8 +101,7 @@ func runDelete(cmd *cobra.Command, args []string) error {
 
 		response = strings.TrimSpace(strings.ToLower(response))
 		if response != "yes" {
-			fmt.Println("Deletion cancelled.")
-			return nil
+			return fmt.Errorf("deletion cancelled")
 		}
 	}
 

internal/api/client.go

@@ -23,7 +23,7 @@
 type Client struct {
 	baseURL    string
 	apiKey     string
 	httpClient *resty.Client
 }
 
 // ClientOption is a function that configures the client
@@ -40,7 +40,7 @@
 		opt(c)
 	}
 
 	c.httpClient = resty.New().
 		SetBaseURL(c.baseURL).
 		SetHeader("api-key", c.apiKey).
 		SetHeader("User-Agent", UserAgent).
@@ -254,7 +254,7 @@
 }
 
 // handleResponse checks for API errors based on HTTP status code
 func (c *Client) handleResponse(resp *resty.Response) error {
 	switch resp.StatusCode() {
 	case http.StatusOK, http.StatusCreated, http.StatusNoContent:
 		return nil
@@ -530,21 +530,6 @@
 	return &result, nil
 }
 
-// DeleteRoutine deletes a routine by ID
-func (c *Client) DeleteRoutine(id string) error {
-	resp, err := c.httpClient.R().
-		Delete("/routines/" + id)
-
-	if err != nil {
-		return &APIError{
-			ErrorCode:    "NETWORK_ERROR",
-			ErrorMessage: fmt.Sprintf("failed to delete routine: %v", err),
-		}
-	}
-
-	return c.handleResponse(resp)
-}
-
 // UpdateRoutineFolder updates an existing routine folder
 func (c *Client) UpdateRoutineFolder(id int, req *UpdateRoutineFolderRequest) (*RoutineFolder, error) {
 	var result RoutineFolderResponse