fix: don't play animation on game over

Author: jim-at-stink

kaggle_environments/envs/open_spiel_env/games/go/visualizer/v2/src/components/GameRenderer.tsx

@@ -42,7 +42,7 @@ export default memo(function GameRenderer(options: GameRendererProps<GoStep[]>)
     setState(game, options);
   }, [options, setState]);
 
-  const gameOver = options.replay.steps.at(options.step)?.winner; // game.isOver();
+  const gameOver = options.replay.steps.at(options.step)?.winner;
   // React 18 doesn't support the `inert` HTML attribute as a prop, so we
   // set it imperatively via a ref callback. This can be replaced with a
   // regular `inert` prop once the project upgrades to React 19+.
@@ -71,7 +71,7 @@ export default memo(function GameRenderer(options: GameRendererProps<GoStep[]>)
       </div>
       {options.step === 0 && <VersusBanner options={options} />}
       {gameOver && <GameOverModal />}
-      {showHeroAnimations && <HeroAnimationModal />}
+      {showHeroAnimations && !gameOver && <HeroAnimationModal />}
     </main>
   );
 });

kaggle_environments/envs/open_spiel_env/games/go/visualizer/v2/src/transformers/goTransformer.ts

@@ -99,12 +99,14 @@ export const goTransformer = (environment: any): GoStep[] => {
     });
   }
 
+  const lastReplayStep = goReplay.steps[goReplay.steps.length - 1];
+
   goSteps.push({
     step: goSteps.length,
     players: extraStepPlayers,
     boardState: goSteps[goSteps.length - 1].boardState,
-    isTerminal: goSteps[goSteps.length - 1].isTerminal,
-    winner: deriveWinner(goReplay.steps[goReplay.steps.length - 1]),
+    isTerminal: lastReplayStep[0].observation.isTerminal,
+    winner: deriveWinner(lastReplayStep),
   });
 
   return goSteps;