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

Bug: Error when try to get most common alert from channel #54

Open
jaschadub opened this issue Aug 17, 2023 · 1 comment
Open

Bug: Error when try to get most common alert from channel #54

jaschadub opened this issue Aug 17, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@jaschadub
Copy link

jaschadub commented Aug 17, 2023

Steps to reproduce the behavior

  1. Go to @ai direct messages
  2. Ask to "in team <name> channel <alerting channel> what is the most common alert?"
  3. See error "Sorry! An error occoured while accessing the LLM. See server logs for details."

In log error: status code: 400, message: '<team>.LookupMattermostUser' does not match '^[a-zA-Z0-9_-]{1,64}$' - 'messages.2.name'"}

Expected behavior

Reply of what the most common alert is

Version and Platform

  • Version: v0.3.2
    Mattermost Version: 7.1.4
    Database Schema Version: 89
    Database: postgres
@crspeller
Copy link
Member

This is partially #44 since you can't do the channel lookup in DMs anyway. (see #44 for why) You did disambiguate but the prompting just isn't there to support that. You have to make that kind of query in the channel itself.
But that shouldn't give you that error. It looks like the LLM came back with a bad function call. (it's hallucinating a function prefixed with a team name)
We should handle this case better.

@crspeller crspeller added the enhancement New feature or request label Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants