Merged
Conversation
Contributor
daisuke-awaji
commented
Nov 1, 2025
daisuke-awaji
commented
- refactor(chat): remove Monaco editor option from MCP server form
- refactor(ToolsSection): inline component implementation from container
Remove the rich editor mode toggle and Monaco JSON editor integration from the MCP server configuration form. The form now exclusively uses a simple textarea for JSON input, simplifying the component and reducing dependencies. Changes: - Remove editorMode state and toggle functionality - Remove MonacoJSONEditor component usage - Simplify UI by removing editor mode switcher button - Keep only the textarea editor for server configuration input This streamlines the user experience and reduces code complexity without losing core functionality.
Replace wrapper component with full ToolsSection implementation by moving all logic from ToolsSectionContainer directly into ToolsSection.tsx. This consolidates the component structure by: - Adding complete state management with custom hooks (useToolsState, useMcpToolsIntegration, useToolsFormatter) - Implementing tab navigation for Available Tools and Tool Detail Settings - Integrating ToolInfoModal for displaying tool information - Adding event handlers and MCP tools integration - Including translation support via react-i18next This refactor simplifies the component architecture by removing the unnecessary container wrapper layer while preserving all existing functionality.
Refactor AgentDetailModal following Clean Architecture and Atomic Design principles by: - Extract business logic into useAgentDetailModal custom hook - Break down monolithic component into smaller, focused components: - AgentHeader, AgentMetadata, SystemPromptSection (Organisms) - MCPServersList, ToolsList, ScenariosList (Molecules) - Replace custom modal implementation with Flowbite-react Modal component for better accessibility and consistency - Improve maintainability by keeping each file under 100 lines - Ensure proper TypeScript type safety throughout - Add comprehensive documentation in README.md This restructuring improves code reusability, testability, and makes the component easier to maintain and extend.
Add visual icons for standard tools using a Proxy-based dynamic icon system. Each tool now displays a distinctive icon with color coding for better visual identification. MCP tools are filtered from the tools list display and assigned a common server icon when needed. Key changes: - Implement toolIcons Proxy for dynamic icon resolution - Add 20+ tool-specific icons with color-coded styling - Filter MCP tools from AgentDetailModal tools list - Display icons alongside tool names in ToolsList component This improves UX by making tools more visually identifiable and handles standard vs MCP tools differently in the UI.
- Standardize tags across all agent YAML files for better categorization - Add 'mcp' tag to MCP-enabled agents - Rename generic tags to more specific ones (e.g., 'cost' → 'cost-optimization', 'developer' → 'development') - Ensure consistent tag ordering with 'aws' as primary tag - Refactor AgentListView filtering logic - Implement switch-based source selection for better maintainability - Improve search filtering to support multiple agent sources - Separate filtering concerns (source selection vs search) These changes improve agent discoverability and make the codebase more maintainable by using consistent tagging conventions and cleaner filtering architecture.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.