UI/UX Refinements, Test Suite Migration to Vitest, Security Hardening, and Infrastructure Updates - #70
Closed
PaulJouvanceau wants to merge 0 commit into
Closed
UI/UX Refinements, Test Suite Migration to Vitest, Security Hardening, and Infrastructure Updates#70PaulJouvanceau wants to merge 0 commit into
PaulJouvanceau wants to merge 0 commit into
Conversation
cgalibern
added a commit
that referenced
this pull request
Sep 22, 2026
UI/UX Refinements, Test Suite Migration to Vitest, Security Hardening, and Infrastructure Updates- #70
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.
Summary
This PR introduces a wide range of improvements across the application, including UI fixes, new features, test refactoring, and bug fixes related to SSE connections, event handling, and authentication.
Changes
UI & UX
Safari compatibility: Fixed badge overflow and aligned badges to the bottom of stat cards
Action menus: Increased z-index of all action menus to appear above EventLogs panel
Destructive actions: Highlighted destructive actions in red and moved them to the bottom of action lists
StatCard: Added dynamic height with configurable max subtitle height
Mobile/scroll: Resolved iOS scroll issues by centralizing scroll in the main container; fixed logs panel resize on iOS and close button visibility
Navigation: Replaced user icon with tag icon for Kinds route; fixed duplicate navigation on Nodes grid by stopping event propagation
Grid2: Replaced undefined Grid2 with Grid for proper mobile rendering
Auth: Show spinner in AuthChoice while authInfo is loading
Features
Added replace mode to setInstanceStatuses and used it on initial fetch to prevent stale data accumulation
Added bottomSpacing prop to LogsViewer for configurable bottom spacing
Added fallback fetch and dynamic configNode switching in ObjectDetails
Reactive config refresh with fallback loading state
Instances in ObjectDetail are now sorted alphabetically
Bug Fixes
SSE/Events: Added objectName parameter to startEventReception for path fix; fixed objectName usage; closed SSE connection on ClusterOverview unmount to prevent memory leaks
OIDC: Use stable base path for OIDC callback URLs instead of deriving from window.location
Auth: Debounced auth check on page resume to prevent redundant token validation
Events: Debounced localStorage persistence and avoided O(n) instance scanning for performance
Heartbeats: Improved GridHeartbeats click handling
Buffer: Reduced debug logging verbosity
Config: Added useConfig and improved eventSourceManager configuration
Test Refactoring & Coverage
Refactored and optimized numerous test suites for clarity, maintainability, and execution speed without reducing coverage:
ObjectDetails.test
eventSourceManager
Objects tests
App.test.jsx
useEventStore (branch coverage for buffer and Safari)
Heartbeats (consolidated, reduced duplication)
Login test file
LogsViewer test suite
NodesTable tests
NavBar tests
Namespaces unit tests
AuthProvider (fixed timing/isolation issues)
WhoAmI tests
ClusterOverview tests (fixed mock alignment)
DarkModeContext (added comprehensive unit tests)
ConfigSection (improved coverage and structure)
Additional test improvements:
Added shared helpers and parameterization for KeysSection tests
Used real MUI RadioGroup/FormControlLabel in KeysSection tests
Fixed flaky action tests in Objects component
Misc
Deleted patch action (cleanup)
Improved GridNodes click handling