feat(sdk): Game manager handles FAIL command and crashes cleanly

Author: chickentuna

engine/core/src/main/java/com/codingame/gameengine/core/GameManager.java

@@ -2,6 +2,8 @@
 
 import java.io.InputStream;
 import java.io.PrintStream;
+import java.io.PrintWriter;
+import java.io.StringWriter;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
@@ -74,7 +76,7 @@ abstract public class GameManager<T extends AbstractPlayer> {
     private int totalTurnTime = 0;
 
     private boolean viewWarning, summaryWarning;
-    
+
     /**
      * GameManager main loop.
      * 
@@ -85,65 +87,72 @@ abstract public class GameManager<T extends AbstractPlayer> {
      */
     void start(InputStream is, PrintStream out) {
         s = new Scanner(is);
-        this.out = out;
-        this.referee = refereeProvider.get();
-
-        // Init ---------------------------------------------------------------
-        log.info("Init");
-        InputCommand iCmd = InputCommand.parse(s.nextLine());
-        int playerCount = s.nextInt();
-        s.nextLine();
-        players = new ArrayList<T>(playerCount);
-
-        for (int i = 0; i < playerCount; i++) {
-            T player = playerProvider.get();
-            player.setIndex(i);
-            players.add(player);
-        }
+        try {
+            this.out = out;
+            this.referee = refereeProvider.get();
+
+            // Init ---------------------------------------------------------------
+            log.info("Init");
+            InputCommand iCmd = InputCommand.parse(s.nextLine());
+            int playerCount = s.nextInt();
+            s.nextLine();
+            players = new ArrayList<T>(playerCount);
+
+            for (int i = 0; i < playerCount; i++) {
+                T player = playerProvider.get();
+                player.setIndex(i);
+                players.add(player);
+            }
 
-        readGameProperties(iCmd, s);
+            readGameProperties(iCmd, s);
 
-        prevViewData = null;
-        currentViewData = new JsonObject();
+            prevViewData = null;
+            currentViewData = new JsonObject();
 
-        referee.init();
-        registeredModules.forEach(Module::onGameInit);
-        initDone = true;
+            referee.init();
+            registeredModules.forEach(Module::onGameInit);
+            initDone = true;
 
-        // Game Loop ----------------------------------------------------------
-        for (turn = 0; turn < getMaxTurns() && !isGameEnd() && !allPlayersInactive(); turn++) {
-            swapInfoAndViewData();
-            log.info("Turn " + turn);
-            newTurn = true;
-            outputsRead = false; // Set as true after first getOutputs() to forbib sendInputs
+            // Game Loop ----------------------------------------------------------
+            for (turn = 0; turn < getMaxTurns() && !isGameEnd() && !allPlayersInactive(); turn++) {
+                swapInfoAndViewData();
+                log.info("Turn " + turn);
+                newTurn = true;
+                outputsRead = false; // Set as true after first getOutputs() to forbib sendInputs
 
-            referee.gameTurn(turn);
-            registeredModules.forEach(Module::onAfterGameTurn);
+                referee.gameTurn(turn);
+                registeredModules.forEach(Module::onAfterGameTurn);
 
-            // reset players' outputs
-            for (AbstractPlayer player : players) {
-                player.resetOutputs();
-                player.setHasBeenExecuted(false);
+                // reset players' outputs
+                for (AbstractPlayer player : players) {
+                    player.resetOutputs();
+                    player.setHasBeenExecuted(false);
+                }
             }
-        }
 
-        log.info("End");
+            log.info("End");
 
-        referee.onEnd();
-        registeredModules.forEach(Module::onAfterOnEnd);
+            referee.onEnd();
+            registeredModules.forEach(Module::onAfterOnEnd);
 
-        // Send last frame ----------------------------------------------------
-        swapInfoAndViewData();
-        newTurn = true;
+            // Send last frame ----------------------------------------------------
+            swapInfoAndViewData();
+            newTurn = true;
 
-        dumpView();
-        dumpInfos();
+            dumpView();
+            dumpInfos();
 
-        dumpGameProperties();
-        dumpMetadata();
-        dumpScores();
+            dumpGameProperties();
+            dumpMetadata();
+            dumpScores();
 
-        s.close();
+            s.close();
+
+        } catch (RuntimeException e) {
+            dumpFail(e);
+            s.close();
+            throw e;
+        }
     }
 
     abstract protected boolean allPlayersInactive();
@@ -231,6 +240,16 @@ private void dumpScores() {
         out.println(data);
     }
 
+    private void dumpFail(RuntimeException e) {
+        OutputData data = new OutputData(OutputCommand.FAIL);
+        StringWriter sw = new StringWriter();
+        PrintWriter pw = new PrintWriter(sw);
+        e.printStackTrace(pw);
+
+        data.add(sw.toString());
+        out.println(data);
+    }
+
     private void dumpView() {
         OutputData data = new OutputData(OutputCommand.VIEW);
         if (newTurn) {

engine/core/src/main/java/com/codingame/gameengine/core/OutputCommand.java

@@ -1,6 +1,6 @@
 package com.codingame.gameengine.core;
 enum OutputCommand {
-    VIEW, INFOS, NEXT_PLAYER_INPUT, NEXT_PLAYER_INFO, SCORES, UINPUT, TOOLTIP, SUMMARY, METADATA;
+    VIEW, INFOS, NEXT_PLAYER_INPUT, NEXT_PLAYER_INFO, SCORES, UINPUT, TOOLTIP, SUMMARY, METADATA, FAIL;
     public String format(int lineCount) {
         return String.format("[[%s] %d]", this.name(), lineCount);
     }