feat: Add UI for summarzing messages #36

sushen123 · 2025-04-05T14:14:01Z

Proposed Changes

I have added a new action button to the UI that triggers a modal for summarizing messages. In modal there is filter to summarize message like (Messages from today, week, unread and specific users).

Action Button

Modal

Video

Screencast.2025-04-05.19.58.12.mp4

Issue(s) Addressed

Closes #35

Steps to Test or Reproduce

Go to any room or channel.
Click on plus button which is in the message box.
Modal will show up.
Select any option
Click on Summarize Button
Messages will be summarize

Further Comments

I have to use app persistence to store the roomId and threadId because the UIKitViewSubmitInteractionContext returns null when attempting to extract these values directly.

public async handleActions(context: UIKitViewSubmitInteractionContext): Promise<IUIKitResponse> {
    const { view, user } = context.getInteractionData(); 
    // When I try to extract the roomId and threadId from the context, they return null.
    
    const { roomId } = await getData(
        this.read.getPersistenceReader(),
        user.id,
        ROOM_ID_KEY,
    );
}

sushen123 · 2025-04-05T14:14:24Z

hey @JeffreytheCoder , can you check this PR?

JeffreytheCoder · 2025-04-07T08:55:23Z

Hi @sushen123 , thanks for implementing this!

The demo video is not playable on my side - could you double check it or provide an alternative link please?

sushen123 · 2025-04-07T09:09:29Z

Hi @sushen123 , thanks for implementing this!

The demo video is not playable on my side - could you double check it or provide an alternative link please?

hey @JeffreytheCoder you can check here https://vimeo.com/1073130710/683a3f2e83?share=copy#t=0

JeffreytheCoder · 2025-04-08T03:32:14Z

app/modal/summarizeModal.ts

