Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Dec 15, 2025

Important

Integrate @roo-code/core for custom tool management, enabling dynamic tool registration and execution across the system.

  • Custom Tool Integration:
    • Integrate customToolRegistry from @roo-code/core to manage custom tools in NativeToolCallParser.ts, presentAssistantMessage.ts, system.ts, and build-tools.ts.
    • Allow custom tools in validateToolUse.ts by checking customToolRegistry.
  • Tool Execution:
    • Modify presentAssistantMessage.ts to execute custom tools and handle errors.
    • Update NativeToolCallParser.ts to parse and validate custom tool arguments.
  • Prompt Generation:
    • Update system.ts to include custom tools in the prompt for non-native protocols.
  • Build and Configuration:
    • Add esbuild as an external dependency in esbuild.mjs and package.json.
  • Testing:
    • Update mocks in autoImportSettings.spec.ts to handle path and OS modules.

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

@roomote
Copy link
Contributor

roomote bot commented Dec 15, 2025

Rooviewer Clock   See task on Roo Cloud

Sync review completed. No new issues found in this update. One outstanding issue remains.

  • Remove debug console.log statements from NativeToolCallParser.ts and Task.ts
  • Remove debug console.log statements from presentAssistantMessage.ts (lines 1063, 1069, 1080)
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 15, 2025
@cte cte marked this pull request as ready for review December 15, 2025 22:49
@cte cte requested review from jr and mrubens as code owners December 15, 2025 22:49
@cte cte merged commit 7b12c60 into cte/core-package Dec 15, 2025
1 check passed
@cte cte deleted the cte/use-core-package branch December 15, 2025 22:49
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Dec 15, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Dec 15, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Dec 15, 2025
@roomote
Copy link
Contributor

roomote bot commented Dec 15, 2025

Rooviewer Clock   See task on Roo Cloud

Sync review completed. No new issues found in this update. One outstanding issue remains.

  • Remove debug console.log statements from NativeToolCallParser.ts and Task.ts
  • Remove debug console.log statements from presentAssistantMessage.ts (lines 1063, 1069, 1080)
Previous reviews

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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