Fix some rich console bugs:

- Large stack traces would loose color
- Large stack traces can loose some output
- Use 'default forground' instead of 'reset all'

Author: stuartwdouglas

core/deployment/src/main/java/io/quarkus/deployment/dev/console/AeshConsole.java

@@ -3,6 +3,8 @@
 import java.util.concurrent.LinkedBlockingDeque;
 import java.util.concurrent.locks.Lock;
 import java.util.concurrent.locks.ReentrantLock;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
 
 import org.aesh.terminal.Attributes;
 import org.aesh.terminal.Connection;
@@ -22,6 +24,7 @@ public class AeshConsole extends QuarkusConsole {
     private String[] messages = new String[0];
     private int totalStatusLines = 0;
     private int lastWriteCursorX;
+    private String lastColorCode; //foreground color code, or reset
     private volatile boolean doingReadline;
     /**
      * if the status area has gotten big then small again
@@ -46,6 +49,8 @@ protected Boolean initialValue() {
         }
     };
 
+    static final Pattern ESCAPE = Pattern.compile("\u001b\\[(\\d\\d?)[\\d;]*m");
+
     public AeshConsole(Connection connection, boolean inputSupport) {
         this.inputSupport = inputSupport;
         INSTANCE = this;
@@ -244,7 +249,7 @@ private void printStatusAndPrompt(StringBuilder buffer) {
     }
 
     private void clearStatusMessages(StringBuilder buffer) {
-        gotoLine(buffer, size.getHeight() - totalStatusLines);
+        gotoLine(buffer, size.getHeight() - totalStatusLines + 1);
         buffer.append("\033[J");
     }
 
@@ -279,6 +284,19 @@ public void write(String s) {
         if (IN_WRITE.get()) {
             return;
         }
+        if (lastColorCode != null) {
+            s = lastColorCode + s;
+        }
+        Matcher m = ESCAPE.matcher(s);
+        while (m.find()) {
+            int val = Integer.parseInt(m.group(1));
+            if (val == 0 || //reset
+                    (val >= 30 && val <= 39) || //foreground colors
+                    (val >= 90 && val <= 97)) { //bright foreground colors
+                lastColorCode = m.group(0);
+            }
+        }
+
         StringBuilder buffer = new StringBuilder();
         synchronized (this) {
             if (outputFilter != null) {
@@ -329,7 +347,8 @@ public void write(String s) {
                     int appendLines = Math.max(Math.min(cursorPos > 1 ? lines - 1 : lines, totalStatusLines), 1);
                     appendLines -= usedBlankSpace;
                     clearStatusMessages(buffer);
-                    buffer.append("\033[").append(size.getHeight() - totalStatusLines - originalBlank).append(";").append(0)
+                    buffer.append("\033[").append(size.getHeight() - totalStatusLines - originalBlank).append(";")
+                            .append(lastWriteCursorX)
                             .append("H");
                     buffer.append(s);
                     buffer.append("\033[").append(size.getHeight()).append(";").append(0).append("H");

core/deployment/src/main/java/io/quarkus/deployment/dev/testing/MessageFormat.java

@@ -5,7 +5,7 @@ public class MessageFormat {
     public static final String RED = "\u001B[91m";
     public static final String GREEN = "\u001b[32m";
     public static final String BLUE = "\u001b[34m";
-    public static final String RESET = "\u001b[0m";
+    public static final String RESET = "\u001b[39m";
 
     private MessageFormat() {
     }

core/deployment/src/main/java/io/quarkus/deployment/dev/testing/TestConsoleHandler.java

@@ -34,7 +34,8 @@ public class TestConsoleHandler implements TestListener {
 
     private static final Logger log = Logger.getLogger("io.quarkus.test");
 
-    public static final String PAUSED_PROMPT = "Tests paused, press [" + BLUE + "r" + RESET + "] to resume, [" + BLUE + "h"
+    public static final String PAUSED_PROMPT = "Tests paused, press [" + BLUE + "r" + RESET + "] to resume, [" + BLUE + "w"
+            + RESET + "] to open the browser, " + RESET + " [" + BLUE + "h"
             + RESET + "] for more options>" + RESET;
     public static final String PAUSED_PROMPT_NO_HTTP = "Tests paused, press [" + BLUE + "r" + RESET + "] to resume, [" + BLUE
             + "s" + RESET + "] to restart with changes, [" + BLUE + "h"
@@ -89,7 +90,7 @@ public void handleInput(int[] keys) {
             for (int k : keys) {
                 if (k == 'h') {
                     printUsage();
-                } else if (k == 'b' && devModeType != DevModeType.TEST_ONLY) {
+                } else if (k == 'w' && devModeType != DevModeType.TEST_ONLY) {
                     browserOpener.accept("/");
                 } else if (k == 'd' && devModeType != DevModeType.TEST_ONLY) {
                     browserOpener.accept("/q/dev");
@@ -166,9 +167,9 @@ public void printUsage() {
             System.out
                     .println(helpOption("i", "Toggle instrumentation based reload", testController.isInstrumentationEnabled()));
             System.out.println(helpOption("l", "Toggle live reload", testController.isLiveReloadEnabled()));
-            System.out.println(helpOption("s", "Force restart with any changes"));
+            System.out.println(helpOption("s", "Force restart"));
             if (hasHttp) {
-                System.out.println(helpOption("b", "Open the application in a browser"));
+                System.out.println(helpOption("w", "Open the application in a browser"));
                 System.out.println(helpOption("d", "Open the Dev UI in a browser"));
             }
         }

docs/src/main/asciidoc/continuous-testing.adoc

@@ -100,7 +100,7 @@ The following commands are available:
 [o] - Toggle test output (disabled)
 [i] - Toggle instrumentation based reload (disabled)
 [l] - Toggle live reload (enabled)
-[s] - Force restart with any changes
+[s] - Force restart
 [h] - Display this help
 [q] - Quit
 ----
@@ -138,9 +138,9 @@ state.
 [l] - Toggle live reload::
 This is not directly related to testing, but allows you to turn live reload on and off.
 
-[s] - Force restart with any changes::
-This will force a scan for changed file, and will perform a live reload if anything has changed. This will still work even
-if live reload is disabled.
+[s] - Force restart::
+This will force a scan for changed files, and will perform a live reload with and changes. Note that even if there are no
+changes the application will still restart. This will still work even if live reload is disabled.
 
 == Continuous Testing Without Dev Mode
 

extensions/vertx-http/deployment/src/main/java/io/quarkus/vertx/http/deployment/devmode/console/DevConsoleProcessor.java

@@ -1,6 +1,5 @@
 package io.quarkus.vertx.http.deployment.devmode.console;
 
-import java.awt.*;
 import java.io.File;
 import java.io.IOException;
 import java.io.Reader;