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

Feature/chat agent pinning #189

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

atahankilc
Copy link

What it does

An "agent pinning" feature has been introduced to enhance the user experience in consecutive agent usage within the chat interface.

  • Automatic Pinning:
    If a chat agent is mentioned in a prompt and there is no prior pinned agent, the mentioned agent is automatically pinned. This eliminates the need for the user to repeatedly mention the agent in subsequent prompts.

  • Handling New Mentions:
    If a different agent is mentioned while another agent is pinned, the pinned agent remains unchanged. The newly mentioned agent is used only for the specific prompt where it is mentioned, without affecting the pinned agent.

  • Manual Control:
    Users can manually unpin the agent through the chat interface if desired.

  • New Session with Pinned Agent:
    A new chat session can now be initiated directly with a pinned agent.

How to test

  • Mention an agent in chat interface
  • Execute AI_CHAT_NEW_CHAT_WINDOW_WITH_PINNED_AGENT_COMMAND with passing desired chat agent

Follow-ups

As part of this contribution, We encountered an issue with the added command ai-chat-ui.new-chat-with-pinned-agent. The command is expected to take a single argument of type ChatAgent | undefined. However, the function call fails when the argument is passed directly in the expected form. To make the command work, We had to pass an argument array instead, where the first item in the array is undefined, followed by the actual ChatAgent value.

This behavior seems to deviate from the intended design. While this workaround resolves the issue for now, the underlying cause still needs to be investigated. The error message encountered when not using this approach is:

Error: The command 'ai-chat-ui.new-chat-with-pinned-agent' cannot be executed. There are no active handlers available for the command.

We are including this as a follow-up issue to address later and investigate why the command handler does not behave as expected when a single argument is passed directly.

Attribution

This feature was developed by @atahankilc, @ZidongWang123, @TANG839, and @mihaela21k as part of the AI Coding Exercises project in the Practical Course Software Engineering for Business Information Systems at Technical University of Munich.

Review checklist

Reminder for reviewers

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.

1 participant