Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jun 13, 2025

  • Fix .rooignore not working for nested project folders
  • Add PR description file

cte added 2 commits June 13, 2025 09:46
- 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
@cte cte requested review from jr and mrubens as code owners June 13, 2025 17:13
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working labels Jun 13, 2025
@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Jun 13, 2025

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]


Generated with ❤️ by ellipsis.dev

@cte cte closed this Jun 13, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 13, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 13, 2025
@cte cte deleted the fix-rooignore-nested-projects branch June 13, 2025 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants