Skip to content
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

frontend: Change message for Agent with no tools #877

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Conversation

tianjing-li
Copy link
Collaborator

@tianjing-li tianjing-li commented Dec 12, 2024

AI Description

This pull request introduces a new feature to the src/backend/config/default_agent.py and src/interfaces/assistants_web/src/components/MessagingContainer/Welcome.tsx files, enabling users to ask questions and receive answers based on their tools and files.

Changes:

  • In src/backend/config/default_agent.py, the description field of the AgentPublic class has been updated to include the phrase "based on your tools and files".
  • In src/interfaces/assistants_web/src/components/MessagingContainer/Welcome.tsx, the useAvailableTools hook has been imported and utilized to filter available tools for the base agent.
  • The toolToggleMessage variable has been introduced to display a message based on the availability of tools. If no tools are available, the message prompts the user to update their agent to add tools.
  • The AssistantTools component now receives the tools array directly, without any filtering.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.21%. Comparing base (f370c46) to head (d43c411).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #877   +/-   ##
=======================================
  Coverage   78.21%   78.21%           
=======================================
  Files         259      259           
  Lines       11088    11088           
=======================================
  Hits         8672     8672           
  Misses       2416     2416           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@EugeneLightsOn EugeneLightsOn left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thank you TJ

@tianjing-li tianjing-li merged commit fc64a71 into main Dec 17, 2024
7 of 8 checks passed
@tianjing-li tianjing-li deleted the toggle-tools branch December 17, 2024 14:18
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