davsclaus opened a new pull request, #23808:
URL: https://github.com/apache/camel/pull/23808

   ## Summary
   
   - **SearchHighlighter**: Extracts duplicated find/highlight/wrap logic from 
`SourceViewer` and `LogTab` into a single reusable class (~300 lines of shared 
search state, key handling, match navigation, and highlight rendering)
   - **MetricsCollector**: Extracts 30+ `ConcurrentHashMap` fields and all 
sliding-window update/cleanup/reset logic from `CamelMonitor` into a dedicated 
class. Tab constructors simplified (e.g. `EndpointsTab` from 10 map params to 1 
`MetricsCollector`)
   - `CamelMonitor` reduced from 3,256 to 2,936 lines
   
   ## Test plan
   
   - [ ] Build passes (`mvn install -DskipTests` in `camel-jbang-plugin-tui`)
   - [ ] Verify find/highlight in Log tab (`/`, `h`, `n`, `N`, `Esc`)
   - [ ] Verify find/highlight in source viewer (files popup → Enter → `/`, `h`)
   - [ ] Verify sparkline charts in Overview, Endpoints, Circuit Breaker, 
Memory tabs
   - [ ] Verify reset stats (F2 → Reset Stats) clears all charts
   - [ ] Verify vanishing integrations clean up chart data after 6 seconds
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code) on behalf of 
Claus Ibsen


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to