fix: resolve UI alignment bug and eliminate code duplication - Fix file tree width calculation for consistent vertical divider alignment - Add formatFileLine helper function for uniform spacing - Move getFileIcon to utils/ui.go to eliminate duplication - Ensure consistent 2-character width for all file list entries - Update both main.go and ui/filelist.go to use shared utility

Author: Cod-e-Codes

main.go

@@ -325,7 +325,7 @@ func (m model) formatDirectoryPreview(dirName string, files []utils.FileInfo, di
 			icon = "📁"
 		} else {
 			// Get file icon based on extension
-			icon = getFileIcon(file.Extension)
+			icon = utils.GetFileIcon(file.Extension)
 		}
 
 		result.WriteString(fmt.Sprintf("  %s %s", icon, file.Path))
@@ -343,72 +343,6 @@ func (m model) formatDirectoryPreview(dirName string, files []utils.FileInfo, di
 	return result.String()
 }
 
-// getFileIcon returns an appropriate icon for the file extension
-func getFileIcon(ext string) string {
-	icons := map[string]string{
-		// Programming languages
-		".go":    "🐹",
-		".py":    "🐍",
-		".js":    "📄",
-		".ts":    "📘",
-		".jsx":   "⚛️",
-		".tsx":   "⚛️",
-		".rs":    "🦀",
-		".java":  "☕",
-		".c":     "📄",
-		".cpp":   "📄",
-		".cc":    "📄",
-		".h":     "📄",
-		".hpp":   "📄",
-		".cs":    "🔷",
-		".php":   "🐘",
-		".rb":    "💎",
-		".swift": "🍎",
-		".kt":    "📱",
-		".scala": "⚖️",
-
-		// Documentation and markup
-		".md":       "📝",
-		".markdown": "📝",
-		".txt":      "📄",
-		".rst":      "📜",
-
-		// Configuration files
-		".json":       "🔧",
-		".yaml":       "⚙️",
-		".yml":        "⚙️",
-		".toml":       "⚙️",
-		".ini":        "⚙️",
-		".cfg":        "⚙️",
-		".conf":       "⚙️",
-		".env":        "🌿",
-		".properties": "⚙️",
-
-		// Data files
-		".xml": "📋",
-		".csv": "📊",
-		".log": "📜",
-
-		// Shell and scripts
-		".sh":   "🐚",
-		".bash": "🐚",
-		".zsh":  "🐚",
-		".fish": "🐠",
-		".ps1":  "💻",
-		".bat":  "💻",
-		".cmd":  "💻",
-
-		// Executables
-		".exe": "⚙️",
-		".bin": "⚙️",
-	}
-
-	if icon, exists := icons[ext]; exists {
-		return icon
-	}
-	return "📄"
-}
-
 // handleFileSelection processes file selection and starts summarization if appropriate
 func (m *model) handleFileSelection(selected *utils.FileInfo) tea.Cmd {
 	if selected == nil {

ui/filelist.go

@@ -169,10 +169,10 @@ func (m FileListModel) View() string {
 	for i := startIdx; i < endIdx; i++ {
 		file := displayFiles[i]
 
-		// Cursor indicator
+		// Cursor indicator (ensure consistent 2-character width)
 		cursor := "  "
 		if m.cursor < len(m.files) && file.Path == m.files[m.cursor].Path {
-			cursor = "> "
+			cursor = "> " // Already 2 characters
 		}
 
 		// File type indicator
@@ -184,23 +184,28 @@ func (m FileListModel) View() string {
 				indicator = "📁"
 			}
 		} else {
-			indicator = getFileIcon(file.Extension)
+			indicator = utils.GetFileIcon(file.Extension)
 		}
 
-		// Selection indicator
+		// Selection indicator (ensure consistent 1-character width)
 		selected := " "
 		if file.Path == m.selected {
-			selected = "✓"
+			selected = "✓" // Single character
 		}
 
 		// File name (truncate if too long)
 		name := file.Path
-		maxNameLength := m.width - 15 // Account for indicators and padding
+
+		// Calculate consistent width accounting for all components
+		// cursor: 2 chars, brackets: 2 chars, selected: 1 char, space: 1 char, indicator: 2 chars, space: 1 char
+		// Total fixed width: 9 characters
+		maxNameLength := m.width - 9
 		if maxNameLength > 0 && len(name) > maxNameLength {
 			name = name[:maxNameLength-3] + "..."
 		}
 
-		line := fmt.Sprintf("%s[%s] %s %s", cursor, selected, indicator, name)
+		// Use helper function to ensure consistent width formatting
+		line := formatFileLine(cursor, selected, indicator, name)
 
 		if m.cursor < len(m.files) && file.Path == m.files[m.cursor].Path {
 			line = m.cursorStyle.Render(line)
@@ -225,107 +230,6 @@ func (m FileListModel) View() string {
 	return m.baseStyle.Render(content)
 }
 
-// getFileIcon returns an appropriate icon for the file extension
-func getFileIcon(ext string) string {
-	icons := map[string]string{
-		// Programming languages
-		".go":    "🐹",
-		".py":    "🐍",
-		".js":    "📄",
-		".ts":    "📘",
-		".jsx":   "⚛️",
-		".tsx":   "⚛️",
-		".rs":    "🦀",
-		".java":  "☕",
-		".c":     "📄",
-		".cpp":   "📄",
-		".cc":    "📄",
-		".h":     "📄",
-		".hpp":   "📄",
-		".cs":    "🔷",
-		".php":   "🐘",
-		".rb":    "💎",
-		".swift": "🍎",
-		".kt":    "📱",
-		".scala": "⚖️",
-
-		// Documentation and markup
-		".md":       "📝",
-		".markdown": "📝",
-		".txt":      "📄",
-		".rst":      "📜",
-		".tex":      "📰",
-
-		// Configuration files
-		".json":       "🔧",
-		".yaml":       "⚙️",
-		".yml":        "⚙️",
-		".toml":       "⚙️",
-		".ini":        "⚙️",
-		".cfg":        "⚙️",
-		".conf":       "⚙️",
-		".env":        "🌿",
-		".properties": "⚙️",
-
-		// Data files
-		".xml": "📋",
-		".csv": "📊",
-		".log": "📜",
-		".sql": "🗄️",
-
-		// Shell and scripts
-		".sh":   "🐚",
-		".bash": "🐚",
-		".zsh":  "🐚",
-		".fish": "🐠",
-		".ps1":  "💻",
-		".bat":  "💻",
-		".cmd":  "💻",
-
-		// Build and package files
-		".dockerfile": "🐳",
-		".makefile":   "🔨",
-		".gradle":     "🐘",
-		".pom":        "📦",
-		".package":    "📦",
-
-		// Web and frontend
-		".html": "🌐",
-		".htm":  "🌐",
-		".css":  "🎨",
-		".scss": "🎨",
-		".sass": "🎨",
-		".less": "🎨",
-
-		// Images
-		".png":  "🖼️",
-		".jpg":  "🖼️",
-		".jpeg": "🖼️",
-		".gif":  "🖼️",
-		".svg":  "🖼️",
-		".ico":  "🖼️",
-
-		// Archives
-		".zip": "📦",
-		".tar": "📦",
-		".gz":  "📦",
-		".rar": "📦",
-		".7z":  "📦",
-
-		// Executables
-		".exe": "⚙️",
-		".bin": "⚙️",
-		".deb": "📦",
-		".rpm": "📦",
-		".msi": "📦",
-	}
-
-	if icon, exists := icons[ext]; exists {
-		return icon
-	}
-	return "📄"
-}
-
 // Helper functions for min/max
 func min(a, b int) int {
 	if a < b {
@@ -340,3 +244,28 @@ func max(a, b int) int {
 	}
 	return b
 }
+
+// formatFileLine ensures consistent width formatting for file list entries
+func formatFileLine(cursor, selected, indicator, name string) string {
+	// Ensure cursor is exactly 2 characters
+	if len(cursor) < 2 {
+		cursor = cursor + strings.Repeat(" ", 2-len(cursor))
+	} else if len(cursor) > 2 {
+		cursor = cursor[:2]
+	}
+
+	// Ensure selected is exactly 1 character
+	if len(selected) < 1 {
+		selected = " "
+	} else if len(selected) > 1 {
+		selected = selected[:1]
+	}
+
+	// Ensure indicator has consistent spacing (add space if needed)
+	indicatorWithSpace := indicator
+	if len(indicator) < 2 {
+		indicatorWithSpace = indicator + " "
+	}
+
+	return fmt.Sprintf("%s[%s] %s %s", cursor, selected, indicatorWithSpace, name)
+}

utils/ui.go

@@ -0,0 +1,102 @@
+package utils
+
+// GetFileIcon returns an appropriate icon for the file extension
+func GetFileIcon(ext string) string {
+	icons := map[string]string{
+		// Programming languages
+		".go":    "🐹",
+		".py":    "🐍",
+		".js":    "📄",
+		".ts":    "📘",
+		".jsx":   "⚛️",
+		".tsx":   "⚛️",
+		".rs":    "🦀",
+		".java":  "☕",
+		".c":     "📄",
+		".cpp":   "📄",
+		".cc":    "📄",
+		".h":     "📄",
+		".hpp":   "📄",
+		".cs":    "🔷",
+		".php":   "🐘",
+		".rb":    "💎",
+		".swift": "🍎",
+		".kt":    "📱",
+		".scala": "⚖️",
+
+		// Documentation and markup
+		".md":       "📝",
+		".markdown": "📝",
+		".txt":      "📄",
+		".rst":      "📜",
+		".tex":      "📰",
+
+		// Configuration files
+		".json":       "🔧",
+		".yaml":       "⚙️",
+		".yml":        "⚙️",
+		".toml":       "⚙️",
+		".ini":        "⚙️",
+		".cfg":        "⚙️",
+		".conf":       "⚙️",
+		".env":        "🌿",
+		".properties": "⚙️",
+
+		// Data files
+		".xml": "📋",
+		".csv": "📊",
+		".log": "📜",
+		".sql": "🗄️",
+
+		// Shell and scripts
+		".sh":   "🐚",
+		".bash": "🐚",
+		".zsh":  "🐚",
+		".fish": "🐠",
+		".ps1":  "💻",
+		".bat":  "💻",
+		".cmd":  "💻",
+
+		// Build and package files
+		".dockerfile": "🐳",
+		".makefile":   "🔨",
+		".gradle":     "🐘",
+		".pom":        "📦",
+		".package":    "📦",
+
+		// Web and frontend
+		".html": "🌐",
+		".htm":  "🌐",
+		".css":  "🎨",
+		".scss": "🎨",
+		".sass": "🎨",
+		".less": "🎨",
+
+		// Images
+		".png":  "🖼️",
+		".jpg":  "🖼️",
+		".jpeg": "🖼️",
+		".gif":  "🖼️",
+		".svg":  "🖼️",
+		".ico":  "🖼️",
+
+		// Archives
+		".zip": "📦",
+		".tar": "📦",
+		".gz":  "📦",
+		".rar": "📦",
+		".7z":  "📦",
+
+		// Executables
+		".exe": "⚙️",
+		".bin": "⚙️",
+		".deb": "📦",
+		".rpm": "📦",
+		".msi": "📦",
+	}
+
+	if icon, exists := icons[ext]; exists {
+		return icon
+	}
+	return "📄"
+}