fix: upload bugs

Author: BhautikChudasama

cmd/projects/add.go

@@ -125,7 +125,10 @@ all required flags:
 					if err != nil {
 						return fmt.Errorf("could not read input: %w", err)
 					}
-					displayName = result
+					displayName = strings.TrimSpace(result)
+					if displayName == "" {
+						return fmt.Errorf("display name is required — please provide a name for your project")
+					}
 				}
 
 				if uniqueName == "" {
@@ -135,7 +138,22 @@ all required flags:
 					if err != nil {
 						return fmt.Errorf("could not read input: %w", err)
 					}
-					uniqueName = result
+					uniqueName = strings.TrimSpace(result)
+					if uniqueName == "" {
+						return fmt.Errorf("unique name is required — must be lowercase, 4-32 characters")
+					}
+					if len(uniqueName) < 4 || len(uniqueName) > 32 {
+						return fmt.Errorf("unique name must be between 4 and 32 characters (got %d)", len(uniqueName))
+					}
+					if uniqueName != strings.ToLower(uniqueName) {
+						return fmt.Errorf("unique name must be lowercase — try %q", strings.ToLower(uniqueName))
+					}
+					available, err := client.CheckUniqueNameAvailable(uniqueName)
+					if err != nil {
+						pterm.Warning.Printf("Could not verify name availability: %s\n", err)
+					} else if !available {
+						return fmt.Errorf("the unique name %q is already taken — please choose a different one", uniqueName)
+					}
 				}
 
 				if projectType == "" {
@@ -519,11 +537,12 @@ func promptFrameworkRuntime(client *api.APIClient, projectType string) (framewor
 	}
 	sel.runtime = runtimeValue
 
-	// For upload projects, standalone runtimes (dockerfile, build-ai) are fine alone.
-	// Regular runtimes need a framework too.
+	// For upload projects, some runtimes (like nodejs) have compatible frameworks.
+	// If compatible frameworks exist, prompt the user to pick one.
+	// Standalone runtimes (dockerfile, build-ai) and runtimes with no compatible
+	// frameworks (golang, rust, python, bun) are fine alone.
 	standaloneRuntimes := map[string]bool{"dockerfile": true, "build-ai": true}
 	if projectType == "upload" && !standaloneRuntimes[entry.Name] {
-		// Need to also pick a framework that's compatible with this runtime
 		var frameworks []api.SupportedProjectType
 		for _, s := range supported {
 			if s.Type != "framework" {
@@ -537,30 +556,28 @@ func promptFrameworkRuntime(client *api.APIClient, projectType string) (framewor
 			}
 		}
 
-		if len(frameworks) == 0 {
-			return frameworkSelection{}, fmt.Errorf("no compatible frameworks found for runtime %q\n\n  For upload projects, try selecting a framework instead", entry.Name)
-		}
-
-		fwOptions := make([]string, len(frameworks))
-		for i, fw := range frameworks {
-			fwOptions[i] = fw.Name
-		}
+		if len(frameworks) > 0 {
+			fwOptions := make([]string, len(frameworks))
+			for i, fw := range frameworks {
+				fwOptions[i] = fw.Name
+			}
 
-		fwSelected, err := pterm.DefaultInteractiveSelect.
-			WithDefaultText("Framework (required for upload projects)").
-			WithOptions(fwOptions).
-			WithFilter(true).
-			Show()
-		if err != nil {
-			return frameworkSelection{}, fmt.Errorf("could not read input: %w", err)
-		}
+			fwSelected, err := pterm.DefaultInteractiveSelect.
+				WithDefaultText("Framework (required for upload projects)").
+				WithOptions(fwOptions).
+				WithFilter(true).
+				Show()
+			if err != nil {
+				return frameworkSelection{}, fmt.Errorf("could not read input: %w", err)
+			}
 
-		sel.framework = fwSelected
-		// Use the framework's editables since they're more specific
-		for _, fw := range frameworks {
-			if fw.Name == fwSelected {
-				sel.editables = fw.Editables
-				break
+			sel.framework = fwSelected
+			// Use the framework's editables since they're more specific
+			for _, fw := range frameworks {
+				if fw.Name == fwSelected {
+					sel.editables = fw.Editables
+					break
+				}
 			}
 		}
 	}

cmd/projects/get.go

@@ -13,6 +13,18 @@ import (
 	"github.com/NodeOps-app/createos-cli/internal/output"
 )
 
+// parseVCSSource extracts the repository full name from a VCS project's source field.
+func parseVCSSource(raw json.RawMessage) string {
+	if len(raw) == 0 {
+		return ""
+	}
+	var src api.VCSSource
+	if err := json.Unmarshal(raw, &src); err != nil {
+		return ""
+	}
+	return src.VCSFullName
+}
+
 func newGetCommand() *cli.Command {
 	return &cli.Command{
 		Name:  "get",
@@ -51,6 +63,13 @@ func newGetCommand() *cli.Command {
 			cyan.Printf("Type:        ")
 			fmt.Println(project.Type)
 
+			if project.Type == "vcs" {
+				if repo := parseVCSSource(project.Source); repo != "" {
+					cyan.Printf("Repository:  ")
+					fmt.Println(repo)
+				}
+			}
+
 			cyan.Printf("Description: ")
 			if project.Description != nil {
 				fmt.Println(*project.Description)
@@ -64,9 +83,6 @@ func newGetCommand() *cli.Command {
 			cyan.Printf("Created At:  ")
 			fmt.Println(project.CreatedAt.Format("2006-01-02 15:04:05"))
 
-			cyan.Printf("Updated At:  ")
-			fmt.Println(project.UpdatedAt.Format("2006-01-02 15:04:05"))
-
 			return nil
 		},
 	}

internal/api/methods.go

@@ -9,14 +9,21 @@ import (
 
 // Project represents a CreateOS project
 type Project struct {
-	ID          string    `json:"id"`
-	UniqueName  string    `json:"uniqueName"`
-	DisplayName string    `json:"displayName"`
-	Description *string   `json:"description"`
-	Status      string    `json:"status"`
-	Type        string    `json:"type"`
-	CreatedAt   time.Time `json:"createdAt"`
-	UpdatedAt   time.Time `json:"updatedAt"`
+	ID          string          `json:"id"`
+	UniqueName  string          `json:"uniqueName"`
+	DisplayName string          `json:"displayName"`
+	Description *string         `json:"description"`
+	Status      string          `json:"status"`
+	Type        string          `json:"type"`
+	Source      json.RawMessage `json:"source,omitempty"`
+	CreatedAt   time.Time       `json:"createdAt"`
+	UpdatedAt   time.Time       `json:"updatedAt"`
+}
+
+// VCSSource represents the source details for a VCS project.
+type VCSSource struct {
+	VCSName     string `json:"vcsName"`
+	VCSFullName string `json:"vcsFullName"`
 }
 
 // CreateProjectRequest is the request body for creating a new project.
@@ -47,6 +54,24 @@ func (c *APIClient) CreateProject(req CreateProjectRequest) (string, error) {
 	return result.Data.ID, nil
 }
 
+// CheckUniqueNameAvailable checks whether a project unique name is available.
+func (c *APIClient) CheckUniqueNameAvailable(uniqueName string) (bool, error) {
+	var result Response[struct {
+		IsAvailable bool `json:"isAvailable"`
+	}]
+	resp, err := c.Client.R().
+		SetResult(&result).
+		SetBody(map[string]string{"uniqueName": uniqueName}).
+		Post("/v1/projects/available-unique-name")
+	if err != nil {
+		return false, err
+	}
+	if resp.IsError() {
+		return false, ParseAPIError(resp.StatusCode(), resp.Body())
+	}
+	return result.Data.IsAvailable, nil
+}
+
 // GithubInstallation represents a connected GitHub account.
 type GithubInstallation struct {
 	InstallationID int64  `json:"installationId"`