feat: real-time notifications + UI fixes for industrial deployment#2960
Open
somi2306 wants to merge 4 commits into
Open
feat: real-time notifications + UI fixes for industrial deployment#2960somi2306 wants to merge 4 commits into
somi2306 wants to merge 4 commits into
Conversation
New features: - Add useNotifications hook: real-time Socket.io notifications with Chainlit native auth - Add NotificationBell component: unread badge, popover list, type indicators - Add LeftSidebar/Search: thread history search functionality Bug fixes: - Fix Alert: use div instead of p to allow block-level children (Skeleton) - Fix AutoResizeTextarea: add forwardRef + useImperativeHandle to expose ref externally - Fix BlinkingCursor: replace static cursor with animated 3-dot typing indicator - Fix Logo/Avatar: resolve absolute URLs for reverse-proxy/standalone deployments - Fix ThreadList: use SidebarMenuAction to avoid invalid HTML nesting - Fix ThreadList: track active thread from URL params for correct highlight state - Fix ThreadList: add stopPropagation on delete/rename/share actions Compatibility: - Update shadcn/ui components to latest API (data-slot, React.ComponentProps) - Fix react-markdown PluggableList import for newer versions Tested in production: Chainlit + Next.js 14 + Nginx reverse proxy PFE ENSMR 2026 - Copilote Industriel OCP Jorf Lasfar
Contributor
There was a problem hiding this comment.
11 issues found across 63 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
- Fix P0: chat/index.tsx — correct useNavigate usage (router.push undefined) - Fix P1: alert-dialog.tsx — restore shadow-root portal container - Fix P1: ThreadOptions.tsx — remove static duplicate id - Fix P1: Video.tsx — wrap ReactPlayer in Suspense boundary - Fix P1: Messages/index.tsx — restore BlinkingCursor for running state - Fix P2: Attachments.tsx — remove unnecessary String() coercion - Fix P2: Attachment.tsx — use isImage && imageUrl for container width - Fix P2: MessagesContainer — add apiClient to useCallback deps - Fix P2: Plotly.tsx — wrap JSON.parse in try/catch - Fix P3: ThreadOptions.tsx — fix BOM and UTF-8 encoding artifacts
Contributor
There was a problem hiding this comment.
1 issue found across 10 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
New features:
Bug fixes:
Compatibility:
Tested in production: Chainlit + Next.js 14 + Nginx reverse proxy PFE ENSMR 2026 - Copilote Industriel OCP Jorf Lasfar
Summary by cubic
Adds real-time notifications with a bell and faster thread search. Improves SSR/media stability, reverse-proxy URL handling, and updates UI primitives to the latest
shadcn/ui.New Features
useNotifications: real-timesocket.io-clientfeed using@chainlit/react-clientauth; tracks unread and “mark all read”.NotificationBell: unread badge and popover list with type indicators and timestamps.Bug Fixes
SidebarMenuAction; stopPropagation on actions; accessible trigger button.AutoResizeTextareaviaforwardRef; safer image previews/progress for attachments.react-plotly.js(with JSON.parse try/catch) andreact-playerinSuspense; restore BlinkingCursor with animated dots; correct NewChat route; fix callback deps.react-markdownplugin import; include KaTeX CSS.shadcn/uiAPIs.MessagesContainer.Written for commit 62fe10d. Summary will update on new commits.