fix(sdk): remove now useless serialization of worldcommits

Author: Julien

engine/modules/entities/src/main/java/com/codingame/gameengine/module/entities/GraphicEntityModule.java

@@ -200,14 +200,12 @@ private void sendFrameData() {
             currentWorldState.updateAllEntities(nextWorldState);
         }
 
-        Optional<String> worldCommits = gameSerializer.serializeWorldCommits(worldCommitsBuilder);
-
         Optional<String> update = gameSerializer.serializeWorldDiff(updateBuilder);
 
         worldStates.clear();
         gameManager.setViewData(
             "entitymodule",
-            Stream.of(load, create, update, worldCommits)
+            Stream.of(load, create, update)
                 .filter(Optional::isPresent)
                 .map(Optional::get)
                 .collect(Collectors.joining("\n"))

engine/modules/entities/src/main/java/com/codingame/gameengine/module/entities/Serializer.java

@@ -71,7 +71,6 @@ class Serializer {
         commands.put("CREATE", "C");
         commands.put("UPDATE", "U");
         commands.put("LOADSPRITESHEET", "L");
-        commands.put("WORLDUPDATE", "W");
 
         separators = new HashMap<>();
         separators.put("COMMAND", ";");
@@ -229,18 +228,6 @@ public Optional<String> serializeLoadSpriteSheets(List<SpriteSheetSplitter> spri
         }
     }
 
-    public Optional<String> serializeWorldCommits(List<String> worldCommits) {
-        if (worldCommits.isEmpty()) {
-            return Optional.empty();
-        } else {
-            return Optional.of(
-                commands.get("WORLDUPDATE") +
-                worldCommits.stream()
-                        .collect(Collectors.joining(separators.get("COMMAND_ARGUMENT")))
-            );
-        }
-    }
-
     public Optional<String> serializeWorldDiff(List<WorldState> diffs) {
         if (diffs.isEmpty()) {
             return Optional.empty();

engine/modules/entities/src/main/resources/view/entity-module/Command.js

@@ -154,16 +154,4 @@ export class PropertiesCommand {
     entity.addState(this.t, { values: this.params, curve: this.curve }, frameInfo.number, frameInfo)
     entity.stateAdded = true
   }
-export class WorldCommitCommand {
-  constructor (args, globalData) {
-    this.times = args.map(v => +v)
-  }
-
-  apply (entities, frameInfo) {
-    entities.forEach(entity => {
-      this.times.forEach(time => {
-        entity.addState(time, { values: {}, curve: {} }, frameInfo.number, frameInfo)
-      })
-    })
-  }
 }

engine/modules/entities/src/main/resources/view/entity-module/CommandParser.js

@@ -1,10 +1,9 @@
-import { CreateCommand, PropertiesCommand, LoadCommand, WorldCommitCommand } from './Command.js'
+import { CreateCommand, PropertiesCommand, LoadCommand } from './Command.js'
 
 const COMMAND_KEY_MAP = {
   C: CreateCommand,
   U: PropertiesCommand,
-  L: LoadCommand,
-  W: WorldCommitCommand
+  L: LoadCommand
 }
 
 function splitOnCharOutsideQuotes (text, charParam) {