Only continue log level if line starts with whitespace; reintroduce exception detection

Signed-off-by: TheKodeToad <TheKodeToad@proton.me>

launcher/launch/LaunchTask.cpp

@@ -257,10 +257,7 @@ bool LaunchTask::parseXmlLogs(QString const& line, MessageLevel::Enum level)
             MessageLevel::Enum newLevel = MessageLevel::fromLine(msg);
 
             if (newLevel == MessageLevel::Unknown)
-                newLevel = LogParser::guessLevel(line);
-
-            if (newLevel == MessageLevel::Unknown)
-                newLevel = model->previousLevel();
+                newLevel = LogParser::guessLevel(line, model->previousLevel());
 
             msg = censorPrivateInfo(msg);
 

launcher/logs/LogParser.cpp

@@ -320,7 +320,7 @@ std::optional<LogParser::ParsedItem> LogParser::parseLog4J()
     throw std::runtime_error("unreachable: already verified this was a complete log4j:Event");
 }
 
-MessageLevel::Enum LogParser::guessLevel(const QString& line)
+MessageLevel::Enum LogParser::guessLevel(const QString& line, MessageLevel::Enum previous)
 {
     static const QRegularExpression LINE_WITH_LEVEL("^\\[(?<timestamp>[0-9:]+)\\] \\[[^/]+/(?<level>[^\\]]+)\\]");
     auto match = LINE_WITH_LEVEL.match(line);
@@ -343,11 +343,17 @@ MessageLevel::Enum LogParser::guessLevel(const QString& line)
             return MessageLevel::Debug;
     }
 
+    if (line.contains("Exception: ") || line.contains("Throwable: "))
+        return MessageLevel::Error;
+
+    if (line.startsWith("Caused by: ") || line.startsWith("Exception in thread"))
+        return MessageLevel::Error;
+
     if (line.contains("overwriting existing"))
         return MessageLevel::Fatal;
 
-    if (line == "---- Minecraft Crash Report ----")
-        return MessageLevel::Error;
+    if (line.startsWith("\t") || line.startsWith(" "))
+        return previous;
 
     return MessageLevel::Unknown;
 }

launcher/logs/LogParser.h

@@ -59,7 +59,7 @@ class LogParser {
     std::optional<Error> getError();
 
     /// guess log level from a line of game log
-    static MessageLevel::Enum guessLevel(const QString& line);
+    static MessageLevel::Enum guessLevel(const QString& line, MessageLevel::Enum previous);
 
    protected:
     std::optional<LogEntry> parseAttributes();

launcher/ui/pages/instance/OtherLogsPage.cpp

@@ -287,10 +287,7 @@ void OtherLogsPage::reload()
             if (!m_instance) {
                 level = MessageLevel::fromLauncherLine(lineTemp);
             } else {
-                level = LogParser::guessLevel(line);
-
-                if (level == MessageLevel::Unknown)
-                    level = last;
+                level = LogParser::guessLevel(line, last);
             }
 
             last = level;

tests/XmlLogs_test.cpp

@@ -126,10 +126,7 @@ class XmlLogParseTest : public QObject {
                     last = entry.level;
                 } else if (std::holds_alternative<LogParser::PlainText>(item)) {
                     auto msg = std::get<LogParser::PlainText>(item).message;
-                    auto level = LogParser::guessLevel(msg);
-
-                    if (level == MessageLevel::Unknown)
-                        level = last;
+                    auto level = LogParser::guessLevel(msg, last);
 
                     out.append(std::make_pair(level, msg));
                     last = level;