fix: fmt

Author: Davincible

cmd/code.go

@@ -48,6 +48,7 @@ func runCode(cmd *cobra.Command, args []string) error {
 
 	// Track reference count
 	procMgr.IncrementRef()
+
 	defer func() {
 		procMgr.DecrementRef()
 		// Only stop service if we started it and no more references
@@ -69,12 +70,14 @@ func runCode(cmd *cobra.Command, args []string) error {
 
 func filterEnv(env []string, key string) []string {
 	var filtered []string
+
 	prefix := key + "="
 	for _, e := range env {
 		if !startsWith(e, prefix) {
 			filtered = append(filtered, e)
 		}
 	}
+
 	return filtered
 }
 

cmd/config.go

@@ -64,23 +64,28 @@ func runConfigInit(cmd *cobra.Command, _ []string) error {
 
 	// Get provider details
 	fmt.Print("\nProvider Name (e.g., openrouter, openai): ")
+
 	providerName, _ := reader.ReadString('\n')
 	providerName = strings.TrimSpace(providerName)
 
 	fmt.Print("API Key: ")
+
 	apiKey, _ := reader.ReadString('\n')
 	apiKey = strings.TrimSpace(apiKey)
 
 	fmt.Print("API Base URL: ")
+
 	baseURL, _ := reader.ReadString('\n')
 	baseURL = strings.TrimSpace(baseURL)
 
 	fmt.Print("Default Model: ")
+
 	model, _ := reader.ReadString('\n')
 	model = strings.TrimSpace(model)
 
 	// Optional router API key
 	fmt.Print("Router API Key (optional, for authentication): ")
+
 	routerAPIKey, _ := reader.ReadString('\n')
 	routerAPIKey = strings.TrimSpace(routerAPIKey)
 
@@ -135,9 +140,11 @@ func runConfigShow(cmd *cobra.Command, _ []string) error {
 	if cfgMgr.HasYAML() {
 		configType = "YAML"
 	}
+
 	fmt.Printf("  %-15s: %s\n", "Format", configType)
 
 	fmt.Println("\nProviders:")
+
 	for _, provider := range cfg.Providers {
 		fmt.Printf("  - Name: %s\n", provider.Name)
 		fmt.Printf("    URL: %s\n", provider.APIBase)
@@ -146,26 +153,33 @@ func runConfigShow(cmd *cobra.Command, _ []string) error {
 		if len(provider.DefaultModels) > 0 {
 			fmt.Printf("    Default Models: %v\n", provider.DefaultModels)
 		}
+
 		if len(provider.ModelWhitelist) > 0 {
 			fmt.Printf("    Model Whitelist: %v\n", provider.ModelWhitelist)
 		}
+
 		if len(provider.Models) > 0 {
 			fmt.Printf("    Models: %v\n", provider.Models)
 		}
+
 		fmt.Println()
 	}
 
 	fmt.Println("Router Configuration:")
 	fmt.Printf("  %-15s: %s\n", "Default", cfg.Router.Default)
+
 	if cfg.Router.Think != "" {
 		fmt.Printf("  %-15s: %s\n", "Think", cfg.Router.Think)
 	}
+
 	if cfg.Router.Background != "" {
 		fmt.Printf("  %-15s: %s\n", "Background", cfg.Router.Background)
 	}
+
 	if cfg.Router.LongContext != "" {
 		fmt.Printf("  %-15s: %s\n", "Long Context", cfg.Router.LongContext)
 	}
+
 	if cfg.Router.WebSearch != "" {
 		fmt.Printf("  %-15s: %s\n", "Web Search", cfg.Router.WebSearch)
 	}
@@ -175,7 +189,7 @@ func runConfigShow(cmd *cobra.Command, _ []string) error {
 
 func runConfigValidate(cmd *cobra.Command, _ []string) error {
 	if !cfgMgr.Exists() {
-		return fmt.Errorf("no configuration found")
+		return errors.New("no configuration found")
 	}
 
 	cfg, err := cfgMgr.Load()
@@ -194,9 +208,11 @@ func runConfigValidate(cmd *cobra.Command, _ []string) error {
 		if provider.Name == "" {
 			errors = append(errors, fmt.Sprintf("provider %d: name is required", i))
 		}
+
 		if provider.APIBase == "" {
 			errors = append(errors, fmt.Sprintf("provider %d: API base URL is required", i))
 		}
+
 		if provider.APIKey == "" {
 			errors = append(errors, fmt.Sprintf("provider %d: API key is required", i))
 		}
@@ -208,13 +224,16 @@ func runConfigValidate(cmd *cobra.Command, _ []string) error {
 
 	if len(errors) > 0 {
 		color.Red("Configuration validation failed:")
+
 		for _, err := range errors {
 			fmt.Printf("  - %s\n", err)
 		}
-		return fmt.Errorf("configuration validation failed")
+
+		return errors.New("configuration validation failed")
 	}
 
 	color.Green("Configuration is valid!")
+
 	return nil
 }
 
@@ -230,6 +249,7 @@ func runConfigGenerate(cmd *cobra.Command, _ []string) error {
 
 		color.Yellow("Configuration file already exists (%s format): %s", configType, cfgMgr.GetPath())
 		color.Cyan("Use --force to overwrite, or 'cco config show' to view current config")
+
 		return nil
 	}
 
@@ -259,8 +279,10 @@ func maskString(s string) string {
 	if s == "" {
 		return "(not set)"
 	}
+
 	if len(s) <= 8 {
 		return strings.Repeat("*", len(s))
 	}
+
 	return s[:4] + strings.Repeat("*", len(s)-8) + s[len(s)-4:]
 }

cmd/root.go

@@ -34,6 +34,7 @@ func init() {
 
 	// Setup directories with backward compatibility
 	var err error
+
 	homeDir, err = os.UserHomeDir()
 	if err != nil {
 		logger.Error("Failed to get home directory", "error", err)
@@ -77,6 +78,7 @@ func getConfigDirectory(homeDir string) string {
 		color.Yellow("Using existing configuration directory: %s", oldDir)
 		color.Cyan("Consider migrating to the new directory: %s", newDir)
 		color.Cyan("You can do this by running: mv %s %s", oldDir, newDir)
+
 		return oldDir
 	}
 
@@ -97,6 +99,7 @@ func directoryHasConfig(dir string) bool {
 	if _, err := os.Stat(yamlConfig); err == nil {
 		return true
 	}
+
 	if _, err := os.Stat(jsonConfig); err == nil {
 		return true
 	}
@@ -140,14 +143,17 @@ func ensureConfigExists() error {
 			color.Green("No configuration file found, but CCO_API_KEY is set - using minimal configuration")
 			return nil
 		}
+
 		color.Yellow("Configuration not found, starting setup...")
+
 		return promptForConfig()
 	}
+
 	return nil
 }
 
 func promptForConfig() error {
 	// This will be implemented in the config command
 	fmt.Println("Please run 'cco config init' to set up your configuration")
-	return fmt.Errorf("configuration required")
+	return errors.New("configuration required")
 }

cmd/start.go

@@ -48,5 +48,6 @@ func runStart(cmd *cobra.Command, _ []string) error {
 
 	// Create and start server
 	srv := server.New(cfgMgr, logger)
+
 	return srv.Start()
 }

cmd/stop.go

@@ -32,5 +32,6 @@ func runStop(cmd *cobra.Command, _ []string) error {
 	procMgr.CleanupRef()
 
 	color.Green("Service stopped successfully")
+
 	return nil
 }

internal/config/config.go

@@ -122,12 +122,14 @@ func (m *Manager) createMinimalConfig() Config {
 }
 
 func (m *Manager) Load() (*Config, error) {
-	var cfg Config
-	var err error
+	var (
+		cfg Config
+		err error
+	)
 
 	// Check if CCO_API_KEY is set - if so, we can run without a config file
 	ccoAPIKey := os.Getenv("CCO_API_KEY")
-	
+
 	// Try YAML first (takes precedence)
 	if _, yamlErr := os.Stat(m.yamlPath); yamlErr == nil {
 		cfg, err = m.loadYAML()
@@ -152,6 +154,7 @@ func (m *Manager) Load() (*Config, error) {
 	}
 
 	m.configValue.Store(&cfg)
+
 	return &cfg, nil
 }
 
@@ -190,6 +193,7 @@ func (m *Manager) applyDefaults(cfg *Config) error {
 	if cfg.Port == 0 {
 		cfg.Port = DefaultPort
 	}
+
 	if cfg.Host == "" {
 		cfg.Host = DefaultHost
 	}
@@ -217,6 +221,7 @@ func (m *Manager) applyDefaults(cfg *Config) error {
 		if len(provider.ModelWhitelist) > 0 && len(provider.DefaultModels) > 0 {
 			// Filter default models based on whitelist
 			var filteredDefaults []string
+
 			for _, model := range provider.DefaultModels {
 				for _, whitelisted := range provider.ModelWhitelist {
 					if strings.Contains(model, whitelisted) || model == whitelisted {
@@ -225,6 +230,7 @@ func (m *Manager) applyDefaults(cfg *Config) error {
 					}
 				}
 			}
+
 			provider.DefaultModels = filteredDefaults
 		}
 	}
@@ -245,6 +251,7 @@ func (m *Manager) Get() *Config {
 			Port: DefaultPort,
 		}
 	}
+
 	return cfg
 }
 
@@ -264,6 +271,7 @@ func (m *Manager) Save(cfg *Config) error {
 	}
 
 	m.configValue.Store(cfg)
+
 	return nil
 }
 
@@ -282,6 +290,7 @@ func (m *Manager) SaveAsYAML(cfg *Config) error {
 	}
 
 	m.configValue.Store(cfg)
+
 	return nil
 }
 
@@ -300,6 +309,7 @@ func (m *Manager) SaveAsJSON(cfg *Config) error {
 	}
 
 	m.configValue.Store(cfg)
+
 	return nil
 }
 
@@ -308,6 +318,7 @@ func (m *Manager) GetPath() string {
 	if _, err := os.Stat(m.yamlPath); err == nil {
 		return m.yamlPath
 	}
+
 	return m.jsonPath
 }
 
@@ -322,6 +333,7 @@ func (m *Manager) GetJSONPath() string {
 func (m *Manager) Exists() bool {
 	_, yamlErr := os.Stat(m.yamlPath)
 	_, jsonErr := os.Stat(m.jsonPath)
+
 	return yamlErr == nil || jsonErr == nil
 }
 
@@ -396,6 +408,7 @@ func (p *Provider) IsModelAllowed(model string) bool {
 			return true
 		}
 	}
+
 	return false
 }
 
@@ -406,10 +419,12 @@ func (p *Provider) GetAllowedModels() []string {
 	}
 
 	var allowed []string
+
 	for _, model := range p.DefaultModels {
 		if p.IsModelAllowed(model) {
 			allowed = append(allowed, model)
 		}
 	}
+
 	return allowed
 }