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: Add persona-based replies and whisper support to LLM triage #1170

Merged
merged 3 commits into from
Mar 6, 2025

Conversation

SamSaffron
Copy link
Member

@SamSaffron SamSaffron commented Mar 6, 2025

This PR enhances the LLM triage automation with several important improvements:

  • Add ability to use AI personas for automated replies instead of canned replies
  • Add support for whisper responses
  • Refactor LLM persona reply functionality into a reusable method
  • Add new settings to configure response behavior in automations
  • Improve error handling and logging
  • Fix handling of personal messages in the triage flow
  • Add comprehensive test coverage for new features
  • Make personas configurable with more flexible requirements

This allows for more dynamic and context-aware responses in automated workflows, with better control over visibility and attribution.

@SamSaffron SamSaffron changed the title llm triage improvements FEATURE: Add persona-based replies and whisper support to LLM triage Mar 6, 2025
raise ArgumentError, "llm_triage: no action specified!"
end

if action == :edit && category_id.blank? && tags.blank? && flag_post.blank? &&
hide_topic.blank?
return
Copy link
Contributor

@nattsw nattsw Mar 6, 2025

Choose a reason for hiding this comment

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

Not sure if this should raise the same argumenterror as above.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think this is correct, cause I am fine with not forcing busywork if people forgot to select "created" from the action type. it is so easy to miss

description: "#{persona.name} (#{persona.user.username})",
}
end
def self.available_persona_choices(require_user: true, require_default_llm: true)
Copy link
Contributor

@nattsw nattsw Mar 6, 2025

Choose a reason for hiding this comment

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

This may return an empty array? Not sure if we can disable the script if this resolves to []

Copy link
Member Author

Choose a reason for hiding this comment

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

sadly automation is missing a lot of stuff here 😢 there is also a bunch of inter key dependencies, we only want to show option A if B is selected and stuff like that. automation does need a bunch of tlc

@SamSaffron SamSaffron merged commit 01893bb into main Mar 6, 2025
6 checks passed
@SamSaffron SamSaffron deleted the llm_triage_improvements branch March 6, 2025 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants