Skip to content

Benr/feat safe meta tools registration#21

Merged
imbenrabi merged 3 commits intomainfrom
benr/feat-safe-meta-tools-registration
Feb 1, 2026
Merged

Benr/feat safe meta tools registration#21
imbenrabi merged 3 commits intomainfrom
benr/feat-safe-meta-tools-registration

Conversation

@imbenrabi
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates meta-tool registration so meta-tools are tracked in ToolRegistry (under a reserved _meta toolset key) to enable collision detection and make meta-tools visible through list_tools / registry listings.

Changes:

  • Add META_TOOLSET_KEY and register meta-tools into ToolRegistry during registerMetaTools().
  • Update ServerOrchestrator and unit tests to pass ToolRegistry into registerMetaTools() and assert meta-tool tracking/collision behavior.
  • Update core documentation and bump package version to 0.6.1.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/metaTools.test.ts Updates tests to reflect meta-tools being tracked in ToolRegistry, adds collision and registry-integration assertions.
src/meta/registerMetaTools.ts Registers meta-tools into ToolRegistry under _meta via META_TOOLSET_KEY.
src/core/ServerOrchestrator.ts Passes the orchestrator’s toolRegistry into registerMetaTools().
src/core/CLAUDE.md Documents meta-tool registration + tracking via _meta and collision implications.
package.json Bumps version from 0.6.0 to 0.6.1.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@imbenrabi imbenrabi merged commit d5a6822 into main Feb 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants