Time bug fix (#23)

Fixed an issue where we use the move of the current depth when not all moves of that depth were searched.

catto.config.js

@@ -1,6 +1,6 @@
 module.exports = {
     // Current version to show in UCI
-    version: "v0.12.2",
+    version: "v0.12.3",
     // Late move reduction config
     lmrFullDepth: 4, // Number of moves to be searched in full depth
     lmrMaxReduction: 3, // Only apply LMR above this depth

package.json

@@ -1,6 +1,6 @@
 {
   "name": "catto",
-  "version": "0.12.2",
+  "version": "0.12.3",
   "description": "The Catto chess engine",
   "main": "index.js",
   "scripts": {

src/core.ts

@@ -508,6 +508,8 @@ export class Engine {
     }
 
     findMove() {
+        let bestMove = "";
+
         // Iterative deepening with aspiration windows
         this.startTime = Date.now();
 
@@ -550,8 +552,12 @@ export class Engine {
                     console.log(`info depth ${depth} score cp ${Math.round(score)} time ${Date.now() - this.startTime} nodes ${this.nodes} pv${pv}`);
                 }
             }
+
+            // This is used to prevent cases where moves at a depth is not completely searched.
+            // We will just use the best move of previous depth if time is up or forced-stopped.
+            bestMove = this.pvTable[0][0];
         }
 
-        console.log(`bestmove ${this.pvTable[0][0]}`);
+        console.log(`bestmove ${bestMove}`);
     }
 }