Fix .rooignore not working for nested project folders

- Modified DirectoryScanner to accept workspace root parameter
- RooIgnoreController now uses workspace root instead of scan directory
- This ensures .rooignore files in workspace root properly filter nested project files
- Fixes issue #4647 where .next folders in nested Next.js projects were being indexed

Changes:
- src/services/code-index/processors/scanner.ts: Added workspaceRoot parameter
- src/services/code-index/service-factory.ts: Pass workspacePath to scanner
- src/services/code-index/processors/__tests__/scanner.spec.ts: Updated test constructor

Author: cte

roo-code-messages.log

@@ -102,6 +102,7 @@ describe("DirectoryScanner", () => {
 			mockCodeParser,
 			mockCacheManager,
 			mockIgnoreInstance,
+			"/mock/workspace", // Add workspace root parameter
 		)
 
 		// Mock default implementations - create proper Stats object

src/services/code-index/processors/__tests__/scanner.spec.ts

@@ -30,6 +30,7 @@ export class DirectoryScanner implements IDirectoryScanner {
 		private readonly codeParser: ICodeParser,
 		private readonly cacheManager: CacheManager,
 		private readonly ignoreInstance: Ignore,
+		private readonly workspaceRoot?: string,
 	) {}
 
 	/**
@@ -53,8 +54,10 @@ export class DirectoryScanner implements IDirectoryScanner {
 		// Filter out directories (marked with trailing '/')
 		const filePaths = allPaths.filter((p) => !p.endsWith("/"))
 
-		// Initialize RooIgnoreController if not provided
-		const ignoreController = new RooIgnoreController(directoryPath)
+		// Initialize RooIgnoreController with workspace root (not the directory being scanned)
+		// This ensures .rooignore files are always looked for in the workspace root
+		const rooIgnoreWorkingDirectory = this.workspaceRoot || directoryPath
+		const ignoreController = new RooIgnoreController(rooIgnoreWorkingDirectory)
 
 		await ignoreController.initialize()
 

src/services/code-index/processors/scanner.ts

@@ -110,7 +110,14 @@ export class CodeIndexServiceFactory {
 		parser: ICodeParser,
 		ignoreInstance: Ignore,
 	): DirectoryScanner {
-		return new DirectoryScanner(embedder, vectorStore, parser, this.cacheManager, ignoreInstance)
+		return new DirectoryScanner(
+			embedder,
+			vectorStore,
+			parser,
+			this.cacheManager,
+			ignoreInstance,
+			this.workspacePath,
+		)
 	}
 
 	/**

src/services/code-index/service-factory.ts

@@ -0,0 +1,53 @@
+// Test script to reproduce the .rooignore issue with nested project folders
+const path = require("path")
+
+// Simulate the issue scenario
+const rootFolder = "/Users/test/Root Folder"
+const nextjsProject = "/Users/test/Root Folder/example-nextjs"
+const nextFolder = "/Users/test/Root Folder/example-nextjs/.next"
+
+// Test the path normalization logic from RooIgnoreController
+function testPathNormalization(cwd, filePath) {
+	console.log(`\nTesting path normalization:`)
+	console.log(`CWD: ${cwd}`)
+	console.log(`File path: ${filePath}`)
+
+	// This is the logic from RooIgnoreController.validateAccess()
+	const absolutePath = path.resolve(cwd, filePath)
+	console.log(`Absolute path: ${absolutePath}`)
+
+	const relativePath = path.relative(cwd, absolutePath)
+	console.log(`Relative path: ${relativePath}`)
+
+	// Convert to POSIX (forward slashes)
+	const posixPath = relativePath.replace(/\\/g, "/")
+	console.log(`POSIX path: ${posixPath}`)
+
+	return posixPath
+}
+
+// Test case 1: .rooignore in root folder, trying to ignore nested .next folder
+console.log("=== Test Case 1: .rooignore in Root Folder ===")
+const rooignorePattern = "example-nextjs/.next/"
+const fileToCheck = "example-nextjs/.next/server/pages/index.js"
+
+const normalizedPath = testPathNormalization(rootFolder, fileToCheck)
+console.log(`\nPattern in .rooignore: ${rooignorePattern}`)
+console.log(`Normalized file path: ${normalizedPath}`)
+console.log(`Should match pattern: ${normalizedPath.startsWith(rooignorePattern.replace(/\/$/, ""))}`)
+
+// Test case 2: What if the file path is already absolute?
+console.log("\n=== Test Case 2: Absolute file path ===")
+const absoluteFileToCheck = path.join(nextjsProject, ".next/server/pages/index.js")
+const normalizedPath2 = testPathNormalization(rootFolder, absoluteFileToCheck)
+console.log(`\nPattern in .rooignore: ${rooignorePattern}`)
+console.log(`Normalized file path: ${normalizedPath2}`)
+console.log(`Should match pattern: ${normalizedPath2.startsWith(rooignorePattern.replace(/\/$/, ""))}`)
+
+// Test case 3: What if we're scanning from the nextjs project directory?
+console.log("\n=== Test Case 3: Scanning from nextjs project directory ===")
+const fileInNext = ".next/server/pages/index.js"
+const normalizedPath3 = testPathNormalization(nextjsProject, fileInNext)
+console.log(`\nIf .rooignore was in nextjs project with pattern: .next/`)
+console.log(`Normalized file path: ${normalizedPath3}`)
+console.log(`Should match pattern: ${normalizedPath3.startsWith(".next/")}`)