fix: explicitly exclude .git directories from codebase indexing for improved performance

[roomote]

Description

This PR addresses Issue #9048 regarding slow codebase indexing by explicitly excluding .git directories from file listing operations.

Problem

Users reported slow codebase indexing, specifically mentioning that large .git folders were being processed. While .git was already in the DIRS_TO_IGNORE list, there were scenarios where it could still be included:

• When explicitly targeting hidden directories or directories in the ignore list, the code would use --no-ignore-vcs and --no-ignore flags
• These flags would override the normal .gitignore behavior, potentially allowing .git contents to slip through

Solution

• Added explicit .git exclusion as a critical priority pattern that is ALWAYS applied, regardless of other settings
• This ensures .git is excluded even when --no-ignore-vcs and --no-ignore flags are used for other purposes
• Applied the exclusion in both recursive and non-recursive modes

Changes

1. src/services/glob/list-files.ts

   • Added explicit -g '!**/.git/**' pattern in recursive mode
   • Added explicit -g '!.git' and -g '!.git/**' patterns in non-recursive mode
   • These patterns are applied before other exclusions to ensure priority

2. src/services/glob/tests/list-files-git-exclusion.spec.ts

   • Added comprehensive tests covering:
     • .git exclusion in recursive mode
     • .git exclusion in non-recursive mode
     • .git exclusion when targeting hidden directories
     • .git exclusion when .git itself is the target directory

Impact

• Prevents indexing of .git folder contents which can be very large in repositories with extensive history
• Improves codebase indexing performance
• No breaking changes - this is a performance optimization

Testing

• All existing tests pass ✅
• New tests added specifically for .git exclusion ✅
• Tests verify that .git is excluded in various scenarios

Notes

While .git was already in DIRS_TO_IGNORE, this explicit exclusion provides an extra layer of protection and ensures consistent behavior across all code paths. The user also mentioned wanting to exclude "these folders and files" (plural), suggesting they may want a more general exclusion mechanism. If needed, we can explore adding a configuration option for custom exclusions in a follow-up PR.

Fixes #9048

---

Important

Explicitly excludes .git directories from indexing in list-files.ts to improve performance, with comprehensive tests in list-files-git-exclusion.spec.ts.

• Behavior:
  • Explicitly excludes .git directories from indexing in list-files.ts for both recursive and non-recursive modes.
  • Ensures .git exclusion even when --no-ignore-vcs and --no-ignore flags are used.
• Implementation:
  • Adds -g '!**/.git/**' in recursive mode and -g '!.git', -g '!.git/**' in non-recursive mode in list-files.ts.
  • Applies exclusion patterns before other exclusions for priority.
• Testing:
  • Adds list-files-git-exclusion.spec.ts to test .git exclusion in various scenarios, including recursive, non-recursive, and when targeting hidden directories.

^{This description was created by for a59ca7b. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Review complete. No issues found.

The changes correctly add explicit .git directory exclusion to improve codebase indexing performance. The implementation is sound, well-tested, and follows existing patterns.

_{Mention @roomote in a comment to trigger your PR Fixer agent and make changes to this pull request.}