fix: logic updates

Author: chenasraf

lib/src/parser.dart

@@ -36,9 +36,7 @@ class ColorParser implements IReader<StringTokenValue> {
   ColorToken? _getToken() {
     var token = ColorToken.empty();
     final char = reader.peek();
-    // print('');
     // print('char: ${_debugString(char ?? '<null>')}');
-    // print('');
 
     switch (char) {
       case null:
@@ -97,6 +95,9 @@ class ColorParser implements IReader<StringTokenValue> {
           if (_checkBetween(color, 100, 107)) {
             token.brightBg = true;
           }
+        } else {
+          // Catch arbitrary/unhandled codes
+          token.setStyle(color);
         }
         colors.removeAt(0);
       }
@@ -113,6 +114,9 @@ class ColorParser implements IReader<StringTokenValue> {
   }
 
   ColorToken _consumeRgbToken(ColorToken token, List<String> colors) {
+    if (colors.length < 5) {
+      return token;
+    }
     final rgb = '${colors[2]};${colors[3]};${colors[4]}';
     if (colors[0] == '38') {
       token.rgbFg = true;
@@ -125,6 +129,9 @@ class ColorParser implements IReader<StringTokenValue> {
   }
 
   ColorToken _consumeXterm256Token(ColorToken token, List<String> colors) {
+    if (colors.length < 3) {
+      return token;
+    }
     final color = int.parse(colors[2]);
     if (colors[0] == '38') {
       token.xterm256 = true;
@@ -144,51 +151,6 @@ class ColorParser implements IReader<StringTokenValue> {
     return min <= value && value <= max;
   }
 
-  // ColorToken _consumeStyleToken(ColorToken token) {
-  //   // print('Consuming style token for $token');
-  //   final color = _consumeUntil('m');
-  //   reader.read();
-  //
-  //   if (!color.contains(';')) {
-  //     //single number color [30-37] / [40-47]
-  //     // or [90-97] / [100-107], fg / bg
-  //     // e.g. ^[40m
-  //     // or just style ? e.g. ^[1m
-  //
-  //     // TODO: this seems to crash on ^[40m, don't understand exactly why
-  //     // doesn't seem to be the 40m, that one gets interpreted well (black bg)
-  //
-  //     int colorValue = -1;
-  //     try {
-  //       colorValue = int.parse(color);
-  //     } on FormatException {
-  //       // ignore, then??
-  //       // print("failing color= " + color);
-  //       // TODO: it keeps logging thousands of empty "failing colors" ?
-  //     }
-  //     if (colorValue > -1) { // init safely, ignore if nothing ?
-  //       if ((30 <= colorValue) && (colorValue <= 37) ||
-  //           (90 <= colorValue) && (colorValue <= 97)) {
-  //         token.fgColor = colorValue;
-  //       } else if ((40 <= colorValue) && (colorValue <= 47) ||
-  //           (100 <= colorValue) && (colorValue <= 107)) {
-  //         token.bgColor = colorValue;
-  //       } else if (colorValue < 30) { // style ?
-  //         token.setStyle(colorValue);
-  //       }
-  //     }
-  //   }
-  //   // things like  ^[1;38;2;114;150;50;48;2;125;70;22m TEXT ^[0m
-  //   else { // multi number madness, trying recursive parser ?
-  //     final colors = color.split(';');
-  //     processTokenStyle(colors, token); //really hope this works by reference
-  //   }
-  //   if (reader.peek() == Consts.esc) {
-  //     return _consumeEscSequence(token);
-  //   }
-  //   return token;
-  // }
-
   ColorToken _consumeText(ColorToken token) {
     // print('Consuming text for $token');
     token.text += _consumeUntil(Consts.esc);
@@ -238,70 +200,6 @@ class ColorParser implements IReader<StringTokenValue> {
     return result;
   }
 
-  // processTokenStyle(List<String> colors, ColorToken token) {
-  //   if (colors.isNotEmpty) {
-  //     //if it's already empty, do nothing more
-  //     int first = int.parse(colors[0]);
-  //     if (first < 30) {
-  //       // bold, underline, etc?
-  //       token.setStyle(first);
-  //       colors.removeAt(0);
-  //     } else if ((30 <= first) && (first <= 37) ||
-  //         (90 <= first) && (first <= 97) ||
-  //         (40 <= first) && (first <= 47) ||
-  //         (100 <= first) && (first <= 107)) {
-  //       if ((30 <= first) && (first <= 37) || (90 <= first) && (first <= 97)) {
-  //         token.fgColor = first;
-  //         colors.removeAt(0);
-  //       } else if ((40 <= first) && (first <= 47) ||
-  //           (100 <= first) && (first <= 107)) {
-  //         token.bgColor = first;
-  //         colors.removeAt(0);
-  //       }
-  //     } else {
-  //       int second = int.parse(colors[1]);
-  //       if (first == 38 && second == 5) {
-  //         token.xterm256 = true;
-  //         int third = int.parse(colors[2]);
-  //         token.fgColor = third;
-  //         colors.removeRange(0, 3);
-  //         // bg = 0;
-  //       } else if (first == 48 && second == 5) {
-  //         token.xterm256 = true;
-  //         int third = int.parse(colors[2]);
-  //         token.bgColor = third;
-  //         colors.removeRange(0, 3);
-  //         // bg = 0;
-  //       } else {
-  //         if (first == 38 && second == 2) {
-  //           //rgb
-  //           String red = colors[2];
-  //           String green = colors[3];
-  //           String blue = colors[4];
-  //           token.rgbFg = true;
-  //           token.rgbFgColor = "$red;$green;$blue";
-  //           colors.removeRange(0, 5);
-  //         } else if (first == 48 && second == 2) {
-  //           //rgb
-  //           String red = colors[2];
-  //           String green = colors[3];
-  //           String blue = colors[4];
-  //           token.rgbBg = true;
-  //           token.rgbBgColor = "$red;$green;$blue";
-  //           colors.removeRange(0, 5);
-  //         } else {
-  //           return;
-  //         }
-  //       }
-  //     }
-  //     //pass the rest of the color codes, hope for the best
-  //     if (colors.isNotEmpty) {
-  //       processTokenStyle(
-  //           colors, token); // really really hoping these go by reference
-  //     }
-  //   }
-  // }
-
   // ignore: unused_element
   _debugString(String string) => string.replaceAll('\x1B', '\\x1B');