Fix pre-release review findings

- resolveLazyChild: remove all alias keys mapping to the same class
  after resolution to prevent duplicate child parsers from stale
  alias entries

- AeshOptionParser: add logging for DefaultValueProvider.fallbackValue()
  exceptions instead of silently swallowing them

- AsciidocRenderer and MarkdownRenderer: add exclusiveWith guard to
  flag clustering so mutually exclusive boolean options are not
  double-rendered (in cluster AND in exclusive group)

Author: stalep

aesh/src/main/java/org/aesh/command/impl/parser/AeshCommandLineParser.java

@@ -155,6 +155,9 @@ private CommandLineParser<CI> resolveLazyChild(String name) {
         Class<? extends Command> clazz = lazyChildClasses.remove(name);
         if (clazz == null)
             return null;
+        // Remove all other keys mapping to the same class (aliases)
+        // to prevent duplicate resolution via stale alias entries
+        lazyChildClasses.values().removeIf(c -> c == clazz);
         try {
             AeshCommandContainerBuilder<CI> builder = new AeshCommandContainerBuilder<>();
             CommandContainer<CI> container = builder.create(clazz);

aesh/src/main/java/org/aesh/command/impl/parser/AeshOptionParser.java

@@ -19,6 +19,8 @@
  */
 package org.aesh.command.impl.parser;
 
+import java.util.logging.Level;
+import java.util.logging.Logger;
 import java.util.regex.Pattern;
 
 import org.aesh.command.impl.internal.OptionType;
@@ -33,6 +35,7 @@
  */
 public class AeshOptionParser implements OptionParser {
 
+    private static final Logger LOGGER = Logger.getLogger(AeshOptionParser.class.getName());
     private static final String EQUALS = "=";
     private static final char DASH = '-';
     private Status status;
@@ -291,7 +294,7 @@ private static void applyOptionalFallback(ProcessedOption option) {
                     return;
                 }
             } catch (Exception e) {
-                // Provider failed — fall through to annotation fallback
+                LOGGER.log(Level.WARNING, "DefaultValueProvider.fallbackValue() failed for --" + option.name(), e);
             }
         }
 

aesh/src/main/java/org/aesh/util/doc/AsciidocRenderer.java

@@ -248,7 +248,7 @@ private String buildSynopsis(CommandLineParser<?> parser, String fullName) {
             if (opt.getVisibility() == OptionVisibility.HIDDEN)
                 continue;
             if (opt.getOptionType() == OptionType.BOOLEAN && opt.shortName() != null
-                    && !opt.isRequired()) {
+                    && !opt.isRequired() && opt.getExclusiveWith().isEmpty()) {
                 shortFlags.append(opt.shortName());
             }
         }
@@ -261,7 +261,8 @@ private String buildSynopsis(CommandLineParser<?> parser, String fullName) {
             if (opt.getVisibility() == OptionVisibility.HIDDEN)
                 continue;
             if (opt.getOptionType() == OptionType.BOOLEAN && opt.shortName() != null
-                    && !opt.isRequired() && !opt.isNegatable()) {
+                    && !opt.isRequired() && !opt.isNegatable()
+                    && opt.getExclusiveWith().isEmpty()) {
                 continue; // already grouped, no separate rendering needed
             }
 

aesh/src/main/java/org/aesh/util/doc/MarkdownRenderer.java

@@ -241,7 +241,7 @@ private String buildSynopsis(CommandLineParser<?> parser, String fullName) {
             if (opt.getVisibility() == OptionVisibility.HIDDEN)
                 continue;
             if (opt.getOptionType() == OptionType.BOOLEAN && opt.shortName() != null
-                    && !opt.isRequired()) {
+                    && !opt.isRequired() && opt.getExclusiveWith().isEmpty()) {
                 shortFlags.append(opt.shortName());
             }
         }
@@ -254,7 +254,8 @@ private String buildSynopsis(CommandLineParser<?> parser, String fullName) {
             if (opt.getVisibility() == OptionVisibility.HIDDEN)
                 continue;
             if (opt.getOptionType() == OptionType.BOOLEAN && opt.shortName() != null
-                    && !opt.isRequired() && !opt.isNegatable()) {
+                    && !opt.isRequired() && !opt.isNegatable()
+                    && opt.getExclusiveWith().isEmpty()) {
                 continue; // already grouped
             }