gnodet-bot commented on code in PR #28150:
URL: https://github.com/apache/camel/pull/28150#discussion_r4323752356
##########
dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/ai/AuthoringTools.java:
##########
@@ -1845,9 +1853,24 @@ public static JsonObject listFiles(Path dir) {
result.put("configFiles", configFiles);
result.put("files", files);
result.put("totalFiles", all.size());
+ // a file consumer moves a file it read into .camel (without
noop=true), which the listing skips
+ Map<String, Integer> consumed = consumedFolders(dir);
+ String moved = null;
+ if (!consumed.isEmpty()) {
+ JsonObject folders = new JsonObject();
+ folders.putAll(consumed);
+ result.put("consumedFiles", folders);
+ moved = String.join(", ", consumed.entrySet().stream().map(e ->
e.getKey() + " holds " + e.getValue()
+ +
(e.getValue() == 1 ? " file" : " files"))
+ .toList())
+ + " that a file consumer moved there after reading
(noop=true leaves the files where they are)";
+ }
if (all.isEmpty()) {
- result.put("message", "The directory has no files");
- } else if (all.size() > MAX_FILES) {
+ result.put("message", moved != null ? "The directory has no files,
but " + moved : "The directory has no files");
+ } else if (moved != null) {
+ result.put("note", moved);
+ }
+ if (all.size() > MAX_FILES) {
Review Comment:
Test comment at all.isEmpty/MAX_FILES block
--
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]
