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

AI generated unread messages summaries #13430

Closed
marcoambrosini opened this issue Oct 1, 2024 · 18 comments
Closed

AI generated unread messages summaries #13430

marcoambrosini opened this issue Oct 1, 2024 · 18 comments
Assignees
Labels
1. to develop enhancement feature: chat 💬 Chat and system messages feature: integration 📦 Integration with 3rd party (chat) service
Milestone

Comments

@marcoambrosini
Copy link
Member

marcoambrosini commented Oct 1, 2024

When we have a lot of unread messages in a conversation, it would be nice to have a short summary of them.

Expectation

A button that allows to generate summaries would appear next to the unread messages divider. Clicking it would start the action

Image

An "info" badge would appear in the new message form, indicating that the task has started

Image

And finally, when the summary is generated, it could look like this.

Image

Warning

The summary box MUST contain a disclaimer that the summary is AI generated and can contain mistakes

@marcoambrosini marcoambrosini self-assigned this Oct 1, 2024
@marcoambrosini marcoambrosini converted this from a draft issue Oct 1, 2024
@marcoambrosini marcoambrosini changed the title AI chat summaries AI generated chat summaries Oct 1, 2024
@marcoambrosini marcoambrosini changed the title AI generated chat summaries AI generated unread messages summaries Oct 1, 2024
@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 💬 Talk team Oct 2, 2024
@nickvergessen nickvergessen moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 💬 Talk team Oct 2, 2024
@nickvergessen nickvergessen added this to the 🌠 Next Minor (30.1) milestone Oct 2, 2024
@marcoambrosini
Copy link
Member Author

Kickoff meeting happening on Monday 7/10

@nickvergessen
Copy link
Member

  • Takes in time range of 10+ seconds to multiple minutes
  • Should be "prompted" but only done once the user requests it to save energy and time on AI
  • Summary state should be shown near the chat (similar to OOO note?, or in topbar?)
  • Once available the summary can be read in a popup and afterwards is deleted again

@nickvergessen nickvergessen added 1. to develop feature: chat 💬 Chat and system messages feature: integration 📦 Integration with 3rd party (chat) service and removed 0. Needs triage labels Oct 16, 2024
@marcoambrosini

This comment has been minimized.

@marcoambrosini marcoambrosini moved this from 📐 At design to 🏗️ At engineering in 🖍 Design team Oct 23, 2024
@nickvergessen
Copy link
Member

Looks like a good place for the button and the summary.
Just have to keep in mind that we should generate the summary from the "button point" because the readmarker might have been updated already (we have visual vs another readmarker), so the UI needs to send the "since message" part to the API.

@nickvergessen

This comment has been minimized.

@nickvergessen
Copy link
Member

nickvergessen commented Nov 11, 2024

@SystemKeeper
Copy link
Contributor

When we have a lot of unread messages in a conversation, it would be nice to have a short summary of them.

Do we have a threshold or conditions for when to show the summary button or not?

It feels like a waste of resources having a summary for something like:

@nickvergessen
Copy link
Member

It feels like a waste of resources having a summary for something like:

Yeah, definitely a threshhold. Not sure what we had discussed. I think 100+ floats around in my head.

@SystemKeeper
Copy link
Contributor

This reminds me of nextcloud/talk-ios#1381, so might be more tricky than I thought ..

@marcoambrosini
Copy link
Member Author

Could the threshold take into account the char count too? That way it would have a more accurate representation of the actual amount of text.

@nickvergessen
Copy link
Member

nickvergessen commented Nov 13, 2024

If all messages have been loaded already, technically yes, but not sure what that does to client performance. If we want to avoid loading 2k or more messages in the future and show it on an "empty page" not anymore.

@SystemKeeper
Copy link
Contributor

If all messages have been loaded already, technically yes, but not sure what that does to client performance

Would need to try it, if we stored the messages locally already on ios, it might work, but if we need to load all those messages first... But the unread message count is already available through the API endpoint, so that would be way easier.

@nickvergessen
Copy link
Member

We could of course also do something like:

  • Show if > 100 messages or
  • if > 20 messages with more than 5.000 characters

@SystemKeeper
Copy link
Contributor

Would that be checked on backend and exposed as an attributed or should we do it in clients?

@nickvergessen
Copy link
Member

config => chat => summary-threshold (local) - Number of unread messages that should exist to show a "Generate summary" option

Added to capabilities for now

@SystemKeeper
Copy link
Contributor

Some notes:

@Antreesy
Copy link
Contributor

Follow-ups:

  • partially failing tasks
  • store task status
  • add copy to clipboard (instead of note to self)

@nickvergessen
Copy link
Member

Overall done

@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💬 Talk team Nov 29, 2024
@github-project-automation github-project-automation bot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design team Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop enhancement feature: chat 💬 Chat and system messages feature: integration 📦 Integration with 3rd party (chat) service
Projects
Status: ☑️ Done
Status: 🎉 Done
Development

No branches or pull requests

4 participants