refactor: remove unused randomSlugSuffix logic and update comments

Author: ramya18101

internal/cli/quickstart_detect.go

@@ -35,15 +35,15 @@ func DetectProject(dir string) DetectionResult {
 	// Read package.json deps early - needed for checks that must precede file-based signals.
 	earlyDeps := readPackageJSONDeps(dir)
 
-	// -- 1. Manage.py (Django) — checked before Ionic to prevent monorepo misdetection.
+	// -- 1. Manage.py (Django) — Checked before Ionic to prevent monorepo misdetection.
 	if fileExists(dir, "manage.py") {
 		result.Framework = "django"
 		result.Type = "regular"
 		result.Detected = true
 		return result
 	}
 
-	// -- 2. Ionic (package.json deps - must check BEFORE angular.json and vite.config) --.
+	// -- 2. Ionic (package.json deps - Must check BEFORE angular.json and vite.config) --.
 	if hasDep(earlyDeps, "@ionic/angular") {
 		result.Framework = "ionic-angular"
 		result.Type = "native"
@@ -80,7 +80,7 @@ func DetectProject(dir string) DetectionResult {
 	if data, ok := readFileContent(dir, "pubspec.yaml"); ok {
 		if strings.Contains(data, "sdk: flutter") {
 			result.Detected = true
-			// android/ios dirs signal native; flutter.web key signals web SPA; default native.
+			// Android/ios dirs signal native; flutter.web key signals web SPA; default native.
 			switch {
 			case dirExists(dir, "android") || dirExists(dir, "ios"):
 				result.Framework = "flutter"
@@ -99,7 +99,7 @@ func DetectProject(dir string) DetectionResult {
 		}
 	}
 
-	// -- 5. composer.json (PHP) — before vite.config to avoid Laravel misdetection.
+	// Composer.json (PHP) — Before vite.config to avoid Laravel misdetection.
 	if data, ok := readFileContent(dir, "composer.json"); ok {
 		result.BuildTool = "composer"
 		result.Type = "regular"
@@ -112,7 +112,7 @@ func DetectProject(dir string) DetectionResult {
 		return result
 	}
 
-	// -- 6. SvelteKit (@sveltejs/kit dep) — before vite.config; only reliable distinguishing signal.
+	// -- 6. SvelteKit (@sveltejs/kit dep) — Before vite.config; only reliable distinguishing signal.
 	if hasDep(earlyDeps, "@sveltejs/kit") {
 		result.Framework = "sveltekit"
 		result.Type = "regular"
@@ -121,7 +121,7 @@ func DetectProject(dir string) DetectionResult {
 		return result
 	}
 
-	// -- 7. nuxt.config.[ts|js] — before vite.config (Nuxt uses Vite internally).
+	// Nuxt.config.[ts|js] — Before vite.config (Nuxt uses Vite internally).
 	if fileExistsAny(dir, "nuxt.config.ts", "nuxt.config.js") {
 		result.Framework = "nuxt"
 		result.Type = "regular"
@@ -155,7 +155,7 @@ func DetectProject(dir string) DetectionResult {
 		return result
 	}
 
-	// -- 10. svelte.config.[js|ts] — only label as sveltekit when @sveltejs/kit dep is confirmed.
+	// -- 10. svelte.config.[js|ts] — Only label as sveltekit when @sveltejs/kit dep is confirmed.
 	if fileExistsAny(dir, "svelte.config.js", "svelte.config.ts") {
 		framework := "sveltekit"
 		appType := "regular"
@@ -192,7 +192,7 @@ func DetectProject(dir string) DetectionResult {
 		}
 	}
 
-	// -- 13. Android (native Java/Kotlin) — before Java build files to avoid vanilla-java misdetection.
+	// -- 13. Android (native Java/Kotlin) — Before Java build files to avoid vanilla-java misdetection.
 	// Excludes React Native projects which also have AndroidManifest.xml in a sub-project.
 	if fileExists(dir, filepath.Join("app", "src", "main", "AndroidManifest.xml")) && !hasDep(earlyDeps, "react-native") {
 		result.Framework = "android"
@@ -203,7 +203,7 @@ func DetectProject(dir string) DetectionResult {
 		return result
 	}
 
-	// -- 14. iOS Swift — .xcodeproj or Package.swift (excludes Vapor server-side Swift).
+	// -- 14. iOS Swift — .xcodeproj or Package.swift (Excludes Vapor server-side Swift).
 	if hasXcodeprojDir(dir) || (fileExists(dir, "Package.swift") && !isVaporSwiftPackage(dir)) {
 		result.Framework = "ios-swift"
 		result.Type = "native"
@@ -258,7 +258,7 @@ func DetectProject(dir string) DetectionResult {
 		}
 	}
 
-	// -- 19. Package.json dep scanning (lowest priority) --
+	// -- 19. Package.json dep scanning (Lowest priority) --
 	// Note: Ionic deps are already handled above (step 2).
 	if len(earlyDeps) > 0 {
 		candidates := collectPackageJSONCandidates(earlyDeps)

internal/cli/quickstarts.go

@@ -2,9 +2,7 @@ package cli
 
 import (
 	"context"
-	"crypto/rand"
 	_ "embed"
-	"encoding/hex"
 	"encoding/json"
 	"fmt"
 	"net/url"
@@ -16,7 +14,6 @@ import (
 	"sort"
 	"strconv"
 	"strings"
-	"time"
 
 	"github.com/auth0/go-auth0/management"
 	"github.com/spf13/cobra"
@@ -822,22 +819,13 @@ func collectName(cmd *cobra.Command, inputs *SetupInputs) error {
 	return nil
 }
 
-// collectAPIInputs prompts for identifier, token lifetime, and signing algorithm.
 func collectAPIInputs(cmd *cobra.Command, cli *cli, inputs *SetupInputs) error {
 	// Identifier.
 	if !setupIdentifier.IsSet(cmd) {
 		defaultID := inputs.Identifier
 		if defaultID == "" && inputs.Name != "" {
 			slug := strings.ToLower(strings.ReplaceAll(inputs.Name, " ", "-"))
 			defaultID = "https://" + slug
-
-			// Check once if the clean slug is taken; if so, append a random suffix
-			// so the rerun doesn't collide. Respects user input — only suggests, doesn't force.
-			if _, err := cli.api.ResourceServer.Read(cmd.Context(), url.PathEscape(defaultID)); err == nil {
-				suffixed := fmt.Sprintf("%s-%s", defaultID, randomSlugSuffix(4))
-				cli.renderer.Warnf("API identifier %q is already in use. Generated a new identifier %q to avoid conflict.", defaultID, suffixed)
-				defaultID = suffixed
-			}
 		}
 		inputs.Identifier = defaultID
 		if err := setupIdentifier.Ask(cmd, &inputs.Identifier, &defaultID); err != nil {
@@ -1436,17 +1424,6 @@ func validateAPIIdentifier(identifier string) error {
 	return nil
 }
 
-// randomSlugSuffix returns a lowercase hex string of the requested byte length
-// (each byte expands to 2 hex chars). Falls back to a timestamp-based suffix if
-// the crypto source fails, which should not happen in practice.
-func randomSlugSuffix(n int) string {
-	b := make([]byte, n)
-	if _, err := rand.Read(b); err != nil {
-		return strconv.FormatInt(time.Now().UnixNano(), 36)
-	}
-	return hex.EncodeToString(b)
-}
-
 func generateClient(input SetupInputs, reqParams auth0.RequestParams) (*management.Client, error) {
 	if input.Name == "" {
 		input.Name = "My App"