- Looks like combat phases are counted from 0 up? (fixes Finest Hour and possibly other cards that trigger "at first combat phase").

forge-game/src/main/java/forge/game/phase/PhaseHandler.java

@@ -855,7 +855,7 @@ public class PhaseHandler implements java.io.Serializable {
     }
 
     public final boolean isFirstCombat() {
-        return (nCombatsThisTurn == 1);
+        return (nCombatsThisTurn == 0);
     }
 
     public final boolean isFirstUpkeep() {