+    if(filterValue === "users") {
+        const participantOptions: MultiStaticSelectOptionsParam = [];
+        try {
+        if (user.id && roomId) {
+            const members = await read.getRoomReader().getMembers(roomId);
+            for (const member of members) {
+                if (member.id) {
+                    participantOptions.push({
+                        text: {
+                            type: TextObjectType.MRKDWN,
+                            text: `${member.name} - @${member.username}`,
+                        },
+                        value: `${member.username}`,
+                    });
+                }
+            }
+            participantOptions.sort((a, b) => {
+                return a.text.text.toUpperCase() < b.text.text.toUpperCase()
+                    ? -1
+                    : 1;
+            });
+        }


What if the room has hundreds of users? It may create a big load on UI. It's worth testing out and adding some safety strategies

I think sorting a large list could add extra load, and it’s probably not that necessary. So I’m thinking of skipping the sort altogether.
Is there anything else that might slow things down? would love to hear your thoughts

JeffreytheCoder · 2025-04-08T03:39:36Z

app/handlers/ExecuteViewSubmitHandler.ts

+                        switch (filter) {
+                            case 'today':
+                                startDate = new Date(
+                                    now.getFullYear(),
+                                    now.getMonth(),
+                                    now.getDate(),
+                                    0,
+                                    0,
+                                    0,
+                                    0
+                                );
+                                break;
+                            case 'week':
+                                startDate = new Date(now.getTime() - 7 * 24 * 60 * 60 * 1000);
+                                break;
+                            case 'unread':
+                                unreadCount = await this.read
+                                    .getUserReader()
+                                    .getUserUnreadMessageCount(user.id);
+                                break;
+                            default:
+                                usernames = view.state?.[SummarizeModalEnum.USER_LISTS_BLOCK_ID]?.[
+                                    SummarizeModalEnum.USER_LISTS_ACTION_ID
+                                ]
+                        }
+                    }
+
+                    const addOns = await this.app
+                        .getAccessors()
+                        .environmentReader.getSettings()
+                        .getValueById('add-ons');
+                    const xAuthToken = await this.app
+                        .getAccessors()
+                        .environmentReader.getSettings()
+                        .getValueById('x-auth-token');
+                    const xUserId = await this.app
+                        .getAccessors()
+                        .environmentReader.getSettings()
+                        .getValueById('x-user-id');
+
+
+                    let messages: string;
+                    if (!threadId) {
+                        messages = await this.getRoomMessages(
+                            room,
+                            this.read,
+                            user,
+                            this.http,
+                            addOns,
+                            xAuthToken,
+                            xUserId,
+                            startDate,
+                            unreadCount,
+                            usernames,
+                            anyMatchedUsername
+                        );
+                    } else {
+                        messages = await this.getThreadMessages(
+                            room,
+                            this.read,
+                            user,
+                            this.http,
+                            threadId,
+                            addOns,
+                            xAuthToken,
+                            xUserId,
+                            startDate,
+                            unreadCount,
+                            usernames,
+                            anyMatchedUsername
+                        );
+                    }
+
+                    if (!messages || messages.trim().length === 0) {
+                        await notifyMessage(
+                            room,
+                            this.read,
+                            user,
+                            'There are no messages to summarize in this channel.',
+                            threadId
+                        );
+                        return this.context.getInteractionResponder().successResponse();;
+                    }
+
+                    await notifyMessage(room, this.read, user, messages, threadId);
+
+                    let summary: string;
+                    if (!threadId) {
+                        const prompt = createSummaryPromptByTopics(messages);
+                        summary = await createTextCompletion(
+                            this.app,
+                            room,
+                            this.read,
+                            user,
+                            this.http,
+                            prompt,
+                            threadId
+                        );
+                    } else {
+                        const prompt = createSummaryPrompt(messages);
+                        summary = await createTextCompletion(
+                            this.app,
+                            room,
+                            this.read,
+                            user,
+                            this.http,
+                            prompt,
+                            threadId
+                        );
+                    }
+                    await notifyMessage(room, this.read, user, summary, threadId);
+
+                    if (addOns.includes('assigned-tasks')) {
+                        const assignedTasksPrompt = createAssignedTasksPrompt(messages);
+                        const assignedTasks = await createTextCompletion(
+                            this.app,
+                            room,
+                            this.read,
+                            user,
+                            this.http,
+                            assignedTasksPrompt,
+                            threadId
+                        );
+                        await notifyMessage(room, this.read, user, assignedTasks, threadId);
+                    }
+
+                    if (addOns.includes('follow-up-questions')) {
+                        const followUpQuestionsPrompt = createFollowUpQuestionsPrompt(messages);
+                        const followUpQuestions = await createTextCompletion(
+                            this.app,
+                            room,
+                            this.read,
+                            user,
+                            this.http,
+                            followUpQuestionsPrompt,
+                            threadId
+                        );
+                        await notifyMessage(room, this.read, user, followUpQuestions, threadId);
+                    }
+
+                    if (addOns.includes('participants-summary')) {
+                        const participantsSummaryPrompt =
+                            createParticipantsSummaryPrompt(messages);
+                        const participantsSummary = await createTextCompletion(
+                            this.app,
+                            room,
+                            this.read,
+                            user,
+                            this.http,
+                            participantsSummaryPrompt,
+                            threadId
+                        );
+                        await notifyMessage(room, this.read, user, participantsSummary, threadId);
+                    }


This part is duplicated from the slash command handling. It's better to unify these to reduce code duplication and ensure future changes on any subcommand works across slash command & UI

*Not only this part, but the entire submit handler

hey i have made the changes can you look into it

JeffreytheCoder · 2025-04-08T03:46:21Z

@sushen123 Thanks for adding the demo video! The UI code is very structured 👍 Left some minor comments

sushen123 · 2025-04-10T07:26:47Z

Sorry, I’ve been working on my lab report for the last 2-3 days. I’ll check this tomorrow.

…

On Tue, 8 Apr 2025, 9:31 am Jeffrey Yu, ***@***.***> wrote: @sushen123 <https://github.com/sushen123> Thanks for adding the demo video! The UI code is very structured 👍 Left some minor comments — Reply to this email directly, view it on GitHub <#36 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AVHF2TN346657IQEUS234LL2YNBCFAVCNFSM6AAAAAB2QRYTRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOOBVGE2DONZTHE> . You are receiving this because you were mentioned.Message ID: ***@***.***> [image: JeffreytheCoder]*JeffreytheCoder* left a comment (RocketChat/Apps.Chat.Summarize#36) <#36 (comment)> @sushen123 <https://github.com/sushen123> Thanks for adding the demo video! The UI code is very structured 👍 Left some minor comments — Reply to this email directly, view it on GitHub <#36 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AVHF2TN346657IQEUS234LL2YNBCFAVCNFSM6AAAAAB2QRYTRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOOBVGE2DONZTHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

JeffreytheCoder · 2025-04-28T08:57:20Z

LGTM! Thanks for the changes

Feel free to merge this PR

sushen123 added 2 commits March 29, 2025 13:34

Add action button

ab35567

add UI for summarizing messages

013a773

Add log

165b820

JeffreytheCoder suggested changes Apr 8, 2025

View reviewed changes

This was referenced Apr 12, 2025

[Feature] Schedule message command for chat summarizer app #37

Open

Feat : Customizable Summary Length/Detail Level #38

Open

sushen123 added 2 commits April 13, 2025 00:03

unify summarization logic

1b36b0d

Remove Logger

8c46699

JeffreytheCoder approved these changes Apr 28, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Add UI for summarzing messages #36

feat: Add UI for summarzing messages #36

Uh oh!

sushen123 commented Apr 5, 2025

Uh oh!

sushen123 commented Apr 5, 2025

Uh oh!

JeffreytheCoder commented Apr 7, 2025

Uh oh!

sushen123 commented Apr 7, 2025

Uh oh!

JeffreytheCoder Apr 8, 2025

Uh oh!

sushen123 Apr 13, 2025 •

edited

Loading

Uh oh!

JeffreytheCoder Apr 8, 2025

Uh oh!

JeffreytheCoder Apr 8, 2025

Uh oh!

sushen123 Apr 13, 2025

Uh oh!

JeffreytheCoder commented Apr 8, 2025

Uh oh!

sushen123 commented Apr 10, 2025 via email

Uh oh!

JeffreytheCoder commented Apr 28, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

feat: Add UI for summarzing messages #36

Are you sure you want to change the base?

feat: Add UI for summarzing messages #36

Uh oh!

Conversation

sushen123 commented Apr 5, 2025

Proposed Changes

Issue(s) Addressed

Steps to Test or Reproduce

Further Comments

Uh oh!

sushen123 commented Apr 5, 2025

Uh oh!

JeffreytheCoder commented Apr 7, 2025

Uh oh!

sushen123 commented Apr 7, 2025

Uh oh!

JeffreytheCoder Apr 8, 2025

Choose a reason for hiding this comment

Uh oh!

sushen123 Apr 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JeffreytheCoder Apr 8, 2025

Choose a reason for hiding this comment

Uh oh!

JeffreytheCoder Apr 8, 2025

Choose a reason for hiding this comment

Uh oh!

sushen123 Apr 13, 2025

Choose a reason for hiding this comment

Uh oh!

JeffreytheCoder commented Apr 8, 2025

Uh oh!

sushen123 commented Apr 10, 2025 via email

Uh oh!

JeffreytheCoder commented Apr 28, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

sushen123 Apr 13, 2025 •

edited

Loading