small

Author: maxdml

cmd/start.go

@@ -20,14 +20,14 @@ var startCmd = &cobra.Command{
 func runStart(cmd *cobra.Command, args []string) error {
 	// Check if config is loaded
 	if config == nil {
-		return fmt.Errorf("failed to load config: dbos-config.yaml not found")
+		return fmt.Errorf("no config provided")
 	}
 
 	if len(config.RuntimeConfig.Start) == 0 {
-		return fmt.Errorf("no start commands found in 'dbos-config.yaml'")
+		return fmt.Errorf("no start commands found in config file")
 	}
 
-	logger.Info("Executing start commands from 'dbos-config.yaml'")
+	logger.Info("Executing start commands from config file")
 
 	for _, command := range config.RuntimeConfig.Start {
 		logger.Info("Executing command", "command", command)
@@ -74,14 +74,14 @@ func runStart(cmd *cobra.Command, args []string) error {
 			}
 		case sig := <-sigChan:
 			logger.Info("Received signal, stopping...", "signal", sig)
-			
+
 			// Kill the process group on Unix-like systems
 			if runtime.GOOS != "windows" {
 				syscall.Kill(-process.Process.Pid, syscall.SIGTERM)
 			} else {
 				process.Process.Kill()
 			}
-			
+
 			// Wait a bit for graceful shutdown
 			select {
 			case <-done:
@@ -91,10 +91,10 @@ func runStart(cmd *cobra.Command, args []string) error {
 					syscall.Kill(-process.Process.Pid, syscall.SIGKILL)
 				}
 			}
-			
+
 			os.Exit(0)
 		}
 	}
 
 	return nil
-}
+}

cmd/utils.go

@@ -16,6 +16,7 @@ func maskPassword(dbURL string) string {
 	parsedURL, err := url.Parse(dbURL)
 	if err != nil {
 		// If we can't parse it, return the original (shouldn't happen with valid URLs)
+		logger.Warn("Failed to parse database URL", "error", err)
 		return dbURL
 	}
 
@@ -57,7 +58,7 @@ func getDBURL(_ *cobra.Command) (string, error) {
 		resolvedURL = envURL
 		source = "environment variable"
 	} else {
-		return "", fmt.Errorf("missing database URL: please set it using the --db-url flag, the DBOS_SYSTEM_DATABASE_URL environment variable, or in your dbos-config.yaml file")
+		return "", fmt.Errorf("missing database URL: please set it using the --db-url flag, your dbos-config.yaml file, or the DBOS_SYSTEM_DATABASE_URL environment variable")
 	}
 
 	// Log the database URL in verbose mode with masked password
@@ -71,12 +72,7 @@ func getDBURL(_ *cobra.Command) (string, error) {
 
 // createDBOSContext creates a new DBOS context with the provided database URL
 func createDBOSContext(dbURL string) (dbos.DBOSContext, error) {
-	// TODO consider removing the CLI flag app-name entirely?
 	appName := "dbos-cli"
-	if viper.IsSet("name") {
-		appName = viper.GetString("name")
-	}
-
 	ctx, err := dbos.NewDBOSContext(dbos.Config{
 		DatabaseURL: dbURL,
 		AppName:     appName,