Final (we can hope) premove cleanup

Author: paulvanwamelen

api/abstractplay.ts

@@ -3400,7 +3400,6 @@ async function submitMove(userid: string, pars: { id: string, move: string, draw
 
     // Parse explorations if fetched (for premove processing)
     let opponentExploration: Exploration[] | null = null;
-    let myExploration: Exploration[] | null = null;
     if (opponentExplorationData?.Item?.tree) {
       try {
         opponentExploration = JSON.parse(opponentExplorationData.Item.tree as string);
@@ -4759,49 +4758,24 @@ async function saveExploration(userid: string, pars: { public: boolean, game: st
 
 async function getExploration(userid: string, pars: { game: string; move: number }) {
   const work: Promise<any>[] = [];
+  let exploration;
   try {
-    // get exploration you did while looking at this position in a previous visit to the game
-    work.push(ddbDocClient.send(
+    // get exploration. At pars.move. submitMove now takes care of moving exploration to the current game state.
+    exploration = await ddbDocClient.send(
       new GetCommand({
         TableName: process.env.ABSTRACT_PLAY_TABLE,
         Key: {
           "pk": "GAMEEXPLORATION#" + pars.game,
           "sk": userid + "#" + pars.move
           },
       })
-    ));
-    // also get exploration you did while you opponent was on move, or if it is his move, the exploration you did for your last move.
-    if (pars.move > 0) {
-      work.push(ddbDocClient.send(
-        new GetCommand({
-          TableName: process.env.ABSTRACT_PLAY_TABLE,
-          Key: {
-            "pk": "GAMEEXPLORATION#" + pars.game,
-            "sk": userid + "#" + (pars.move - 1)
-            },
-        })
-      ));
-    }
-    // and exploration you did when it was last your move. No need to further back because that exploration would have been merged when it was
-    // last your move (unless you turned off exploration for that move, but...)
-    if (pars.move > 1) {
-      work.push(ddbDocClient.send(
-        new GetCommand({
-          TableName: process.env.ABSTRACT_PLAY_TABLE,
-          Key: {
-            "pk": "GAMEEXPLORATION#" + pars.game,
-            "sk": userid + "#" + (pars.move - 2)
-            },
-        })
-      ));
-    }
+    );
   }
   catch (error) {
     logGetItemError(error);
     return formatReturnError(`Unable to get exploration data for game ${pars.game} from table ${process.env.ABSTRACT_PLAY_TABLE}`);
   }
-  const data = await Promise.all(work);
-  const trees = data.map((d: any) => d.Item);
+  const trees = [exploration.Item,,]; // return as an array as was done in the past. Not really needed, but makes transition safer.
   return {
     statusCode: 200,
     body: JSON.stringify(trees),