davsclaus commented on PR #26750: URL: https://github.com/apache/camel/pull/26750#issuecomment-5781773268
One more piece, from trying the workflow the tool is for: in the camel-jbang views `camel_edit_file` went straight to the shared implementation, so it wrote without the `/write confirm` dialog and without the live replay in the Source editor. Since the agent now prefers the edit tool over a whole-file write, `/write live` would quietly have stopped showing most changes. The matching part of the shared tool is a method of its own now (`editedContent`, returning the new content or the not-found/ambiguous answer). The views resolve the snippet with it and write the result through the facade, so an edit is confirmed, replayed and validated exactly as a write is. With a directory given, or without a selected integration, it still takes the shared path as before. Test in `McpFacadeWriteFileTest`, the views' 1383 tests pass, and the TUI page says so. -- 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]
