fix test bug and path resolution bug

Author: j-east

src/services/ripgrep/__tests__/regex-search-files.gitignore.integration.spec.ts

@@ -62,10 +62,18 @@ vi.mock("child_process", () => {
 
 // Ensure fs/promises and file utils are mockable from tests
 // Provide explicit mock factory so readdir/readFile are defined vi.fn()
-vi.mock("fs/promises", () => ({
-	readdir: vi.fn(),
-	readFile: vi.fn(),
-}))
+vi.mock("fs/promises", () => {
+	const mockReaddir = vi.fn()
+	const mockReadFile = vi.fn()
+	return {
+		readdir: mockReaddir,
+		readFile: mockReadFile,
+		default: {
+			readdir: mockReaddir,
+			readFile: mockReadFile,
+		},
+	}
+})
 vi.mock("../../../utils/fs")
 // Mock fs so BaseIgnoreController's realpathSync won't touch the real filesystem
 vi.mock("fs", () => ({

src/services/ripgrep/index.ts

@@ -235,7 +235,9 @@ function formatResults(fileResults: SearchFileResult[], cwd: string): string {
 
 	// Group results by file name
 	fileResults.slice(0, MAX_RESULTS).forEach((file) => {
-		const relativeFilePath = path.relative(cwd, file.file)
+		// If file.file is already a relative path, use it as-is
+		// Otherwise, make it relative to cwd
+		const relativeFilePath = path.isAbsolute(file.file) ? path.relative(cwd, file.file) : file.file
 		if (!groupedResults[relativeFilePath]) {
 			groupedResults[relativeFilePath] = []