Skip to content

Conversation

porkpink
Copy link
Contributor

@porkpink porkpink commented Oct 13, 2025

✨ Add tool category feature. #1362

image

@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 08:45
Copy link

@Copilot 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 adds a tool category feature to enable secondary grouping of tools within the tool pool interface. The implementation includes backend database schema updates, Python tool categorization, and frontend UI enhancements for better tool organization.

Key changes:

  • Introduces a ToolCategory enum with predefined categories (search, file, email, terminal)
  • Updates all local tools to include appropriate category assignments
  • Enhances the frontend tool pool UI with category-based filtering and navigation

Reviewed Changes

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

Show a summary per file
File Description
sdk/nexent/core/utils/constants.py Defines new ToolCategory enum with standard tool categories
sdk/nexent/core/tools/*.py Updates all tool classes to include category assignments
frontend/types/agentConfig.ts Adds category field to Tool interface and ToolSubGroup type
frontend/services/agentConfigService.ts Updates API service to handle category field
frontend/app/[locale]/setup/agents/components/tool/ToolPool.tsx Implements category-based UI with sidebar navigation
backend/database/db_models.py Adds category column to ToolInfo database model
backend/services/tool_configuration_service.py Updates tool processing to handle category field
docker/sql/1013_add_tool_group_to_ag_tool_info.sql Database migration script for category column

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

codecov bot commented Oct 13, 2025

Codecov Report

❌ Patch coverage is 92.75362% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/services/tool_configuration_service.py 84.00% 1 Missing and 3 partials ⚠️
...dk/nexent/core/tools/knowledge_base_search_tool.py 87.50% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@porkpink porkpink changed the title ✨ Add tool category feature. #1362 [WIP]✨ Add tool category feature. #1362 Oct 13, 2025
@porkpink porkpink force-pushed the jpl/jpl_0829 branch 2 times, most recently from c658be3 to 7cd2fa8 Compare October 13, 2025 11:55
@porkpink porkpink changed the title [WIP]✨ Add tool category feature. #1362 ✨ Add tool category feature. #1362 Oct 13, 2025
@Phinease
Copy link
Member

二级菜单建议采用统一组件风格,不要产生不同风格

@porkpink porkpink changed the title ✨ Add tool category feature. #1362 [WIP]✨ Add tool category feature. #1362 Oct 14, 2025
@porkpink porkpink force-pushed the jpl/jpl_0829 branch 2 times, most recently from e29fb4e to 9e178f8 Compare October 15, 2025 08:29
@porkpink porkpink changed the title [WIP]✨ Add tool category feature. #1362 ✨ Add tool category feature. #1362 Oct 15, 2025
✨ Add tool category. #1362

✨ Add tool category. #1362
@porkpink
Copy link
Contributor Author

二级菜单建议采用统一组件风格,不要产生不同风格

@porkpink porkpink closed this Oct 15, 2025
@porkpink
Copy link
Contributor Author

二级菜单建议采用统一组件风格,不要产生不同风格

已修改

@porkpink porkpink reopened this Oct 15, 2025
@Phinease Phinease merged commit b1024a6 into develop Oct 16, 2025
18 checks 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.

3 participants