Improve Sri and Lobby.pong

src/main/scala/Controller.scala

@@ -242,7 +242,7 @@ final class Controller(
               )
 
   def api(header: RequestHeader) =
-    val req = Req(header, Sri.random, None).copy(flag = Some(Flag.api))
+    val req = Req(header, Sri.random(), None).copy(flag = Some(Flag.api))
     Future.successful(
       endpoint(
         name = "api",

src/main/scala/LilaWs.scala

@@ -63,7 +63,7 @@ final class LilaWsServer(
       Bus.publish(_.all, ipc.ClientCtrl.Broom(nowSeconds - 30))
 
     scheduler.scheduleWithFixedDelay(4.seconds, 1201.millis): () =>
-      val counters = lobby.pong.get
+      val counters = lobby.pong.get()
       lila.emit.lobby(ipc.LilaIn.Counters(counters.members, counters.rounds))
 
     scheduler.scheduleWithFixedDelay(1.seconds, 1879.millis): () =>

src/main/scala/Lobby.scala

@@ -17,7 +17,7 @@ final class Lobby(lila: Lila, groupedWithin: util.GroupedWithin):
 
     private var value = LobbyPong(0, 0)
 
-    def get = value
+    def get() = value
 
     def update(members: Int, rounds: Int): Unit =
       value = LobbyPong(members, rounds)

src/main/scala/actor/LobbyClientActor.scala

@@ -50,7 +50,7 @@ object LobbyClientActor:
               case Some(s) => apply(state.copy(site = s), deps)
 
           case msg: ClientOut.Ping =>
-            clientIn(services.lobby.pong.get)
+            clientIn(services.lobby.pong.get())
             apply(state.copy(site = sitePing(state.site, deps, msg)), deps)
 
           case ClientOut.LobbyJoin(payload) =>

src/main/scala/model.scala

@@ -106,8 +106,8 @@ object Racer:
 
 opaque type Sri = String
 object Sri extends OpaqueString[Sri]:
-  def random                         = Sri(SecureRandom.nextString(12))
-  def from(str: String): Option[Sri] = if str contains ' ' then None else Some(str)
+  def random()                       = Sri(SecureRandom.nextString(12))
+  def from(str: String): Option[Sri] = Option.unless(str.contains(' '))(str)
 
 opaque type Flag = String
 object Flag extends OpaqueString[Flag]: