-
Notifications
You must be signed in to change notification settings - Fork 0
refactor(tools): rename meta tools to utility tools with tool_* prefix #128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Rename tool naming convention to avoid confusion with Facebook's Meta brand. The "meta_*" prefix was ambiguous and could be misinterpreted. File renames: - stackone_ai/meta_tools.py -> stackone_ai/utility_tools.py - tests/test_meta_tools.py -> tests/test_utility_tools.py - examples/meta_tools_example.py -> examples/utility_tools_example.py Tool name changes: - meta_search_tools -> tool_search - meta_execute_tool -> tool_execute - meta_collect_tool_feedback -> tool_feedback API changes: - Tools.meta_tools() -> Tools.utility_tools() Internal class renames: - MetaToolSearchResult -> ToolSearchResult - create_meta_search_tools() -> create_tool_search() - create_meta_execute_tool() -> create_tool_execute() - MetaSearchTool -> ToolSearchTool - MetaExecuteTool -> ToolExecuteTool All tests pass and documentation has been updated accordingly.
There was a problem hiding this 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 renames the tool naming convention from meta_* to tool_* prefix to avoid confusion with Facebook's Meta brand, affecting tool names, API methods, file names, and all associated documentation.
Changes:
- Tool names updated:
meta_search_tools→tool_search,meta_execute_tool→tool_execute,meta_collect_tool_feedback→tool_feedback - API method renamed:
Tools.meta_tools()→Tools.utility_tools() - Files renamed:
stackone_ai/meta_tools.py→stackone_ai/utility_tools.py, along with corresponding test and example files
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/test_utility_tools.py | Updated all test cases, class names, and assertions to reflect the new tool_* naming convention |
| tests/test_feedback.py | Updated feedback tool name from meta_collect_tool_feedback to tool_feedback in test assertions |
| stackone_ai/utility_tools.py | Renamed classes, functions, and updated all references from meta_* to tool_* prefix; updated comments from "Meta tools" to "Utility tools" |
| stackone_ai/models.py | Renamed meta_tools() method to utility_tools() and updated imports and documentation |
| stackone_ai/feedback/tool.py | Changed feedback tool name from meta_collect_tool_feedback to tool_feedback |
| examples/utility_tools_example.py | Updated all function names, variable names, and documentation to use new utility_tools naming |
| examples/test_examples.py | Updated reference to example file from meta_tools_example.py to utility_tools_example.py |
| README.md | Updated documentation to reflect new naming: tool_* prefix for tools and utility_tools() method |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 issue found across 8 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="README.md">
<violation number="1" location="README.md:308">
P3: Update the Features list to say “Utility Tools (Beta)” so the renamed section and feature overview stay consistent.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| - Only call the tool after the user explicitly agrees. | ||
|
|
||
| ## Meta Tools (Beta) | ||
| ## Utility Tools (Beta) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P3: Update the Features list to say “Utility Tools (Beta)” so the renamed section and feature overview stay consistent.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At README.md, line 308:
<comment>Update the Features list to say “Utility Tools (Beta)” so the renamed section and feature overview stay consistent.</comment>
<file context>
@@ -305,23 +305,23 @@ result = feedback_tool.call(
- Only call the tool after the user explicitly agrees.
-## Meta Tools (Beta)
+## Utility Tools (Beta)
-Meta tools enable dynamic tool discovery and execution without hardcoding tool names.
</file context>
Summary
Renames the tool naming convention from
meta_*totool_*prefix to avoid confusion with Facebook's Meta brand.What Changed
Tool names:
meta_search_tools→tool_searchmeta_execute_tool→tool_executemeta_collect_tool_feedback→tool_feedbackAPI changes:
Tools.meta_tools()→Tools.utility_tools()File renames:
stackone_ai/meta_tools.py→stackone_ai/utility_tools.pytests/test_meta_tools.py→tests/test_utility_tools.pyexamples/meta_tools_example.py→examples/utility_tools_example.pyWhy
The
meta_*prefix was ambiguous and could be misinterpreted as relating to Facebook's Meta. The newtool_*naming is clearer and more descriptive of the functionality (tool search, tool execute, tool feedback).Testing
Summary by cubic
Refactors “meta” tools to “utility” tools and renames meta_* to tool_* across code, tests, and docs. Clarifies naming and avoids confusion with the “Meta” brand.
Refactors
Migration
Written for commit c28436d. Summary will update on new commits.