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

   ## Summary
   
   - Fix bug where `camel debug --breakpoint=setBody` stops at the route input 
(timer/from) instead of the `setBody` EIP when it is the first step in the route
   - The `singleStepIncludeStartEnd` route-input replacement now only applies 
to `_all_routes_` breakpoints, not explicit user-specified breakpoints
   
   ## Root cause
   
   In `DefaultChannel.backlogDebuggerSetupInitialBreakpoints()`, when a 
breakpoint pattern matches and the matched node is the first output in the 
route, the code replaces the breakpoint target with the route input node (the 
`from`). This was intended for `_all_routes_` breakpoints (where you want to 
start from the beginning of each route) but was incorrectly applied to all 
breakpoints including explicit ones like `setBody`.
   
   ## Fix
   
   Added an `allRoutes` flag to track whether the match came from 
`_all_routes_`. The route-input replacement now requires `allRoutes && first && 
singleStepIncludeStartEnd`.
   
   ## Test plan
   
   - [x] Verified manually with `camel debug route.yaml --breakpoint=setBody` — 
now stops at `setBody` instead of the timer
   
   _Claude Code on behalf of Claus Ibsen_
   
   Co-Authored-By: Claude Opus 4.6 <[email protected]>


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