fix: remove unnecessary nil checks in configimport

Fixes staticcheck S1009 and S1031 lint errors:
- len() for nil maps/slices is defined as zero
- range over nil map is a no-op

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: Dumbris

internal/configimport/claude_code.go

@@ -38,7 +38,7 @@ func (p *ClaudeCodeParser) Parse(content []byte) ([]*ParsedServer, error) {
 		}
 	}
 
-	if cfg.MCPServers == nil || len(cfg.MCPServers) == 0 {
+	if len(cfg.MCPServers) == 0 {
 		return nil, &ImportError{
 			Type:    "no_servers",
 			Message: "no MCP servers found in Claude Code config",

internal/configimport/claude_desktop.go

@@ -36,7 +36,7 @@ func (p *ClaudeDesktopParser) Parse(content []byte) ([]*ParsedServer, error) {
 		}
 	}
 
-	if cfg.MCPServers == nil || len(cfg.MCPServers) == 0 {
+	if len(cfg.MCPServers) == 0 {
 		return nil, &ImportError{
 			Type:    "no_servers",
 			Message: "no MCP servers found in Claude Desktop config",

internal/configimport/codex.go

@@ -57,7 +57,7 @@ func (p *CodexParser) Parse(content []byte) ([]*ParsedServer, error) {
 		}
 	}
 
-	if cfg.MCPServers == nil || len(cfg.MCPServers) == 0 {
+	if len(cfg.MCPServers) == 0 {
 		return nil, &ImportError{
 			Type:    "no_servers",
 			Message: "no MCP servers found in Codex config (looking for [mcp_servers.*] sections)",

internal/configimport/cursor.go

@@ -48,7 +48,7 @@ func (p *CursorParser) Parse(content []byte) ([]*ParsedServer, error) {
 		}
 	}
 
-	if cfg.MCPServers == nil || len(cfg.MCPServers) == 0 {
+	if len(cfg.MCPServers) == 0 {
 		return nil, &ImportError{
 			Type:    "no_servers",
 			Message: "no MCP servers found in Cursor config",

internal/configimport/gemini.go

@@ -72,7 +72,7 @@ func (p *GeminiParser) Parse(content []byte) ([]*ParsedServer, error) {
 		}
 	}
 
-	if cfg.MCPServers == nil || len(cfg.MCPServers) == 0 {
+	if len(cfg.MCPServers) == 0 {
 		return nil, &ImportError{
 			Type:    "no_servers",
 			Message: "no MCP servers found in Gemini config",

internal/configimport/import.go

@@ -129,11 +129,9 @@ func Import(content []byte, opts *ImportOptions) (*ImportResult, error) {
 	}
 
 	// Validate filter - check if requested servers were found
-	if filterSet != nil {
-		for name := range filterSet {
-			if !foundServers[name] {
-				result.Warnings = append(result.Warnings, fmt.Sprintf("requested server '%s' not found in config", name))
-			}
+	for name := range filterSet {
+		if !foundServers[name] {
+			result.Warnings = append(result.Warnings, fmt.Sprintf("requested server '%s' not found in config", name))
 		}
 	}