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

   ## Summary
   
   _Claude Code on behalf of Guillaume Nodet_
   
   Extracts ~48 MCP-facing accessor methods (~650 lines) from 
`CamelMonitor.java` into a new `McpFacade` class. This is **step 1** of the 
CamelMonitor decomposition plan ([tracking 
gist](https://gist.github.com/gnodet/8152a5c6b56ee80166895391f5d11b63)).
   
   - **New `McpFacade.java`**: Contains all MCP accessor, navigation, data, and 
control methods; `TAB_NAMES`/`MORE_TAB_NAMES` constants; `PendingKey` record; 
and `parseKey()` method
   - **New `MonitorBridge` callback interface**: Decouples `McpFacade` from 
`CamelMonitor` internals (tab handling, rendering, process control)
   - **`AtomicReference<TapeRecorder>`**: Thread-safe sharing between the 
monitor event loop and MCP facade
   - **`TuiMcpServer` updated**: Now depends only on `McpFacade`, not the full 
`CamelMonitor`
   - **`CamelMonitor` reduced**: From 3,200 to ~2,600 lines (638 lines removed)
   - **`CamelMonitorParseKeyTest` updated**: References moved 
`McpFacade.parseKey()` method
   
   This is a **pure refactor** — zero behavior change. All 260 tests pass.
   
   ## Test plan
   
   - [x] `mvn formatter:format impsort:sort` — 1 file formatted, 2 imports 
sorted
   - [x] `mvn compile` — compilation succeeds with only pre-existing 
deprecation warnings
   - [x] `mvn test -pl dsl/camel-jbang/camel-jbang-plugin-tui` — all 260 tests 
pass (0 failures, 0 errors)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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