fix: addressed suggested changes

Author: BhautikChudasama

cmd/cronjobs/create.go

@@ -133,7 +133,7 @@ Examples:
 					}
 					path = pterm.RemoveColorFromString(path)
 				}
-				if method == "GET" {
+				if !c.IsSet("method") {
 					methods := []string{"GET", "POST", "PUT", "DELETE", "PATCH", "HEAD"}
 					selected, selErr := pterm.DefaultInteractiveSelect.
 						WithOptions(methods).

cmd/cronjobs/update.go

@@ -62,7 +62,9 @@ Examples:
 			// Decode existing settings for defaults in both TTY and non-TTY.
 			var currentSettings api.HTTPCronjobSettings
 			if existing.Settings != nil {
-				_ = json.Unmarshal(*existing.Settings, &currentSettings)
+				if err := json.Unmarshal(*existing.Settings, &currentSettings); err != nil {
+					return fmt.Errorf("could not parse existing cron job settings: %w", err)
+				}
 			}
 
 			if !terminal.IsInteractive() {
@@ -121,10 +123,10 @@ Examples:
 						defaultMethod = currentSettings.Method
 					}
 					methods := []string{"GET", "POST", "PUT", "DELETE", "PATCH", "HEAD"}
-					_ = defaultMethod // used as visual reference; select will show all options
 					selected, selErr := pterm.DefaultInteractiveSelect.
 						WithOptions(methods).
 						WithDefaultText("HTTP method").
+						WithDefaultOption(defaultMethod).
 						Show()
 					if selErr != nil {
 						return fmt.Errorf("could not read method: %w", selErr)

cmd/env/helpers.go

@@ -65,6 +65,7 @@ func resolveProjectEnv(c *cli.Context, client *api.APIClient) (string, *api.Envi
 				return projectID, &envs[i], nil
 			}
 		}
+		return "", nil, fmt.Errorf("no environment selected")
 	}
 
 	// envID was known — fetch the full object so callers have UniqueName etc.

cmd/scale/scale.go

@@ -98,6 +98,7 @@ func resolveProjectAndEnv(c *cli.Context, client *api.APIClient) (string, string
 				return projectID, envs[i].ID, nil
 			}
 		}
+		return "", "", fmt.Errorf("no environment selected")
 	}
 
 	return projectID, envID, nil