feat(lightspeed): add conversation sorting with persistence, persisting pinned chats and pinned chats toggle per-user #2003

its-mitesh-kumar · 2026-01-06T11:19:23Z

Description

Adds conversation sorting and per-user settings persistence using Backstage's StorageApi. Users can now sort conversations by date (newest/oldest) or name (A-Z/Z-A) via a dropdown next to the search input. Pinned chats, the pinned chats toggle state, and sort order now persist across sessions, scoped per-user. Upgraded @patternfly/chatbot to 6.5.0-prerelease.28 for searchActionEnd prop support.

Fixes

Fixed https://issues.redhat.com/browse/RHIDP-9564

UI after Changes

Screen.Recording.2026-01-05.at.10.04.13.PM.mov

✔️ Checklist

A changeset describing the change and affected packages. (more info)
Added or Updated documentation
Tests for new functionality and regression tests for bug fixes
Screenshots attached (for UI changes)

…toggle pinned chat Signed-off-by: its-mitesh-kumar <[email protected]>

Signed-off-by: its-mitesh-kumar <[email protected]>

rhdh-gh-app · 2026-01-06T11:19:42Z

Important

This PR includes changes that affect public-facing API. Please ensure you are adding/updating documentation for new features or behavior.

Changed Packages

Package Name	Package Path	Changeset Bump	Current Version
app	workspaces/lightspeed/packages/app	none	`v0.0.0`
@red-hat-developer-hub/backstage-plugin-lightspeed	workspaces/lightspeed/plugins/lightspeed	minor	`v1.1.1`

Signed-off-by: its-mitesh-kumar <[email protected]>

HusneShabbir

works as expected,
/lgtm

chore(e2e): enhance existing lightspeed e2e tests

openshift-ci · 2026-01-07T09:10:41Z

New changes are detected. LGTM label has been removed.

fix(e2e): unskip file attachemnet tests

workspaces/lightspeed/plugins/lightspeed/src/hooks/usePinnedChatsSettings.ts

Signed-off-by: its-mitesh-kumar <[email protected]>

sonarqubecloud · 2026-01-08T08:32:57Z

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

debsmita1

Tested the changes locally

Screen.Recording.2026-01-09.at.12.42.57.PM.mov

debsmita1 · 2026-01-09T06:35:25Z

workspaces/lightspeed/plugins/lightspeed/src/hooks/usePinnedChatsSettings.ts

