Fix Android client game events failing to deserialize in cross-platform network play (Card-Forge#10304)

JVM hardcodes serialVersionUID=0 for records, but Android's D8 desugaring
computes a different UID, causing InvalidClassException for all GameEvent
types in GameEventProxy's inner deserialization. Override readClassDescriptor()
to use the local class descriptor when UIDs mismatch, matching the existing
pattern in SaveFileData and CObjectInputStream.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: MostCromulent

forge-game/src/main/java/forge/game/event/GameEvent.java

@@ -4,5 +4,5 @@
 
 public interface GameEvent extends Event, Serializable {
 
-    public abstract <T> T visit(IGameEventVisitor<T> visitor);
+    <T> T visit(IGameEventVisitor<T> visitor);
 }

forge-gui/src/main/java/forge/gamemodes/match/AbstractGuiGame.java

@@ -240,10 +240,8 @@ public boolean mayView(final CardView c) {
             } catch (NullPointerException e) {
                 return true; // return true so it will work as normal
             }
-        } else {
-            if (getGameController().mayLookAtAllCards()) {
-                return true;
-            }
+        } else if (getGameController().mayLookAtAllCards()) {
+            return true;
         }
         return c.canBeShownToAny(getLocalPlayers());
     }

forge-gui/src/main/java/forge/gamemodes/net/GameEventProxy.java

@@ -216,6 +216,24 @@ boolean hasUnresolvedRefs() {
             return unresolvedRefs;
         }
 
+        // needed for cross-platform play because Android implements Records via desugaring to regular classes,
+        // causing the serialVersionUID to auto-compute instead of default 0L
+        // (this approach avoids having to hardcode it on each individual GameEvent instead)
+        @Override
+        protected ObjectStreamClass readClassDescriptor() throws IOException, ClassNotFoundException {
+            ObjectStreamClass streamDesc = super.readClassDescriptor();
+            try {
+                Class<?> localClass = Class.forName(streamDesc.getName());
+                ObjectStreamClass localDesc = ObjectStreamClass.lookup(localClass);
+                if (localDesc != null && streamDesc.getSerialVersionUID() != localDesc.getSerialVersionUID()) {
+                    return localDesc;
+                }
+            } catch (ClassNotFoundException ignored) {
+                // Class not found locally — fall through to stream descriptor
+            }
+            return streamDesc;
+        }
+
         @Override
         protected Object resolveObject(Object obj) {
             if (obj instanceof IdRef ref) {

forge-gui/src/main/java/forge/player/PlayerControllerHuman.java

@@ -148,25 +148,16 @@ public void setDisableAutoYields(final boolean disableAutoYields0) {
     public boolean mayLookAtAllCards() {
         return mayLookAtAllCards;
     }
-    /**
-     * Set this to {@code true} to enable this player to see all cards any other
-     * player can see.
-     *
-     * @param mayLookAtAllCards the mayLookAtAllCards to set
-     */
-    public void setMayLookAtAllCards(final boolean mayLookAtAllCards) {
-        this.mayLookAtAllCards = mayLookAtAllCards;
-    }
 
     private final ArrayList<Card> tempShownCards = new ArrayList<>();
 
     public <T> void tempShow(final Iterable<T> objects) {
         for (final T t : objects) {
             // assume you may see any card passed through here
-            if (t instanceof Card) {
-                tempShowCard((Card) t);
-            } else if (t instanceof CardView) {
-                tempShowCard(getCard((CardView) t));
+            if (t instanceof Card c) {
+                tempShowCard(c);
+            } else if (t instanceof CardView c) {
+                tempShowCard(getCard(c));
             }
         }
     }