Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Dec 21, 2025

Important

Adds support for loading custom tools from multiple directories, including global directories, with UI and test updates.

  • Behavior:
    • CustomToolRegistry now supports loading tools from multiple directories with loadFromDirectories() and loadFromDirectoriesIfStale().
    • Tools can be overridden by later directories in the list.
    • Handles non-existent directories and empty directory arrays gracefully.
  • UI:
    • Updates CustomToolsSettings.tsx to display tool source paths.
    • Adds translations for new functionality in multiple language files.
  • Tests:
    • Adds tests in custom-tool-registry.spec.ts for loading tools from multiple directories and handling stale directories.
  • Misc:
    • Adds StoredCustomTool type in types.ts to include source information.
    • Updates build-tools.ts and webviewMessageHandler.ts to use new directory loading functions.

This description was created by Ellipsis for f6429c7. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested review from jr and mrubens as code owners December 21, 2025 18:41
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. Enhancement New feature or request labels Dec 21, 2025
@roomote
Copy link
Contributor

roomote bot commented Dec 21, 2025

Rooviewer Clock   See task on Roo Cloud

Reviewed all changes since last review (f6429c7). No issues found.

The commit updates test snapshots to include the new source field for custom tools. The source: undefined entries reflect that test fixtures don't specify source paths.

Previous reviews

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Dec 21, 2025
@cte cte merged commit 0083691 into cte/core-package Dec 21, 2025
@cte cte deleted the cte/global-custom-tools branch December 21, 2025 19:58
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Dec 21, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Dec 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants