Skip to content

Conversation

@Wallgau
Copy link
Collaborator

@Wallgau Wallgau commented Nov 30, 2025

  • Adds the chat header component with session management capabilities to the playground sliding container.
    Changes
    -> Add ChatHeader component with title, actions, and session dropdown
    -> Add ChatSidebar component for session list in fullscreen mode
    -> Add SessionSelector component with rename and delete functionality
    -> Add SessionLogsModal for inspecting session messages
    -> Add session rename API integration with optimistic updates
    -> Extract session management logic into useSessionManagement hook
    -> Add tests for ChatSidebar component

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 30, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/chat-header-feature

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Nov 30, 2025
@Wallgau Wallgau force-pushed the feat/chat-header-feature branch from b0700d6 to 7064bba Compare November 30, 2025 21:31
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Nov 30, 2025
@Wallgau Wallgau marked this pull request as ready for review November 30, 2025 23:37
Olfa Maslah added 7 commits November 30, 2025 19:09
- Add complete chat-header component with all sub-components
- Add chat-header hooks (rename, session actions)
- Add chat-header utils (title, error checking)
- Add chat-header types and tests
- Replace placeholder with FlowPageSlidingContainerContent
- Add API query fixes for session management
- Full chat-header UI is now visible and functional
…ion management to hook

- Extract session management logic into useSessionManagement hook
- Simplify FlowPageSlidingContainerContent component (85 lines, down from 213)
- Remove complex isManuallySet and renamedSessions tracking
- Follow IOModal pattern for simpler session selection logic
- Add border-radius 8px to flow canvas divs
- Update Share button styling (white background, black text)
- Revert dialog z-index changes (will be handled in separate PR)
@Wallgau Wallgau force-pushed the feat/chat-header-feature branch from 6ea70fd to 391970c Compare December 1, 2025 01:29
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Dec 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 16%
15.68% (4397/28032) 8.85% (1888/21330) 9.76% (602/6168)

Unit Test Results

Tests Skipped Failures Errors Time
1693 0 💤 0 ❌ 0 🔥 21.681s ⏱️

@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

❌ Patch coverage is 32.60341% with 277 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (feat/sliding-container-infrastructure@87e8f8f). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...r/components/session-selector/session-selector.tsx 0.00% 52 Missing ⚠️
...hat-view/chat-header/components/session-rename.tsx 0.00% 43 Missing ⚠️
...omponent/chat-view/hooks/use-session-management.ts 0.00% 39 Missing ⚠️
...ntainer/components/flow-page-sliding-container.tsx 0.00% 32 Missing ⚠️
.../chat-header/components/chat-sessions-dropdown.tsx 0.00% 31 Missing ⚠️
...-view/chat-header/components/session-more-menu.tsx 47.22% 11 Missing and 8 partials ⚠️
...t-view/chat-header/hooks/use-chat-header-rename.ts 44.82% 16 Missing ⚠️
...groundComponent/chat-view/hooks/use-get-flow-id.ts 0.00% 10 Missing ⚠️
...view/chat-header/components/session-logs-modal.tsx 0.00% 9 Missing ⚠️
...at-header/hooks/use-chat-header-session-actions.ts 61.53% 5 Missing ⚠️
... and 8 more
Additional details and impacted files

Impacted file tree graph

@@                           Coverage Diff                            @@
##             feat/sliding-container-infrastructure   #10797   +/-   ##
========================================================================
  Coverage                                         ?   32.42%           
========================================================================
  Files                                            ?     1393           
  Lines                                            ?    63923           
  Branches                                         ?     9494           
========================================================================
  Hits                                             ?    20726           
  Misses                                           ?    42144           
  Partials                                         ?     1053           
Flag Coverage Δ
frontend 14.52% <32.60%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...t/chat-view/chat-header/components/chat-header.tsx 100.00% <100.00%> (ø)
...t/chat-view/chat-header/utils/get-session-title.ts 100.00% <100.00%> (ø)
...w/chat-header/components/session-selector/index.ts 0.00% <0.00%> (ø)
...playgroundComponent/chat-view/chat-header/index.ts 0.00% <0.00%> (ø)
src/frontend/src/pages/FlowPage/index.tsx 0.00% <0.00%> (ø)
...iew/chat-header/components/chat-header-actions.tsx 86.66% <86.66%> (ø)
...-view/chat-header/components/chat-header-title.tsx 81.25% <81.25%> (ø)
.../chat-view/chat-header/components/chat-sidebar.tsx 88.23% <88.23%> (ø)
src/frontend/src/stores/messagesStore.ts 80.39% <0.00%> (ø)
...at-header/hooks/use-chat-header-session-actions.ts 61.53% <61.53%> (ø)
... and 10 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Dec 1, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Dec 3, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Dec 3, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Dec 3, 2025
@Wallgau Wallgau requested a review from lucaseduoli December 4, 2025 15:41
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Dec 16, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants