From 84213b17e172e2a073a82c28f6e60bc33daeb3fe Mon Sep 17 00:00:00 2001
From: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Date: Thu, 7 Aug 2025 12:09:41 -0300
Subject: [PATCH] fix: improve ignore

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
---
 internal/fsext/ls.go                          | 61 +++++++++----------
 internal/tui/components/chat/editor/editor.go |  2 +-
 2 files changed, 29 insertions(+), 34 deletions(-)

diff --git a/internal/fsext/ls.go b/internal/fsext/ls.go
index e800e921de74e41a9d46f4d1fafd6fc59bcf65a0..5bd45dc4da20bacd805a4f0165d9701e5d5acc6e 100644
--- a/internal/fsext/ls.go
+++ b/internal/fsext/ls.go
@@ -1,8 +1,12 @@
 package fsext
 
 import (
+	"bytes"
+	"io"
+	"log/slog"
 	"os"
 	"path/filepath"
+	"strings"
 
 	"github.com/charlievieth/fastwalk"
 	ignore "github.com/sabhiram/go-gitignore"
@@ -67,10 +71,8 @@ var CommonIgnorePatterns = []string{
 }
 
 type DirectoryLister struct {
-	gitignore    *ignore.GitIgnore
-	crushignore  *ignore.GitIgnore
-	commonIgnore *ignore.GitIgnore
-	rootPath     string
+	ignores  *ignore.GitIgnore
+	rootPath string
 }
 
 func NewDirectoryLister(rootPath string) *DirectoryLister {
@@ -78,26 +80,29 @@ func NewDirectoryLister(rootPath string) *DirectoryLister {
 		rootPath: rootPath,
 	}
 
-	// Load gitignore if it exists
-	gitignorePath := filepath.Join(rootPath, ".gitignore")
-	if _, err := os.Stat(gitignorePath); err == nil {
-		if gi, err := ignore.CompileIgnoreFile(gitignorePath); err == nil {
-			dl.gitignore = gi
-		}
-	}
+	dl.ignores = ignore.CompileIgnoreLines(append(CommonIgnorePatterns, strings.Split(parseIgnores(rootPath), "\n")...)...)
+
+	return dl
+}
 
-	// Load crushignore if it exists
-	crushignorePath := filepath.Join(rootPath, ".crushignore")
-	if _, err := os.Stat(crushignorePath); err == nil {
-		if ci, err := ignore.CompileIgnoreFile(crushignorePath); err == nil {
-			dl.crushignore = ci
+func parseIgnores(path string) string {
+	var b bytes.Buffer
+	for _, ign := range []string{".crushignore", ".gitignore"} {
+		p := filepath.Join(path, ign)
+		if _, err := os.Stat(p); err == nil {
+			f, err := os.Open(p)
+			if err != nil {
+				_ = f.Close()
+				slog.Error("Failed to open ignore file", "path", p, "error", err)
+				continue
+			}
+			if _, err := io.Copy(&b, f); err != nil {
+				slog.Error("Failed to read ignore file", "path", p, "error", err)
+			}
+			_ = f.Close()
 		}
 	}
-
-	// Create common ignore patterns
-	dl.commonIgnore = ignore.CompileIgnoreLines(CommonIgnorePatterns...)
-
-	return dl
+	return b.String()
 }
 
 func (dl *DirectoryLister) shouldIgnore(path string, ignorePatterns []string) bool {
@@ -106,18 +111,7 @@ func (dl *DirectoryLister) shouldIgnore(path string, ignorePatterns []string) bo
 		relPath = path
 	}
 
-	// Check common ignore patterns
-	if dl.commonIgnore.MatchesPath(relPath) {
-		return true
-	}
-
-	// Check gitignore patterns if available
-	if dl.gitignore != nil && dl.gitignore.MatchesPath(relPath) {
-		return true
-	}
-
-	// Check crushignore patterns if available
-	if dl.crushignore != nil && dl.crushignore.MatchesPath(relPath) {
+	if dl.ignores.MatchesPath(relPath) {
 		return true
 	}
 
@@ -144,6 +138,7 @@ func ListDirectory(initialPath string, ignorePatterns []string, limit int) ([]st
 		ToSlash: fastwalk.DefaultToSlash(),
 		Sort:    fastwalk.SortDirsFirst,
 	}
+
 	err := fastwalk.Walk(&conf, initialPath, func(path string, d os.DirEntry, err error) error {
 		if err != nil {
 			return nil // Skip files we don't have permission to access
diff --git a/internal/tui/components/chat/editor/editor.go b/internal/tui/components/chat/editor/editor.go
index 4c7df84daad5b911be66dcd7f7cf6d832d714293..52884847ba65d96c24a939b227061fd4cc556f1a 100644
--- a/internal/tui/components/chat/editor/editor.go
+++ b/internal/tui/components/chat/editor/editor.go
@@ -473,7 +473,7 @@ func (m *editorCmp) SetPosition(x, y int) tea.Cmd {
 }
 
 func (m *editorCmp) startCompletions() tea.Msg {
-	files, _, _ := fsext.ListDirectory(".", []string{}, 0)
+	files, _, _ := fsext.ListDirectory(".", nil, 0)
 	completionItems := make([]completions.Completion, 0, len(files))
 	for _, file := range files {
 		file = strings.TrimPrefix(file, "./")