fix: indefinite wait for bestmove after time loss (#237)

Author: Disservin

src/engines/uci_engine.cpp

@@ -145,4 +145,13 @@ int UciEngine::lastScore() const {
     return str_utils::findElement<int>(lastInfo(), lastScoreType() == ScoreType::CP ? "cp" : "mate")
         .value_or(0);
 }
+
+bool UciEngine::outputIncludesBestmove() const {
+    for (const auto &line : output_) {
+        if (line.find("bestmove") != std::string::npos) return true;
+    }
+
+    return false;
+}
+
 }  // namespace fast_chess

src/engines/uci_engine.hpp

@@ -72,6 +72,8 @@ class UciEngine : Process {
     /// @return
     [[nodiscard]] int lastScore() const;
 
+    [[nodiscard]] bool outputIncludesBestmove() const;
+
     [[nodiscard]] const std::vector<std::string> &output() const noexcept { return output_; }
     [[nodiscard]] const EngineConfiguration &getConfig() const noexcept { return config_; }
 

src/matchmaking/match/match.cpp

@@ -193,8 +193,10 @@ bool Match::playMove(Player& us, Player& opponent) {
 
         us.engine.writeEngine("stop");
 
-        // wait for bestmove, indefinitely
-        us.engine.readEngine("bestmove", 0ms);
+        if (!us.engine.outputIncludesBestmove()) {
+            // wait for bestmove, indefinitely
+            us.engine.readEngine("bestmove", 0ms);
+        }
 
         return false;
     }