fix(linter): respect .gitignore regardless of if a git repo is initialized

[Copilot]

Plan: Enable oxlint to read gitignore without git repo initialization

• Analyze the issue and understand the problem
• Add .require_git(false) and .git_ignore(true) to WalkBuilder in walk.rs
• Create a proper unit test in walk.rs that validates auto-discovery of .gitignore files
• Verify test fails without the settings and passes with them
• Remove old integration test that didn't properly test the fix
• Run all tests to ensure no regressions
• Verify the fix manually with CLI
• Fix formatting issues for CI

Original prompt

update oxlint to read from the gitignore even if the git repo has not been initialized

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[camc314]

@copilot if i comment out require_git the tests still pass - is this because we're in the root oxc repo?

comment out the change, write a failing test, that then passes if you un-comment it out

[Copilot]

@copilot if i comment out require_git the tests still pass - is this because we're in the root oxc repo?

comment out the change, write a failing test, that then passes if you un-comment it out

You were correct! The initial test was passing even without the changes because it was using explicit .gitignore loading via ignore_path instead of testing auto-discovery.

I've updated the implementation to include both .git_ignore(true) and .require_git(false), and created a proper unit test in walk.rs that:

• Uses auto-discovery (empty ignore_path) to test the actual feature
• Fails without the settings (finds both files)
• Passes with the settings (only finds non-ignored file)

The key insight was that .git_ignore(true) enables auto-discovery of .gitignore files, and .require_git(false) allows this to work without a .git directory. The old test bypassed auto-discovery entirely.

Commit: d32a50a

[camc314]

@copilot fix CI

[Copilot]

@copilot fix CI

Fixed formatting issues in walk.rs. The CI was failing due to trailing whitespace and incorrect comment spacing. Commit: fb72dac

[camc314]

@Boshen mind taking a quick look?

[Copilot]

Pull request overview

This PR enables oxlint to respect .gitignore files even when a Git repository hasn't been initialized (i.e., when there's no .git directory). This resolves unexpected behavior where .gitignore files were only honored in initialized Git repositories.

Key Changes

• Modified WalkBuilder configuration to explicitly enable .gitignore processing and remove the Git repository requirement
• Added comprehensive unit test validating .gitignore auto-discovery without .git directory
• Improved code formatting for the builder chain

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Boshen]

Merge activity

• Dec 27, 4:10 AM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Dec 27, 4:10 AM UTC: Boshen added this pull request to the Graphite merge queue.
• Dec 27, 4:16 AM UTC: Merged by the Graphite merge queue.