davsclaus commented on code in PR #28209:
URL: https://github.com/apache/camel/pull/28209#discussion_r4333646791
##########
dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/ai/AuthoringTools.java:
##########
@@ -1701,8 +1701,9 @@ public static JsonObject writeFile(ToolContext ctx, Path
dir, String file, Strin
+ (output.size() > 1 ? " (and " + (output.size() -
1) + " more in reload.output)" : "");
}
final String loggedAfter = logged;
+ String reloadedWhat = SourceValidator.isValidatableFile(file) ?
"the file" : "its routes, which load the file";
result.put("message", switch (status) {
- case "reloaded" -> "The running integration reloaded the
file." + loggedAfter;
+ case "reloaded" -> "The running integration reloaded " +
reloadedWhat + "." + loggedAfter;
case "properties" -> "The running integration reloaded the
properties.";
case "failed" -> "The running integration FAILED to reload the
file, the route is not running; fix the"
+ " content and write again (see
reload.message).";
Review Comment:
Applied in d48009744f13: the `failed` message now uses `reloadedWhat` as
well, so a failed reload of a script names the routes that load it.
_Claude Code on behalf of davsclaus_
--
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]