+      const sortData = sortSnapshot.value ?? {};
+
+      setIsPinningChatsEnabled(
+        (enabledData[user] as boolean | undefined) ?? true,


for Guest users, avoid persisting the preferences

debsmita1 · 2026-01-09T07:09:28Z

workspaces/lightspeed/plugins/lightspeed/src/components/LightSpeedChat.tsx

+  }, []);
+
+  const onSortSelect = useCallback(
+    (_event?: React.MouseEvent<Element>, value?: string | number) => {


Suggested change

(_event?: React.MouseEvent<Element>, value?: string | number) => {

(_event?: MouseEvent<Element>, value?: string | number) => {

debsmita1 · 2026-01-09T07:09:37Z

workspaces/lightspeed/plugins/lightspeed/src/components/LightSpeedChat.tsx

+  );
+
+  const sortToggle = useCallback(
+    (toggleRef: React.Ref<MenuToggleElement>) => (


Suggested change

(toggleRef: React.Ref<MenuToggleElement>) => (

(toggleRef: Ref<MenuToggleElement>) => (

debsmita1 · 2026-01-09T07:11:45Z

workspaces/lightspeed/plugins/lightspeed/src/hooks/usePinnedChatsSettings.ts

+ * Hook to manage pinned chats settings with persistence using Backstage StorageApi.
+ * Settings are scoped per-user to support multi-user environments.
+ *
+ * @param user - The user entity ref (e.g., "user:default/guest")


for Guest users, avoid persisting the preferences

rohitkrai03 · 2026-01-09T08:09:26Z

workspaces/lightspeed/plugins/lightspeed/src/components/LightSpeedChat.tsx

+    handlePinningChatsToggle,
+    pinChat,
+    unpinChat,
+    handleSortChange,


Is there a reason to combine sorting and pinning logic into one hook? It would be better if sorting and pinning logic are separated into there own hooks. Easier to read and maintain. Plus sorting action won't affect pinning data to cause rerenders and vice versa.

rohitkrai03 · 2026-01-09T08:16:29Z

workspaces/lightspeed/plugins/lightspeed/package.json

    "@backstage/theme": "^0.7.0",
    "@material-ui/core": "^4.9.13",
    "@material-ui/lab": "^4.0.0-alpha.61",
+    "@monaco-editor/react": "^4.7.0",


Why is this PR adding monaco editor as a dependency here?

@monaco-editor/react and @monaco-editor are required peer dependencies of @patternfly/[email protected]. We upgraded to this prerelease version to use the new searchActionEnd prop on ChatbotConversationHistoryNav for the sort dropdown feature.

The chatbot pre-release verison adds monaco-editor as a peer-dependency, without this in our dependency shows module not found errs

debsmita1 · 2026-01-09T08:58:04Z

@its-mitesh-kumar
I have tested this on RHDH with the following configuration:

red-hat-developer-hub.backstage-plugin-lightspeed:
      translationResources:
        - importName: lightspeedTranslations
          module: Alpha
          ref: lightspeedTranslationRef
      appIcons:
        - name: LightspeedIcon
          module: LightspeedPlugin
          importName: LightspeedIcon
      dynamicRoutes:
        - path: /lightspeed
          importName: LightspeedPage
          module: LightspeedPlugin
          menuItem:
            icon: LightspeedIcon
            text: Lightspeed

userSettings:
  persistence: browser

My chat preferences persist, but when switching to the database

userSettings:
  persistence: database

My preferences are not saved

karthikjeeyar · 2026-01-09T17:38:49Z

workspaces/lightspeed/plugins/lightspeed/src/hooks/usePinnedChatsSettings.ts

+          const chatsData = { ...chatsSnapshot.value };
+          chatsData[user] = updated;
+          bucket.set(PINNED_CHATS_KEY, chatsData);


Remove the user information in the user settings bucket value.

Usersettings backend will work based on the user token attached in the request header, you do not need to save or send the user information explicitly. The backend automatically attaches the logged in userEntityRef and uses that for all the GET, PUT, DELETE methods.

karthikjeeyar · 2026-01-09T17:39:19Z

workspaces/lightspeed/plugins/lightspeed/src/hooks/usePinnedChatsSettings.ts

+          chatsData[user] = updated;
+          bucket.set(PINNED_CHATS_KEY, chatsData);


same here, refer my above comment.

karthikjeeyar · 2026-01-09T17:46:34Z

workspaces/lightspeed/plugins/lightspeed/src/translations/fr.ts

Yi is currently adding it and ja locales in rhdh-plugins repo in this PR #2009. Since these are new strings can you also use cursor to add strings for those locales?

So that we will at-least have the AI generated strings rather than completely missing out.

its-mitesh-kumar added 6 commits January 6, 2026 16:47

feat(lightspeed): sorting chats, persisting chat order, pinned chat, …

e29ffcf

…toggle pinned chat Signed-off-by: its-mitesh-kumar <[email protected]>

adding changeset

23449a0

Signed-off-by: its-mitesh-kumar <[email protected]>

different icons for different sorting

073d002

Signed-off-by: its-mitesh-kumar <[email protected]>

adding api report

8901948

Signed-off-by: its-mitesh-kumar <[email protected]>

updating the version of monaco-editor

3a2e907

Signed-off-by: its-mitesh-kumar <[email protected]>

yarn dedupe

63223bb

Signed-off-by: its-mitesh-kumar <[email protected]>

its-mitesh-kumar requested review from a team, karthikjeeyar, rohitkrai03 and yangcao77 as code owners January 6, 2026 11:19

its-mitesh-kumar changed the title ~~Feat/lightspeed chat sorting clean~~ feat(lightspeed): add conversation sorting with persistence, persisting pinned chats and pinned chats toggle per-user Jan 6, 2026

its-mitesh-kumar mentioned this pull request Jan 6, 2026

feat(lightspeed): add conversation sorting with persistence, persisting pinned chats and pinned chats toggle per-user #1998

Closed

4 tasks

its-mitesh-kumar added 2 commits January 6, 2026 17:34

updating test

50b2924

Signed-off-by: its-mitesh-kumar <[email protected]>

resolving sonarqube issues

596c41a

Signed-off-by: its-mitesh-kumar <[email protected]>

HusneShabbir reviewed Jan 6, 2026

View reviewed changes

openshift-ci bot assigned HusneShabbir Jan 6, 2026

openshift-ci bot added the lgtm label Jan 6, 2026

HusneShabbir and others added 2 commits January 7, 2026 14:30

enhance existing e2e

b57ca2b

Merge pull request #6 from HusneShabbir/pr/its-mitesh-kumar/2003

daf0e66

chore(e2e): enhance existing lightspeed e2e tests

openshift-ci bot removed the lgtm label Jan 7, 2026

its-mitesh-kumar and others added 5 commits January 7, 2026 14:41

Merge branch 'main' into feat/lightspeed-chat-sorting-clean

4f674b7

unskip file attachemnets

1de36ad

fix sonar

bfb27dd

test re-str

0dd5d60

Merge pull request #7 from HusneShabbir/pr/its-mitesh-kumar/2003

42c14cd

fix(e2e): unskip file attachemnet tests

debsmita1 reviewed Jan 8, 2026

View reviewed changes

workspaces/lightspeed/plugins/lightspeed/src/hooks/usePinnedChatsSettings.ts Show resolved Hide resolved

its-mitesh-kumar added 2 commits January 8, 2026 14:00

adding tests

e2af1cb

Signed-off-by: its-mitesh-kumar <[email protected]>

Merge branch 'main' into feat/lightspeed-chat-sorting-clean

960e422

debsmita1 reviewed Jan 9, 2026

View reviewed changes

rohitkrai03 requested changes Jan 9, 2026

View reviewed changes

karthikjeeyar reviewed Jan 9, 2026

View reviewed changes

	(_event?: React.MouseEvent<Element>, value?: string \| number) => {
	(_event?: MouseEvent<Element>, value?: string \| number) => {

	(toggleRef: React.Ref<MenuToggleElement>) => (
	(toggleRef: Ref<MenuToggleElement>) => (

		chatsData[user] = updated;
		bucket.set(PINNED_CHATS_KEY, chatsData);

feat(lightspeed): add conversation sorting with persistence, persisting pinned chats and pinned chats toggle per-user #2003

Are you sure you want to change the base?

feat(lightspeed): add conversation sorting with persistence, persisting pinned chats and pinned chats toggle per-user #2003

Uh oh!

Conversation

its-mitesh-kumar commented Jan 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Fixes

UI after Changes

✔️ Checklist

Uh oh!

rhdh-gh-app bot commented Jan 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changed Packages

Uh oh!

HusneShabbir left a comment

Choose a reason for hiding this comment

Uh oh!

openshift-ci bot commented Jan 7, 2026

Uh oh!

Uh oh!

sonarqubecloud bot commented Jan 8, 2026

Quality Gate passed

Uh oh!

debsmita1 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

debsmita1 Jan 9, 2026 • 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!

debsmita1 commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

its-mitesh-kumar commented Jan 6, 2026 •

edited

Loading

rhdh-gh-app bot commented Jan 6, 2026 •

edited

Loading

debsmita1 Jan 9, 2026 •

edited

Loading

debsmita1 commented Jan 9, 2026 •

edited

Loading